[118533] trunk/dports/net/pidgin/Portfile

[Daniel]

David,

Would you please take over as maintainer of the port?




On Fri, Apr 4, 2014 at 4:54 PM, David Evans <devans at macports.org> wrote:

> On 4/4/14 4:01 PM, Ryan Schmidt wrote:
> >> On 4/4/14 1:22 AM, Ryan Schmidt wrote:
> >>> You should use a specific version of perl instead, like perl5.16.
> > Maybe I take that back, because:
> >
> > On Apr 4, 2014, at 07:44, Eric Gallager wrote:
> >
> >> I thought we were doing variants now, when the dependency is for more
> >> than just the "perl" executable, at least:
> >> https://trac.macports.org/ticket/43193
> > In this case, the dependency isn’t for more than just the perl
> executable, is it? The port does build and install perl modules, but it
> installs them in a pidgin-specific location, not in the versioned directory
> for whatever version of perl it found. And it doesn’t appear that the path
> to perl gets baked into any installed files. So maybe the perl version
> doesn’t matter after all.
> Mostly true with the exception of the baked-in part.  The port builds
>
> /opt/local/lib/purple-2/perl.so
>
> which has a binary dependency on the versioned libperl.dylib that
> belongs to the perl it configured with.
>
> In my mixed versioned test case this is perl5.12
>
> otool -L /opt/local/lib/purple-2/perl.so
>
> /opt/local/lib/purple-2/perl.so:
>     ...
>
> /opt/local/lib/perl5/5.12.4/darwin-thread-multi-2level/CORE/libperl.dylib
> (compatibility
> version 5.12.0, current version 5.12.4)
>     ...
>
> This is caused by this line in configure.ac:
>
> PERL_LIBS=`$perlpath -MExtUtils::Embed -e ldopts 2>/dev/null |$sedpath
> 's/-lgdbm //'`
>
> On Apr 4, 2014, at 10:59,
>
> David Evans wrote:
> >> Unfortunately that doesn't work in this situation.  configure.ac needs
> >> to be modified to respect
> >> configure.perl before a version specific perl can be used.
> >>
> >> configure.ac uses this code to test for perl:
> >>
> >>        AC_PATH_PROG(perlpath, perl)
> >>        AC_MSG_CHECKING(for Perl compile flags)
> >>        PERL_CFLAGS=`$perlpath -MExtUtils::Embed -e ccopts 2>/dev/null`
> > I’ve changed my mind about the necessity of this, but if we did want to
> use a specific perl, patching the configure file would not be necessary;
> you can just add this to configure.args:
> >
> > ac_cv_path_perlpath=${prefix}/bin/perl5.16
> >
> >
> Yes, this works (just tried it on a test build).
>
> Actually, if I were the maintainer of this port, I'd put perl, tcl and
> tk in variants as I'm not sure how much demand
> there is for these bindings to libpurple and installing them in pidgin's
> own tree makes them inconvenient to use.
>
> Unfortunately, the maintainer has been AWOL for some time so it's
> cumbersome to make changes.
>
>
>
>
>
>
> _______________________________________________
> macports-dev mailing list
> macports-dev at lists.macosforge.org
> https://lists.macosforge.org/mailman/listinfo/macports-dev
>



-- 
"America was founded by men who understood that the threat of domestic
tyranny is as great as any threat from abroad. If we want to be worthy of
their legacy, we must resist the rush toward ever-increasing state control
of our society. Otherwise, our own government will become a greater threat
to our freedoms than any foreign terrorist."
 - Ron Paul, Texas Straight Talk, May 31, 2004
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-dev/attachments/20140423/613d5969/attachment-0001.html>