Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: Use c11 rather than gnu11 and define vendor extensions separately #302

Merged
merged 2 commits into from Feb 10, 2021

Conversation

pwithnall
Copy link
Contributor

This makes the code a little more amenable to being built as a
subproject for projects like gnome-software, which use standard C.

See man 7 feature_test_macros for details of the vendor extension
macros.

Signed-off-by: Philip Withnall pwithnall@endlessos.org

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
This makes the code a little more amenable to being built as a
subproject for projects like gnome-software, which use standard C.

See `man 7 feature_test_macros` for details of the vendor extension
macros.

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
@ximion ximion merged commit b4dcf81 into ximion:master Feb 10, 2021
@ximion
Copy link
Owner

ximion commented Feb 10, 2021

I would probably just have enabled _GNU_SOURCE to be done with it, but it's probably indeed better to be a bit more restrictive.

@pwithnall pwithnall deleted the work-with-c11 branch September 20, 2021 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants