all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] Do not disable gnurl testcase 1022
@ 2014-02-18 20:57 Sree Harsha Totakura
  2014-02-18 20:57 ` [PATCH] gnu: gnurl: Do not disable " Sree Harsha Totakura
  2014-02-18 21:55 ` [PATCH] Do not disable gnurl " Andreas Enge
  0 siblings, 2 replies; 3+ messages in thread
From: Sree Harsha Totakura @ 2014-02-18 20:57 UTC (permalink / raw)
  To: guix-devel

Upstream has fixed the previously failing testcase 1022.  Hence,
it is no longer required to be disabled.

Sree Harsha Totakura (1):
  gnu: gnurl: Do not disable testcase 1022.

 gnu/packages/gnunet.scm | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

-- 
1.8.5.3

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

* [PATCH] gnu: gnurl: Do not disable testcase 1022.
  2014-02-18 20:57 [PATCH] Do not disable gnurl testcase 1022 Sree Harsha Totakura
@ 2014-02-18 20:57 ` Sree Harsha Totakura
  2014-02-18 21:55 ` [PATCH] Do not disable gnurl " Andreas Enge
  1 sibling, 0 replies; 3+ messages in thread
From: Sree Harsha Totakura @ 2014-02-18 20:57 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/gnunet.scm (gnurl): Do not disable testcase 1022.
---
 gnu/packages/gnunet.scm | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/gnu/packages/gnunet.scm b/gnu/packages/gnunet.scm
index b084796..dd54911 100644
--- a/gnu/packages/gnunet.scm
+++ b/gnu/packages/gnunet.scm
@@ -167,8 +167,7 @@ and support for SSL3 and TLS.")
                           "--disable-file" "--disable-ftp")
      #:test-target "test"
      #:parallel-tests? #f
-     ;; We have to patch runtests.pl in tests/ directory and add a failing
-     ;; test due to curl->gnurl name change to tests/data/DISABLED
+     ;; We have to patch runtests.pl in tests/ directory
      #:phases
       (alist-cons-before
        'check 'patch-runtests
@@ -176,11 +175,7 @@ and support for SSL3 and TLS.")
          (with-directory-excursion "tests"
            (substitute* "runtests.pl"
              (("/bin/sh")
-              (which "sh")))
-           (let* ((port (open-file "data/DISABLED" "a")))
-             (newline port)
-             (display "1022" port)
-             (close port))))
+              (which "sh")))))
        %standard-phases)))
    (synopsis "Microfork of cURL with support for the HTTP/HTTPS/GnuTLS subset of cURL")
    (description
-- 
1.8.5.3

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

* Re: [PATCH] Do not disable gnurl testcase 1022
  2014-02-18 20:57 [PATCH] Do not disable gnurl testcase 1022 Sree Harsha Totakura
  2014-02-18 20:57 ` [PATCH] gnu: gnurl: Do not disable " Sree Harsha Totakura
@ 2014-02-18 21:55 ` Andreas Enge
  1 sibling, 0 replies; 3+ messages in thread
From: Andreas Enge @ 2014-02-18 21:55 UTC (permalink / raw)
  To: Sree Harsha Totakura

On Tue, Feb 18, 2014 at 09:57:42PM +0100, Sree Harsha Totakura wrote:
> Upstream has fixed the previously failing testcase 1022.  Hence,
> it is no longer required to be disabled.

Applied, thanks! I dropped the "with-directory-excursion" line as there
was only one file to be patched.

Andreas

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

end of thread, other threads:[~2014-02-18 21:59 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-02-18 20:57 [PATCH] Do not disable gnurl testcase 1022 Sree Harsha Totakura
2014-02-18 20:57 ` [PATCH] gnu: gnurl: Do not disable " Sree Harsha Totakura
2014-02-18 21:55 ` [PATCH] Do not disable gnurl " Andreas Enge

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.