Skip to content

Conversation

DavidSpickett
Copy link
Contributor

8fc27f4 pinned MarkupSafe to 1.1.1, which lead to this error on our buildbots that are using Python 3.10 to run the lnt client:
error: MarkupSafe 1.1.1 is installed but MarkupSafe>=2.0 is required by {'Jinja2'}

Jinja2 2.11.3 is the last version that will work with that version of MarkupSafe:
https://github.com/pallets/jinja/blob/2.11.3/setup.py

The 3.x series moved to >= 2.0.0:
https://github.com/pallets/jinja/blob/3.0.0rc1/setup.py

8fc27f4 pinned MarkupSafe to
1.1.1, which lead to this error on our buildbots that are using
Python 3.10 to run the lnt client:
error: MarkupSafe 1.1.1 is installed but MarkupSafe>=2.0 is required by {'Jinja2'}

Jinja2 2.11.3 is the last version that will work with that version of
MarkupSafe:
https://github.com/pallets/jinja/blob/2.11.3/setup.py

The 3.x series moved to >= 2.0.0:
https://github.com/pallets/jinja/blob/3.0.0rc1/setup.py
@DavidSpickett
Copy link
Contributor Author

I know I'm risking blowing up some other workflow here, but it seems one worth taking to fix this short term. I see tox failures but those were mentioned elsewhere and you want to drop 3.9 anyway.

If it would help to move the buildbots away from the setup.py install method, I can do that. As long as we have some overlapping time where the old and new methods work, it's not hard to fix.

@DavidSpickett DavidSpickett merged commit 11c2d9e into llvm:main Oct 7, 2025
2 of 3 checks passed
@DavidSpickett DavidSpickett deleted the fix-jinja2 branch October 7, 2025 10:29
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.

1 participant