The pre-commit hooks do not work on macOS #110652

Activity

1. erlend-aasland

   added

   OS-mac

   on Oct 10, 2023
2. erlend-aasland commented on Oct 10, 2023

   

   erlend-aasland

   on Oct 10, 2023

   ContributorAuthor

   More actions

   Simply executing these scripts directly would also do:

   $ head -n1 Tools/patchcheck/*.py
   ==> Tools/patchcheck/patchcheck.py <==
   #!/usr/bin/env python3
   
   ==> Tools/patchcheck/reindent.py <==
   #! /usr/bin/env python3
   
   ==> Tools/patchcheck/untabify.py <==
   #! /usr/bin/env python3
3. erlend-aasland

   added a commit that references this issue on Oct 10, 2023

   pythongh-110652: Fix pre-commit hooks on macOS

   6509e94
4. hugovk commented on Oct 10, 2023

   

   hugovk

   on Oct 10, 2023

   Member

   More actions

   Yep, see also comments at #109891 (comment).

   We can't use python3 because that won't work on Windows...

   Let's check if direct execution works on Windows.
5. AlexWaygood commented on Oct 10, 2023

   

   AlexWaygood

   on Oct 10, 2023

   Member

   More actions

   We can't use python3 because that won't work on Windows...

   Let's check if direct execution works on Windows.

   Unfortunately neither approach works on Windows: see #110653 (comment) ://
6. AlexWaygood commented on Oct 11, 2023

   

   AlexWaygood

   on Oct 11, 2023

   Member

   More actions

   I think we should revert these commits until we figure out how to fix this properly (if it can be "fixed properly"):

   • f5edb56
   • 08ec4a1

   👍React with 👍1Reacted by Hugo van Kemenade
7. hugovk commented on Oct 11, 2023

   

   hugovk

   on Oct 11, 2023

   Member

   More actions

   Agree to revert.

   I'd prefer to replace Tools/patchcheck/reindent.py and Tools/patchcheck/untabify.py with other pre-commit hooks.

   👍React with 👍1Reacted by Erlend E. Aasland
8. erlend-aasland commented on Oct 11, 2023

   

   erlend-aasland

   on Oct 11, 2023

   ContributorAuthor

   More actions

   I'd prefer to replace Tools/patchcheck/reindent.py and Tools/patchcheck/untabify.py with other pre-commit hooks.

   Surely there must be existing hooks we can use.

   If we need CPython specific hooks, we can create a repo for them under the GitHub python org. (for example python/pre-commit)

   👍React with 👍2Reacted by Hugo van Kemenade and Thomas Grainger
9. erlend-aasland

   closed this as not plannedon Oct 11, 2023