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

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

Open
danjde opened this issue Oct 10, 2020 · 3 comments

Comments

@danjde
Copy link

danjde commented Oct 10, 2020

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
Copy link
Collaborator

t-book commented Oct 10, 2020

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
Copy link
Author

danjde commented Oct 10, 2020

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
Copy link
Collaborator

t-book commented Oct 10, 2020

+1 thanks @danjde

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

No branches or pull requests

2 participants