Skip to content

[GlibcModuleMap] Add sys/file.h to glibc modulemap #5131

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

Merged
merged 1 commit into from
Oct 10, 2016

Conversation

aciidgh
Copy link
Contributor

@aciidgh aciidgh commented Oct 5, 2016

Add sys/file.h in glibc modulemap to export flock() on Linux platforms
Ref: swiftlang/swift-package-manager#720

@ddunbar
Copy link
Contributor

ddunbar commented Oct 5, 2016

@dabrahams, @bitjammer who owns the Glibc overlay now?

@ddunbar
Copy link
Contributor

ddunbar commented Oct 5, 2016

@aciidb0mb3r do you know for sure this is present on FreeBSD and Cygwin?

@aciidgh
Copy link
Contributor Author

aciidgh commented Oct 5, 2016

I think so (from quick google searches before), but not completely sure

@dabrahams
Copy link
Contributor

@ddunbar: @moiseev is the one to ask about overlays; he knows all.

@moiseev
Copy link
Contributor

moiseev commented Oct 5, 2016

@kstaring might now about FreeBSD, and @tinysun212 did cygwin related work elsewhere.

@aciidgh
Copy link
Contributor Author

aciidgh commented Oct 7, 2016

Is it okay if I remove freebsd and cygwin and only add it for linux to get this in?

@tinysun212
Copy link
Contributor

tinysun212 commented Oct 7, 2016

Please don't remove for Cygwin, sys/file.h is needed for Package Manager.

Tagging @kstaring, @dcci and @landonf, they are working for FreeBSD in #4804.

@dcci
Copy link
Member

dcci commented Oct 9, 2016

sys/file.h is present on FreeBSD, probably since forever, and won't go away anytime soon.
http://fxr.watson.org/fxr/source/sys/file.h
So I think the change is safe, but an additional round of testing wouldn't hurt.

@aciidgh
Copy link
Contributor Author

aciidgh commented Oct 10, 2016

@swift-ci please smoke test

@aciidgh aciidgh merged commit 5d5334a into swiftlang:master Oct 10, 2016
@aciidgh aciidgh deleted the glibc-modulemap-file branch October 10, 2016 05:38
@tinysun212 tinysun212 mentioned this pull request Oct 10, 2016
1 task
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.

6 participants