Date: Fri, 07 Jun 2002 19:02:18 +0300 From: Maxim Sobolev <sobomax@FreeBSD.org> To: Doug Barton <DougB@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: ports/x11/xscreensaver Makefile distinfo pkg-plistports/x11/xscreensaver/files patch-ab patch-driver::Makefile.inpatch-driver_Makefile.in Message-ID: <3D00D90A.9B67BDB0@FreeBSD.org> References: <20020606134542.O4933-100000@master.gorean.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Doug Barton wrote: > > On Thu, 6 Jun 2002, Maxim Sobolev wrote: > > > Maxim Sobolev wrote: > > > > > > Please put PLIST_SUB for nognome case back, because otherwise PLIST is > > > broken. > > Please explain why that is true, since it works perfectly fine for me as > is. This is necessary when the GNOME is detected, but isn't explicitly turned on. Unfortunately our bsd.gnome.mk is broken in this sense, because it thinks that if gnome-config script is installed then all GNOME1 desktop should be pulled in (gnome-config script is part of gnomelibs), which screws up GNOME2. I've added an ugly workarond into xscreensaver to avoid this, but it needs this PLIST_SUB. In a nutshell, just trust me - it can't cause any problems, because it is only defined in the nognome case. > > > Attached is the patch which fixes this problem and another problem > > with PLIST (webcollage-helper) > > Please see my commit message as to why this isn't valid. One other thought > did occur to me though... is gdk-pixbuf always installed for gnome users? > If so, we could put webcollage-helper in under the PLIST_SUB for gnome > stuff. Yes, it does. Actually I do not see why you don't want to just include gdk-pixbuf into the LIB_DEPENDS - this library is very slim and doesn't have any dependencies apart from usual graphics libs and gtk+. This would resolve all problems. > > as well as fixes libintl detection and enables localisation of the > > frontend. Please either commit this patch or allow me commit it. > > I'll take a look at these bits and let you know, thanks. Any news on this? -Maxim To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3D00D90A.9B67BDB0>