Hey grahamc (he/him) -- I'm looking at ImpactedMaintainers::calculate, and I notice this expression:

        let mut cmd = nix.safely_evaluate_expr_cmd(
            checkout,
            include_str!("./maintainers.nix"),
            argstrs,
            &[path_file.path(), attr_file.path()],
        );

Why do you add the 2 tempfiles (path_file, attr_file) to safe_paths, which seemingly just adds them to NIX_PATH? Do they need to be there for whatever reason? I ask since maintainers.nix just imports them by absolute path, not <...>.

In reply to @winterqt:nixos.dev
Is the sandbox enabled on the Darwin builders?

In reply to @hexa:lossy.network
I don't believe there is a working sandbox on darwin

In reply to @winterqt:nixos.dev
Is there any way that I can run OfBorg locally just to debug a specific task? (calculating changed attributes)
Or should I just do the things it does to do that manually, idk.
The latter is probably simpler

In reply to @grahamc:nixos.org
I think “thelounge:” is only for triggering builds, not @ing maintainers. This is to encourage good commit message format, but still @ing relevant people even if the author didn’t do it quite right

In reply to @grahamc:nixos.org
I think “thelounge:” is only for triggering builds, not @ing maintainers. This is to encourage good commit message format, but still @ing relevant people even if the author didn’t do it quite right