Skip to content

Conversation

@mati865
Copy link
Collaborator

@mati865 mati865 commented Jul 15, 2023

One less downstream patch to maintain (once the fix reaches the release....).

Upstream PR: rust-lang/rust#113725

@mati865 mati865 force-pushed the simplify-rust-packaging branch 2 times, most recently from a768a47 to 3b19c7c Compare August 2, 2023 18:44
@mati865 mati865 force-pushed the simplify-rust-packaging branch from 3b19c7c to 706daae Compare August 2, 2023 22:31
@mati865 mati865 changed the title Disable mingw packaging via configuration option Rust: disable mingw packaging via configuration option Aug 2, 2023
@mati865 mati865 marked this pull request as ready for review August 2, 2023 22:31
@lazka lazka merged commit 21e86e4 into msys2:master Aug 4, 2023
@lazka
Copy link
Member

lazka commented Aug 4, 2023

thanks!

@mati865 mati865 deleted the simplify-rust-packaging branch August 4, 2023 16:36
@lazka
Copy link
Member

lazka commented Aug 4, 2023

Looks like this broke things (in staging): https://github.com/msys2/msys2-tests/actions/runs/5768428800

Should I revert?

lazka added a commit that referenced this pull request Aug 5, 2023
This reverts commit 706daae.

This results in a broken rust (or some other change since the last rebuild):

error: linking with `x86_64-w64-mingw32-clang` failed: exit code: 1
note: x86_64-w64-mingw32-clang: error: no such file or directory: 'crt2.o'

See #17777 (comment)
@lazka
Copy link
Member

lazka commented Aug 5, 2023

Reverted via 1717c9e for now

@mati865
Copy link
Collaborator Author

mati865 commented Aug 5, 2023

Sorry, looks like we cannot get rid of 0008-disable-self-contained.patch.
I must have tested intermediate version with rm -f dest-rust/${MINGW_PREFIX}/lib/rustlib/$OSTYPE/lib/self-contained/* commented out.

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.

2 participants