| 7 Dec 2025 |
thubrecht | Welp https://github.com/NixOS/nixpkgs/commit/b3cf9ce0f9d01b9c1dc4b7e04fb8b28f79ee68c4 | 09:21:41 |
raitobezarius | sounds like it's worth sending a revert | 14:13:32 |
Rutile (rootile) | what context am i missing? | 19:11:45 |
Sergei Zimmerman (xokdvium) | In reply to @commentator2.0:elia.garden what context am i missing? The env variable is CppNix specific and is kind of a kludge | 19:12:21 |
Sergei Zimmerman (xokdvium) | Also builtins.warn hasn’t landed yet I think because of the whole env variable? | 19:12:55 |
Sergei Zimmerman (xokdvium) | Anyway minver is 2.18 and the abort on warm behavior can’t be relied on to be supported if it’s polyfilled | 19:13:45 |
Sergei Zimmerman (xokdvium) | * Anyway minver is 2.18 and the abort on warn behavior can’t be relied on to be supported if it’s polyfilled | 19:13:58 |
Sergei Zimmerman (xokdvium) | Just grepping for “warn” would have been a better approach. Or do a scopedImport to make builtins.warn into a throw. | 19:16:20 |
thubrecht | The issue at play I think is that lix now has builtins.warn but it righfully has no knowledge of the env variable, hence the lib polyfill does not execute and no throws happen | 19:54:04 |
thubrecht | What is weird though is that there are similar checks that are done before, but they still pass... | 19:55:25 |
Sergei Zimmerman (xokdvium) | In reply to @thubrecht:matrix.org What is weird though is that there are similar checks that are done before, but they still pass... Exit code is not checked? | 20:15:37 |
thubrecht | It is checked | 20:53:56 |