diff mbox series

[bug#54515,v4,2/3] gnu: python-lxml: Update to 4.7.1.

Message ID 20220324173702.22599-3-chadwainholness@gmail.com
State Accepted
Headers show
Series [bug#54515,v4,1/3] gnu: python-pycountry: Update to 22.3.5. | expand

Commit Message

Chadwain Holness March 24, 2022, 5:37 p.m. UTC 
* gnu/packages/xml.scm (python-lxml): Update to 4.7.1.
---
 gnu/packages/xml.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Ludovic Courtès April 1, 2022, 4:03 p.m. UTC | #1 
Chadwain Holness <chadwainholness@gmail.com> skribis:

> * gnu/packages/xml.scm (python-lxml): Update to 4.7.1.

Unfortunately this one has too many dependencies, as reported by
‘guix refresh -l python-lxml’.

So I can apply it on the ‘core-updates’ branch, but it won’t be merged
before a few months.  Or you can define it as a variant, and we’d keep
both versions in ‘master’.

WDYT?

Ludo’.
Chadwain Holness April 1, 2022, 7:30 p.m. UTC | #2 
This doesn't need to be applied right away, so I think simply applying it
to core-updates would be fine.


On Fri, Apr 1, 2022, 12:03 PM Ludovic Courtès <ludo@gnu.org> wrote:

> Chadwain Holness <chadwainholness@gmail.com> skribis:
>
> > * gnu/packages/xml.scm (python-lxml): Update to 4.7.1.
>
> Unfortunately this one has too many dependencies, as reported by
> ‘guix refresh -l python-lxml’.
>
> So I can apply it on the ‘core-updates’ branch, but it won’t be merged
> before a few months.  Or you can define it as a variant, and we’d keep
> both versions in ‘master’.
>
> WDYT?
>
> Ludo’.
>
Ludovic Courtès April 15, 2022, 5:20 p.m. UTC | #3 
Hi!

I ended up with a middleground solution in commit
ff3355dcbf155fcdb3ff1502efe13d969c5d6979.

Let me know if anything’s amiss.

Thanks,
Ludo’.
Chadwain Holness April 19, 2022, 2:30 a.m. UTC | #4 
Looks good to me, thanks!
diff mbox series

Patch

diff --git a/gnu/packages/xml.scm b/gnu/packages/xml.scm
index 49275c3e0c..50f552e048 100644
--- a/gnu/packages/xml.scm
+++ b/gnu/packages/xml.scm
@@ -2500,13 +2500,13 @@  (define-public python-elementpath
 (define-public python-lxml
   (package
     (name "python-lxml")
-    (version "4.6.3")
+    (version "4.7.1")
     (source
      (origin
        (method url-fetch)
        (uri (pypi-uri "lxml" version))
        (sha256
-        (base32 "0s14r1w2x9sdlcsw8mxiqgw4rz5zs5lpqpxrfyn4a1mkndqqbdrr"))
+        (base32 "090viyanaki4q7w7i000xl0qh4in52bkl3qal55sz2bbm8w3hqd1"))
        ;; Adapt a test to libxml2 2.9.12, taken from this commit:
        ;; https://github.com/lxml/lxml/commit/852ed1092bd80b6b9a51db24371047e
        (modules '((guix build utils)))