unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
@ 2016-02-04 19:58 Mark H Weaver
  2016-02-04 20:22 ` Mark H Weaver
  0 siblings, 1 reply; 13+ messages in thread
From: Mark H Weaver @ 2016-02-04 19:58 UTC (permalink / raw)
  To: 22558

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

gnupg-2.1.11 on x86_64 failed its tests many times on Hydra:

  http://hydra.gnu.org:3000/build/990803#tabs-buildsteps

On all other architectures (i686, armhf, mips64el) it succeeded on the
first try.  So far, we've heard no reports of it failing for anyone else
on x86_64.  Ludovic built it 3 times on his x86_64 machine and it never
failed.

It failed in at least two different ways:

* With only 1 test failure: gpgtar.test.  This happened on at least two
  different build slaves.

* With 14 test failures.  This happened at least twice, on
  hydra.gnunet.org.

I've attached 4 of the failure logs retrieved from Hydra.  There are two
logs of each of the two failure modes described above.

It would be good to understand what happened here.

     Mark



[-- Attachment #2: hydra gnupg-x86_64 failure logs --]
[-- Type: application/octet-stream, Size: 30788 bytes --]

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-04 19:58 bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra Mark H Weaver
@ 2016-02-04 20:22 ` Mark H Weaver
  2016-02-04 20:40   ` Mark H Weaver
  0 siblings, 1 reply; 13+ messages in thread
From: Mark H Weaver @ 2016-02-04 20:22 UTC (permalink / raw)
  To: 22558

Mark H Weaver <mhw@netris.org> writes:

> gnupg-2.1.11 on x86_64 failed its tests many times on Hydra:
>
>   http://hydra.gnu.org:3000/build/990803#tabs-buildsteps
>
> On all other architectures (i686, armhf, mips64el) it succeeded on the
> first try.  So far, we've heard no reports of it failing for anyone else
> on x86_64.  Ludovic built it 3 times on his x86_64 machine and it never
> failed.
>
> It failed in at least two different ways:
>
> * With only 1 test failure: gpgtar.test.  This happened on at least two
>   different build slaves.
>
> * With 14 test failures.  This happened at least twice, on
>   hydra.gnunet.org.

Eric Bavier reported on #guix that it failed to build on his machine
twice in a row, with 14 test failures.

       Mark

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-04 20:22 ` Mark H Weaver
@ 2016-02-04 20:40   ` Mark H Weaver
  0 siblings, 0 replies; 13+ messages in thread
From: Mark H Weaver @ 2016-02-04 20:40 UTC (permalink / raw)
  To: 22558

Eric Bavier found this:

  https://bugs.gnupg.org/gnupg/issue2229

Title: "make check 14 0f 35 tests failed"

     Mark


PS: It doesn't load in our IceCat with the error "Cannot communicate
    securely with peer: no common encryption algorithm(s). (Error code:
    ssl_error_no_cypher_overlap)", but it works in Epiphany.

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
       [not found]     ` <87d1scxkd6.fsf@netris.org>
@ 2016-02-05 15:25       ` Eric Bavier
  2016-02-05 18:58         ` Leo Famulari
  2016-02-07 20:50         ` Ludovic Courtès
  0 siblings, 2 replies; 13+ messages in thread
From: Eric Bavier @ 2016-02-05 15:25 UTC (permalink / raw)
  To: 22558

> > * With 14 test failures.  This happened at least twice, on
> >   hydra.gnunet.org.
> 
> Eric Bavier reported on #guix that it failed to build on his machine
> twice in a row, with 14 test failures.

I used 'git bisect' on the gnupg repository to narrow the failures down. 
  Commit ee87c653bf is the commit that introduces the failures.

Knowing this however doesn't explain why Ludo was not able to reproduce 
on his machine or why Hydra was eventually able to build it 
successfully.

-- 
`~Eric

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-05 15:25       ` Eric Bavier
@ 2016-02-05 18:58         ` Leo Famulari
  2016-02-07 20:50         ` Ludovic Courtès
  1 sibling, 0 replies; 13+ messages in thread
From: Leo Famulari @ 2016-02-05 18:58 UTC (permalink / raw)
  To: Eric Bavier; +Cc: 22558

On Fri, Feb 05, 2016 at 09:25:10AM -0600, Eric Bavier wrote:
> >> * With 14 test failures.  This happened at least twice, on
> >>   hydra.gnunet.org.
> >
> >Eric Bavier reported on #guix that it failed to build on his machine
> >twice in a row, with 14 test failures.
> 
> I used 'git bisect' on the gnupg repository to narrow the failures down.
> Commit ee87c653bf is the commit that introduces the failures.
> 
> Knowing this however doesn't explain why Ludo was not able to reproduce on
> his machine or why Hydra was eventually able to build it successfully.

It fails on my x86_64 machine as well, also with those 14 test failures.
Will somebody for whom it builds successfully see if it builds
reproducibly on their machine?

> 
> -- 
> `~Eric
> 
> 
> 

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-05 15:25       ` Eric Bavier
  2016-02-05 18:58         ` Leo Famulari
@ 2016-02-07 20:50         ` Ludovic Courtès
  2016-02-12 23:22           ` Mark H Weaver
  1 sibling, 1 reply; 13+ messages in thread
From: Ludovic Courtès @ 2016-02-07 20:50 UTC (permalink / raw)
  To: Eric Bavier; +Cc: 22558

Eric Bavier <ericbavier@openmailbox.org> skribis:

>> > * With 14 test failures.  This happened at least twice, on
>> >   hydra.gnunet.org.
>>
>> Eric Bavier reported on #guix that it failed to build on his machine
>> twice in a row, with 14 test failures.
>
> I used 'git bisect' on the gnupg repository to narrow the failures
> down. Commit ee87c653bf is the commit that introduces the failures.

Thanks for digging!  Could you report it upstream?

> Knowing this however doesn't explain why Ludo was not able to
> reproduce on his machine or why Hydra was eventually able to build it
> successfully.

This could be due to assembly using instructions available on one
machine and not the other.

Ludo’.

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-07 20:50         ` Ludovic Courtès
@ 2016-02-12 23:22           ` Mark H Weaver
  2016-02-13  0:12             ` Leo Famulari
  0 siblings, 1 reply; 13+ messages in thread
From: Mark H Weaver @ 2016-02-12 23:22 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: Eric Bavier, 22558

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

ludo@gnu.org (Ludovic Courtès) writes:

> Eric Bavier <ericbavier@openmailbox.org> skribis:
>
>>> > * With 14 test failures.  This happened at least twice, on
>>> >   hydra.gnunet.org.
>>>
>>> Eric Bavier reported on #guix that it failed to build on his machine
>>> twice in a row, with 14 test failures.
>>
>> I used 'git bisect' on the gnupg repository to narrow the failures
>> down. Commit ee87c653bf is the commit that introduces the failures.
>
> Thanks for digging!  Could you report it upstream?

I reported it upstream, and now they have a fix in their repo.
I've attached a patch to Guix which adds this upstream patch.

Eric, would you be willing to test it and see if it fixes the problem
for you?

      Mark


[-- Attachment #2: [PATCH] gnu: gnupg: Add upstream fix for test failures on x86_64 --]
[-- Type: text/x-patch, Size: 7514 bytes --]

From bd025179efca777678f39a782f851e07719db05f Mon Sep 17 00:00:00 2001
From: Mark H Weaver <mhw@netris.org>
Date: Fri, 12 Feb 2016 18:19:05 -0500
Subject: [PATCH] gnu: gnupg: Add upstream fix for test failures on x86_64.

Fixes <https://debbugs.gnu.org/22558>.

* gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch: New
  file.
* gnu-system.am (dist_patch_DATA): Add it.
* gnu/packages/gnupg.scm (gnupg)[source]: Add patch.
---
 gnu-system.am                                      |   1 +
 gnu/packages/gnupg.scm                             |   7 +-
 ...gnupg-simple-query-ignore-status-messages.patch | 142 +++++++++++++++++++++
 3 files changed, 148 insertions(+), 2 deletions(-)
 create mode 100644 gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch

diff --git a/gnu-system.am b/gnu-system.am
index 9a2988c..3a47e00 100644
--- a/gnu-system.am
+++ b/gnu-system.am
@@ -506,6 +506,7 @@ dist_patch_DATA =						\
   gnu/packages/patches/gmp-arm-asm-nothumb.patch		\
   gnu/packages/patches/gmp-faulty-test.patch			\
   gnu/packages/patches/gnucash-price-quotes-perl.patch		\
+  gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch \
   gnu/packages/patches/gobject-introspection-absolute-shlib-path.patch \
   gnu/packages/patches/gobject-introspection-cc.patch		\
   gnu/packages/patches/gobject-introspection-girepository.patch	\
diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
index a35e8fc..6cd37bc 100644
--- a/gnu/packages/gnupg.scm
+++ b/gnu/packages/gnupg.scm
@@ -2,7 +2,7 @@
 ;;; Copyright © 2012, 2013, 2014, 2015 Ludovic Courtès <ludo@gnu.org>
 ;;; Copyright © 2013, 2015 Andreas Enge <andreas@enge.fr>
 ;;; Copyright © 2014 Eric Bavier <bavier@member.fsf.org>
-;;; Copyright © 2014, 2015 Mark H Weaver <mhw@netris.org>
+;;; Copyright © 2014, 2015, 2016 Mark H Weaver <mhw@netris.org>
 ;;; Copyright © 2015 Paul van der Walt <paul@denknerd.org>
 ;;; Copyright © 2015, 2016 Efraim Flashner <efraim@flashner.co.il>
 ;;; Copyright © 2016 Christopher Allan Webber <cwebber@dustycloud.org>
@@ -206,7 +206,10 @@ compatible to GNU Pth.")
                                   ".tar.bz2"))
               (sha256
                (base32
-                "06mn2viiwsyq991arh5i5fhr9jyxq2bi0jkdj7ndfisxihngpc5p"))))
+                "06mn2viiwsyq991arh5i5fhr9jyxq2bi0jkdj7ndfisxihngpc5p"))
+              (patches
+               (list (search-patch
+                      "gnupg-simple-query-ignore-status-messages.patch")))))
     (build-system gnu-build-system)
     (native-inputs
      `(("pkg-config" ,pkg-config)))
diff --git a/gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch b/gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch
new file mode 100644
index 0000000..153f71c
--- /dev/null
+++ b/gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch
@@ -0,0 +1,142 @@
+Copied from upstream:
+http://git.gnupg.org/cgi-bin/gitweb.cgi?p=gnupg.git;a=commitdiff;h=acac103ba5772ae738ce5409d17feab80596cde6
+
+Fixes: https://debbugs.gnu.org/22558
+Upstream bug: https://bugs.gnupg.org/gnupg/issue2229
+
+From acac103ba5772ae738ce5409d17feab80596cde6 Mon Sep 17 00:00:00 2001
+From: "Neal H. Walfield" <neal@g10code.com>
+Date: Fri, 12 Feb 2016 22:12:21 +0100
+Subject: [PATCH] common: Change simple_query to ignore status messages.
+
+* common/simple-pwquery.c (simple_query): Ignore status messages.
+
+--
+Signed-off-by: Neal H. Walfield <neal@g10code.com>
+GnuPG-bug-id: 2229
+---
+ common/simple-pwquery.c | 95 ++++++++++++++++++++++++++++++++++---------------
+ 1 file changed, 67 insertions(+), 28 deletions(-)
+
+diff --git a/common/simple-pwquery.c b/common/simple-pwquery.c
+index 90d04c0..b2d666c 100644
+--- a/common/simple-pwquery.c
++++ b/common/simple-pwquery.c
+@@ -618,6 +618,7 @@ simple_query (const char *query)
+   int fd = -1;
+   int nread;
+   char response[500];
++  int have = 0;
+   int rc;
+ 
+   rc = agent_open (&fd);
+@@ -628,40 +629,78 @@ simple_query (const char *query)
+   if (rc)
+     goto leave;
+ 
+-  /* get response */
+-  nread = readline (fd, response, 499);
+-  if (nread < 0)
+-    {
+-      rc = -nread;
+-      goto leave;
+-    }
+-  if (nread < 3)
++  while (1)
+     {
+-      rc = SPWQ_PROTOCOL_ERROR;
+-      goto leave;
+-    }
++      if (! have || ! strchr (response, '\n'))
++        /* get response */
++        {
++          nread = readline (fd, &response[have],
++                            sizeof (response) - 1 /* NUL */ - have);
++          if (nread < 0)
++            {
++              rc = -nread;
++              goto leave;
++            }
++          have += nread;
++          if (have < 3)
++            {
++              rc = SPWQ_PROTOCOL_ERROR;
++              goto leave;
++            }
++          response[have] = 0;
++        }
+ 
+-  if (response[0] == 'O' && response[1] == 'K')
+-    /* OK, do nothing.  */;
+-  else if ((nread > 7 && !memcmp (response, "ERR 111", 7)
+-            && (response[7] == ' ' || response[7] == '\n') )
+-           || ((nread > 4 && !memcmp (response, "ERR ", 4)
+-                && (strtoul (response+4, NULL, 0) & 0xffff) == 99)) )
+-    {
+-      /* 111 is the old Assuan code for canceled which might still
+-         be in use by old installations. 99 is GPG_ERR_CANCELED as
+-         used by modern gpg-agents; 0xffff is used to mask out the
+-         error source.  */
++      if (response[0] == 'O' && response[1] == 'K')
++        /* OK, do nothing.  */;
++      else if ((nread > 7 && !memcmp (response, "ERR 111", 7)
++                && (response[7] == ' ' || response[7] == '\n') )
++               || ((nread > 4 && !memcmp (response, "ERR ", 4)
++                    && (strtoul (response+4, NULL, 0) & 0xffff) == 99)) )
++        {
++          /* 111 is the old Assuan code for canceled which might still
++             be in use by old installations. 99 is GPG_ERR_CANCELED as
++             used by modern gpg-agents; 0xffff is used to mask out the
++             error source.  */
+ #ifdef SPWQ_USE_LOGGING
+-      log_info (_("canceled by user\n") );
++          log_info (_("canceled by user\n") );
+ #endif
+-    }
+-  else
+-    {
++        }
++      else if (response[0] == 'S' && response[1] == ' ')
++        {
++          char *nextline;
++          int consumed;
++
++          nextline = strchr (response, '\n');
++          if (! nextline)
++            /* Point to the NUL.  */
++            nextline = &response[have];
++          else
++            /* Move past the \n.  */
++            nextline ++;
++
++          consumed = (size_t) nextline - (size_t) response;
++
++          /* Skip any additional newlines.  */
++          while (consumed < have && response[consumed] == '\n')
++            consumed ++;
++
++          have -= consumed;
++
++          if (have)
++            memmove (response, &response[consumed], have + 1);
++
++          continue;
++        }
++      else
++        {
+ #ifdef SPWQ_USE_LOGGING
+-      log_error (_("problem with the agent\n"));
++          log_error (_("problem with the agent (unexpected response \"%s\"\n"),
++                     response);
+ #endif
+-      rc = SPWQ_ERR_RESPONSE;
++          rc = SPWQ_ERR_RESPONSE;
++        }
++
++      break;
+     }
+ 
+  leave:
+-- 
+2.6.3
+
-- 
2.6.3


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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-12 23:22           ` Mark H Weaver
@ 2016-02-13  0:12             ` Leo Famulari
  2016-02-13  2:13               ` Mark H Weaver
  0 siblings, 1 reply; 13+ messages in thread
From: Leo Famulari @ 2016-02-13  0:12 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: Eric Bavier, 22558

On Fri, Feb 12, 2016 at 06:22:46PM -0500, Mark H Weaver wrote:
> ludo@gnu.org (Ludovic Courtès) writes:
> 
> > Eric Bavier <ericbavier@openmailbox.org> skribis:
> >
> >>> > * With 14 test failures.  This happened at least twice, on
> >>> >   hydra.gnunet.org.
> >>>
> >>> Eric Bavier reported on #guix that it failed to build on his machine
> >>> twice in a row, with 14 test failures.
> >>
> >> I used 'git bisect' on the gnupg repository to narrow the failures
> >> down. Commit ee87c653bf is the commit that introduces the failures.
> >
> > Thanks for digging!  Could you report it upstream?
> 
> I reported it upstream, and now they have a fix in their repo.
> I've attached a patch to Guix which adds this upstream patch.
> 
> Eric, would you be willing to test it and see if it fixes the problem
> for you?

I'm not Eric, but with this patch, I can successfully build on the
machine that was previously exhibiting the failures.

> 
>       Mark
> 

> From bd025179efca777678f39a782f851e07719db05f Mon Sep 17 00:00:00 2001
> From: Mark H Weaver <mhw@netris.org>
> Date: Fri, 12 Feb 2016 18:19:05 -0500
> Subject: [PATCH] gnu: gnupg: Add upstream fix for test failures on x86_64.
> 
> Fixes <https://debbugs.gnu.org/22558>.
> 
> * gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch: New
>   file.
> * gnu-system.am (dist_patch_DATA): Add it.
> * gnu/packages/gnupg.scm (gnupg)[source]: Add patch.
> ---
>  gnu-system.am                                      |   1 +
>  gnu/packages/gnupg.scm                             |   7 +-
>  ...gnupg-simple-query-ignore-status-messages.patch | 142 +++++++++++++++++++++
>  3 files changed, 148 insertions(+), 2 deletions(-)
>  create mode 100644 gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch
> 
> diff --git a/gnu-system.am b/gnu-system.am
> index 9a2988c..3a47e00 100644
> --- a/gnu-system.am
> +++ b/gnu-system.am
> @@ -506,6 +506,7 @@ dist_patch_DATA =						\
>    gnu/packages/patches/gmp-arm-asm-nothumb.patch		\
>    gnu/packages/patches/gmp-faulty-test.patch			\
>    gnu/packages/patches/gnucash-price-quotes-perl.patch		\
> +  gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch \
>    gnu/packages/patches/gobject-introspection-absolute-shlib-path.patch \
>    gnu/packages/patches/gobject-introspection-cc.patch		\
>    gnu/packages/patches/gobject-introspection-girepository.patch	\
> diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
> index a35e8fc..6cd37bc 100644
> --- a/gnu/packages/gnupg.scm
> +++ b/gnu/packages/gnupg.scm
> @@ -2,7 +2,7 @@
>  ;;; Copyright © 2012, 2013, 2014, 2015 Ludovic Courtès <ludo@gnu.org>
>  ;;; Copyright © 2013, 2015 Andreas Enge <andreas@enge.fr>
>  ;;; Copyright © 2014 Eric Bavier <bavier@member.fsf.org>
> -;;; Copyright © 2014, 2015 Mark H Weaver <mhw@netris.org>
> +;;; Copyright © 2014, 2015, 2016 Mark H Weaver <mhw@netris.org>
>  ;;; Copyright © 2015 Paul van der Walt <paul@denknerd.org>
>  ;;; Copyright © 2015, 2016 Efraim Flashner <efraim@flashner.co.il>
>  ;;; Copyright © 2016 Christopher Allan Webber <cwebber@dustycloud.org>
> @@ -206,7 +206,10 @@ compatible to GNU Pth.")
>                                    ".tar.bz2"))
>                (sha256
>                 (base32
> -                "06mn2viiwsyq991arh5i5fhr9jyxq2bi0jkdj7ndfisxihngpc5p"))))
> +                "06mn2viiwsyq991arh5i5fhr9jyxq2bi0jkdj7ndfisxihngpc5p"))
> +              (patches
> +               (list (search-patch
> +                      "gnupg-simple-query-ignore-status-messages.patch")))))
>      (build-system gnu-build-system)
>      (native-inputs
>       `(("pkg-config" ,pkg-config)))
> diff --git a/gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch b/gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch
> new file mode 100644
> index 0000000..153f71c
> --- /dev/null
> +++ b/gnu/packages/patches/gnupg-simple-query-ignore-status-messages.patch
> @@ -0,0 +1,142 @@
> +Copied from upstream:
> +http://git.gnupg.org/cgi-bin/gitweb.cgi?p=gnupg.git;a=commitdiff;h=acac103ba5772ae738ce5409d17feab80596cde6
> +
> +Fixes: https://debbugs.gnu.org/22558
> +Upstream bug: https://bugs.gnupg.org/gnupg/issue2229
> +
> +From acac103ba5772ae738ce5409d17feab80596cde6 Mon Sep 17 00:00:00 2001
> +From: "Neal H. Walfield" <neal@g10code.com>
> +Date: Fri, 12 Feb 2016 22:12:21 +0100
> +Subject: [PATCH] common: Change simple_query to ignore status messages.
> +
> +* common/simple-pwquery.c (simple_query): Ignore status messages.
> +
> +--
> +Signed-off-by: Neal H. Walfield <neal@g10code.com>
> +GnuPG-bug-id: 2229
> +---
> + common/simple-pwquery.c | 95 ++++++++++++++++++++++++++++++++++---------------
> + 1 file changed, 67 insertions(+), 28 deletions(-)
> +
> +diff --git a/common/simple-pwquery.c b/common/simple-pwquery.c
> +index 90d04c0..b2d666c 100644
> +--- a/common/simple-pwquery.c
> ++++ b/common/simple-pwquery.c
> +@@ -618,6 +618,7 @@ simple_query (const char *query)
> +   int fd = -1;
> +   int nread;
> +   char response[500];
> ++  int have = 0;
> +   int rc;
> + 
> +   rc = agent_open (&fd);
> +@@ -628,40 +629,78 @@ simple_query (const char *query)
> +   if (rc)
> +     goto leave;
> + 
> +-  /* get response */
> +-  nread = readline (fd, response, 499);
> +-  if (nread < 0)
> +-    {
> +-      rc = -nread;
> +-      goto leave;
> +-    }
> +-  if (nread < 3)
> ++  while (1)
> +     {
> +-      rc = SPWQ_PROTOCOL_ERROR;
> +-      goto leave;
> +-    }
> ++      if (! have || ! strchr (response, '\n'))
> ++        /* get response */
> ++        {
> ++          nread = readline (fd, &response[have],
> ++                            sizeof (response) - 1 /* NUL */ - have);
> ++          if (nread < 0)
> ++            {
> ++              rc = -nread;
> ++              goto leave;
> ++            }
> ++          have += nread;
> ++          if (have < 3)
> ++            {
> ++              rc = SPWQ_PROTOCOL_ERROR;
> ++              goto leave;
> ++            }
> ++          response[have] = 0;
> ++        }
> + 
> +-  if (response[0] == 'O' && response[1] == 'K')
> +-    /* OK, do nothing.  */;
> +-  else if ((nread > 7 && !memcmp (response, "ERR 111", 7)
> +-            && (response[7] == ' ' || response[7] == '\n') )
> +-           || ((nread > 4 && !memcmp (response, "ERR ", 4)
> +-                && (strtoul (response+4, NULL, 0) & 0xffff) == 99)) )
> +-    {
> +-      /* 111 is the old Assuan code for canceled which might still
> +-         be in use by old installations. 99 is GPG_ERR_CANCELED as
> +-         used by modern gpg-agents; 0xffff is used to mask out the
> +-         error source.  */
> ++      if (response[0] == 'O' && response[1] == 'K')
> ++        /* OK, do nothing.  */;
> ++      else if ((nread > 7 && !memcmp (response, "ERR 111", 7)
> ++                && (response[7] == ' ' || response[7] == '\n') )
> ++               || ((nread > 4 && !memcmp (response, "ERR ", 4)
> ++                    && (strtoul (response+4, NULL, 0) & 0xffff) == 99)) )
> ++        {
> ++          /* 111 is the old Assuan code for canceled which might still
> ++             be in use by old installations. 99 is GPG_ERR_CANCELED as
> ++             used by modern gpg-agents; 0xffff is used to mask out the
> ++             error source.  */
> + #ifdef SPWQ_USE_LOGGING
> +-      log_info (_("canceled by user\n") );
> ++          log_info (_("canceled by user\n") );
> + #endif
> +-    }
> +-  else
> +-    {
> ++        }
> ++      else if (response[0] == 'S' && response[1] == ' ')
> ++        {
> ++          char *nextline;
> ++          int consumed;
> ++
> ++          nextline = strchr (response, '\n');
> ++          if (! nextline)
> ++            /* Point to the NUL.  */
> ++            nextline = &response[have];
> ++          else
> ++            /* Move past the \n.  */
> ++            nextline ++;
> ++
> ++          consumed = (size_t) nextline - (size_t) response;
> ++
> ++          /* Skip any additional newlines.  */
> ++          while (consumed < have && response[consumed] == '\n')
> ++            consumed ++;
> ++
> ++          have -= consumed;
> ++
> ++          if (have)
> ++            memmove (response, &response[consumed], have + 1);
> ++
> ++          continue;
> ++        }
> ++      else
> ++        {
> + #ifdef SPWQ_USE_LOGGING
> +-      log_error (_("problem with the agent\n"));
> ++          log_error (_("problem with the agent (unexpected response \"%s\"\n"),
> ++                     response);
> + #endif
> +-      rc = SPWQ_ERR_RESPONSE;
> ++          rc = SPWQ_ERR_RESPONSE;
> ++        }
> ++
> ++      break;
> +     }
> + 
> +  leave:
> +-- 
> +2.6.3
> +
> -- 
> 2.6.3
> 

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-13  0:12             ` Leo Famulari
@ 2016-02-13  2:13               ` Mark H Weaver
  2016-02-13  4:02                 ` Eric Bavier
  0 siblings, 1 reply; 13+ messages in thread
From: Mark H Weaver @ 2016-02-13  2:13 UTC (permalink / raw)
  To: Leo Famulari; +Cc: Eric Bavier, 22558-done

Leo Famulari <leo@famulari.name> writes:

> On Fri, Feb 12, 2016 at 06:22:46PM -0500, Mark H Weaver wrote:
>> ludo@gnu.org (Ludovic Courtès) writes:
>> 
>> > Eric Bavier <ericbavier@openmailbox.org> skribis:
>> >
>> >>> > * With 14 test failures.  This happened at least twice, on
>> >>> >   hydra.gnunet.org.
>> >>>
>> >>> Eric Bavier reported on #guix that it failed to build on his machine
>> >>> twice in a row, with 14 test failures.
>> >>
>> >> I used 'git bisect' on the gnupg repository to narrow the failures
>> >> down. Commit ee87c653bf is the commit that introduces the failures.
>> >
>> > Thanks for digging!  Could you report it upstream?
>> 
>> I reported it upstream, and now they have a fix in their repo.
>> I've attached a patch to Guix which adds this upstream patch.
>> 
>> Eric, would you be willing to test it and see if it fixes the problem
>> for you?
>
> I'm not Eric, but with this patch, I can successfully build on the
> machine that was previously exhibiting the failures.

Thanks!  I went ahead and pushed this to master, commit 3a7261bf96.
I'm closing this bug now.

      Mark

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-13  2:13               ` Mark H Weaver
@ 2016-02-13  4:02                 ` Eric Bavier
  2016-02-13  7:41                   ` Leo Famulari
  0 siblings, 1 reply; 13+ messages in thread
From: Eric Bavier @ 2016-02-13  4:02 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: 22558-done

On Fri, 12 Feb 2016 21:13:55 -0500
Mark H Weaver <mhw@netris.org> wrote:

> Leo Famulari <leo@famulari.name> writes:
> 
> > On Fri, Feb 12, 2016 at 06:22:46PM -0500, Mark H Weaver wrote:
> >> ludo@gnu.org (Ludovic Courtès) writes:
> >> 
> >> > Eric Bavier <ericbavier@openmailbox.org> skribis:
> >> >
> >> >>> > * With 14 test failures.  This happened at least twice, on
> >> >>> >   hydra.gnunet.org.
> >> >>>
> >> >>> Eric Bavier reported on #guix that it failed to build on his machine
> >> >>> twice in a row, with 14 test failures.
> >> >>
> >> >> I used 'git bisect' on the gnupg repository to narrow the failures
> >> >> down. Commit ee87c653bf is the commit that introduces the failures.
> >> >
> >> > Thanks for digging!  Could you report it upstream?
> >> 
> >> I reported it upstream, and now they have a fix in their repo.
> >> I've attached a patch to Guix which adds this upstream patch.
> >> 
> >> Eric, would you be willing to test it and see if it fixes the problem
> >> for you?
> >
> > I'm not Eric, but with this patch, I can successfully build on the
> > machine that was previously exhibiting the failures.
> 
> Thanks!  I went ahead and pushed this to master, commit 3a7261bf96.
> I'm closing this bug now.

I realize I'm a bit late, but I just wanted to let you know that this
appears to fix the gnupg-2.1.11 build for me too:

@ build-succeeded /gnu/store/qyjrqxnrka55p5g5vh1p80ac1rxn5nfm-gnupg-2.1.11.drv -
/gnu/store/w6fbikpwzpjkwvy6z1hwbszq0x83yspv-gnupg-2.1.11

But I had to add "#:use-module (gnu packages)" to the
gnu/packages/gnupg.scm before it would build, otherwise "could not
find symbol search-path".

Cheers,
`~Eric

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-13  4:02                 ` Eric Bavier
@ 2016-02-13  7:41                   ` Leo Famulari
  2016-02-13  7:46                     ` Leo Famulari
  2016-02-13 14:36                     ` Mark H Weaver
  0 siblings, 2 replies; 13+ messages in thread
From: Leo Famulari @ 2016-02-13  7:41 UTC (permalink / raw)
  To: Eric Bavier; +Cc: 22558-done

On Fri, Feb 12, 2016 at 10:02:32PM -0600, Eric Bavier wrote:
> On Fri, 12 Feb 2016 21:13:55 -0500
> Mark H Weaver <mhw@netris.org> wrote:
> 
> > Leo Famulari <leo@famulari.name> writes:
> > 
> > > On Fri, Feb 12, 2016 at 06:22:46PM -0500, Mark H Weaver wrote:
> > >> ludo@gnu.org (Ludovic Courtès) writes:
> > >> 
> > >> > Eric Bavier <ericbavier@openmailbox.org> skribis:
> > >> >
> > >> >>> > * With 14 test failures.  This happened at least twice, on
> > >> >>> >   hydra.gnunet.org.
> > >> >>>
> > >> >>> Eric Bavier reported on #guix that it failed to build on his machine
> > >> >>> twice in a row, with 14 test failures.
> > >> >>
> > >> >> I used 'git bisect' on the gnupg repository to narrow the failures
> > >> >> down. Commit ee87c653bf is the commit that introduces the failures.
> > >> >
> > >> > Thanks for digging!  Could you report it upstream?
> > >> 
> > >> I reported it upstream, and now they have a fix in their repo.
> > >> I've attached a patch to Guix which adds this upstream patch.
> > >> 
> > >> Eric, would you be willing to test it and see if it fixes the problem
> > >> for you?
> > >
> > > I'm not Eric, but with this patch, I can successfully build on the
> > > machine that was previously exhibiting the failures.
> > 
> > Thanks!  I went ahead and pushed this to master, commit 3a7261bf96.
> > I'm closing this bug now.
> 
> I realize I'm a bit late, but I just wanted to let you know that this
> appears to fix the gnupg-2.1.11 build for me too:
> 
> @ build-succeeded /gnu/store/qyjrqxnrka55p5g5vh1p80ac1rxn5nfm-gnupg-2.1.11.drv -
> /gnu/store/w6fbikpwzpjkwvy6z1hwbszq0x83yspv-gnupg-2.1.11
> 
> But I had to add "#:use-module (gnu packages)" to the
> gnu/packages/gnupg.scm before it would build, otherwise "could not
> find symbol search-path".

Weird, that did not happen to me when building from 3a7261bf96b2 on
x86_64 with a recent guix-daemon.

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-13  7:41                   ` Leo Famulari
@ 2016-02-13  7:46                     ` Leo Famulari
  2016-02-13 14:36                     ` Mark H Weaver
  1 sibling, 0 replies; 13+ messages in thread
From: Leo Famulari @ 2016-02-13  7:46 UTC (permalink / raw)
  To: Eric Bavier; +Cc: 22558-done

On Sat, Feb 13, 2016 at 02:41:18AM -0500, Leo Famulari wrote:
> On Fri, Feb 12, 2016 at 10:02:32PM -0600, Eric Bavier wrote:
> > On Fri, 12 Feb 2016 21:13:55 -0500
> > Mark H Weaver <mhw@netris.org> wrote:
> > 
> > > Leo Famulari <leo@famulari.name> writes:
> > > 
> > > > On Fri, Feb 12, 2016 at 06:22:46PM -0500, Mark H Weaver wrote:
> > > >> ludo@gnu.org (Ludovic Courtès) writes:
> > > >> 
> > > >> > Eric Bavier <ericbavier@openmailbox.org> skribis:
> > > >> >
> > > >> >>> > * With 14 test failures.  This happened at least twice, on
> > > >> >>> >   hydra.gnunet.org.
> > > >> >>>
> > > >> >>> Eric Bavier reported on #guix that it failed to build on his machine
> > > >> >>> twice in a row, with 14 test failures.
> > > >> >>
> > > >> >> I used 'git bisect' on the gnupg repository to narrow the failures
> > > >> >> down. Commit ee87c653bf is the commit that introduces the failures.
> > > >> >
> > > >> > Thanks for digging!  Could you report it upstream?
> > > >> 
> > > >> I reported it upstream, and now they have a fix in their repo.
> > > >> I've attached a patch to Guix which adds this upstream patch.
> > > >> 
> > > >> Eric, would you be willing to test it and see if it fixes the problem
> > > >> for you?
> > > >
> > > > I'm not Eric, but with this patch, I can successfully build on the
> > > > machine that was previously exhibiting the failures.
> > > 
> > > Thanks!  I went ahead and pushed this to master, commit 3a7261bf96.
> > > I'm closing this bug now.
> > 
> > I realize I'm a bit late, but I just wanted to let you know that this
> > appears to fix the gnupg-2.1.11 build for me too:
> > 
> > @ build-succeeded /gnu/store/qyjrqxnrka55p5g5vh1p80ac1rxn5nfm-gnupg-2.1.11.drv -
> > /gnu/store/w6fbikpwzpjkwvy6z1hwbszq0x83yspv-gnupg-2.1.11
> > 
> > But I had to add "#:use-module (gnu packages)" to the
> > gnu/packages/gnupg.scm before it would build, otherwise "could not
> > find symbol search-path".
> 
> Weird, that did not happen to me when building from 3a7261bf96b2 on
> x86_64 with a recent guix-daemon.

I do have a different output hash:
/gnu/store/bky2i5n7wvcnbzpxpnb4y5r5p795sc7v-gnupg-2.1.11

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

* bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra
  2016-02-13  7:41                   ` Leo Famulari
  2016-02-13  7:46                     ` Leo Famulari
@ 2016-02-13 14:36                     ` Mark H Weaver
  1 sibling, 0 replies; 13+ messages in thread
From: Mark H Weaver @ 2016-02-13 14:36 UTC (permalink / raw)
  To: Leo Famulari; +Cc: Eric Bavier, 22558-done

Leo Famulari <leo@famulari.name> writes:

> On Fri, Feb 12, 2016 at 10:02:32PM -0600, Eric Bavier wrote:
>> But I had to add "#:use-module (gnu packages)" to the
>> gnu/packages/gnupg.scm before it would build, otherwise "could not
>> find symbol search-path".
>
> Weird, that did not happen to me when building from 3a7261bf96b2 on
> x86_64 with a recent guix-daemon.

Eric presumably applied the patch to an older git checkout.  It turns
out that #:use-module (gnu packages) was only added to gnupg.scm about a
week ago in commit a05d1d82c0.

      Mark

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

end of thread, other threads:[~2016-02-13 14:37 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-04 19:58 bug#22558: gnupg-2.1.11 on x86_64 failed its tests many times on Hydra Mark H Weaver
2016-02-04 20:22 ` Mark H Weaver
2016-02-04 20:40   ` Mark H Weaver
     [not found] <20160204152848.GA4379@debian>
     [not found] ` <87r3gs9zvf.fsf@gnu.org>
     [not found]   ` <20160204193649.GA8785@debian>
     [not found]     ` <87d1scxkd6.fsf@netris.org>
2016-02-05 15:25       ` Eric Bavier
2016-02-05 18:58         ` Leo Famulari
2016-02-07 20:50         ` Ludovic Courtès
2016-02-12 23:22           ` Mark H Weaver
2016-02-13  0:12             ` Leo Famulari
2016-02-13  2:13               ` Mark H Weaver
2016-02-13  4:02                 ` Eric Bavier
2016-02-13  7:41                   ` Leo Famulari
2016-02-13  7:46                     ` Leo Famulari
2016-02-13 14:36                     ` Mark H Weaver

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).