mapbox / mapbox-cli-py

Command line interface to Mapbox Web Services
MIT License
164 stars 44 forks source link

Latest version on pypi is no longer installing properly #139

Open SiggyF opened 1 year ago

SiggyF commented 1 year ago
### What you've tried already I tried to install the latest version of mapboxcli using pipx install. This failed I looked at the sourcecode to find the cause of the failure. ### Expected outcome `$ pipx install mapboxcli ` should install the mapbox command line tool ### Actual outcome I got the following error: ``` packaging.requirements.InvalidRequirement: Expected closing RIGHT_PARENTHESIS python-dateutil (>=2.5.0requests) ``` ### Other information * Version numbers for `mapboxcli` and `mapbox` (from, e.g., `pip list`) : mapboxcli 0.8.0, mapbox 0.16.1 * Operating System : MacOS 13 * Python version number : py39, py310 * How did you install the mapbox cli? Homebrew, pip, other? : pipx My suggested solution would be to release a new version of mapbox-cli-py to pypi with an upgraded mapbox-sdk-py The bug (missing comma, resulting in concatenation of 2 dependencies python-dateutil and requests) has been fixed in the 0.18.0 release in this commit: https://github.com/mapbox/mapbox-sdk-py/commit/f52fa309387584daf96191bc12527ee8e02c83cc The packaging module has become stricter in checking semantic version numbers, that is why 2.5.0requests is no longer accepted as a valid version string. The requests package is still installed due to the boto3 dependency. ### Proposed solution Upgrade mapbox dependency to 0.18 in setup.py and make a new release to pypi of mapbox-cli-py.