This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Compiler warning in Modules/posixmodule.c
Type: compile error Stage: resolved
Components: Extension Modules Versions: Python 3.7
process
Status: closed Resolution: duplicate
Dependencies: Superseder: Include major(), minor(), makedev() from sysmacros
View: 31343
Assigned To: Nosy List: fweimer, louielu, serhiy.storchaka, vstinner
Priority: normal Keywords:

Created on 2017-04-07 09:56 by louielu, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 1024 closed louielu, 2017-04-07 10:02
Messages (3)
msg291260 - (view) Author: Louie Lu (louielu) * Date: 2017-04-07 09:56
Using gcc-6.3.1 20170306 on Linux 4.10.1, it gave the warning:

gcc -pthread -c -Wno-unused-result -Wsign-compare -g -Og -Wall -Wstrict-prototypes    -std=c99 -Wextra -Wno-unused-result -Wno-unused-parameter -Wno-missing-field-initializers -Werror=implicit-function-declaration   -I. -I./Include    -DPy_BUILD_CORE -o Python/pyctype.o Python/pyctype.c
:./Modules/posixmodule.c: In function ‘os_major_impl’:
./Modules/posixmodule.c:8584:13: warning: In the GNU C Library, "major" is defined
 by <sys/sysmacros.h>. For historical compatibility, it is
 currently defined by <sys/types.h> as well, but we plan to
 remove this soon. To use "major", include <sys/sysmacros.h>
 directly. If you did not intend to use a system-defined macro
 "major", you should undefine it after including <sys/types.h>.
     return major(device);
             ^~~~~~~~~~~~~                                                                                                                                                                                                                                                                                                                       
./Modules/posixmodule.c: In function ‘os_minor_impl’:
./Modules/posixmodule.c:8601:13: warning: In the GNU C Library, "minor" is defined
 by <sys/sysmacros.h>. For historical compatibility, it is
 currently defined by <sys/types.h> as well, but we plan to
 remove this soon. To use "minor", include <sys/sysmacros.h>
 directly. If you did not intend to use a system-defined macro
 "minor", you should undefine it after including <sys/types.h>.
     return minor(device);
             ^~~~~~~~~~~~~                                                                                                                                                                                                                                                                                                                       
./Modules/posixmodule.c: In function ‘os_makedev_impl’:
./Modules/posixmodule.c:8619:13: warning: In the GNU C Library, "makedev" is defined
 by <sys/sysmacros.h>. For historical compatibility, it is
 currently defined by <sys/types.h> as well, but we plan to
 remove this soon. To use "makedev", include <sys/sysmacros.h>
 directly. If you did not intend to use a system-defined macro
 "makedev", you should undefine it after including <sys/types.h>.
     return makedev(major, minor);
             ^~~~~~~~~~~~~~~~~~~~~          


The problem introduce in glibc 2.25, going to deprecate the definition of 'major', 'minor', and 'makedev' by  sys/types.h. And the autoconf didn't change the behavior of `AC_HEADER_MAJOR`, see:

https://lists.gnu.org/archive/html/autoconf/2016-08/msg00014.html


There is a workaround path for this in libvirt, which take from autoconf patch, see:

https://www.redhat.com/archives/libvir-list/2016-September/msg00459.html
msg292308 - (view) Author: Louie Lu (louielu) * Date: 2017-04-26 06:44
serhiy, haypo, what do you think about this warning's fixed?

Thanks!
msg304500 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-10-17 12:05
Sorry, Louie Lu, this already has been fixed in issue31343.
History
Date User Action Args
2022-04-11 14:58:45adminsetgithub: 74199
2017-10-17 12:05:57serhiy.storchakasetstatus: open -> closed
superseder: Include major(), minor(), makedev() from sysmacros
messages: + msg304500

resolution: duplicate
stage: resolved
2017-10-17 11:40:48fweimersetnosy: + fweimer
2017-04-26 06:44:56louielusetnosy: + vstinner, serhiy.storchaka
messages: + msg292308
2017-04-07 10:02:14louielusetpull_requests: + pull_request1184
2017-04-07 09:58:49louielusettitle: Compiling warning in Modules/posixmodule.c -> Compiler warning in Modules/posixmodule.c
2017-04-07 09:56:44louielucreate