2023-02-17 Nix team meeting minutes #33

[fricklerhandwerk]

Attendees: @edolstra @Ericson2314 @tomberek @roberth @fricklerhandwerk

Agenda

• Triage (30 min)
  • extended to 60 min, postponed the other topics in favor of working off the triage list.
• re-open auto-closed issues (10 min) (postponed)
• re-evaluate team goals based on 2022 community survey (15 min) (postponed)

Triage

• Triage and break up the global settings · Issue #5638 · NixOS/nix · GitHub
  • to discuss
• Make command infra less stateful and more regular by Ericson2314 · Pull Request #7750 · NixOS/nix · GitHub
  • to discuss
• nix-hash: support base-64 and SRI format by ShamrockLee · Pull Request #7690 · NixOS/nix · GitHub
  • to discuss if we want to add features to the old CLI
• New logger: JSON diff output by thomasjm · Pull Request #7760 · NixOS/nix · GitHub
  • to discuss
• Add pre-commit hook and CI check, excluding currently unformatted files by roberth · Pull Request #7745 · NixOS/nix · GitHub
  • to discuss how to find an approach to code formatting
• `nix flake check`: skip derivations for foreign systems by peterbecich · Pull Request #7759 · NixOS/nix · GitHub
  • to discuss, the sentiment is this is probably the wrong thing to do
• fix(build): Use nix from the buildir instead of the installed one by Et7f3 · Pull Request #7747 · NixOS/nix · GitHub
  • to discuss how to solve the underlying problem
• Upgrading nix suggest nix upgrade-nix by lemastero · Pull Request #7794 · NixOS/nix · GitHub
  • to discuss in order to clarify underlying issues
• API docs · Issue #7814 · NixOS/nix · GitHub
  • agreement that we want this
  • to discuss tooling and how to proceed
• Remove workarounds for broken GCC · Issue #7815 · NixOS/nix · GitHub
  • idea approved
• `nix add-derivation` · Issue #7828 · NixOS/nix · GitHub
  • idea approved
• null flake input · Issue #7807 · NixOS/nix · GitHub
  • to discuss, this is low priority
• `nix describe-stores --json` format is not extensible · Issue #7824 · NixOS/nix · GitHub
  • this is a blocker for stabilising nix describe-stores
  • agreement this should be fixed
• Guidelines for extensible JSON · Issue #7823 · NixOS/nix · GitHub
  • to discuss design
• Move `nix {show-config,doctor}` under subcommands · Issue #7672 · NixOS/nix · GitHub
  • related to stabilisation
  • to discuss design
• Can't add a file from stdin · Issue #7773 · NixOS/nix · GitHub
  • postponed, to be addressed together with other global issues related to stabilisation of new CLI
• Support GNU-style long options with `=` · Issue #7789 · NixOS/nix · GitHub
  • postponed, this is low priority

[peterbecich]

@fricklerhandwerk

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

• to discuss, the sentiment is this is probably the wrong thing to do

Would this be more likely to be accepted if:

• default behavior does not change
• --current-system-only flag provided to restrict nix flake check to current system type

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

Thanks