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

meson: define _GNU_SOURCE to detect copy_file_range() #7734

Merged
merged 1 commit into from Dec 25, 2017

Conversation

yuwata
Copy link
Member

@yuwata yuwata commented Dec 24, 2017

Follow-up for bad7a0c81f501fbbcc79af9eaa4b8254441c4a1f of git repository for glibc.

Recently glibc adds copy_file_range() but to use it, _GNU_SOURCE needs to be defined. This adds the flag in meson.build to detect the function by meson correctly.

@yuwata yuwata added the meson label Dec 24, 2017
Follow-up for bad7a0c81f501fbbcc79af9eaa4b8254441c4a1f of git
repository for glibc.

Recently glibc added `copy_file_range()`, but to use it,
`_GNU_SOURCE` needs to be defined. This adds the flag in
meson.build to detect the function by meson correctly.
@poettering
Copy link
Member

lgtm, though I wonder why we are not setting that unconditionally. I mean, all the new Linux-specific stuff will need this. @keszybz?

@poettering poettering added the good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed label Dec 24, 2017
@poettering
Copy link
Member

KVM: entry failed, hardware error 0x0

@poettering poettering merged commit 0e50bfa into systemd:master Dec 25, 2017
@yuwata yuwata deleted the copy_file_range branch December 26, 2017 00:31
floppym pushed a commit to gentoo/systemd that referenced this pull request Jan 13, 2018
Follow-up for bad7a0c81f501fbbcc79af9eaa4b8254441c4a1f of git
repository for glibc.

Recently glibc added `copy_file_range()`, but to use it,
`_GNU_SOURCE` needs to be defined. This adds the flag in
meson.build to detect the function by meson correctly.

(cherry picked from commit 0e50bfa)
SergioAtSUSE pushed a commit to SergioAtSUSE/systemd_systemd that referenced this pull request Jun 7, 2018
Follow-up for bad7a0c81f501fbbcc79af9eaa4b8254441c4a1f of git
repository for glibc.

Recently glibc added `copy_file_range()`, but to use it,
`_GNU_SOURCE` needs to be defined. This adds the flag in
meson.build to detect the function by meson correctly.

(cherry picked from commit 0e50bfa)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed meson
Development

Successfully merging this pull request may close these issues.

None yet

2 participants