Merge pull request #35644 from thockin/build-fix-rsync

Automatic merge from submit-queue Build: Use checksum to rsync results If dependencies are touched, the Makefile will detect that tools like deepcopy-gen need to be rebuilt. It will rebuild them, but if Go decides the dependencies are not ACTUALLY updated (maybe it was a test file - known limitation of the current Makefile), it does not touch the resulting binary. The Makefile touches the result explicitly to mark that the dep change has been handled. But rsync blows away that change with the older file, and helpfully preserves the timestamp. This repeats on every build. Now rsync ignores it if the checksum is the same. Result = faster build. @wojtek-t @gmarek

Merge pull request #35644 from thockin/build-fix-rsync
d9873f45 · Kubernetes Submit Queue · GitHub · c31dc0c7 · 7aa91bbf · d9873f45
Commit d9873f45 authored Oct 26, 2016 by Kubernetes Submit Queue Committed by GitHub Oct 26, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

golang.sh hack/lib/golang.sh +1 -1

No files found.
--- a/hack/lib/golang.sh
+++ b/hack/lib/golang.sh
@@ -383,7 +383,7 @@ kube::golang::place_bins() {
    if [[ -d "${full_binpath_src}" ]]; then
      mkdir -p "${KUBE_OUTPUT_BINPATH}/${platform}"
      find "${full_binpath_src}" -maxdepth 1 -type f -exec \
-        rsync -pt {} "${KUBE_OUTPUT_BINPATH}/${platform}" \;
+        rsync -ptc {} "${KUBE_OUTPUT_BINPATH}/${platform}" \;
    fi
  done
 }