Skip to content

[ZEPPELIN-1180] Update publish_release.sh to publish scala 210 and scala 211 artifacts - #1209

Closed
minahlee wants to merge 5 commits into
apache:masterfrom
minahlee:ZEPPELIN-1180
Closed

[ZEPPELIN-1180] Update publish_release.sh to publish scala 210 and scala 211 artifacts#1209
minahlee wants to merge 5 commits into
apache:masterfrom
minahlee:ZEPPELIN-1180

Conversation

@minahlee

@minahleeminahlee commented Jul 20, 2016

Copy link
Copy Markdown
Member

What is this PR for?

  • Update publish_release.sh to publish scala 2.10 and scala 2.11 artifacts to maven repository.
  • Set build spark version to 2.0

This PR supposed to work once #1186 and #1195 is merged

What type of PR is it?

Improvement

What is the Jira issue?

ZEPPELIN-1180

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@minahlee
minahleeforce-pushed the ZEPPELIN-1180 branch 2 times, most recently from a4a669e to 5b160eeCompareJuly 22, 2016 02:55
Comment threaddev/publish_release.sh Outdated
echo "mvn clean install -Ppublish-distr \
-Dmaven.repo.local=${tmp_repo} -Dscala-2.11 \
${PUBLISH_PROFILES} ${PROJECT_OPTIONS}"
mvn clean install -Ppublish-distr -Dmaven.repo.local="${tmp_repo}" -Dscala-2.11\

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

to be consistent with other places such as travis ci config, I would use p_scalal-2.11

@minahlee

minahlee commented Jul 23, 2016

Copy link
Copy Markdown
MemberAuthor

@lresende thank you for review. I addressed your comments in 0a91932

@lresende

Copy link
Copy Markdown
Member

LGTM

@minahlee

Copy link
Copy Markdown
MemberAuthor

Merge if there is no more discussion

asfgit pushed a commit that referenced this pull request Jul 24, 2016
…ala 211 artifacts
### What is this PR for?
- Update publish_release.sh to publish scala 2.10 and scala 2.11 artifacts to maven repository.
- Set build spark version to 2.0
This PR supposed to work once #1186 and #1195 is merged
### What type of PR is it?
Improvement
### What is the Jira issue?
[ZEPPELIN-1180](https://issues.apache.org/jira/browse/ZEPPELIN-1180)
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
Author: Mina Lee <minalee@apache.org>
Closes#1209 from minahlee/ZEPPELIN-1180 and squashes the following commits:
40e5718 [Mina Lee] Use underscore as word separator in file name for consistency
0a91932 [Mina Lee] Include all module for scala 2.11 artifact build
c1d6439 [Mina Lee] publish 2.11 artifact to maven
a7918f0 [Mina Lee] Fix mod from 755 to 644
bca1af1 [Mina Lee] Fix indentation
(cherry picked from commit 1077921)
Signed-off-by: Mina Lee <minalee@apache.org>
PhilippGrulich pushed a commit to SWC-SENSE/zeppelin that referenced this pull request Aug 8, 2016
…ala 211 artifacts
### What is this PR for?
- Update publish_release.sh to publish scala 2.10 and scala 2.11 artifacts to maven repository.
- Set build spark version to 2.0
This PR supposed to work once apache#1186 and apache#1195 is merged
### What type of PR is it?
Improvement
### What is the Jira issue?
[ZEPPELIN-1180](https://issues.apache.org/jira/browse/ZEPPELIN-1180)
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
Author: Mina Lee <minalee@apache.org>
Closesapache#1209 from minahlee/ZEPPELIN-1180 and squashes the following commits:
40e5718 [Mina Lee] Use underscore as word separator in file name for consistency
0a91932 [Mina Lee] Include all module for scala 2.11 artifact build
c1d6439 [Mina Lee] publish 2.11 artifact to maven
a7918f0 [Mina Lee] Fix mod from 755 to 644
bca1af1 [Mina Lee] Fix indentation
@minahlee
minahlee deleted the ZEPPELIN-1180 branch November 2, 2016 04:59
Sign up for freeto 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.

2 participants

@minahlee@lresende