| 22 Dec 2025 |
helle (just a stray cat girl) | ah, nice, missed that, sorry | 08:58:29 |
helle (just a stray cat girl) | still waking up | 08:58:35 |
Robert Hensing (roberth) | my bad :) | 08:58:36 |
raitobezarius | I'm running smoketesting, roberth on your PR, once this passes, I will approve as a Lix code owner | 08:59:09 |
Robert Hensing (roberth) | awesome, thanks! | 08:59:20 |
raitobezarius | thanks for pulling the patch! | 08:59:28 |
helle (just a stray cat girl) | but yes, ty, I read the message like crossways, lol | 09:02:01 |
| @cinerealkiara:matrix.org left the room. | 10:53:47 |
| 23 Dec 2025 |
piegames | Hm. Unfortunately, the Nix language is not amenable to single-parse-pass compilation because of parsetime attrset merging | 09:56:00 |
piegames | Everything else can be done with placeholders and a fixup pass afterwards, but attrsets really require using some treemaps during construction | 09:56:40 |
piegames | Also let's move this over to Zulip https://zulip.lix.systems/#narrow/channel/6-Eval/topic/Parser.20rewrite/with/3389 | 10:36:24 |
XYenon | Should I do anything now, or just wait? My CL hasn't had any new reviews for a week.
https://gerrit.lix.systems/c/lix/+/4783 | 13:44:10 |
piegames | Asking here is fine | 14:10:07 |
piegames | Oh I see, there has been a disagreement between reviewers. Maybe we should escalate it to a core team discusssion then | 14:11:59 |
raitobezarius | where? | 14:15:48 |
piegames | The CL has both +1 and -1 reviews | 14:18:02 |
helle (just a stray cat girl) | is that not just purely for the test impl | 14:18:14 |
raitobezarius | i think you are misreading the reviews | 14:18:13 |
piegames | Or am I reading it wrongly (I'm on the phone so lol@Gerrit UI) | 14:18:19 |
raitobezarius | +1 was given over the F2 part | 14:18:40 |
raitobezarius | -1 was given over the actual contents | 14:18:46 |
raitobezarius | * -1 was given over the non testing changes | 14:18:49 |
raitobezarius | anyway, this stalled because:
(a) flakes are frozen in principle and we recommend reaching out here before sending changes
(b) selected reviewers did not interact with the CL
(c) it's holidays for many people :P | 14:19:59 |
XYenon | Get it. I thought this was just a small backport change. | 14:47:52 |
raitobezarius | No worries, unfortunately, as we touched upon it in https://lix.systems/blog/2025-11-18-lix-2.94-release/ (section "Flakes freeze"), too many inconsistences kept being discovered in how these features are rolled in CppNix | 14:49:37 |
raitobezarius | And as you have seen, horrors flagged such a problem on that backport, alas | 14:49:57 |
K900 | Oh yeah does anyone want a trivial UX improvement | 15:16:00 |
K900 | Make it so -L doesn't print the failing drv logs twice | 15:16:24 |
K900 | For bonus points make it so it prints the full list of all failing drvs with keep-going | 15:16:42 |
raitobezarius | open an issue so that someone gets to it eventually | 15:19:25 |