Skip to content

Markup: re-export libcmark_static #28998

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 4, 2020

Conversation

compnerd
Copy link
Member

@compnerd compnerd commented Jan 3, 2020

Unfortunately, Swift exports the library targets for LLDB. This
requires the transitive dependencies be exported. libcmark_swift was
not in the export set. This breaks the unified build.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@compnerd
Copy link
Member Author

compnerd commented Jan 3, 2020

CC: @xiaobai @JDevlieghere

@compnerd
Copy link
Member Author

compnerd commented Jan 3, 2020

@swift-ci please test Windows platform

@bulbazord
Copy link
Contributor

How does this break the unified build, exactly? Are you not building lldb in that unified setup?

@compnerd
Copy link
Member Author

compnerd commented Jan 3, 2020

@compnerd
Copy link
Member Author

compnerd commented Jan 3, 2020

I am building LLDB, but, thats not the issue, the issue is that swiftMarkup is being exported, but that is a static library, and that library has a dependency on libcmark_static which is not being exported.

CMark upstream now provides an exports target entry that we can use.
@compnerd compnerd force-pushed the export-restrictions branch from 86f6611 to 5f428f5 Compare January 3, 2020 22:41
@compnerd
Copy link
Member Author

compnerd commented Jan 3, 2020

Please test with following PRs:
swiftlang/swift-cmark#15

@swift-ci please test

@shahmishal shahmishal merged commit bc14367 into swiftlang:master Jan 4, 2020
@compnerd compnerd deleted the export-restrictions branch January 4, 2020 01:11
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