[PATCH] D97625: fix check-clang-tools tests that fail due to Windows CRLF line endings
Richard via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 10 09:57:04 PST 2022
LegalizeAdulthood added a comment.
I have run into this line ending problem with python scripts as well.
If you follow the advice on the "getting started" pages, then you will
have checked out the entire repo with Unix line endings, but when you
run python scripts like add_new_check.py for clang-tidy, it writes out
the files with native line endings and ends up flagging the entirety of
the modified file as changing due to the flip-flopping of line endings.
I have an open bug <https://github.com/llvm/llvm-project/issues/52968> on this,
but I think we need to have a wider discussion in order to reach a
more global fix.
Your limited scope fix looks good to me.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97625/new/
https://reviews.llvm.org/D97625
More information about the cfe-commits
mailing list