This is an archive of the discontinued LLVM Phabricator instance.

Differential D143523

[mlir] Relax version requirement for PyYAML in mlir
ClosedPublic

Authored by stella.stamenova on Feb 7 2023, 12:59 PM.

Details

Reviewers
stellaraccident
ftynse
Commits
rG97fbbd657f9a: [mlir] Relax version requirement for PyYAML in mlir
Summary

Some Ubuntu 20.04 images come with PyYAML 5.3.1 pre-installed through distutils. This makes pip very angry. See https://github.com/yaml/pyyaml/issues/349.

Since older versions of PyYAML should work for mlir, relax the version requirement to ease developer setup.

Diff Detail

Event Timeline

stella.stamenova created this revision.Feb 7 2023, 12:59 PM
Herald added a reviewer: ftynse. · View Herald TranscriptFeb 7 2023, 12:59 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added subscribers: Moerafaat, zero9178, bzcheeseman and 21 others. · View Herald Transcript
stella.stamenova requested review of this revision.Feb 7 2023, 12:59 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 7 2023, 12:59 PM
Herald added subscribers: stephenneuendorffer, nicolasvasilache. · View Herald Transcript
stellaraccident accepted this revision.Feb 7 2023, 1:22 PM
This revision is now accepted and ready to land.Feb 7 2023, 1:22 PM
Harbormaster completed remote builds in B212458: Diff 495620.Feb 7 2023, 2:18 PM
Closed by commit rG97fbbd657f9a: [mlir] Relax version requirement for PyYAML in mlir (authored by stella.stamenova). · Explain WhyFeb 7 2023, 2:25 PM
This revision was automatically updated to reflect the committed changes.
stella.stamenova added a commit: rG97fbbd657f9a: [mlir] Relax version requirement for PyYAML in mlir.

Revision Contents

PathSize
mlir/
python/
2 lines

Diff 495644

mlir/python/requirements.txt

numpy>=1.19.5, <=1.23.5 numpy>=1.19.5, <=1.23.5
pybind11>=2.8.0, <=2.10.3 pybind11>=2.8.0, <=2.10.3
PyYAML==6.0 PyYAML>= 5.3.1, <=6.0
dataclasses>=0.6, <=0.8 dataclasses>=0.6, <=0.8