I haven't looked at the code yet, but after a user question about missing entries for crossover office I found out that two things in the relevant .desktop files caused the problem: 1) It was missing the Categories field. We should really just put those in 'Other'. 2) It use " quotes around the Exec= field (Exec="/opt/cxoffice/bin/wine" ...). We should probably check for that, shouldn't we?
To clarify, the quotes were around the executable, not the entire field. Exec="/opt/cxoffice/bin/wine" --check --cx-app "C://Program Files//Microsoft Office//Office//WINWORD.EXE"
There *is* an Other menu, but I guess it isn't being used or set up properly. I guess I haven't looked at that code in a while, so I'm not sure what it's doing. As for the quoting... this is really a PITA, and I don't think xfdesktop should have to do this. Since xfce_exec() is just taking command lines, it should handle breaking up the commands into an argv array, and stripping any quoting as needed. If xfdesktop strips the quoting, and there's a space in one of the arguments, it won't get executed properly.
#1 should be fixed on both branches, though i haven't tested beyond a compile test. what do you think about #2? i'd much rather see this fixed in xfce_exec(). (tsk tsk, two bugs in one report ^_~)
(In reply to comment #3) > #1 should be fixed on both branches, though i haven't tested beyond a compile test. > Yeah, I think so. > what do you think about #2? i'd much rather see this fixed in xfce_exec(). > Yeah, i agree, I'll have a look unless you do so first. > (tsk tsk, two bugs in one report ^_~) Hehe, I knew you were going to say that ;-)
Should be fixed now.