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

Ilya Biryukov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 31 01:59:41 PDT 2017


ilya-biryukov updated this revision to Diff 113372.
ilya-biryukov marked 3 inline comments as done.
ilya-biryukov added a comment.

Addressed review comments.

- Renamed proc_file to process_file.
- Don't feed number of files to the hasher, it's redundant.
- Removed next_dirs local var.


https://reviews.llvm.org/D37099

Files:
  utils/docker/build_docker_image.sh
  utils/docker/debian8/build/Dockerfile
  utils/docker/example/build/Dockerfile
  utils/docker/nvidia-cuda/build/Dockerfile
  utils/docker/scripts/build_install_llvm.sh
  utils/docker/scripts/llvm_checksum/llvm_checksum.py
  utils/docker/scripts/llvm_checksum/llvm_checksum_utils.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37099.113372.patch
Type: text/x-patch
Size: 17924 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170831/63f999e4/attachment-0001.bin>


More information about the llvm-commits mailing list