| 30 May 2021 |
LnL | master looks pretty good compared to 21.03 https://hydra.nixos.org/eval/1674155?filter=.x86_64-darwin&compare=1674101&full=#tabs-still-fail | 19:14:36 |
LnL | I'd say create a draft pr based against 7e9e1b6351bb4acc71f88d3fc4defa3fa899e121 so we can easily compare and then I'll trigger an eval with it | 19:15:43 |
toonn | That's a commit on master? The current PR is targetting staging should I change that over? | 19:18:15 |
LnL | well staging is kind of unstable and hard to compare | 19:18:51 |
LnL | I'd start with master and then change the base once everything looks good enough | 19:19:06 |
LnL | I did the same with eg. https://github.com/NixOS/nixpkgs/pull/85151 | 19:19:39 |
toonn | Ok, I'll rebase on master. | 19:24:01 |
toonn | I've rebased, I'll make sure the bootstrap-tools builds before changing the PR. | 19:34:14 |
toonn | This takes hours to verify though. | 19:34:24 |
toonn | I'll probably pick it back up on Tuesday. Run the builds in the background. | 19:34:42 |
LnL | sure, no rush | 19:34:56 |
toonn | Pinging sterni because this ^ discussion is relevent if they want to look into this bootstrap-tools bump still. | 19:40:25 |
toonn | *relevant. | 19:55:43 |
ris_ | 💦 llvm being in the curl rebuild loop | 21:11:20 |
toonn | Rebuild loop? You mean the LLVM 7 workaround? | 21:20:13 |
ris_ | whatever causes a change to curl to need a full llvm build before the "full" curl can be built | 21:34:25 |
Sandro | Maybe stdenv? | 21:35:27 |
LnL | if I recall correctly llvm -> cmake -> curl | 21:38:37 |
toonn | Maybe because curl is used in bootstrapping? | 21:43:01 |
LnL | actually that does make me realise that this is a native input only, so it's probably possible to propagate curl from bootstrap tools in more stages since nothing that needs it will keep references to it | 21:43:10 |
LnL | unlike clang or make this isn't part of the stdenv so we don't need a version without bootstrap tools references except for the very toplevel one | 21:44:49 |
toonn | Is curl rebuilt multiple times? | 22:02:03 |
ris_ | there are at least 3 curls | 22:03:32 |
ris_ | curl used in bootstrapping (anonymous?) | 22:03:46 |
ris_ | curlMinimal | 22:03:52 |
ris_ | and curl | 22:03:56 |
ris_ | that's not even counting stdenv.fetchurlBoot, which i don't even | 22:04:55 |
| 31 May 2021 |
LnL | for which packages is curlMinimal a dependency? | 05:40:38 |
thefloweringash | In reply to @kloenk:petabyte.dev The wrote back. It is fixed in Zulu: 16.30.19. thefloweringash would you mind bumping it in your pr? I didn't even notice that x86_64 and aarch64 were at different zulu versions. PR updated | 07:22:37 |
| 0x4A6F changed their display name from [0x4A6F] to 0x4A6F. | 08:23:32 |