[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
autoconf: AC_SYS_LARGEFILE should output to CPPFLAGS
From: |
Thorsten Glaser |
Subject: |
autoconf: AC_SYS_LARGEFILE should output to CPPFLAGS |
Date: |
Wed, 25 Jan 2017 17:57:03 +0100 (CET) |
User-agent: |
Alpine 2.20 (DEB 67 2015-01-07) |
On Wed, 25 Jan 2017, Thorsten Glaser wrote:
> Looking at the source, AC_SYS_LARGEFILE calls AC_DEFINE_UNQUOTED,
> which appends to DEFS (which I agree with Tom Dickey is wrong) as
> Florian Weimer said, but — even more confusingly — only when
> AC_CONFIG_HEADERS is not used.
>
> If AC_CONFIG_HEADERS is used, the definition is only written to
> its output file (config.h by default).
>
> This is even more unfortunate, because now, if any program includes
> a glibc system header before "config.h", the definition will be
> ignored, because it comes too late.
>
> CPPFLAGS is r̲e̲a̲l̲l̲y̲ the correct place for this. Again.
Let’s keep track of t̲h̲i̲s̲ bug here <address@hidden>.
Again, please forward this upstream, too.
The workaround I was forced to use in a real-world package is thus:
https://anonscm.debian.org/cgit/pkg-remote/xrdp.git/plain/debian/patches/lfs.diff?id=e17430063641d44f5596b5bfc1b32ac4ba39f9f1
This feels completely wrong (and breaks with nōn-GCC compilers).
Thanks in advance,
//mirabilos (current hat: Debian Developer)
--
tarent solutions GmbH
Rochusstraße 2-4, D-53123 Bonn • http://www.tarent.de/
Tel: +49 228 54881-393 • Fax: +49 228 54881-235
HRB 5168 (AG Bonn) • USt-ID (VAT): DE122264941
Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg
- autoconf: AC_SYS_LARGEFILE should output to CPPFLAGS,
Thorsten Glaser <=
- Re: autoconf: AC_SYS_LARGEFILE should output to CPPFLAGS, Eric Blake, 2017/01/25
- Re: autoconf: AC_SYS_LARGEFILE should output to CPPFLAGS, Thorsten Glaser, 2017/01/25
- Re: autoconf: AC_SYS_LARGEFILE should output to CPPFLAGS, Zack Weinberg, 2017/01/25
- Re: autoconf: AC_SYS_LARGEFILE should output to CPPFLAGS, Thorsten Glaser, 2017/01/25
- Re: autoconf: AC_SYS_LARGEFILE should output to CPPFLAGS, Zack Weinberg, 2017/01/25
- Re: autoconf: AC_SYS_LARGEFILE should output to CPPFLAGS, Paul Eggert, 2017/01/25
- Re: autoconf: AC_SYS_LARGEFILE should output to CPPFLAGS, Thorsten Glaser, 2017/01/25