Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented Jan 22, 2015

Here's one way to make the hashes match what Maven's plugins would create. It takes a little extra footwork since OS X doesn't have the same command line tools. An alternative is just to make Maven output these of course - would that be better? I ask in case there is a reason I'm missing, like, we need to hash files that Maven doesn't build.

@SparkQA
Copy link

SparkQA commented Jan 22, 2015

Test build #25968 has started for PR 4161 at commit e25eff8.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Jan 22, 2015

Test build #25968 has finished for PR 4161 at commit e25eff8.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25968/
Test PASSed.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor style comment: Could we use $(...) instead of backticks? I believe use of backticks is discouraged in bash.

@SparkQA
Copy link

SparkQA commented Jan 22, 2015

Test build #25973 has started for PR 4161 at commit 70d09d0.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Jan 22, 2015

Test build #25973 has finished for PR 4161 at commit 70d09d0.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25973/
Test PASSed.

@pwendell
Copy link
Contributor

Thanks Sean - pulling this in.

asfgit pushed a commit that referenced this pull request Jan 27, 2015
…n output

Here's one way to make the hashes match what Maven's plugins would create. It takes a little extra footwork since OS X doesn't have the same command line tools. An alternative is just to make Maven output these of course - would that be better? I ask in case there is a reason I'm missing, like, we need to hash files that Maven doesn't build.

Author: Sean Owen <sowen@cloudera.com>

Closes #4161 from srowen/SPARK-5308 and squashes the following commits:

70d09d0 [Sean Owen] Use $(...) syntax
e25eff8 [Sean Owen] Generate MD5, SHA1 hashes in a format like Maven's plugin

(cherry picked from commit ff356e2)
Signed-off-by: Patrick Wendell <patrick@databricks.com>
@asfgit asfgit closed this in ff356e2 Jan 27, 2015
@srowen srowen deleted the SPARK-5308 branch January 29, 2015 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants