Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump xmlsec version #22000

Closed
wants to merge 1 commit into from
Closed

Bump xmlsec version #22000

wants to merge 1 commit into from

Conversation

aspicer
Copy link
Contributor

@aspicer aspicer commented May 1, 2024

xmlsec released a new minor version that doesn't have the insane build issue mentioned on the "develop locally" page for macs anymore
Friendly tip: If you see ERROR: Could not build wheels for xmlsec, refer to this [issue](https://github.com/xmlsec/python-xmlsec/issues/254).

As discussed here: xmlsec/python-xmlsec#284

Bump it.

@aspicer aspicer closed this May 1, 2024
@aspicer aspicer deleted the aspicer/bump_xml branch May 1, 2024 00:37
@aphexcx
Copy link

aphexcx commented May 10, 2024

I just noticed this too trying to build posthog locally, why not merge this in?

@aphexcx
Copy link

aphexcx commented May 10, 2024

Oh, I just noticed I get this when manually bumping it to 1.3.14:

...site-packages/onelogin/saml2/auth.py", line 14, in <module>
    import xmlsec
xmlsec.InternalError: (-1, 'lxml & xmlsec libxml2 library version mismatch')

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.

2 participants