DRILL-7169: Rename drill-root ArtifactID to apache-drill#1746
DRILL-7169: Rename drill-root ArtifactID to apache-drill#1746vdiravka wants to merge 4 commits intoapache:masterfrom
Conversation
There was a problem hiding this comment.
@vdiravka, with this change, a folder with the unpacked build in distribution was renamed from apache-drill-1.16.0-SNAPSHOT to distribution-1.16.0-SNAPSHOT, so for now built version is in the distribution/target/distribution-1.16.0-SNAPSHOT/distribution-1.16.0-SNAPSHOT/ instead of distribution/target/apache-drill-1.16.0-SNAPSHOT/apache-drill-1.16.0-SNAPSHOT/.
I don't think that we should rename these folders.
Thanks for finding this, it is random mistake. |
distribution/pom.xml
Outdated
There was a problem hiding this comment.
all the <location>target/... can be replaced by <location>${project.build.directory}/....
pom.xml
Outdated
There was a problem hiding this comment.
is this path correct ? I don't see any src/main/protobuf path ?
Also there are other places like line 549 which can be modified to replace target/classes with ${project.build.outputDirectory}
There was a problem hiding this comment.
-
Looks like it was introduced for a long time ago for
maven-antrun-plugin.
b53933f#diff-0c2b5a9caaab6aa53c1c6c7dead7be77R118
But a little bit later this usage was removed. I think it is safe to remove it. -
Modified
vvysotskyi
left a comment
vvysotskyi
left a comment
There was a problem hiding this comment.
Changes look good, but as we discussed before, it would be good to hear community opinion about this change.
Please send a letter to the dev and user mailing lists about this change.
- change 'project.artifactId' from 'drill-root' to 'apache-drill'
…hes with ${project.build.directory},
${project.build.outputDirectory} and ${project.build.testOutputDirectory} properties accordingly
|
@vvysotskyi @sohami I have rebased the changes onto latest Drill master branch and added one commit with minor improvement. |
|
@vdiravka / @vvysotskyi what is the status of this PR? |
|
Currently, there is no consensus for this change, here is mail thread with the discussion: https://lists.apache.org/thread.html/99a2098ae04d56dd5994288b5fc05780d7de09a2f4a5ffbc5ce39cde@%3Cdev.drill.apache.org%3E |
4 participants
Change 'project.artifactId' from 'drill-root' to 'apache-drill'
Note: it includes changes for #1743 also (it is expected that PR will be merged first)