-
Notifications
You must be signed in to change notification settings - Fork 13.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build dist for x86_64-pc-solaris and sparcv9-sun-solaris #138699
base: master
Are you sure you want to change the base?
Conversation
r? @marcoieni rustbot has assigned @marcoieni. Use |
rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead. cc @rust-lang/rust-analyzer These commits modify the If this was unintentional then you should revert the changes before this PR is merged. |
@@ -33,6 +33,7 @@ inherits = "release" | |||
debug = 2 | |||
|
|||
[patch.'crates-io'] | |||
nix = { git = "https://github.com/psumbera/nix.git", branch = "mypatch-v28" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not going to happen, this will break our crates-io releases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is waiting for new nix
version the only option here? Last nix
release is from May 2024..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately I think that's the case, AFAIK neither r-a nor the root manifest here can use cargo patch functionality.
For the root manifest here, I believe we can't be pointing to non-rust-lang/*
forks via git dependencies too for supply-chain security reasons, cc @pietroalbini.
This comment has been minimized.
This comment has been minimized.
[patch.crates-io] | ||
nix = { git = "https://github.com/psumbera/nix.git", branch = "mypatch" } | ||
libffi-sys = { git = "https://github.com/psumbera/libffi-rs.git", branch = "sparcv9" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These should be fixed upstream, we really cannot be repointing these deps to the non-upstream forks.
3b9fbda
to
0deec39
Compare
The job Click to see the possible cause of the failure (guessed by this bot)
|
(That being said, thank you for investigating fixes for the Solaris CI) |
FIx for #126025
Note that this needs Nix with following commit:
nix-rust/nix@0f45593
Unfortunatelly there is no new release.
And for sparcv9 it needs also:
tov/libffi-rs#103
For now it therefore uses cargo patch infrastructure for these.