Skip to content
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

Manifest parse errors despite using latest version of Rust #355

Closed
ilyvion opened this issue Aug 1, 2022 · 11 comments · Fixed by #356
Closed

Manifest parse errors despite using latest version of Rust #355

ilyvion opened this issue Aug 1, 2022 · 11 comments · Fixed by #356

Comments

@ilyvion
Copy link

ilyvion commented Aug 1, 2022

❯ cargo build
    Finished dev [unoptimized + debuginfo] target(s) in 0.23s

❯ cargo geiger --version
cargo-geiger 0.11.3

❯ cargo geiger
error: failed to parse manifest at `C:\Path\To\Project\Cargo.toml`

Caused by:
  namespaced features with the `dep:` prefix are only allowed on the nightly channel and requires the `-Z namespaced-features` flag on the command-line

Which is odd, because namespaced features has been stable since 1.60 and none of the other cargo tools I have are choking on my Cargo.toml.

@pinkforest
Copy link
Collaborator

pinkforest commented Aug 1, 2022

Thanks for the report!

What does rustup show say and which is used by default just off curiosity ?

There is some logic going how the used rustc etc. gets selected. Manifest parsing is more involved.

We had ~this before: #181 so it may be related.

We are currently on latest cargo dependency which I think was 0.63

I will need to replicate and see what is going on.

@pinkforest
Copy link
Collaborator

@alexschrod Could you share your Cargo.toml please or the relevant bit re: what it is complaining is fine. Thanks!

@ilyvion
Copy link
Author

ilyvion commented Aug 1, 2022

❯ rustup show
Default host: x86_64-pc-windows-msvc
rustup home:  C:\Users\alexs\.rustup

installed toolchains
--------------------

stable-x86_64-pc-windows-msvc
nightly-2022-04-05-x86_64-pc-windows-msvc
nightly-2022-07-08-x86_64-pc-windows-msvc
nightly-x86_64-pc-windows-msvc
1.56-x86_64-pc-windows-msvc
1.60-x86_64-pc-windows-msvc
1.42.0-x86_64-pc-windows-msvc

installed targets for active toolchain
--------------------------------------

wasm32-unknown-unknown
x86_64-pc-windows-msvc

active toolchain
----------------

stable-x86_64-pc-windows-msvc (default)
rustc 1.62.1 (e092d0b6b 2022-07-16)

I'm using the stable, default toolchain as my default, and there are no overrides active in my project as far as I can tell:

❯ rustc --version
rustc 1.62.1 (e092d0b6b 2022-07-16)

❯ cargo --version
cargo 1.62.1 (a748cf5a3 2022-06-08)

You can clone the repo I'm getting this behavior on if that helps: https://github.com/alexschrod/nombytes

@pinkforest
Copy link
Collaborator

pinkforest commented Aug 1, 2022

EDIT: I should have realised straight off it was the cargo (as a library) dependency :D I had a brainfreeze 🤣

I have a that itchy feeling this is some rust target specific thing.. namedly windows

I have to investigate but I don't have windows machine so have to figure MS ISO or VM

It works fine for me on both rustup default stable and nightly in Ubuntu and docker

$ uname -av
Linux foobar-MS-7C77 5.11.0-46-generic #51~20.04.1-Ubuntu SMP Fri Jan 7 06:51:40 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

$ rustup show
Default host: x86_64-unknown-linux-gnu
rustup home: /home/foobar/.rustup

installed toolchains

stable-x86_64-unknown-linux-gnu (default)
nightly-2022-01-17-x86_64-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu
1.46-x86_64-unknown-linux-gnu
1.47-x86_64-unknown-linux-gnu
1.48-x86_64-unknown-linux-gnu
1.49-x86_64-unknown-linux-gnu
1.51-x86_64-unknown-linux-gnu
1.52-x86_64-unknown-linux-gnu
stage1
1.62.1-x86_64-unknown-linux-gnu

installed targets for active toolchain

wasm32-unknown-unknown
wasm32-wasi
x86_64-unknown-linux-gnu

active toolchain

stable-x86_64-unknown-linux-gnu (default)
rustc 1.61.0 (fe5b13d68 2022-05-18)

@ilyvion
Copy link
Author

ilyvion commented Aug 1, 2022

Oh, I just noticed something; not sure if it matters, but both on my Windows and my WSL2 Ubuntu installs, I'm getting

...
Compiling cargo v0.60.0
...

when doing cargo install cargo-geiger. Now 1.60 was when the feature was stabilized, so if cargo's version stays in line with that, then it still should've supported it, but it's not 0.63 in any case.

@pinkforest
Copy link
Collaborator

pinkforest commented Aug 1, 2022

Oh yeah 🤦‍♀️ ... I should have released our 0.63 dependency change

@pinkforest
Copy link
Collaborator

Did you pull the release from crates.io or main from github ? I should really push a release out ! thnaks for reminding 💜

@ilyvion
Copy link
Author

ilyvion commented Aug 1, 2022

I've just done cargo install cargo-geiger like the README says. 😅

@pinkforest
Copy link
Collaborator

yeah it pulls it from the crates.io.. this is my neglect of forgetting to release. I will do one now. Thank you so much 💜

@pinkforest
Copy link
Collaborator

I have a feeling we solved #181 as well 🥳

@pinkforest
Copy link
Collaborator

@alexschrod fyi the release is 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 a pull request may close this issue.

2 participants