!UNVBThoJtlIiVwiDjU:nixos.org

Staging

189 Members
Staging merges | Running staging cycles: https://github.com/NixOS/nixpkgs/pulls?q=is%3Apr+is%3Aopen+head%3Astaging-next+head%3Astaging-next-24.05 | Review Reports: https://malob.github.io/nix-review-tools-reports/71 Servers

Load older messages


SenderMessageTime
16 Oct 2024
@emilazy:matrix.orgemilycompat stubs just weren't tested on Rust packages on Linux it looks like19:35:59
@emilazy:matrix.orgemilybecause of the evil stuff the old SDK pattern needed to do for Rust in the first place19:36:14
@justscreamy:matrix.orgjustscreamy joined the room.20:02:10
@reckenrode:matrix.orgRandy Eckenrode
In reply to @emilazy:matrix.org
K900: here you go https://github.com/NixOS/nixpkgs/pull/349117
Thanks for fixing. I’m surprised it got last ofborg.
23:10:16
@emilazy:matrix.orgemilyAIUI it only breaks evaluation of the insides of Rust packages23:10:47
@emilazy:matrix.orgemily I believe ofborg only catches eval failures that will "break eval", i.e. stuff that breaks before inside the body of whatever is callPackaged 23:11:07
@emilazy:matrix.orgemilybut I'm not sure, since it also evaluates some tests and stuff…23:11:12
@lily:lily.flowersLily FosterOfborg will catch failure to eval any outPath of hydra jobset23:14:10
@emilazy:matrix.orgemily🤔23:14:47
@lily:lily.flowersLily Fosternot sure how that could have gotten past ofborg either tbh23:14:55
@emilazy:matrix.orgemilyI think the answer is that it didn't actually finishing evaluating the final version because we rushed before the freeze, and IIRC on previous revisions there was some other eval-breaking thing.23:15:36
@lily:lily.flowersLily Foster * Ofborg will catch failure to eval any outPath of hydra jobset (with aliases turned off)23:15:39
@emilazy:matrix.orgemily(unrelated to the change)23:15:41
@emilazy:matrix.orgemilyOTOH I'm not sure how ofborg hasn't complained about every PR since?23:15:49
@emilazy:matrix.orgemilyso maybe it is actually just some accursed mystery23:15:56
@lily:lily.flowersLily Fosteroh it's probably because of the system fuckery that happens23:17:58
@emilazy:matrix.orgemilydoes ofborg somehow evaluate as all systems simultaneously23:21:10
@lily:lily.flowersLily Fosterwell, it uses the hydra jobset23:21:21
@lily:lily.flowersLily Fosterso yes23:21:24
@lily:lily.flowersLily Fosterin a sense23:21:32
@reckenrode:matrix.orgRandy Eckenrode
In reply to @emilazy:matrix.org
I think the answer is that it didn't actually finishing evaluating the final version because we rushed before the freeze, and IIRC on previous revisions there was some other eval-breaking thing.
I ran the ofborg check locally. I did confirm eval with the SDK rework before I marked it ready. It’s very strange that it failed on Hydra, but I’ve also encountered a case where my tree passed the ofborg check, but something I tried to build failed to eval.
23:21:48
17 Oct 2024
@lily:lily.flowersLily Foster
In reply to @emilazy:matrix.org
so maybe it is actually just some accursed mystery
config.allowUnsupportedSystem = true
00:00:31
@emilazy:matrix.orgemilyah00:00:39
@emilazy:matrix.orgemilynot so accursed00:00:42
@emilazy:matrix.orgemilybut also, come on00:00:44
@lily:lily.flowersLily Fosterthat was what i suspected when i realized the hydra jobset was what was failing to eval (i didn't read context)00:00:50
@lily:lily.flowersLily Fosterlet me double check though00:01:02
@lily:lily.flowersLily Fosteroh no it's not that at all00:02:08
@lily:lily.flowersLily Foster https://github.com/NixOS/ofborg/blob/5a4e743f192fb151915fcbe8789922fa401ecf48/ofborg/src/outpaths.nix#L22-L25 00:02:09
@lily:lily.flowersLily Fosterbut also00:02:11

Show newer messages


Back to Room ListRoom Version: 6