Skip to content

[Swift Build] Wire up --jobs#9367

Merged
bkhouri merged 2 commits into
swiftlang:mainfrom
owenv:owenv/jobs
Dec 18, 2025
Merged

[Swift Build] Wire up --jobs#9367
bkhouri merged 2 commits into
swiftlang:mainfrom
owenv:owenv/jobs

Conversation

@owenv

@owenv owenv commented Nov 12, 2025

Copy link
Copy Markdown
Contributor

Adopt the new API on SWBBuildRequest

Depends on #9417
Fixes: #9321

@owenv

owenv commented Nov 12, 2025

Copy link
Copy Markdown
Contributor Author

Closes #9321

@owenv

owenv commented Nov 12, 2025

Copy link
Copy Markdown
Contributor Author

@swift-ci test

@owenv

owenv commented Nov 12, 2025

Copy link
Copy Markdown
Contributor Author

@swift-ci test

@bkhouri bkhouri left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I know it may be benign, but can we add an automated test against the makeBuildSystem that ensure the SWBBuildRequest has the correct value set vain the various buildParmeters.workers value?

@bkhouri

bkhouri commented Dec 5, 2025

Copy link
Copy Markdown
Contributor

@swift-ci test

@bkhouri
bkhouri marked this pull request as draft December 5, 2025 05:21
@bkhouri

bkhouri commented Dec 5, 2025

Copy link
Copy Markdown
Contributor

I know it may be benign, but can we add an automated test against the makeBuildSystem that ensure the SWBBuildRequest has the correct value set vain the various buildParmeters.workers value?

Test has been added.

@bkhouri bkhouri self-assigned this Dec 5, 2025
@bkhouri

bkhouri commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

@swift-ci test

@bkhouri
bkhouri marked this pull request as ready for review December 9, 2025 14:53
@bkhouri

bkhouri commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

@swift-ci test windows

@bkhouri

bkhouri commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

@swift-ci test

1 similar comment
@bkhouri

bkhouri commented Dec 18, 2025

Copy link
Copy Markdown
Contributor

@swift-ci test

@bkhouri
bkhouri enabled auto-merge (squash) December 18, 2025 04:21
@bkhouri
bkhouri merged commit af38fe0 into swiftlang:main Dec 18, 2025
82 of 83 checks passed
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.

[Parity] Pass the --jobs to SwiftBuild

3 participants