Re: wip/phantomjs upgrade needed to avoid X dependency

On Sun, Aug 05, 2012 at 04:05:48PM +0400, Aleksej Saushev wrote:
> Mayuresh <[email protected]> writes:
> 
> > On Sat, Aug 04, 2012 at 05:08:42PM -0400, Matthew Mondor wrote:
> >> library.  This looks like Linux-centric code, I think that patches to
> >> the configure script (or its input config.in etc, with running
> >> autoconf) would be necessary.
> >
> > Thanks. If there are any comments on this patch, please share. Or I'll
> > commit.
> 
> Yes. This patch is wrong. See pkgsrc/mk/dlopen.buildlink3.mk

Thanks. I wasn't sure about the usage. Copied the BUILDLINK_TRANSFORM line
from other packages that use dlopen buildlink.

Is this change ok?

===================================================================
RCS file: /cvsroot/pkgsrc-wip/wip/phantomjs/Makefile,v
retrieving revision 1.1.1.1
diff -r1.1.1.1 Makefile
26a27,29
> .include "../../mk/dlopen.buildlink3.mk"
> BUILDLINK_TRANSFORM+=   opt:-ldl:${BUILDLINK_LDADD.dl:M*}
> 
Index: distinfo
===================================================================

Mayuresh

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/