Bugfix: do not set TESTMODULES environment variable if JDK is not modular

[Jiri Vanek]

On 12/20/2017 05:35 PM, Jiri Vanek wrote:
> On 12/15/2017 11:37 AM, Miloslav Zezulka wrote:
>> This is a very small patch fixing the issue pointed out in $subject. I've
>> added checks for both compile and test JDK - is this an overkill, since
>> most of the time the implication goes COMPILEJDK.modular =>
>> TESTJDK.modular, or a necessary thing to have for foolproofness?
> 
ping?
> 
> Works fine. Johnatan, can you push it? Or do you wont mila to create regular webrev? Is there some 
> place to add test for it?
> 
> Tahnx a lot!
>    J.
> 
> 


-- 
Jiri Vanek
Senior QE engineer, OpenJDK QE lead, Mgr.
Red Hat Czech
jvanek at redhat.com    M: +420775390109