[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#65853] bug#65857: [PATCH v2 2/2] gnu: scilab: Set SCIHOME default t
From: |
Nicolas Graves |
Subject: |
[bug#65853] bug#65857: [PATCH v2 2/2] gnu: scilab: Set SCIHOME default to respect XDG base dirs. |
Date: |
Thu, 05 Oct 2023 17:09:20 +0200 |
On 2023-10-05 16:51, Ludovic Courtès wrote:
> Hi,
>
> Nicolas Graves <ngraves@ngraves.fr> skribis:
>
>> + (add-after 'install 'hardcode-script-inputs
>> + (lambda _
>> + (substitute* (string-append #$output "/bin/scilab")
>> + (("grep")
>> + (search-inputs-file #$inputs "/bin/grep"))
>
> I’ll contradict Efraim, but hopefully they’ll pardon me: I’d write it
> like so:
>
> (lambda* (#:key inputs #:allow-other-keys)
> (substitute* …
> (("grep")
> (search-input-file inputs "/bin/grep"))))
>
> That way it’ll DTRT even when creating a variant of this package.
>
> (The code above had a typo too, with “inputs” as plural.)
>
> Could you send one last updated patch?
Yes, it also has to be rebased now that scilab is version 2023.
>
> Thanks,
> Ludo’.
--
Best regards,
Nicolas Graves