all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Ben Sturmfels <ben@sturm.com.au>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: Ricardo Wurmus <rekado@elephly.net>, 26647@debbugs.gnu.org
Subject: [bug#26647] Upgrade fabric to 1.13.2, remove upstreamed patch.
Date: Tue, 07 Nov 2017 23:54:29 +1100	[thread overview]
Message-ID: <87bmkefdve.fsf@sturm.com.au> (raw)
In-Reply-To: <87k21cr6i9.fsf@gumble.sturm.com.au>

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


On Wed, 06 Sep 2017, Ben Sturmfels wrote:

> On Sat, 02 Sep 2017, Ludovic Courtès wrote:
>
>> Did you have a chance to look into this?
>> (See <https://bugs.gnu.org/26647> for context.)

It looks like Tobias beat me to the fabric 1.13.2 upgrade in f58be417 -
thanks Tobias! But the build hanging issue Ricardo mentioned still
remains. Should be fixed by the attached patch.

I've re-added the "Requires < X.X" comments as I think these will help
anyone trying to debug build problems if/when these dependencies are
upgraded.

If that's all ok, I can look at the upgrade to 0.14.0 in a separate patch.

Regards,
Ben

--
Ben Sturmfels

Sturm - Software Engineering
www.sturm.com.au
(03) 9024 2467

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-fabric-Disable-problematic-tests.patch --]
[-- Type: text/x-patch, Size: 1906 bytes --]

From 1a88d0aede4aee911617dc4ad5359d43a1193f22 Mon Sep 17 00:00:00 2001
From: Ben Sturmfels <ben@sturm.com.au>
Date: Tue, 7 Nov 2017 23:37:38 +1100
Subject: [PATCH] gnu: fabric: Disable problematic tests.

* gnu/packages/admin.scm (fabric): Add custom check phase to disable two tests.
---
 gnu/packages/admin.scm | 21 +++++++++++++++++----
 1 file changed, 17 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
index 7b8e41fce..dcc5ecbff 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -2122,11 +2122,24 @@ Intel DRM Driver.")
          "0k944dxr41whw7ib6380q9x15wyskx7fqni656icdn8rzshn9bwq"))))
     (build-system python-build-system)
     (arguments
-     `(#:python ,python-2))             ; Python 2 only
+     `(#:python ,python-2               ; Python 2 only
+       #:phases
+       (modify-phases %standard-phases
+         (replace 'check
+           (lambda _
+             (zero?
+              (system*
+               "nosetests" "-v" "tests/"
+               ;; This test hangs indefinitely when run on a single core VM
+               ;; (see GNU bug #26647 and Debian bug #850230).
+               "--exclude=test_nested_execution_with_explicit_ports"
+               ;; This test randomly fails in certain environments causing too
+               ;; much noise to be useful (see Debian bug #854686).
+               "--exclude=test_should_use_sentinel_for_tasks_that_errored")))))))
     (native-inputs
-     `(("python2-fudge" ,python2-fudge)
-       ("python2-jinja2" ,python2-jinja2)
-       ("python2-nose" ,python2-nose)))
+     `(("python2-fudge" ,python2-fudge) ; Requires < 1.0
+       ("python2-jinja2" ,python2-jinja2) ; Requires < 3.0
+       ("python2-nose" ,python2-nose))) ; Requires < 2.0
     (propagated-inputs
      `(("python2-paramiko" ,python2-paramiko)))
     (home-page "http://fabfile.org")
-- 
2.14.2


  reply	other threads:[~2017-11-07 12:55 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-25  2:40 bug#26647: Upgrade fabric to 1.13.2, remove upstreamed patch Ben Sturmfels
2017-04-25 13:18 ` Ricardo Wurmus
2017-04-26  0:26   ` Ben Sturmfels
2017-09-01 22:48     ` [bug#26647] " Ludovic Courtès
2017-09-06  0:54       ` Ben Sturmfels
2017-11-07 12:54         ` Ben Sturmfels [this message]
2017-11-07 13:21           ` Ben Sturmfels
2017-11-07 13:24             ` Ben Sturmfels
2017-11-07 13:46           ` Ricardo Wurmus
2017-12-02  0:06             ` Ben Sturmfels
2018-06-10  5:18               ` bug#26647: " 宋文武
2018-06-12  2:14                 ` [bug#26647] " Ben Sturmfels
2018-06-12 11:51                   ` bug#26647: " 宋文武

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87bmkefdve.fsf@sturm.com.au \
    --to=ben@sturm.com.au \
    --cc=26647@debbugs.gnu.org \
    --cc=ludo@gnu.org \
    --cc=rekado@elephly.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.