Re: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms
Hi Sergey, Then the fix looks good to me. Thanks and regards, Shashi -Original Message- From: Sergey Bylokhov Sent: Friday, March 8, 2019 3:20 AM To: Shashidhara Veerabhadraiah ; Dmitry Markov Cc: awt-dev@openjdk.java.net Subject: Re: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms Hi, Shashi. On 07/03/2019 07:45, Shashidhara Veerabhadraiah wrote: > Hi Sergey, I think you need to add "@run main" command and this bug id to the > "@bug" list of bugs. Also you can update the copyright year to this year as > well. - The "@run main" tag is optional, this is default action for the test. - The @bug tag contains a bugid for which the test was written or the bugid which was found by this test later. It can also contain the bugid of bugfix in the test if the fix rewrote the test. - I'll update the year before the push. > > Thanks and regards, > Shashi > > -Original Message- > From: Dmitry Markov > Sent: Thursday, March 7, 2019 2:59 PM > To: Sergey Bylokhov > Cc: awt-dev@openjdk.java.net > Subject: Re: [13] Review Request: 8219504 Test for > JDK-8211435 can be run on all platforms > > Hi Sergey, > > The fix looks good to me. > > Thanks, > Dmitry > >> On 7 Mar 2019, at 06:59, Sergey Bylokhov wrote: >> >> Thank you for review! >> Looking for the second +1 from someone else. >> >> On 20/02/2019 22:20, Krishna Addepalli wrote: >>> +1 >>> Krishna >>> -----Original Message----- >>> From: Sergey Bylokhov >>> Sent: Thursday, February 21, 2019 8:01 AM >>> To: awt-dev@openjdk.java.net >>> Subject: [13] Review Request: 8219504 Test for JDK-8211435 >>> can be run on all platforms Hello. >>> Please review the fix for jdk 13. >>> Bug: https://bugs.openjdk.java.net/browse/JDK-8219504 >>> Fix: http://cr.openjdk.java.net/~serb/8219504/webrev.00 >>> Currently the test for JDK-8211435 is executed on macOS only, because of >>> JDK-8204142[1][2]: >>> But since JDK-8204142 was fixed we can run it on all platforms. >>> [1] https://bugs.openjdk.java.net/browse/JDK-8204142 >>> [2] >>> http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014492.h >>> t >>> ml >> >> >> -- >> Best regards, Sergey. > -- Best regards, Sergey.
Re: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms
Hi, Shashi. On 07/03/2019 07:45, Shashidhara Veerabhadraiah wrote: Hi Sergey, I think you need to add "@run main" command and this bug id to the "@bug" list of bugs. Also you can update the copyright year to this year as well. - The "@run main" tag is optional, this is default action for the test. - The @bug tag contains a bugid for which the test was written or the bugid which was found by this test later. It can also contain the bugid of bugfix in the test if the fix rewrote the test. - I'll update the year before the push. Thanks and regards, Shashi -Original Message- From: Dmitry Markov Sent: Thursday, March 7, 2019 2:59 PM To: Sergey Bylokhov Cc: awt-dev@openjdk.java.net Subject: Re: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms Hi Sergey, The fix looks good to me. Thanks, Dmitry On 7 Mar 2019, at 06:59, Sergey Bylokhov wrote: Thank you for review! Looking for the second +1 from someone else. On 20/02/2019 22:20, Krishna Addepalli wrote: +1 Krishna -Original Message- From: Sergey Bylokhov Sent: Thursday, February 21, 2019 8:01 AM To: awt-dev@openjdk.java.net Subject: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms Hello. Please review the fix for jdk 13. Bug: https://bugs.openjdk.java.net/browse/JDK-8219504 Fix: http://cr.openjdk.java.net/~serb/8219504/webrev.00 Currently the test for JDK-8211435 is executed on macOS only, because of JDK-8204142[1][2]: But since JDK-8204142 was fixed we can run it on all platforms. [1] https://bugs.openjdk.java.net/browse/JDK-8204142 [2] http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014492.ht ml -- Best regards, Sergey. -- Best regards, Sergey.
Re: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms
Hi Sergey, I think you need to add "@run main" command and this bug id to the "@bug" list of bugs. Also you can update the copyright year to this year as well. Thanks and regards, Shashi -Original Message- From: Dmitry Markov Sent: Thursday, March 7, 2019 2:59 PM To: Sergey Bylokhov Cc: awt-dev@openjdk.java.net Subject: Re: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms Hi Sergey, The fix looks good to me. Thanks, Dmitry > On 7 Mar 2019, at 06:59, Sergey Bylokhov wrote: > > Thank you for review! > Looking for the second +1 from someone else. > > On 20/02/2019 22:20, Krishna Addepalli wrote: >> +1 >> Krishna >> -Original Message- >> From: Sergey Bylokhov >> Sent: Thursday, February 21, 2019 8:01 AM >> To: awt-dev@openjdk.java.net >> Subject: [13] Review Request: 8219504 Test for JDK-8211435 >> can be run on all platforms Hello. >> Please review the fix for jdk 13. >> Bug: https://bugs.openjdk.java.net/browse/JDK-8219504 >> Fix: http://cr.openjdk.java.net/~serb/8219504/webrev.00 >> Currently the test for JDK-8211435 is executed on macOS only, because of >> JDK-8204142[1][2]: >> But since JDK-8204142 was fixed we can run it on all platforms. >> [1] https://bugs.openjdk.java.net/browse/JDK-8204142 >> [2] >> http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014492.ht >> ml > > > -- > Best regards, Sergey.
Re: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms
Hi Sergey, The fix looks good to me. Thanks, Dmitry > On 7 Mar 2019, at 06:59, Sergey Bylokhov wrote: > > Thank you for review! > Looking for the second +1 from someone else. > > On 20/02/2019 22:20, Krishna Addepalli wrote: >> +1 >> Krishna >> -Original Message- >> From: Sergey Bylokhov >> Sent: Thursday, February 21, 2019 8:01 AM >> To: awt-dev@openjdk.java.net >> Subject: [13] Review Request: 8219504 Test for JDK-8211435 can be >> run on all platforms >> Hello. >> Please review the fix for jdk 13. >> Bug: https://bugs.openjdk.java.net/browse/JDK-8219504 >> Fix: http://cr.openjdk.java.net/~serb/8219504/webrev.00 >> Currently the test for JDK-8211435 is executed on macOS only, because of >> JDK-8204142[1][2]: >> But since JDK-8204142 was fixed we can run it on all platforms. >> [1] https://bugs.openjdk.java.net/browse/JDK-8204142 >> [2] http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014492.html > > > -- > Best regards, Sergey.
Re: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms
Thank you for review! Looking for the second +1 from someone else. On 20/02/2019 22:20, Krishna Addepalli wrote: +1 Krishna -Original Message- From: Sergey Bylokhov Sent: Thursday, February 21, 2019 8:01 AM To: awt-dev@openjdk.java.net Subject: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms Hello. Please review the fix for jdk 13. Bug: https://bugs.openjdk.java.net/browse/JDK-8219504 Fix: http://cr.openjdk.java.net/~serb/8219504/webrev.00 Currently the test for JDK-8211435 is executed on macOS only, because of JDK-8204142[1][2]: But since JDK-8204142 was fixed we can run it on all platforms. [1] https://bugs.openjdk.java.net/browse/JDK-8204142 [2] http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014492.html -- Best regards, Sergey.
Re: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms
+1 Krishna -Original Message- From: Sergey Bylokhov Sent: Thursday, February 21, 2019 8:01 AM To: awt-dev@openjdk.java.net Subject: [13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms Hello. Please review the fix for jdk 13. Bug: https://bugs.openjdk.java.net/browse/JDK-8219504 Fix: http://cr.openjdk.java.net/~serb/8219504/webrev.00 Currently the test for JDK-8211435 is executed on macOS only, because of JDK-8204142[1][2]: But since JDK-8204142 was fixed we can run it on all platforms. [1] https://bugs.openjdk.java.net/browse/JDK-8204142 [2] http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014492.html -- Best regards, Sergey.
[13] Review Request: 8219504 Test for JDK-8211435 can be run on all platforms
Hello. Please review the fix for jdk 13. Bug: https://bugs.openjdk.java.net/browse/JDK-8219504 Fix: http://cr.openjdk.java.net/~serb/8219504/webrev.00 Currently the test for JDK-8211435 is executed on macOS only, because of JDK-8204142[1][2]: But since JDK-8204142 was fixed we can run it on all platforms. [1] https://bugs.openjdk.java.net/browse/JDK-8204142 [2] http://mail.openjdk.java.net/pipermail/awt-dev/2018-October/014492.html -- Best regards, Sergey.