2024-02-28 Nix team meeting minutes #129

thufschmitt · February 29, 2024, 7:20am

Triage

nix repl repl-flake behaves differently than :lf · Issue #8059 · NixOS/nix · GitHub

github.com/NixOS/nix

nix repl repl-flake behaves differently than :lf

opened 04:11PM - 17 Mar 23 UTC

sg-qwt

bug flakes repl

Not sure of this is a bug or desired behavior of the new experimental repl-flake…. With repl-flake enabled and entered my nixos flake, when I run `nix repl .#`, only 5 variables added. In my previous workflow with `:lf` inside repl, I can see more variables added. Things like input etc are missing by `nix repl .#` ``` > nix repl .# Welcome to Nix 2.13.3. Type :? for help. Loading installable 'git+file:///home/me/nixos#'... Added 5 variables. nix-repl> :lf .# Added 17 variables ```

(Towards) stable C bindings for libutil, libexpr by yorickvP · Pull Request #8699 · NixOS/nix · GitHub

@thufschmitt: Should we merge it as-it-is and iterate from master?
@roberth working on Rust bindings and finding things to change on the way. Can merge in a couple weeks once it’s clarified
Still assigned to @roberth

add a purpose statement to the front page by fricklerhandwerk · Pull Request #9872 · NixOS/nix · GitHub

Waiting for it to be reworded, dropped from the board

reword the introduction in the manual by fricklerhandwerk · Pull Request #9873 · NixOS/nix · GitHub

Same

Refactor generation deletion for better code reuse between CLI tools by MichaelOultram · Pull Request #9894 · NixOS/nix · GitHub

Fine on the principle, deferred to @ericson2314 for review

Add nixos-unstable based outputs, bdwgc 8.2.4, Nixpkgs lib tests, cleanup by roberth · Pull Request #9900 · NixOS/nix · GitHub

Add position information to `while evaluating the attribute` errors in the debugger by 9999years · Pull Request #9915 · NixOS/nix · GitHub

Assigned to @roberth

Tracy-based expr evaluation profiler by picnoir · Pull Request #9967 · NixOS/nix · GitHub

To discuss, see whether that’s worth pursuing

feat: tab support for indentation stripping by thecaralice · Pull Request #9971 · NixOS/nix · GitHub

Likely to break things. Wontfix

parser rewrite by pennae · Pull Request #9989 · NixOS/nix · GitHub

`ValidPathInfo` JSON format should use `null` not omit field by Ericson2314 · Pull Request #9995 · NixOS/nix · GitHub

github.com/NixOS/nix

Comment by thufschmitt to `ValidPathInfo` JSON format should use `null` not omit field

NixOS:master ← NixOS:json-empty-sigs

Discussed during the Nix maintainers meeting on 2024-02-26. Idea approved, but …: - Should be mentioned in the release notes - Should be reflected in the json guidelines policy <details> - @edolstra: Potentially breaking change, although probably not a big one - Also only happens in the new command-line - @thufschmitt: Do we care about having these `null` fields? - @roberth: Have a slightly different semantics, potentially interesting to distinguish - Idea approved - Should be mentioned in the release notes - Should be reflected in the json guidelines policy

Fix readfile from special files by thufschmitt · Pull Request #10019 · NixOS/nix · GitHub

Make `outputHashAlgo` accept `"nar"`, stay in sync by Ericson2314 · Pull Request #10021 · NixOS/nix · GitHub

Fix 0. in lexer by MrQubo · Pull Request #10025 · NixOS/nix · GitHub

Blocked on the parser rewrite
Assigned to @roberth

profile install: fail on conflicting name by bobvanderlinden · Pull Request #10065 · NixOS/nix · GitHub

Stabilise fetchTree by thufschmitt · Pull Request #10068 · NixOS/nix · GitHub

github.com/NixOS/nix

Comment by thufschmitt to Stabilise fetchTree

NixOS:master ← NixOS:stabilize-fetchtree

Discussed during the Nix maintainers meeting on 2024-02-26. Approved, but needs …to be more restrictive, and blocked on https://github.com/NixOS/nix/issues/10115 <details> - Stabilizing everything but git was too eager, we need to just stabilize the tarball one - Should it return a path value as the output path? - Lazy trees require that - We can also just keep `fetchTree` not lazy - Any reason not to return a path? - Will cause errors in some usages - Any complexity in making it return a path? - Hopefully a one-line change - Already the case on lazy-trees - @edolstra can backport it from lazy-trees - https://github.com/NixOS/nix/issues/10115

`install-multi-user.sh`: `_sudo`: add proxy variables to sudo by ramboman · Pull Request #10070 · NixOS/nix · GitHub

Merged
Would be nice to have it tested, but scarily complex