Re: RFR: JDK-8230522: rename "--linux-bundle-name", and "--temp-root" options.
On 9/9/2019 6:31 PM, Alexander Matveev wrote: Hi Andy, http://cr.openjdk.java.net/~herrick/8230522/webrev.01/src/jdk.jpackage/linux/classes/jdk/jpackage/internal/resources/LinuxResources.properties.frames.html Line 44 "mp." -> "map" This is a typo I will fix. http://cr.openjdk.java.net/~herrick/8230522/webrev.01/src/jdk.jpackage/share/classes/jdk/jpackage/internal/Arguments.java.frames.html Do we want to rename internal names like MAC_BUNDLE_NAMEĀ -> MAC_PACKAGE_NAME? Generally, I didn't want to change anything but the CLI option name, and that includes the deploy parameter name used internally. If we want a follow up issue to make all internal deploy parameter identifiers correspond more closely to the CLI options they reflect, I have no problem with that, but it would apply to a lot more than the 4-5 options being renamed here. I think we should at least as follow up cleanup issue, to make code less confusing with different name. Otherwise looks fine. Thanks, Alexander On 9/9/2019 12:15 PM, Andy Herrick wrote: Please review the jpackage fix for bug [1] at [2]. This is a fix for the JDK-8200758-branch branch of the open sandbox repository (jpackage). This fix: modifies the name of options: --temp-root, --linux-bundle-name , --mac-bundle-name, --mac-bundle-identifier, and --mac-bundle-signing-prefix. new names: --temp, --linux-package-name , --mac-package-name, --mac-package-identifier, and --mac-package-signing-prefix. This change also also removes the period at the end of all error messages. [1] https://bugs.openjdk.java.net/browse/JDK-8230522 [2] http://cr.openjdk.java.net/~herrick/8230522 Thanks Andy,
Re: RFR: JDK-8230522: rename "--linux-bundle-name", and "--temp-root" options.
Hi Andy, http://cr.openjdk.java.net/~herrick/8230522/webrev.01/src/jdk.jpackage/linux/classes/jdk/jpackage/internal/resources/LinuxResources.properties.frames.html Line 44 "mp." -> "map" http://cr.openjdk.java.net/~herrick/8230522/webrev.01/src/jdk.jpackage/share/classes/jdk/jpackage/internal/Arguments.java.frames.html Do we want to rename internal names like MAC_BUNDLE_NAMEĀ -> MAC_PACKAGE_NAME? I think we should at least as follow up cleanup issue, to make code less confusing with different name. Otherwise looks fine. Thanks, Alexander On 9/9/2019 12:15 PM, Andy Herrick wrote: Please review the jpackage fix for bug [1] at [2]. This is a fix for the JDK-8200758-branch branch of the open sandbox repository (jpackage). This fix: modifies the name of options: --temp-root, --linux-bundle-name , --mac-bundle-name, --mac-bundle-identifier, and --mac-bundle-signing-prefix. new names: --temp, --linux-package-name , --mac-package-name, --mac-package-identifier, and --mac-package-signing-prefix. This change also also removes the period at the end of all error messages. [1] https://bugs.openjdk.java.net/browse/JDK-8230522 [2] http://cr.openjdk.java.net/~herrick/8230522 Thanks Andy,
Re: RFR: JDK-8230522: rename "--linux-bundle-name", and "--temp-root" options.
Looks good. - Alexey On 9/9/2019 3:15 PM, Andy Herrick wrote: Please review the jpackage fix for bug [1] at [2]. This is a fix for the JDK-8200758-branch branch of the open sandbox repository (jpackage). This fix: modifies the name of options: --temp-root, --linux-bundle-name , --mac-bundle-name, --mac-bundle-identifier, and --mac-bundle-signing-prefix. new names: --temp, --linux-package-name , --mac-package-name, --mac-package-identifier, and --mac-package-signing-prefix. This change also also removes the period at the end of all error messages. [1] https://bugs.openjdk.java.net/browse/JDK-8230522 [2] http://cr.openjdk.java.net/~herrick/8230522 Thanks Andy,
RFR: JDK-8230522: rename "--linux-bundle-name", and "--temp-root" options.
Please review the jpackage fix for bug [1] at [2]. This is a fix for the JDK-8200758-branch branch of the open sandbox repository (jpackage). This fix: modifies the name of options: --temp-root, --linux-bundle-name , --mac-bundle-name, --mac-bundle-identifier, and --mac-bundle-signing-prefix. new names: --temp, --linux-package-name , --mac-package-name, --mac-package-identifier, and --mac-package-signing-prefix. This change also also removes the period at the end of all error messages. [1] https://bugs.openjdk.java.net/browse/JDK-8230522 [2] http://cr.openjdk.java.net/~herrick/8230522 Thanks Andy,