<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 5/14/15 12:18 PM, <a
class="moz-txt-link-abbreviated"
href="mailto:larryv@macports.org"><a class="moz-txt-link-abbreviated" href="mailto:larryv@macports.org">larryv@macports.org</a></a> wrote:<br>
</div>
<blockquote cite="mid:20150514191842.4D304140F5D@svn.macports.org"
type="cite">
<meta http-equiv="content-type" content="text/html;
charset=windows-1252">
<title>[136318] trunk/dports/devel/autoconf/Portfile</title>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt>
<dd><a moz-do-not-send="true"
href="https://trac.macports.org/changeset/136318">136318</a></dd>
<dt>Author</dt>
<dd><a class="moz-txt-link-abbreviated"
href="mailto:larryv@macports.org">larryv@macports.org</a></dd>
<dt>Date</dt>
<dd>2015-05-14 12:18:42 -0700 (Thu, 14 May 2015)</dd>
</dl>
<h3>Log Message</h3>
<pre>autoconf: Use the system Perl
Autoconf's Perl requirements are light, and it's nice to eliminate
dependencies for such a core tool.</pre>
<h3>Modified Paths</h3>
<ul>
<li><a moz-do-not-send="true"
href="#trunkdportsdevelautoconfPortfile">trunk/dports/devel/autoconf/Portfile</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<div class="modfile">
<h4>Modified: trunk/dports/devel/autoconf/Portfile (136317
=> 136318)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/dports/devel/autoconf/Portfile 2015-05-14 19:10:41 UTC (rev 136317)
+++ trunk/dports/devel/autoconf/Portfile 2015-05-14 19:18:42 UTC (rev 136318)
</span><span class="lines">@@ -6,7 +6,7 @@
</span><span class="cx"> name autoconf
</span><span class="cx"> epoch 1
</span><span class="cx"> version 2.69
</span><del>-revision 3
</del><ins>+revision 4
</ins><span class="cx"> categories devel
</span><span class="cx"> # the license is GPL-3+ with an exception:
</span><span class="cx"> # <a class="moz-txt-link-freetext" href="http://www.gnu.org/licenses/autoconf-exception.html">http://www.gnu.org/licenses/autoconf-exception.html</a>
</span><span class="lines">@@ -28,9 +28,7 @@
</span><span class="cx"> homepage <a class="moz-txt-link-freetext" href="http://www.gnu.org/software/autoconf/">http://www.gnu.org/software/autoconf/</a>
</span><span class="cx">
</span><span class="cx"> installs_libs no
</span><del>-depends_lib port:gettext \
- port:m4 \
- port:perl5
</del><ins>+depends_lib port:gettext port:m4
</ins><span class="cx">
</span><span class="cx"> master_sites gnu
</span><span class="cx"> use_xz yes
</span><span class="lines">@@ -43,7 +41,9 @@
</span><span class="cx"> touch ${worksrcpath}/man/autoreconf.1
</span><span class="cx"> }
</span><span class="cx">
</span><del>-configure.perl ${prefix}/bin/perl
</del><ins>+# It should be safe to use the system Perl, since the scripts only use
+# core modules. The README prescribes 5.6 or later, and Tiger has 5.8.6.
+configure.perl /usr/bin/perl
</ins><span class="cx">
</span><span class="cx"> test.run yes
</span><span class="cx"> test.target check
</span></span></pre>
</div>
</div>
<br>
</blockquote>
Larry, <br>
<br>
I understand what you're saying, but this seems to me to be in the
category of fixing things that aren't broken. In addition, it
complicates maintenance by opening us up to perl version specific
problems on the various platforms, particularly older ones.<br>
<br>
I would favor depending not on the system perl or perl5 but use the
default perl (currently perl5.16). This is in line with our general
perl guidelines and ensures consistent behavior on all platforms
regardless of the system perl version.<br>
<br>
Dave<br>
<br>
<br>
</body>
</html>