You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by "Eugene Kuleshov (JIRA)" <ji...@codehaus.org> on 2005/10/12 07:02:11 UTC
[jira] Created: (MNG-1181) MavenEmbedder.execute() doesn't run reactor modules
MavenEmbedder.execute() doesn't run reactor modules
---------------------------------------------------
Key: MNG-1181
URL: http://jira.codehaus.org/browse/MNG-1181
Project: Maven 2
Type: Bug
Components: maven-embedder
Reporter: Eugene Kuleshov
Priority: Blocker
MavenEmbedder.execute() doesn't run reactor modules.
I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Commented: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "Jochen Wiedmann (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=comments#action_59191 ]
Jochen Wiedmann commented on MNG-1181:
--------------------------------------
Jason, I can follow your points, including the "formatted differently". This one wasn't intended, btw, I went as far as creating a special Maven formatter in my Eclipse.
However, please understand the following points:
- I am not proposing a patch. I am posting a "proof of concept". I am mainly asking for guidance.
- Without help, I am most possibly unable to fix the current MavenEmbedder, because it is too
complex for me. You're the Maven expert, I am the novice. At least, I won't be able to fix it
with a small patch.
- I do not even understand the intended lifecycle of the MavenEmbedder, the DefaultMaven,
and the objects they are using.
- I am happy, if you decide to fix the MavenEmbedder without following my extended proposals.
In that case I'll step aside and terminate my work on this request.
If you want me to carry on, here's my plan. Understand, that I make the steps as small as possible:
- I propose an object (MavenRequestConfiguration?), that may serve as a replacement of the CLI
within DefaultMaven. The object will receive additional properties, as required by the
MavenEmbedder. This first step doesn't include other changes than proposing this object.
- You split the objects properties into two categories: Those, which should be per
MavenEmbedder instance and those, which are per request. Additionally, you should provide
notes on using these properties, if they have lifcycle considerations, or something similar.
- The former properties are removed from the object.
- I propose a patch for DefaultMaven, the Maven interface, and the MavenCli, which makes
the DefaultMaven use the above object (get/setMavenRequestConfiguration(), and getters/
setters for the per embedder properties).
- You accept the patch.
- I propose a patch for the MavenEmbedder, which makes it use the DefaultMaven internally.
- You accept the patch.
- I propose a patch for the MavenCli, which makes it use the MavenEmbedder.
Changes to the above proposal are fine for me. My intention is to get this thing going. (Remember,
this is a five months old blocker!)
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: Embedding
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.4
> Attachments: MNG-1181.tar.gz, MavenEmbedder2.java, MavenEmbedder2.java
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Updated: (MNG-1181) MavenEmbedder.execute() doesn't run reactor modules
Posted by "John Casey (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=all ]
John Casey updated MNG-1181:
----------------------------
Description:
MavenEmbedder.execute() doesn't run reactor modules.
I've been trying to run it with "generate-sources" goal, but it only build the root project.
was:
MavenEmbedder.execute() doesn't run reactor modules.
I've been trying to run it with "generate-sources" goal, but it only build the root project.
Fix Version: 2.0.1
Environment:
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: maven-embedder
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.1
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Commented: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "Jochen Wiedmann (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=comments#action_59089 ]
Jochen Wiedmann commented on MNG-1181:
--------------------------------------
> the embedder2 class seems to be missing some functionality of the original,
> namely the reading of models and MavenProjects.
Yes, that's true, and I already noted that. I see three possible solutions:
- My personal favourite: Open the DefaultMaven class to provide this functionality.
Possibly open it as protected methods only and make MavenEmbedder2 a
subclass of DefaultMaven, which opens this.
- Copy code from DefaultMaven, or MavenEmbedder. Personally, I do not like
this, because copying code is bad.
- Change the MavenEmbedders execute methods to work with or like the
MavenEmbedder. I also do not like this, because it still leaves the problem
of doing things different than the "raw Maven".
> can you elaborate on if/how MNG-1665 will work with your embedder2?
The test case attached to MNG-1181 doesn't check that. However, considering
that the MavenEmbedder2 really does nothing different than the command line
Maven, I see no reason to believe the it won't work.
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: Embedding
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.4
> Attachments: MNG-1181.tar.gz, MavenEmbedder2.java
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Commented: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "Jason van Zyl (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=comments#action_59183 ]
Jason van Zyl commented on MNG-1181:
------------------------------------
I apologize for not looking at these issues sooner and I do appreciate the effort and I would like to integrate some of your work but the embedder is not simply a repackaging of DefaultMaven. The intention is to also provide access/manipulation of models, artifacts and repositories. I'm not intentionally trying to duplicate the code, I was attemping to entirely decouple the embedder from the CLI notions. Eventually the embedder will be used in the MavenCli, the plugin integration testing code and the Ant tasks. I'm not trying to dupe things here intentionally.
I will start looking at integrating bits and pieces of your code, but patches and some discussion will get you further then just rewriting the embedder. It's not likely I'm going to replace it with something I'm unfamiliar with, formatted differently and working differently then I intended. I'm not trying to discourage you and gladly welcome the help but some smaller patches and dialog will get you much further.
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: Embedding
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.4
> Attachments: MNG-1181.tar.gz, MavenEmbedder2.java, MavenEmbedder2.java
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Updated: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "Jochen Wiedmann (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=all ]
Jochen Wiedmann updated MNG-1181:
---------------------------------
Attachment: MNG-1181.tar.gz
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: Embedding
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.3
> Attachments: MNG-1181.tar.gz
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Updated: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "Jochen Wiedmann (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=all ]
Jochen Wiedmann updated MNG-1181:
---------------------------------
Attachment: MavenEmbedder2.java
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: Embedding
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.3
> Attachments: MNG-1181.tar.gz, MavenEmbedder2.java
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Commented: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "Jason van Zyl (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=comments#action_59900 ]
Jason van Zyl commented on MNG-1181:
------------------------------------
You will want to take a look here:
http://svn.apache.org/viewcvs.cgi/maven/components/branches/maven-embedder-refactor/
The MavenCli class now uses the embedder and I've pushed a lot of the code that was in the MavenCli class into DefaultMaven. Many of your points there have been addressed and next week I'll be actively working on this code so we probably need to exchange some email or chat on IRC if you want to do some work that I will incorporate.
I'm not sure if you are subscribed to the dev mailing list but I will be sending a message there talking about incorporating what is on the branch into the trunk.
Now to address your points above:
- Everything that goes into DefaultMaven should be couched in terms of the MavenExecutionRequest. Internally Maven should deal with a MavenExecutionRequest and the MavenSession. Anything that comes in via the CLI should be translated into a request and the Settings object that is used throughout the core should be taken out of the core.
- I've has started figuring out what is required per request to make sure that the embedder works in a multi threaded environment. So this you could certainly help me with. The embedder is pretty chopped up on the branch but you can see how the embedder is being using in the MavenCli class in the branch.
- I would be happy to work with you to figure out how to best configure the embedder.
- This I've done by allowing the configuration of the request itself. You can see exactly what's happening by looking at the MavenCli in the branch.
- I think what I've started is heading down that path.
- No patch is ever going to be accepted unless we are actually in discussion on a regular basis. That's how I can guide you, with some dialog.
- This has been done on the branch.
- Same as above regarding integrating large patches/changes.
- This has been done on the branch.
Everyday this week and on the weekend I'll be doing some work on the embedder to prepare for a presentation on the 7th of March.
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: Embedding
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.4
> Attachments: MNG-1181.tar.gz, MavenEmbedder2.java, MavenEmbedder2.java
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Updated: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "Jochen Wiedmann (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=all ]
Jochen Wiedmann updated MNG-1181:
---------------------------------
Attachment: MavenEmbedder2.java
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: Embedding
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.4
> Attachments: MNG-1181.tar.gz, MavenEmbedder2.java, MavenEmbedder2.java
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Updated: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "Brett Porter (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=all ]
Brett Porter updated MNG-1181:
------------------------------
Fix Version: (was: 2.0.1)
2.0.2
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: maven-embedder
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.2
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Commented: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "Jochen Wiedmann (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=comments#action_59113 ]
Jochen Wiedmann commented on MNG-1181:
--------------------------------------
Sorry, not MNGECLIPSE-75, but MNGECLIPSE-71.
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: Embedding
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.4
> Attachments: MNG-1181.tar.gz, MavenEmbedder2.java, MavenEmbedder2.java
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Commented: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "Milos Kleint (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=comments#action_59080 ]
Milos Kleint commented on MNG-1181:
-----------------------------------
the embedder2 class seems to be missing some functionality of the original, namely the reading of models and MavenProjects.
can you elaborate on if/how MNG-1665 will work with your embedder2? that's a crucial functionality for my netbeans ide integration.
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: Embedding
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.4
> Attachments: MNG-1181.tar.gz, MavenEmbedder2.java
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
[jira] Updated: (MNG-1181) MavenEmbedder.execute() doesn't run
reactor modules
Posted by "John Casey (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MNG-1181?page=all ]
John Casey updated MNG-1181:
----------------------------
Fix Version: (was: 2.0.3)
2.0.4
> MavenEmbedder.execute() doesn't run reactor modules
> ---------------------------------------------------
>
> Key: MNG-1181
> URL: http://jira.codehaus.org/browse/MNG-1181
> Project: Maven 2
> Type: Bug
> Components: Embedding
> Reporter: Eugene Kuleshov
> Priority: Blocker
> Fix For: 2.0.4
> Attachments: MNG-1181.tar.gz, MavenEmbedder2.java
>
>
> MavenEmbedder.execute() doesn't run reactor modules.
> I've been trying to run it with "generate-sources" goal, but it only build the root project.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org