2023-03-17 Nix team meeting minutes #41

[fricklerhandwerk]

Attendees: @thufschmitt @Ericson2314 @roberth @edolstra @tomberek

Triage

• nix-push was removed without a stable replacement · Issue #8011 · NixOS/nix · GitHub

  • to discuss

• Nix build issue · Issue #7809 · NixOS/nix · GitHub

  • assigned to @ericson2314

• nix: Add support for --keep-going and --keep-failed by bgamari · Pull Request #2106 · NixOS/nix · GitHub

  • closed
    • already fixed

• Cache flake-registry by centromere · Pull Request #6750 · NixOS/nix · GitHub

  • not sure what the tradeoffs are
  • provisionally assigned to @garbas as the de-facto maintainer of the Docker infra

• add path to 'very large path' error by bburdette · Pull Request #4608 · NixOS/nix · GitHub

  • assigned to @ericson2314 to assess whether that’s still relevant

• Warn about untracked files in git repositories by rti · Pull Request #7244 · NixOS/nix · GitHub

  • the approach is too eager, we should only warn on access
  • postponed

• `#! nix-shell -i bash` use bash from shell.nix by Radvendii · Pull Request #6472 · NixOS/nix · GitHub

  • waiting for some tests to be added
  • postponed

• ssh-ng: Set log-fd for ssh to `4` by default by Ma27 · Pull Request #7659 · NixOS/nix · GitHub

  • assigned to @edolstra for another review, asked for by @ericson2314

• log all ifd by cleverca22 · Pull Request #3494 · NixOS/nix · GitHub

  • might be too verbose, but since it’s hidden behind a flag, should be OK
  • idea approved

Discussion

• nix-push was removed without a stable replacement · Issue #8011 · NixOS/nix · GitHub

  • in scope of CLI stabilisation

• Hide experimental settings by Ericson2314 · Pull Request #7609 · NixOS/nix · GitHub

  • annoying because of the churn coming with it
  • but also very useful as long as we have experimental features.
  • idea approved, assigned to @roberth

• Add pre-commit hook and CI check, excluding currently unformatted files by roberth · Pull Request #7745 · NixOS/nix · GitHub

  • idea approved, as an experiment

• `nix flake check`: skip derivations for foreign systems by peterbecich · Pull Request #7759 · NixOS/nix · GitHub

  • default behavior should stay the same
  • no conclusion

[garbas]

@fricklerhandwerk

Cache flake-registry by centromere · Pull Request #6750 · NixOS/nix · GitHub

• not sure what the tradeoffs are
• provisionally assigned to @garbas as the de-facto maintainer of the Docker infra

I’ve resolved the PR and merged it thank you for ping.

but … I don’t know where my life went wrong by I guess I’m “de-facto maintainer of the Docker infra”

[peterbecich]

@fricklerhandwerk

nix flake check: skip derivations for foreign systems by peterbecich · Pull Request #7759 · NixOS/nix · GitHub

• default behavior should stay the same
• no conclusion

I have changed the PR. The default behavior is now unchanged, please review again:

https://github.com/NixOS/nix/pull/7759#issuecomment-1484198541