[bug#27548] [PATCH] gnu: Add python-xenon

[bug#27548] [PATCH] gnu: Add python-xenon

[Muriithi Frederick Muriuki]

* gnu/packages/python.scm (python-xenon, python2-xenon) New variables.
---
 gnu/packages/python.scm | 46 ++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 46 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 6e1e289e9..1167b0933 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -15512,3 +15512,49 @@ pure Python module.")
 
 (define-public python2-rencode
   (package-with-python2 python-rencode))
+
+(define-public python-xenon
+  (package
+    (name "python-xenon")
+    (version "0.5.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (pypi-uri "xenon" version))
+       (sha256
+        (base32
+         "14kby2y48vp3sgwxqlm5d5789yibqwb1qli5fwcmdqg3iayrbklc"))))
+    (build-system python-build-system)
+    (native-inputs
+     `(("python-pyyaml" ,python-pyyaml)
+       ("python-radon" ,python-radon)
+       ("python-requests" ,python-requests)
+      ("python-flake8" ,python-flake8)
+       ("python-tox" ,python-tox)))
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         (add-before 'build 'patch-test-requirements
+          (lambda* (#:key inputs #:allow-other-keys)
+             ;; Update requirements from dependecy==version
+             ;; to dependency>=version
+             (substitute* "requirements.txt"
+               (("==") ">=")
+               ((",<1.5.0") ""))
+            ;; Remove httpretty dependency for tests
+             (substitute* "setup.py"
+               (("httpretty") ""))
+             #t)))))
+    (home-page "https://xenon.readthedocs.org/")
+    (synopsis
+     "Monitor code metrics for Python on your CI server")
+   (description
+     "@code{Xenon} is a monitoring tool based on Radon.  It monitors code’s
+complexity.  Ideally, @code{xenon} is run every time code is committed.  Through
+command line options, various thresholds can be set for the complexity of code.
+It will fail (i.e.  it will exit with a non-zero exit code) when any of these
+equirements is not met.")
+    (license license:expat)))
+
+(define-public python2-xenon
+  (package-with-python2 python-xenon))
-- 
2.13.2

[bug#27548] [PATCH] gnu: Add python-xenon

[Marius Bakke]

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

Hello!

Sorry for the slooow response :-)

The patch mostly LGTM. A few minor comments:

Muriithi Frederick Muriuki <fredmanglis@gmail.com> writes:

> * gnu/packages/python.scm (python-xenon, python2-xenon) New variables.

[...]

> +(define-public python-xenon
> +  (package
> +    (name "python-xenon")
> +    (version "0.5.1")

It looks like 0.5.2 was recently released. Can you try updating to it?

> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (pypi-uri "xenon" version))
> +       (sha256
> +        (base32
> +         "14kby2y48vp3sgwxqlm5d5789yibqwb1qli5fwcmdqg3iayrbklc"))))
> +    (build-system python-build-system)
> +    (native-inputs
> +     `(("python-pyyaml" ,python-pyyaml)
> +       ("python-radon" ,python-radon)
> +       ("python-requests" ,python-requests)
> +      ("python-flake8" ,python-flake8)

Indentation is off :)

> +       ("python-tox" ,python-tox)))
> +    (arguments
> +     `(#:phases
> +       (modify-phases %standard-phases
> +         (add-before 'build 'patch-test-requirements
> +          (lambda* (#:key inputs #:allow-other-keys)
> +             ;; Update requirements from dependecy==version
Missing letter here:                             ^^^
> +             ;; to dependency>=version
> +             (substitute* "requirements.txt"
> +               (("==") ">=")
> +               ((",<1.5.0") ""))
> +            ;; Remove httpretty dependency for tests
> +             (substitute* "setup.py"
> +               (("httpretty") ""))

Why is this necessary? Can you expand on this comment?

> +             #t)))))
> +    (home-page "https://xenon.readthedocs.org/")
> +    (synopsis
> +     "Monitor code metrics for Python on your CI server")
> +   (description
> +     "@code{Xenon} is a monitoring tool based on Radon.  It monitors code’s

I think we can drop @code here since it's used as a name. Side note: It
would be cool if we could cross-reference packages in descriptions.

Sorry for the nit-picks! I'll go through the other patches shortly.

> +complexity.  Ideally, @code{xenon} is run every time code is committed.  Through
> +command line options, various thresholds can be set for the complexity of code.
> +It will fail (i.e.  it will exit with a non-zero exit code) when any of these
> +equirements is not met.")
> +    (license license:expat)))

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

bug#27548: [PATCH] gnu: Add python-xenon

[Ludovic Courtès]

Muriithi Frederick Muriuki <fredmanglis@gmail.com> skribis:

> * gnu/packages/python.scm (python-xenon, python2-xenon) New variables.

This was pushed as 2038a48f4d a while back.

Thanks again,
Ludo'.

[bug#27548] [PATCH] gnu: Add python-xenon

[Frederick Muriithi]

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

No problem, thanks!

[-- Attachment #2: Type: text/html, Size: 95 bytes --]