colcon / colcon.readthedocs.org

https://colcon.readthedocs.org
Creative Commons Attribution 4.0 International
10 stars 27 forks source link

Update requirements.txt to pin EmPy version #98

Closed sdmcgrath closed 3 months ago

sdmcgrath commented 3 months ago

Reflects changes here: https://github.com/colcon/colcon-core/pull/603/

Otherwise building colcon for the first time per https://colcon.readthedocs.io/en/released/developer/bootstrap.html fails due to

ImportError: cannot import name 'OVERRIDE_OPT' from 'em' (/$USER_DIR/colcon-venv/lib/python3.11/site-packages/em.py) The Python package 'empy' must be installed and 'em' must not be installed since both packages share the same namespace

nuclearsandwich commented 3 months ago

This change was actually already proposed as #95, which I've merged. Thanks for contributing!