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.

Sign up for GitHub

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

Jump to bottom

correct Tkinter (TclTk)-related instructions in installation manual #36710

Closed
2 tasks done
dimpase opened this issue Nov 12, 2023 · 1 comment · Fixed by #36719
Closed
2 tasks done

correct Tkinter (TclTk)-related instructions in installation manual #36710

dimpase opened this issue Nov 12, 2023 · 1 comment · Fixed by #36719
Labels
t: bug
Milestone
sage-10.3

Comments

@dimpase
Copy link
Member

dimpase commented Nov 12, 2023

Steps To Reproduce

the docs only say what to do if Python is built by Sage

Expected Behavior

if Python comes from OS, then no Python (re)building in Sage is needed

Actual Behavior

nothing is said for this system Python case, so people try building Sage's Python for no good reason

Additional Information

No response

Environment

all platforms

Checklist

  • I have searched the existing issues for a bug report that matches the one I want to file, without success.
  • I have read the documentation and troubleshoot guide
@dimpase dimpase added the t: bug label Nov 12, 2023
@mkoeppe
Copy link
Member

mkoeppe commented Nov 12, 2023

See also #31206

@dimpase dimpase changed the title correct Tinker (TclTk)-related instructions in installation manual correct Tkinter (TclTk)-related instructions in installation manual Nov 12, 2023
dimpase added a commit to dimpase/sage that referenced this issue Nov 14, 2023
@dimpase
rectify docs around Tkinter
f8e0f60 
Will fix sagemath#36710
@dimpase dimpase mentioned this issue Nov 14, 2023
Merged
vbraun pushed a commit to vbraun/sage that referenced this issue Dec 13, 2023
sagemathgh-36719: rectify docs around Tkinter
07a5b27 
    
our instructions on Tkinter are outdated, as they don't take into
account possible use of system Python in Sage.
This is a fix for this issue, which confuses users, see e.g. sage-
develop [here](https://groups.google.com/g/sage-
devel/c/IeGdueQOmEw/m/NNVAYqp4AQAJ)

The Italian FAQ has been done by Matteo, a native speaker.

Will fix sagemath#36710
    
URL: sagemath#36719
Reported by: Dima Pasechnik
Reviewer(s): Dima Pasechnik, Kwankyu Lee, Matthias Köppe
@vbraun vbraun closed this as completed in 28a7848 Dec 14, 2023
@mkoeppe mkoeppe added this to the sage-10.3 milestone Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
t: bug
Projects
None yet
Milestone
sage-10.3
Development

Successfully merging a pull request may close this issue.

rectify docs around Tkinter dimpase/sage
2 participants
@dimpase @mkoeppe