RFR(S): 8025606: [TESTBUG] Move ctw-tests to /testlibrary_tests
Evgeniya Stepanova
evgeniya.stepanova at oracle.com
Thu Sep 18 08:02:40 UTC 2014
Vladimir,
Thanks for review!
Igor, could you please push changes to the repo?
Thanks,
Jane
On 17.09.2014 20:58, Vladimir Kozlov wrote:
> On 9/17/14 2:36 AM, Igor Ignatyev wrote:
>> Vladimir,
>>
>>> What about testlibrary/ctw/Makefile? Should we move/modify it?
>> we don't need to move or modify Makefile, since it doesn't contain
>> any references to tests.
>
> Okay.
>
>>
>>> Do we need it at all?
>> it depends on point of view, hotspot doesn't require it to build (as
>> well as testlibrary/whitebox/Makefile), but I'd
>> prefer to leave them, since the intent why I added these Makefiles
>> was to generate ctw.jar/wb.jar from hotpost sources
>> during the build cycle of another testbase, e.g. vm testbase, ctw.
>
> Okay. Changes look fine then.
>
> Thanks,
> Vladimir
>
>>
>> Thanks,
>> Igor
>>
>> On 09/17/2014 12:30 PM, Vladimir Kozlov wrote:
>>> What about testlibrary/ctw/Makefile? Should we move/modify it? Do we
>>> need it at all?
>>>
>>> Thanks,
>>> Vladimir
>>>
>>> On 9/17/14 12:43 AM, Evgeniya Stepanova wrote:
>>>> Hi,
>>>>
>>>> Please review this small patch.
>>>> Tests for testlibrary moved to testlibrary_tests to exclude them from
>>>> the classpath when running other tests
>>>>
>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8025606
>>>> webrev:
>>>> http://cr.openjdk.java.net/~iignatyev/eistepan/8025606/webrev.00/
>>>>
>>>>
>>>> Thanks,
>>>> Jane
>>>> --
>>>> /Evgeniya Stepanova/
--
/Evgeniya Stepanova/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20140918/b892cded/attachment.html>
More information about the hotspot-compiler-dev
mailing list