[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