Re: [9] Review Request for several test bugs: 8150535, 8151033, 8151037 etc.

2016-03-29 Thread Yuri Nesterenko
Hi Semyon, (see also my answer in https://bugs.openjdk.java.net/browse/JDK-8152693 ) Unfortunately, this changeset is already pushed. Generally, some of the tests is not hard to adapt for non-modules execution mode, and let's try that sometimes, however (1) some of them already run well with

Re: [9] Review Request for several test bugs: 8150535, 8151033, 8151037 etc.

2016-03-24 Thread Alexander Scherbatiy
The fix looks good to me. Thanks, Alexandr. On 24/03/16 18:17, Yuri Nesterenko wrote: Hi, please review this test-only fix, a leftover from Jigsaw M3 integration. Some 7 tests fixed but note that not all of them always pass even fixed. Webrev: http://cr.openjdk.java.net/~yan/8150535/webrev.01

Re: [9] Review Request for several test bugs: 8150535, 8151033, 8151037 etc.

2016-03-24 Thread Yuri Nesterenko
Sergey, I'm sorry, filing JDK-8152693, I found an error in that version. A new one is http://cr.openjdk.java.net/~yan/8150535/webrev.02 The spawned processes were provided with addExports. Shame on me! -yan On 03/24/2016 05:34 PM, Sergey Bylokhov wrote: cc swing-dev On 24.03.16 17:17, Yuri

Re: [9] Review Request for several test bugs: 8150535, 8151033, 8151037 etc.

2016-03-24 Thread Sergey Bylokhov
Looks fine. On 24.03.16 17:17, Yuri Nesterenko wrote: Hi, please review this test-only fix, a leftover from Jigsaw M3 integration. Some 7 tests fixed but note that not all of them always pass even fixed. Webrev: http://cr.openjdk.java.net/~yan/8150535/webrev.01 I need to file a bug about the

[9] Review Request for several test bugs: 8150535, 8151033, 8151037 etc.

2016-03-24 Thread Yuri Nesterenko
Hi, please review this test-only fix, a leftover from Jigsaw M3 integration. Some 7 tests fixed but note that not all of them always pass even fixed. Webrev: http://cr.openjdk.java.net/~yan/8150535/webrev.01 I need to file a bug about the fixed here manual test