RFR [15] : 8249036 : clean up FileInstaller $test.src $cwd in vmTestbase_nsk_stress tests
Igor Ignatyev
igor.ignatyev at oracle.com
Mon Jul 13 21:29:18 UTC 2020
http://cr.openjdk.java.net/~iignatyev//8249036/webrev.00/
> 44 lines changed: 0 ins; 23 del; 21 mod;
Hi all,
could you please review this clean-up which removes unnecessary `FileInstaller` actions for :vmTestbase_nsk_stress tests?
from the main issue(8204985):
> all vmTestbase tests have '@run driver jdk.test.lib.FileInstaller . .' to mimic old test harness behavior and copy all files from a test source directory to a current work directory. some tests depend on this step, so we need 1st identify such tests and then either rewrite them not to have this dependency or leave FileInstaller only in these tests.
none of vmTestbase_nsk_stress tests need FileInstaller, hence the patch is just `ag -l '@run driver jdk.test.lib.FileInstaller . .' vmTestbase/nsk/stress xargs -I{} gsed -i '/@run driver jdk.test.lib.FileInstaller \. \./d' {}`.
JBS: https://bugs.openjdk.java.net/browse/JDK-8249036
webrev: http://cr.openjdk.java.net/~iignatyev//8249036/webrev.00/
testing: :vmTestbase_nsk_stress on linux-x64
Thanks,
-- Igor
More information about the hotspot-compiler-dev
mailing list