Re: [PATCH] Changed java-tests module to require module java

2016-12-22 Thread Nadav Har'El
On Thu, Dec 22, 2016 at 2:38 PM, Waldek Kozaczuk wrote: > Also as I suggested in one of my older emails it might be worth to ponder > the idea of changing the default from java-isolated to java-non-isolated as > the former one requires java.beans package which does not exist in compact > profiles

Re: [PATCH] Changed java-tests module to require module java

2016-12-22 Thread Waldek Kozaczuk
Also as I suggested in one of my older emails it might be worth to ponder the idea of changing the default from java-isolated to java-non-isolated as the former one requires java.beans package which does not exist in compact profiles. Also java-isolated jar will not work in Java 9 without making so

Re: [PATCH] Changed java-tests module to require module java

2016-12-22 Thread Waldek Kozaczuk
Please see my answers below: On Thu, Dec 22, 2016 at 5:24 AM, Nadav Har'El wrote: > Hi, I committed this patch, but I also have a question below: > > > -- > Nadav Har'El > n...@scylladb.com > > On Thu, Dec 22, 2016 at 7:20 AM, Waldemar Kozaczuk > wrote: > >> This patch addresses a build issue w

Re: [PATCH] Changed java-tests module to require module java

2016-12-22 Thread Nadav Har'El
Hi, I committed this patch, but I also have a question below: -- Nadav Har'El n...@scylladb.com On Thu, Dec 22, 2016 at 7:20 AM, Waldemar Kozaczuk wrote: > This patch addresses a build issue where this build command would fail due > to both jdk 7 and 8 included: > scripts/build -j4 image=java-

[PATCH] Changed java-tests module to require module java

2016-12-21 Thread Waldemar Kozaczuk
This patch addresses a build issue where this build command would fail due to both jdk 7 and 8 included: scripts/build -j4 image=java-tests,openjdk8-fedora && scripts/test.py --name java.* Following build commands should work now: scripts/build -j4 image=java-tests && scripts/test.py --name java