On Wed, Jan 5, 2011 at 1:31 PM, Ken Thomases ken@codeweavers.com wrote:
On Jan 5, 2011, at 5:25 AM, Damjan Jovanovic wrote:
It doesn't seem like a good idea to introduce a platform-specific dependency for ICNS support, but you might want to get feedback from other Wine developers.
Hmm. I guess I don't see how ICNS is anything other than platform-specific. Is it used anywhere else? Is it ever expected to be?
By the way, as for your other patch, freedesktop also supports multiple icon sizes, but it uses a strange directory structure to do it which I haven't quite gotten to work yet.
OK. Do you think my patch should be held up until you find an approach that works for both?
The patch seems fine (doesn't seem to regress anything), but aren't ICNS files invisible anyway with winemenubuilder writing .desktop files which MacOS can't use?
Thanks for your quick reply!
-Ken
Damjan