Skip to content

mach: Break the shackle that restricts cross-compilation from Windows#41748

Merged
yezhizhen merged 1 commit intoservo:mainfrom
yezhizhen:unshackle
Jan 8, 2026
Merged

mach: Break the shackle that restricts cross-compilation from Windows#41748
yezhizhen merged 1 commit intoservo:mainfrom
yezhizhen:unshackle

Conversation

@yezhizhen
Copy link
Copy Markdown
Member

@yezhizhen yezhizhen commented Jan 8, 2026

I never forget that day: @dependabot updates aws-lc-rs, and cross-compilation Servo from Windows fails. We were left with two options:

  1. Get a local clone of aws-lc-rs, and modify it in a hacky way
  2. Use Git Bash, but still has to set up MSYS2 etc.

Later, we've been restricted since #36070 to only use option 2 due to restrictions imposed here. But the system is innocent, and it is aws-lc-rs's responsibility to fix the rules. Still, @d-desiatkin and I believed that day would come, when we would return justice to the victim.

Freedom is better served late than never. It is now working normally since aws-lc-rs has fixed the rules.

.
Signed-off-by: Euclid Ye <[email protected]>
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jan 8, 2026
Copy link
Copy Markdown
Member

@sagudev sagudev left a comment

Choose a reason for hiding this comment

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

LGTM, but can you also add link to offending/workaround PR into this PR description (before merge so it will also get into commit).

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jan 8, 2026
@yezhizhen
Copy link
Copy Markdown
Member Author

LGTM, but can you also add link to offending/workaround PR into this PR description (before merge so it will also get into commit).

Updated.

@yezhizhen yezhizhen added this pull request to the merge queue Jan 8, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 8, 2026
Merged via the queue into servo:main with commit 7aa2dd2 Jan 8, 2026
36 checks passed
@yezhizhen yezhizhen deleted the unshackle branch January 8, 2026 08:29
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jan 8, 2026
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