Re: [2/2] winemenubuilder: add thumbnail mode for .lnk files

[Hans Leidekker]

On Fri, 2010-05-21 at 07:52 +0200, Damjan Jovanovic wrote:
> --- winemenubuilder00.c 2010-05-21 07:44:48.195864397 +0200
> +++ winemenubuilder.c   2010-05-21 07:47:25.591878489 +0200

This will not apply from the root. This patch set needs your previous
patches to make sense, right?
> Changelog:
> * winemenubuilder: add thumbnail mode for .lnk files

winemenubuilder is a winelib app, it may not run in the prefix
that the .lnk file was created in. So while these patches get rid of
duplicate launchers on the desktop, we lose support for multiple
prefixes.

[2/2] winemenubuilder: add thumbnail mode for .lnk files

[Damjan Jovanovic]

On Fri, May 21, 2010 at 12:48 PM, Hans Leidekker <hans(a)codeweavers.com> wrote:
> On Fri, 2010-05-21 at 07:52 +0200, Damjan Jovanovic wrote:
> > --- winemenubuilder00.c 2010-05-21 07:44:48.195864397 +0200
> > +++ winemenubuilder.c   2010-05-21 07:47:25.591878489 +0200
>
> This will not apply from the root. This patch set needs your previous
> patches to make sense, right?

Yes. And this particular patch is optional - the previous one alone
allegedly helps Macs, since .xpm is completely foreign there. So
please apply the prevoius one even if this one is rejected.
> > Changelog:
> > * winemenubuilder: add thumbnail mode for .lnk files
>
> winemenubuilder is a winelib app, it may not run in the prefix
> that the .lnk file was created in. So while these patches get rid of
> duplicate launchers on the desktop, we lose support for multiple
> prefixes.

The wineprefix is already lost. With the Git from about 36 hours ago,
you can double-click any .lnk file to run it. This patch merely adds
thumbnails.

Regards
Damjan Jovanovic