Re: RFR: JDK-8264244: BasicShortcutHintTest shortcut can not be found

2020-06-17 Thread Alexey Semenyuk
Yeh, you are right. It was copy/paste error. Thank you for the correction! - Alexey On 6/16/2020 6:13 PM, Alexander Matveev wrote: Hi Alexey, Looks good. I think you got links and bug ID incorrect. It should be JDK-8246244 and you have 8264244. Links also does not work. Working links are: h

Re: RFR: JDK-8264244: BasicShortcutHintTest shortcut can not be found

2020-06-16 Thread Andy Herrick
looks good with these links /Andy On 6/16/2020 6:13 PM, Alexander Matveev wrote: Hi Alexey, Looks good. I think you got links and bug ID incorrect. It should be JDK-8246244 and you have 8264244. Links also does not work. Working links are: https://bugs.openjdk.java.net/browse/JDK-8246244 ht

RFR: JDK-8264244: BasicShortcutHintTest shortcut can not be found

2020-06-16 Thread Alexey Semenyuk
Please review fix [2] for jpackage bug [1]. The fix is to put value of `Exec` property of .desktop files generated by jpackage in double quotes if the value of the propery contains whitespace characters. - Alexey [1] https://bugs.openjdk.java.net/browse/JDK-8264244 [2] http://cr.openjdk.jav

Re: RFR: JDK-8264244: BasicShortcutHintTest shortcut can not be found

2020-06-16 Thread Alexander Matveev
Hi Alexey, Looks good. I think you got links and bug ID incorrect. It should be JDK-8246244 and you have 8264244. Links also does not work. Working links are: https://bugs.openjdk.java.net/browse/JDK-8246244 http://cr.openjdk.java.net/~asemenyuk/8246244/webrev.00/ Thanks, Alexander On 6/16/2