Re: RFR: 8236127: Use value of --icon CLI option to set icon for exe installers [v3]

2021-04-06 Thread Alexander Matveev
On Wed, 7 Apr 2021 00:22:14 GMT, Alexey Semenyuk wrote: >> Use value of `--icon` jpackage CLI option to set icon for exe installers. > > Alexey Semenyuk has updated the pull request incrementally with one > additional commit since the last revision: > > Fix

Re: RFR: 8248418: jpackage fails to extract main class and version from app module linked in external runtime

2021-03-29 Thread Alexander Matveev
On Mon, 29 Mar 2021 18:11:26 GMT, Alexey Semenyuk wrote: > Remove the test for failing jpackage use case scenario that we don't intent > to fix Marked as reviewed by almatvee (Committer). - PR: https://git.openjdk.java.net/jdk/pull/3249

Re: RFR: 8220266: add support for additional metadata in add/remove programs

2021-03-24 Thread Alexander Matveev
On Wed, 24 Mar 2021 19:01:04 GMT, Alexey Semenyuk wrote: > Add support for `--about-url`, `--win-update-url` and `--win-help-url` > parameters. > Implementation for `--about-url` is provided on Windows only. > This parameter may be supported on Linux as well. >

Re: RFR: 8189198: Add "forRemoval = true" to Applet API deprecations

2021-03-12 Thread Alexander Matveev
On Wed, 10 Mar 2021 18:33:37 GMT, Andy Herrick wrote: > implementation of > JDK-8256145: JEP 398: Deprecate the Applet API for Removal Marked as reviewed by almatvee (Committer). - PR: https://git.openjdk.java.net/jdk/pull/2920

Re: RFR: 8263536: Add missing @compile tags to jpackage tests

2021-03-12 Thread Alexander Matveev
On Fri, 12 Mar 2021 17:52:30 GMT, Alexey Semenyuk wrote: > 8263536: Add missing @compile tags to jpackage tests Marked as reviewed by almatvee (Committer). - PR: https://git.openjdk.java.net/jdk/pull/2975

Re: RFR: JDK-8263545: Convert jpackage to use Stream.toList()

2021-03-18 Thread Alexander Matveev
On Thu, 18 Mar 2021 19:51:06 GMT, Ian Graves wrote: >> @igraves as a "best practice" try to avoid doing a force-push to a branch >> with an active code review. It makes it hard for reviewers to see what has >> changed. If you need to merge changes from master, use "git merge master" >>

Re: RFR: JDK-8263154: [macos] DMG builds have finder errors

2021-03-15 Thread Alexander Matveev
On Sat, 13 Mar 2021 14:39:40 GMT, Andy Herrick wrote: > JDK-8263154: [macos] DMG builds have finder errors Can you explain what problem is and how it is fixed? Does it fails when "install-dir" is specified? "install-dir" is not relevant for DMG image and we should ignore it. -

Re: RFR: JDK-8263545: Convert jpackage to use Stream.toList() [v2]

2021-03-15 Thread Alexander Matveev
On Mon, 15 Mar 2021 18:53:26 GMT, Ian Graves wrote: >> This converts jpackage to use `Stream.toList()` instead of >> `Stream.collect(Collectors.toList())`. One piece of code was modified to not >> mutate a list in addition to one test that used a mutating sort on a list. >> The rest of the

Re: RFR: JDK-8263154: [macos] DMG builds have finder errors

2021-03-16 Thread Alexander Matveev
On Sat, 13 Mar 2021 14:39:40 GMT, Andy Herrick wrote: > JDK-8263154: [macos] DMG builds have finder errors Marked as reviewed by almatvee (Committer). - PR: https://git.openjdk.java.net/jdk/pull/2987

Re: RFR: 8241716: Jpackage functionality to let users choose whether to create shortcuts [v2]

2021-03-09 Thread Alexander Matveev
On Tue, 9 Mar 2021 16:50:40 GMT, Alexey Semenyuk wrote: >> Add support to insert dialog with prompts to create shortcuts in dialog >> installation sequence of Windows installers created by jpackage. >> >> Dialog look: >>

Re: RFR: JDK-8248904: Add support to jpackage for the Mac App Store

2021-03-08 Thread Alexander Matveev
On Wed, 24 Feb 2021 21:59:22 GMT, Andy Herrick wrote: > Implementation of Mac App Support including three new mac specific CLI > options. Marked as reviewed by almatvee (Committer). - PR: https://git.openjdk.java.net/jdk/pull/2716

Re: RFR: JDK-8261518: jpackage looks for main module in current dir when there is no module-path

2021-03-01 Thread Alexander Matveev
On Mon, 1 Mar 2021 15:55:39 GMT, Andy Herrick wrote: > when the app modules have already been jlinked with the runtime, and there is > no need for module-path, jpackage was acting as if the module-path was "." > and picking up jars in the current directory. Marked as reviewed by almatvee

Re: RFR: JDK-8261839: Error creating runtime package on macos without mac-pack…

2021-02-25 Thread Alexander Matveev
On Thu, 25 Feb 2021 21:15:44 GMT, Andy Herrick wrote: > …age-identifier Marked as reviewed by almatvee (Committer). - PR: https://git.openjdk.java.net/jdk/pull/2730

Re: RFR: 8262300: jpackage app-launcher fails on linux when using JDK11 based runtime

2021-03-04 Thread Alexander Matveev
On Thu, 4 Mar 2021 12:32:11 GMT, Alexey Semenyuk wrote: > Fix jpackage app launcher to also look for JLI lib in "lib/jli/libjli.so" > subdirectory of runtime Marked as reviewed by almatvee (Committer). - PR: https://git.openjdk.java.net/jdk/pull/2827

Integrated: 8261845: File permissions of packages built by jpackage

2021-03-06 Thread Alexander Matveev
On Thu, 4 Mar 2021 03:59:27 GMT, Alexander Matveev wrote: > - Fixed by adding write permissions to .exe package. This pull request has now been integrated. Changeset: fa43f926 Author: Alexander Matveev URL: https://git.openjdk.java.net/jdk/commit/fa43f926 Stats: 3 lines in 1 f

RFR: 8261845: File permissions of packages built by jpackage

2021-03-03 Thread Alexander Matveev
- Fixed by adding write permissions to .exe package. - Commit messages: - 8261845: File permissions of packages built by jpackage Changes: https://git.openjdk.java.net/jdk/pull/2822/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=2822=00 Issue:

RFR: 8263154: [macos] DMG builds have finder errors

2021-04-14 Thread Alexander Matveev
- Issue was reproducible when install-dir points to some invalid location. - Fixed by defaulting DMG drag and drop location to /Applications folder and --install-dir will be ignored with warning for DMG. - I do not see any way to support other valid, but uncommon locations for drag and drop.

Re: RFR: 8263154: [macos] DMG builds have finder errors

2021-04-15 Thread Alexander Matveev
On Thu, 15 Apr 2021 01:24:04 GMT, Alexander Matveev wrote: > - Issue was reproducible when install-dir points to some invalid location. > - Fixed by defaulting DMG drag and drop location to /Applications folder and > --install-dir will be ignored with warning for DMG. > - I do

RFR: 8263157: [macos]: java.library.path is being set incorrectly

2021-04-12 Thread Alexander Matveev
This is regression from JDK-8242302. Fixed by setting java.library.path to same values as it was before JDK-8242302. - Commit messages: - 8263157: [macos]: java.library.path is being set incorrectly Changes: https://git.openjdk.java.net/jdk/pull/3443/files Webrev:

Integrated: 8263154: [macos] DMG builds have finder errors

2021-04-19 Thread Alexander Matveev
On Thu, 15 Apr 2021 01:24:04 GMT, Alexander Matveev wrote: > - Issue was reproducible when install-dir points to some invalid location. > - Fixed by defaulting DMG drag and drop location to /Applications folder and > --install-dir will be ignored with warning for DMG. > - I do

Integrated: 8263157: [macos]: java.library.path is being set incorrectly

2021-04-13 Thread Alexander Matveev
On Mon, 12 Apr 2021 20:18:46 GMT, Alexander Matveev wrote: > This is regression from JDK-8242302. Fixed by setting java.library.path to > same values as it was before JDK-8242302. This pull request has now been integrated. Changeset: 55d56495 Author:Alexander Matveev URL:

Re: RFR: 8263154: [macos] DMG builds have finder errors

2021-04-16 Thread Alexander Matveev
On Thu, 15 Apr 2021 01:24:04 GMT, Alexander Matveev wrote: > - Issue was reproducible when install-dir points to some invalid location. > - Fixed by defaulting DMG drag and drop location to /Applications folder and > --install-dir will be ignored with warning for DMG. > - I do

Re: RFR: JDK-8272639: jpackaged applications using microphone on mac

2021-08-19 Thread Alexander Matveev
On Thu, 19 Aug 2021 14:12:00 GMT, Andy Herrick wrote: > JDK-8272639: jpackaged applications using microphone on mac Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/5186

Re: RFR: 8272815: jpackage --type rpm produces an error: Invalid or unsupported type: [null] [v2]

2021-09-14 Thread Alexander Matveev
On Tue, 14 Sep 2021 21:47:36 GMT, Alexey Semenyuk wrote: >> Fix jpackage error output when "--type" CLI option is missing and jpackage >> detects that it can't build native packages in the environment. > > Alexey Semenyuk has updated the pull request incrementally with one > additional commit

Re: [jdk17] RFR: JDK-8272639: jpackaged applications using microphone on mac

2021-09-09 Thread Alexander Matveev
On Thu, 9 Sep 2021 20:14:01 GMT, Andy Herrick wrote: > backport from jdk-18 Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk17/pull/306

Re: RFR: JDK-8269387: jpackage --add-launcher should have option to not create shortcuts for additional launchers [v4]

2021-07-14 Thread Alexander Matveev
On Wed, 14 Jul 2021 16:50:39 GMT, Andy Herrick wrote: >> JDK-8269387: jpackage --add-launcher should have option to not create >> shortcuts for additional launchers > > Andy Herrick has updated the pull request incrementally with one additional > commit since the last revision: > >

Re: RFR: JDK-8271344: Windows product version issue

2021-08-02 Thread Alexander Matveev
On Mon, 2 Aug 2021 13:45:17 GMT, Andy Herrick wrote: > JDK-8271344: Windows product version issue Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/4958

Re: RFR: 8271170: Add unit test for what jpackage app launcher puts in the environment [v2]

2021-08-09 Thread Alexander Matveev
On Mon, 9 Aug 2021 18:19:03 GMT, Alexey Semenyuk wrote: >> Added jtreg test to verify jpackage launcher adds app dir to >> `java.library.path` system property. >> Removed unused test/jdk/tools/jpackage/apps/installer/Hello.java. >> Moved test/jdk/tools/jpackage/apps/image/Hello.java to >>

Re: RFR: 8271868: Warn user when using mac-sign option with unsigned app-image. [v2]

2021-08-05 Thread Alexander Matveev
On Thu, 5 Aug 2021 17:07:13 GMT, Andy Herrick wrote: >> 8271868: Warn user when using mac-sign option with unsigned app-image. > > Andy Herrick has updated the pull request incrementally with one additional > commit since the last revision: > > JDK-8271868: Warn user when using mac-sign

Re: RFR: 8271868: Warn user when using mac-sign option with unsigned app-image.

2021-08-04 Thread Alexander Matveev
On Wed, 4 Aug 2021 20:00:37 GMT, Andy Herrick wrote: > 8271868: Warn user when using mac-sign option with unsigned app-image. I think it will not work. User might modify app image after it was created and it will invalidate signature. - PR:

Re: RFR: 8271868: Warn user when using mac-sign option with unsigned app-image.

2021-08-04 Thread Alexander Matveev
On Wed, 4 Aug 2021 20:00:37 GMT, Andy Herrick wrote: > 8271868: Warn user when using mac-sign option with unsigned app-image. Changes requested by almatvee (Reviewer). src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/MacAppImageBuilder.java line 877: > 875: try { > 876:

Re: RFR: 8271868: Warn user when using mac-sign option with unsigned app-image.

2021-08-04 Thread Alexander Matveev
On Wed, 4 Aug 2021 20:00:37 GMT, Andy Herrick wrote: > 8271868: Warn user when using mac-sign option with unsigned app-image. First of all why we only want to cover case when app-image was produced without --mac-sign? If we want to cover such case only, then we need to use approach suggested

Re: RFR: 8272328: java.library.path is not set properly by Windows jpackage app launcher

2021-08-11 Thread Alexander Matveev
On Wed, 11 Aug 2021 17:52:57 GMT, Alexey Semenyuk wrote: > Restart window app launcher to make changes made to `PATH` env variable > visible to JLI code in > https://github.com/openjdk/jdk/blob/master/src/hotspot/os/windows/os_windows.cpp#L347 Marked as reviewed by almatvee (Reviewer).

Re: RFR: JDK-8263155: Allow additional contents for DMG [v2]

2021-10-21 Thread Alexander Matveev
On Thu, 21 Oct 2021 15:36:36 GMT, Andy Herrick wrote: >> JDK-8263155: Allow additional contents for DMG > > Andy Herrick has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains five commits: > > - Merge branch 'master' into

Re: RFR: 8274346: Support for additional content in an app-image. [v2]

2021-10-06 Thread Alexander Matveev
On Wed, 6 Oct 2021 18:53:39 GMT, Andy Herrick wrote: >> 8274346: Support for additional content in an app-image. > > Andy Herrick has updated the pull request incrementally with one additional > commit since the last revision: > > JDK-8274346: Support for additional content in an app-image.

Re: RFR: JDK-8274087: Windows DLL path not set correctly.

2021-09-23 Thread Alexander Matveev
On Thu, 23 Sep 2021 20:03:29 GMT, Andy Herrick wrote: > JDK-8274087: Windows DLL path not set correctly. Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/5663

[jdk18] Integrated: 8278970: [macos] SigningPackageTest is failed with runtime exception

2021-12-17 Thread Alexander Matveev
On Fri, 17 Dec 2021 03:28:12 GMT, Alexander Matveev wrote: > This is regression from JDK-8263155. MacHelper.java is now calling test > verification callback with all content in DMG root. SigningPackageTest > expects only path with app name in it and thus it fails when trying to veri

Re: [jdk18] RFR: 8279370: jdk.jpackage/share/native/applauncher/JvmLauncher.cpp fails to build with GCC 6.3.0

2022-01-03 Thread Alexander Matveev
On Mon, 3 Jan 2022 08:10:27 GMT, Aleksey Shipilev wrote: > Seems like a missing include. C++ docs say `offsetof` is from ``, > adding that include explicitly fixes the build. Seems to only happen with > older GCCs, but it seems to be a happy accident it works on newer ones, > probably through

Re: RFR: 8277507: Add jlink.debug system property while launching jpackage tests to help diagonize recent intermittent failures

2021-11-22 Thread Alexander Matveev
On Sun, 21 Nov 2021 12:39:17 GMT, Jaikiran Pai wrote: > Can I please get a review of this change which proposes to add more > diagnostics when a failure occurs in the jlink tool during the jpackage tests? > > As noted in https://bugs.openjdk.java.net/browse/JDK-8277507, so far 3 > failures

Re: RFR: 8277429: Conflicting jpackage static library name

2021-11-19 Thread Alexander Matveev
On Sat, 20 Nov 2021 03:26:51 GMT, Alexey Semenyuk wrote: > 8277429: Conflicting jpackage static library name Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/6485

Re: RFR: 8276084: Linux DEB Bundler: release number in outputted .deb file should be optional

2021-11-12 Thread Alexander Matveev
On Thu, 11 Nov 2021 04:07:18 GMT, Alexey Semenyuk wrote: > 8276084: Linux DEB Bundler: release number in outputted .deb file should be > optional Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/6345

Re: RFR: JDKJDK-8276150: Quarantined jpackage apps are labeled as "damaged"

2021-11-16 Thread Alexander Matveev
On Mon, 15 Nov 2021 19:56:28 GMT, Andy Herrick wrote: > JDKJDK-8276150: Quarantined jpackage apps are labeled as "damaged" Looks good. Just fix pull request title. - Marked as reviewed by almatvee (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/6396

Re: RFR: 8274856: Failing jpackage tests with fastdebug/release build

2021-11-06 Thread Alexander Matveev
On Fri, 5 Nov 2021 19:58:01 GMT, Alexey Semenyuk wrote: > The fix is to isolate C++ calls in the separate forked child process on > Linux. > This change requires the passing of JLI command line arguments and values of > environment variables between two processes. Marked as reviewed by

Re: RFR: JDK-8276562: Fix to JDK-8263155 left out the help text changes

2021-11-06 Thread Alexander Matveev
On Fri, 5 Nov 2021 17:21:01 GMT, Andy Herrick wrote: > JDK-8276562: Fix to JDK-8263155 left out the help text changes Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/6281

[jdk18] RFR: 8278233: [macos] tools/jpackage tests timeout due to /usr/bin/osascript

2021-12-13 Thread Alexander Matveev
This is regression from JDK-8276837. exec() was passing INFINITE_TIMEOUT instead of actual value of timeout variable. Execution of osascript was running without timeout and thus several tests timeout. Osascript hang during test execution is intermittent issue. Also, removed IconTest.java and

[jdk18] RFR: 8278907: JPackage test - SigningPackageTest is failed with runtime exception

2021-12-16 Thread Alexander Matveev
This is regression from JDK-8263155. MacHelper.java is now calling test verification callback with all content in DMG root. SigningPackageTest expects only path with app name in it and thus it fails when trying to verify app inside ".background" folder. Fixed by checking that provided path for

Re: RFR: JDK-8276837: [macos]: Error when signing the additional launcher

2021-12-01 Thread Alexander Matveev
On Wed, 1 Dec 2021 21:35:01 GMT, Andy Herrick wrote: > - We need to unsign all executables and libraries in the app-image before > signing. (not just those in the runtime). > - Clean up excessive output by executing the individual file sign and > unsigning commands in quiet mode. > - Add

Re: RFR: 8278311: Debian packaging doesn't work

2021-12-06 Thread Alexander Matveev
On Mon, 6 Dec 2021 19:20:00 GMT, Alexey Semenyuk wrote: > 8278311: Debian packaging doesn't work Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/6726

[jdk18] Integrated: 8278233: [macos] tools/jpackage tests timeout due to /usr/bin/osascript

2021-12-14 Thread Alexander Matveev
On Tue, 14 Dec 2021 06:10:51 GMT, Alexander Matveev wrote: > This is regression from JDK-8276837. exec() was passing INFINITE_TIMEOUT > instead of actual value of timeout variable. Execution of osascript was > running without timeout and thus several tests timeout. Osascript ha

Re: [jdk17] RFR: 8271155: Wrong path separator in env variable

2021-07-22 Thread Alexander Matveev
On Thu, 22 Jul 2021 19:35:59 GMT, Alexey Semenyuk wrote: > Replace `";"` with `FileUtils::pathSeparator` in the expression adding 'app' > dir to env variable in jpackage app launcher. Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk17/pull/271

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description [v2]

2022-02-14 Thread Alexander Matveev
On Fri, 11 Feb 2022 22:11:44 GMT, Alexey Semenyuk wrote: >> Alexander Matveev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8279995: jpackage --add-launcher option should allow overriding >> descr

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description [v3]

2022-02-14 Thread Alexander Matveev
> Added ability to override description for additional launchers via > "description" property. Alexander Matveev has updated the pull request incrementally with one additional commit since the last revision: 8279995: jpackage --add-launcher option should allow overriding

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description

2022-02-11 Thread Alexander Matveev
On Wed, 9 Feb 2022 07:37:42 GMT, Alexander Matveev wrote: > Added ability to override description for additional launchers via > "description" property. Added automated tests for .exe files in Windows and .desktop files on Linux. - PR: https://git.openjdk.java.net/jdk/pull/7399

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description [v2]

2022-02-11 Thread Alexander Matveev
> Added ability to override description for additional launchers via > "description" property. Alexander Matveev has updated the pull request incrementally with one additional commit since the last revision: 8279995: jpackage --add-launcher option should allow overriding

RFR: 8282407: Missing ')' in MacResources.properties

2022-03-11 Thread Alexander Matveev
- Fixed by adding missing ']'. - I changed '()' to '[]', since other error messages use '[]' and not '()'. - Commit messages: - 8282407: Missing ')' in MacResources.properties Changes: https://git.openjdk.java.net/jdk/pull/7797/files Webrev:

Re: RFR: JDK-8236128: Allow jpackage create installers for services

2022-03-14 Thread Alexander Matveev
On Fri, 11 Mar 2022 23:37:06 GMT, Alexey Semenyuk wrote: > Implementation of [JDK-8275062: "Allow jpackage create installers for > services"](https://bugs.openjdk.java.net/browse/JDK-8275062) > CSR Marked as reviewed by almatvee (Reviewer). - PR:

Integrated: 8282407: Missing ')' in MacResources.properties

2022-03-17 Thread Alexander Matveev
On Sat, 12 Mar 2022 03:12:30 GMT, Alexander Matveev wrote: > - Fixed by adding missing ']'. > - I changed '()' to '[]', since other error messages use '[]' and not '()'. This pull request has now been integrated. Changeset: d83cee98 Author:Alexander Matveev URL:

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description [v6]

2022-02-24 Thread Alexander Matveev
> Added ability to override description for additional launchers via > "description" property. Alexander Matveev has updated the pull request incrementally with one additional commit since the last revision: 8279995: jpackage --add-launcher option should allow overriding

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description [v4]

2022-02-24 Thread Alexander Matveev
On Thu, 17 Feb 2022 17:53:57 GMT, Alexey Semenyuk wrote: >> Alexander Matveev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8279995: jpackage --add-launcher option should allow overriding >> descr

Re: RFR: 8282007: Assorted enhancements to jpackage testing framework [v3]

2022-02-16 Thread Alexander Matveev
On Thu, 17 Feb 2022 03:50:39 GMT, Alexey Semenyuk wrote: >> 8282007: Assorted enhancements to jpackage testing framework > > Alexey Semenyuk has updated the pull request incrementally with one > additional commit since the last revision: > > Copyright years fixed Marked as reviewed by

Re: RFR: 8282007: Assorted enhancements to jpackage testing framework [v2]

2022-02-16 Thread Alexander Matveev
On Wed, 16 Feb 2022 23:51:30 GMT, Alexey Semenyuk wrote: >> 8282007: Assorted enhancements to jpackage testing framework > > Alexey Semenyuk has updated the pull request incrementally with one > additional commit since the last revision: > > Don't fail if requested to uninstall not installed

Re: RFR: 8282011: test/jdk/tools/jpackage/windows/WinL10nTest.java test fails if light.exe is not in %PATH%

2022-02-16 Thread Alexander Matveev
On Wed, 16 Feb 2022 17:52:43 GMT, Alexey Semenyuk wrote: > 8282011: test/jdk/tools/jpackage/windows/WinL10nTest.java test fails if > light.exe is not in %PATH% Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/7500

Integrated: 8279995: jpackage --add-launcher option should allow overriding description

2022-02-25 Thread Alexander Matveev
On Wed, 9 Feb 2022 07:37:42 GMT, Alexander Matveev wrote: > Added ability to override description for additional launchers via > "description" property. This pull request has now been integrated. Changeset: fb8bf818 Author:Alexander Matveev URL: https://git.open

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description [v5]

2022-02-17 Thread Alexander Matveev
> Added ability to override description for additional launchers via > "description" property. Alexander Matveev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits: - Merge master - 8279995: jpackag

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description [v4]

2022-02-16 Thread Alexander Matveev
> Added ability to override description for additional launchers via > "description" property. Alexander Matveev has updated the pull request incrementally with one additional commit since the last revision: 8279995: jpackage --add-launcher option should allow overriding

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description [v3]

2022-02-17 Thread Alexander Matveev
On Tue, 15 Feb 2022 15:48:02 GMT, Alexey Semenyuk wrote: >> Alexander Matveev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8279995: jpackage --add-launcher option should allow overriding >> descr

Re: RFR: 8281874: Can't unpack msi installers from test/jdk/tools/jpackage/windows/test/jdk/tools/jpackage/windows/WinShortcutPromptTest.java test

2022-02-15 Thread Alexander Matveev
On Tue, 15 Feb 2022 22:56:35 GMT, Alexey Semenyuk wrote: > 8281874: Can't unpack msi installers from > test/jdk/tools/jpackage/windows/test/jdk/tools/jpackage/windows/WinShortcutPromptTest.java > test Marked as reviewed by almatvee (Reviewer). - PR:

Re: RFR: 8281170: Test jdk/tools/jpackage/windows/WinInstallerIconTest always fails on Windows 11

2022-02-15 Thread Alexander Matveev
On Tue, 15 Feb 2022 17:44:24 GMT, Alexey Semenyuk wrote: > Code clean up. Remove obsolete comments, unused imports, and unneeded jtreg > test parameter. > Fix the code to handle the case when installers are not created and there is > nothing to verify in the test. Marked as reviewed by

RFR: 8279995: jpackage --add-launcher option should allow overriding description

2022-02-08 Thread Alexander Matveev
Added ability to override description for additional launchers via "description" property. - Commit messages: - 8279995: jpackage --add-launcher option should allow overriding description Changes: https://git.openjdk.java.net/jdk/pull/7399/files Webrev:

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description

2022-02-09 Thread Alexander Matveev
On Wed, 9 Feb 2022 12:35:59 GMT, Alexey Semenyuk wrote: >> Added ability to override description for additional launchers via >> "description" property. > > test/jdk/tools/jpackage/share/AddLauncherTest.java line 93: > >> 91: new AdditionalLauncher("Baz2") >> 92:

Re: RFR: 8279995: jpackage --add-launcher option should allow overriding description

2022-02-09 Thread Alexander Matveev
On Wed, 9 Feb 2022 07:37:42 GMT, Alexander Matveev wrote: > Added ability to override description for additional launchers via > "description" property. AddLauncherArguments.java class reads parameters for additional launcher from property file and if this class contains par

Re: Integrated: 8257434: jpackage fails to create rpm on Fedora Linux

2022-01-18 Thread Alexander Matveev
On Tue, 1 Dec 2020 23:54:11 GMT, Alexey Semenyuk wrote: > Add missing quotes to `%if` expressions in spec template. Required by > rpmbuild 4.16.0 Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/1551

Re: RFR: 8284067: jpackage'd launcher reports non-zero exit codes with error prompt

2022-04-05 Thread Alexander Matveev
On Thu, 31 Mar 2022 20:08:13 GMT, Alexey Semenyuk wrote: > Add missing `exit(exitCode)` call. > Add relevant unit test. Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/8064

Re: RFR: 8277493: [REDO] Quarantined jpackage apps are labeled as "damaged"

2022-05-12 Thread Alexander Matveev
On Wed, 4 May 2022 03:56:10 GMT, Alexander Matveev wrote: > - No changes to code provided by original fix. > - Added ad hoc signing on macOS aarch64, since macOS aarch64 cannot execute > unsigned code and code should be at least ad hoc signed. > - Signing of app bun

Re: RFR: 8277493: [REDO] Quarantined jpackage apps are labeled as "damaged" [v2]

2022-05-12 Thread Alexander Matveev
undle structure with added files into Content folder. Thus test > is executed but failure is always expected on macOS aarch64. Alexander Matveev has updated the pull request incrementally with one additional commit since the last revision: 8277493: [REDO] Quarantined jpackage apps are l

Re: [External] : Re: RFR: 8286122: [macos]: App bundle cannot upload to Mac App Store due to info.plist embedded in java exe

2022-05-16 Thread Alexander Matveev
Hi Michael, > >>> >>> I’m not real familiar with the build process. But would it be possible for >>> the user to build their own jdk that substitutes something else for the >>> colliding identifier that gets embedded? >> Yes, it should be possible and in theory such JDK with native commands

Re: [External] : Re: RFR: 8286122: [macos]: App bundle cannot upload to Mac App Store due to info.plist embedded in java exe

2022-05-16 Thread Alexander Matveev
jdk.jpackage module with your app, you should able to use jpackage functionality from your app via ToolProvider without actual jpackage native command. Thanks, Alexander > On May 16, 2022, at 6:03 PM, Michael Hall wrote: > > > >> On May 16, 2022, at 7:09 PM, Alexander Matveev &g

Integrated: 8286122: [macos]: App bundle cannot upload to Mac App Store due to info.plist embedded in java exe

2022-05-18 Thread Alexander Matveev
On Wed, 11 May 2022 21:31:44 GMT, Alexander Matveev wrote: > - It is not possible to support native JDK commands such as "java" inside Mac > App Store bundles due to embedded info.plist. Workarounds suggested in > JDK-8286122 does not seems to be visible. >

RFR: 8287125: [macos] Multiple jpackage tests fail/timeout on same host

2022-05-26 Thread Alexander Matveev
Looks like regression from JDK-8277493. JDK-8277493 will always un-sign app image. Un-signing takes time since we enumerating all files and un-signing binaries one by one. Average increase 2-3 minutes for tests which generates multiple app images. Fixed by increasing timeout for reported tests.

RFR: 8286850: [macos] Add support for signing user provided app image

2022-06-01 Thread Alexander Matveev
- Added support for signing predefined application image. - Following command can be used to sign predefined application images: jpackage --type app-image --app-image Test.app --mac-sign [additional signing options] - Main class and if --mac-app-store was specified will be saved in

Integrated: 8287125: [macos] Multiple jpackage tests fail/timeout on same host

2022-06-02 Thread Alexander Matveev
On Fri, 27 May 2022 03:12:39 GMT, Alexander Matveev wrote: > Looks like regression from JDK-8277493. JDK-8277493 will always un-sign app > image. Un-signing takes time since we enumerating all files and un-signing > binaries one by one. Average increase 2-3 minutes for tests which

Re: RFR: 8286122: [macos]: App bundle cannot upload to Mac App Store due to info.plist embedded in java exe

2022-05-11 Thread Alexander Matveev
Hi Michael, > On May 11, 2022, at 3:17 PM, Michael Hall wrote: > > Is this restricted somehow to Mac App Store applications? Yes, helper tools (in our case JDK native commands) in Mac App Store applications cannot use same bundle ID as another application. Since we have bundle ID embedded in

Integrated: 8277493: [REDO] Quarantined jpackage apps are labeled as "damaged"

2022-05-13 Thread Alexander Matveev
On Wed, 4 May 2022 03:56:10 GMT, Alexander Matveev wrote: > - No changes to code provided by original fix. > - Added ad hoc signing on macOS aarch64, since macOS aarch64 cannot execute > unsigned code and code should be at least ad hoc signed. > - Signing of app bun

Re: RFR: 8286122: [macos]: App bundle cannot upload to Mac App Store due to info.plist embedded in java exe [v2]

2022-05-11 Thread Alexander Matveev
On Wed, 11 May 2022 21:58:46 GMT, Alexey Semenyuk wrote: >> Alexander Matveev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8286122: [macos]: App bundle cannot upload to Mac App Store due to >> info.

Re: RFR: 8286122: [macos]: App bundle cannot upload to Mac App Store due to info.plist embedded in java exe [v2]

2022-05-11 Thread Alexander Matveev
tion for > jlink, so native JDK commands are not included when generating Mac App Store > bundles. > - Custom runtime provided via --runtime-image should not contain native > commands as well, otherwise jpackage will throw error. > - Added two tests to validate fix.

Re: RFR: 8281682: Redundant .ico files in Windows app-image cause unnecessary bloat

2022-05-19 Thread Alexander Matveev
On Thu, 19 May 2022 19:13:12 GMT, Alexey Semenyuk wrote: > 8281682: Redundant .ico files in Windows app-image cause unnecessary bloat Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/8794

Re: RFR: 8286122: [macos]: App bundle cannot upload to Mac App Store due to info.plist embedded in java exe

2022-05-19 Thread Alexander Matveev
to run in order to cleanup extracted files. Thanks, Alexander On May 18, 2022, at 6:44 PM, Michael Hall mailto:mik3h...@gmail.com>> wrote: On May 11, 2022, at 4:39 PM, Alexander Matveev mailto:almat...@openjdk.java.net>> wrote: - It is not possible to support native

Re: RFR: 8286850: [macos] Add support for signing user provided app image [v2]

2022-06-06 Thread Alexander Matveev
Hi Michael, See below. On Jun 5, 2022, at 5:58 PM, Michael Hall mailto:mik3h...@gmail.com>> wrote: ./build/*/images/jdk/bin/jpackage --app-image ~/HalfPipe/halfpipe_jpkg/outputdir/HalfPipe.app --mac-sign --mac-signing-key-user-name "Developer ID Application: Michael Hall (5X6BXQB3Q7)"

Re: RFR: 8286850: [macos] Add support for signing user provided app image [v2]

2022-06-06 Thread Alexander Matveev
Hi Michael, Printing "Warning: Using unsigned app-image to build signed dmg.” with signed application should be fixed with JDK-8286850. Did you reproduce it on build containing JDK-8286850? Thanks, Alexander > On Jun 5, 2022, at 3:06 PM, Michael Hall wrote: > > > >> On Jun 5, 2022, at

RFR: 8287971: Throw exception for missing values in .jpackage.xml

2022-06-08 Thread Alexander Matveev
- Warning will be printed if .jpackage.xml has old version number. - Error will be thrown if main-class and app-store values are missing from .jpackage.xml. - Both main-class and app-store are required values for JDK-8286850 and they might be missing if user uses app images generated by

Re: [External] : Re: RFR: 8286850: [macos] Add support for signing user provided app image [v2]

2022-06-07 Thread Alexander Matveev
Hi Michael, Yes, this is correct. It is a three step process as you outlined it below. Thanks, Alexander On Jun 7, 2022, at 12:00 AM, Michael Hall mailto:mik3h...@gmail.com>> wrote: Alexander, I had an existing local GitHub repo for the jdk I updated that appeared to accept the parameters

Integrated: 8286850: [macos] Add support for signing user provided app image

2022-06-03 Thread Alexander Matveev
On Thu, 2 Jun 2022 05:17:25 GMT, Alexander Matveev wrote: > - Added support for signing predefined application image. > - Following command can be used to sign predefined application images: > jpackage --type app-image --app-image Test.app --mac-sign [additional signing > option

Re: RFR: 8286850: [macos] Add support for signing user provided app image [v2]

2022-06-02 Thread Alexander Matveev
app images are supported. App image will be > completely unsigned before signing. Alexander Matveev has updated the pull request incrementally with one additional commit since the last revision: 8286850: [macos] Add support for signing user provided app image [v2] - Changes: -

Re: RFR: 8286850: [macos] Add support for signing user provided app image [v2]

2022-06-02 Thread Alexander Matveev
On Fri, 3 Jun 2022 01:56:01 GMT, Alexander Matveev wrote: >> - Added support for signing predefined application image. >> - Following command can be used to sign predefined application images: >> jpackage --type app-image --app-image Test.app --mac-sign [additional

Re: RFR: 8286850: [macos] Add support for signing user provided app image [v3]

2022-06-02 Thread Alexander Matveev
app images are supported. App image will be > completely unsigned before signing. Alexander Matveev has updated the pull request incrementally with one additional commit since the last revision: 8286850: [macos] Add support for signing user provided app image [v3] - Changes: -

Re: RFR: 8250950: Allow per-user and system wide configuration of a jpackaged app

2022-06-03 Thread Alexander Matveev
On Fri, 3 Jun 2022 21:07:47 GMT, Alexey Semenyuk wrote: > 8250950: Allow per-user and system wide configuration of a jpackaged app test/jdk/tools/jpackage/share/PerUserCfgTest.java line 174: > 172: action.accept(cmd); > 173: } finally { > 174:

Re: RFR: 8250950: Allow per-user and system wide configuration of a jpackaged app [v2]

2022-06-06 Thread Alexander Matveev
On Mon, 6 Jun 2022 20:37:11 GMT, Alexey Semenyuk wrote: >> 8250950: Allow per-user and system wide configuration of a jpackaged app > > Alexey Semenyuk has updated the pull request incrementally with one > additional commit since the last revision: > > Add TKit.createDirectories() method

RFR: 8277493: [REDO] Quarantined jpackage apps are labeled as "damaged"

2022-05-03 Thread Alexander Matveev
- No changes to code provided by original fix. - Added ad hoc signing on macOS aarch64, since macOS aarch64 cannot execute unsigned code and code should be at least ad hoc signed. - Signing of app bundle produced by AppContentTest.java fails, since it has invalid app bundle structure with

Re: RFR: 8282351: jpackage does not work if class file has `$$` in the name on windows

2022-05-09 Thread Alexander Matveev
On Mon, 9 May 2022 21:25:37 GMT, Alexey Semenyuk wrote: > 8282351: jpackage does not work if class file has `$$` in the name on windows Marked as reviewed by almatvee (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/8613

Re: RFR: 8285616: [macos] Incorrect path for launcher-as-service.txt in .cfg file

2022-05-05 Thread Alexander Matveev
On Thu, 5 May 2022 20:00:54 GMT, Alexey Semenyuk wrote: > - Replace `System.getProperty("java.io.tmpdir")` call with hardcoded `/tmp` > string to get root folder for test files. > - Fix test cleanup - the test didn't delete test files upon completion Marked as reviewed by almatvee (Reviewer).

<    1   2   3   4   5   6   >