Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

installation via pypi does not install requirements automatically #14

Closed
Meibes opened this issue Aug 11, 2021 · 2 comments
Closed

installation via pypi does not install requirements automatically #14

Meibes opened this issue Aug 11, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@Meibes
Copy link

Meibes commented Aug 11, 2021

Hello,

I just created a new python environment and installed ogr2osm via "pip3 install ogr2osm". The installation was successful but pip did not mention any installation of the requirements. I tried to convert a file and got the "ModuleNotFoundError: No module named 'lxml'". I think there are issues with the automatic installation of the requirements via PyPi. I did install GDAL manually before I installed ogr2osm, so there is probably another ModuleNotFoundError I just did not encounter on my setup.

I did a little research and it seems the setup.py needs to be modified like this:

setuptools.setup( ... install_requires=['lxml>=4.3.0', 'GDAL>=3.0.0'], ... )

more information on this topic can be found here.

I think it's better for the user if "install_requires" is used here.

Greetings

@roelderickx
Copy link
Owner

Thanks for your report 🙂

It's indeed a good idea to include this in the setup script, but allow me some time to test the modified package on an empty docker image.

However, it came to my attention there is currently no support for lxml on the apple M1 cpu. For now nobody complained and eventually M1 will be supported, but I may have to reinstate pnorman's fallback to ElementTree for apple users. In that case lxml would become strongly recommended in stead of required, I am not sure if this can be achieved with the install_requires parameter.

@roelderickx roelderickx added the enhancement New feature or request label Aug 11, 2021
@roelderickx
Copy link
Owner

Fixed in version 1.1.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants