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

Minimum version may need to be bumped (in documentation) #386

Closed
arifd opened this issue Nov 9, 2022 · 1 comment · Fixed by #397
Closed

Minimum version may need to be bumped (in documentation) #386

arifd opened this issue Nov 9, 2022 · 1 comment · Fixed by #397
Labels
S-bug Severity: bug

Comments

@arifd
Copy link
Contributor

arifd commented Nov 9, 2022

What crate(s) in this repo are involved in the problem?

console-subscriber

What is the issue?

with this in my Cargo.toml

tokio = { version = "=1.17.0", features = ["tracing"] }
console-subscriber = "=0.1.8"

and tokio_unstable in my RUSTFLAGS

Something refuses to build (even after cargo clean), with this error:

error[E0599]: no method named `unwrap` found for struct `tokio::task::JoinHandle` in the current scope
    --> /home/arif/.cargo/registry/src/github.com-1ecc6299db9ec823/console-subscriber-0.1.8/src/lib.rs:1066:64
     |
1066 |     return tokio::task::Builder::new().name(_name).spawn(task).unwrap();
     |                                                                ^^^^^^ method not found in `tokio::task::JoinHandle<T>`

For more information about this error, try `rustc --explain E0599`.
error: could not compile `console-subscriber` due to previous error

This was resolved when i bumped tokio to =1.21.2

This is what the documentation says:
image

How can the bug be reproduced?

.

Logs, error output, etc

.

Versions

.

Possible solution

.

Additional context

.

Would you like to work on fixing this bug?

maybe

@arifd arifd added the S-bug Severity: bug label Nov 9, 2022
@hawkw
Copy link
Member

hawkw commented Nov 9, 2022

Ah, yeah, that's because the console-subscriber crate uses the unstable task::Builder API to name the tasks it spawns, and that API changed slightly in Tokio 1.21.0 (Builder::spawn was made fallible): https://github.com/tokio-rs/tokio/blob/master/tokio/CHANGELOG.md#unstable

We should update console-subscriber's tokio dependency here:

tokio = { version = "^1.15", features = ["sync", "time", "macros", "tracing"] }
as well as the documentation.

Thanks for the report, good catch!

@hawkw hawkw closed this as completed in #397 Feb 6, 2023
hawkw pushed a commit that referenced this issue Feb 6, 2023
hawkw pushed a commit that referenced this issue Sep 29, 2023
hawkw added a commit that referenced this issue Sep 29, 2023
# Changelog

All notable changes to this project will be documented in this file.
This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## console-subscriber-v0.2.0 - (2023-09-29)

[0b0c1af](https://github.com/tokio-rs/console/commit/0b0c1aff18c3260d3a45a78f6c0d6f4206af1cbb)...[0b0c1af](https://github.com/tokio-rs/console/commit/0b0c1aff18c3260d3a45a78f6c0d6f4206af1cbb)

### <a id = "console-subscriber-v0.2.0-breaking"></a>Breaking Changes
- **Update Tonic and Prost dependencies ([#364](#364 ([f9b8e03](https://github.com/tokio-rs/console/commit/f9b8e03bd7ee1d0edb441c94a93a350d5b06ed3b))<br />This commit updates the public dependencies `prost` and `tonic` to
semver-incompatible versions (v0.11.0 and v0.8.0, respectively). This is
a breaking change for users who are integrating the `console-api` protos
with their own `tonic` servers or clients.
- **Update `tonic` to v0.10 and increase MSRV to 1.64 ([#464](#464 ([96e62c8](https://github.com/tokio-rs/console/commit/96e62c83ef959569bb062dc8fee98fa2b2461e8d))<br />This is a breaking change for users of `console-api` and
`console-subscriber`, as it changes the public `tonic` dependency to a
semver-incompatible version. This breaks compatibility with `tonic`
0.9.x and `prost` 0.11.x.

### Added

- [**breaking**](#console-subscriber-v0.2.0-breaking) Update Tonic and Prost dependencies ([#364](#364)) ([f9b8e03](f9b8e03))
- Add support for Unix domain sockets ([#388](#388)) ([a944dbc](a944dbc), closes [#296](#296))
- Add scheduled time per task ([#406](#406)) ([f280df9](f280df9))
- Add task scheduled times histogram ([#409](#409)) ([d92a399](d92a399))
- Update `tonic` to 0.9 ([#420](#420)) ([48af1ee](48af1ee))
- Update MSRV to Rust 1.60.0 ([b18ee47](b18ee47))
- Expose server parts ([#451](#451)) ([e51ac5a](e51ac5a))
- Add cfg `console_without_tokio_unstable` ([#446](#446)) ([7ed6673](7ed6673))
- Add warning for tasks that never yield ([#439](#439)) ([d05fa9e](d05fa9e))
- [**breaking**](#console-subscriber-v0.2.0-breaking) Update `tonic` to v0.10 and increase MSRV to 1.64 ([#464](#464)) ([96e62c8](96e62c8))

### Documented

- Fix unclosed code block ([#463](#463)) ([362bdbe](362bdbe))
- Update MSRV version docs to 1.64 ([#467](#467)) ([94a5a51](94a5a51))

### Fixed

- Fix build on tokio 1.21.0 ([#374](#374)) ([c34ac2d](c34ac2d))
- Fix off-by-one indexing for `callsites` ([#391](#391)) ([43891ab](43891ab))
- Bump minimum Tokio version ([#397](#397)) ([bbb8f25](bbb8f25), fixes [#386](#386))
- Fix self wakes count ([#430](#430)) ([d308935](d308935))
- Remove clock skew warning in `start_poll` ([#434](#434)) ([4a88b28](4a88b28))
- Do not report excessive polling ([#378](#378)) ([#440](#440)) ([8b483bf](8b483bf), closes [#378](#378))
- Correct retain logic ([#447](#447)) ([36ffc51](36ffc51))

Signed-off-by: Eliza Weisman <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-bug Severity: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants