unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: gmime: Remove gpg to gpg2 patch.
@ 2016-07-25 11:47 ng0
  2016-07-25 15:21 ` Leo Famulari
  2016-07-26 23:44 ` Leo Famulari
  0 siblings, 2 replies; 8+ messages in thread
From: ng0 @ 2016-07-25 11:47 UTC (permalink / raw)
  To: guix-devel

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

gmime building broke with gpg@2.1 --build-gpg2-as-gpg or what it
was. This fixes it.
Please also fix the typo in the commit message ("peviously").

From 3bc7d2c8d94da0a81ecc16b2e12791f9ed466c95 Mon Sep 17 00:00:00 2001
From: ng0 <ng0@we.make.ritual.n0.is>
Date: Mon, 25 Jul 2016 11:38:37 +0000
Subject: [PATCH] gnu: gmime: Remove gpg to gpg2 patch.

* gnu/packages/mail.scm (gmime)[arguments]: Remove the patch which peviously
changed gpg to gpg2 in 'patch-paths-in-tests phase.
---
 gnu/packages/mail.scm | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
index 9214b73..f7a7623 100644
--- a/gnu/packages/mail.scm
+++ b/gnu/packages/mail.scm
@@ -15,6 +15,7 @@
 ;;; Copyright © 2016 Lukas Gradl <lgradl@openmailbox.org>
 ;;; Copyright © 2016 Alex Kost <alezost@gmail.com>
 ;;; Copyright © 2016 Troy Sankey <sankeytms@gmail.com>
+;;; Copyright © 2016 ng0 <ng0@we.make.ritual.n0.is>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -240,14 +241,14 @@ operating systems.")
           'unpack 'patch-paths-in-tests
           (lambda _
             ;; The test programs run several programs using 'system' with
-            ;; hard-coded paths.  Here we patch them all.  We also change "gpg"
-            ;; to "gpg2".  We use ISO-8859-1 here because test-iconv.c contains
+            ;; hard-coded paths.  Here we patch them all.
+            ;; We use ISO-8859-1 here because test-iconv.c contains
             ;; raw byte sequences in several different encodings.
             (with-fluids ((%default-port-encoding #f))
               (substitute* (find-files "tests" "\\.c$")
                 (("(system *\\(\")(/[^ ]*)" all pre prog-path)
                  (let* ((base (basename prog-path))
-                        (prog (which (if (string=? base "gpg") "gpg2" base))))
+                        (prog (which base)))
                    (string-append pre
                                   (or prog (error "not found: " base))))))))))))
     (home-page "http://spruce.sourceforge.net/gmime/")
-- 
2.9.1


-- 
♥Ⓐ  ng0
Current Keys: https://we.make.ritual.n0.is/ng0.txt
For non-prism friendly talk find me on http://www.psyced.org

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

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

* Re: [PATCH] gnu: gmime: Remove gpg to gpg2 patch.
  2016-07-25 11:47 [PATCH] gnu: gmime: Remove gpg to gpg2 patch ng0
@ 2016-07-25 15:21 ` Leo Famulari
  2016-07-26 12:35   ` Ludovic Courtès
  2016-07-26 17:48   ` Mark H Weaver
  2016-07-26 23:44 ` Leo Famulari
  1 sibling, 2 replies; 8+ messages in thread
From: Leo Famulari @ 2016-07-25 15:21 UTC (permalink / raw)
  To: ng0; +Cc: guix-devel

On Mon, Jul 25, 2016 at 11:47:51AM +0000, ng0 wrote:
>              (with-fluids ((%default-port-encoding #f))
>                (substitute* (find-files "tests" "\\.c$")
>                  (("(system *\\(\")(/[^ ]*)" all pre prog-path)
>                   (let* ((base (basename prog-path))
> -                        (prog (which (if (string=? base "gpg") "gpg2" base))))
> +                        (prog (which base)))
>                     (string-append pre
>                                    (or prog (error "not found: " base))))))))))))

I can confirm this fixes the build failure. But, I don't fully
understand the code that was changed. Can somebody double-check it?

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

* Re: [PATCH] gnu: gmime: Remove gpg to gpg2 patch.
  2016-07-25 15:21 ` Leo Famulari
@ 2016-07-26 12:35   ` Ludovic Courtès
  2016-07-26 14:35     ` Andreas Enge
  2016-07-26 17:48   ` Mark H Weaver
  1 sibling, 1 reply; 8+ messages in thread
From: Ludovic Courtès @ 2016-07-26 12:35 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Leo Famulari <leo@famulari.name> skribis:

> On Mon, Jul 25, 2016 at 11:47:51AM +0000, ng0 wrote:
>>              (with-fluids ((%default-port-encoding #f))
>>                (substitute* (find-files "tests" "\\.c$")
>>                  (("(system *\\(\")(/[^ ]*)" all pre prog-path)
>>                   (let* ((base (basename prog-path))
>> -                        (prog (which (if (string=? base "gpg") "gpg2" base))))
>> +                        (prog (which base)))
>>                     (string-append pre
>>                                    (or prog (error "not found: " base))))))))))))
>
> I can confirm this fixes the build failure. But, I don't fully
> understand the code that was changed. Can somebody double-check it?

LGTM.  The comment above this hunk should also be changed.

Thanks,
Ludo’.

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

* Re: [PATCH] gnu: gmime: Remove gpg to gpg2 patch.
  2016-07-26 12:35   ` Ludovic Courtès
@ 2016-07-26 14:35     ` Andreas Enge
  2016-07-26 23:59       ` Leo Famulari
  0 siblings, 1 reply; 8+ messages in thread
From: Andreas Enge @ 2016-07-26 14:35 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

Hello,

speaking of gmime and gpg, in the latest evaluation of master on hydra:
   http://hydra.gnu.org/eval/109021
gnupg@2.1 fails its tests on i686:
   http://hydra.gnu.org/build/1315032

Maybe someone involved in the recent gnupg changes could have a look?

I understand that the gmime failure is solved by the patch under
discussion; if not, please also have a look :-)

Andreas

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

* Re: [PATCH] gnu: gmime: Remove gpg to gpg2 patch.
  2016-07-25 15:21 ` Leo Famulari
  2016-07-26 12:35   ` Ludovic Courtès
@ 2016-07-26 17:48   ` Mark H Weaver
  1 sibling, 0 replies; 8+ messages in thread
From: Mark H Weaver @ 2016-07-26 17:48 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Leo Famulari <leo@famulari.name> writes:

> On Mon, Jul 25, 2016 at 11:47:51AM +0000, ng0 wrote:
>>              (with-fluids ((%default-port-encoding #f))
>>                (substitute* (find-files "tests" "\\.c$")
>>                  (("(system *\\(\")(/[^ ]*)" all pre prog-path)
>>                   (let* ((base (basename prog-path))
>> -                        (prog (which (if (string=? base "gpg") "gpg2" base))))
>> +                        (prog (which base)))
>>                     (string-append pre
>>                                    (or prog (error "not found: " base))))))))))))
>
> I can confirm this fixes the build failure. But, I don't fully
> understand the code that was changed. Can somebody double-check it?

The patch looks good to me.

     Thanks!
       Mark

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

* Re: [PATCH] gnu: gmime: Remove gpg to gpg2 patch.
  2016-07-25 11:47 [PATCH] gnu: gmime: Remove gpg to gpg2 patch ng0
  2016-07-25 15:21 ` Leo Famulari
@ 2016-07-26 23:44 ` Leo Famulari
  1 sibling, 0 replies; 8+ messages in thread
From: Leo Famulari @ 2016-07-26 23:44 UTC (permalink / raw)
  To: ng0; +Cc: guix-devel

On Mon, Jul 25, 2016 at 11:47:51AM +0000, ng0 wrote:
> Subject: [PATCH] gnu: gmime: Remove gpg to gpg2 patch.
> 
> * gnu/packages/mail.scm (gmime)[arguments]: Remove the patch which peviously
> changed gpg to gpg2 in 'patch-paths-in-tests phase.

Thanks, pushed as ae83dcf6ba9687b7ff123fdd4a81eb952b7b7801 !

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

* Re: [PATCH] gnu: gmime: Remove gpg to gpg2 patch.
  2016-07-26 14:35     ` Andreas Enge
@ 2016-07-26 23:59       ` Leo Famulari
  2016-07-27  8:26         ` Andreas Enge
  0 siblings, 1 reply; 8+ messages in thread
From: Leo Famulari @ 2016-07-26 23:59 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

On Tue, Jul 26, 2016 at 04:35:28PM +0200, Andreas Enge wrote:
> Hello,
> 
> speaking of gmime and gpg, in the latest evaluation of master on hydra:
>    http://hydra.gnu.org/eval/109021
> gnupg@2.1 fails its tests on i686:
>    http://hydra.gnu.org/build/1315032
> 
> Maybe someone involved in the recent gnupg changes could have a look?
> 
> I understand that the gmime failure is solved by the patch under
> discussion; if not, please also have a look :-)

I'm confused — that gnupg@2.1 build looks like it succeeded. I read the
log of the test suite and didn't see any failures that were ignored.

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

* Re: [PATCH] gnu: gmime: Remove gpg to gpg2 patch.
  2016-07-26 23:59       ` Leo Famulari
@ 2016-07-27  8:26         ` Andreas Enge
  0 siblings, 0 replies; 8+ messages in thread
From: Andreas Enge @ 2016-07-27  8:26 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

On Tue, Jul 26, 2016 at 07:59:32PM -0400, Leo Famulari wrote:
> I'm confused — that gnupg@2.1 build looks like it succeeded. I read the
> log of the test suite and didn't see any failures that were ignored.

It built successfully a few hours after my report. Maybe someone restarted
it, and it was only a transient failure. Thanks for checking!

Andreas

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

end of thread, other threads:[~2016-07-27  8:26 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-25 11:47 [PATCH] gnu: gmime: Remove gpg to gpg2 patch ng0
2016-07-25 15:21 ` Leo Famulari
2016-07-26 12:35   ` Ludovic Courtès
2016-07-26 14:35     ` Andreas Enge
2016-07-26 23:59       ` Leo Famulari
2016-07-27  8:26         ` Andreas Enge
2016-07-26 17:48   ` Mark H Weaver
2016-07-26 23:44 ` Leo Famulari

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).