winemenubuilder: appbundle patch 20110618

[Per Johansson]

Hello again,

17 aug 2011 kl. 00:06 skrev Steven Edwards:

> I've finally gotten back around to hacking the *.app bundle support in
> winemenubuilder and come close to having something that mostly works.

I'm attaching my current work, since I'm not sure when I'll work on it next (might be tomorrow, might be never).

> - The first icon/bundle always seems to get the wrong icon, and uses
> the default wine icon. I need to take some time to understand the code
> a bit better and see why this is.

I don't seem to have this problem. I commented out identifier from Info.plist, since I think it's a bad idea to have a non-unique one. Perhaps that was it?
I also commented out the signature and PkgInfo files, since they're not really needed. Xcode on Lion does not create them anymore AFAICT.

> - Bundles are created in $HOME/Applications/wine and icon's are
> generated using the icns code. Support for building bundles for
> shortcuts on $HOME/Desktop has not been hacked on yet.

I've added bundles for desktop and also for URLs. I decided to create subdirectories in ~/Applications/Wine since a Readme icon will not tell which readme it'll open without living in the correct folder. Also, if you add the folder to the Dock and set view type to list, it kind of works as a Start menu.

> I welcome your feedback.

There's a few things left to be done. The pass by globals into platform_write_icon is kind of getting in the way, those values should be merged into the main execution path. I also think the xdg specific functions should also be extracted to a separate file for clarity (but perhaps that's a separate task).

The two files are diff vs. the patch from Steven and diff vs current HEAD. My individual commits can be seen at https://github.com/morth/wine/commits/master (should create a separate branch I guess).

Regards,
-- 
Per Johansson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: appbundle-diff-head.patch
Type: application/octet-stream
Size: 32209 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20110820/c2cad193/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: appbundle-diff-steven.patch
Type: application/octet-stream
Size: 31000 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20110820/c2cad193/attachment-0003.obj>