This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Change shebang from python to python3
ClosedPublic

Authored by hokein on Apr 22 2021, 2:23 AM.

Details

Diff Detail

Event Timeline

hokein created this revision.Apr 22 2021, 2:23 AM
hokein requested review of this revision.Apr 22 2021, 2:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 22 2021, 2:23 AM
hokein edited the summary of this revision. (Show Details)Apr 22 2021, 2:36 AM
hokein added reviewers: tdl-g, aaron.ballman.
hokein edited the summary of this revision. (Show Details)
keith added a subscriber: keith.Apr 22 2021, 9:26 AM

I also have this patch https://reviews.llvm.org/D100692 which changes run-clang-tidy.py this way as well, specifically because it utilizes some python3 features in shutil.which

tdl-g accepted this revision.Apr 22 2021, 10:34 AM

I'll defer to the consensus on https://lists.llvm.org/pipermail/cfe-dev/2021-April/068047.html regarding whether or not there are gotchas for requiring python3,, but assuming tests have confirmed that each of these scripts are python3 compatible, this seems reasonable to me.

This revision is now accepted and ready to land.Apr 22 2021, 10:34 AM
keith added a comment.Jul 9 2021, 10:15 AM

Doesn't seem like anything changed on the mailing list side, should we land this now?

keith added a comment.Sep 10 2021, 6:05 PM

@hokein any plans to land this?

+1 to committing this.

xgupta closed this revision.Jul 23 2023, 6:55 AM
xgupta added a subscriber: xgupta.

https://reviews.llvm.org/rG475440703238ca32adab6c3fe5e0039c3f96d1a5 committed the remaining shebang change from python to python3.

Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: PiotrZSL. · View Herald Transcript