<div dir="ltr">Hi Ryan,<div><br></div><div>Why is using a patchfile instead of reinplace preferable?</div><div><br></div><div>David</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Jun 8, 2014 at 3:33 PM, Ryan Schmidt <span dir="ltr">&lt;<a href="mailto:ryandesign@macports.org" target="_blank">ryandesign@macports.org</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Jun 5, 2014, at 12:30 PM, <a href="mailto:mf2k@macports.org">mf2k@macports.org</a> wrote:<br>
<br>
&gt; Revision<br>
&gt; 120684<br>
&gt; Author<br>
&gt; <a href="mailto:mf2k@macports.org">mf2k@macports.org</a><br>
&gt; Date<br>
&gt; 2014-06-05 10:30:25 -0700 (Thu, 05 Jun 2014)<br>
&gt; Log Message<br>
&gt;<br>
&gt; srecord: Update to version 1.63. (#43916)<br>
&gt; Modified Paths<br>
&gt;<br>
&gt;       • trunk/dports/devel/srecord/Portfile<br>
<br>
&gt; @@ -30,8 +30,17 @@<br>
&gt;<br>
&gt;                    offsets, split and unsplit for memory striping schemes.<br>
&gt;<br>
&gt;  homepage          <a href="http://srecord.sourceforge.net/" target="_blank">http://srecord.sourceforge.net/</a><br>
&gt;  master_sites      sourceforge<br>
&gt;<br>
&gt; -checksums         md5 8fce124d47f23b4aa187c3b8eebc9fd7<br>
&gt;<br>
&gt; +checksums         rmd160  668d5dc75960666a7c99509f39ecd2602891c384 \<br>
&gt; +                  sha256  78fec76d04424506e319f59b19a520428a7449ed087a67e1779fa2996992bf1a<br>
&gt; +depends_build     port:libtool<br>
&gt;<br>
&gt;  configure.cflags-append      &quot;-I${prefix}/include&quot;<br>
&gt;<br>
&gt; -configure.args    --mandir=${destroot}${prefix}/share/man<br>
&gt;<br>
&gt; +configure.env-append &quot;LIBTOOL=glibtool&quot;<br>
<br>
Consider specifying the absolute path to glibtool i.e. ${prefix}/bin/glibtool, just in case the user has an unusual binpath configured.<br>
<br>
<br>
&gt;  destroot.destdir  prefix=${destroot}${prefix}<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; +pre-configure {<br>
&gt; +    reinplace {s|@bindir@|$(prefix)/bin|} ${worksrcpath}/Makefile.in<br>
&gt; +    reinplace {s|@mandir@|$(prefix)/share/man|} ${worksrcpath}/Makefile.in<br>
&gt; +    reinplace {s|@datarootdir@|$(prefix)/share|} ${worksrcpath}/Makefile.in<br>
&gt; +    reinplace {s|@libdir@|$(prefix)/lib|} ${worksrcpath}/Makefile.in<br>
&gt; +    reinplace {s|@includedir@|$(prefix)/include|} ${worksrcpath}/Makefile.in<br>
&gt; +}<br>
<br>
Could this be done as a normal patchfile instead? I notice you&#39;re using parentheses e.g. &quot;$(prefix)&quot; which is Makefile variable expansion syntax, not curly brackets e.g. &quot;${prefix}&quot; which would be the Tcl variable expansion syntax, so no variable substitution is happening here at reinplace time anyway.<br>

<br>
_______________________________________________<br>
macports-dev mailing list<br>
<a href="mailto:macports-dev@lists.macosforge.org">macports-dev@lists.macosforge.org</a><br>
<a href="https://lists.macosforge.org/mailman/listinfo/macports-dev" target="_blank">https://lists.macosforge.org/mailman/listinfo/macports-dev</a><br>
</blockquote></div><br></div>