[9] RFR 4154028: Several hotspot tests need to be updated after 8153737 (Unsupported Module)
Can I get a quick review for this fix that's blocking the synching of jdk9/hs with jdk9/dev: Basically, https://bugs.openjdk.java.net/browse/JDK-8153737 introduced some changes that need to be applied to several hotspot tests that were not in jdk9/dev by the time that fix was pushed. They can't be fixed on either repo, the need to be fixed in the merging repo before being pushed back to jdk9/hs (first) Bug https://bugs.openjdk.java.net/browse/JDK-8154028 Webrev: http://cr.openjdk.java.net/~amurillo/9/8154028/ I'm running a sanity test job now, but wanted to get the review going as I need to get the synch done ASAP to start PIT Thanks -- Alejandro
Looks fine Alejandro; thanks, -Joe On 4/11/2016 5:14 PM, Alejandro Murillo wrote:
Can I get a quick review for this fix that's blocking the synching of jdk9/hs with jdk9/dev:
Basically, https://bugs.openjdk.java.net/browse/JDK-8153737 introduced some changes that need to be applied to several hotspot tests that were not in jdk9/dev by the time that fix was pushed.
They can't be fixed on either repo, the need to be fixed in the merging repo before being pushed back to jdk9/hs (first)
Bug https://bugs.openjdk.java.net/browse/JDK-8154028 Webrev: http://cr.openjdk.java.net/~amurillo/9/8154028/
I'm running a sanity test job now, but wanted to get the review going as I need to get the synch done ASAP to start PIT
Thanks
Thanks Joe. Alejandro On 4/11/2016 6:44 PM, joe darcy wrote:
Looks fine Alejandro; thanks,
-Joe
On 4/11/2016 5:14 PM, Alejandro Murillo wrote:
Can I get a quick review for this fix that's blocking the synching of jdk9/hs with jdk9/dev:
Basically, https://bugs.openjdk.java.net/browse/JDK-8153737 introduced some changes that need to be applied to several hotspot tests that were not in jdk9/dev by the time that fix was pushed.
They can't be fixed on either repo, the need to be fixed in the merging repo before being pushed back to jdk9/hs (first)
Bug https://bugs.openjdk.java.net/browse/JDK-8154028 Webrev: http://cr.openjdk.java.net/~amurillo/9/8154028/
I'm running a sanity test job now, but wanted to get the review going as I need to get the synch done ASAP to start PIT
Thanks
-- Alejandro
Looks good. Thanks, Vladimir On 4/11/16 5:14 PM, Alejandro Murillo wrote:
Can I get a quick review for this fix that's blocking the synching of jdk9/hs with jdk9/dev:
Basically, https://bugs.openjdk.java.net/browse/JDK-8153737 introduced some changes that need to be applied to several hotspot tests that were not in jdk9/dev by the time that fix was pushed.
They can't be fixed on either repo, the need to be fixed in the merging repo before being pushed back to jdk9/hs (first)
Bug https://bugs.openjdk.java.net/browse/JDK-8154028 Webrev: http://cr.openjdk.java.net/~amurillo/9/8154028/
I'm running a sanity test job now, but wanted to get the review going as I need to get the synch done ASAP to start PIT
Thanks
Thanks Vladimir Alejandro On 4/11/2016 7:19 PM, Vladimir Kozlov wrote:
Looks good.
Thanks, Vladimir
On 4/11/16 5:14 PM, Alejandro Murillo wrote:
Can I get a quick review for this fix that's blocking the synching of jdk9/hs with jdk9/dev:
Basically, https://bugs.openjdk.java.net/browse/JDK-8153737 introduced some changes that need to be applied to several hotspot tests that were not in jdk9/dev by the time that fix was pushed.
They can't be fixed on either repo, the need to be fixed in the merging repo before being pushed back to jdk9/hs (first)
Bug https://bugs.openjdk.java.net/browse/JDK-8154028 Webrev: http://cr.openjdk.java.net/~amurillo/9/8154028/
I'm running a sanity test job now, but wanted to get the review going as I need to get the synch done ASAP to start PIT
Thanks
-- Alejandro
participants (3)
-
Alejandro Murillo
-
joe darcy
-
Vladimir Kozlov