-
Notifications
You must be signed in to change notification settings - Fork 100
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
With latest nightly rust, builds failing due to changes in cargo output #398
Comments
I think rust-lang/cargo#13311 might be the cause |
Actually, more likely rust-lang/cargo#12914 |
It looks like these landed in rustc in seperate bumps: And I believe these were split across separate nightlies, so it is possible this will be resolved in the next nightly bump. |
This is also affecting |
That's my plan
…On Thu, Jan 18, 2024, 3:33 PM David Hewitt ***@***.***> wrote:
This is also affecting maturin, I guess we see if it resolves again
tomorrow then?
—
Reply to this email directly, view it on GitHub
<#398 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAAGBA6FZ5EX526D7LCGZTYPGBKBAVCNFSM6AAAAABCAFWZ7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJZGE3DGOJSGQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Ok we're good with the nightly that came out a few minutes ago. Declaring victory here. |
I'm currently seeing all builds failing on the latest nightly Rust with:
This error comes from: https://github.com/PyO3/setuptools-rust/blob/main/setuptools_rust/build.py#L301-L310
The cause appears to be that the format of the
package_id
variable changed. On stable,package_id
for me is:cryptography-rust 0.1.0 (path+file:///Users/alex_gaynor/projects/cryptography/src/rust)
, while on nightly, thepackage_id
ispath+file:///Users/alex_gaynor/projects/cryptography/src/rust#cryptography-rust@0.1.0
.However, the format of package IDs found in the cargo JSON build output is unchanged, which results in
_find_cargo_artifacts
no longer matching things.I'm not sure if this is a cargo bug or a setuptools-rust bug.
The text was updated successfully, but these errors were encountered: