You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@storm.apache.org by Lewuathe <gi...@git.apache.org> on 2015/01/30 16:10:44 UTC
[GitHub] storm pull request: [STORM-639] storm-maven-plugin not found
GitHub user Lewuathe opened a pull request:
https://github.com/apache/storm/pull/403
[STORM-639] storm-maven-plugin not found
maven compile is fail due to missing storm-maven-plugin.
This plugin should be installed before compiling storm-core project.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Lewuathe/storm maven-plugin-failure
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/403.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #403
----
commit 715dfb9a430990835f531c316c93e0b99df626c8
Author: lewuathe <le...@me.com>
Date: 2015-01-30T15:08:55Z
[STORM-639] storm-maven-plugin not found
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm pull request: [STORM-639] storm-maven-plugin not found
Posted by revans2 <gi...@git.apache.org>.
Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/403#issuecomment-89378042
I forgot to add +1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm pull request: [STORM-639] storm-maven-plugin not found
Posted by Lewuathe <gi...@git.apache.org>.
Github user Lewuathe commented on the pull request:
https://github.com/apache/storm/pull/403#issuecomment-75163410
I changed documentation. Can someone review it?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm pull request: [STORM-639] storm-maven-plugin not found
Posted by d2r <gi...@git.apache.org>.
Github user d2r commented on the pull request:
https://github.com/apache/storm/pull/403#issuecomment-72554412
> Update documents to install storm-maven-plugin before compiling storm-core project.
Yeah... I was afraid of that. It is a binary dependency, and maven does not support making this kind of thing work.
So I think your suggestion of changing the documents is better.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm pull request: [STORM-639] storm-maven-plugin not found
Posted by Lewuathe <gi...@git.apache.org>.
Github user Lewuathe commented on the pull request:
https://github.com/apache/storm/pull/403#issuecomment-72356826
It seems that there is no way to install `storm-maven-plugin` in local repository before compiling `storm-core` because with `compile` phase there is no jar files enough to install. So I think there are two options.
1. Update documents to install `storm-maven-plugin` before compiling `storm-core` project.
2. Deploy `storm-maven-plugin` on remote repository.
What do you think? If someone has any other ideas, please let me know.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm pull request: [STORM-639] storm-maven-plugin not found
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/403
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm pull request: [STORM-639] storm-maven-plugin not found
Posted by d2r <gi...@git.apache.org>.
Github user d2r commented on the pull request:
https://github.com/apache/storm/pull/403#issuecomment-72268036
I am seeing this happen too. Even with this change, if I try to compile with `mvn compile` from the top level, I see the following error:
```
Error resolving version for plugin 'org.apache.storm:storm-maven-plugins'
```
After `mvn install`, it does compile OK.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm pull request: [STORM-639] storm-maven-plugin not found
Posted by Lewuathe <gi...@git.apache.org>.
Github user Lewuathe commented on the pull request:
https://github.com/apache/storm/pull/403#issuecomment-72318223
@d2r Thank you for reporting. I'll fix to succeed compile with `mvn compile`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] storm pull request: [STORM-639] storm-maven-plugin not found
Posted by d2r <gi...@git.apache.org>.
Github user d2r commented on the pull request:
https://github.com/apache/storm/pull/403#issuecomment-75309369
> Update documents to install storm-maven-plugin before compiling storm-core project.
It looks as though the documents already say how to build it so that it works. It does not claim that doing the build directly from storm-core without the plugin jar on the classpath will succeed. So now I am not sure we want to add to the document one way not to build storm.
Really, it would be good if the plugin were pushed to maven repositories separately from storm, like @revans2 [suggested](https://issues.apache.org/jira/browse/STORM-639?focusedCommentId=14324305&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14324305).
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---