RFR: 8038966 JAX-WS handles wrongly xsd:any arguments for Web services
sean.coffey at oracle.com
Thu Sep 18 09:49:59 UTC 2014
On 18/09/2014 10:12, Miroslav Kos wrote:
> Thanks, Sean, good catch ...
> I changed the destination for generated files:
> 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 ...
Ah - I see the dependency now. Thanks for the information.
Just spotted one other issue. The deleteGeneratedFiles() method scans
for files in the
test src path (for deletion) - No files should be created there now. You
can change that
to scan the test classes path now. You could just use the current
working directory I guess
since that's where jtreg will generate the new files.
once that's fixed up, I can run this through JPRT and it should be good
to push then.
> On 17/09/14 16:16, Seán Coffey wrote:
>> 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
>> 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 ?
>> 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
>>> 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?
More information about the core-libs-dev