Skip to content

#277 - update lxml requirements - #278

Closed
eljeffeg wants to merge 1 commit into
xmlsec:masterfrom
eljeffeg:patch-1
Closed

eljeffeg wants to merge 1 commit into
xmlsec:masterfrom
eljeffeg:patch-1

Conversation

@eljeffeg

@eljeffeg eljeffeg commented Jan 4, 2024

Copy link
Copy Markdown

lxml 5.0.0 is incompatible with the current version of python-xmlsec

@Jasper-Koops Jasper-Koops left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are doing the same thing to fix an issue on our side.

@nosnilmot

Copy link
Copy Markdown
Contributor

Use pip install --no-binary lxml lxml to install lxml from source and link to system libxml2, same as xmlsec, to avoid conflicting libxml2 libraries.
See https://bugs.launchpad.net/lxml/+bug/1960668#yui_3_10_3_1_1705879387759_1639
Also see PR #246

@eljeffeg

eljeffeg commented Mar 1, 2024

Copy link
Copy Markdown
Author

Closing in favor of #246 (though I haven't tested that fix - trusting @nosnilmot )

@eljeffeg eljeffeg closed this Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants