all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#54153] gnu: mblaze: Offer contrib executables.
@ 2022-02-25  6:42 elaexuotee--- via Guix-patches via
  2022-03-16 14:44 ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: elaexuotee--- via Guix-patches via @ 2022-02-25  6:42 UTC (permalink / raw)
  To: 54153

Upstream offers several extra useful utilities under the `contrib' directory.
These are intentially not installed by the `install' make target, as the
scripts are provided "as-is" without portability, testing, etc. guarantees.

I would be nice to offer these somehow. Attached is a patch that installs them
into a "contrib" output. It might also make sense to just throw them in the
default output, instead.

Cheers,

#x-patch 0001-gnu-mblaze-Offer-contrib-executables.patch




^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#54153] gnu: mblaze: Offer contrib executables.
  2022-02-25  6:42 [bug#54153] gnu: mblaze: Offer contrib executables elaexuotee--- via Guix-patches via
@ 2022-03-16 14:44 ` Ludovic Courtès
  2022-03-17  2:31   ` elaexuotee--- via Guix-patches via
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2022-03-16 14:44 UTC (permalink / raw)
  To: elaexuotee; +Cc: 54153

Hi,

elaexuotee@wilsonb.com skribis:

> Upstream offers several extra useful utilities under the `contrib' directory.
> These are intentially not installed by the `install' make target, as the
> scripts are provided "as-is" without portability, testing, etc. guarantees.
>
> I would be nice to offer these somehow. Attached is a patch that installs them
> into a "contrib" output. It might also make sense to just throw them in the
> default output, instead.
>
> Cheers,
>
> #x-patch 0001-gnu-mblaze-Offer-contrib-executables.patch

Looks like the patch isn’t actually there.  ↑

Could you send it?

Thanks in advance!  :-)

Ludo’.




^ permalink raw reply	[flat|nested] 4+ messages in thread

* [bug#54153] gnu: mblaze: Offer contrib executables.
  2022-03-16 14:44 ` Ludovic Courtès
@ 2022-03-17  2:31   ` elaexuotee--- via Guix-patches via
       [not found]     ` <25a020feaa0d60db6f6606ad9cc2e81ae77650c8.camel@telenet.be>
  0 siblings, 1 reply; 4+ messages in thread
From: elaexuotee--- via Guix-patches via @ 2022-03-17  2:31 UTC (permalink / raw)
  To: Ludovic Courtès

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

Ludovic Courtès <ludo@gnu.org> wrote:
> Looks like the patch isn’t actually there.  ↑
> 
> Could you send it?

Oops! Here you go.


[-- Attachment #2: 0001-gnu-mblaze-Offer-contrib-executables.patch --]
[-- Type: text/x-patch, Size: 2082 bytes --]

From e389b54cb0347bef0d31185472df7206f2b91aa3 Mon Sep 17 00:00:00 2001
From: "B. Wilson" <elaexuotee@wilsonb.com>
Date: Fri, 25 Feb 2022 15:31:05 +0900
Subject: [PATCH] gnu: mblaze: Offer contrib executables.
To: guix-patches@gnu.org

* gnu/packages/mail.scm(mblaze): New contrib output.
---
 gnu/packages/mail.scm | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
index b0b9ae9f76..7c7aa07428 100644
--- a/gnu/packages/mail.scm
+++ b/gnu/packages/mail.scm
@@ -2722,6 +2722,7 @@ (define-public mblaze
        (file-name (git-file-name name version))
        (sha256
         (base32 "0fa8s9dp5ilwmfcwkx72x2b5i0maa5sl97hv2cdknqmc27gv0b1c"))))
+    (outputs '("out" "contrib"))
     (build-system gnu-build-system)
     (native-inputs
      (list perl))
@@ -2730,9 +2731,26 @@ (define-public mblaze
        #:make-flags (list (string-append "CC=" ,(cc-for-target))
                           "PREFIX="
                           (string-append "DESTDIR=" %output))
+       #:modules ((ice-9 ftw)
+                  (guix build utils)
+                  (guix build gnu-build-system))
        #:phases
        (modify-phases %standard-phases
-         (delete 'configure))))
+         (delete 'configure)
+         (add-after 'install 'install-contrib
+           (lambda _
+             (let* ((out (assoc-ref %outputs "contrib"))
+                    (bin (string-append out "/bin"))
+                    (exe? (lambda (file)
+                            (let ((s (stat file)))
+                              (and (eq? 'regular (stat:type s))
+                                   (logtest #o100 (stat:perms s)))))))
+               (mkdir-p bin)
+               (with-directory-excursion "contrib"
+                 (for-each
+                   (lambda (file)
+                     (install-file file bin))
+                   (scandir "." exe?)))))))))
     (home-page "https://github.com/leahneukirchen/mblaze")
     (synopsis "Unix utilities to deal with Maildir")
     (description
-- 
2.34.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* bug#54153: gnu: mblaze: Offer contrib executables.
       [not found]       ` <3SCWHAVD365EF.2KODNTO4XF57G@"@WILSONB.COM>
@ 2022-04-15 15:55         ` Ludovic Courtès
  0 siblings, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2022-04-15 15:55 UTC (permalink / raw)
  To: elaexuotee; +Cc: Maxime Devos, 54153-done

Hi,

elaexuotee@wilsonb.com skribis:

> From 5f14f56a51cb340a54472a76610cb4672d36134f Mon Sep 17 00:00:00 2001
> From: "B. Wilson" <elaexuotee@wilsonb.com>
> Date: Fri, 25 Feb 2022 15:31:05 +0900
> Subject: [PATCH] gnu: mblaze: Offer contrib executables.
> To: guix-patches@gnu.org
>
> * gnu/packages/mail.scm(mblaze): New contrib output.

Applied…

>        #:make-flags
> -      #~(list #$(string-append "CC=" (cc-for-target))
> +      #~(list (string-append "CC=" #$(cc-for-target))
>                "PREFIX="
> -              (string-append "DESTDIR=" #$output))
> +              (string-append "DESTDIR=" %output))

… without this hunk.

I also added details to the commit log.

Thanks!

Ludo’.




^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-04-15 15:56 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-25  6:42 [bug#54153] gnu: mblaze: Offer contrib executables elaexuotee--- via Guix-patches via
2022-03-16 14:44 ` Ludovic Courtès
2022-03-17  2:31   ` elaexuotee--- via Guix-patches via
     [not found]     ` <25a020feaa0d60db6f6606ad9cc2e81ae77650c8.camel@telenet.be>
     [not found]       ` <3SCWHAVD365EF.2KODNTO4XF57G@"@WILSONB.COM>
2022-04-15 15:55         ` bug#54153: " Ludovic Courtès

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.