Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

nix: use go1.22.4#63372

Merged
keegancsmith merged 1 commit into
mainfrom
k/go_1_22
Jun 20, 2024
Merged

nix: use go1.22.4#63372
keegancsmith merged 1 commit into
mainfrom
k/go_1_22

Conversation

@keegancsmith

Copy link
Copy Markdown
Member

Tired of seeing the go toolchain being easier to use than nix.

Test Plan: nix develop on linux amd64 and macbook arm64 followed by running "go test ./internal/search" working. Also confirming that "go env GOROOT" points into the nix store.

Tired of seeing the go toolchain being easier to use than nix.

Test Plan: nix develop on linux amd64 and macbook arm64 followed by
running "go test ./internal/search" working. Also confirming that "go
env GOROOT" points into the nix store.
@cla-bot cla-bot Bot added the cla-signed label Jun 20, 2024
@github-actions github-actions Bot added team/product-platform team/search-platform Issues owned by the search platform team labels Jun 20, 2024
@Strum355

Copy link
Copy Markdown
Contributor

The curse of the nixpkgs monorepo

@keegancsmith

Copy link
Copy Markdown
Member Author

@Strum355 when you build will it be cached by your cachix instance? If so do you do both amd64 linux and arm64 darwin? If not, is there a way I can start pushing to that cache?

@keegancsmith keegancsmith merged commit d42a99b into main Jun 20, 2024
@keegancsmith keegancsmith deleted the k/go_1_22 branch June 20, 2024 09:12
@Strum355

Copy link
Copy Markdown
Contributor

@Strum355 when you build will it be cached by your cachix instance? If so do you do both amd64 linux and arm64 darwin? If not, is there a way I can start pushing to that cache?

I generally do both yeah, its a manual process but its uncommon & easy enough.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed team/product-platform team/search-platform Issues owned by the search platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants