Skip to content

[build] [hotfix] Fix make-distribution.sh for Scala 2.11. #5002

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

vanzin
Copy link
Contributor

@vanzin vanzin commented Mar 12, 2015

No description provided.

@SparkQA
Copy link

SparkQA commented Mar 12, 2015

Test build #28518 has started for PR 5002 at commit ced65f7.

  • This patch merges cleanly.

@srowen
Copy link
Member

srowen commented Mar 12, 2015

Is the fix the $@? LGTM.

@vanzin
Copy link
Contributor Author

vanzin commented Mar 12, 2015

Yeah. I also formatted things to look more like the other, similar code. Thanks to @nightwolfzor for pointing it out.

@srowen
Copy link
Member

srowen commented Mar 12, 2015

CC @pwendell in case this has implications for the 1.3 release. Obviously looks OK to merge to master (jenkins won't test this anyway) but might even be needed at the last moment for 1.3.

@vanzin
Copy link
Contributor Author

vanzin commented Mar 12, 2015

No, this doesn't affect 1.3.

@srowen
Copy link
Member

srowen commented Mar 12, 2015

Ah right, this was just introduced with SPARK-4924. OK good to go

@SparkQA
Copy link

SparkQA commented Mar 12, 2015

Test build #28518 has finished for PR 5002 at commit ced65f7.

  • 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/28518/
Test PASSed.

@asfgit asfgit closed this in 8f1bc79 Mar 12, 2015
@vanzin vanzin deleted the mkdist-hotfix branch March 20, 2015 23:34
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.

4 participants