Issue 36086: Split IDLE into separate feature in Windows installer (original) (raw)

Created on 2019-02-23 01:03 by kimsey0, last changed 2022-04-11 14:59 by admin. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 12023 closed kimsey0,2019-02-24 21:43
Messages (8)
msg336352 - (view) Author: Jacob Bundgaard (kimsey0) * Date: 2019-02-23 01:03
I don't use IDLE to edit Python files, but do use tcl/tk for Python projects on Windows. Therefore, it would be useful for me to be able to install tcl/tk without also installing IDLE. However, in the Windows installer, tcl/tk and IDLE are bundled together into one feature. Splitting them into two features (the IDLE feature requiring the tcl/tk one) would reduce installation time, storage use, Explorer context menu cluttering, etc. for users like me.
msg336378 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2019-02-23 10:26
The installed space for /lib/idlelib it is roughly 2% of the total, depending on other options. The time saved is, I think, roughly comparable to the extra time it would take to read and think about another option. So the main issue would be making 'Edit with IDLE' optional.
msg336379 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2019-02-23 14:03
Right, Idle is tiny and painless compared to Tk, and adding additional options to the installer isn't trivial. I might consider a patch, but I'm not doing this myself.
msg336484 - (view) Author: Jacob Bundgaard (kimsey0) * Date: 2019-02-24 21:45
I'm glad to hear that you might consider a patch. I've created a draft pull request for this, but I still need to figure out how to encode the dependency from IDLE to Tcl/Tk in WIX. Do you have any insights?
msg337404 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2019-03-07 16:42
For the dependency you'll need to modify the bootstrapper to manually enforce checkbox selection. It'll also need to force the variable at planning time to support command line installs, so the checkbox is really just to help prevent users from becoming confused, but we'll want both. IIRC, there's already something along those lines in there for another component (maybe file associations requiring the launcher?), so you can copy from that.
msg385622 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2021-01-25 12:19
turtledemo currently requires a couple of modules from idlelib. One to syntax color examples, another to display the help file. So turtledemo would have to be patched if IDLE were separate from tkinter.
msg385654 - (view) Author: Andre Roberge (aroberge) * Date: 2021-01-25 19:44
I do not use IDLE (except for testing) and would not be affected by such a change. However, many tutorials and books intended for beginners instruct users to use IDLE, with the assumption that it is available out of the box. Removing IDLE would immediately make instructions from those tutorials and books obsolete and needlessly confuse beginners who often struggle installing anything additional to a basic Python installation. I just checked and if I right-click on a Python file, I see 29 entries in the contextual menu, 28 of which would be unaffected if IDLE would be removed. If IDLE were not included by default (or if it required clicking on a checkbox), I think that the number of questions in various beginners forum about installing IDLE would quickly become a real annoyance for people that strive to help beginners.
msg385841 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2021-01-28 06:19
I concur with the other commenters that this isn't worth it. Installing Tcl/Tk without IDLE saves almost nothing. Thank you for the suggestion, but I'm going to mark it as rejected.
History
Date User Action Args
2022-04-11 14:59:11 admin set github: 80267
2021-01-28 06:19:11 rhettinger set status: open -> closednosy: + rhettingermessages: + resolution: rejectedstage: patch review -> resolved
2021-01-25 19:44:18 aroberge set nosy: + arobergemessages: +
2021-01-25 12:19:58 terry.reedy set messages: +
2019-03-07 16:42:20 steve.dower set messages: +
2019-02-24 21:45:45 kimsey0 set messages: +
2019-02-24 21:43:18 kimsey0 set keywords: + patchstage: patch reviewpull_requests: + <pull%5Frequest12055>
2019-02-23 14:03:24 steve.dower set messages: +
2019-02-23 10:26:50 terry.reedy set messages: +
2019-02-23 07:42:26 xtreak set nosy: + terry.reedy
2019-02-23 07:02:37 SilentGhost set nosy: + paul.moore, tim.golden, zach.ware, steve.dower
2019-02-23 01:03:02 kimsey0 create