all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] gnu: python-click: Fix test failure.
@ 2017-01-11 20:18 Danny Milosavljevic
  2017-01-16 21:20 ` Ludovic Courtès
  0 siblings, 1 reply; 7+ messages in thread
From: Danny Milosavljevic @ 2017-01-11 20:18 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/python.scm (python-click)[arguments]: Fix test failure.
---
 gnu/packages/python.scm | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 4dc284d03..2f1bd8315 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -2383,6 +2383,8 @@ is used by the Requests library to verify HTTPS requests.")
        (modify-phases %standard-phases
          (add-after 'unpack 'fix-paths
            (lambda* (#:key inputs #:allow-other-keys)
+             ;; Causes test failure if present.
+             (delete-file "click/_winconsole.py")
              (let ((glibc (assoc-ref inputs ,(if (%current-target-system)
                                                  "cross-libc" "libc"))))
                (substitute* "click/_unicodefun.py"

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

* Re: [PATCH] gnu: python-click: Fix test failure.
  2017-01-11 20:18 [PATCH] gnu: python-click: Fix test failure Danny Milosavljevic
@ 2017-01-16 21:20 ` Ludovic Courtès
  2017-01-17 19:33   ` Danny Milosavljevic
  0 siblings, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2017-01-16 21:20 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel

Danny Milosavljevic <dannym@scratchpost.org> skribis:

> * gnu/packages/python.scm (python-click)[arguments]: Fix test failure.
> ---
>  gnu/packages/python.scm | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
> index 4dc284d03..2f1bd8315 100644
> --- a/gnu/packages/python.scm
> +++ b/gnu/packages/python.scm
> @@ -2383,6 +2383,8 @@ is used by the Requests library to verify HTTPS requests.")
>         (modify-phases %standard-phases
>           (add-after 'unpack 'fix-paths
>             (lambda* (#:key inputs #:allow-other-keys)
> +             ;; Causes test failure if present.
> +             (delete-file "click/_winconsole.py")

If you know more details, you’re welcome to add them in the comment, but
otherwise feel free to push!

Thanks,
Ludo’.

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

* Re: [PATCH] gnu: python-click: Fix test failure.
  2017-01-16 21:20 ` Ludovic Courtès
@ 2017-01-17 19:33   ` Danny Milosavljevic
  2017-01-17 19:39     ` Marius Bakke
  2017-01-17 22:50     ` Ludovic Courtès
  0 siblings, 2 replies; 7+ messages in thread
From: Danny Milosavljevic @ 2017-01-17 19:33 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

Hi Ludo,

> If you know more details, you’re welcome to add them in the comment, but
> otherwise feel free to push!

Does that mean push to master or to python-tests or to both?

Also, I don't really know more details other than that the test framework does autodiscovery and picks up a module for the wrong platform (Windows) just because it's there.

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

* Re: [PATCH] gnu: python-click: Fix test failure.
  2017-01-17 19:33   ` Danny Milosavljevic
@ 2017-01-17 19:39     ` Marius Bakke
  2017-01-17 22:50     ` Ludovic Courtès
  1 sibling, 0 replies; 7+ messages in thread
From: Marius Bakke @ 2017-01-17 19:39 UTC (permalink / raw)
  To: Danny Milosavljevic, Ludovic Courtès; +Cc: guix-devel

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

Danny Milosavljevic <dannym@scratchpost.org> writes:

> Hi Ludo,
>
>> If you know more details, you’re welcome to add them in the comment, but
>> otherwise feel free to push!
>
> Does that mean push to master or to python-tests or to both?
>
> Also, I don't really know more details other than that the test
> framework does autodiscovery and picks up a module for the wrong
> platform (Windows) just because it's there.

I came across this in python-tests as well, and fixed it by executing
"make test" instead of relying on 'setup.py test'.

Pro tip: Check how upstream runs their test, if they have a CI system
(.travis.yml etc).

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

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

* Re: [PATCH] gnu: python-click: Fix test failure.
  2017-01-17 19:33   ` Danny Milosavljevic
  2017-01-17 19:39     ` Marius Bakke
@ 2017-01-17 22:50     ` Ludovic Courtès
  2017-01-17 23:27       ` Marius Bakke
  1 sibling, 1 reply; 7+ messages in thread
From: Ludovic Courtès @ 2017-01-17 22:50 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel

Danny Milosavljevic <dannym@scratchpost.org> skribis:

> Hi Ludo,
>
>> If you know more details, you’re welcome to add them in the comment, but
>> otherwise feel free to push!
>
> Does that mean push to master or to python-tests or to both?

Good question!  That can go to master, but Marius and Leo know better
whether ‘python-tests’ is a better fit.  Thoughts?

Ludo’.

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

* Re: [PATCH] gnu: python-click: Fix test failure.
  2017-01-17 22:50     ` Ludovic Courtès
@ 2017-01-17 23:27       ` Marius Bakke
  2017-01-18 21:44         ` Ludovic Courtès
  0 siblings, 1 reply; 7+ messages in thread
From: Marius Bakke @ 2017-01-17 23:27 UTC (permalink / raw)
  To: Ludovic Courtès, Danny Milosavljevic; +Cc: guix-devel

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

Ludovic Courtès <ludo@gnu.org> writes:

> Danny Milosavljevic <dannym@scratchpost.org> skribis:
>
>> Hi Ludo,
>>
>>> If you know more details, you’re welcome to add them in the comment, but
>>> otherwise feel free to push!
>>
>> Does that mean push to master or to python-tests or to both?
>
> Good question!  That can go to master, but Marius and Leo know better
> whether ‘python-tests’ is a better fit.  Thoughts?

This is already fixed in 'python-tests' by
97be1bc23128cf42c49867d984c43f6fea3820d6. I don't think we need to fix
it in 'master', since it "succeeds" anyway.

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

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

* Re: [PATCH] gnu: python-click: Fix test failure.
  2017-01-17 23:27       ` Marius Bakke
@ 2017-01-18 21:44         ` Ludovic Courtès
  0 siblings, 0 replies; 7+ messages in thread
From: Ludovic Courtès @ 2017-01-18 21:44 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

Marius Bakke <mbakke@fastmail.com> skribis:

> Ludovic Courtès <ludo@gnu.org> writes:
>
>> Danny Milosavljevic <dannym@scratchpost.org> skribis:
>>
>>> Hi Ludo,
>>>
>>>> If you know more details, you’re welcome to add them in the comment, but
>>>> otherwise feel free to push!
>>>
>>> Does that mean push to master or to python-tests or to both?
>>
>> Good question!  That can go to master, but Marius and Leo know better
>> whether ‘python-tests’ is a better fit.  Thoughts?
>
> This is already fixed in 'python-tests' by
> 97be1bc23128cf42c49867d984c43f6fea3820d6. I don't think we need to fix
> it in 'master', since it "succeeds" anyway.

Oh good, makes sense.

Ludo’.

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

end of thread, other threads:[~2017-01-18 21:44 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-01-11 20:18 [PATCH] gnu: python-click: Fix test failure Danny Milosavljevic
2017-01-16 21:20 ` Ludovic Courtès
2017-01-17 19:33   ` Danny Milosavljevic
2017-01-17 19:39     ` Marius Bakke
2017-01-17 22:50     ` Ludovic Courtès
2017-01-17 23:27       ` Marius Bakke
2017-01-18 21:44         ` Ludovic Courtès

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.