| 19 Aug 2025 |
raitobezarius | WeetHet https://git.lix.systems/lix-project/lix/issues/966#issuecomment-14125 sadness | 20:59:19 |
raitobezarius | also darwin CI still not my friend | 21:00:10 |
raitobezarius | https://gerrit.lix.systems/c/lix/+/4004 | 21:00:14 |
jade_ | https://bytes.zone/posts/tree-grepper/ wow this is cool. sorta off topic. i am going to probably write a nixpkgs package. | 21:19:13 |
piegames | Woow | 21:40:04 |
piegames | Need | 21:40:05 |
emily | we have https://github.com/ast-grep/ast-grep iirc | 21:41:13 |
emily | I believe people have used it for Nixpkgs work | 21:41:19 |
emily | probably more Production Ready™ | 21:41:47 |
jade_ | it, very frustratingly, does not take tree sitter queries in s-expr form | 21:49:10 |
jade_ | i have tried it before, it has a bunch of opinions which annoy me a varying amount depending on the day | 21:49:34 |
jade_ | mostly just that repos should be configured for it (which is totally fair and defensible, but someone might whine about it in nixpkgs) | 22:00:14 |
jade_ | https://github.com/NixOS/nixpkgs/pull/435115 have! (with review request attached :P) | 22:27:22 |
| 20 Aug 2025 |
| Federico Damián Schonborn (he/they) changed their profile picture. | 01:06:55 |
raitobezarius | In reply to @raitobezarius:matrix.org https://gerrit.lix.systems/c/lix/+/4004 that revert might not make sense in the end | 10:20:22 |
raitobezarius | i think we are facing a combination of two things: | 10:20:29 |
raitobezarius | hangs due to certificates | 10:20:31 |
raitobezarius | and emilazy's changes might be doing something bad to darwin | 10:20:38 |
emily | I tested with temp-dir = /nix/tmp. possible that I regressed something with temp-dir = /tmp. though it would be a bit weird since most of the refactoring around temp stuff was fairly mechanical | 11:32:47 |
| jopejoe1 changed their display name from jopejoe1 (4094@GPN23) to jopejoe1. | 18:50:44 |
| 21 Aug 2025 |
WeetHet | What the | 07:58:55 |
WeetHet | nix flake update should have nothing to do with that code | 08:01:19 |
WeetHet | Fails:
$ nix --extra-experimental-features nix-command --extra-experimental-features flakes flake lock --override-input lix git+https://git.lix.systems/lix-project/lix?rev=20f84eb6bfa874a1c4f45b38b949335f580ea0d7
error:
… while updating the lock file of flake 'path:/Users/weethet/Projects/Trash?lastModified=1755765056&narHash=sha256-gTvZ/vT/V3E8DVb2V%2BH03/0mNxHr7V0RG2FTU1V9B7U%3D'
… while updating the flake input 'lix'
… while fetching the input 'git+https://git.lix.systems/lix-project/lix?rev=20f84eb6bfa874a1c4f45b38b949335f580ea0d7'
error: failed to extract archive (Cannot extract through symlink /tmp/nix-7s5dsxncvd8wyr6wd9wpkv5f4y/.clang-format)
Succeeds:
$ TMPDIR=/private/tmp nix --extra-experimental-features nix-command --extra-experimental-features flakes flake lock --override-input lix git+https://git.lix.systems/lix-project/lix?rev=20f84eb6bfa874a1c4f45b38b949335f580ea0d7
| 09:15:11 |
WeetHet | Probably this would be the fix:
Path defaultTempDir()
{
auto dir = settings.tempDir.get().or_else([] {
return getEnvNonEmpty("TMPDIR").and_then([](auto val) -> std::optional<Path> {
#if __APPLE__
/* On macOS, don't use the per-session TMPDIR (as set e.g. by
sshd). This breaks build users because they don't have access
to the TMPDIR, in particular in ‘nix-store --serve’. */
if (val.starts_with("/var/folders/")) {
return std::nullopt;
}
#endif
return val;
});
}).value_or("/tmp");
return canonPath(dir, true);
}
| 09:16:15 |
emily | why would it need to change? did I break the behaviour of something in the temp directory functions? | 10:27:50 |
emily | tbh that #if __APPLE__ seems obsolete | 10:28:21 |
emily | given that builds are now in /nix | 10:28:26 |
emily | maybe we could just drop it… | 10:28:30 |
raitobezarius | emily given https://git.lix.systems/lix-project/lix/issues/975 i intend to submit a revert of the series starting from there fwiw | 14:34:33 |
raitobezarius | maybe a clean revert is better hm | 14:35:21 |