Skip to content

Fix duplicate feature test macros#455

Merged
jrybar-rh merged 1 commit intopolkit-org:mainfrom
notpeelz:fix-duplicate-feature-test-macros
Aug 22, 2024
Merged

Fix duplicate feature test macros#455
jrybar-rh merged 1 commit intopolkit-org:mainfrom
notpeelz:fix-duplicate-feature-test-macros

Conversation

@notpeelz
Copy link
Contributor

I removed feature test macro definitions (i.e _GNU_SOURCE/_XOPEN_SOURCE) from polkitbackend because _GNU_SOURCE is already defined in the root meson.build.
Explicitly defining _XOPEN_SOURCE is redundant because _GNU_SOURCE implies _XOPEN_SOURCE 700 (according to the comments in glibc's features.h)

Depends on #454

@notpeelz notpeelz mentioned this pull request May 17, 2024
@notpeelz notpeelz force-pushed the fix-duplicate-feature-test-macros branch from 49a43a4 to e37250c Compare June 7, 2024 16:33
@notpeelz notpeelz force-pushed the fix-duplicate-feature-test-macros branch 3 times, most recently from cfd4bf3 to 418db17 Compare June 26, 2024 16:50
@notpeelz notpeelz marked this pull request as ready for review June 26, 2024 16:51
@notpeelz notpeelz force-pushed the fix-duplicate-feature-test-macros branch from 418db17 to 40e65bf Compare June 27, 2024 18:02
@notpeelz
Copy link
Contributor Author

@jrybar-rh Now that all the other PRs are merged, this should be ready for review.

@jrybar-rh jrybar-rh merged commit 82f4a62 into polkit-org:main Aug 22, 2024
@notpeelz notpeelz deleted the fix-duplicate-feature-test-macros branch August 22, 2024 21:05
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.

2 participants