This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] fix None tmpdir when exporting fixes in run-clang-tidy

Authored by 5chmidti on Aug 12 2023, 3:57 AM.



Differential removed the check for (yaml and args.export_fixes) in line 303 to skip looking for the clang-apply-replacements binary. However, the tmpdir variable was set in this true branch when exporting fixes and therefore is None when invoking run-clang-tidy with run-clang-tidy -p . -export-fixes fixes.yaml.

Diff Detail

Event Timeline

5chmidti created this revision.Aug 12 2023, 3:57 AM
5chmidti requested review of this revision.Aug 12 2023, 3:57 AM
This revision is now accepted and ready to land.Aug 12 2023, 4:01 AM

Thanks. I don't have commit access, could somebody please commit this for me with Julian Schmidt <>?