Mark H Weaver <m...@netris.org> writes: > David Thompson <dthomps...@worcester.edu> writes: > >> Paul van der Walt <p...@denknerd.org> writes: > [...] >>> + (arguments >>> + '(#:configure-flags >>> + '("BOOST_LIB_SUFFIX=") >>> + #:phases >>> + (alist-cons-after >>> + 'unpack 'autogen >>> + (lambda _ >>> + (substitute* "autogen.sh" >>> + (("/bin/sh") (which "bash"))) >> >> Is "autogen.sh" not executable? If it starts with a shebang, Guix >> should patch the file appropriately. > > The problem here is that this 'autogen' phase is run before the > 'patch-source-shebangs' phase, at my suggestion. It has to be that way, > because 'patch-source-shebangs' (and 'patch-usr-bin-file') will need to > be run on the files created by 'autogen.sh'.
Oh, okay. Right. Thanks for explaining. > >>> + (setenv "NOCONFIGURE" "true") >>> + (zero? (system* "bash" "autogen.sh"))) >> >> If its executable, you can say: >> >> (zero? (system* "./autogen.sh")) > > Yes, after the 'substitute*' above, this should presumably work. My > guess is that Paul added the "bash" here before he realized that he > also needed the 'substitute*'. Makes sense. -- David Thompson Web Developer - Free Software Foundation - http://fsf.org GPG Key: 0FF1D807 Support the FSF: https://fsf.org/donate