[PATCH] gnu: %static-inputs: Use bash from PATH in bootstrap tar.

[PATCH] gnu: %static-inputs: Use bash from PATH in bootstrap tar.

[Carlos Sánchez de La Lama]

See
<http://lists.gnu.org/archive/html/guix-devel/2016-09/msg02272.html>.

gnu/packages/make-bootstrap.scm (%static-inputs): Use bash from PATH.
---
 gnu/packages/make-bootstrap.scm | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/gnu/packages/make-bootstrap.scm b/gnu/packages/make-bootstrap.scm
index def9c23..180ca72 100644
--- a/gnu/packages/make-bootstrap.scm
+++ b/gnu/packages/make-bootstrap.scm
@@ -186,6 +186,16 @@ for `sh' in $PATH, and without nscd, and with static NSS modules."
                 (inputs (if (%current-target-system)
                             `(("bash" ,%bash-static))
                             '()))))
+	(tar (package (inherit tar)
+	       (arguments
+		'(#:phases (modify-phases %standard-phases
+			     (add-before 'build 'set-shell-file-name
+			       (lambda _
+				 ;; Do not use "/bin/sh" to run programs.
+				 (substitute* "src/system.c"
+				   (("/bin/sh") "sh")
+				   (("execv ") "execvp "))
+				 #t)))))))
         (finalize (compose static-package
                            package-with-relocatable-glibc)))
     `(,@(map (match-lambda
-- 
2.9.2

Re: [PATCH] gnu: %static-inputs: Use bash from PATH in bootstrap tar.

[Ludovic Courtès]

csanchezdll@gmail.com (Carlos Sánchez de La Lama) skribis:

> See
> <http://lists.gnu.org/archive/html/guix-devel/2016-09/msg02272.html>.

Could you put this URL next to the “Do not use /bin/sh” comment?

> gnu/packages/make-bootstrap.scm (%static-inputs): Use bash from PATH.

Otherwise LGTM, thanks!

Ludo’.

Re: [PATCH] gnu: %static-inputs: Use bash from PATH in bootstrap tar.

[Efraim Flashner]

[-- Attachment #1: Type: text/plain, Size: 844 bytes --]

On Wed, Oct 19, 2016 at 05:34:17PM +0200, Ludovic Courtès wrote:
> csanchezdll@gmail.com (Carlos Sánchez de La Lama) skribis:
> 
> > See
> > <http://lists.gnu.org/archive/html/guix-devel/2016-09/msg02272.html>.
> 
> Could you put this URL next to the “Do not use /bin/sh” comment?
> 
> > gnu/packages/make-bootstrap.scm (%static-inputs): Use bash from PATH.
> 
> Otherwise LGTM, thanks!
> 
> Ludo’.
> 

Was this tested with the other bootstrap binaries? I'm still having
trouble with the aarch64 ones, didn't check if/how it affected the ones
that are already in Guix (which I assume would be not at all)

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [PATCH] gnu: %static-inputs: Use bash from PATH in bootstrap tar.

[Ludovic Courtès]

Efraim Flashner <efraim@flashner.co.il> skribis:

> On Wed, Oct 19, 2016 at 05:34:17PM +0200, Ludovic Courtès wrote:
>> csanchezdll@gmail.com (Carlos Sánchez de La Lama) skribis:
>> 
>> > See
>> > <http://lists.gnu.org/archive/html/guix-devel/2016-09/msg02272.html>.
>> 
>> Could you put this URL next to the “Do not use /bin/sh” comment?
>> 
>> > gnu/packages/make-bootstrap.scm (%static-inputs): Use bash from PATH.
>> 
>> Otherwise LGTM, thanks!
>> 
>> Ludo’.
>> 
>
> Was this tested with the other bootstrap binaries?

It changes execv("/bin/sh", …) to execvp("sh", …) so it looks right.

> I'm still having trouble with the aarch64 ones, didn't check if/how it
> affected the ones that are already in Guix (which I assume would be
> not at all)

What kind of problems?  After this patch?

Thanks,
Ludo’.

Re: [PATCH] gnu: %static-inputs: Use bash from PATH in bootstrap tar.

[Efraim Flashner]

On October 19, 2016 10:48:11 PM GMT+03:00, ludo@gnu.org wrote:
>Efraim Flashner <efraim@flashner.co.il> skribis:
>
>> On Wed, Oct 19, 2016 at 05:34:17PM +0200, Ludovic Courtès wrote:
>>> csanchezdll@gmail.com (Carlos Sánchez de La Lama) skribis:
>>> 
>>> > See
>>> >
><http://lists.gnu.org/archive/html/guix-devel/2016-09/msg02272.html>.
>>> 
>>> Could you put this URL next to the “Do not use /bin/sh” comment?
>>> 
>>> > gnu/packages/make-bootstrap.scm (%static-inputs): Use bash from
>PATH.
>>> 
>>> Otherwise LGTM, thanks!
>>> 
>>> Ludo’.
>>> 
>>
>> Was this tested with the other bootstrap binaries?
>
>It changes execv("/bin/sh", …) to execvp("sh", …) so it looks right.
>
>> I'm still having trouble with the aarch64 ones, didn't check if/how
>it
>> affected the ones that are already in Guix (which I assume would be
>> not at all)
>
>What kind of problems?  After this patch?
>
>Thanks,
>Ludo’.

Sorry, unrelated to this patch. Nvm and carry on

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Re: [PATCH] gnu: %static-inputs: Use bash from PATH in bootstrap tar.

[Carlos Sánchez de La Lama]

Hi!

>> See
>> <http://lists.gnu.org/archive/html/guix-devel/2016-09/msg02272.html>.
>
> Could you put this URL next to the “Do not use /bin/sh” comment?
>
>> gnu/packages/make-bootstrap.scm (%static-inputs): Use bash from PATH.
>
> Otherwise LGTM, thanks!

Done, and untabify'ed also (I forgot on first version).

Thanks!

Carlos

gnu: %static-inputs: Use bash from PATH in bootstrap tar.

gnu/packages/make-bootstrap.scm (%static-inputs): Use bash from PATH.
---
 gnu/packages/make-bootstrap.scm | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/gnu/packages/make-bootstrap.scm b/gnu/packages/make-bootstrap.scm
index def9c23..28b1549 100644
--- a/gnu/packages/make-bootstrap.scm
+++ b/gnu/packages/make-bootstrap.scm
@@ -186,6 +186,17 @@ for `sh' in $PATH, and without nscd, and with static NSS modules."
                 (inputs (if (%current-target-system)
                             `(("bash" ,%bash-static))
                             '()))))
+        (tar (package (inherit tar)
+               (arguments
+                '(#:phases (modify-phases %standard-phases
+                             (add-before 'build 'set-shell-file-name
+                               (lambda _
+                                 ;; Do not use "/bin/sh" to run programs. See
+                                 ;; <http://lists.gnu.org/archive/html/guix-devel/2016-09/msg02272.html>.
+                                 (substitute* "src/system.c"
+                                   (("/bin/sh") "sh")
+                                   (("execv ") "execvp "))
+                                 #t)))))))
         (finalize (compose static-package
                            package-with-relocatable-glibc)))
     `(,@(map (match-lambda
-- 
2.9.2

Re: [PATCH] gnu: %static-inputs: Use bash from PATH in bootstrap tar.

[Ludovic Courtès]

csanchezdll@gmail.com (Carlos Sánchez de La Lama) skribis:

> Hi!
>
>>> See
>>> <http://lists.gnu.org/archive/html/guix-devel/2016-09/msg02272.html>.
>>
>> Could you put this URL next to the “Do not use /bin/sh” comment?
>>
>>> gnu/packages/make-bootstrap.scm (%static-inputs): Use bash from PATH.
>>
>> Otherwise LGTM, thanks!
>
> Done, and untabify'ed also (I forgot on first version).
>
> Thanks!

I had done it for you in 80c9164fd7741e3bb3da368666f41c49a456d990 when I
realized that I’d have to push on your behalf.

Sorry for the confusion!

Thanks,
Ludo’.

Re: [PATCH] gnu: %static-inputs: Use bash from PATH in bootstrap tar.

[Carlos Sánchez de La Lama]

Hi Ludo,

>> Done, and untabify'ed also (I forgot on first version).
>
> I had done it for you in 80c9164fd7741e3bb3da368666f41c49a456d990 when I
> realized that I’d have to push on your behalf.

Great, thanks :)

Carlos