After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 607829 - sawfish doesn't start if a desktop file contains an error
sawfish doesn't start if a desktop file contains an error
Status: RESOLVED FIXED
Product: Sawfish
Classification: Deprecated
Component: Window Manager
1.6.x
Other Linux
: Normal normal
: 1.6.x
Assigned To: sawfish-maint
sawfish-maint
Depends on:
Blocks:
 
 
Reported: 2010-01-23 00:32 UTC by liam
Modified: 2011-02-20 20:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
A .desktop file that causes sawfish to fail. (269 bytes, text/plain)
2010-01-23 10:53 UTC, Shlomi Fish
Details

Description liam 2010-01-23 00:32:42 UTC
If any desktop file should happen to contain an error, such as a leading semicolon in the category list, sawfish won't start, and gives an error about end of input (not naming the faulty file).

Although the desktop files should be fixed, it's not always possible (e.g. a user might not have root access)...

Maybe the parsing could be made more robust?
Comment 1 Shlomi Fish 2010-01-23 10:53:32 UTC
Created attachment 152072 [details]
A .desktop file that causes sawfish to fail.

This is a .desktop file that causes sawfish to give up and not load. It is taken from the old package of the Mandriva prboom.
Comment 2 Christopher Roy Bratusek 2010-02-10 09:15:21 UTC
Thanks for the catch. I've just commited a patch by Matthew Love (the contributor who volounteered the apps-menu), which should fix the issue.

Can you give it a try?
Comment 3 Teika Kazura 2010-02-13 05:54:10 UTC
The patch is available at:
http://mail.gnome.org/archives/sawfish-list/2010-February/msg00031.html

Save the file somewhere, and do
$ cd /usr/share/sawfish/1.6.x/
$ patch -p1 < [patch-file-name]

and restart Sawfish.

I think we can release the Sawfish-1.6.2 which contains the above fix this month.
Thanks.
Comment 4 Teika Kazura 2010-02-13 05:57:54 UTC
Oops, if *.desktop is not readable, like by a broken symlink, Sawfish still doesn't start, but it'll be fixed.
Comment 5 Christopher Roy Bratusek 2010-02-13 09:41:08 UTC
O.K. current HEAD does no longer crash on non-readable files.
actually it doesn't crash, it "just" stops the startup-process.

Symlinks should also be no problem, neither normal, circular nor broken. But I haven't do any testing on hardlinks.
Comment 6 Teika Kazura 2010-02-16 06:01:38 UTC
Now we think the fix is complete. The latest code is available in git:
http://git.gnome.org/browse/sawfish/tree/lisp/sawfish/wm/ext/apps-menu.jl

For Sawfish-1.6.2 release, sorry, it may not be so soon.