[PATCH] build-system/gnu: Make libraries writable before stripping.

[PATCH] build-system/gnu: Make libraries writable before stripping.

[Marius Bakke]

* guix/build/gnu-build-system.scm (strip)[strip-dir]: Change mode of
files before running strip-command.
---
 guix/build/gnu-build-system.scm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/guix/build/gnu-build-system.scm b/guix/build/gnu-build-system.scm
index 1dfd854..59394c2 100644
--- a/guix/build/gnu-build-system.scm
+++ b/guix/build/gnu-build-system.scm
@@ -393,6 +393,8 @@ makefiles."
                      (or (elf-file? file) (ar-file? file))
                      (or (not debug-output)
                          (make-debug-file file))
+                     ;; Ensure libraries are writable.
+                     (chmod file #o755)
                      (zero? (apply system* strip-command
                                    (append strip-flags (list file))))
                      (or (not debug-output)
-- 
2.10.2

Re: [PATCH] build-system/gnu: Make libraries writable before stripping.

[Ludovic Courtès]

Marius Bakke <mbakke@fastmail.com> skribis:

> * guix/build/gnu-build-system.scm (strip)[strip-dir]: Change mode of
> files before running strip-command.

OK for ‘core-updates’.

Note that a few packages did that by hand.  While you’re at it, you
can adjust them.

Thanks!

Ludo’.

Re: [PATCH] build-system/gnu: Make libraries writable before stripping.

[Marius Bakke]

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

Ludovic Courtès <ludo@gnu.org> writes:

> Marius Bakke <mbakke@fastmail.com> skribis:
>
>> * guix/build/gnu-build-system.scm (strip)[strip-dir]: Change mode of
>> files before running strip-command.
>
> OK for ‘core-updates’.

Cool, pushed!

> Note that a few packages did that by hand.  While you’re at it, you
> can adjust them.

That was the motivation. I will submit a patch removing most users
after next master merge.

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