| 9 Jan 2022 |
Winter (she/her) | what is it? | 01:24:45 |
piegames | A Codeowners reimplementation without stupid limitations. Let me grab you the issue(s) | 01:25:57 |
Winter (she/her) | In reply to @winterqt:nixos.dev tbh i might dedicate time to implementing it, if i can get a grasp for ofborg’s codebase yeah this is not bad at all, other than the fact that a local instance apparently can’t be setup easily
i’ll take a shot at it :) | 01:28:10 |
piegames | In reply to @piegames:matrix.org A Codeowners reimplementation without stupid limitations. Let me grab you the issue(s) https://github.com/NixOS/nixpkgs/issues/143441 | 01:28:24 |
piegames | In reply to @winterqt:nixos.dev yeah this is not bad at all, other than the fact that a local instance apparently can’t be setup easily
i’ll take a shot at it :) IIRC andi- hacked something together that allows you to do this. But otherwise, it's really a "if it compiles, then deploy and pray" | 01:29:45 |
Winter (she/her) | that’s wonderful lmao | 01:30:06 |
Winter (she/her) | Should changes be based off of release? | 02:16:50 |
hexa | best to check existing PRs | 02:36:00 |
Winter (she/her) | yeah, I did and figured it was okay -- just wanted to double check thoug | 02:46:50 |
Winter (she/her) | * yeah, I did and figured it was okay -- just wanted to double check though | 02:46:52 |
cole-h | In reply to @winterqt:nixos.dev Should changes be based off of release? Yes indeed :) | 05:23:31 |
Winter (she/her) | Oh god I just noticed I forgot to squash #589, whoops. Will do that in the morning. | 05:25:02 |
Winter (she/her) | Also i believe technically its title/commit message is wrong, since both building and rust-analyzer were broken until I applied the changes, so I don’t think it’s really Darwin specific | 05:28:44 |
Winter (she/her) | * Also I believe technically its title/commit message is wrong, since both building and rust-analyzer were broken until I applied the changes, so I don’t think it’s really Darwin specific
Should I even bother changing it? Probably will just because I have to rebase anyways but 🤷♀️ | 05:35:41 |
Winter (she/her) | Is sandbox enabled on the Darwin builders? | 22:34:46 |
Winter (she/her) | * Is the sandbox enabled on the Darwin builders? | 22:34:49 |
Winter (she/her) | 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 <...>.
| 22:56:00 |
Winter (she/her) | Thanks! | 22:56:10 |
Winter (she/her) | Oh, never mind. | 22:57:04 |
Winter (she/her) | * Oh, nevermind. | 22:57:07 |
Winter (she/her) | I should've dug a bit more deeper -- restrict-eval only allows paths in NIX_PATH to be accessed. | 22:57:20 |
Winter (she/her) | That makes sense, sorry for the ping. | 22:57:25 |
hexa | In reply to @winterqt:nixos.dev Is the sandbox enabled on the Darwin builders? I don't believe there is a working sandbox on darwin | 23:50:14 |
| 10 Jan 2022 |
Winter (she/her) | In reply to @hexa:lossy.network I don't believe there is a working sandbox on darwin There definitely is (I have it enabled) | 00:10:18 |
Winter (she/her) | But it’s not without its breakages | 00:10:25 |
Winter (she/her) | 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 | 00:12:29 |
Winter (she/her) | (I know I asked a similar question earlier so I don’t know why I’m asking again, sorry.$ | 00:13:08 |
Winter (she/her) | * (I know I asked a similar question earlier so I don’t know why I’m asking again, sorry.) | 00:13:12 |
piegames | 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 Ah, I found it https://github.com/NixOS/ofborg/pull/562 (FWIW) | 00:16:46 |
Winter (she/her) | I’ll take a look, thanks. | 00:26:56 |