| 7 Dec 2025 |
 Rutile (Commentator2.0) feel free to ping | 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 |
| thubrecht | But the other tests that use NIX_ABORT_ON_WARN don't actually fail because the warning in considered an error | 20:54:24 |
| thubrecht | But because of error: cannot convert a thunk to JSON | 20:54:31 |
| Sergei Zimmerman (xokdvium) | Ugh, wth | 20:54:57 |
| Sergei Zimmerman (xokdvium) | That error doesn't make sense, nothing in user code should be able to observe thunks | 20:55:50 |
| thubrecht | It's evaluation an option | 20:57:29 |
| thubrecht | * It's evaluating an option | 20:57:34 |
| Sergei Zimmerman (xokdvium) | Is this just for the nix-instantiate output right? | 20:59:09 |
| thubrecht | It's at least for this, I did not investiguate further | 21:00:09 |
| thubrecht | The command was:
nix-instantiate -E 'import ./default.nix { modules = [ ./deprecated-wrapped.nix ];}' -A 'options.mergedEither.type.nestedTypes.right.functor.wrapped' --eval-only --json --show-trace
| 21:00:22 |
| thubrecht | * The command was:
nix-instantiate -E 'import ./default.nix { modules = [ ./deprecated-wrapped.nix ];}' -A 'options.mergedEither.type.nestedTypes.right.functor.wrapped' --eval-only --json
| 21:00:37 |
| thubrecht | Redacted or Malformed Event | 21:00:53 |
| Sergei Zimmerman (xokdvium) | Yeah then this is just a nix-instantiate not forcing thunks | 21:01:10 |
| thubrecht | Even if it was forcing chunks, then the error would be cannot convert the built-in function 'isString' to JSON or something along the lines | 21:02:51 |
| thubrecht | At the end of the day, the test passes but for the wrong reasons | 21:03:44 |
| thubrecht | https://gerrit.lix.systems/c/lix/+/4679/16/misc/nixpkgs-lib-tests.patch this makes the tests behave correctly btw @xokdvium:matrix.org | 22:38:46 |
| 8 Dec 2025 |
 jade_ | I told the people implementing fixing this to set NIX_CONFIG to make it use the new functionality and deprecate the weird environment variable O_o | 00:02:03 |
| jade_ | confused | 00:02:09 |
| jade_ | oh they snuck more legacy nonsense into there | 00:02:55 |
| jade_ | agh | 00:02:56 |
| thubrecht | I sent https://github.com/NixOS/nixpkgs/pull/468901 | 08:43:53 |
