Re: libidn-1.23 bug in lib/Makefile cross-building for Windows

[Simon Josefsson]

René Berber <address@hidden> writes:

> On 12/19/2011 7:25 AM, Simon Josefsson wrote:
>
>> Is support for -Wl,--output-def detected properly on your platform?
>> There should be a message like 'checking if gcc/ld supports
>> -Wl,--output-def' when you run ./configure.
>
> Yes, the configure message is there with result "yes".
>
> That line is the only occurrence in the complete build log of that
> option.  It wasn't used, and I was wrong before, it should be used if
> the library is made.
>
> Looking at the Makefile targets:
>
> libidn_la_LINK = $(LIBTOOL) $(AM_V_lt) --tag=CC $(AM_LIBTOOLFLAGS) \
>         $(LIBTOOLFLAGS) --mode=link $(CCLD) $(AM_CFLAGS) $(CFLAGS) \
>         $(libidn_la_LDFLAGS) $(LDFLAGS) -o $@
>
> libidn_la_LDFLAGS = -version-info \
>         $(LT_CURRENT):$(LT_REVISION):$(LT_AGE) -no-undefined \
>         $(am__append_3) $(am__append_4) $(am__append_5)
>
> At this point the .def file is made, but another make rule which has
> defexec-DATA as dependency fired before this one: all-am (has DATA as
> dependency, and DATA is defined as defexec_DATA which is the .def file).

Then it is the ordering issue that I suspected, and I was able to
reproduce the problem with 'make -j' in the cross-build environment.
The problem should be fixed with this commit:

http://git.savannah.gnu.org/cgit/libidn.git/commit/?id=616039509ebc13586ab61ba5d78a06bf500d6cf6

Thanks,
/Simon