Skip to content

dependency upgrades#819

Merged
problame merged 13 commits intomasterfrom
problame/dependency-upgrades
Oct 18, 2024
Merged

dependency upgrades#819
problame merged 13 commits intomasterfrom
problame/dependency-upgrades

Conversation

@problame
Copy link
Member

@problame problame commented Sep 8, 2024

No description provided.

@problame problame force-pushed the problame/dependency-upgrades branch 4 times, most recently from 2701759 to c0565b5 Compare September 8, 2024 16:31
Go upgrade:
- Go 1.23 is current => use that for release builds
- Go 1.22 is less than one year old, it's desirable to support it.
- The [`Go Toolchains`](https://go.dev/doc/toolchain) stuff is available
  in both of these (would also be in Go 1.21). That is quite nice stuff,
  but required some changes to how we versions we use in CircleCI and
  the `release-docker` Makefile target.

Protobuf upgrade:
- Go to protobuf GH release website
- Download latest locally
- run `sha256sum`
- replace existing pinned hashes
- `make generate`

Deps upgrade:
- `go get -t -u all`
- repository moves aren't handled well automatically, fix manually
- repeat until no changes
@problame problame force-pushed the problame/dependency-upgrades branch from 3189801 to 699936c Compare September 8, 2024 21:57
@problame
Copy link
Member Author

problame commented Sep 8, 2024

I think work on this is done.
I'm deploying the build to my personal infrastructure to see if anything breaks.

This also fixes a deprecation warning.
math/rand.Read is deprecated in newer Go versions.

Also, it appears that crypto/rand is faster when used from multiple
goroutines: https://gist.github.com/problame/0699acd6f99db4163f26f0b8a61569f3
Doesn't matter much because everything happens inside Docker.
The binary packaging workflow has long since
been moved to this repo (I don't think the external
workflow work ever completed).
@problame problame force-pushed the problame/dependency-upgrades branch from 4cf0fe1 to 3df5e22 Compare September 8, 2024 23:19
@problame problame marked this pull request as ready for review October 18, 2024 12:54
@problame
Copy link
Member Author

Nothing broke, let's merge this.

@problame problame merged commit 904c151 into master Oct 18, 2024
@problame problame deleted the problame/dependency-upgrades branch October 18, 2024 13:00
lapo-luchini pushed a commit to lapo-luchini/zrepl that referenced this pull request Oct 22, 2024
…pgrades"

Revert all changes of Go code. I already done all of them long time ago.

This reverts commit 904c151, reversing
changes made to affe00a.
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.

1 participant