GNU bug report logs - #61889
[PATCH] gnu: Add mairix. (WIP)

Previous Next

Package: guix-patches;

Reported by: Martin Marshall <law <at> martinmarshall.com>

Date: Wed, 1 Mar 2023 03:56:01 UTC

Severity: normal

Tags: patch

Done: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 61889 in the body.
You can then email your comments to 61889 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#61889; Package guix-patches. (Wed, 01 Mar 2023 03:56:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Martin Marshall <law <at> martinmarshall.com>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Wed, 01 Mar 2023 03:56:01 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Martin Marshall <law <at> martinmarshall.com>
To: guix-patches <at> gnu.org
Cc: Martin Marshall <law <at> martinmarshall.com>
Subject: [PATCH] gnu: Add mairix. (WIP)
Date: Tue, 28 Feb 2023 22:55:23 -0500
* gnu/packages/mail.scm (mairix): Add variable
---
Greetings,

Building with "--system=armhf" fails the check phase.  However,
building with "--system=i686" or "--system=aarch64" succeeds, as does
building without emulation on x86_64.

On an actual armhf device running a debian-based OS (a "Beaglebone
Black" single-board-computer), compilation via "./configure" and
"make" was successful.  And all tests passed "make check".  But that
only proves the upstream source can work on armhf.  It doesn't explain
why the "make check" fails for armhf on Guix.

Any suggestions or fixes for getting this package definition ready to
commit would be appreciated.

 gnu/packages/mail.scm | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
index 6ef4c6fdab..3cb3b4cc3f 100644
--- a/gnu/packages/mail.scm
+++ b/gnu/packages/mail.scm
@@ -408,6 +408,40 @@ (define-public mailutils
      ;; Libraries are under LGPLv3+, and programs under GPLv3+.
      (list license:gpl3+ license:lgpl3+))))
 
+(define-public mairix
+  (let ((commit "1cc06f4a73ba4b940008c1ffc398d2ac708cd6d6")
+        (revision "0"))
+    (package
+      (name "mairix")
+      (version (git-version "0.24" revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://github.com/vandry/mairix")
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "12bhmk5j77cl3vjda48cmdysq1c2yjzvfv6zm4hlky6d5g3l49d7"))))
+      (build-system gnu-build-system)
+      (arguments
+       (list #:parallel-tests? #f
+             #:phases #~(modify-phases %standard-phases
+                          (replace 'configure
+                            (lambda* (#:key inputs #:allow-other-keys)
+                              (setenv "LC_ALL" "en_US.UTF-8")
+                              (invoke "./configure"
+                                      (string-append "--prefix="
+                                                     #$output)))))))
+      (propagated-inputs (list zlib openssl bzip2 xz))
+      (native-inputs (list flex bison))
+      (home-page "https://github.com/vandry/mairix")
+      (synopsis "Program for indexing and searching email messages")
+      (description
+       "Mairix is a program for indexing and searching email messages stored in
+Maildir, MH, MMDF or mbox folders.")
+      (license license:gpl2))))
+
 (define-public go-gitlab.com-shackra-goimapnotify
   (package
     (name "go-gitlab.com-shackra-goimapnotify")
-- 
2.39.1





Information forwarded to guix-patches <at> gnu.org:
bug#61889; Package guix-patches. (Wed, 01 Mar 2023 04:03:01 GMT) Full text and rfc822 format available.

Message #8 received at 61889 <at> debbugs.gnu.org (full text, mbox):

From: Martin Marshall <law <at> martinmarshall.com>
To: Martin Marshall <law <at> martinmarshall.com>
Cc: 61889 <at> debbugs.gnu.org
Subject: Re: [PATCH] gnu: Add mairix. (WIP)
Date: Tue, 28 Feb 2023 23:02:46 -0500
Correction to the above, "--system=armhf" should read as
"--system=armhf-linux", and likewise for the other "--system="
arguments.

Best regards.

-- 
Martin Marshall
law <at> martinmarshall.com




Information forwarded to guix-patches <at> gnu.org:
bug#61889; Package guix-patches. (Fri, 10 Mar 2023 23:30:02 GMT) Full text and rfc822 format available.

Message #11 received at 61889 <at> debbugs.gnu.org (full text, mbox):

From: Martin Marshall <law <at> martinmarshall.com>
To: 61889 <at> debbugs.gnu.org
Subject: [PATCH] gnu: Add mairix.
Date: Fri, 10 Mar 2023 18:29:46 -0500
[Message part 1 (text/plain, inline)]
Per Guix QA, all builds succeed except for "i586-gnu".  Still not sure
why my system fails the check phase when emulating "armhf-linux".  Will
attribute to user-error.

Attached is an updated patch which builds successfully on my
x86_64-linux system.

Changes from the original patch:
* Moved propagated-inputs to inputs.
* Added perl to inputs.
* Removed (setenv "LC_ALL" "en_US.UTF-8") from configure phase.
* Added (supported-systems ...) at the end to reflect the systems
indicated by Guix QA as building correctly.

Best regards.

-- 
Martin Marshall
law <at> martinmarshall.com

[0001-Add-mairix.patch (text/x-patch, attachment)]

Reply sent to Nicolas Goaziou <mail <at> nicolasgoaziou.fr>:
You have taken responsibility. (Sun, 02 Apr 2023 21:18:02 GMT) Full text and rfc822 format available.

Notification sent to Martin Marshall <law <at> martinmarshall.com>:
bug acknowledged by developer. (Sun, 02 Apr 2023 21:18:02 GMT) Full text and rfc822 format available.

Message #16 received at 61889-done <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: Martin Marshall <law <at> martinmarshall.com>
Cc: 61889-done <at> debbugs.gnu.org
Subject: Re: [bug#61889] [PATCH] gnu: Add mairix.
Date: Sun, 02 Apr 2023 23:16:47 +0200
Hello,

Martin Marshall <law <at> martinmarshall.com> writes:

> Attached is an updated patch which builds successfully on my
> x86_64-linux system.

Applied. Thank you.
>

> * Added (supported-systems ...) at the end to reflect the systems
> indicated by Guix QA as building correctly.

I removed the supported-systems field since it is not strictly necessary here.

Regards,
-- 
Nicolas Goaziou




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Mon, 01 May 2023 11:24:14 GMT) Full text and rfc822 format available.

This bug report was last modified 358 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.