FS#18300 - [rss-glx] Fix for path for installed executables

Attached to Project: Community Packages
Opened by Gerardo Exequiel Pozzi (djgera) - Friday, 12 February 2010, 00:19 GMT
Last edited by Ionut Biru (wonder) - Tuesday, 16 March 2010, 15:39 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Ionut Biru (wonder)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description: Executables are currently installed on /usr/bin but should be in /usr/lib/xscreensaver. This is the path that xscreensaver will look first, and the only path that kxsrun (KDE) understands, so at this moment these screensaver does not work under KDE. If you look at xscreensaver, it install executables in this path, not in /usr/bin.

Also fixing this, avoid rename of "plasma", and make it to work.

Patch attached against latest trunk.

Additional info:
rss-glx-0.9.1-1
This task depends upon

Closed by  Ionut Biru (wonder)
Tuesday, 16 March 2010, 15:39 GMT
Reason for closing:  Fixed
Additional comments about closing:  1.9.1-4
Comment by Corrado Primier (bardo) - Friday, 12 February 2010, 01:14 GMT
I'm not sure about what to do, but I don't really like this. The RSS screensavers are independent from xscreensaver or any other screensaver manager, and as far as I could find from the man page kxsrun supports path as a part of the screensaver name. The only real gain I can see is avoiding the renaming of plasma...
Comment by Gerardo Exequiel Pozzi (djgera) - Friday, 12 February 2010, 01:49 GMT
mmm not only gain skiping the rename, also make it to work with KDE, currently KDE list all screensavers but does not work.

Nope "kxsrun /usr/bin/lattice" will look at "/usr/lib/xscreensave/usr/bin/lattice". (I was tried this) So changing *.desktop is not useful.

1) Other option is generating symlinks at /usr/bin to /usr/lib/xscreensaver/*. (with special symlink name for "plasma")
2) Keep like now, but generate symlinks at /usr/lib/xscreensaver to /usr/bin/*. (rename executable needed)
2) Keep like now, but leave a msg in .install that for work with KDE should create a symlinks for each executable at /usr/lib/xscreensaver.

Otherwise all /usr/share/applnk/System/ScreenSavers/* and /usr/share/applnk/System/ScreenSavers/* does not work.

I prefer the first option.
Comment by Ionut Biru (wonder) - Monday, 08 March 2010, 15:32 GMT
currently any solution doesn't make gnome-screensaver list any screensaver. what i'm thinking to do is doing 1) with --program-prefix=rss-glx- do avoid conflicts and trying to figure out how incorporate simple a template in our pkgbuild to generate valid .desktops file for gnome.

[Desktop Entry]
Name=${name}
Comment=${desc}
TryExec=${trycmd}
Exec=${cmd} -r
StartupNotify=false
Terminal=false
Type=Application
Categories=GNOME;Screensaver;
Comment by Gerardo Exequiel Pozzi (djgera) - Tuesday, 09 March 2010, 03:05 GMT
Currently *.desktop files are installed on /usr/share/applnk/System/ScreenSavers/ and /usr/share/applications/screensavers/, these containst the same *.desktop files, but these are for KDE. And seems that are GNOME incompatible.

In AUR there are two packages, one for GNOME [#1] and one for KDE [#2]. Acording to comments in rss-glx-gnome, just with symlinks executables works in gnome. (My proposed option 1). And rss-glx-kde apparently seems that is not working.

[#1] http://aur.archlinux.org/packages.php?ID=31769
[#2] http://aur.archlinux.org/packages.php?ID=21733
Comment by Ionut Biru (wonder) - Monday, 15 March 2010, 21:08 GMT
rss-glx-gnome never worked for me. i can only make it work if i add the correct .desktop. i think that i'll add new .desktop files in svn and use that.
Comment by Ionut Biru (wonder) - Monday, 15 March 2010, 21:51 GMT
new packages in community-testing. please test them. for me now gnome-screensaver is detecting after i added new .desktop files
Comment by Gerardo Exequiel Pozzi (djgera) - Monday, 15 March 2010, 23:37 GMT
Thanks!

Does not work on KDE. Screensavers are not listed under KDE screensaver configuration (.desktop files are not KDE compatible). Only works if cp /usr/share/applnk/System/ScreenSavers/biof.desktop /usr/share/applications/screensavers/biof.desktop

What about doing some merge betweent these two files, and make both desktopworlds, compatible? :P Can you try this patch for .desktop file for GNOME? apply to /usr/share/applications/screensavers/biof.desktop

This change make screensaver KDE compatible ;)
Comment by Ionut Biru (wonder) - Tuesday, 16 March 2010, 00:04 GMT
nope. doesn't work. this make me think to have two more packages that provides only .desktop :D rss-glx-gnome and rss-glx-kde
Comment by Gerardo Exequiel Pozzi (djgera) - Tuesday, 16 March 2010, 00:08 GMT
ouch! :( Yes, I guess for now, is a good choice.
Comment by Ionut Biru (wonder) - Tuesday, 16 March 2010, 00:15 GMT
what about if you remove all .desktop from usr/share/applications/screensavers/. i don't understand what's the point of having kde4 .desktop compatible in usr/share/applnk/System/ScreenSavers/
Comment by Gerardo Exequiel Pozzi (djgera) - Tuesday, 16 March 2010, 00:39 GMT
Another patch.

in reverse? I can remove /usr/share/applnk/System/ScreenSavers/* without any issues. (at least on my system there are no more files under /usr/share/applnk/), but can't work if remove /usr/share/applications/screensavers/
Comment by Ionut Biru (wonder) - Tuesday, 16 March 2010, 00:43 GMT
success
Comment by Gerardo Exequiel Pozzi (djgera) - Tuesday, 16 March 2010, 00:54 GMT
wiiiiii :)
Comment by Ionut Biru (wonder) - Tuesday, 16 March 2010, 11:14 GMT
take 2, please test
Comment by Gerardo Exequiel Pozzi (djgera) - Tuesday, 16 March 2010, 14:33 GMT
Perfectly, works fine on my user and on new clean user.
Comment by Gerardo Exequiel Pozzi (djgera) - Tuesday, 16 March 2010, 15:30 GMT
minor issue, only screensaver that does not work is plasma, .desktop does not reflect the filename change (in KDE entries)

Loading...