Skip to content

Upgrade nix, allow windows-sys >=0.59, <=0.61#35

Merged
passcod merged 3 commits intowatchexec:mainfrom
ColinFinck:upgrade-deps-2026-01
Jan 20, 2026
Merged

Upgrade nix, allow windows-sys >=0.59, <=0.61#35
passcod merged 3 commits intowatchexec:mainfrom
ColinFinck:upgrade-deps-2026-01

Conversation

@ColinFinck
Copy link
Contributor

This syntax is recommended by Microsoft in https://crates.io/crates/windows-sys/0.61.2.
It helps to eradicate duplicate windows-sys dependencies from the dependency tree of a complex project.

I'm particularly interested in windows-sys 0.61.x, because it consistently uses raw-dylib for importing APIs from Windows DLLs. This makes a huge difference when using non-MSVC toolchains to build Windows binaries, e.g. x86_64-pc-windows-gnullvm.

I also took the opportunity to fix a warning that occurs when compiling with newer Rust versions.

CC @passcod

@passcod
Copy link
Member

passcod commented Jan 19, 2026

Oh, interesting. I think because nix is also not exposed in the public API, the same thing could be done there, if you want to try that out.

@ColinFinck
Copy link
Contributor Author

@passcod: CI answered it for us (https://github.com/watchexec/clearscreen/actions/runs/21143630360/job/60830301599): The changes from nix 0.29.x to 0.30.x were fundamental enough that they also required code changes inside clearscreen.

My commit b535537 should pass CI now and ready to be merged.

@passcod passcod merged commit 9666a9a into watchexec:main Jan 20, 2026
6 checks passed
@passcod
Copy link
Member

passcod commented Jan 20, 2026

Thanks! Released as 4.0.3

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