search

ubarsc / rios

A raster processing layer on top of GDAL
https://www.rioshome.org
GNU General Public License v3.0
15 stars 8 forks source link

Update INSTALL.txt to reflect recommendation for using pip around the… #46

Closed neilflood closed 2 years ago

neilflood commented 2 years ago

… setup.py

neilflood commented 2 years ago

The only other place I can find any mention of "setup.py install" is in the .github/workflows/python-app.yml. Do you think we should change it to also use "pip install ." ?

gillins commented 2 years ago

The only other place I can find any mention of "setup.py install" is in the .github/workflows/python-app.yml. Do you think we should change it to also use "pip install ." ?

I'd say yes just so we are checking our instructions. But no strong thoughts. Perhaps we should be doing this for the conda recipe also (from the next release)?

neilflood commented 2 years ago

OK, updated the Github checks, and that seems to work fine. Less verbose, too, which is a minor plus.

Yes, you are probably right, the conda recipe should also change.