Skip to content

CI: apt-get update before apt-get install gnome-keyring#109

Merged
nuttycom merged 2 commits into
masterfrom
aphelionz/apt-update-before-install
Sep 4, 2025
Merged

CI: apt-get update before apt-get install gnome-keyring#109
nuttycom merged 2 commits into
masterfrom
aphelionz/apt-update-before-install

Conversation

@aphelionz
Copy link
Copy Markdown
Member

This PR is intended to fix the mirror+file 404s that started showing up in CI.

apt-get update seemed the simplest place to start. Open to other ideas!

This PR is intended to fix the mirror+file 404s that started [showing up in CI](https://github.com/zcash/zcash-test-vectors/actions/runs/17371436288/job/49407384424#step:3:94).

`apt-get update` seemed the simplest place to start. Open to other ideas!
@aphelionz aphelionz requested review from daira and nuttycom September 2, 2025 14:04
@aphelionz
Copy link
Copy Markdown
Member Author

What say you, @daira ?

Comment thread .github/workflows/test_vectors.yml Outdated
nuttycom
nuttycom previously approved these changes Sep 4, 2025
Copy link
Copy Markdown
Contributor

@nuttycom nuttycom left a comment

Choose a reason for hiding this comment

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

utACK with nonblocking suggestion

Co-authored-by: Kris Nuttycombe <kris@nutty.land>
@aphelionz
Copy link
Copy Markdown
Member Author

Whoops, accepting your suggestion dismissed your review @nuttycom, sorry about that. Re-requested.

Copy link
Copy Markdown
Contributor

@nuttycom nuttycom left a comment

Choose a reason for hiding this comment

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

utACK

@nuttycom nuttycom merged commit c98a8a0 into master Sep 4, 2025
3 checks passed
@nuttycom nuttycom deleted the aphelionz/apt-update-before-install branch September 4, 2025 19:27
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