Fix 'evaluation warning: ... buildGoModule: specify CGO_ENABLED with env.CGO_ENABLED instead.'

[onnimonni]

I'm using your module in one of my servers and I started to see following error recently:

evaluation warning: /nix/store/1dyb7shmrf9v3xy5g2y7sajl5hhcppjd-source/pkgs/infrastructure-agent.nix:36: buildGoModule: specify CGO_ENABLED with env.CGO_ENABLED instead.

This should fix it 👍

[DavSanchez]

Hi! Thanks a lot.

I saw the CI failed with a certain error. Trying it from a local macOS also shows it:

error: The `env` attribute set cannot contain any attributes passed to derivation. The following attributes are overlapping:
  - CGO_ENABLED: in `env`: "1"; in derivation arguments: 1

Reading on similar occurrences on other places seem to be back from a 2023 problem that should be solved. Moreover, the upstream packages on unstable already include this change and could be built without issues. The diff only shows a difference in versions and corresponding hashes 🤔

Not sure what could be the issue. I assume this worked on your side. What version of Nix did you use to build the packages?

• CI: 2.22.1
• On local: 2.24.11

I'm updating the inputs and the version of Nix used in CI on separate PRs, after this is done, could you rebase your changes so the package builds with the latest dependency versions?

[DavSanchez]

Done with #26 and #27. Could you please rebase your changes with the latest master?

[onnimonni]

Not sure what could be the issue. I assume this worked on your side. What version of Nix did you use to build the packages?

Well I got these errors with the latest Nix 2.26. I didn't actually even try them locally since I'm not sure how to do that (I'm Nix noob still).

I did the rebasing anyway so let's see 👍

[onnimonni]

I tested these changes by merging this PR into my own fork and using that as an input to my flake.

It worked just fine with nix versions 2.26 and 2.25.3. The error went away using this and my machine is still connected to New relic so I assume it works just fine 👍

EDIT: clarified my message.

[DavSanchez]

This indeed worked. Thanks! Merging 🙂