Skip to content

Conversation

ADD-SP
Copy link
Member

@ADD-SP ADD-SP commented Sep 29, 2025

See https://app.netlify.com/projects/tokio-rs/deploys/68d8b0ea0ad8820008da8f26, I opened a issue to the lock_api (Amanieu/parking_lot#494).

11:53:23 AM: error[E0557]: feature has been removed
11:53:23 AM:   --> /opt/buildhome/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/lock_api-0.4.13/src/lib.rs:89:29
11:53:23 AM:    |
11:53:23 AM: 89 | #![cfg_attr(docsrs, feature(doc_auto_cfg))]
11:53:23 AM:    |                             ^^^^^^^^^^^^ feature has been removed
11:53:23 AM:    |
11:53:23 AM:    = note: removed in 1.58.0; see <[https://github.com/rust-lang/rust/pull/138907&gt;](https://github.com/rust-lang/rust/pull/138907%3E) for more information
11:53:23 AM:    = note: merged into `doc_cfg`
11:53:23 AM: For more information about this error, try `rustc --explain E0557`.
11:53:23 AM: error: could not compile `lock_api` (lib) due to 1 previous error
11:53:23 AM: warning: build failed, waiting for other jobs to finish...

@ADD-SP ADD-SP added the A-ci Area: The continuous integration setup label Sep 29, 2025
# 89 | #![cfg_attr(docsrs, feature(doc_auto_cfg))]
# | ^^^^^^^^^^^^ feature has been removed
# |
# = note: removed in 1.58.0; see <https://github.com/rust-lang/rust/pull/138907; for more information
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed in one point fifty eight? That's ancient.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This version also surprised me, so I checked the PR in the error message, it was merged two days ago: rust-lang/rust@c0ee51f.

Copy link
Member

@taiki-e taiki-e Sep 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just a mistake in the error message and will be fixed in the next nightly: rust-lang/rust#147123

# = note: merged into `doc_cfg`
command = """
rustup install nightly --profile minimal && cargo doc --no-deps --all-features
rustup install nightly-2025-01-25 --profile minimal && cargo doc --no-deps --all-features
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this same version as what's used for docs builds?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the same version in ci.yml, there are also some cargo docs in this yml.

rust_nightly: nightly-2025-01-25

@ADD-SP ADD-SP merged commit 0248697 into tokio-rs:master Sep 29, 2025
93 checks passed
@ADD-SP ADD-SP deleted the add_sp/ci-freeze-nightly-rust branch September 29, 2025 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ci Area: The continuous integration setup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants