saiddfhi / gwt-maven

Automatically exported from code.google.com/p/gwt-maven
0 stars 0 forks source link

mergewebxml shouldn't run in a non-war packaged project #174

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Is this Maven 1 or Maven 2?

Maven 2

What steps will reproduce the problem?
1. Have a pom-packaged project that includes the gwt plugin, including the
mergewebxml
2. run mvn compile
3. observe an error

What is the expected output? What do you see instead?

Would expect no errors. Instead I get:
[INFO] copy source web.xml -
C:\home\devel\ganymede\master\master-java\master-gwt\src\main\webapp\WEB-INF\web
.xml
to build dir (source web.xml required if mergewebxml execution is enabled)
[INFO] ------------------------------------------------------------------------
[ERROR] BUILD ERROR
[INFO] ------------------------------------------------------------------------
[INFO] Unable to merge web.xml

Embedded error:
C:\home\devel\ganymede\master\master-java\master-gwt\src\main\webapp\WEB-INF\web
.xml
(The system cannot find the path specified)

What version of the product are you using? On what operating system?
2.0-beta-25, windows

Please provide any additional information below.

Basically, I'm trying to create a master GWT pom that all GWT-using
projects would extend. The goal of this is to reduce as much configuration
as possible from the individual GWT-using projects. However, I can't
include the mergewebxml goal in the plugin configuration as a result of
this bug. It seems like a pretty easy fix. I have plugins that only operate
on EAR projects, so I have this in the plugin code:
        if ("ear".equals(packaging)) {
            innerExecute();
        } else {
            getLog().debug(
                    String.format("Mojo %s only runs on EAR-packaged
projects. Skipping", getClass().getName()));
        }

Original issue reported on code.google.com by justined...@gmail.com on 7 Nov 2008 at 6:55

GoogleCodeExporter commented 9 years ago
Just so I understand this, you want to include the mergewebxml goal in your
executions, but you don't want it to run if the packaging isn't war?  

What does your POM look like?  And, do you have a patch for this plugin, or is 
that
last part of code just a general note?  

Original comment by charlie....@gmail.com on 8 Nov 2008 at 11:47

GoogleCodeExporter commented 9 years ago
Unless this issue gets some more feedback, so I can understand it, it will be 
closed.
I am going to leave it open a few weeks here, but not indefinitely. 

Original comment by charlie....@gmail.com on 13 Nov 2008 at 4:55

GoogleCodeExporter commented 9 years ago
Got no further feedback, closing (see previous comments).

Original comment by charlie....@gmail.com on 19 Nov 2008 at 7:45

GoogleCodeExporter commented 9 years ago
sorry about my lack of response. I always forget that I can't reply to Google 
Code
via email for some reason.

In any case, this is pretty much moot for me now until #88 is fixed.

Original comment by justined...@gmail.com on 19 Nov 2008 at 10:13