Repository: spark
Updated Branches:
  refs/heads/branch-2.0 96c011d5b -> 8ee93eed9


[SPARK-12712] Fix failure in ./dev/test-dependencies when run against empty .m2 
cache

This patch fixes a bug in `./dev/test-dependencies.sh` which caused spurious 
failures when the script was run on a machine with an empty `.m2` cache. The 
problem was that extra log output from the dependency download was conflicting 
with the grep / regex used to identify the classpath in the Maven output. This 
patch fixes this issue by adjusting the regex pattern.

Tested manually with the following reproduction of the bug:

```
rm -rf ~/.m2/repository/org/apache/commons/
./dev/test-dependencies.sh
```

Author: Josh Rosen <joshro...@databricks.com>

Closes #13568 from JoshRosen/SPARK-12712.

(cherry picked from commit 921fa40b14082bfd1094fa49fb3b0c46a79c1aaa)
Signed-off-by: Josh Rosen <joshro...@databricks.com>


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

Branch: refs/heads/branch-2.0
Commit: 8ee93eed931b185b887882cc77c6fe8ddc907611
Parents: 96c011d
Author: Josh Rosen <joshro...@databricks.com>
Authored: Thu Jun 9 00:51:24 2016 -0700
Committer: Josh Rosen <joshro...@databricks.com>
Committed: Thu Jun 9 00:51:51 2016 -0700

----------------------------------------------------------------------
 dev/test-dependencies.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/8ee93eed/dev/test-dependencies.sh
----------------------------------------------------------------------
diff --git a/dev/test-dependencies.sh b/dev/test-dependencies.sh
index 5ea643e..28e3d4d 100755
--- a/dev/test-dependencies.sh
+++ b/dev/test-dependencies.sh
@@ -79,7 +79,7 @@ for HADOOP_PROFILE in "${HADOOP_PROFILES[@]}"; do
   echo "Generating dependency manifest for $HADOOP_PROFILE"
   mkdir -p dev/pr-deps
   $MVN $HADOOP2_MODULE_PROFILES -P$HADOOP_PROFILE dependency:build-classpath 
-pl assembly \
-    | grep "Building Spark Project Assembly" -A 5 \
+    | grep "Dependencies classpath:" -A 1 \
     | tail -n 1 | tr ":" "\n" | rev | cut -d "/" -f 1 | rev | sort \
     | grep -v spark > dev/pr-deps/spark-deps-$HADOOP_PROFILE
 done


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to