Skip to content

build: attempt to bundle files for the modules #243

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

compnerd
Copy link
Member

@compnerd compnerd commented Mar 3, 2025

This adds a helper to bundle files into the resource directory for the module.

This adds a helper to bundle files into the resource directory for the
module.
@compnerd
Copy link
Member Author

compnerd commented Mar 3, 2025

@swift-ci please test

@jakepetroules
Copy link
Collaborator

What about SWBApplePlatform, SWBUniversalPlatform, and SWBCore?

@compnerd
Copy link
Member Author

compnerd commented Mar 3, 2025

I wanted to get the basic files packaged up first. There was the question of whether the Specs directory needed to be preserved and I figured that we should do those in a follow up.

@owenv
Copy link
Collaborator

owenv commented Mar 4, 2025

fwiw preserving the Specs/ directory structure is not important, it's just a way of organizing the sources

@owenv
Copy link
Collaborator

owenv commented May 5, 2025

@compnerd would you mind if I picked these changes over to another PR? I've been testing them out with some local additions to begin bundling the rest of the specs and simplify the resource accessor codegen a bit.

@compnerd
Copy link
Member Author

compnerd commented May 5, 2025

@owenv nope, by all means! Sorry, I've not been able to push this forward, been trying to finish up the static linking support to get the early swift-driver enabled on Windows.

@owenv
Copy link
Collaborator

owenv commented May 5, 2025

No problem, thanks! it's been a good opportunity to get more familiar with the toolchain build

For reference: opened as #488

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