2023-05-12 Nix team meeting minutes #54

Attendees: @tomberek @thufschmitt @edolstra @ericson2314 @fricklerhandwerk @kranzes @roberth

Agenda

Triage

--keep-going does not keep going on various build failures · Issue #8321 · NixOS/nix · GitHub

  • idea approved

ci: Always run with sandbox, even on Darwin by yorickvP · Pull Request #8240 · NixOS/nix · GitHub

reword documentation on `builtins` by fricklerhandwerk · Pull Request #8314 · NixOS/nix · GitHub

  • merged

reword introduction to built-in functions by fricklerhandwerk · Pull Request #8319 · NixOS/nix · GitHub

  • merged

document `builtins.currentTime` by fricklerhandwerk · Pull Request #8318 · NixOS/nix · GitHub

  • approved, needs small fixup

Allow CLI to pass environment variables to daemon by dramforever · Pull Request #5890 · NixOS/nix · GitHub

  • pinged author if there is interest to continue with the alternative approach

add input scheme for forgejo by getchoo · Pull Request #7970 · NixOS/nix · GitHub

  • to discuss

Rename attribute selection to attribute lookup · Issue #8197 · NixOS/nix · GitHub

  • low priority

Reviews

Make more string values work as installables by Ericson2314 · Pull Request #7601 · NixOS/nix · GitHub

  • Is now ready, next on RFC 92 agenda
  • @tomberek will take a look again

Give `nix daemon` and `nix-store --serve` protocols separate serializers with version info by Ericson2314 · Pull Request #6223 · NixOS/nix · GitHub

  • to review collaboratively

Refactor `Store` hierarchy with a new `IndirectRootStore` interface by Ericson2314 · Pull Request #8243 · NixOS/nix · GitHub

Document user files of nix by balsoft · Pull Request #8141 · NixOS/nix · GitHub

  • approved, may need a minor fixup before merging

Notes

  • @thufschmitt: should we put the manual into ownership of the docs team? we can’t address many PRs in a timely manner, and while we care about contents and semantics, many changes are also about structure
  • @ericson2314: in favor, helps with parallelism and we have a docs team for a reason.
  • @edolstra: have opinions on both structure and contents, we should still look at changes
  • @fricklerhandwerk: yes, we should continue running non-trivial changes through the team so we get the subtleties right
1 Like

The PR shows that @thufschmitt is assigned, who also reviewed it, is this a mistake somewhere?

Possibly, not sure any more.