unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#62922] [PATCH] gnu: python-lazr-restfulclient: Add missing dependency
@ 2023-04-18  8:12 Tom Willemse
  2023-05-16 20:31 ` Tom Willemse
  2023-05-22 18:38 ` [bug#62922] [PATCH v2] gnu: python-lazr-restfulclient: Add missing dependencies Ivan Vilata i Balaguer
  0 siblings, 2 replies; 4+ messages in thread
From: Tom Willemse @ 2023-04-18  8:12 UTC (permalink / raw)
  To: 62922; +Cc: Tom Willemse

* gnu/packages/python-web.scm (python-lazr-restfulclient): Add missing
dependency: python-six
---
Trying to rebuild my guix home config I ran into the following error:

starting phase `sanity-check'
validating 'lazr.restfulclient' /gnu/store/llar0l72885zagk7km0hvjlq4hb3g2sx-python-lazr-restfulclient-0.14.4/lib/python3.9/site-packages
...checking requirements: ERROR: lazr.restfulclient==0.14.4 DistributionNotFound(Requirement.parse('six'), {'lazr.restfulclient'})
error: in phase 'sanity-check': uncaught exception:
%exception #<&invoke-error program: "python" arguments: ("/gnu/store/35ix1m6m8a5s21j02ajhdyqxb2xkshfb-sanity-check.py" "/gnu/store/llar0l72885zagk7km0hvjlq4hb3g2sx-python-lazr-restfulclient-0.14.4/lib/python3.9/site-packages") exit-status: 1 term-signal: #f stop-signal: #f>
phase `sanity-check' failed after 0.2 seconds
command "python" "/gnu/store/35ix1m6m8a5s21j02ajhdyqxb2xkshfb-sanity-check.py" "/gnu/store/llar0l72885zagk7km0hvjlq4hb3g2sx-python-lazr-restfulclient-0.14.4/lib/python3.9/site-packages" failed with status 1

From what I understand from the text it indicates that the requirements.txt of
the lazr restfulclient project specifies a dependency on six, but that package
hasn't been installed in the environment. Adding this dependency fixes the issue
for me.

 gnu/packages/python-web.scm | 22 ++++++++++------------
 1 file changed, 10 insertions(+), 12 deletions(-)

diff --git a/gnu/packages/python-web.scm b/gnu/packages/python-web.scm
index d84b5efdf9..051346c66a 100644
--- a/gnu/packages/python-web.scm
+++ b/gnu/packages/python-web.scm
@@ -127,21 +127,19 @@ (define-public python-lazr-restfulclient
   (package
     (name "python-lazr-restfulclient")
     (version "0.14.4")
-    (source
-     (origin
-       (method url-fetch)
-       (uri (pypi-uri "lazr.restfulclient" version))
-       (sha256
-        (base32 "11yhlqmdf2cqbdfzn8gdmzvmcivh4fflr18zf412sflvfjrdc3xz"))))
+    (source (origin
+              (method url-fetch)
+              (uri (pypi-uri "lazr.restfulclient" version))
+              (sha256
+               (base32
+                "11yhlqmdf2cqbdfzn8gdmzvmcivh4fflr18zf412sflvfjrdc3xz"))))
     (build-system python-build-system)
     ;; Disable the test suite to avoid the lazr.authentication requirement,
     ;; which requires the ancient 'oauth', a Python 2 only library.
-    (arguments (list #:tests? #f))
-    (propagated-inputs
-     (list python-distro
-           python-httplib2
-           python-oauthlib
-           python-wadllib))
+    (arguments
+     (list #:tests? #f))
+    (propagated-inputs (list python-distro python-httplib2 python-oauthlib
+                             python-wadllib python-six))
     (home-page "https://launchpad.net/lazr.restfulclient")
     (synopsis "Web client Python library extending wadlib")
     (description "This package provides a programmable client library that

base-commit: 79b2221ce43b8026eae6461a9269466cae946cac
-- 
2.40.0





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

* [bug#62922] [PATCH] gnu: python-lazr-restfulclient: Add missing dependency
  2023-04-18  8:12 [bug#62922] [PATCH] gnu: python-lazr-restfulclient: Add missing dependency Tom Willemse
@ 2023-05-16 20:31 ` Tom Willemse
  2023-05-22 18:38 ` [bug#62922] [PATCH v2] gnu: python-lazr-restfulclient: Add missing dependencies Ivan Vilata i Balaguer
  1 sibling, 0 replies; 4+ messages in thread
From: Tom Willemse @ 2023-05-16 20:31 UTC (permalink / raw)
  To: 62922; +Cc: lars, jgart

Hello!

Tom Willemse <tom@ryuslash.org> writes:

> * gnu/packages/python-web.scm (python-lazr-restfulclient): Add missing
> dependency: python-six
> ---
> Trying to rebuild my guix home config I ran into the following error:
>
> starting phase `sanity-check'
> validating 'lazr.restfulclient' /gnu/store/llar0l72885zagk7km0hvjlq4hb3g2sx-python-lazr-restfulclient-0.14.4/lib/python3.9/site-packages
> ...checking requirements: ERROR: lazr.restfulclient==0.14.4 DistributionNotFound(Requirement.parse('six'), {'lazr.restfulclient'})
> error: in phase 'sanity-check': uncaught exception:
> %exception #<&invoke-error program: "python" arguments: ("/gnu/store/35ix1m6m8a5s21j02ajhdyqxb2xkshfb-sanity-check.py" "/gnu/store/llar0l72885zagk7km0hvjlq4hb3g2sx-python-lazr-restfulclient-0.14.4/lib/python3.9/site-packages") exit-status: 1 term-signal: #f stop-signal: #f>
> phase `sanity-check' failed after 0.2 seconds
> command "python" "/gnu/store/35ix1m6m8a5s21j02ajhdyqxb2xkshfb-sanity-check.py" "/gnu/store/llar0l72885zagk7km0hvjlq4hb3g2sx-python-lazr-restfulclient-0.14.4/lib/python3.9/site-packages" failed with status 1
>
> From what I understand from the text it indicates that the requirements.txt of
> the lazr restfulclient project specifies a dependency on six, but that package
> hasn't been installed in the environment. Adding this dependency fixes the issue
> for me.
>
>  gnu/packages/python-web.scm | 22 ++++++++++------------
>  1 file changed, 10 insertions(+), 12 deletions(-)
>
> diff --git a/gnu/packages/python-web.scm b/gnu/packages/python-web.scm
> index d84b5efdf9..051346c66a 100644
> --- a/gnu/packages/python-web.scm
> +++ b/gnu/packages/python-web.scm
> @@ -127,21 +127,19 @@ (define-public python-lazr-restfulclient
>    (package
>      (name "python-lazr-restfulclient")
>      (version "0.14.4")
> -    (source
> -     (origin
> -       (method url-fetch)
> -       (uri (pypi-uri "lazr.restfulclient" version))
> -       (sha256
> -        (base32 "11yhlqmdf2cqbdfzn8gdmzvmcivh4fflr18zf412sflvfjrdc3xz"))))
> +    (source (origin
> +              (method url-fetch)
> +              (uri (pypi-uri "lazr.restfulclient" version))
> +              (sha256
> +               (base32
> +                "11yhlqmdf2cqbdfzn8gdmzvmcivh4fflr18zf412sflvfjrdc3xz"))))
>      (build-system python-build-system)
>      ;; Disable the test suite to avoid the lazr.authentication requirement,
>      ;; which requires the ancient 'oauth', a Python 2 only library.
> -    (arguments (list #:tests? #f))
> -    (propagated-inputs
> -     (list python-distro
> -           python-httplib2
> -           python-oauthlib
> -           python-wadllib))
> +    (arguments
> +     (list #:tests? #f))
> +    (propagated-inputs (list python-distro python-httplib2 python-oauthlib
> +                             python-wadllib python-six))
>      (home-page "https://launchpad.net/lazr.restfulclient")
>      (synopsis "Web client Python library extending wadlib")
>      (description "This package provides a programmable client library that
>
> base-commit: 79b2221ce43b8026eae6461a9269466cae946cac
> -- 
> 2.40.0

Apologies if this is the wrong way to go about it to add some people to
the CC, but it was pointed out to me that I should have included the
area owners in the first place.

If you would be so kind as to have a look, this issue is preventing me
from upgrading my guix installation because one of the packages I
install uses Bazaar to get the source, which is actually aliased to
Breezy, which requires the package in this patch.




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

* [bug#62922] [PATCH v2] gnu: python-lazr-restfulclient: Add missing dependencies.
  2023-04-18  8:12 [bug#62922] [PATCH] gnu: python-lazr-restfulclient: Add missing dependency Tom Willemse
  2023-05-16 20:31 ` Tom Willemse
@ 2023-05-22 18:38 ` Ivan Vilata i Balaguer
  2023-05-23  7:27   ` Ivan Vilata i Balaguer
  1 sibling, 1 reply; 4+ messages in thread
From: Ivan Vilata i Balaguer @ 2023-05-22 18:38 UTC (permalink / raw)
  To: 62922

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

Fixes <https://issues.guix.gnu.org/62922>.

* gnu/packages/python-web.scm (python-lazr-restfulclient): Add missing
dependencies.
[propagated-inputs] Add python-pyparsing and python-six.
---
The original patch from Tom Willemse still issued a DistributionNotFound error
on pyparsing.  This alternative version adds both dependencies (without extra
reformatting).  Thanks Tom for the initiative!

 gnu/packages/python-web.scm | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/gnu/packages/python-web.scm b/gnu/packages/python-web.scm
index b6ad489626..262309032f 100644
--- a/gnu/packages/python-web.scm
+++ b/gnu/packages/python-web.scm
@@ -59,6 +59,7 @@
 ;;; Copyright © 2022 Michael Rohleder <mike@rohleder.de>
 ;;; Copyright © 2022 Baptiste Strazzulla <bstrazzull@hotmail.fr>
 ;;; Copyright © 2023 John Kehayias <john.kehayias@protonmail.com>
+;;; Copyright © 2023 Ivan Vilata-i-Balaguer <ivan@selidor.net>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -143,6 +144,8 @@ (define-public python-lazr-restfulclient
      (list python-distro
            python-httplib2
            python-oauthlib
+           python-pyparsing
+           python-six
            python-wadllib))
     (home-page "https://launchpad.net/lazr.restfulclient")
     (synopsis "Web client Python library extending wadlib")

base-commit: dff1689bb37e5303868584d3f1d7a33cbcb7f51e
-- 
2.39.2


-- 
Ivan Vilata i Balaguer -- https://elvil.net/

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

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

* [bug#62922] [PATCH v2] gnu: python-lazr-restfulclient: Add missing dependencies.
  2023-05-22 18:38 ` [bug#62922] [PATCH v2] gnu: python-lazr-restfulclient: Add missing dependencies Ivan Vilata i Balaguer
@ 2023-05-23  7:27   ` Ivan Vilata i Balaguer
  0 siblings, 0 replies; 4+ messages in thread
From: Ivan Vilata i Balaguer @ 2023-05-23  7:27 UTC (permalink / raw)
  To: Ivan Vilata i Balaguer; +Cc: 62922

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

An equivalent patch was commited by jgart:
87d75d915152b150c3800d7011b04b4d11cd339c.

This issue can be closed now, thanks!


Ivan Vilata i Balaguer (2023-05-22 20:38:17 +0200) wrote:

> Fixes <https://issues.guix.gnu.org/62922>.
> 
> * gnu/packages/python-web.scm (python-lazr-restfulclient): Add missing
> dependencies.
> [propagated-inputs] Add python-pyparsing and python-six.
> ---
> The original patch from Tom Willemse still issued a DistributionNotFound error
> on pyparsing.  This alternative version adds both dependencies (without extra
> reformatting).  Thanks Tom for the initiative!
> 
>  gnu/packages/python-web.scm | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/gnu/packages/python-web.scm b/gnu/packages/python-web.scm
> index b6ad489626..262309032f 100644
> --- a/gnu/packages/python-web.scm
> +++ b/gnu/packages/python-web.scm
> @@ -59,6 +59,7 @@
>  ;;; Copyright © 2022 Michael Rohleder <mike@rohleder.de>
>  ;;; Copyright © 2022 Baptiste Strazzulla <bstrazzull@hotmail.fr>
>  ;;; Copyright © 2023 John Kehayias <john.kehayias@protonmail.com>
> +;;; Copyright © 2023 Ivan Vilata-i-Balaguer <ivan@selidor.net>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -143,6 +144,8 @@ (define-public python-lazr-restfulclient
>       (list python-distro
>             python-httplib2
>             python-oauthlib
> +           python-pyparsing
> +           python-six
>             python-wadllib))
>      (home-page "https://launchpad.net/lazr.restfulclient")
>      (synopsis "Web client Python library extending wadlib")
> 
> base-commit: dff1689bb37e5303868584d3f1d7a33cbcb7f51e
> -- 
> 2.39.2

-- 
Ivan Vilata i Balaguer -- https://elvil.net/

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

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

end of thread, other threads:[~2023-05-23  7:28 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-18  8:12 [bug#62922] [PATCH] gnu: python-lazr-restfulclient: Add missing dependency Tom Willemse
2023-05-16 20:31 ` Tom Willemse
2023-05-22 18:38 ` [bug#62922] [PATCH v2] gnu: python-lazr-restfulclient: Add missing dependencies Ivan Vilata i Balaguer
2023-05-23  7:27   ` Ivan Vilata i Balaguer

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).