Farrukh Najmi wrote:
> Hi Wilfred,
>
> Thanks very much for the patch.
>
> The high level comments (which I feel are very important) are:
>
> * We should not try and support ant in addition to maven, It creates
> a horses front end with a zebra's rear end ;-)
> * We should go with maven default patterns all the way and not try
> and keep any file structures or other patterns from the ant build
> process.
>
>
> I have made changes locally in my workspace exemplifying above
> suggestion on top of your patch and attaching the cumalitive patch for
> your review. The new patch is relative to the wadl directory (top
> level) and r89 in repo. The patched pom does not work for me as your
> patch was missing the parent pom.
>
> Please consider using this patch and then add missing files to it.
> Then apply patch to a cleanly checked out r89 workspace and make sure
> things build before sending next patch. Thanks.
>
It seems that the patch did not include files that were added due to a
moved :-(
There seems to be a bug in "svn diff" that it does not pick up files
that were added due to a move.
I am attaching the "svn status" output so you can get an idea of what I
moved with "svn move".
Marc, it would be much faster progress if we could just commit interim
changes and aim to reach stability ASAP. Is it possible to have svn be
unstable for say 1 week? If not then we should do what Wilfred suggested
earlier which is to create a branch and get it stable and then merge it
back to trunk?
Thoughts?
--
Regards,
Farrukh
Web: http://www.wellfleetsoftware.com
? patch-mavenized.txt
? source/wadl2java/catalog.xml
D source/wadl2java/test/com
D source/wadl2java/test/com/sun
D source/wadl2java/test/com/sun/research
D source/wadl2java/test/com/sun/research/ws
D source/wadl2java/test/com/sun/research/ws/wadl
D source/wadl2java/test/com/sun/research/ws/wadl/util
D source/wadl2java/test/com/sun/research/ws/wadl/util/UriBuilderTest.java
D source/wadl2java/test/com/sun/research/ws/wadl/util/URIUtilTest.java
D source/wadl2java/test/com/sun/research/ws/wadl2java
D source/wadl2java/test/com/sun/research/ws/wadl2java/GeneratorUtilTest.java
D source/wadl2java/test/com/sun/research/ws/wadl2java/MainTest.java
D source/wadl2java/test/com/sun/research/ws/wadl2java/ast
D source/wadl2java/test/com/sun/research/ws/wadl2java/ast/PathSegmentTest.java
D source/wadl2java/wadl.xsd
D source/wadl2java/nbproject
D source/wadl2java/nbproject/project.properties
D source/wadl2java/nbproject/project.xml
D source/wadl2java/nbproject/genfiles.properties
D source/wadl2java/nbproject/jax-ws.xml
D source/wadl2java/nbproject/build-impl.xml
D source/wadl2java/lib
D source/wadl2java/lib/localizer.jar
D source/wadl2java/lib/jaxb-xjc.jar
D source/wadl2java/lib/comresrcgen.jar
D source/wadl2java/lib/sjsxp.jar
D source/wadl2java/lib/jaxb-api.jar
D source/wadl2java/lib/activation.jar
D source/wadl2java/lib/resolver.jar
D source/wadl2java/lib/ant.jar
D source/wadl2java/lib/jsr173_api.jar
D source/wadl2java/lib/jaxb-impl.jar
A source/wadl2java/src/test
A source/wadl2java/src/test/java
A + source/wadl2java/src/test/java/com
A source/wadl2java/src/main
A source/wadl2java/src/main/java
A + source/wadl2java/src/main/java/com
A source/wadl2java/src/main/resources
A source/wadl2java/src/main/resources/xsd
A + source/wadl2java/src/main/resources/xsd/wadl.xsd
D source/wadl2java/src/com
D source/wadl2java/src/com/sun
D source/wadl2java/src/com/sun/research
D source/wadl2java/src/com/sun/research/ws
D source/wadl2java/src/com/sun/research/ws/wadl
D source/wadl2java/src/com/sun/research/ws/wadl/util
D source/wadl2java/src/com/sun/research/ws/wadl/util/UriSegment.java
D source/wadl2java/src/com/sun/research/ws/wadl/util/StreamDataSource.java
D source/wadl2java/src/com/sun/research/ws/wadl/util/UriBuilder.java
D source/wadl2java/src/com/sun/research/ws/wadl/util/JAXBDispatcher.java
D source/wadl2java/src/com/sun/research/ws/wadl/util/URIUtil.java
D source/wadl2java/src/com/sun/research/ws/wadl/util/DSDispatcher.java
D source/wadl2java/src/com/sun/research/ws/wadl/util/package.html
D source/wadl2java/src/com/sun/research/ws/wadl2java
D source/wadl2java/src/com/sun/research/ws/wadl2java/UnresolvableReferenceException.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/Main.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/ResourceClassGenerator.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/GeneratorUtil.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/ReferenceTargetException.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/JavaDocUtil.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/Wadl2Java.properties
D source/wadl2java/src/com/sun/research/ws/wadl2java/WJCTask.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/ast
D source/wadl2java/src/com/sun/research/ws/wadl2java/ast/PathSegment.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/ast/ResourceTypeNode.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/ast/FaultNode.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/ast/ast.properties
D source/wadl2java/src/com/sun/research/ws/wadl2java/ast/MethodNode.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/ast/RepresentationNode.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/ast/ResourceNode.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/ast/package.html
D source/wadl2java/src/com/sun/research/ws/wadl2java/Wadl2Java.java
D source/wadl2java/src/com/sun/research/ws/wadl2java/package.html
D source/wadl2java/manifest.mf
D source/wadl2java/xml.xsd
D source/wadl2java/build.xml
A source/wadl2java-maven-plugin
A source/wadl2java-maven-plugin/src
A source/wadl2java-maven-plugin/src/main
A source/wadl2java-maven-plugin/src/main/java
A source/wadl2java-maven-plugin/src/main/java/com
A source/wadl2java-maven-plugin/src/main/java/com/sun
A source/wadl2java-maven-plugin/src/main/java/com/sun/research
A source/wadl2java-maven-plugin/src/main/java/com/sun/research/ws
A source/wadl2java-maven-plugin/src/main/java/com/sun/research/ws/wadl2java
A source/wadl2java-maven-plugin/src/main/java/com/sun/research/ws/wadl2java/maven
A source/wadl2java-maven-plugin/src/main/java/com/sun/research/ws/wadl2java/maven/Wadl2JavaMojo.java
A source/wadl2java-maven-plugin/pom.xml