[136318] trunk/dports/devel/autoconf/Portfile

[David Evans]

On 5/14/15 12:18 PM, larryv at macports.org wrote:
> [136318] trunk/dports/devel/autoconf/Portfile
>
> Revision
>     136318 <https://trac.macports.org/changeset/136318>
> Author
>     larryv at macports.org
> Date
>     2015-05-14 12:18:42 -0700 (Thu, 14 May 2015)
>
>
>       Log Message
>
> autoconf: Use the system Perl
>
> Autoconf's Perl requirements are light, and it's nice to eliminate
> dependencies for such a core tool.
>
>
>       Modified Paths
>
>   * trunk/dports/devel/autoconf/Portfile
>     <#trunkdportsdevelautoconfPortfile>
>
>
>       Diff
>
>
>         Modified: trunk/dports/devel/autoconf/Portfile (136317 => 136318)
>
> --- 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) @@ -6,7 +6,7 @@ name autoconf epoch 1 version 2.69 
> -revision 3 +revision 4 categories devel # the license is GPL-3+ with 
> an exception: # http://www.gnu.org/licenses/autoconf-exception.html @@ 
> -28,9 +28,7 @@ homepage http://www.gnu.org/software/autoconf/ 
> installs_libs no -depends_lib port:gettext \ - port:m4 \ - port:perl5 
> +depends_lib port:gettext port:m4 master_sites gnu use_xz yes @@ -43,7 
> +41,9 @@ touch ${worksrcpath}/man/autoreconf.1 } -configure.perl 
> ${prefix}/bin/perl +# 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 test.run yes 
> test.target check
>
Larry,

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.

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.

Dave


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-changes/attachments/20150514/28cce7e9/attachment.html>