-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Add support for building under glibc 2.26 #13785
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM after fixing the nit.
src/pal/src/include/pal/context.h
Outdated
@@ -21,6 +21,8 @@ Module Name: | |||
#ifndef _PAL_CONTEXT_H_ | |||
#define _PAL_CONTEXT_H_ | |||
|
|||
#include "config.h" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A nit - this is not needed. The config.h is included in palinternal.h that needs to be always included before this header. We already use a couple of HAVE_xxxx macros in this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops. Fixed now.
glibc 2.26 renames a number of identifiers so they are reserved under POSIX. Specifically, `padding` becomes `__glibc_reserved1`. Add a configure test for it and use the appropriate field name. See https://sourceware.org/bugzilla/show_bug.cgi?id=21457 for more information. Resolves #13009
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
@dotnet-bot test OSX10.12 x64 Checked Build and Test Failure was regexdna which is known flaky. |
Thanks for the review and merging this! |
glibc 2.26 renames a number of identifiers so they are reserved under POSIX. Specifically, `padding` becomes `__glibc_reserved1`. Add a configure test for it and use the appropriate field name. See https://sourceware.org/bugzilla/show_bug.cgi?id=21457 for more information. Resolves #13009
glibc 2.26 renames a number of identifiers so they are reserved under POSIX. Specifically, `padding` becomes `__glibc_reserved1`. Add a configure test for it and use the appropriate field name. See https://sourceware.org/bugzilla/show_bug.cgi?id=21457 for more information. Resolves #13009
glibc 2.26 renames a number of identifiers so they are reserved under POSIX. Specifically,
padding
becomes__glibc_reserved1
. Add a configure test for it and use the appropriate field name.See https://sourceware.org/bugzilla/show_bug.cgi?id=21457 for more information.
Resolves #13009