Hi,

sending fix for jimage bug JDK-8170114[1]. I'm not sure whether this is correct list. If it is not, please direct me somewhere else.

I don't have an openjdk account, so webrev is on my fedora public space. I will need a sponsor for this.

webrev: 
https://michalvala.fedorapeople.org/webrevs/JDK-8170114/webrev.00/webrev/

Patch validates output directory before any jimage extracting happen. I've moved validation to extra private method as it is few lines of code. I've also added proper error message for case when output path is not a directory (JImageTask.java#449).

I've implemented beforeTest method, that cleans cwd. This must be done because files were extracted to cwd and constantly overwriting and tests were non-deterministic (order did matter). Plus some tests added.




I want to fix JDK-8170120 with IOException when file is not an jimage. It is blocked by this. This will fix 3 now problematic and excluded tests.

[1] - https://bugs.openjdk.java.net/browse/JDK-8170114

--
Michal Vala
OpenJDK QE
Red Hat Czech

Reply via email to