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 manual: Fix documentation of python prerequisite (sage-bootstrap-python) #31192

Closed
mkoeppe opened this issue Jan 6, 2021 · 14 comments

Comments

@mkoeppe
Copy link
Member

mkoeppe commented Jan 6, 2021

(split out from #30551)

sage-bootstrap-python which has much wider range of supported python versions, including Python >= 2.6 and Python >= 3.4 - according to build/tox.ini.

CC: @jhpalmieri @tobiasdiez @dimpase

Component: documentation

Author: Tobias Diez, Matthias Koeppe

Branch: 4265656

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/31192

@mkoeppe mkoeppe added this to the sage-9.3 milestone Jan 6, 2021
@tobiasdiez
Copy link
Contributor

comment:1

What is wrong with the current description? If an old Python is installed, then sage bootstrap downloads and (locally) installs a new python, right? This seems to be a huge enough workaround to justify the terminology "deprecated" (which implies that it does work, but is no longer recommended). Or do you mean something else?

@mkoeppe
Copy link
Member Author

mkoeppe commented Jan 6, 2021

@mkoeppe
Copy link
Member Author

mkoeppe commented Jan 6, 2021

Commit: 4265656

@mkoeppe
Copy link
Member Author

mkoeppe commented Jan 6, 2021

Author: Tobias Diez, Matthias Koeppe

@mkoeppe
Copy link
Member Author

mkoeppe commented Jan 6, 2021

New commits:

4265656src/doc/en/installation/source.rst: Fix documentation of python prerequisite (sage-bootstrap-python)

@dimpase
Copy link
Member

dimpase commented Jan 10, 2021

comment:4

is Python 2.6 actually tested to work? IMHO asking for 2.7 is fine.

@tobiasdiez
Copy link
Contributor

comment:5

This part of the documentation is for "normal" users right? Then maybe the reference to sage-bootstrap-python is more confusing than helpful for the target group. Personally, I would also add a "Version 3.7 or later is recommended".

@jhpalmieri
Copy link
Member

comment:6

I think the phrasing is okay: I think it makes it clear that it is a technical term and people can freely ignore it. I haven't actually tested to see whether Python 2.6 will work, but I think we should merge this. Any objections?

@dimpase
Copy link
Member

dimpase commented Jan 13, 2021

Reviewer: Dima Pasechnik

@dimpase
Copy link
Member

dimpase commented Jan 13, 2021

comment:7

ok, fine

@mkoeppe
Copy link
Member Author

mkoeppe commented Jan 13, 2021

comment:8

Thanks.

@vbraun
Copy link
Member

vbraun commented Jan 24, 2021

@mkoeppe
Copy link
Member Author

mkoeppe commented Mar 4, 2021

comment:10

Follow-up in #20023

@mkoeppe
Copy link
Member Author

mkoeppe commented Mar 4, 2021

Changed commit from 4265656 to none

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

No branches or pull requests

5 participants