Skip to content

Allow building swift-syntax as dynamic libraries using an environment variable #2867

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 5, 2024

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Oct 1, 2024

This allows us to build swift-syntax as dynamic libraries, which in turn allows us to build SourceKit-LSP using SwiftPM on Windows. Linking swift-syntax statically into sourcekit-lsp exceeds the maximum number of exported symbols on Windows.

@ahoppen ahoppen requested a review from bnbarham as a code owner October 1, 2024 15:42
@ahoppen
Copy link
Member Author

ahoppen commented Oct 1, 2024

@swift-ci Please test

Copy link
Contributor

@Matejkob Matejkob left a comment

Choose a reason for hiding this comment

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

A bit off-top: but perhaps it would be a good idea to document all the environment variables in one place? There are quite a few of them in this repository.

… variable

This allows us to build swift-syntax as dynamic libraries, which in turn allows us to build SourceKit-LSP using SwiftPM on Windows. Linking swift-syntax statically into sourcekit-lsp exceeds the maximum number of exported symbols on Windows.
@ahoppen
Copy link
Member Author

ahoppen commented Oct 2, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented Oct 2, 2024

@swift-ci Please test Windows

@ahoppen
Copy link
Member Author

ahoppen commented Oct 2, 2024

A bit off-top: but perhaps it would be a good idea to document all the environment variables in one place? There are quite a few of them in this repository.

We actually had an issue for this: #1922

Addressing it in #2870.

@ahoppen
Copy link
Member Author

ahoppen commented Oct 4, 2024

@swift-ci Please test macOS

1 similar comment
@ahoppen
Copy link
Member Author

ahoppen commented Oct 4, 2024

@swift-ci Please test macOS

@ahoppen ahoppen merged commit 2a731fb into swiftlang:main Oct 5, 2024
3 checks passed
@ahoppen ahoppen deleted the build-dylibs branch October 5, 2024 14: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.

4 participants