Skip to content

Commit 58d2a42

Browse files
Maven execution ordering will mean distribution must be dependent upon integration-tests... or the conformance tests much be a separate module which distribution depends upon. WORK NEEDED.
1 parent 468b85f commit 58d2a42

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

integration-tests/pom.xml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,11 @@
8181
8282
NOTE TOO: If we want to include the results of this test in
8383
the source distribution archives, this needs to run before
84-
"package". We may want to consider having
85-
pre-integration-test ensure that xalan-test is indeed
86-
available. See also
87-
https://maven.apache.org/guides/introduction/introduction-to-the-lifecycle.html
84+
"package". THAT IS NOT WHAT IS CURRENTLY HAPPENING.
85+
Maven only supports dependencies at the module level,
86+
with phase ordering only guaranteed within each module,
87+
so the fix appears to be to break this out into
88+
another module and make distribution depend upon it
8889
-->
8990
<plugin>
9091
<groupId>org.codehaus.mojo</groupId>
@@ -93,7 +94,7 @@
9394
<executions>
9495
<execution>
9596
<id>xalan-test attempt</id>
96-
<phase>integration-test</phase>
97+
<phase>test</phase>
9798
<goals>
9899
<goal>exec</goal>
99100
</goals>

0 commit comments

Comments
 (0)