Looks good for me. On 2016-09-02 10:33, Sharath Ballal wrote: > Hi Dmitry, > I have done the changes: > http://cr.openjdk.java.net/~sballal/8164943/webrev.01/ > > > -Sharath Ballal > > > > -----Original Message----- > From: Dmitry Samersoff > Sent: Thursday, September 01, 2016 6:04 PM > To: Sharath Ballal; serviceability-dev@openjdk.java.net > Subject: Re: RFR: JDK-8164943: sun/tools/jhsdb/HeapDumpTest failed with Can't > find library: /test/lib/share/classes > > Sharath, > > Please remove > > 30 * @build jdk.testlibrary.* > 31 * @build jdk.test.lib.apps.* > > as well. > > -Dmitry > > On 2016-09-01 07:37, Sharath Ballal wrote: >> Please review this fix to change /test/lib/share/classes to /test/lib/ >> in the test: >> >> >> >> Issue: https://bugs.openjdk.java.net/browse/JDK-8164943 >> >> >> >> Webrev: http://cr.openjdk.java.net/~sballal/8164943/webrev.00/ >> >> >> >> -Sharath Ballal >> > > > -- > Dmitry Samersoff > Oracle Java development team, Saint Petersburg, Russia > * I would love to change the world, but they won't give me the sources. >
-- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.