On Thu, Apr 22, 2021 at 06:40:46PM +0200, Maxime Devos wrote:
Xinglu Chen schreef op wo 21-04-2021 om 23:45 [+0200]:
On Wed, Apr 21 2021, Roel Janssen wrote:
[...]
+ (arguments
+ `(#:phases
+ (modify-phases %standard-phases
+ (add-after 'unpack 'autogen
+ (lambda _
+ (zero? (system* "sh" "autogen.sh")))))))
IIRC, phases don’t have to return #t, so you could remove ‘zero?’.
Try running (system* "does-not-exist"). It will fail by returning
something non-zero. If I recall how to call "invoke" correctly,
I would recommend (invoke "sh" "autogen.sh") here. "invoke" raises
an exception when the command fails, instead of returning something.
While we're at it, can this phase replace 'bootstrap? It seems to me we
shouldn't need both phases.