diff options
| author | Khem Raj <raj.khem@gmail.com> | 2016-12-18 22:39:55 -0800 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-12-20 15:22:16 +0000 |
| commit | c9786e1e1278ff9569ec90cafe1b8ae9f16259b2 (patch) | |
| tree | bb1e920feaedecafad6ae87a3e4d5621e0c3a947 | |
| parent | f87751165deb58b2e4f967e1b0da7cd3853fcbff (diff) | |
| download | poky-c9786e1e1278ff9569ec90cafe1b8ae9f16259b2.tar.gz | |
autoconf: Fix AC_HEADER_MAJOR with glibc 2.25
Backport the patch from master for 2.69
(From OE-Core rev: 83081afc08695ea42046208ca8ab077e482d3018)
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta/recipes-devtools/autoconf/autoconf/AC_HEADER_MAJOR-port-to-glibc-2.25.patch | 162 | ||||
| -rw-r--r-- | meta/recipes-devtools/autoconf/autoconf_2.69.bb | 1 |
2 files changed, 163 insertions, 0 deletions
diff --git a/meta/recipes-devtools/autoconf/autoconf/AC_HEADER_MAJOR-port-to-glibc-2.25.patch b/meta/recipes-devtools/autoconf/autoconf/AC_HEADER_MAJOR-port-to-glibc-2.25.patch new file mode 100644 index 0000000000..fc37236bf8 --- /dev/null +++ b/meta/recipes-devtools/autoconf/autoconf/AC_HEADER_MAJOR-port-to-glibc-2.25.patch | |||
| @@ -0,0 +1,162 @@ | |||
| 1 | From e17a30e987d7ee695fb4294a82d987ec3dc9b974 Mon Sep 17 00:00:00 2001 | ||
| 2 | From: Eric Blake <eblake@redhat.com> | ||
| 3 | Date: Wed, 14 Sep 2016 08:17:06 -0500 | ||
| 4 | Subject: [PATCH] AC_HEADER_MAJOR: port to glibc 2.25 | ||
| 5 | |||
| 6 | glibc 2.25 is deprecating the namespace pollution of <sys/types.h> | ||
| 7 | injecting major(), minor(), and makedev() into the compilation | ||
| 8 | environment, with a warning that insists that users include | ||
| 9 | <sys/sysmacros.h> instead. However, because the expansion of | ||
| 10 | AC_HEADER_MAJOR didn't bother checking sys/sysmacros.h until | ||
| 11 | after probing whether sys/types.h pollutes the namespace, it was | ||
| 12 | not defining MAJOR_IN_SYSMACROS, with the result that code | ||
| 13 | compiled with -Werror chokes on the deprecation warnings because | ||
| 14 | it was not including sysmacros.h. | ||
| 15 | |||
| 16 | In addition to fixing autoconf (which only benefits projects | ||
| 17 | that rebuild configure after this fix is released), we can also | ||
| 18 | give a hint to distros on how they can populate config.site with | ||
| 19 | a cache variable to force pre-existing configure scripts without | ||
| 20 | the updated macro to behave sanely in the presence of glibc 2.25 | ||
| 21 | (the documentation is especially useful since that cache variable | ||
| 22 | is no longer present in autoconf after this patch). | ||
| 23 | |||
| 24 | Note that mingw lacks major/minor/makedev in any of its standard | ||
| 25 | headers; for that platform, the behavior of this macro is unchanged | ||
| 26 | (code using the recommended include formula will get a compile error | ||
| 27 | when trying to use major(), whether before or after this patch); but | ||
| 28 | for now, it is assumed that programs actually concerned with | ||
| 29 | creating devices are not worried about portability to mingw. If | ||
| 30 | desired, a later patch could tighten AC_HEADER_MAJOR to fail at | ||
| 31 | configure time if the macros are unavailable in any of the three | ||
| 32 | system headers, but that semantic change is not worth mixing into | ||
| 33 | this patch. | ||
| 34 | |||
| 35 | * lib/autoconf/headers.m4 (AC_HEADER_MAJOR): Drop check for | ||
| 36 | major within sys/types.h; it interferes with the need to check | ||
| 37 | sysmacros.h first. | ||
| 38 | * doc/autoconf.texi (Particular Headers) <AC_HEADER_MAJOR>: Expand | ||
| 39 | details on usage, and on workarounds for non-updated projects. | ||
| 40 | |||
| 41 | Signed-off-by: Eric Blake <eblake@redhat.com> | ||
| 42 | --- | ||
| 43 | Upstream-Status: Backport | ||
| 44 | |||
| 45 | doc/autoconf.texi | 35 +++++++++++++++++++++++++++++++---- | ||
| 46 | lib/autoconf/headers.m4 | 30 ++++++++++++++---------------- | ||
| 47 | 2 files changed, 45 insertions(+), 20 deletions(-) | ||
| 48 | |||
| 49 | Index: autoconf-2.69/doc/autoconf.texi | ||
| 50 | =================================================================== | ||
| 51 | --- autoconf-2.69.orig/doc/autoconf.texi | ||
| 52 | +++ autoconf-2.69/doc/autoconf.texi | ||
| 53 | @@ -15,7 +15,7 @@ | ||
| 54 | @c The ARG is an optional argument. To be used for macro arguments in | ||
| 55 | @c their documentation (@defmac). | ||
| 56 | @macro ovar{varname} | ||
| 57 | -@r{[}@var{\varname\}@r{]}@c | ||
| 58 | +@r{[}@var{\varname\}@r{]} | ||
| 59 | @end macro | ||
| 60 | |||
| 61 | @c @dvar(ARG, DEFAULT) | ||
| 62 | @@ -23,7 +23,7 @@ | ||
| 63 | @c The ARG is an optional argument, defaulting to DEFAULT. To be used | ||
| 64 | @c for macro arguments in their documentation (@defmac). | ||
| 65 | @macro dvar{varname, default} | ||
| 66 | -@r{[}@var{\varname\} = @samp{\default\}@r{]}@c | ||
| 67 | +@r{[}@var{\varname\} = @samp{\default\}@r{]} | ||
| 68 | @end macro | ||
| 69 | |||
| 70 | @c Handling the indexes with Texinfo yields several different problems. | ||
| 71 | @@ -5926,10 +5926,37 @@ Also see @code{AC_STRUCT_DIRENT_D_INO} a | ||
| 72 | @cvindex MAJOR_IN_SYSMACROS | ||
| 73 | @hdrindex{sys/mkdev.h} | ||
| 74 | @hdrindex{sys/sysmacros.h} | ||
| 75 | -If @file{sys/types.h} does not define @code{major}, @code{minor}, and | ||
| 76 | -@code{makedev}, but @file{sys/mkdev.h} does, define | ||
| 77 | -@code{MAJOR_IN_MKDEV}; otherwise, if @file{sys/sysmacros.h} does, define | ||
| 78 | -@code{MAJOR_IN_SYSMACROS}. | ||
| 79 | +Detect the headers required to use @code{makedev}, @code{major}, and | ||
| 80 | +@code{minor}. These functions may be defined by @file{sys/mkdev.h}, | ||
| 81 | +@code{sys/sysmacros.h}, or @file{sys/types.h}. | ||
| 82 | + | ||
| 83 | +@code{AC_HEADER_MAJOR} defines @code{MAJOR_IN_MKDEV} if they are in | ||
| 84 | +@file{sys/mkdev.h}, or @code{MAJOR_IN_SYSMACROS} if they are in | ||
| 85 | +@file{sys/sysmacros.h}. If neither macro is defined, they are either in | ||
| 86 | +@file{sys/types.h} or unavailable. | ||
| 87 | + | ||
| 88 | +To properly use these functions, your code should contain something | ||
| 89 | +like: | ||
| 90 | + | ||
| 91 | +@verbatim | ||
| 92 | +#include <sys/types.h> | ||
| 93 | +#ifdef MAJOR_IN_MKDEV | ||
| 94 | +# include <sys/mkdev.h> | ||
| 95 | +#elif defined MAJOR_IN_SYSMACROS | ||
| 96 | +# include <sys/sysmacros.h> | ||
| 97 | +#endif | ||
| 98 | +@end verbatim | ||
| 99 | + | ||
| 100 | +Note: Configure scripts built with Autoconf 2.69 or earlier will not | ||
| 101 | +detect a problem if @file{sys/types.h} contains definitions of | ||
| 102 | +@code{major}, @code{minor}, and/or @code{makedev} that trigger compiler | ||
| 103 | +warnings upon use. This is known to occur with GNU libc 2.25, where | ||
| 104 | +those definitions are being deprecated to reduce namespace pollution. | ||
| 105 | +If it is not practical to use Autoconf 2.70 to regenerate the configure | ||
| 106 | +script of affected software, you can work around the problem by setting | ||
| 107 | +@samp{ac_cv_header_sys_types_h_makedev=no}, as an argument to | ||
| 108 | +@command{configure} or as part of a @file{config.site} site default file | ||
| 109 | +(@pxref{Site Defaults}). | ||
| 110 | @end defmac | ||
| 111 | |||
| 112 | @defmac AC_HEADER_RESOLV | ||
| 113 | Index: autoconf-2.69/lib/autoconf/headers.m4 | ||
| 114 | =================================================================== | ||
| 115 | --- autoconf-2.69.orig/lib/autoconf/headers.m4 | ||
| 116 | +++ autoconf-2.69/lib/autoconf/headers.m4 | ||
| 117 | @@ -502,31 +502,29 @@ fi | ||
| 118 | |||
| 119 | # AC_HEADER_MAJOR | ||
| 120 | # --------------- | ||
| 121 | +# Thanks to glibc 2.25 deprecating macros in sys/types.h, coupled with | ||
| 122 | +# back-compat to autoconf 2.69, we need the following logic: | ||
| 123 | +# Check whether <sys/types.h> compiles. | ||
| 124 | +# If <sys/mkdev.h> compiles, assume it provides major/minor/makedev. | ||
| 125 | +# Otherwise, if <sys/sysmacros.h> compiles, assume it provides the macros. | ||
| 126 | +# Otherwise, either the macros were provided by <sys/types.h>, or do | ||
| 127 | +# not exist on the platform. Code trying to use these three macros is | ||
| 128 | +# assumed to not care about platforms that lack the macros. | ||
| 129 | AN_FUNCTION([major], [AC_HEADER_MAJOR]) | ||
| 130 | AN_FUNCTION([makedev], [AC_HEADER_MAJOR]) | ||
| 131 | AN_FUNCTION([minor], [AC_HEADER_MAJOR]) | ||
| 132 | AN_HEADER([sys/mkdev.h], [AC_HEADER_MAJOR]) | ||
| 133 | AC_DEFUN([AC_HEADER_MAJOR], | ||
| 134 | -[AC_CACHE_CHECK(whether sys/types.h defines makedev, | ||
| 135 | - ac_cv_header_sys_types_h_makedev, | ||
| 136 | -[AC_LINK_IFELSE([AC_LANG_PROGRAM([[@%:@include <sys/types.h>]], | ||
| 137 | - [[return makedev(0, 0);]])], | ||
| 138 | - [ac_cv_header_sys_types_h_makedev=yes], | ||
| 139 | - [ac_cv_header_sys_types_h_makedev=no]) | ||
| 140 | -]) | ||
| 141 | - | ||
| 142 | -if test $ac_cv_header_sys_types_h_makedev = no; then | ||
| 143 | +[AC_CHECK_HEADERS_ONCE([sys/types.h]) | ||
| 144 | AC_CHECK_HEADER(sys/mkdev.h, | ||
| 145 | [AC_DEFINE(MAJOR_IN_MKDEV, 1, | ||
| 146 | [Define to 1 if `major', `minor', and `makedev' are | ||
| 147 | declared in <mkdev.h>.])]) | ||
| 148 | - | ||
| 149 | - if test $ac_cv_header_sys_mkdev_h = no; then | ||
| 150 | - AC_CHECK_HEADER(sys/sysmacros.h, | ||
| 151 | - [AC_DEFINE(MAJOR_IN_SYSMACROS, 1, | ||
| 152 | - [Define to 1 if `major', `minor', and `makedev' | ||
| 153 | - are declared in <sysmacros.h>.])]) | ||
| 154 | - fi | ||
| 155 | +if test $ac_cv_header_sys_mkdev_h = no; then | ||
| 156 | + AC_CHECK_HEADER(sys/sysmacros.h, | ||
| 157 | + [AC_DEFINE(MAJOR_IN_SYSMACROS, 1, | ||
| 158 | + [Define to 1 if `major', `minor', and `makedev' | ||
| 159 | + are declared in <sysmacros.h>.])]) | ||
| 160 | fi | ||
| 161 | ])# AC_HEADER_MAJOR | ||
| 162 | |||
diff --git a/meta/recipes-devtools/autoconf/autoconf_2.69.bb b/meta/recipes-devtools/autoconf/autoconf_2.69.bb index aa1877a1fb..fd01585441 100644 --- a/meta/recipes-devtools/autoconf/autoconf_2.69.bb +++ b/meta/recipes-devtools/autoconf/autoconf_2.69.bb | |||
| @@ -14,6 +14,7 @@ SRC_URI += "file://check-automake-cross-warning.patch \ | |||
| 14 | file://autotest-automake-result-format.patch \ | 14 | file://autotest-automake-result-format.patch \ |
| 15 | file://add_musl_config.patch \ | 15 | file://add_musl_config.patch \ |
| 16 | file://performance.patch \ | 16 | file://performance.patch \ |
| 17 | file://AC_HEADER_MAJOR-port-to-glibc-2.25.patch \ | ||
| 17 | " | 18 | " |
| 18 | 19 | ||
| 19 | SRC_URI[md5sum] = "82d05e03b93e45f5a39b828dc9c6c29b" | 20 | SRC_URI[md5sum] = "82d05e03b93e45f5a39b828dc9c6c29b" |
