Navigation Menu

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

go_2-dev: change pname/version to use unstable format #108270

Merged
merged 1 commit into from Jan 3, 2021
Merged

go_2-dev: change pname/version to use unstable format #108270

merged 1 commit into from Jan 3, 2021

Conversation

zowoq
Copy link
Contributor

@zowoq zowoq commented Jan 2, 2021

otherwise this causes problems with stable go versions and nix-env

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

otherwise this causes problems with stable go versions and nix-env
Copy link
Member

@mdlayher mdlayher left a comment

Choose a reason for hiding this comment

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

The change seems reasonable but this appears not to bootstrap in the PR review environment (at least here):

$ nixpkgs-review pr 108270
$ git -c fetch.prune=false fetch --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0 pull/108270/head:refs/nixpkgs-review/1
remote: Enumerating objects: 6, done.
remote: Counting objects: 100% (6/6), done.
remote: Total 7 (delta 6), reused 6 (delta 6), pack-reused 1
Unpacking objects: 100% (7/7), 642 bytes | 321.00 KiB/s, done.
From ssh://github.com/NixOS/nixpkgs
 + a1e9a94785c...2e7ccd2c2c9 refs/pull/108270/head -> refs/nixpkgs-review/1  (forced update)
$ git worktree add /home/matt/.cache/nixpkgs-review/pr-108270/nixpkgs 712830d101d94ae29a2310d91b4d54c0132f5716
Preparing worktree (detached HEAD 712830d101d)
HEAD is now at 712830d101d Merge pull request #108310 from mdlayher/mdl-bump-corerad
$ git merge --no-commit 2e7ccd2c2c929b05172a33d685b8635211e33eee
Automatic merge went well; stopped before committing as requested
$ nix build --no-link --keep-going --option build-use-sandbox relaxed -f /home/matt/.cache/nixpkgs-review/pr-108270/build.nix
builder for '/nix/store/2yyzgpc9yiv0fkn1qx97naghv99bkh2g-go2-unstable-2020-12-08.drv' failed with exit code 1; last 10 log lines:
  ok          cmd/link/internal/loader        0.091s
  ok     cmd/link/internal/sym   0.083s
  ok     cmd/nm  2.760s
  ok     cmd/objdump     1.792s
  ok     cmd/oldlink/internal/sym        0.052s
  ok     cmd/pack        0.940s
  ok     cmd/trace       0.078s
  ok     cmd/vet 4.452s
  FAIL
  go tool dist: Failed: exit status 1
cannot build derivation '/nix/store/j6xd9pyc8i711h3yzv158w6fjs14ykrd-env.drv': 1 dependencies couldn't be built
[0 built (1 failed), 6 copied (379.7 MiB), 61.2 MiB DL]
error: build of '/nix/store/j6xd9pyc8i711h3yzv158w6fjs14ykrd-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/108270
1 package failed to build:
go_2-dev

@zowoq
Copy link
Contributor Author

zowoq commented Jan 3, 2021

ofborg doesn't seem to like it either, failed on x86_64-linux here and the original PR.

aarch64-linux on ofborg is fine, builds locally on x86_64-nixos.

@zowoq zowoq merged commit fe13e4f into NixOS:master Jan 3, 2021
@zowoq zowoq deleted the go2 branch January 3, 2021 20:49
@fishy
Copy link

fishy commented Jan 3, 2021

Thanks for doing this! After this is merged I'm still getting upgraded from go-1.15.6 to g-2:

$ nix-channel --update && nix-env --dry-run -u
unpacking channels...
(dry run; not doing anything)
upgrading 'go-1.15.6' to 'go-2'
these derivations will be built:
  /nix/store/gsx301g781gdc7fx7nnahkmvfsnlcqri-go-2.drv

Is there anything else I need to do (or maybe wait for some other release from nixpkgs-channels repo)?

@zowoq
Copy link
Contributor Author

zowoq commented Jan 3, 2021

Just need to wait for the channels to update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants