| 28 Oct 2025 |
emily | (they were added to the aliases file many years ago) | 17:54:32 |
emily | but I opted for being conservative about that since it's post-freeze | 17:54:39 |
Artturin | Oh yeah we still have those aliases, yeah we should make them warns | 17:55:58 |
emily | they're still used in pkgs/test even 🙃 | 17:56:13 |
emily | can we make stdenv.isFoo warn as well? 😆 | 17:56:16 |
emily | maybe something for early 26.05 | 17:56:21 |
emily | looking at the diff, runCommandNoCC, runCommandNoCCLocal, and maybe utillinux seem like the main things worth turning into warnings instead of throws | 18:00:22 |
Artturin | You sure? | 18:00:30 |
emily | might also be good to adjust the python throw error message | 18:00:47 |
emily | since it currently recommends people use python2 😆 | 18:00:51 |
emily | I'll PR those | 18:00:53 |
emily | https://github.com/NixOS/nixpkgs/actions/runs/18828351853/job/53715047703?pr=456065 | 18:01:01 |
emily | this is what happened when I forgot the commit reverting turning those into throws | 18:01:08 |
emily | vcunat: btw, warning aliases used in NixOS tests won't block/fail anything on Hydra, right? because the warning check there is only in the release checks? | 18:01:40 |
emily | is it possible to make Hydra do the equivalent of NIX_ABORT_ON_WARN instead of checking it in release checks, so that those are caught in the same way warnings in packages are? | 18:02:10 |
Artturin | Ok I can take care of those | 18:03:18 |
emily | no rush | 18:04:09 |