| 6 Oct 2021 |
|  kamadorueda joined the room. | 02:56:06 |
| kamadorueda changed their display name from Kevin Amado to kamadorueda. | 03:10:05 |
|  Rosario Pulella changed their display name from rosariopulella to Rosuavio. | 10:38:28 |
| Rosario Pulella changed their display name from Rosuavio to Rosario Pulella. | 10:44:53 |
| 7 Oct 2021 |
|  infinisil set a profile picture. | 00:08:00 |
 niksnut | If there are no objections, I'd like to branch 2.4 now. | 12:53:34 |
 balsoft | If there are still known bugs that's OK I'm assuming? | 12:53:57 |
| balsoft | E.g. https://github.com/NixOS/nix/issues/5289 | 12:54:13 |
| niksnut | Yeah, if they're not regressions | 12:54:38 |
 Alyssa Ross | isn't the musl issue a regression? | 14:07:06 |
| niksnut | Don't think so, since it was not part of the previous release iirc | 14:50:21 |
| niksnut | But it would be great if somebody can look into it | 14:50:32 |
| balsoft | While you're here sometimes I'm not getting any output even with -L on nix (Nix) 2.4pre20211001_4f49615 (and also I'm not getting the list of all derivation that failed), and it's a bit painful to reproduce. Is this a known bug that was fixed in the meantime? | 14:51:33 |
| balsoft | I would update and try to check on master, but I can't quite figure out when it happens | 14:51:46 |
| balsoft | And also I would think it's a regression | 14:52:00 |
 sterni | niksnut: nixStatic 2.3.x is exposed top level in nixpkgs which does use musl | 14:53:08 |
| niksnut | In reply to @balsoft:balsoft.ru
While you're here sometimes I'm not getting any output even with -L on nix (Nix) 2.4pre20211001_4f49615 (and also I'm not getting the list of all derivation that failed), and it's a bit painful to reproduce. Is this a known bug that was fixed in the meantime? Sounds like https://github.com/NixOS/nix/pull/5339 | 14:53:22 |
| balsoft | No, I think mine is different | 14:53:57 |
| balsoft | It's just in the regular terminal | 14:54:03 |
| balsoft | And not consistently so | 14:54:19 |
| balsoft | E.g. two consequtive runs of nix-store --realise on the same derivation:
error: 1 dependencies of derivation '/nix/store/<...>-nixos-system-foo.drv' failed to build
And second one:
error: builder for '/nix/store/<...>-nginx.conf.drv' failed with exit code 1;
last 10 log lines:
> }
> }
>
> ==================== Summary ===================
> Total issues:
> Unspecified: 0
> Low: 0
> Medium: 1
> High: 0
>
For full logs, run 'nix log /nix/store/<...>-nginx.conf.drv'.
error: 1 dependencies of derivation '/nix/store/<...>-unit-nginx.service.drv' failed to build
error: 1 dependencies of derivation '/nix/store/<...>-system-units.drv' failed to build
error: 1 dependencies of derivation '/nix/store/<...>-etc.drv' failed to build
error: 1 dependencies of derivation '/nix/store/<...>-nixos-system-foo.drv' failed to build
| 14:56:18 |
| balsoft | Also reproducible with passing -L to new nix commands | 14:56:32 |
| balsoft | (and by "reproducible" I mean it happens... sometimes) | 14:57:40 |
 andi- | Before diving into doing the change I had a thought yesterday while looking at related code: Any reason we don't implement Expr{Sub,Add,...} like we do for ExprNotOp etc? In the case of subtraction it could get rid of about 30k Symbol table looks, a few thousand new objects and execution wouldn't probably do a few less function calls. That would probably break overriding substraction by providing a __sub (or whatever) function in the current scope (if that even works right now). | 14:58:58 |
| balsoft | In reply to @balsoft:balsoft.ru
E.g. two consequtive runs of nix-store --realise on the same derivation:
error: 1 dependencies of derivation '/nix/store/<...>-nixos-system-foo.drv' failed to build
And second one:
error: builder for '/nix/store/<...>-nginx.conf.drv' failed with exit code 1;
last 10 log lines:
> }
> }
>
> ==================== Summary ===================
> Total issues:
> Unspecified: 0
> Low: 0
> Medium: 1
> High: 0
>
For full logs, run 'nix log /nix/store/<...>-nginx.conf.drv'.
error: 1 dependencies of derivation '/nix/store/<...>-unit-nginx.service.drv' failed to build
error: 1 dependencies of derivation '/nix/store/<...>-system-units.drv' failed to build
error: 1 dependencies of derivation '/nix/store/<...>-etc.drv' failed to build
error: 1 dependencies of derivation '/nix/store/<...>-nixos-system-foo.drv' failed to build
Interestingly, I think it never happens once logs for the derivation were shown at least once | 14:59:15 |
| andi- | * Before diving into doing the change I had a thought yesterday while looking at related code: Any reason we don't implement Expr{Sub,Add,...} like we do for ExprNotOp etc? In the case of subtraction it could get rid of about 30k Symbol table looks, a few thousand new objects (while building a NixOS test). And execution wouldn't probably do a few less function calls. That would probably break overriding substraction by providing a __sub (or whatever) function in the current scope (if that even works right now). | 14:59:20 |
| balsoft | I don't mind at all if this ends up in the release, but just wanted to note that it happens sometimes | 15:01:23 |
 baloo | niksnut: any chance we could get a 2.3 release or merge this? https://github.com/NixOS/nixpkgs/pull/137197 I think it should fix the errors we're seeing on https://r13y.com | 15:03:53 |
| niksnut | Sure, I can do a release | 15:04:16 |
| niksnut | In reply to @andi:kack.it
Before diving into doing the change I had a thought yesterday while looking at related code: Any reason we don't implement Expr{Sub,Add,...} like we do for ExprNotOp etc? In the case of subtraction it could get rid of about 30k Symbol table looks, a few thousand new objects (while building a NixOS test). And execution wouldn't probably do a few less function calls. That would probably break overriding substraction by providing a __sub (or whatever) function in the current scope (if that even works right now). No, I think the reason it's implemented that way is because builtins.sub etc. already existed | 15:14:19 |
