From: Dan Mick <dan.mick@redhat.com>
Date: Thu, 16 Mar 2017 05:06:20 +0000 (-0700)
Subject: ceph-pr-submodules: fix submodule check.
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F665%2Fhead;p=ceph-build.git

ceph-pr-submodules: fix submodule check.

Prior code never generated any output at all.

Fixes: http://tracker.ceph.com/issues/19244
Signed-off-by: Dan Mick <dan.mick@redhat.com>
---

diff --git a/ceph-pr-submodules/build/build b/ceph-pr-submodules/build/build
index 9428aa61..d7081ae0 100644
--- a/ceph-pr-submodules/build/build
+++ b/ceph-pr-submodules/build/build
@@ -4,14 +4,14 @@ set -ex
 
 cd "$WORKSPACE"
 
-# The command below has access to the variables $name, $path, $sha1 and $toplevel:
-# $name is the name of the relevant submodule section in .gitmodules, $path is
-# the name of the submodule directory relative to the superproject, $sha1 is
-# the commit as recorded in the superproject, and $toplevel is the absolute
-# path to the top-level of the superproject.
-modified_submodules=$(git submodule foreach 'cd $toplevel; git diff --ignore-submodules=dirty origin/master -- $path')
+# show diffs between $ghprbTargetBranch (where the merge is going) and
+# $ghprbActualCommit (the tip of the branch that's merging) with '...',
+# which is equivalent to diff $(git merge-base TB AC) AC, or "show
+# diff from common ancestor of the target branch and this branch with the
+# tip of this branch".  With --submodule, also show detail of diff in submodules.
+modified_submodules="$(git diff --submodule=log origin/$ghprbTargetBranch...$ghprbActualCommit | grep ^Submodule || true)"
 
-if [[ $modified_submodules ]]; then
+if [[ -n "$modified_submodules" ]]; then
     echo "Project has modified submodules!"
     echo $modified_submodules
     exit 1