Skip to content

ci: fix failing udeps install due to rust nightly #4

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

Merged
merged 1 commit into from
Jun 30, 2023

Conversation

9oelM
Copy link
Owner

@9oelM 9oelM commented Jun 30, 2023

Problem: starting from nightly-2023-06-28, Rust nightly changed the name of a preview feature, and proc-macro2 is referring to that feature that does not exist anymore. More about this problem at: rust-lang/rust#113152

cargo-udeps is using an old version of proc-macro2 which is 1.0.59. The problem is fixed in versions higher than 1.0.59. But obviously we need cargo-udeps to have it updated. So it will take some time for them to work on this problem. I just submitted an issue to their repo as well: est31/cargo-udeps#180.

What we can do for now, to get the CI working so that we can just work, is to anchor the nightly version at nightly-2023-06-27 and wait for the fix, and revert to the latest version of nightly after the fix. I already tested with my own repo and confirmed it works with nightly-2023-06-27 and fails with versions after that:

@9oelM 9oelM self-assigned this Jun 30, 2023
@9oelM 9oelM force-pushed the ci/udeps-install-fix branch from 179e128 to cd18284 Compare June 30, 2023 03:05
@9oelM 9oelM changed the title ci: fix udeps install due to latest nightly version ci: fix failing udeps install due to rust nightly Jun 30, 2023
@9oelM 9oelM requested a review from xJonathanLEI June 30, 2023 03:13
Copy link
Contributor

@xJonathanLEI xJonathanLEI left a comment

Choose a reason for hiding this comment

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

LGTM thanks for the fix.

@xJonathanLEI xJonathanLEI merged commit f29bde5 into master Jun 30, 2023
@xJonathanLEI xJonathanLEI deleted the ci/udeps-install-fix branch June 30, 2023 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants