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

v1.16: pin ahash version #34645

Closed
wants to merge 1 commit into from
Closed

v1.16: pin ahash version #34645

wants to merge 1 commit into from

Conversation

yihau
Copy link
Member

@yihau yihau commented Jan 4, 2024

Problem

ci error on v1.16: https://buildkite.com/solana-labs/solana/builds/106239#018cd29e-8257-4530-9a36-ac08d70f88b3

Summary of Changes

pin ahash to 0.8.4.

(I thought I can only pin it to 0.8.3 but I failed 😢: https://buildkite.com/solana-labs/solana/builds/106258#018cd2c6-0e42-4cc8-8d43-94b1516e1d26. I haven't delved into it too deeply. If we feel pinning ahash 0.8.4 is too aggressive, I can spend more time on it.)

@yihau yihau marked this pull request as ready for review January 4, 2024 05:36
@yihau yihau added the automerge Merge this Pull Request automatically once CI passes label Jan 4, 2024
@CriesofCarrots CriesofCarrots removed the automerge Merge this Pull Request automatically once CI passes label Jan 4, 2024
Copy link
Contributor

@CriesofCarrots CriesofCarrots left a comment

Choose a reason for hiding this comment

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

It looks like they've yanked everything before v0.8.4 back to v0.4.1 😞
I don't love how many things in the Cargo.lock this touches, but I don't have a better suggestion to fix the build. I'm approving, but give Will or Trent a chance to weigh in before merging.

@t-nelson
Copy link
Contributor

t-nelson commented Jan 4, 2024

i would like to abandon this dependency entirely

tkaitchuck/aHash@b583310#r136011618


i think this worked locally...

#34650

@yihau yihau closed this Jan 5, 2024
@yihau yihau deleted the pin-ahash branch January 5, 2024 02:50
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.

3 participants