buildGoModule might produce different checksum depending on the platform (linux <> darwin)

[Mic92]

Describe the bug

Discovered in https://github.com/NixOS/nixpkgs/pull/129649/files#r666294752

To investigate this we would need someone with both Linux and macOS to provide the module derivations for both.

[zowoq]

diff -rq /nix/store/9d1rp6pmfaa1sxj5ni40jd5xwwzwdhfc-telegraf-1.19.1-go-modules /nix/store/izq65v2jbqjfb29289hcpkwfkwb8yfqc-telegraf-1.19.1-go-modules
Only in /nix/store/izq65v2jbqjfb29289hcpkwfkwb8yfqc-telegraf-1.19.1-go-modules/github.com/Microsoft: ApplicationInsights-Go
Only in /nix/store/9d1rp6pmfaa1sxj5ni40jd5xwwzwdhfc-telegraf-1.19.1-go-modules/github.com: microsoft~nix~case~hack~1

---

nixpkgs/pkgs/tools/misc/opentelemetry-collector/default.nix

Lines 18 to 20 in 3f92843

	vendorSha256 = if stdenv.isDarwin
	then "0anw3l6pq8yys2g2607ndhklb9m1i9krgjrw4wb99igavjzp3wpj"
	else "04h463d2d7g6wqp5mzkqlszwzdbq0pix6j7n2s9s80lwg7nh8k3h";

diff -rq /nix/store/x3mqwlqzrpgbnczsy8ca02ynnc1nsijv-opentelemetry-collector-0.26.0-go-modules /nix/store/naii9fxpfcr2kcj845g3b7hp9ghh6xbm-opentelemetry-collector-0.26.0-go-modules
Only in /nix/store/x3mqwlqzrpgbnczsy8ca02ynnc1nsijv-opentelemetry-collector-0.26.0-go-modules/github.com/Microsoft: ApplicationInsights-Go
Only in /nix/store/naii9fxpfcr2kcj845g3b7hp9ghh6xbm-opentelemetry-collector-0.26.0-go-modules/github.com: microsoft~nix~case~hack~1

[Mic92]

diff -rq /nix/store/9d1rp6pmfaa1sxj5ni40jd5xwwzwdhfc-telegraf-1.19.1-go-modules /nix/store/izq65v2jbqjfb29289hcpkwfkwb8yfqc-telegraf-1.19.1-go-modules
Only in /nix/store/izq65v2jbqjfb29289hcpkwfkwb8yfqc-telegraf-1.19.1-go-modules/github.com/Microsoft: ApplicationInsights-Go
Only in /nix/store/9d1rp6pmfaa1sxj5ni40jd5xwwzwdhfc-telegraf-1.19.1-go-modules/github.com: microsoft~nix~case~hack~1

nixpkgs/pkgs/tools/misc/opentelemetry-collector/default.nix

Lines 18 to 20 in 3f92843

	vendorSha256 = if stdenv.isDarwin
	then "0anw3l6pq8yys2g2607ndhklb9m1i9krgjrw4wb99igavjzp3wpj"
	else "04h463d2d7g6wqp5mzkqlszwzdbq0pix6j7n2s9s80lwg7nh8k3h";

diff -rq /nix/store/x3mqwlqzrpgbnczsy8ca02ynnc1nsijv-opentelemetry-collector-0.26.0-go-modules /nix/store/naii9fxpfcr2kcj845g3b7hp9ghh6xbm-opentelemetry-collector-0.26.0-go-modules
Only in /nix/store/x3mqwlqzrpgbnczsy8ca02ynnc1nsijv-opentelemetry-collector-0.26.0-go-modules/github.com/Microsoft: ApplicationInsights-Go
Only in /nix/store/naii9fxpfcr2kcj845g3b7hp9ghh6xbm-opentelemetry-collector-0.26.0-go-modules/github.com: microsoft~nix~case~hack~1

I don't really understand the diff. Are there different files in both?

[zowoq]

Seems to be a case-insensitive conflict with github.com/Microsoft and github.com/microsoft.

[Mic92]

Could we normalize this in our code for buildGoModule?

[zowoq]

I'd be a bit concerned that any normalization we do ourselves may end up causing more problems with other workarounds or the go 1.17 vendor changes, especially if we end up with 1.17 as default on linux and keeping 1.16 for darwin.

I think I've got a fix that just uses go itself, I'll open a PR in the next couple of days.

[zowoq]

Draft fix: #131452

[zowoq]

Should be resolved by #131452