Re: [PATCH] menubar: Handle .desktop files with multiple sections

2012-12-01 Thread Arvydas Sidorenko
I am trying to read and understand the specification. If I understand this correctly, then a Desktop Entry group is required and this is what we are really interested in. The only other kind of group that I have found are Actions which e.g. is Play/Pause/Previous/... in the case of totem. None

Re: [PATCH] menubar: Handle .desktop files with multiple sections

2012-12-01 Thread Uli Schlachter
On 01.12.2012 15:59, Arvydas Sidorenko wrote: I am trying to read and understand the specification. If I understand this correctly, then a Desktop Entry group is required and this is what we are really interested in. The only other kind of group that I have found are Actions which e.g. is

Re: [PATCH] menubar: Handle .desktop files with multiple sections

2012-12-01 Thread Arvydas Sidorenko
Thanks, merged. Should this be enough to fix FS#1057 or am I missing anything? Yes, it does fix the reported issue. -- To unsubscribe, send mail to awesome-devel-unsubscr...@naquadah.org.

Re: [PATCH] menubar: Handle .desktop files with multiple sections

2012-11-29 Thread Arvydas Sidorenko
Thoughts on this one are welcome. I would change .desktop group name regexp to ^%s*%[([^%]]+)%]%s*$ since white spaces do come in once in a while. Towards the right direction, but still doesn't solve FS#1057. Overall I think I would ignore all groups except [Desktop Entry]. It would solve the

[PATCH] menubar: Handle .desktop files with multiple sections

2012-11-28 Thread Alexander Yakushev
that use of GDB is not necessary. -- Branimir Karadžić From 7fcb8bd535eddf8532922ec4c51a9d1df0ff44db Mon Sep 17 00:00:00 2001 From: Alexander Yakushev a...@bytopia.org Date: Wed, 28 Nov 2012 15:12:05 +0200 Subject: [PATCH] menubar: Handle .desktop files with multiple sections Desktop file standard