Thanks, Sean, good catch ...

I changed the destination for generated files:
http://cr.openjdk.java.net/~mkos/8038966/jdk.02/

Regarding usage ProcessBuilder instead of shell script - the problem is, that to compile test classes, wsdl must be compiled first - shell script is setup required before javac. The test would have to be run in two phases - first compiling+running java classes that compile wsdl (ProcessBuilder) and after that compiling other java classes using result of previous run.

I was solving this with jtreg some time ago already, but I haven't found other way than using shell script. If you know better solution, it would be great ...

Thanks
Miran



On 17/09/14 16:16, Seán Coffey wrote:
Miran,

the src change looks ok but I think there's a problem with the testcase.

You've defined generated classes for wsimport to be output to the TESTSRC
directory. This is often read only and won't work.

TESTCLASSES is the variable you're probably looking for. In any case, I think it's possible for you to avoid use of scripting. Can you invoke wsimport via
the ProcessBuilder API like that done for other wsimport tests ?

regards,
Sean.

On 17/09/14 14:00, Miroslav Kos wrote:
Hi everybody,

please review patch fixing following issue:

JBS: https://bugs.openjdk.java.net/browse/JDK-8038966
webrev:
http://cr.openjdk.java.net/~mkos/8038966/jaxws.00/
http://cr.openjdk.java.net/~mkos/8038966/jdk.01/

It is second part of fix ensuring that content of type xsd:any/content=mixed stays unchanged when processed with JAX-B RI. Regression test within webrev.

Sean, could you sponsor it for me?

Thanks
Miran





Reply via email to