Skip to content

SR-3771 and SR-3751: libdispatch should not depend on external libkqueue #207

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
Feb 1, 2017

Conversation

dgrove-oss
Copy link
Contributor

Revert back to libkqueue version that has the fix for
SR-2941 but does not have the changes to remove the
autotools-based build system.

Revert back to libkqueue version that has the fix for
SR-2941 but does not have the changes to remove the
autotools-based build system.
@dgrove-oss
Copy link
Contributor Author

@swift-ci Please test

@dgrove-oss dgrove-oss changed the title SR-3771 and SR-3751: libdispatch dependency on libkqueue SR-3771 and SR-3751: libdispatch dependency on external libkqueue Jan 30, 2017
@dgrove-oss dgrove-oss changed the title SR-3771 and SR-3751: libdispatch dependency on external libkqueue SR-3771 and SR-3751: libdispatch should not depend on external libkqueue Jan 30, 2017
@dgrove-oss
Copy link
Contributor Author

@MadCoder would you mind merging this to master? I screwed up in #203 and moved libkqueue too far forward. I didn't notice (and neither did the CI testing) that because the latest libkqueue removed the autotools based build system it meant that we stopped building libkqueue from source when building libdispatch. On both my machine and the Swift CI machines, libkqueue is installed, so the build just defaults back to using the system libkqueue which is not what we want.

Thanks & sorry for the churn.

@MadCoder MadCoder merged commit 3767ac7 into swiftlang:master Feb 1, 2017
@dgrove-oss dgrove-oss deleted the SR-3771 branch February 1, 2017 18:44
das pushed a commit that referenced this pull request Feb 21, 2017
SR-3771 and SR-3751: libdispatch should not depend on external libkqueue

Signed-off-by: Daniel A. Steffen <dsteffen@apple.com>
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.

3 participants