all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#67244] [PATCH] gnu: mu: Update to 1.10.8 and re-enable Guile bindings
@ 2023-11-17 17:21 Thiago Jung Bauermann
  2023-11-23 16:20 ` bug#67244: " Christopher Baines
  0 siblings, 1 reply; 3+ messages in thread
From: Thiago Jung Bauermann @ 2023-11-17 17:21 UTC (permalink / raw)
  To: 67244

Version 1.10.8 fixes the installation of the Guile bindings, so we can
enable it again.

* gnu/packages/mail.scm (mu): Update to 1.10.7.
  [inputs]: Add guile-3.0.
  [arguments]: Add "-Dguile-extension-dir" to #:configure-flags.
  Add fix-ffi phase to allow Guile module to load its native library.

Change-Id: I2a8250d8d6298712ad116b1d88157a4344175722
---

Hello,

I find the Guile bindings useful, so I'd like to reenable them.
I've tested that my script that uses them is working.

Kind regards,
Thiago

 gnu/packages/mail.scm | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
index 256bd4b03a5a..353c8c7bd722 100644
--- a/gnu/packages/mail.scm
+++ b/gnu/packages/mail.scm
@@ -1206,14 +1206,14 @@ (define-public emacs-mew
 (define-public mu
   (package
     (name "mu")
-    (version "1.10.7")
+    (version "1.10.8")
     (source
      (origin
        (method url-fetch)
        (uri (string-append "https://github.com/djcb/mu/releases/download/v"
                            version "/mu-" version ".tar.xz"))
        (sha256
-        (base32 "089w1m6sd0nk9l9j40d357fjym8kxmz7kwh3bclk58jxa6xckapa"))))
+        (base32 "129m6rz8vbd7370c3h3ma66bxqdkm6wsdix5qkmv1vm7sanxh4bb"))))
     (build-system meson-build-system)
     (native-inputs
      (list pkg-config
@@ -1221,7 +1221,7 @@ (define-public mu
            gnupg                        ; for tests
            texinfo))
     (inputs
-     (list glib gmime xapian))
+     (list glib gmime guile-3.0 xapian))
     (arguments
      (list
       #:modules '((guix build meson-build-system)
@@ -1229,6 +1229,8 @@ (define-public mu
                   (guix build utils))
       #:imported-modules `(,@%meson-build-system-modules
                            (guix build emacs-utils))
+      #:configure-flags
+      #~(list (format #f "-Dguile-extension-dir=~a/lib" #$output))
       #:phases
       #~(modify-phases %standard-phases
           (add-after 'unpack 'patch-bin-references
@@ -1241,6 +1243,11 @@ (define-public mu
               (substitute* '("lib/tests/bench-indexer.cc"
                              "lib/utils/mu-test-utils.cc")
                 (("/bin/rm") (which "rm")))))
+          (add-after 'install 'fix-ffi
+            (lambda _
+              (substitute* (find-files #$output "mu.scm")
+                (("\"libguile-mu\"")
+                 (format #f "\"~a/lib/libguile-mu\"" #$output)))))
           (add-after 'install 'install-emacs-autoloads
             (lambda* (#:key outputs #:allow-other-keys)
               (emacs-generate-autoloads

base-commit: fc6bdaad57bf91609849623c5f485403c030cb49




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

* bug#67244: [PATCH] gnu: mu: Update to 1.10.8 and re-enable Guile bindings
  2023-11-17 17:21 [bug#67244] [PATCH] gnu: mu: Update to 1.10.8 and re-enable Guile bindings Thiago Jung Bauermann
@ 2023-11-23 16:20 ` Christopher Baines
  2023-11-23 16:48   ` [bug#67244] " Thiago Jung Bauermann
  0 siblings, 1 reply; 3+ messages in thread
From: Christopher Baines @ 2023-11-23 16:20 UTC (permalink / raw)
  To: Thiago Jung Bauermann; +Cc: 67244-done, guix-patches

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


Thiago Jung Bauermann <thiago.bauermann@linaro.org> writes:

> Version 1.10.8 fixes the installation of the Guile bindings, so we can
> enable it again.
>
> * gnu/packages/mail.scm (mu): Update to 1.10.7.
>   [inputs]: Add guile-3.0.
>   [arguments]: Add "-Dguile-extension-dir" to #:configure-flags.
>   Add fix-ffi phase to allow Guile module to load its native library.
>
> Change-Id: I2a8250d8d6298712ad116b1d88157a4344175722
> ---
>
> Hello,
>
> I find the Guile bindings useful, so I'd like to reenable them.
> I've tested that my script that uses them is working.
>
> Kind regards,
> Thiago
>
>  gnu/packages/mail.scm | 13 ++++++++++---
>  1 file changed, 10 insertions(+), 3 deletions(-)

Thanks for the patch, I've pushed this to master as
c26ed5d6b971af11894015979f1e260df571a2be.

Chris

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

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

* [bug#67244] [PATCH] gnu: mu: Update to 1.10.8 and re-enable Guile bindings
  2023-11-23 16:20 ` bug#67244: " Christopher Baines
@ 2023-11-23 16:48   ` Thiago Jung Bauermann
  0 siblings, 0 replies; 3+ messages in thread
From: Thiago Jung Bauermann @ 2023-11-23 16:48 UTC (permalink / raw)
  To: Christopher Baines; +Cc: 67244-done, 67244


Christopher Baines <mail@cbaines.net> writes:

> Thanks for the patch, I've pushed this to master as
> c26ed5d6b971af11894015979f1e260df571a2be.

Thank you!

-- 
Thiago




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

end of thread, other threads:[~2023-11-23 22:00 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-11-17 17:21 [bug#67244] [PATCH] gnu: mu: Update to 1.10.8 and re-enable Guile bindings Thiago Jung Bauermann
2023-11-23 16:20 ` bug#67244: " Christopher Baines
2023-11-23 16:48   ` [bug#67244] " Thiago Jung Bauermann

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.