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: Fix the build in non-root sandboxes #144398

Merged
merged 1 commit into from
Feb 4, 2022
Merged

Conversation

thufschmitt
Copy link
Member

Motivation for this change

Workaround for golang/go#42525

(Also related to NixOS/nix#3245)

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all packages 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/)
  • 21.11 Release Notes (or backporting 21.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@SuperSandro2000
Copy link
Member

Maybe we want to throw that on staging?

@zowoq
Copy link
Contributor

zowoq commented Nov 13, 2021

Is this only an issue with go_1_16 or does it need to be applied to go_1_17 as well?

@thufschmitt
Copy link
Member Author

Is this only an issue with go_1_16 or does it need to be applied to go_1_17 as well?

I actually didn’t try it with other go versions, but I guess it would need to be ported here as well since the upstream bug is still open

@hhoeflin
Copy link
Contributor

hhoeflin commented Jan 9, 2022

Will this be merged? One of my builds is also affected by this. Thanks

@SuperSandro2000
Copy link
Member

The same needs to be applied to go 1.17 and we should target staging because this change is not very important and does not need to trigger so many rebuilds on master.

@gurgl
Copy link

gurgl commented Jan 27, 2022

Just chiming in to say im affacted by this too. Running nix on ubuntu 20.04, if that matters.

@zowoq zowoq marked this pull request as draft January 27, 2022 22:13
@zowoq zowoq changed the base branch from master to staging January 27, 2022 22:14
@ofborg ofborg bot requested review from Mic92 and c00w January 27, 2022 22:43
@zowoq
Copy link
Contributor

zowoq commented Jan 27, 2022

@ofborg build go_1_17

@zowoq
Copy link
Contributor

zowoq commented Feb 3, 2022

I added the patch to the go 1.18 PR in #157650 so it doesn't get overlooked.

I'll rebase this after the patch lands in staging and merge.

@zowoq
Copy link
Contributor

zowoq commented Feb 4, 2022

@jonringer Can I squeeze this into the new staging-next?

@jonringer
Copy link
Contributor

jonringer commented Feb 4, 2022

I just started this, 5 mins before you posted :(

@zowoq
Copy link
Contributor

zowoq commented Feb 4, 2022

I was waiting for the merge action to push the patch from master to staging when I saw you do it manually. 😆

This waited a couple of months until I picked it up, it can wait until the next cycle.

@zowoq zowoq marked this pull request as ready for review February 4, 2022 07:32
@zowoq zowoq merged commit a66d9c8 into NixOS:staging Feb 4, 2022
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.

8 participants