nixUnstable: pre20210326_dd77f71 -> pre20210429_d15a196

[thufschmitt]

Motivation for this change

(amongst other things) several fixes to make the ca-derivations experimental feature usable on a daily basis

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.

[r-rmcgibbo]

Result of nixpkgs-review pr 121263 at 5ce13431 run on aarch64-linux 1

2 packages failed to build:

• dydisnix
• hydra-unstable

6 packages built successfully:

• disnix
• disnixos
• nix-direnv
• nix-update
• nixFlakes (nixUnstable)
• nixpkgs-review

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

---

Result of nixpkgs-review pr 121263 at 5ce13431 run on x86_64-linux 1

2 packages failed to build:

• dydisnix
• hydra-unstable

6 packages built successfully:

• disnix
• disnixos
• nix-direnv
• nix-update
• nixFlakes (nixUnstable)
• nixpkgs-review

2 suggestions:

• warning: missing-patch-comment

  Please add a comment on the line above, explaining the purpose of this patch.
  Near pkgs/tools/package-management/nix/default.nix:205:16:

      |
  205 |     patches = [(
      |                ^
  

• warning: missing-patch-comment

  Please add a comment on the line above, explaining the purpose of this patch.
  Near pkgs/tools/package-management/nix/default.nix:67:17:

     |
  67 |                 ./aws-sdk-cpp-TransferManager-ContentEncoding.patch
     |                 ^
  

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

[SuperSandro2000]

Not sure if we want to upgrade if the tests didn't fully run https://github.com/NixOS/nix/runs/2470165209

[thufschmitt]

Not sure if we want to upgrade if the tests didn't fully run https://github.com/NixOS/nix/runs/2470165209

That test is a know flaky test (see for example https://github.com/NixOS/nix/runs/2419910217 for another instance of it failing on master).

I’ve updated hydra too to catch-up with the Nix update. The dydisnix failure seems unrelated − the failure comes from libnixxml which doesn’t depend on nixUnstable

[Ma27]

@regnat OK, this shouldn't be merged as-is. nix-shell currently fails like this for me on nixpkgs on 78effd79f10:

→ ./result/bin/nix-shell -A hello                                                           [78effd79f10]
error: derivation '/nix/store/zipc1j00q8i30xaylg0fz64lf2ypx752-bash-interactive-4.4-p23.drv' does not have wanted outputs '*'

[Ma27]

According to git bisect the commit 255d145 is the problem.

[Ma27]

Opened NixOS/nix#4759 to fix this.

[Ma27]

Ok, seemed to be false alarm. You have to update your nix-daemon as well for getting this to work. New nix and 2.3 daemon appears to work, but not older(tm) unstable daemon & newer CLI. As this is still unstable software, this seems OK to me.

EDIT: the version number of the daemon protocol should be bumped retroactively to entirely get rid of this incompatibility.

[thufschmitt]

Thanks for catching that @Ma27. Let’s wait for NixOS/nix#4763 to be merged first then

[thufschmitt]

Just updated Nix with the fix mentionned above, should be good now!

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/tweag-nix-dev-update-11/12886/1