2023-02-10 Nix team meeting minutes #31

[fricklerhandwerk]

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

Agenda

• Triage (30 min)
• Discussion (30 min)

Discussed in the Nix team meeting 2023-02-10:

Triage

• Label approved ideas to lead contributions · Issue #7663 · NixOS/nix · GitHub
  • idea approved
  • assigned to @fricklerhandwerk to implement
• Test everything before merging with merge queue + hercules-ci · Issue #7674 · NixOS/nix · GitHub
  • to discuss
• Make `initLibStore` a viable alternative by roberth · Pull Request #7732 · NixOS/nix · GitHub
  • idea had been approved previously
  • assigned to @edolstra
• Factor out `InstallableStorePath` to its own file, dedup by Ericson2314 · Pull Request #7744 · NixOS/nix · GitHub
  • assigned to @tomberek for review
• Keep splitting libcmd headers & files by Ericson2314 · Pull Request #7748 · NixOS/nix · GitHub
  • assigned to @edolstra
• disallow selecting shell prompt in code samples by fricklerhandwerk · Pull Request #7403 · NixOS/nix · GitHub
  • merged
• document multi-user install · Issue #929 · NixOS/nix · GitHub
  • closed as fixed
• The release process is not clear · Issue #3679 · NixOS/nix · GitHub
  • assigned to @roberth
• primops: make nature of foldl' strictness clearer by sternenseemann · Pull Request #7158 · NixOS/nix · GitHub
  • assigned to @fricklerhandwerk
• CLI stabilization effort · Issue #7701 · NixOS/nix · GitHub
  • decision:
    • work through one complete command at a time and write down the process
    • once finished and everyone is confident to reproduce that process on one’s own, do the rest of the work in parallel
• Ensure that `self.outPath == ./.` by roberth · Pull Request #7796 · NixOS/nix · GitHub
  • to discuss
• Use `std::set<StringContextElem>` not `PathSet` for string contexts by Ericson2314 · Pull Request #7710 · NixOS/nix · GitHub
  • assigned to @thufschmitt for review
• Add option --color by sivizius · Pull Request #7650 · NixOS/nix · GitHub
  • to discuss
• Test `nix build --json` return output paths in floating CA case by Ericson2314 · Pull Request #4589 · NixOS/nix · GitHub
  • assigned to @tomberek

Discussion

• Parse JSON using simdjson by yorickvP · Pull Request #7249 · NixOS/nix · GitHub

  • decision: closed
    • if simdjson gets better error messages, we will reconsider

• Deduplicate code between Hydra and Nix · Issue #1164 · NixOS/hydra · GitHub

  • no decision