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

linuxHeaders cleanup #15347

Merged
merged 2 commits into from
May 14, 2016
Merged

linuxHeaders cleanup #15347

merged 2 commits into from
May 14, 2016

Conversation

joachifm
Copy link
Contributor

No rebuilds.

@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @thoughtpolice, @bluescreen303 and @edolstra to be potential reviewers

@joachifm joachifm changed the title Kernel headers cleanup linuxHeaders cleanup May 10, 2016
@dezgeg
Copy link
Contributor

dezgeg commented May 10, 2016

Building against anything other than the headers used to build libc
doesn't really make any sense.

Why not? It wouldn't be particularly nice to have a mass rebuild every couple of months in case we had a package which really depended on bleeding-edge headers.

@joachifm
Copy link
Contributor Author

I don't follow, can you point to an example to help me understand?

@joachifm
Copy link
Contributor Author

Oh, I think I see what you mean ... well, I'm not saying it should be impossible to override it, but it doesn't make sense in these specific cases, and, I'd think, most cases.

Building against anything other than the headers used to build libc
doesn't really make any sense.
Building against anything other than the headers used to build libc
doesn't really make any sense.

Also: this package is Linux specific, so mark it as such.
@joachifm joachifm merged commit 7702212 into NixOS:master May 14, 2016
@joachifm joachifm deleted the kernel-headers-cleanup branch May 15, 2016 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants