unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Leo Famulari <leo@famulari.name>
To: Frederick Muriithi <fredmanglis@gmail.com>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 1/4] gnu: Add python-pytest-3.0.7
Date: Mon, 27 Mar 2017 11:09:24 -0400	[thread overview]
Message-ID: <20170327150924.GA16027@jasmine> (raw)
In-Reply-To: <CALjrZwbo_h2wUPKX-QW5b5FbcepvqOzPmAUjWQsBkR-dir1yRQ@mail.gmail.com>


[-- Attachment #1.1: Type: text/plain, Size: 960 bytes --]

On Sun, Mar 26, 2017 at 06:29:04PM +0300, Frederick Muriithi wrote:
> I don't really see a reason why not. I have to point out though, that when
> writing the definition, I found python-pytest-2.9.2 also defined, with a
> note that it's there until all packages can be upgraded to use a more
> recent pytest.
> 
> Since I don't know what the progress on that is, I decided to err on the
> side of caution, and just add it with a note on why it's needed.

I added pytest 2.9.2 when I realized that many packages are picky about
which version of pytest they can use.

I checked, and all the packages that are using pytest 2.9.2 can use
pytest 3.0.7 instead.

So, here's a patch that updates the newer pytest across the board. It
goes after your patch series, unless you want to rebase your patches on
to it.

Soon we'll do a "python-updates" branch where we try to default to the
newer pytest, and keep an older one around for the outliers.

[-- Attachment #1.2: 0001-gnu-python-pytest-2.9.2-Replace-with-python-pytest-3.patch --]
[-- Type: text/plain, Size: 5116 bytes --]

From f44c8d0dec2af7677ad16cfad877fb73407b6c30 Mon Sep 17 00:00:00 2001
From: Leo Famulari <leo@famulari.name>
Date: Mon, 27 Mar 2017 03:10:53 -0400
Subject: [PATCH] gnu: python-pytest-2.9.2: Replace with python-pytest-3.0.

* gnu/packages/python.scm (python-pytest-2.9.2): Replace with ...
(python-pytest-3.0): ... new variable.
(python2-pytest-3.0): New variable.
(python-cryptography, python2-cryptography)[native-inputs]: Replace
python-pytest-2.9.2 with python-pytest-3.0.
* gnu/packages/backup.scm (borg)[native-inputs]: Likewise.
gnu/packages/calendar.scm (khal)[native-inputs]: Likewise.
---
 gnu/packages/backup.scm   |  2 +-
 gnu/packages/calendar.scm |  2 +-
 gnu/packages/python.scm   | 40 +++++++++++++++++-----------------------
 3 files changed, 19 insertions(+), 25 deletions(-)

diff --git a/gnu/packages/backup.scm b/gnu/packages/backup.scm
index 7e9ae18f3..8e1f19515 100644
--- a/gnu/packages/backup.scm
+++ b/gnu/packages/backup.scm
@@ -529,7 +529,7 @@ detection, and lossless compression.")
        ("python-setuptools-scm" ,python-setuptools-scm)
        ;; Borg 1.0.8's test suite uses 'tmpdir_factory', which was introduced in
        ;; pytest 2.8.
-       ("python-pytest" ,python-pytest-2.9.2)
+       ("python-pytest" ,python-pytest-3.0)
        ;; For generating the documentation.
        ("python-sphinx" ,python-sphinx)
        ("python-sphinx-rtd-theme" ,python-sphinx-rtd-theme)))
diff --git a/gnu/packages/calendar.scm b/gnu/packages/calendar.scm
index 1e9a93ca9..977111bfe 100644
--- a/gnu/packages/calendar.scm
+++ b/gnu/packages/calendar.scm
@@ -113,7 +113,7 @@ data units.")
             (zero? (system* "py.test" "tests")))))))
     (native-inputs
       ;; XXX Uses tmpdir_factory, introduced in pytest 2.8.
-     `(("python-pytest" ,python-pytest-2.9.2)
+     `(("python-pytest" ,python-pytest-3.0)
        ("python-pytest-cov" ,python-pytest-cov)
        ("python-setuptools-scm" ,python-setuptools-scm)
        ;; Required for tests
diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 8788e9d77..12d298bbd 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -1818,19 +1818,18 @@ and many external plugins.")
 (define-public python2-pytest
   (package-with-python2 python-pytest))
 
-;; This package is used by Borg until we can upgrade all our Python packages to
-;; use a more recent pytest.
-(define-public python-pytest-2.9.2
+;; Some packages require a newer pytest.
+(define-public python-pytest-3.0
   (package
     (inherit python-pytest)
     (name "python-pytest")
-    (version "2.9.2")
+    (version "3.0.7")
     (source (origin
               (method url-fetch)
               (uri (pypi-uri "pytest" version))
               (sha256
                (base32
-                "1n6igbc1b138wx1q5gca4pqw1j6nsyicfxds5n0b5989kaxqmh8j"))))
+                "1asc4b2nd2a4f0g3r12y97rslq5wliji7b73wwkvdrm5s7mrc1mp"))))
     (arguments
      `(#:phases
        (modify-phases %standard-phases
@@ -1841,25 +1840,20 @@ and many external plugins.")
                 (string-append "@pytest.mark.skip"
                                "(reason=\"Assumes that /usr exists.\")\n    "
                                line)))
-             #t)))))))
-
-;; This package is used by Sphinx version 1.5.2 and up
-(define-public python-pytest-3.0.7
-  (package
-    (inherit python-pytest-2.9.2)
-    (name "python-pytest")
-    (version "3.0.7")
-    (source
-     (origin
-       (method url-fetch)
-       (uri (pypi-uri "pytest" version))
-       (sha256
-        (base32
-         "1asc4b2nd2a4f0g3r12y97rslq5wliji7b73wwkvdrm5s7mrc1mp"))))
+             #t)))))
     (native-inputs
      `(("python-nose" ,python-nose)
        ("python-mock" ,python-mock)
-       ("python-hypothesis" ,python-hypothesis)))))
+       ("python-hypothesis" ,python-hypothesis)))
+    (properties `((python2-variant . ,(delay python2-pytest-3.0))))))
+
+(define-public python2-pytest-3.0
+  (let ((base (package-with-python2
+                (strip-python2-variant python-pytest-3.0))))
+    (package (inherit base)
+      (native-inputs
+        `(("python2-enum34" ,python2-enum34)
+          ,@(package-native-inputs base))))))
 
 (define-public python-pytest-cov
   (package
@@ -3119,7 +3113,7 @@ sources.")
        ("python-html5lib" ,python-html5lib)
        ("python-mock" ,python-mock)
        ("python-nose" ,python-nose)
-       ("python-pytest" ,python-pytest-3.0.7)))))
+       ("python-pytest" ,python-pytest-3.0)))))
 
 (define-public python-sphinx-1.5.3
   (package
@@ -7331,7 +7325,7 @@ responses, rather than doing any computation.")
        ("python-pyasn1" ,python-pyasn1)
        ("python-pyasn1-modules" ,python-pyasn1-modules)
        ("python-pytz" ,python-pytz)
-       ("python-pytest" ,python-pytest-2.9.2)))
+       ("python-pytest" ,python-pytest-3.0)))
     (home-page "https://github.com/pyca/cryptography")
     (synopsis "Cryptographic recipes and primitives for Python")
     (description
-- 
2.12.0


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

  reply	other threads:[~2017-03-27 15:09 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-26  9:23 [PATCH 1/4] gnu: Add python-pytest-3.0.7 Muriithi Frederick Muriuki
2017-03-26  9:23 ` [PATCH 2/4] gnu: Add python-sphinx-1.5.2, python-sphinx-1.5.3 Muriithi Frederick Muriuki
2017-03-26 15:10   ` Pjotr Prins
2017-03-26 15:32     ` Frederick Muriithi
2017-03-26 16:40       ` Pjotr Prins
2017-03-27 19:10         ` Ricardo Wurmus
2017-03-27 15:11       ` Leo Famulari
2017-03-27 15:12         ` Leo Famulari
2017-03-28 20:03           ` Leo Famulari
2017-03-26  9:23 ` [PATCH 3/4] gnu: Add python-paramunittest Muriithi Frederick Muriuki
2017-03-26 15:12   ` Pjotr Prins
2017-03-26 15:47     ` Frederick Muriithi
2017-03-26 23:32       ` Chris Marusich
2017-03-27  2:41         ` Frederick Muriithi
2017-03-28  5:16           ` Frederick Muriithi
2017-03-28 19:25             ` Ludovic Courtès
2017-03-28 20:03   ` Leo Famulari
2017-03-26  9:23 ` [PATCH 4/4] gnu: Add python-mando Muriithi Frederick Muriuki
2017-03-26 15:16   ` Pjotr Prins
2017-03-27 15:14   ` Leo Famulari
2017-03-28 19:28     ` Ludovic Courtès
2017-03-28 20:04   ` Leo Famulari
2017-03-26 15:10 ` [PATCH 1/4] gnu: Add python-pytest-3.0.7 Pjotr Prins
     [not found]   ` <CALjrZwaRUvrctcwwFQ_5isqwoaaEq=77P2qFHwk1FL2mJ5T-dg@mail.gmail.com>
     [not found]     ` <CALjrZwaah9_Vv+MVTm2aiA_-WHycYhn=KCnk5KjsXo=8z0+QDA@mail.gmail.com>
2017-03-26 15:29       ` Frederick Muriithi
2017-03-27 15:09         ` Leo Famulari [this message]
2017-03-28 20:05           ` Leo Famulari
2017-03-28 20:04 ` Leo Famulari

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

  List information: https://guix.gnu.org/

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

  git send-email \
    --in-reply-to=20170327150924.GA16027@jasmine \
    --to=leo@famulari.name \
    --cc=fredmanglis@gmail.com \
    --cc=guix-devel@gnu.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 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).