want some opinions on this.
see thread: https://lists.llvm.org/pipermail/cfe-dev/2021-April/068047.html
Differential D101037
[clang-tidy] Change shebang from python to python3 hokein on Apr 22 2021, 2:23 AM. Authored by
Details
want some opinions on this. see thread: https://lists.llvm.org/pipermail/cfe-dev/2021-April/068047.html
Diff Detail
Unit Tests Event TimelineComment Actions 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 Comment Actions 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. Comment Actions Doesn't seem like anything changed on the mailing list side, should we land this now? Comment Actions https://reviews.llvm.org/rG475440703238ca32adab6c3fe5e0039c3f96d1a5 committed the remaining shebang change from python to python3. |