Skip to content
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

[cmake] Install libraries in standard directories #685

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Steelskin
Copy link

Previously, libraries were installed under lib/swift/${os}/. They should be installed in the default library directory for the relevant target system.
In addition, swiftmodules were installed in the older layout format on non-Darwin platforms. This changes to use the standard modern layout format for swiftmodules.

Checklist

  • I've added at least one test that validates that my change is working, if appropriate
  • I've followed the code style of the rest of the project
  • I've read the Contribution Guidelines
  • I've updated the documentation if necessary

Previously, libraries were installed under `lib/swift/${os}/`. They
should be installed in the default library directory for the relevant
target system.
In addition, swiftmodules were installed in the older layout format on
non-Darwin platforms. This changes to use the standard modern layout
format for swiftmodules.
@Steelskin
Copy link
Author

@compnerd Before I get started on other libraries, is this what you had in mind?

@compnerd compnerd requested a review from etcwilde November 20, 2024 23:26
@Steelskin Steelskin requested a review from compnerd November 20, 2024 23:57
@Steelskin Steelskin force-pushed the cmake-install-folders branch from 9ea1bc0 to 34f4919 Compare November 20, 2024 23:57
@compnerd
Copy link
Collaborator

@swift-ci please test

@compnerd
Copy link
Collaborator

This is going to require a cross-repository test to ensure that the toolchain builds don't break.

@Steelskin Steelskin requested a review from etcwilde November 21, 2024 01:50
cmake/modules/SwiftSupport.cmake Outdated Show resolved Hide resolved
cmake/modules/SwiftSupport.cmake Outdated Show resolved Hide resolved
@Steelskin Steelskin requested a review from etcwilde December 2, 2024 16:44
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