gnome-menus doesn't send notification for new .desktop files

[Chris Lord]

Binary package hint: alacarte

If you add/edit menus in Edgy, the menu shown in alacarte isn't updated until you reload alacarte and the actual gnome menu isn't updated until either a package upgrade updates it or you kill gnome-panel.

[Travis Watkins (amaranth)]

Alacarte and the menu both get their updates from gnome-menus.

[Pete Ryland (pdr)]

Ok, I've been looking at this and I think I've worked out what is going wrong. It is indeed in gnome-menus, and it's not passing on the notifications from gnome-vfs that the .desktop files have been created/changed/etc. I'll post a patch shortly. This will also close five or six other duplicate bugs.

[Pete Ryland (pdr)]

Ok, here is the patch. This seems to make everything work as it should.

[Pete Ryland (pdr)]

And here is another patch, which fixes the handling of .desktop files with a space in their name (amongst other uri-encodable characters). The problem was that gnomvfs converts spaces to %20 encoding. I've fixed this by using gnomevfs's helper functions to do the encoding and decoding.

[Sebastien Bacher (seb128)]

Pete, thanks for your work on that. What was that gnome-menus upload you did for edgy (which is frozen)?

[wouter bolsterlee (wbolster)]

Could you please use unified diff (diff -u) when sending diffs? (also to gnome mailing lists)

[Pete Ryland (pdr)]

Seb, no problems. The upload I tried to make was just for the above two patches. I put the first one into the debian/patches directory as 11_desktop_notifications.patch and the second one was simply applied (to the gnomevfs patch). I should have known it wasn't going to work. :-/ I can send you a debdiff or something if you like.

[Sebastien Bacher (seb128)]

attaching a "diff -u" version of the patches to the bug would be nice

[Sebastien Bacher (seb128)]

Comment from upstream:

"Doesn't look right to me.

Take the simple case of a .desktop file being deleted. We only want to invoke
the monitors if something has actually changed in our view of the world, so we
only want to invoke monitors if cached_dir_remove_entry() returns TRUE

Also marking this bug as NEEDINFO, as there isn't enough info for me to figure
out the bug myself - e.g. what exactly is alacarte doing when you copy an item,
what does the gnome-menus verbose log show etc. ?"

[Philip Paquette (pcpaquette)]

Subscribing Travis Watkins to this bug, since upstream developers have marked this bug as "Needs Info" in February and Travis is probably the person who can answer best the question: "what exactly is alacarte doing when you copy an item, what does the gnome-menus verbose log show etc. ?""

[Sebastien Bacher (seb128)]

gnome-menus (2.19.90-0ubuntu2) gutsy; urgency=low

  * debian/patches/07_gnomevfs.patch:
    - patch change by Pete Ryland, fix notification for path with spaces
      and special chars (LP: #64264)

 -- Sebastien Bacher <email address hidden> Fri, 24 Aug 2007 15:48:17 +0200