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.

Sign up for GitHub

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

Jump to bottom

Update cargo and other dependencies #403

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update cargo and other dependencies #403

wants to merge 3 commits into from
+1,379 −791

Conversation

MonterraByte
Copy link

@MonterraByte MonterraByte commented Oct 3, 2024 •
edited
Loading

Resolves #402. Resolves #401. Resolves #384. Resolves #404.

This MR updates all dependencies to their latest versions. The only update that required code changes was the cargo crate, which was caused by:

  • cargo::util::Config being renamed to cargo::util::context::GlobalContext.
  • cargo::sources::source::Source::query_vec() returning a Vec<IndexSummary> instead of a Vec<Summary>.

BarbossHack
BarbossHack approved these changes Oct 5, 2024
View reviewed changes
Cargo.toml Outdated Show resolved Hide resolved
@BarbossHack
Copy link

BarbossHack commented Oct 5, 2024 •
edited
Loading

Also Fixes #385

@MonterraByte
Copy link
Author

Also Fixes #385

That one is already fixed in master, as far as I can tell.

@kaimast
Copy link

kaimast commented Oct 17, 2024

Maybe bump directly to cargo 0.83?

@MonterraByte
Copy link
Author

Maybe bump directly to cargo 0.83?

That wasn't out when I made the PR. :)
I'll update it.

@kaimast
Copy link

kaimast commented Oct 17, 2024

Yeah, I figured it might not have been out yet.

I just tried the PR, and it works fine for me. Thanks for creating it! Hopefully, the maintainers can merge it soon.

@weihanglo weihanglo mentioned this pull request Oct 22, 2024
Closed
@MaxG87 MaxG87 mentioned this pull request Oct 23, 2024
Open
@zerosnacks
Copy link

zerosnacks commented Oct 29, 2024 •
edited
Loading

Ping @kbknapp, install currently throws - should be resolved by this PR:

error[E0282]: type annotations needed for `Box<_>`
  --> /home/<USER>/.cargo/registry/src/index.crates.io-6f17d22bba15001f/time-0.3.30/src/format_description/parse/mod.rs:83:9
   |
83 |     let items = format_items
   |         ^^^^^
...
86 |     Ok(items.into())
   |              ---- type must be known at this point
   |
   = note: this is an inference error on crate `time` caused by an API change in Rust 1.80.0; update `time` to version `>=0.3.35` by calling `cargo update

@frewsxcv frewsxcv mentioned this pull request Nov 2, 2024
Open
@kbknapp
Copy link
Owner

kbknapp commented Nov 28, 2024

Hi all. Sorry for the lack of communication. It's been a very crazy year for me. I'm open to adding maintainer to this project if anyone is willing as i think the other current maintainers are in a similar situation. Feel free to email me and I can review your account and/or other contributions. I'd plan on keeping this repo here, to try and keep an eye on new maintainers as best I can. However, if the Rust or Cargo project wished me to transfer to their control as has been discussed previously, I'd be open to that too.

@MonterraByte
chore: update Cargo.lock
83a737e 
This updates the `time` dependency, fixing a build error
with Rust >= 1.80.0.
@MonterraByte
chore: update cargo to v0.84.0
73f9c5f 
git2-curl is also updated to v0.20.0 to satisfy dependency constraints.
@MonterraByte
chore: update env_logger to v0.11.5 and strum to v0.26.3
52da5b1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BarbossHack BarbossHack BarbossHack approved these changes

@paolobarbolini paolobarbolini paolobarbolini approved these changes

@jaztec jaztec jaztec approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
7 participants
@MonterraByte @BarbossHack @kaimast @zerosnacks @kbknapp @jaztec @paolobarbolini