2023-03-27 Nix team meeting minutes #44

[fricklerhandwerk]

Attendees: @edolstra @fricklerhandwerk @thufschmitt @roberth @Ericson2314

Agenda

• shrink to discussion column
• work off the remaining discussion items

Clean up Discussion column

• Rename 'nix shell' · Issue #4715 · NixOS/nix · GitHub
  • postponed, will discuss in context of stabilisation when it’s time
• review process: all PRs must be triaged by fricklerhandwerk · Pull Request #7579 · NixOS/nix · GitHub
  • should constrain to ‘non-trivial’ PRs
• adr: 0001-condense-information-around-nix-3 by kamadorueda · Pull Request #6232 · NixOS/nix · GitHub
  • postponed
• https://github.com/NixOS/nix/pull/7156
  • agreement: should approach this differently
    • closed
• feat: add pre-http-request-hooks config entry and handle it by olebedev · Pull Request #7391 · NixOS/nix · GitHub
  • decide how to deal with this issue
• Deduplicate code between Hydra and Nix · Issue #1164 · NixOS/hydra · GitHub
  • Nix team doesn’t have capacity to deal with this globally
    • there should be a dedicated maintainer team for Hydra
    • as a first step, @fricklerhandwerk will help with getting current maintainers to answer a few questions on priorities
• Trustless remote building for input-addressed drvs by Ericson2314 · Pull Request #3921 · NixOS/nix · GitHub
  • @ericson2314: PR is not ready, but the underlying issue is worth discussing
  • @edolstra: making remote builds work is one the most tricky things for users
    • adding another flag to potentially get wrong is probably not the right way
  • underlying issue distributed builds require a trusted remote user · Issue #2789 · NixOS/nix · GitHub therefore moved to triage section
  • removed the PR from the board
• Triage and break up the global settings · Issue #5638 · NixOS/nix · GitHub
  • assigned to @roberth

Discussion

• Allow system certs access to fixed-output derivations by layus · Pull Request #7312 · NixOS/nix · GitHub
  • idea approved, assigned to @edolstra for review
• Add option --color by sivizius · Pull Request #7650 · NixOS/nix · GitHub
  • assigned to @Ericson2314
• New logger: JSON diff output by thomasjm · Pull Request #7760 · NixOS/nix · GitHub
  • assigned to @roberth without commitment
• `nix flake check`: skip derivations for foreign systems by peterbecich · Pull Request #7759 · NixOS/nix · GitHub
  • agreement: make --current-system-only the default and make --all-systems work exactly as in nix flake show
  • assigned to @thufschmitt
• Reorganise include files by andreabedini · Pull Request #7847 · NixOS/nix · GitHub
  • postponed
• Reorganize this repo · Issue #7876 · NixOS/nix · GitHub
  • agreement: move the tests as a first step, discuss the rest later
• Test everything before merging with merge queue + hercules-ci · Issue #7674 · NixOS/nix · GitHub
  • idea approved, assigned to @edolstra to set it up with help of @roberth
    • will experiment it for ~1 month and re-evaluate
• distributed builds require a trusted remote user · Issue #2789 · NixOS/nix · GitHub
  • agreement: general idea approved, but implementation approach needs more deliberation
    • assigned to @ericson2314