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

an unknown tool name found in scoped lint: rustdoc #156

Closed
Oliboy50 opened this issue Jul 18, 2021 · 3 comments
Closed

an unknown tool name found in scoped lint: rustdoc #156

Oliboy50 opened this issue Jul 18, 2021 · 3 comments

Comments

@Oliboy50
Copy link

Oliboy50 commented Jul 18, 2021

Bug Report

Version

├── aws-sdk-cloudwatch v0.0.10-alpha (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   ├── aws-auth v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   ├── smithy-http v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   │   ├── smithy-types v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   ├── aws-endpoint v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   ├── aws-types v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   └── smithy-http v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   ├── aws-http v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   ├── aws-types v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   ├── smithy-http v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   ├── smithy-types v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   ├── aws-hyper v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   ├── aws-auth v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   ├── aws-endpoint v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   ├── aws-http v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   ├── aws-sig-auth v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   │   ├── aws-auth v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   │   ├── aws-types v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   │   ├── smithy-http v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   ├── protocol-test-helpers v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   ├── smithy-client v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   │   ├── protocol-test-helpers v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   │   ├── smithy-http v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   │   ├── smithy-http-tower v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   │   │   ├── smithy-http v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   │   ├── smithy-types v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   ├── smithy-http v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   ├── smithy-http-tower v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   │   ├── smithy-types v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   ├── aws-sig-auth v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   ├── aws-types v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   ├── smithy-http v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   ├── smithy-query v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
│   │   ├── smithy-types v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   ├── smithy-types v0.0.1 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57) (*)
│   └── smithy-xml v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)

Platform

docker run --rm -it ekidd/rust-musl-builder uname -a

Linux 9aabf45e6f7a 5.10.25-linuxkit #1 SMP Tue Mar 23 09:27:39 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Description

The following error happens when I try to simply compile a bit of code using docker run --rm -it ekidd/rust-musl-builder cargo build --release (the purpose of this docker image is to cross build from MacOS to AWS Lambda)

Compiling smithy-client v0.1.0 (https://github.com/awslabs/aws-sdk-rust?tag=v0.0.10-alpha#abe0ae57)
error[E0710]: an unknown tool name found in scoped lint: `rustdoc::all`
 --> /home/rust/.cargo/git/checkouts/aws-sdk-rust-be00ef1d3f696844/abe0ae5/sdk/smithy-client/src/lib.rs:9:5
  |
9 |     rustdoc::all,
  |     ^^^^^^^
error[E0710]: an unknown tool name found in scoped lint: `rustdoc::private_doc_tests`
  --> /home/rust/.cargo/git/checkouts/aws-sdk-rust-be00ef1d3f696844/abe0ae5/sdk/smithy-client/src/lib.rs:18:9
   |
18 | #[allow(rustdoc::private_doc_tests)]
   |         ^^^^^^^
error: aborting due to 2 previous errors
For more information about this error, try `rustc --explain E0710`.
error: could not compile `smithy-client`
To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: build failed

I think the issue here is that the "ekidd/rust-musl-builder" compiler misses some "rustdoc dependencies"... so I opened an issue on their GitHub repository... but, aws-sdk-rust is the first dependency I have an issue with (I was using rusoto before, and it worked well)... so I'm not sure who's wrong here.

Would it be possible to stop allowing rustdoc stuff in your codebase?
So that we don't have to add extra dependencies to compile our projects 😢

Note: Compiling works fine when we remove these 2 lines (I forked your repo to try it)

@rcoh
Copy link
Contributor

rcoh commented Jul 18, 2021

what Rust version is this? I believe this was added in 1.52.0.

@Oliboy50
Copy link
Author

this is Rust version 1.51.0 😮

@Oliboy50
Copy link
Author

indeed, this was added in 1.52.0 https://github.com/rust-lang/rust/blob/master/RELEASES.md#rustdoc-1

thank you, I'll just wait for emk/rust-musl-builder to release a compatible Rust 1.52 version :)

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

No branches or pull requests

2 participants