Description

vstinner

opened on Sep 30, 2022

Issue body actions

The Tools/scripts/ directory contains more than 70+ example scripts. Most of them are outdated and short scripts Python which are simple to rewrite. When I asked other core developers, they suggest to remove all of them, see the discussion:

https://discuss.python.org/t/remove-outdated-tools-scripts-scripts/19571

I created this issue to look into these scripts and see what to do with them.

Activity

1. vstinner

   added

   type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

   on Sep 30, 2022
2. vstinner commented on Sep 30, 2022

   

   vstinner

   on Sep 30, 2022

   MemberAuthor

   More actions

   analyze_dxp.py: I created issue #97670 to remove the sys.getdxp() function and the Tools/scripts/analyze_dxp.py script.
3. vstinner commented on Sep 30, 2022

   

   vstinner

   on Sep 30, 2022

   MemberAuthor

   More actions

   See also issue #97649: Do not install Tools directory on Windows.
4. vstinner commented on Sep 30, 2022

   

   vstinner

   on Sep 30, 2022

   MemberAuthor

   More actions

   I created PR #97675 to remove a first batch of outdated example scripts.
5. vstinner commented on Oct 3, 2022

   

   vstinner

   on Oct 3, 2022

   MemberAuthor

   More actions

   @warsaw @gpshead @gvanrossum: Would you mind to approve formally this PR since you are in favor of removing Tools/scripts/ scripts according to your messages on https://discuss.python.org/t/remove-outdated-tools-scripts-scripts/19571?
6. vstinner

   added a commit that references this issue on Oct 4, 2022

   gh-97669: Remove outdated example scripts (#97675)

   6cbbc26
7. vstinner commented on Oct 6, 2022

   

   vstinner

   on Oct 6, 2022

   MemberAuthor

   More actions

   I created a draft PR #97963 to create a new Tools/build/ sub-directory which contains scripts used to build Python. I moved 13 scripts there. Not sure if it's a good idea or not.

   With PR #97963 and PR #97964, the remaining scripts in Tools/scripts/ are:

   Programs (3):

   • 2to3
   • idle3
   • pydoc3

   Scripts (19):

   • abitype.py
   • checkpip.py
   • combinerefs.py
   • eptags.py
   • gprof2html.py
   • md5sum.py
   • nm2def.py
   • patchcheck.py
   • pathfix.py
   • pep384_macrocheck.py
   • ptags.py
   • reindent.py
   • run_tests.py
   • startuptime.py
   • summarize_stats.py
   • untabify.py: used by patchcheck.py
   • var_access_benchmark.py
   • verify_ensurepip_wheels.py
   • win_add2path.py

   patchcheck.py, run_tests.py and verify_ensurepip_wheels.py are used by the Python workflow. Does it deserve its own sub-directory?
8. vstinner commented on Oct 6, 2022

   

   vstinner

   on Oct 6, 2022

   MemberAuthor

   More actions

   I created PR #97964 to move diff.py and ndiff.py scripts to the difflib documentation (to Doc/includes/).
9. ambv

   added a commit that references this issue on Oct 7, 2022

   gh-97669: Move difflib examples to Doc/includes/ (#97964)

   002252c
10. vstinner commented on Oct 10, 2022

    

    vstinner

    on Oct 10, 2022

    MemberAuthor

    More actions

    Oh nice, 4ed00be removed eptags.py and ptags.py.

    Reacted by Gregory P. Smith
11. vstinner commented on Oct 10, 2022

    

    vstinner

    on Oct 10, 2022

    MemberAuthor

    More actions

    PR to remove 5 more scripts: PR #98167

    PR to remove abitype.py and pep384_macrocheck.py: PR #98165
12. mpage

    added a commit that references this issue on Oct 11, 2022

    pythongh-97669: Move difflib examples to Doc/includes/ (python#97964)

    97ae2e9
13. vstinner

    added 5 commits that reference this issue on Oct 11, 2022

    gh-97669: Remove outdated example scripts (#97675) (#98167)

    e0ae9dd

    gh-97669: Remove abitype.py and pep384_macrocheck.py (#98165)

    454a6d6

    gh-97669: Create Tools/patchcheck/ directory (#98186)

    0895c2a

    gh-97669: Fix test_tools reference leak (#98216)

    a8c8526

    gh-97669: Remove Tools/scripts/startuptime.py (#98214)

    d09d2c7
14. vstinner commented on Oct 12, 2022

    

    vstinner

    on Oct 12, 2022

    MemberAuthor

    More actions

    Updated list of Tools/scripts/ scripts. With PR #97963, there are only 8 remaining scripts.

    Programs (3):

    • 2to3
    • idle3
    • pydoc3

    Misc (5):

    • checkpip.py
    • combinerefs.py: I started a discussion about this script
    • run_tests.py
    • summarize_stats.py
    • var_access_benchmark.py: see issue Move Benchmarks From Tools/ to the pyperformance Repo #97680

    Update: I moved verify_ensurepip_wheels.py to Tools/build/ in my PR #97963.
15. carljm

    added 2 commits that reference this issue on Oct 14, 2022

    Merge branch 'main' into typewatch

    6e53514

    Merge branch 'main' into cached_property

    0bb0c69
16. vstinner

    added a commit that references this issue on Oct 17, 2022

    gh-97669: Create Tools/build/ directory (#97963)

    1863302
17. vstinner commented on Oct 17, 2022

    

    vstinner

    on Oct 17, 2022

    MemberAuthor

    More actions

    I moved 16 scripts to a new Tools/build/ directory: 1863302

    There are 9 remaining files in Tools/scripts/, including the README file (so 8 scripts):

    2to3
    checkpip.py
    combinerefs.py
    idle3
    pydoc3
    README
    run_tests.py
    summarize_stats.py
    var_access_benchmark.py
    
18. carljm

    added a commit that references this issue on Oct 17, 2022

    Merge branch 'main' into typewatch

    cdbe93b
19. vstinner commented on Nov 3, 2022

    

    vstinner

    on Nov 3, 2022

    MemberAuthor

    More actions

    I removed enough scripts, I close the issue :-)
20. vstinner

    closed this as completedon Nov 3, 2022

Issue actions