Skip to content

maturin: Add 1.9.6#5586

Merged
lazka merged 7 commits intomsys2:masterfrom
lazka:maturin-1.9.3
Oct 27, 2025
Merged

maturin: Add 1.9.6#5586
lazka merged 7 commits intomsys2:masterfrom
lazka:maturin-1.9.3

Conversation

@lazka
Copy link
Copy Markdown
Member

@lazka lazka commented Aug 22, 2025

Doesn't build, feel free to improve it. (I'd assume it needs rust ecosystem fixes, so not easy)

Disabling rusttls makes it fail in https://github.com/bytecodealliance/target-lexicon I think

@lazka lazka changed the title maturin: Update to 1.9.3 maturin: Add 1.9.3 Aug 22, 2025
@ognevny
Copy link
Copy Markdown
Collaborator

ognevny commented Aug 22, 2025

some low-level rust packages require changes to make work on cygwin. in most cases it's adding some cfg's (like #if #endif) or even link specific Cygwin API...

edit: compile error output literally tells to add some extra cfg conditions

@ognevny
Copy link
Copy Markdown
Collaborator

ognevny commented Aug 22, 2025

bytecodealliance/rustix#1410
bytecodealliance/rustix#1428

^ backport these + ensure that some deps are up-to-date, then try again...

@lazka
Copy link
Copy Markdown
Member Author

lazka commented Aug 22, 2025

thanks.

I also got these some time (this log is from testing with another package):

error: failed to run custom build command for `target-lexicon v0.13.2`

Caused by:
  process didn't exit successfully: `/tmp/build-via-sdist-umdsri5d/bcrypt-4.3.0/src/_bcrypt/target/release/build/target-lexicon-567ae6ae729fbcec/build-script-build` (exit status: 101)
  --- stderr

  thread 'main' panicked at /home/user/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/target-lexicon-0.13.2/build.rs:52:54:
  Invalid target name: 'x86_64-pc-cygwin'
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

@ognevny
Copy link
Copy Markdown
Collaborator

ognevny commented Aug 22, 2025

I've just looked at source. seems easy to fix

@ognevny
Copy link
Copy Markdown
Collaborator

ognevny commented Aug 24, 2025

that's how far I get. the problem is that getrandom v0.3.2 is required, but reverse dependency is abandoned so it's stuck at 0.2.x

@ognevny
Copy link
Copy Markdown
Collaborator

ognevny commented Aug 24, 2025

rustls is still compiled even when it's disabled

@lazka
Copy link
Copy Markdown
Member Author

lazka commented Aug 27, 2025

thanks, blocked on PyO3/maturin#1858 then I guess

@lazka lazka changed the title maturin: Add 1.9.3 maturin: Add 1.9.4 Aug 27, 2025
@lazka lazka added reported-upstream Issues reported or fixed in upstream repository and removed help-wanted labels Aug 31, 2025
disable all features except cli-completion to make it build
@lazka lazka marked this pull request as ready for review October 27, 2025 08:32
@lazka lazka changed the title maturin: Add 1.9.4 maturin: Add 1.9.6 Oct 27, 2025
@lazka lazka merged commit c0072e2 into msys2:master Oct 27, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reported-upstream Issues reported or fixed in upstream repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants