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

[Jiri Vanek]

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?


Works fine. Johnatan, can you push it? Or do you wont mila to create regular webrev? Is there some 
palce 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