[PATCH] D37099: Added optional validation of svn sources to Dockerfiles.

Ilya Biryukov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 09:27:38 PDT 2017


ilya-biryukov marked an inline comment as done.
ilya-biryukov added a comment.

In https://reviews.llvm.org/D37099#873837, @jlebar wrote:

> Are the checksums intended to be valid across multiple platforms?  I recall that @zturner noticed that we use SVN in weird ways to munge newlines for some files on Windows.


Currently the checksums are only intended to match on Linux. I haven't checked, but some files will definitely have different line-endings on Windows.


Repository:
  rL LLVM

https://reviews.llvm.org/D37099





More information about the llvm-commits mailing list