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

CI error due to deprecated feature in dependency package ordered-multimap #538

Closed
mario-nt opened this issue Mar 15, 2024 · 3 comments
Closed
Assignees
Labels
Continuous Integration Workflows and Automation Dependencies Related to Dependencies
Milestone

Comments

@mario-nt
Copy link
Contributor

mario-nt commented Mar 15, 2024

We are getting this CI error due to the ordered-multimap package:

image

I think it has been fixed in the v0.7.0, but we are using the 0.6.0.

The dependency tree is the following:

Torrust Index -> config-rs v0.14.0 -> rust-ini v0.19.0 -> ordered-multimap 0.6.0.

The error is fixed in the v0.7.1 of ordered-multimap, but to get that newer version, the config-rs package needs to bump the rust-ini v0.19.0 to v0.20.0.

That dependency update is already present in the Cargo.toml file of the github repo for the config-rs package, but the crates.io package that we download in the project, still uses the v0.19.0.

I have already opened an issue in the config-rs project so they can update the crates.io package with the latest changes:
rust-cli/config-rs#548

@mario-nt mario-nt self-assigned this Mar 15, 2024
@mario-nt mario-nt added Dependencies Related to Dependencies Continuous Integration Workflows and Automation labels Mar 15, 2024
@mario-nt mario-nt added this to the v3.0.0 milestone Mar 15, 2024
@mario-nt
Copy link
Contributor Author

One of the contributors answered telling that dependency change will be out in the next release of config, so we need to wait until the next version.

@mario-nt mario-nt changed the title CI error due to deprecated feature in dependency package oredered-multimap CI error due to deprecated feature in dependency package ordered-multimap Mar 16, 2024
@josecelano
Copy link
Member

This issue would be solved if we migrate to Figment.

@josecelano
Copy link
Member

The config crate was removed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Continuous Integration Workflows and Automation Dependencies Related to Dependencies
Projects
Status: Done
Development

No branches or pull requests

2 participants