[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#57050] [PATCH v2 04/13] gnu: Add Zuo.
From: |
Maxime Devos |
Subject: |
[bug#57050] [PATCH v2 04/13] gnu: Add Zuo. |
Date: |
Tue, 23 Aug 2022 11:20:25 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.12.0 |
[...]
I will use GUIX_SH in v3 of this series.
My concern with it originally was that it's generic enough that it might be
used in other ways elsewhere in Guix, but, since I'm hoping it's only going to
be a medium-term solution, it seems good enough, and I haven't heard any
objections to it.
_PATH_BSHELL seems better to me, as it is not Guix-specific and does not
require adding preprocessor arguments. Even simpler would be to
substitute* the /bin/sh to (search-input-file inputs "bin/sh") like done
for other packages.
I have previously objected to it, though not explicitly:
I haven't been following the discussion on the other patches, but
didn't I give an example of something independent of the Racket
component in use and even independent of Racket itself? See the
suggestion of using the already existing _PATH_BSHELL from <paths.h>.
It's even not Guix-specific, apparently it's a BSD-ism!
I don't see the point of a GUIX_SH macro when the standard-ish
_PATH_BSHELL appears to suffice.
On 23-08-2022 03:40, Philip McGrath wrote:
First of all, I have found that there*is* another Unix-like system where "/bin/sh"
doesn't exist: on Android, the POSIX shell is usually at "/system/bin/sh". Also, at least
on some versions, _PATH_BSHELL isn't a compile-time constant. It is:
#define _PATH_BSHELL __bionic_get_shell_path()
Looking at the patch, it not being a compile-time constant does not
appear to be a problem to me.
Greetings,
Maxime.
OpenPGP_0x49E3EE22191725EE.asc
Description: OpenPGP public key
OpenPGP_signature
Description: OpenPGP digital signature
- [bug#57050] [PATCH v2 03/13] gnu: chez-scheme: Fix use of "/bin/sh"., (continued)
- [bug#57050] [PATCH v2 03/13] gnu: chez-scheme: Fix use of "/bin/sh"., Philip McGrath, 2022/08/11
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Philip McGrath, 2022/08/11
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Liliana Marie Prikler, 2022/08/11
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Philip McGrath, 2022/08/11
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Liliana Marie Prikler, 2022/08/11
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Philip McGrath, 2022/08/11
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Maxime Devos, 2022/08/16
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Philip McGrath, 2022/08/22
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Maxime Devos, 2022/08/23
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Philip McGrath, 2022/08/23
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo.,
Maxime Devos <=
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Philip McGrath, 2022/08/23
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Liliana Marie Prikler, 2022/08/24
- [bug#57050] [PATCH v2 04/13] gnu: Add Zuo., Philip McGrath, 2022/08/24
- [bug#57050] [PATCH v3 00/14] gnu: Update Racket to 8.6. Add Zuo., Philip McGrath, 2022/08/25
- [bug#57050] [PATCH v3 01/14] gnu: stex: Update to 1.2.2-2.afa6075., Philip McGrath, 2022/08/25
- [bug#57050] [PATCH v3 03/14] etc: teams: Add racket team., Philip McGrath, 2022/08/25
- [bug#57050] [PATCH v3 02/14] gnu: stex: Fix read-only gifs and math directories., Philip McGrath, 2022/08/25
- [bug#57050] [PATCH v3 04/14] etc: teams: Add entry for Philip McGrath., Philip McGrath, 2022/08/25
- [bug#57050] [PATCH v3 05/14] gnu: racket: Adjust patch for "/bin/sh" in rktio., Philip McGrath, 2022/08/25
- [bug#57050] [PATCH v3 05/14] gnu: racket: Adjust patch for "/bin/sh" in rktio., Liliana Marie Prikler, 2022/08/25