Skip to content

Final steps towards rel.#310

Merged
jixuan1989 merged 3 commits intorel/0.8from
rename-parent
Jul 26, 2019
Merged

Final steps towards rel.#310
jixuan1989 merged 3 commits intorel/0.8from
rename-parent

Conversation

@JulianFeinauer
Copy link
Contributor

Removed "build" module and moved this to parent pom.
Renamed parent pom without incubating.
Added incubating artefact names in assembly plugin.
Added SHA512 for all artefacts.

Big props to @chrisdutz for his support!

@JulianFeinauer JulianFeinauer removed the request for review from jixuan1989 July 26, 2019 12:07
See https://issues.apache.org/jira/browse/MNG-5454 sigh.
-->
<configuration combine.self="append">
<finalName>apache-iotdb-${project.version}-incubating</finalName>
Copy link
Member

Choose a reason for hiding this comment

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

Oh, now I get what Justin means... just add "incubating" here is fine..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah.. therefore I changed the name of the module back to "iotdb-parent" which is more standard : )

@jixuan1989
Copy link
Member

Hi, I tried this branch but after running mvn package, I do not get the .zip file...

@JulianFeinauer
Copy link
Contributor Author

@jixuan1989 you have to use mvn package -Papache-release to enable these steps.

Copy link
Member

@jixuan1989 jixuan1989 left a comment

Choose a reason for hiding this comment

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

+1. Learned a lot!
And because the generated binary file is changed, I will modify the dockerfile quickly in a new PR.

@jixuan1989 jixuan1989 merged commit 005eb31 into rel/0.8 Jul 26, 2019
@qiaojialin qiaojialin deleted the rename-parent branch November 28, 2019 07:28
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.

3 participants