You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by bu...@apache.org on 2017/11/09 15:05:15 UTC

[1/5] hbase git commit: HBASE-19184 Update nightly source artifact check to use git archive for making the tarball.

Repository: hbase
Updated Branches:
  refs/heads/branch-1 5f8b9de3f -> d1f8fc849
  refs/heads/branch-1.1 51f06c742 -> f25100d3c
  refs/heads/branch-1.2 eb8560cab -> 607e9c643
  refs/heads/branch-1.3 557194830 -> 9ae75667c
  refs/heads/branch-1.4 93c91666d -> 51680b97b


HBASE-19184 Update nightly source artifact check to use git archive for making the tarball.

Signed-off-by: Apekshit Sharma <ap...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/d1f8fc84
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/d1f8fc84
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/d1f8fc84

Branch: refs/heads/branch-1
Commit: d1f8fc849a7dfed0164d8b8c180f59e72ea2411b
Parents: 5f8b9de
Author: Sean Busbey <bu...@apache.org>
Authored: Mon Nov 6 14:14:18 2017 -0600
Committer: Sean Busbey <bu...@apache.org>
Committed: Thu Nov 9 09:03:32 2017 -0600

----------------------------------------------------------------------
 dev-support/hbase_nightly_source-artifact.sh | 41 +++++++++++++----------
 1 file changed, 23 insertions(+), 18 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/d1f8fc84/dev-support/hbase_nightly_source-artifact.sh
----------------------------------------------------------------------
diff --git a/dev-support/hbase_nightly_source-artifact.sh b/dev-support/hbase_nightly_source-artifact.sh
index 4e4c6a2..c66c41b 100755
--- a/dev-support/hbase_nightly_source-artifact.sh
+++ b/dev-support/hbase_nightly_source-artifact.sh
@@ -125,38 +125,31 @@ mvn --version --offline | tee "${working_dir}/maven_version"
 echo "Do a clean building of the source artifact using code in ${component_dir}"
 cd "${component_dir}"
 echo "Clean..."
-mvn --batch-mode -DskipTests clean >"${working_dir}/component_clean_before.log" 2>&1
-echo "Step 3 Build the source tarball"
-mvn -Prelease --batch-mode -Dmaven.repo.local="${m2_initial}" install -DskipTests assembly:single \
-    -Dassembly.file=hbase-assembly/src/main/assembly/src.xml \
+git clean -xdfff >"${working_dir}/component_git_clean.log" 2>&1
+echo "Follow the ref guide section on making a RC: Step 6 Build the source tarball"
+git archive --format=tar.gz --output="${working_dir}/hbase-src.tar.gz" \
+    --prefix="hbase-SOMEVERSION/" HEAD \
     >"${working_dir}/component_build_src_tarball.log" 2>&1
 
 cd "${unpack_dir}"
 echo "Unpack the source tarball"
-tar --strip-components=1 -xzf "${component_dir}"/hbase-assembly/target/hbase-*-src.tar.gz \
+tar --strip-components=1 -xzf "${working_dir}/hbase-src.tar.gz" \
     >"${working_dir}/srctarball_unpack.log" 2>&1
-echo "Building from source artifact."
-mvn -DskipTests -Prelease --batch-mode -Dmaven.repo.local="${m2_tarbuild}" clean install \
-    assembly:single >"${working_dir}/srctarball_install.log" 2>&1
-
-echo "Clean up after checking ability to build."
-mvn -DskipTests --batch-mode clean >"${working_dir}/srctarball_clean.log" 2>&1
-
 
 cd "${component_dir}"
-echo "Clean up the source checkout"
-mvn --batch-mode -DskipTests clean >"${working_dir}/component_clean_after.log" 2>&1
 echo "Diff against source tree"
 diff --binary --recursive . "${unpack_dir}" >"${working_dir}/diff_output" || true
 
 cd "${working_dir}"
 # expectation check largely based on HBASE-14952
-echo "Checking against things we don't expect to include in the source tarball (git related, hbase-native-client, etc.)"
+echo "Checking against things we don't expect to include in the source tarball (git related, etc.)"
+# Add in lines to show differences between the source tarball and this branch, in the same format diff would give.
+# e.g. prior to HBASE-19152 we'd have the following lines (ignoring the bash comment marker):
+#Only in .: .gitattributes
+#Only in .: .gitignore
+#Only in .: hbase-native-client
 cat >known_excluded <<END
 Only in .: .git
-Only in .: .gitattributes
-Only in .: .gitignore
-Only in .: hbase-native-client
 END
 if ! diff known_excluded diff_output >"${working_dir}/unexpected.diff" ; then
   echo "Any output here are unexpected differences between the source artifact we'd make for an RC and the current branch."
@@ -166,3 +159,15 @@ if ! diff known_excluded diff_output >"${working_dir}/unexpected.diff" ; then
 else
   echo "Everything looks as expected."
 fi
+
+cd "${unpack_dir}"
+echo "Follow the ref guide section on making a RC: Step 8 Build the binary tarball."
+if mvn -DskipTests -Prelease --batch-mode -Dmaven.repo.local="${m2_tarbuild}" clean install \
+    assembly:single >"${working_dir}/srctarball_install.log" 2>&1; then
+  echo "Building a binary tarball from the source tarball succeeded."
+else
+  echo "Building a binary tarball from the source tarball failed. see srtarball_install.log for details."
+  exit 1
+fi
+
+# TODO check the layout of the binary artifact we just made.


[5/5] hbase git commit: HBASE-19184 Update nightly source artifact check to use git archive for making the tarball.

Posted by bu...@apache.org.
HBASE-19184 Update nightly source artifact check to use git archive for making the tarball.

Signed-off-by: Apekshit Sharma <ap...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/f25100d3
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/f25100d3
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/f25100d3

Branch: refs/heads/branch-1.1
Commit: f25100d3c999c8f51515f1f890340a7f3af7b973
Parents: 51f06c7
Author: Sean Busbey <bu...@apache.org>
Authored: Mon Nov 6 14:14:18 2017 -0600
Committer: Sean Busbey <bu...@apache.org>
Committed: Thu Nov 9 09:04:50 2017 -0600

----------------------------------------------------------------------
 dev-support/hbase_nightly_source-artifact.sh | 41 +++++++++++++----------
 1 file changed, 23 insertions(+), 18 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/f25100d3/dev-support/hbase_nightly_source-artifact.sh
----------------------------------------------------------------------
diff --git a/dev-support/hbase_nightly_source-artifact.sh b/dev-support/hbase_nightly_source-artifact.sh
index 4e4c6a2..c66c41b 100755
--- a/dev-support/hbase_nightly_source-artifact.sh
+++ b/dev-support/hbase_nightly_source-artifact.sh
@@ -125,38 +125,31 @@ mvn --version --offline | tee "${working_dir}/maven_version"
 echo "Do a clean building of the source artifact using code in ${component_dir}"
 cd "${component_dir}"
 echo "Clean..."
-mvn --batch-mode -DskipTests clean >"${working_dir}/component_clean_before.log" 2>&1
-echo "Step 3 Build the source tarball"
-mvn -Prelease --batch-mode -Dmaven.repo.local="${m2_initial}" install -DskipTests assembly:single \
-    -Dassembly.file=hbase-assembly/src/main/assembly/src.xml \
+git clean -xdfff >"${working_dir}/component_git_clean.log" 2>&1
+echo "Follow the ref guide section on making a RC: Step 6 Build the source tarball"
+git archive --format=tar.gz --output="${working_dir}/hbase-src.tar.gz" \
+    --prefix="hbase-SOMEVERSION/" HEAD \
     >"${working_dir}/component_build_src_tarball.log" 2>&1
 
 cd "${unpack_dir}"
 echo "Unpack the source tarball"
-tar --strip-components=1 -xzf "${component_dir}"/hbase-assembly/target/hbase-*-src.tar.gz \
+tar --strip-components=1 -xzf "${working_dir}/hbase-src.tar.gz" \
     >"${working_dir}/srctarball_unpack.log" 2>&1
-echo "Building from source artifact."
-mvn -DskipTests -Prelease --batch-mode -Dmaven.repo.local="${m2_tarbuild}" clean install \
-    assembly:single >"${working_dir}/srctarball_install.log" 2>&1
-
-echo "Clean up after checking ability to build."
-mvn -DskipTests --batch-mode clean >"${working_dir}/srctarball_clean.log" 2>&1
-
 
 cd "${component_dir}"
-echo "Clean up the source checkout"
-mvn --batch-mode -DskipTests clean >"${working_dir}/component_clean_after.log" 2>&1
 echo "Diff against source tree"
 diff --binary --recursive . "${unpack_dir}" >"${working_dir}/diff_output" || true
 
 cd "${working_dir}"
 # expectation check largely based on HBASE-14952
-echo "Checking against things we don't expect to include in the source tarball (git related, hbase-native-client, etc.)"
+echo "Checking against things we don't expect to include in the source tarball (git related, etc.)"
+# Add in lines to show differences between the source tarball and this branch, in the same format diff would give.
+# e.g. prior to HBASE-19152 we'd have the following lines (ignoring the bash comment marker):
+#Only in .: .gitattributes
+#Only in .: .gitignore
+#Only in .: hbase-native-client
 cat >known_excluded <<END
 Only in .: .git
-Only in .: .gitattributes
-Only in .: .gitignore
-Only in .: hbase-native-client
 END
 if ! diff known_excluded diff_output >"${working_dir}/unexpected.diff" ; then
   echo "Any output here are unexpected differences between the source artifact we'd make for an RC and the current branch."
@@ -166,3 +159,15 @@ if ! diff known_excluded diff_output >"${working_dir}/unexpected.diff" ; then
 else
   echo "Everything looks as expected."
 fi
+
+cd "${unpack_dir}"
+echo "Follow the ref guide section on making a RC: Step 8 Build the binary tarball."
+if mvn -DskipTests -Prelease --batch-mode -Dmaven.repo.local="${m2_tarbuild}" clean install \
+    assembly:single >"${working_dir}/srctarball_install.log" 2>&1; then
+  echo "Building a binary tarball from the source tarball succeeded."
+else
+  echo "Building a binary tarball from the source tarball failed. see srtarball_install.log for details."
+  exit 1
+fi
+
+# TODO check the layout of the binary artifact we just made.


[3/5] hbase git commit: HBASE-19184 Update nightly source artifact check to use git archive for making the tarball.

Posted by bu...@apache.org.
HBASE-19184 Update nightly source artifact check to use git archive for making the tarball.

Signed-off-by: Apekshit Sharma <ap...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/9ae75667
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/9ae75667
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/9ae75667

Branch: refs/heads/branch-1.3
Commit: 9ae75667c43eae6bbc5d3b52a6117b0fd5971cce
Parents: 5571948
Author: Sean Busbey <bu...@apache.org>
Authored: Mon Nov 6 14:14:18 2017 -0600
Committer: Sean Busbey <bu...@apache.org>
Committed: Thu Nov 9 09:04:42 2017 -0600

----------------------------------------------------------------------
 dev-support/hbase_nightly_source-artifact.sh | 41 +++++++++++++----------
 1 file changed, 23 insertions(+), 18 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/9ae75667/dev-support/hbase_nightly_source-artifact.sh
----------------------------------------------------------------------
diff --git a/dev-support/hbase_nightly_source-artifact.sh b/dev-support/hbase_nightly_source-artifact.sh
index 4e4c6a2..c66c41b 100755
--- a/dev-support/hbase_nightly_source-artifact.sh
+++ b/dev-support/hbase_nightly_source-artifact.sh
@@ -125,38 +125,31 @@ mvn --version --offline | tee "${working_dir}/maven_version"
 echo "Do a clean building of the source artifact using code in ${component_dir}"
 cd "${component_dir}"
 echo "Clean..."
-mvn --batch-mode -DskipTests clean >"${working_dir}/component_clean_before.log" 2>&1
-echo "Step 3 Build the source tarball"
-mvn -Prelease --batch-mode -Dmaven.repo.local="${m2_initial}" install -DskipTests assembly:single \
-    -Dassembly.file=hbase-assembly/src/main/assembly/src.xml \
+git clean -xdfff >"${working_dir}/component_git_clean.log" 2>&1
+echo "Follow the ref guide section on making a RC: Step 6 Build the source tarball"
+git archive --format=tar.gz --output="${working_dir}/hbase-src.tar.gz" \
+    --prefix="hbase-SOMEVERSION/" HEAD \
     >"${working_dir}/component_build_src_tarball.log" 2>&1
 
 cd "${unpack_dir}"
 echo "Unpack the source tarball"
-tar --strip-components=1 -xzf "${component_dir}"/hbase-assembly/target/hbase-*-src.tar.gz \
+tar --strip-components=1 -xzf "${working_dir}/hbase-src.tar.gz" \
     >"${working_dir}/srctarball_unpack.log" 2>&1
-echo "Building from source artifact."
-mvn -DskipTests -Prelease --batch-mode -Dmaven.repo.local="${m2_tarbuild}" clean install \
-    assembly:single >"${working_dir}/srctarball_install.log" 2>&1
-
-echo "Clean up after checking ability to build."
-mvn -DskipTests --batch-mode clean >"${working_dir}/srctarball_clean.log" 2>&1
-
 
 cd "${component_dir}"
-echo "Clean up the source checkout"
-mvn --batch-mode -DskipTests clean >"${working_dir}/component_clean_after.log" 2>&1
 echo "Diff against source tree"
 diff --binary --recursive . "${unpack_dir}" >"${working_dir}/diff_output" || true
 
 cd "${working_dir}"
 # expectation check largely based on HBASE-14952
-echo "Checking against things we don't expect to include in the source tarball (git related, hbase-native-client, etc.)"
+echo "Checking against things we don't expect to include in the source tarball (git related, etc.)"
+# Add in lines to show differences between the source tarball and this branch, in the same format diff would give.
+# e.g. prior to HBASE-19152 we'd have the following lines (ignoring the bash comment marker):
+#Only in .: .gitattributes
+#Only in .: .gitignore
+#Only in .: hbase-native-client
 cat >known_excluded <<END
 Only in .: .git
-Only in .: .gitattributes
-Only in .: .gitignore
-Only in .: hbase-native-client
 END
 if ! diff known_excluded diff_output >"${working_dir}/unexpected.diff" ; then
   echo "Any output here are unexpected differences between the source artifact we'd make for an RC and the current branch."
@@ -166,3 +159,15 @@ if ! diff known_excluded diff_output >"${working_dir}/unexpected.diff" ; then
 else
   echo "Everything looks as expected."
 fi
+
+cd "${unpack_dir}"
+echo "Follow the ref guide section on making a RC: Step 8 Build the binary tarball."
+if mvn -DskipTests -Prelease --batch-mode -Dmaven.repo.local="${m2_tarbuild}" clean install \
+    assembly:single >"${working_dir}/srctarball_install.log" 2>&1; then
+  echo "Building a binary tarball from the source tarball succeeded."
+else
+  echo "Building a binary tarball from the source tarball failed. see srtarball_install.log for details."
+  exit 1
+fi
+
+# TODO check the layout of the binary artifact we just made.


[4/5] hbase git commit: HBASE-19184 Update nightly source artifact check to use git archive for making the tarball.

Posted by bu...@apache.org.
HBASE-19184 Update nightly source artifact check to use git archive for making the tarball.

Signed-off-by: Apekshit Sharma <ap...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/607e9c64
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/607e9c64
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/607e9c64

Branch: refs/heads/branch-1.2
Commit: 607e9c64376f8092fe2f5111bf183c8c963328ca
Parents: eb8560c
Author: Sean Busbey <bu...@apache.org>
Authored: Mon Nov 6 14:14:18 2017 -0600
Committer: Sean Busbey <bu...@apache.org>
Committed: Thu Nov 9 09:04:46 2017 -0600

----------------------------------------------------------------------
 dev-support/hbase_nightly_source-artifact.sh | 41 +++++++++++++----------
 1 file changed, 23 insertions(+), 18 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/607e9c64/dev-support/hbase_nightly_source-artifact.sh
----------------------------------------------------------------------
diff --git a/dev-support/hbase_nightly_source-artifact.sh b/dev-support/hbase_nightly_source-artifact.sh
index 4e4c6a2..c66c41b 100755
--- a/dev-support/hbase_nightly_source-artifact.sh
+++ b/dev-support/hbase_nightly_source-artifact.sh
@@ -125,38 +125,31 @@ mvn --version --offline | tee "${working_dir}/maven_version"
 echo "Do a clean building of the source artifact using code in ${component_dir}"
 cd "${component_dir}"
 echo "Clean..."
-mvn --batch-mode -DskipTests clean >"${working_dir}/component_clean_before.log" 2>&1
-echo "Step 3 Build the source tarball"
-mvn -Prelease --batch-mode -Dmaven.repo.local="${m2_initial}" install -DskipTests assembly:single \
-    -Dassembly.file=hbase-assembly/src/main/assembly/src.xml \
+git clean -xdfff >"${working_dir}/component_git_clean.log" 2>&1
+echo "Follow the ref guide section on making a RC: Step 6 Build the source tarball"
+git archive --format=tar.gz --output="${working_dir}/hbase-src.tar.gz" \
+    --prefix="hbase-SOMEVERSION/" HEAD \
     >"${working_dir}/component_build_src_tarball.log" 2>&1
 
 cd "${unpack_dir}"
 echo "Unpack the source tarball"
-tar --strip-components=1 -xzf "${component_dir}"/hbase-assembly/target/hbase-*-src.tar.gz \
+tar --strip-components=1 -xzf "${working_dir}/hbase-src.tar.gz" \
     >"${working_dir}/srctarball_unpack.log" 2>&1
-echo "Building from source artifact."
-mvn -DskipTests -Prelease --batch-mode -Dmaven.repo.local="${m2_tarbuild}" clean install \
-    assembly:single >"${working_dir}/srctarball_install.log" 2>&1
-
-echo "Clean up after checking ability to build."
-mvn -DskipTests --batch-mode clean >"${working_dir}/srctarball_clean.log" 2>&1
-
 
 cd "${component_dir}"
-echo "Clean up the source checkout"
-mvn --batch-mode -DskipTests clean >"${working_dir}/component_clean_after.log" 2>&1
 echo "Diff against source tree"
 diff --binary --recursive . "${unpack_dir}" >"${working_dir}/diff_output" || true
 
 cd "${working_dir}"
 # expectation check largely based on HBASE-14952
-echo "Checking against things we don't expect to include in the source tarball (git related, hbase-native-client, etc.)"
+echo "Checking against things we don't expect to include in the source tarball (git related, etc.)"
+# Add in lines to show differences between the source tarball and this branch, in the same format diff would give.
+# e.g. prior to HBASE-19152 we'd have the following lines (ignoring the bash comment marker):
+#Only in .: .gitattributes
+#Only in .: .gitignore
+#Only in .: hbase-native-client
 cat >known_excluded <<END
 Only in .: .git
-Only in .: .gitattributes
-Only in .: .gitignore
-Only in .: hbase-native-client
 END
 if ! diff known_excluded diff_output >"${working_dir}/unexpected.diff" ; then
   echo "Any output here are unexpected differences between the source artifact we'd make for an RC and the current branch."
@@ -166,3 +159,15 @@ if ! diff known_excluded diff_output >"${working_dir}/unexpected.diff" ; then
 else
   echo "Everything looks as expected."
 fi
+
+cd "${unpack_dir}"
+echo "Follow the ref guide section on making a RC: Step 8 Build the binary tarball."
+if mvn -DskipTests -Prelease --batch-mode -Dmaven.repo.local="${m2_tarbuild}" clean install \
+    assembly:single >"${working_dir}/srctarball_install.log" 2>&1; then
+  echo "Building a binary tarball from the source tarball succeeded."
+else
+  echo "Building a binary tarball from the source tarball failed. see srtarball_install.log for details."
+  exit 1
+fi
+
+# TODO check the layout of the binary artifact we just made.


[2/5] hbase git commit: HBASE-19184 Update nightly source artifact check to use git archive for making the tarball.

Posted by bu...@apache.org.
HBASE-19184 Update nightly source artifact check to use git archive for making the tarball.

Signed-off-by: Apekshit Sharma <ap...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/51680b97
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/51680b97
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/51680b97

Branch: refs/heads/branch-1.4
Commit: 51680b97b8a50b30576ae3fb07a5e500f6c39026
Parents: 93c9166
Author: Sean Busbey <bu...@apache.org>
Authored: Mon Nov 6 14:14:18 2017 -0600
Committer: Sean Busbey <bu...@apache.org>
Committed: Thu Nov 9 09:04:38 2017 -0600

----------------------------------------------------------------------
 dev-support/hbase_nightly_source-artifact.sh | 41 +++++++++++++----------
 1 file changed, 23 insertions(+), 18 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/51680b97/dev-support/hbase_nightly_source-artifact.sh
----------------------------------------------------------------------
diff --git a/dev-support/hbase_nightly_source-artifact.sh b/dev-support/hbase_nightly_source-artifact.sh
index 4e4c6a2..c66c41b 100755
--- a/dev-support/hbase_nightly_source-artifact.sh
+++ b/dev-support/hbase_nightly_source-artifact.sh
@@ -125,38 +125,31 @@ mvn --version --offline | tee "${working_dir}/maven_version"
 echo "Do a clean building of the source artifact using code in ${component_dir}"
 cd "${component_dir}"
 echo "Clean..."
-mvn --batch-mode -DskipTests clean >"${working_dir}/component_clean_before.log" 2>&1
-echo "Step 3 Build the source tarball"
-mvn -Prelease --batch-mode -Dmaven.repo.local="${m2_initial}" install -DskipTests assembly:single \
-    -Dassembly.file=hbase-assembly/src/main/assembly/src.xml \
+git clean -xdfff >"${working_dir}/component_git_clean.log" 2>&1
+echo "Follow the ref guide section on making a RC: Step 6 Build the source tarball"
+git archive --format=tar.gz --output="${working_dir}/hbase-src.tar.gz" \
+    --prefix="hbase-SOMEVERSION/" HEAD \
     >"${working_dir}/component_build_src_tarball.log" 2>&1
 
 cd "${unpack_dir}"
 echo "Unpack the source tarball"
-tar --strip-components=1 -xzf "${component_dir}"/hbase-assembly/target/hbase-*-src.tar.gz \
+tar --strip-components=1 -xzf "${working_dir}/hbase-src.tar.gz" \
     >"${working_dir}/srctarball_unpack.log" 2>&1
-echo "Building from source artifact."
-mvn -DskipTests -Prelease --batch-mode -Dmaven.repo.local="${m2_tarbuild}" clean install \
-    assembly:single >"${working_dir}/srctarball_install.log" 2>&1
-
-echo "Clean up after checking ability to build."
-mvn -DskipTests --batch-mode clean >"${working_dir}/srctarball_clean.log" 2>&1
-
 
 cd "${component_dir}"
-echo "Clean up the source checkout"
-mvn --batch-mode -DskipTests clean >"${working_dir}/component_clean_after.log" 2>&1
 echo "Diff against source tree"
 diff --binary --recursive . "${unpack_dir}" >"${working_dir}/diff_output" || true
 
 cd "${working_dir}"
 # expectation check largely based on HBASE-14952
-echo "Checking against things we don't expect to include in the source tarball (git related, hbase-native-client, etc.)"
+echo "Checking against things we don't expect to include in the source tarball (git related, etc.)"
+# Add in lines to show differences between the source tarball and this branch, in the same format diff would give.
+# e.g. prior to HBASE-19152 we'd have the following lines (ignoring the bash comment marker):
+#Only in .: .gitattributes
+#Only in .: .gitignore
+#Only in .: hbase-native-client
 cat >known_excluded <<END
 Only in .: .git
-Only in .: .gitattributes
-Only in .: .gitignore
-Only in .: hbase-native-client
 END
 if ! diff known_excluded diff_output >"${working_dir}/unexpected.diff" ; then
   echo "Any output here are unexpected differences between the source artifact we'd make for an RC and the current branch."
@@ -166,3 +159,15 @@ if ! diff known_excluded diff_output >"${working_dir}/unexpected.diff" ; then
 else
   echo "Everything looks as expected."
 fi
+
+cd "${unpack_dir}"
+echo "Follow the ref guide section on making a RC: Step 8 Build the binary tarball."
+if mvn -DskipTests -Prelease --batch-mode -Dmaven.repo.local="${m2_tarbuild}" clean install \
+    assembly:single >"${working_dir}/srctarball_install.log" 2>&1; then
+  echo "Building a binary tarball from the source tarball succeeded."
+else
+  echo "Building a binary tarball from the source tarball failed. see srtarball_install.log for details."
+  exit 1
+fi
+
+# TODO check the layout of the binary artifact we just made.