Skip to content

build: use siso instead of reclient#47534

Merged
jkleinsc merged 1 commit intomainfrom
sam/siso
Aug 22, 2025
Merged

build: use siso instead of reclient#47534
jkleinsc merged 1 commit intomainfrom
sam/siso

Conversation

@MarshallOfSound
Copy link
Copy Markdown
Member

@MarshallOfSound MarshallOfSound commented Jun 24, 2025 •
edited by release-clerk bot
Loading

Chromium is switching from Ninja to Siso: https://groups.google.com/a/chromium.org/g/build/c/J5UBTvrj9oo/m/qz4m7dDEBQAJ. This PR updates our builds to use Siso instead of reclient.

Notes: none

@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Jun 24, 2025
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Jul 1, 2025
@jkleinsc jkleinsc force-pushed the sam/siso branch 2 times, most recently from d03f923 to ece6e7c Compare July 2, 2025 18:59
@jkleinsc jkleinsc force-pushed the sam/siso branch 3 times, most recently from 8e0f5fc to 5de14fb Compare July 15, 2025 18:39
@jkleinsc jkleinsc force-pushed the sam/siso branch 2 times, most recently from 8db6bea to a6d9e2d Compare July 23, 2025 14:29
@jkleinsc jkleinsc force-pushed the sam/siso branch 3 times, most recently from dd0b72b to 52db97a Compare August 12, 2025 19:01
@jkleinsc jkleinsc changed the title [EXPERIMENTAL] build: use siso instead of reclient build: use siso instead of reclient Aug 12, 2025
@jkleinsc jkleinsc added the target/38-x-y PR should also be added to the "38-x-y" branch. label Aug 12, 2025
@jkleinsc jkleinsc marked this pull request as ready for review August 12, 2025 20:32
@jkleinsc jkleinsc requested a review from a team as a code owner August 12, 2025 20:32
@jkleinsc jkleinsc requested a review from a team as a code owner August 21, 2025 21:20
@jkleinsc jkleinsc merged commit a67aad0 into main Aug 22, 2025
146 of 149 checks passed
@jkleinsc jkleinsc deleted the sam/siso branch August 22, 2025 18:17
@release-clerk
Copy link
Copy Markdown

release-clerk bot commented Aug 22, 2025

No Release Notes

@trop trop bot mentioned this pull request Aug 22, 2025
Merged
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Aug 22, 2025

I have automatically backported this PR to "38-x-y", please check out #48154

@trop trop bot added in-flight/38-x-y and removed target/38-x-y PR should also be added to the "38-x-y" branch. labels Aug 22, 2025
@jkleinsc jkleinsc mentioned this pull request Aug 25, 2025
Merged
3 tasks
@trop trop bot added merged/38-x-y PR was merged to the "38-x-y" branch. and removed in-flight/38-x-y labels Aug 25, 2025
jkleinsc pushed a commit that referenced this pull request Aug 25, 2025
@MarshallOfSound @jkleinsc
build: use siso instead of reclient (#47534)
d184625
@jkleinsc jkleinsc mentioned this pull request Aug 29, 2025
Merged
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Aug 29, 2025

@jkleinsc has manually backported this PR to "37-x-y", please check out #48227

@jkleinsc jkleinsc mentioned this pull request Aug 29, 2025
Merged
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Aug 29, 2025

@jkleinsc has manually backported this PR to "36-x-y", please check out #48228

@trop trop bot added in-flight/36-x-y merged/36-x-y PR was merged to the "36-x-y" branch. merged/37-x-y PR was merged to the "37-x-y" branch. and removed needs-manual-bp/36-x-y in-flight/36-x-y in-flight/37-x-y labels Aug 29, 2025
dodolalorc pushed a commit to dodolalorc/electron that referenced this pull request Sep 11, 2025
@MarshallOfSound @dodolalorc
build: use siso instead of reclient (electron#47534)
481844d
@jkleinsc jkleinsc mentioned this pull request Sep 15, 2025
Merged
2 tasks
kigh-ota pushed a commit to kigh-ota/electron that referenced this pull request Sep 30, 2025
@MarshallOfSound @kigh-ota
build: use siso instead of reclient (electron#47534)
4c3bca8
@jkleinsc jkleinsc mentioned this pull request Oct 6, 2025
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jkleinsc jkleinsc jkleinsc approved these changes

@VerteDinde VerteDinde VerteDinde approved these changes

Assignees

No one assigned

Labels

merged/36-x-y PR was merged to the "36-x-y" branch. merged/37-x-y PR was merged to the "37-x-y" branch. merged/38-x-y PR was merged to the "38-x-y" branch. semver/none

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MarshallOfSound @jkleinsc @VerteDinde