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

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

[Ludovic Courtès]

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

> Hi!
>
>>> See
>>> .
>>
>> 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!

>> See
>> .
>
> 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
+ ;; 
.
+ (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.

[Efraim Flashner]

On October 19, 2016 10:48:11 PM GMT+03:00, l...@gnu.org wrote:
>Efraim Flashner  skribis:
>
>> On Wed, Oct 19, 2016 at 05:34:17PM +0200, Ludovic Courtès wrote:
>>> csanchez...@gmail.com (Carlos Sánchez de La Lama) skribis:
>>> 
>>> > See
>>> >
>.
>>> 
>>> 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.

[Ludovic Courtès]

Efraim Flashner  skribis:

> On Wed, Oct 19, 2016 at 05:34:17PM +0200, Ludovic Courtès wrote:
>> csanchez...@gmail.com (Carlos Sánchez de La Lama) skribis:
>> 
>> > See
>> > .
>> 
>> 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 Wed, Oct 19, 2016 at 05:34:17PM +0200, Ludovic Courtès wrote:
> csanchez...@gmail.com (Carlos Sánchez de La Lama) skribis:
> 
> > See
> > .
> 
> 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  אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted


signature.asc
Description: PGP signature

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

[Ludovic Courtès]

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

> See
> .

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’.

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

[Carlos Sánchez de La Lama]

See
.

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