Re: /bin/sh in configure tests

[Ludovic Courtès]

Hello!

Andreas Enge <address@hidden> skribis:

> Yes, it does! Attached is a patch for a new package. If it is okay, I can 
> push it.

Just minor nitpicking below, but if it builds in a chroot without /bin
or anything, please push!

> +   (inputs `(("perl" ,perl)
> +             ))

Please put closing parentheses at the end of the line.

> +   (arguments
> +    (lambda (system)

Since the ‘system’ variable isn’t used, you don’t need a lambda here.
Instead you can just write:

  (arguments
    `(#:phases ...))

> +             (substitute* "configure"
> +                           (("`/bin/sh")
> +                           (string-append "`" (which "bash"))))

The opening parenthesis on the second line is normally aligned with the
‘u’ of ‘substitute*’.  :-)

Thanks,
Ludo’.