hai shi <shihai1...@126.com> added the comment: Sorry for my delay~
>I think some of these work with any Python, but some need the locally built >interpreter to give correct results. >It could avoid confusion and mistakes to remove the shebangs and executable >bits from the scripts that need a local Python, and make dure their >documentation mentions `./python path/to/tool`. I update Objects/typeslots.py in #PR21913 recently and using f-format in this script(it means that just local python or python3 can run). So I will accept your suggestion from this file after this PR merged:) >(Oh and by the way, Tools/scripts/fixps.py is a tool to replace >`/usr/local/bin/python` shebangs with `/usr/bin/env python`… You can judge if >that is useful and/or correct.) Thanks, I will take a look. > So if things are changed (let’s get a couple opinions here), maybe this needs > separate PRs for 1/2/3, and make sure to get reviews to ensure we don’t break > IDLE / Mac installers / pegen / etc. Agree. To corner case, a single sepereate PR is good. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <https://bugs.python.org/issue41626> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
