6 Sep 2024 |
emily | also because probably there are janky homemade ./configure scripts that accept --prefix but not all the others? | 13:04:32 |
Alyssa Ross | that's probably the main reason | 13:04:57 |
Alyssa Ross | I missed that line before :( | 13:05:15 |
emily | almost as if assuming all the world is autotools is bad š« | 13:05:45 |
Alyssa Ross | But I'm glad to have learned about it, because now maybe I can fix https://github.com/NixOS/nix/issues/11389 | 13:05:58 |
emily | hm | 13:06:11 |
emily | but there's setOutputFlags as an escape hatch | 13:06:15 |
emily | and a lot of packages already set that | 13:06:21 |
emily | maybe we can just get rid of [ "$(getAllOutputNames)" = "out" ] for a staging cycle and then I fix sbin the one after? | 13:06:37 |
Artturin | zlib
./configure: interpreter directive changed from "#!/bin/sh" to "/nix/store/razasrvdg7ckplfmvdxv4ia3wbayr94s-bootstrap-tools/bin/sh"
configure flags: --prefix=/nix/store/m88zpjjm4cgk9i6hfkyymb7a20fnkj6q-zlib-1.3.1 --bindir=/nix/store/m88zpjjm4cgk9i6hfkyymb7a20fnkj6q-zlib-1.3.1/bin --sbindir=/nix/store/m88zpjjm4cgk9i6hfkyymb7a20fnkj6q-zlib-1.3.1/sbin --includedir=/nix/store/5hdwfmdcv170>
unknown option: --bindir=/nix/store/m88zpjjm4cgk9i6hfkyymb7a20fnkj6q-zlib-1.3.1/bin
./configure --help for help
** ./configure aborting.
| 13:07:11 |
emily | I don't know if that's too ambitious, but "set one boolean flag on packages that actually currently work but have a bad home-grown configure script" soundsā¦ tenable? | 13:07:31 |
emily | I feel like I could triage that without burning out. | 13:07:41 |
Artturin | It has multiple outputs and setOutputFlags=false so at least that part has to be kept | 13:07:57 |
Alyssa Ross | although to fix my Nix issue I'd have to add setOutputFlags support to the Meson hook | 13:08:04 |
Alyssa Ross | I guess I'll do that | 13:08:06 |
emily | In reply to @artturin:matrix.org It has multiple outputs and setOutputFlags=false so at least that part has to be kept right | 13:09:52 |
emily | I mean any package that already sets it is fine | 13:09:55 |
emily | I wouldn't want to get rid of that flag | 13:09:59 |
emily | I just think we should be doing this stuff even if outputs = ["out"] | 13:10:08 |
emily | let's see if I can get to hello with that | 13:10:42 |
Alyssa Ross | https://github.com/NixOS/nixpkgs/pull/340089 | 13:14:12 |
emily | if [ -z "${dontUseMesonConfigure-}" ] && [ -z "${configurePhase-}" ]; then
# shellcheck disable=SC2034
setOutputFlags=
configurePhase=mesonConfigurePhase
fi
| 13:15:17 |
emily | don't you need to adjust that too? | 13:15:21 |
Alyssa Ross | Oops, yes | 13:18:12 |
Alyssa Ross | I should have tested more thoroughly but I was distracted by Rust update being broken again | 13:18:28 |
Alyssa Ross | Drafted | 13:18:39 |
emily | I'm bootstrapping a stdenv without the [ "$(getAllOutputNames)" = "out" ] so we'll see if maybe you won't need to justify the divergence :) | 13:20:21 |
emily | I really want a world where we have autotoolsHook | 13:23:52 |
K900 | Who doesn't | 13:25:04 |
emily | the users of the dearly-departed gnu module | 13:26:15 |