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

cargo-udeps fails due to compilation error of ahash crate #2686

Closed
rina23q opened this issue Feb 6, 2024 · 1 comment
Closed

cargo-udeps fails due to compilation error of ahash crate #2686

rina23q opened this issue Feb 6, 2024 · 1 comment
Labels
bug Something isn't working ci/cd Repository management and pipeline topics

Comments

@rina23q
Copy link
Member

rina23q commented Feb 6, 2024

Describe the bug
cargo-udeps starts failing. See this run.

   Compiling ahash v0.7.7
    Checking hex v0.4.3
error[E0635]: unknown feature `stdsimd`
  --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/ahash-0.7.7/src/lib.rs:33:42
   |
33 | #![cfg_attr(feature = "stdsimd", feature(stdsimd))]
   |                                          ^^^^^^^
For more information about this error, try `rustc --explain E0635`.
error: could not compile `ahash` (lib) due to previous error                                ^^^^^^^

This issue has already been captured in tkaitchuck/aHash#200, however, we don't have direct dependency on ahash. Actually chain dependency starting from rumqttd.

rumqttd 0.18.0 (upgrading to 0.19.0 has breaking change, meaning that we have to change our code)
-> metrics-exporter-prometheus 0.12.2: ^0.12.1 (latest 0.13.0)
-> indexmap 1.9.3: ^1 (latest 2.x)
-> hashbrown 0.12.3: ^0.12 (latest 0.14.3)
-> ahash 0.7.7: ^0.7.0 (latest 0.8.7, which has the fix)

Unfortunately, upgrading rumqttd to 0.19.0 won't help as it still has a dependency on metrics-exporter-prometheus ^0.12.1.

To Reproduce
Run cargo-udeps job.

Expected behavior
No compilation error.

Screenshots

Environment (please complete the following information):

  • OS [incl. version]
  • Hardware [incl. revision]
  • System-Architecture [e.g. result of "uname -a"]
  • thin-edge.io version [e.g. 0.1.0]

Additional context

@rina23q rina23q added the bug Something isn't working label Feb 6, 2024
@rina23q rina23q added the ci/cd Repository management and pipeline topics label Feb 6, 2024
@reubenmiller
Copy link
Contributor

Pinning to a specific nightly build has fixed the issue for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ci/cd Repository management and pipeline topics
Projects
None yet
Development

No branches or pull requests

2 participants