GeoNode / documentation

GeoNode Documentation
https://docs.geonode.org/
Other
14 stars 73 forks source link

Wrong commands to run to set python variables on GeoNode Advanced Installation -> GeoNode Installation/Create the GeoNode Virtual Environment #56

Open danjde opened 3 years ago

danjde commented 3 years ago

Hi Devs,

Building the services as your detailed indications, on GeoNode Advanced Installation -> GeoNode Installation/Create the GeoNode Virtual Environment, are present two different command, producing two different results.

The following command set Python environment to 3.6.x and not to 3.8.x: mkvirtualenv --python=/usr/bin/python3 geonode

The right one should be: mkvirtualenv --python=/usr/local/bin/python3.8 geonode

please check

Thanks!

t-book commented 3 years ago

Hi @danjde doesn't exactly this warning inform a user how to create the env?

Bildschirmfoto 2020-10-10 um 16 13 33

and later

Bildschirmfoto 2020-10-10 um 16 11 53

https://docs.geonode.org/en/master/install/advanced/core/index.html?highlight=3.7#packages-installation

There might be the case that /usr/bin/python3 id 3.7+ in that case you will not have to install a later python version. However your contribution is very welcome. If you think this can be optimized feel free to create a PR that we can review.

danjde commented 3 years ago

All true, but the "! warning" from my point of view, should be used for non-ordinary situations. Ubuntu 18.04 doesn't natively have Python 3.8, but at least 3.6.

So this "! warning", always in my opinion, should take the place of the "! note" and remove the current "! note" .

The current "! note" brings the user who followed the previous procedure (all Ubuntu 18.04 standard set systems) to set Python to 3.6 version.

This is not needed just makes a mess, and it also creates problems for anyone who is using a native version (or just installed):

Schermata del 2020-10-10 18-57-40

Because contains this wrong (pointing to Python 3.6) path.

And this point I don't know what else to tell you, you do.

t-book commented 3 years ago

+1 thanks @danjde