-
Notifications
You must be signed in to change notification settings - Fork 6
fix: Windows - run-clang-tidy.py not found in your PATH (IDFGH-14140) #50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Windows - run-clang-tidy.py not found in your PATH (IDFGH-14140) #50
Conversation
6562648
to
64098a5
Compare
@mfialaf LGTM, I'm wondering if adding |
LGTM, I would try to add |
64098a5
to
3d1bda8
Compare
@fhrbata While debugging I've tried to extend the The added dot did not come to my mind. Good point, it works! |
Thank you @fhrbata and @dobairoland for reviewing and @peterdragun for help with debugging setup. |
3d1bda8
to
4969ec9
Compare
@mfialaf Thank you for trying it. Still LGTM. Thanks |
@mfialaf Thank you. Could you please make a release as well so we can integrate this fix back to ESP-IDF? |
The bug causes the
shutil.which()
function on Windows. When used, it tries to append all possible extensions to the searched command fromos.environ['PATHEXT']
. Since the searched command isrun-clang-tidy
without extension, the search fails.https://docs.python.org/3/library/shutil.html#shutil.which
Adding '.' to
PATHEXT
ensures to find the full path torun-clang-tidy
.As stated in the documentation, lookup for files without extension works in Python 3.12, without modification.