KDE checks for xscreensaver-gnome in wrong place

[Adam Weinberger]

Joe Marcus Clarke wrote:
> On Fri, 2005-03-25 at 21:24 -0500, Adam Weinberger wrote:
> 
>>kdeartwork3 and xscreensaver-kde do some incorrect GNOME-related checking.
>>
>>They both check to see if xscreensaver-gnome is installed by looking for
>>${X11BASE}/share/gnome/control-center-2.0/capplets/screensaver-properties.desktop
>>
>>But, that file is now 
>>${X11BASE}/share/gnome/applications/gnome-screensaver-properties.desktop
> 
> 
> This is only in xscreensaver-gnome-4.21 which has not been committed to
> the ports tree due to the freeze.  Therefore, what is in the tre now
> should be correct.

Oh, geez. I completely forgot that I wasn't running xscreensaver-gnome 
from the FreeBSD ports tree. *shakes head*

>>It would seem to me that this change would be good to commit before 5.4 
>>is tagged. As it stands now, kdeartwork3 cannot build with 
>>xscreensaver-gnome installed.
> 
> 
> It can with a pure FreeBSD ports tree.  Though it might be a good idea
> to add
> ${X11BASE}/share/gnome/applications/gnome-screensaver-properties.desktop
> as an additional file to check.

Would such a change be admissible for 5.4?

# Adam


-- 

Adam Weinberger
adamw at magnesium.net || adamw at FreeBSD.org
adamw at vectors.cx    ||   adamw at gnome.org
http://www.vectors.cx