Skip to content

[CMake] Include apinotes in StandaloneOverlay.cmake #21928

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
Jan 17, 2019

Conversation

jrose-apple
Copy link
Contributor

...because the Darwin overlay is responsible for installing them.

Context: Within Apple, overlays are sometimes built separately from the rest of the Swift project. This doesn't currently work with just the public repo (rdar://problem/47324189), but it could, and I broke it with my recent API notes cleanup. This should get things working again.

rdar://problem/47294352

...because the Darwin overlay is responsible for installing them.

Context: Within Apple, overlays are sometimes built separately from
the rest of the Swift project. This doesn't currently work with just
the public repo (rdar://problem/47324189), but it could, and I broke
it with my recent API notes cleanup. This should get things working
again.

rdar://problem/47294352
@jrose-apple
Copy link
Contributor Author

@swift-ci Please smoke test

@jrose-apple
Copy link
Contributor Author

(I did check that this works using the Apple-internal repository.)

@jrose-apple
Copy link
Contributor Author

Failure is an unrelated LLDB timeout…

@swift-ci Please smoke test macOS

Copy link
Contributor

@Rostepher Rostepher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@jrose-apple jrose-apple merged commit bb6c643 into swiftlang:master Jan 17, 2019
@jrose-apple jrose-apple deleted the slightly-less-standalone branch January 17, 2019 18:34
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