all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Ben Sturmfels <ben@sturm.com.au>
To: 宋文武 <iyzsong@member.fsf.org>
Cc: 26647-done@debbugs.gnu.org
Subject: [bug#26647] Upgrade fabric to 1.13.2, remove upstreamed patch.
Date: Tue, 12 Jun 2018 12:14:05 +1000	[thread overview]
Message-ID: <87lgbk3g5u.fsf@sturm.com.au> (raw)
In-Reply-To: <87lgbngqxh.fsf@member.fsf.org>

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

Hi 宋文武,



Ben Sturmfels

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

On Sun, 10 Jun 2018, 宋文武 wrote:

> Ben Sturmfels <ben@sturm.com.au> writes:
>
>> From a5f28bdc67c4427cfcf213fb288c811ba3e0788c 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: Update to 1.14.0 and disable problematic tests.
>>
>> * gnu/packages/admin.scm (fabric): Update to 1.14.0 and add custom check phase to disable two tests.
>
> Well, this patch no longer needed, as we already had fabric-1.14.0.
>
> closing now, Thank you!

Thanks for following up on this. I believe that the patch to disable two
tests is still required though to address Ricardo's earlier issues with
tests timing out.

I've attached again without the upgrade to 0.14.0.

Regards,
Ben

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

From ace313161f2a3c8fda198b7bf5f7be91c9de4945 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.

Reported by Ricardo Wurmus <rekado@elephly.net> at
<https://debbugs.gnu.org/cgi/bugreport.cgi?bug=26647#8>.

* 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 6b8df3fb9..c1acfce8e 100644
--- a/gnu/packages/admin.scm
+++ b/gnu/packages/admin.scm
@@ -2359,11 +2359,24 @@ Intel DRM Driver.")
          "13r0b0hllgf8j9rh6x1knmbgvingbdmx046aazv6vck2ll120mw1"))))
     (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
        ("python2-pynacl" ,python2-pynacl)
        ("python2-bcrypt" ,python2-bcrypt)))
     (propagated-inputs
-- 
2.17.0


  reply	other threads:[~2018-06-12  2:15 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
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                 ` Ben Sturmfels [this message]
2018-06-12 11:51                   ` 宋文武

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=87lgbk3g5u.fsf@sturm.com.au \
    --to=ben@sturm.com.au \
    --cc=26647-done@debbugs.gnu.org \
    --cc=iyzsong@member.fsf.org \
    /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.