[PATCH] configure: Check for linker delayload support on mingw hosts.

[Jacek Caban]

Fixes PE-only builds with llvm-mingw.

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com>
---
  configure.ac | 5 +++++
  1 file changed, 5 insertions(+)

[Martin Storsjö]

On Fri, 8 Jan 2021, Jacek Caban wrote:
> Fixes PE-only builds with llvm-mingw.
>
> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com>
> ---
> configure.ac | 5 +++++
> 1 file changed, 5 insertions(+)

Hi,

Is this a fix for a regression - or were just delayloaded libs something 
that doesn't end up tested very often (as wine overall seemed to work fine 
recently, but maybe I didn't run into it)?

// Martin

[Jacek Caban]

On 08.01.2021 15:10, Martin Storsjö wrote:
> On Fri, 8 Jan 2021, Jacek Caban wrote:
> > Fixes PE-only builds with llvm-mingw.
> >
> > Signed-off-by: Jacek Caban <jacek(a)codeweavers.com>
> > ---
> > configure.ac | 5 +++++
> > 1 file changed, 5 insertions(+)
>
> Hi,
>
> Is this a fix for a regression - or were just delayloaded libs 
> something that doesn't end up tested very often (as wine overall 
> seemed to work fine recently, but maybe I didn't run into it)?

It's for PE-only builds, which you get when you set configure 
--host=...-mingw32. This is different than a regular Unix build with 
mingw available. It never worked out of the box with llvm-mingw 
(although it's easy to work around by passing DELAYLOADFLAG to configure 
manually).


Jacek

[Martin Storsjö]

On Fri, 8 Jan 2021, Jacek Caban wrote:
> On 08.01.2021 15:10, Martin Storsjö wrote:
> > On Fri, 8 Jan 2021, Jacek Caban wrote:
> > > Fixes PE-only builds with llvm-mingw.
> > >
> > > Signed-off-by: Jacek Caban <jacek(a)codeweavers.com>
> > > ---
> > > configure.ac | 5 +++++
> > > 1 file changed, 5 insertions(+)
> >
> > Hi,
> >
> > Is this a fix for a regression - or were just delayloaded libs something 
> > that doesn't end up tested very often (as wine overall seemed to work fine 
> > recently, but maybe I didn't run into it)?
>
> It's for PE-only builds, which you get when you set configure 
> --host=...-mingw32. This is different than a regular Unix build with mingw 
> available. It never worked out of the box with llvm-mingw (although it's easy 
> to work around by passing DELAYLOADFLAG to configure manually).

Oh, thanks, that's the distinction I missed. Yeah I haven't tested that 
case, only the usual mixed ELF/PE cases.

// Martin