Thanks David, pushed to jdk15.
-- Igor > On Jul 13, 2020, at 7:47 PM, David Holmes <david.hol...@oracle.com> wrote: > > Looks good! > > Thanks, > David > > On 9/07/2020 3:21 am, Igor Ignatyev wrote: >> http://cr.openjdk.java.net/~iignatyev/8249028/webrev.00/ >>> 547 lines changed: 0 ins; 361 del; 186 mod; >> Hi all, >> could you please review the patch which removes `FileInstaller . .` jtreg >> action from :vmTestbase_nsk_monitoring 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. >> effectively, the patch is just `ag -l '@run driver >> jdk.test.lib.FileInstaller . .' vmTestbase/nsk/monitoring | xargs -I{} >> gsed -i '/@run driver jdk.test.lib.FileInstaller \. \./d' {}` >> testing: :vmTestbase_nsk_monitoring on linux-x64 >> webrev: http://cr.openjdk.java.net/~iignatyev//8249018/webrev.00 >> JBS: https://bugs.openjdk.java.net/browse/JDK-8249028 >> Thanks, >> -- Igor
