GitHub user JoshRosen opened a pull request:
https://github.com/apache/spark/pull/10543
[SPARK-10359][PROJECT-INFRA] Multiple fixes to dev/test-dependencies.sh
script
This patch includes multiple fixes for the `dev/test-dependencies.sh`
script (which was introduced in #10461):
- Use `build/mvn --force` instead of `mvn` in one additional place.
- Explicitly set a zero exit code on success.
- Set `LC_ALL=C` before calling `sort` in order to make the results agree
across machines (see https://stackoverflow.com/questions/28881/)
- To improve readability, split a complicated shell pipeline across
multiple lines.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/JoshRosen/spark dep-script-fixes
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/10543.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #10543
----
commit 38da65fc111a83e28aa9070587f59c2b481a45d5
Author: Josh Rosen <[email protected]>
Date: 2016-01-01T03:01:26Z
Use our own MVN script
commit bf92b0356661834549f64e0a2f0259b4d33e35f2
Author: Josh Rosen <[email protected]>
Date: 2016-01-01T03:01:51Z
Set explicit zero exit code on success
commit ee489ef3a4df74e9a9a3304fd1bb799c1627c834
Author: Josh Rosen <[email protected]>
Date: 2016-01-01T03:04:37Z
Split complex pipeline across multiple lines
commit 40aa7a181a282ad2a33fb4683b1fea9da50cd7d8
Author: Josh Rosen <[email protected]>
Date: 2016-01-01T03:07:42Z
Use LC_ALL=C for portability
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
