[8u60] review request for RT-22993: Native installer - allow user to choose installation directory

2015-02-24 Thread Dmitry Cherepanov
Danno, Could you please review the fix for RT-22993 JIRA: https://javafx-jira.kenai.com/browse/RT-22993 Webrev: http://cr.openjdk.java.net/~dcherepanov/RT-22993/webrev.0 Thanks, Dmitry

Re: [sandbox-9-jake] Review Request 8080531 - fix ant-javafx ant tasks

2015-10-19 Thread Dmitry Cherepanov
Looks fine to me. Dmitry On 10/19/15 11:02 PM, Danno Ferrin wrote: Kevin, Chris, Dmitry Because of the way resources are being handled in current revs of Jake we need to pull the main ant task classes back into ant-javafx and not deploy them into the main packager module. Properly

[9] Review request for 8144922: [packager] intermittent test failures

2015-12-08 Thread Dmitry Cherepanov
Chris, Please review the following fix https://bugs.openjdk.java.net/browse/JDK-8144922 http://cr.openjdk.java.net/~dcherepanov/8144922/webrev.v0/ Thanks, Dmitry

[9] Review request for 8144619: [packager] GetSystemJRE fails after changes for new version string

2015-12-04 Thread Dmitry Cherepanov
Chris, Kevin, Please review the following fix https://bugs.openjdk.java.net/browse/JDK-8144619 http://cr.openjdk.java.net/~dcherepanov/8144619/webrev.v0/ Thanks, Dmitry

[9] Review request for 8091625: [Packager, Windows] launcher.exe stub should have version resource

2015-11-30 Thread Dmitry Cherepanov
Chris, Danno, Kevin, Please review the following fix https://bugs.openjdk.java.net/browse/JDK-8091625 http://cr.openjdk.java.net/~dcherepanov/8091625/webrev.v0/ Thanks, Dmitry

[8u-dev] Review request: 8143314: Runtime not respected with INI-configuration while creating native bundle

2015-11-19 Thread Dmitry Cherepanov
Chris, Danno, Please review the following fix: https://bugs.openjdk.java.net/browse/JDK-8143314 http://cr.openjdk.java.net/~dcherepanov/8143314/webrev.0/ Dmitry

[9] Review request for 8088039: [launcher] Change Windows and Linux to use JLI rather than JNI

2016-01-12 Thread Dmitry Cherepanov
Chris, Please review the following fix https://bugs.openjdk.java.net/browse/JDK-8088039 http://cr.openjdk.java.net/~dcherepanov/8088039/webrev.v0/ Thanks, Dmitry