From 715dfb9a430990835f531c316c93e0b99df626c8 Mon Sep 17 00:00:00 2001 From: lewuathe Date: Sat, 31 Jan 2015 00:08:55 +0900 Subject: [PATCH 1/2] [STORM-639] storm-maven-plugin not found --- storm-buildtools/storm-maven-plugins/pom.xml | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/storm-buildtools/storm-maven-plugins/pom.xml b/storm-buildtools/storm-maven-plugins/pom.xml index 5579d03baef..385c93e8cac 100644 --- a/storm-buildtools/storm-maven-plugins/pom.xml +++ b/storm-buildtools/storm-maven-plugins/pom.xml @@ -76,6 +76,16 @@ + + org.apache.maven.plugins + maven-install-plugin + + + default-install + package + + + From 50001f55ef1a1d87bdca2f39618dbe91be09db04 Mon Sep 17 00:00:00 2001 From: lewuathe Date: Tue, 3 Feb 2015 18:42:12 +0900 Subject: [PATCH 2/2] [STORM-639] Update docs for storm-maven-plugin --- DEVELOPER.md | 2 ++ storm-buildtools/storm-maven-plugins/pom.xml | 10 ---------- 2 files changed, 2 insertions(+), 10 deletions(-) diff --git a/DEVELOPER.md b/DEVELOPER.md index 9cd49e83081..6c4dbb907bb 100644 --- a/DEVELOPER.md +++ b/DEVELOPER.md @@ -228,6 +228,8 @@ To pull in a merge request you should generally follow the command line instruct The following commands must be run from the top-level directory. # Build the code and run the tests (requires nodejs, python and ruby installed) + # `mvn clean package` will fail because storm-core requires storm-maven-plugin. + # This plugin should be installed before compiling storm-core. $ mvn clean install # Build the code and run the tests, with specifying default test timeout (in millisecond) diff --git a/storm-buildtools/storm-maven-plugins/pom.xml b/storm-buildtools/storm-maven-plugins/pom.xml index 385c93e8cac..5579d03baef 100644 --- a/storm-buildtools/storm-maven-plugins/pom.xml +++ b/storm-buildtools/storm-maven-plugins/pom.xml @@ -76,16 +76,6 @@ - - org.apache.maven.plugins - maven-install-plugin - - - default-install - package - - -