Add upper bound for lxml

[cdce8p]

lxml==4.9.3 was released yesterday which caused the wheel builder to fail again.
https://github.com/mypyc/mypy_mypyc-wheels/actions/runs/5473072463/jobs/9966070811#step:4:6979

The lxml project decided to change the manylinux builds from manylinux2014 to manylinux_2_28. Thus to test our wheels we would need to compile lxml from source. Until will update to manylinux_2_28 ourselves, I think pinning lxml<4.9.3 is the best option.

https://pypi.org/project/lxml/4.9.2/#files
https://pypi.org/project/lxml/4.9.3/#files

[AlexWaygood]

Should we also add an upper bound here?

mypy/setup.py

Line 231 in e0b159e

"reports": "lxml",

[cdce8p]

Should we also add an upper bound here?

mypy/setup.py

Line 231 in e0b159e

"reports": "lxml",

Not sure. It's not impossible to build lxml from source. Just the docker image to build the mypy wheels isn't setup to do that. Furthermore if the target OS already ships with glibc>=2.28 the wheels for lxml can be installed as usual.

Also, we don't prevent anyone from installing mypy[reports] on windows. We just don't install it as test requirement there.

[github-actions]

According to mypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅

[hauntsaninja]

Thanks! I looked into whether it makes sense for us to use 2_28 too, and I'm feeling like nah

[hauntsaninja]

Thanks! I looked into whether it makes sense for us to use 2_28 too, and I'm feeling like nah