Re: [PATCH] x11/nitrogen: enable Xinerama

2012-09-19 Thread Gregor Best
On Wed, Sep 19, 2012 at 08:26:26PM +0200, David Coppa wrote: > [...] > Also, do not hardcode /usr/X11R6/: use "-L${X11BASE}/lib" instead. > [...] Done. Index: Makefile === RCS file: /cvs/ports/x11/nitrogen/Makefile,v retrieving revis

Re: [PATCH] x11/nitrogen: enable Xinerama

2012-09-19 Thread Gregor Best
On Wed, Sep 19, 2012 at 03:02:56PM -0500, Kent R. Spillner wrote: > [...] > Why does nitrogen need the extra hint to find Xinerama but not X11? > [...] I think nitrogen itself will build fine without -L${X11BASE}/lib once configure generated the makefile, it's just that the test configure runs to

Re: [PATCH] x11/nitrogen: enable Xinerama

2012-09-19 Thread Kent R. Spillner
Hey, dude- > x11/nitrogen has --disable-xinerama in the configure args. This > is presumably because the build failed without it, not fixing the > real issue. The attached patch simply removes the configure flag > and adds /usr/X11R6/lib to the LDFLAGS to make configure's Xinerama > test compile a

Re: [PATCH] x11/nitrogen: enable Xinerama

2012-09-19 Thread David Coppa
On Wed, Sep 19, 2012 at 8:05 PM, Gregor Best wrote: > Bumped > > Index: Makefile > === > RCS file: /cvs/ports/x11/nitrogen/Makefile,v > retrieving revision 1.8 > diff -u -r1.8 Makefile > --- Makefile15 Jun 2012 08:30:24 -

Re: [PATCH] x11/nitrogen: enable Xinerama

2012-09-19 Thread Gregor Best
Bumped Index: Makefile === RCS file: /cvs/ports/x11/nitrogen/Makefile,v retrieving revision 1.8 diff -u -r1.8 Makefile --- Makefile15 Jun 2012 08:30:24 - 1.8 +++ Makefile19 Sep 2012 18:05:13 - @@ -3,7 +3,7 @@ COM

Re: [PATCH] x11/nitrogen: enable Xinerama

2012-09-19 Thread David Coppa
On Wed, Sep 19, 2012 at 5:55 PM, Gregor Best wrote: > Ahoi, > > x11/nitrogen has --disable-xinerama in the configure args. This is presumably > because the build failed without it, not fixing the > real issue. The attached patch simply removes the configure flag and adds > /usr/X11R6/lib to the

[PATCH] x11/nitrogen: enable Xinerama

2012-09-19 Thread Gregor Best
Ahoi, x11/nitrogen has --disable-xinerama in the configure args. This is presumably because the build failed without it, not fixing the real issue. The attached patch simply removes the configure flag and adds /usr/X11R6/lib to the LDFLAGS to make configure's Xinerama test compile and work. Ind