cloud-custodian / cel-python

Pure Python implementation of the Common Expression Language
Apache License 2.0
116 stars 22 forks source link

Fix pip sdist install #37

Closed eandersson closed 2 years ago

eandersson commented 2 years ago

With the current implementation of setup.py the source distribution cannot work because requirements.txt isn't included. This is reproducible with the following command pip install cel-python --no-binary :all:.

FileNotFoundError: [Errno 2] No such file or directory: 'requirements.txt'

linux-foundation-easycla[bot] commented 2 years ago

CLA Signed

The committers listed above are authorized under a signed CLA.

kapilt commented 2 years ago

Thanks for the pull request! If you don't mind could you sign the Linux Foundation/CNCF CLA referenced in the other comment.

eandersson commented 2 years ago

Thanks for the pull request! If you don't mind could you sign the Linux Foundation/CNCF CLA referenced in the other comment.

Thanks for the reminder. Fixed.