FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-97669: Remove outdated example scripts (#97675) by vstinner · Pull Request #98167 · python/cpython · GitHub

Repository navigation

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

Merged
vstinner merged 2 commits into
python:mainfrom
vstinner:remove_scripts2
Oct 11, 2022
Merged

vstinner merged 2 commits into
python:mainfrom
vstinner:remove_scripts2

Conversation

vstinner commented Oct 10, 2022 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

Remove outdated example scripts of the Tools/scripts/ directory:

  • gprof2html.py
  • md5sum.py
  • nm2def.py
  • pathfix.py
  • win_add2path.py

Remove test_gprof2html, test_md5sum and test_pathfix of test_tools.

Remove outdated example scripts of the Tools/scripts/ directory:

* gprof2html.py
* md5sum.py
* nm2def.py
* pathfix.py
* win_add2path.py

Remove test_gprof2html, test_md5sum and test_pathfix of test_tools.

Copy link
Copy Markdown
Member Author

gprof2html.py

Created in 2002 by @gvanrossum. Following changes are mostly coding styles changes, no feature or bugfix.

I didn't use gprof format for a long time. There are now many tools to profile programs with nice UI. I don't think that maintaining such tool in Python with a HTML output is still relevant today.

md5sum.py

If someone wants such feature as part of Python, I would suggest to add a command line interface (CLI) to the hashlib module instead.

The script doesn't support multithreading, whereas Python releases the GIL in hashlib. The script uses the old getopt module.

@gvanrossum wrote in 2001:

This is a trivial command line utility to print MD5 checksums.
I published it on the web as http://www.python.org/2.1/md5sum.py
so I thought I might as well check it in.

Well, I don't think that it's still relevant in 2022. There are now many ways to easily distribute Python scripts (pip, GitHub, whatever works).

nm2def.py

Helpers to extract symbols from Unix libs and auto-generate
Windows definition files from them. Depends on nm(1). Tested
on Linux and Solaris only (-p option to nm is for Solaris only).

@malemburg: if you still use this script, it should be maintained outside Python. Python doesn't use it.

pathfix.py

@hroncok: Script used by Fedora, see: https://discuss.python.org/t/remove-outdated-tools-scripts-scripts/19571/35

It can be maintained somewhere else.

win_add2path.py

Not used by the Windows installer: https://discuss.python.org/t/remove-outdated-tools-scripts-scripts/19571/20

gvanrossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Kill 'em with fire!

Copy link
Copy Markdown

Hi @vstinner ,
Is there any alternate ways to use the pathfix.py or patch updates for consuming the file? Refer: https://gitlab.com/redhat/centos-stream/rpms/python3.14/-/blob/c10s/python3.14.spec?ref_type=heads#L1325

Copy link
Copy Markdown
Member

Is there any alternate ways to use the pathfix.py or patch updates for consuming the file?

Yes, by copying the script out of an old branch into your repo.

Copy link
Copy Markdown
Member Author

Is there any alternate ways to use the pathfix.py or patch updates for consuming the file? Refer: https://gitlab.com/redhat/centos-stream/rpms/python3.14/-/blob/c10s/python3.14.spec?ref_type=heads#L1325

The python3.14.spec that you linked refers to %{_rpmconfigdir}/redhat/pathfix.py which gets pathfix.py from /usr/lib/rpm/redhat/pathfix.py. The file is installed by python-rpm-macros on my Fedora. So pathfix.py is now maintained by the python-rpm-macros project.

The CentOS Stream package doesn't get pathfix.py from Python but from python-rpm-macros.

vstinner commented Aug 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

Update: Ah, in fact pathfix.py is maintained in the fedora-python/pathfix repository.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL