[PATCH v2 0/7] Updated: Some simple or even trivial changes.

[PATCH v2 0/7] Updated: Some simple or even trivial changes.

[Hartmut Goebel]

Hi,

hopefully I got all requested changes.

 * The updates for urls and descriptions I split into three commits. Hope this
   is wat you wanted.
 * I added to remove the duplicate definition of python2-msgpack, too. I uses
   one commit for, hope this is okay, too.

Hartmut Goebel (7):
  gnu: python2-pytest-runner, python2-msgpack: Remove duplicate
    definitions.
  gnu: python-pytest-xdist: Remove python byte-code files from source.
  gnu: python-zope-testing: Strip byte-code and backup-files from
    source.
  gnu: python-pylockfile: Update home-page url.
  gnu: python-keyring: Update home-page url.
  gnu: python-pathlib: Update description.
  guix: python-build-system: Fix an outdated comment.

 gnu/packages/python.scm            | 35 ++++++++++++++++++++++++-----------
 guix/build/python-build-system.scm |  3 +--
 2 files changed, 25 insertions(+), 13 deletions(-)

-- 
2.7.4

[PATCH v2 1/7] gnu: python2-pytest-runner, python2-msgpack: Remove duplicate definitions.

[Hartmut Goebel]

* gnu/packages/python.scm (python2-pytest-runner): Was defined twice, remove
  duplicate definition. (python2-msgpack): Was defined twice, but differently,
  remove the obviously wrong second definition.
---
 gnu/packages/python.scm | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index a9fd8eb..3c3a00c 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -1715,9 +1715,6 @@ supports coverage of subprocesses.")
       (native-inputs `(("python2-setuptools" ,python2-setuptools)
                        ,@(package-native-inputs base))))))
 
-(define-public python2-pytest-runner
-  (package-with-python2 python-pytest-runner))
-
 (define-public python-pytest-mock
   (package
     (name "python-pytest-mock")
@@ -5754,9 +5751,6 @@ reading and writing MessagePack data.")
     (native-inputs
      `(("python2-setuptools" ,python2-setuptools)))))
 
-(define-public python2-msgpack
-  (package-with-python2 python-msgpack))
-
 (define-public python-netaddr
   (package
     (name "python-netaddr")
-- 
2.7.4

[PATCH v2 2/7] gnu: python-pytest-xdist: Remove pre-compiled .pyc files from source.

[Hartmut Goebel]

* gnu/packages/python.scm (python-pytest-xdist,
  python2-pytest-xdist)[source]: Add snippet.
---
 gnu/packages/python.scm | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 3c3a00c..eca60bc 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -1764,7 +1764,15 @@ same arguments.")
        (uri (pypi-uri "pytest-xdist" version ".zip"))
        (sha256
         (base32
-         "08rn2l39ds60xshs4js787l84pfckksqklfq2wq9x8ig2aci2pja"))))
+         "08rn2l39ds60xshs4js787l84pfckksqklfq2wq9x8ig2aci2pja"))
+       (modules '((guix build utils)))
+       (snippet
+        '(begin
+           ;; Remove pre-compiled .pyc files from source.
+           (for-each delete-file-recursively
+                     (find-files "." "__pycache__" #:directories? #t))
+           (for-each delete-file (find-files "." "\\.pyc$"))
+           #t))))
     (build-system python-build-system)
     (native-inputs
      `(("unzip" ,unzip)
-- 
2.7.4

[PATCH v2 3/7] gnu: python-zope-testing: Strip byte-code and backup-files from source.

[Hartmut Goebel]

* gnu/package/python.scm (python-zope-testing, python2-zope-testing)[source]:
  Add snippet to strip byte-code and backup-files.
---
 gnu/packages/python.scm | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index eca60bc..ec56c85 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -6644,7 +6644,13 @@ that have uses outside of the Zope framework.")
                            "/zope.testing/zope.testing-" version ".tar.gz"))
        (sha256
         (base32
-         "1yvglxhzvhl45mndvn9gskx2ph30zz1bz7rrlyfs62fv2pvih90s"))))
+         "1yvglxhzvhl45mndvn9gskx2ph30zz1bz7rrlyfs62fv2pvih90s"))
+       (modules '((guix build utils)))
+       (snippet
+        '(begin
+           ;; Remove pre-compiled .pyc files backup files from source.
+           (for-each delete-file (find-files "." "(\\.pyc|~)$"))
+           #t))))
     (build-system python-build-system)
     (native-inputs
      `(("python-zope-exceptions" ,python-zope-exceptions)))
-- 
2.7.4

[PATCH v2 4/7] gnu: python-pylockfile: Update home-page url.

[Hartmut Goebel]

* gnu/packages/python.scm (python-pylockfile, python2-pylockfile): Update
  home-page url.
---
 gnu/packages/python.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index ec56c85..a1c2088 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -763,7 +763,7 @@ concepts.")
     (arguments '(#:test-target "check"))
     (native-inputs
      `(("python-pbr" ,python-pbr)))
-    (home-page "http://code.google.com/p/pylockfile/")
+    (home-page "https://launchpad.net/pylockfile")
     (synopsis "Platform-independent file locking module")
     (description
      "The lockfile package exports a LockFile class which provides a simple
-- 
2.7.4

[PATCH v2 5/7] gnu: python-keyring: Update home-page url.

[Hartmut Goebel]

* gnu/packages/python.scm (python-keyring, python2-keyring): Update home-page
  url.
---
 gnu/packages/python.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index a1c2088..c890c68 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -900,7 +900,7 @@ etc.).  The package is structured to make adding new modules easy.")
      `(("python-pycrypto" ,python-pycrypto)))
     (arguments
      `(#:tests? #f))                      ;TODO: tests require pytest
-    (home-page "http://bitbucket.org/kang/python-keyring-lib")
+    (home-page "https://github.com/jaraco/keyring")
     (synopsis "Store and access your passwords safely")
     (description
      "The Python keyring lib provides a easy way to access the system keyring
-- 
2.7.4

[PATCH v2 6/7] gnu: python-pathlib: Update description.

[Hartmut Goebel]

* gnu/packages/python.scm (python-pathlib, python2-pathlib): Update
  description.
---
 gnu/packages/python.scm | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index c890c68..6207896 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -8849,7 +8849,12 @@ be done easily through operators, attribute accesses, and method calls.
 comparing Windows paths ignores casing.
 @item Well-defined semantics, eliminating any inconsistencies or
 ambiguities (forward vs. backward slashes, etc.).
-@end enumerate\n")
+@end enumerate
+
+Note: In Python 3.4, pathlib is now part of the standard library.  For other
+Python versions please consider python-pathlib2 instead, which tracks the
+standard library module.  This module (python-pathlib) isn't maintained
+anymore.")
     (license license:expat)))
 
 (define-public python2-pathlib
-- 
2.7.4

[PATCH v2 7/7] guix: python-build-system: Fix an outdated comment.

[Hartmut Goebel]

The python-build-system uses phases the build and install, but not
configure. So the old comment was plain wrong since Sept. 2013, when the build
phase has been added.
---
 guix/build/python-build-system.scm | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/guix/build/python-build-system.scm b/guix/build/python-build-system.scm
index 9109fb4..e906e60 100644
--- a/guix/build/python-build-system.scm
+++ b/guix/build/python-build-system.scm
@@ -137,8 +137,7 @@ installed with setuptools."
     #t))
 
 (define %standard-phases
-  ;; 'configure' and 'build' phases are not needed.  Everything is done during
-  ;; 'install'.
+  ;; 'configure' phase is not needed.
   (modify-phases gnu:%standard-phases
     (add-after 'unpack 'ensure-no-mtimes-pre-1980 ensure-no-mtimes-pre-1980)
     (delete 'configure)
-- 
2.7.4

Re: [PATCH v2 0/7] Updated: Some simple or even trivial changes.

[Danny Milosavljevic]

Hi,

all LGTM!

Re: [PATCH v2 0/7] Updated: Some simple or even trivial changes.

[Hartmut Goebel]

Am 13.10.2016 um 15:27 schrieb Danny Milosavljevic:
> all LGTM!

commited. Thanks for the review.

-- 
Regards
Hartmut Goebel

| Hartmut Goebel          | h.goebel@crazy-compilers.com               |
| www.crazy-compilers.com | compilers which you thought are impossible |