[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 08/11] gnu: libtool: support cross-libtool mingw.
From: |
Andy Wingo |
Subject: |
Re: [PATCH 08/11] gnu: libtool: support cross-libtool mingw. |
Date: |
Mon, 09 May 2016 09:39:12 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
On Sun 08 May 2016 22:42, Jan Nieuwenhuizen <address@hidden> writes:
> diff --git a/gnu/packages/patches/libtool-mingw.patch
> b/gnu/packages/patches/libtool-mingw.patch
> new file mode 100644
> index 0000000..1246bd7
> --- /dev/null
> +++ b/gnu/packages/patches/libtool-mingw.patch
> @@ -0,0 +1,40 @@
> +I do not know why stat (and other functions here) get #define'd to their
> +underscore variants; I find no apparent need or rationale for that. However,
> +redefining stat also impacts struct stat, breaking lstat's signature. That
> is
> +fixed be #define'ing lstat along.
> +
> +Jan Nieuwenhuizen
> +
> +Upstream status: not yet presented upstream.
> +
> +--- libtool-2.4.6/build-aux/ltmain.in~ 2015-02-06 13:57:56.000000000
> +0100
> ++++ libtool-2.4.6/build-aux/ltmain.in 2016-05-06 07:46:29.425142546
> +0200
> +@@ -3658,12 +3658,10 @@
> + #if defined _MSC_VER
> + # define setmode _setmode
> +-# define stat _stat
> + # define chmod _chmod
> + # define getcwd _getcwd
> + # define putenv _putenv
> + # define S_IXUSR _S_IEXEC
> + #elif defined __MINGW32__
> + # define setmode _setmode
> +-# define stat _stat
> + # define chmod _chmod
> + # define getcwd _getcwd
This doesn't look right to me. Stat was actually the first of this set
to be added to the weird define list, via:
commit 781fc82e1b2bceaa5c55388d6ab1d1744663f992
Author: Peter Rosin <address@hidden>
Date: Sun Jul 22 17:57:10 2007 +0000
* libltdl/config/ltmain.m4sh (func_emit_cwrapperexe_src): Add
support for Microsoft Visual C. Also, older MinGW versions
seem to need stdint.h to find intptr_t.
And there does appear to exist "struct _stat" in the API
(https://msdn.microsoft.com/en-us/library/14h5k7ff.aspx). Does it mean
that "lstat" is coming from somewhere else (i.e. not the system libc),
expecting to have a "struct stat" as an argument? I don't know very
much about this stuff but this change looks fishy to me.
Andy
- Re: [PATCH 03/11] gnu: Add function libiconv-if-needed., (continued)
- [PATCH 10/11] gnu: readline: support mingw., Jan Nieuwenhuizen, 2016/05/08
- [PATCH 09/11] gnu: ncurses: build mingw with libtool., Jan Nieuwenhuizen, 2016/05/08
- [PATCH 08/11] gnu: libtool: support cross-libtool mingw., Jan Nieuwenhuizen, 2016/05/08
- Re: [PATCH 08/11] gnu: libtool: support cross-libtool mingw.,
Andy Wingo <=
- [PATCH 11/11] gnu: guile-2.0: support mingw., Jan Nieuwenhuizen, 2016/05/08