RFR: 8264144: Add handling of "--about-url" CLI parameter for RPM/DEB packages

2021-06-08 Thread Alexey Semenyuk
…ages - Commit messages: - 8264144: Add handling of "--about-url" CLI parameter for RPM/DEB packages Changes: https://git.openjdk.java.net/jdk/pull/4415/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=4415=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8264144

Re: RFR: 8264144: Add handling of "--about-url" CLI parameter for RPM/DEB packages [v2]

2021-06-08 Thread Alexey Semenyuk
> …ages Alexey Semenyuk has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: - Merge branch 'mas

Integrated: 8264144: Add handling of "--about-url" CLI parameter for RPM/DEB packages

2021-06-08 Thread Alexey Semenyuk
On Tue, 8 Jun 2021 16:45:27 GMT, Alexey Semenyuk wrote: > …ages This pull request has now been integrated. Changeset: bcaa2cb1 Author: Alexey Semenyuk URL: https://git.openjdk.java.net/jdk/commit/bcaa2cb154ae5d23a067f6e38a19a21eef8fe8e8 Stats: 115 lines in 5 files changed:

Re: jpackage issue with --dest

2021-06-17 Thread Alexey Semenyuk
Thanks. I'll look into it. - Alexey On 6/17/2021 2:01 PM, Maurizio Cimadamore wrote: Filed this: https://bugs.openjdk.java.net/browse/JDK-8268974 Maurizio On 17/06/2021 18:52, Alexey Semenyuk wrote: Seems like a new issue. Please file a bug in Jira. - Alexey On 6/16/2021 5:14 PM

Re: jpackage issue with --dest

2021-06-17 Thread Alexey Semenyuk
Maurizio On 16/06/2021 22:11, Maurizio Cimadamore wrote: Hi, I've built my JDK this morning. I checked now with `git log` and I do have: https://bugs.openjdk.java.net/browse/JDK-8267598 Maurizio On 16/06/2021 21:12, Alexey Semenyuk wrote: Hi Maurizio, Thank you for the provided output.

Re: [jdk17] RFR: 8269036: tools/jpackage/share/AppImagePackageTest.java failed with "hdiutil: create failed - Resource busy"

2021-06-22 Thread Alexey Semenyuk
On Tue, 22 Jun 2021 21:59:43 GMT, Alexander Matveev wrote: > Looks like another "Resource busy" issue similar to recent fixes for "hdiutil > convert" and "hdiutil detach". Workaround in same way by repeating "create" > command. Modified RetryExecutor to pass write to file flag, otherwise >

Re: RFR: 8268974: jpackage fails to handle --dest option containing "bin" folder [v2]

2021-06-21 Thread Alexey Semenyuk
looks for "/lib/". But this is wrong order as it should > look for "/lib/" first. I.e. TruncatePath() should look for "/bin/" and then > for "/lib/" if called from GetApplicationHome() and for "/lib/" first and > then for "/bin/" i

Re: RFR: 8268974: jpackage fails to handle --dest option containing "bin" folder

2021-06-21 Thread Alexey Semenyuk
On Fri, 18 Jun 2021 22:46:24 GMT, Alexey Semenyuk wrote: > GetApplicationHomeFromDll() fails if the path to libjli.so contains "bin" > component (/tmp/bin/HelloWorld/lib/runtime/lib/libjli.so). TruncatePath() > looks for "/bin/" first in "/tmp/bin/He

Re: [jdk17] RFR: JDK-8268404: [TESTBUG] tools/jpackage/windows/WinInstallerIconTest.java failed "AssertionError: Failed: Check icon"

2021-06-21 Thread Alexey Semenyuk
On Mon, 21 Jun 2021 20:21:58 GMT, Andy Herrick wrote: > …t.java failed "AssertionError: Failed: Check icon" Changes requested by asemenyuk (Reviewer). test/jdk/tools/jpackage/windows/WinInstallerIconTest.java line 75: > 73: > 74: // Create another installer with custom icon. > 75:

Re: [jdk17] RFR: JDK-8268404: [TESTBUG] tools/jpackage/windows/WinInstallerIconTest.java failed "AssertionError: Failed: Check icon" [v2]

2021-06-22 Thread Alexey Semenyuk
On Tue, 22 Jun 2021 12:34:04 GMT, Andy Herrick wrote: >> …t.java failed "AssertionError: Failed: Check icon" > > Andy Herrick has updated the pull request incrementally with one additional > commit since the last revision: > > JDK-8268404: [TESTBUG]

RFR: 8268974: jpackage fails to handle --dest option containing "bin" folder

2021-06-18 Thread Alexey Semenyuk
GetApplicationHomeFromDll() fails if the path to libjli.so contains "bin" component (/tmp/bin/HelloWorld/lib/runtime/lib/libjli.so). TruncatePath() looks for "/bin/" first in "/tmp/bin/HelloWorld/lib/runtime/lib/libjli.so" string and then it looks for "/lib/". But this is wrong order as it

Re: RFR: JDK-8267764: jpackage cannot handle window screensaver files when EXE renamed as SCR [v2]

2021-06-08 Thread Alexey Semenyuk
On Tue, 8 Jun 2021 14:06:50 GMT, Andy Herrick wrote: >> JDK-8267764: jpackage cannot handle window screensaver files when EXE >> renamed as SCR > > Andy Herrick has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated

Integrated: 8266162: Remove JPackage duplicate tests

2021-05-13 Thread Alexey Semenyuk
On Wed, 12 May 2021 20:34:08 GMT, Alexey Semenyuk wrote: > 8266162: Remove JPackage duplicate tests This pull request has now been integrated. Changeset: f3c6cda4 Author: Alexey Semenyuk URL: https://git.openjdk.java.net/jdk/commit/f3c6cda47631cc123dbcddbfb627dc05cf7bc13b St

Re: RFR: 8266456: Replace direct TKit.run() calls with jdk.jpackage.test.Annotations.Test annotation [v2]

2021-05-10 Thread Alexey Semenyuk
On Fri, 7 May 2021 23:07:02 GMT, Alexander Matveev wrote: >> test/jdk/tools/jpackage/helpers/jdk/jpackage/test/Functional.java line 161: >> >>> 159: } >>> 160: >>> 161: if >>> (throwable.getClass().getName().equals("jtreg.SkippedException")) { >> >> Would it make sense to

Re: RFR: 8266456: Replace direct TKit.run() calls with jdk.jpackage.test.Annotations.Test annotation [v2]

2021-05-10 Thread Alexey Semenyuk
On Sat, 8 May 2021 01:44:46 GMT, Alexander Matveev wrote: >> - Replaced direct TKit.run() calls with Test annotation. >> - Increased timeout for SigningPackageTest from default to 360 due to >> timeout. This is regression from JDK-8248904 due to changes done in signing >> and

Re: RFR: 8266456: Replace direct TKit.run() calls with jdk.jpackage.test.Annotations.Test annotation

2021-05-07 Thread Alexey Semenyuk
On Fri, 7 May 2021 02:48:44 GMT, Alexander Matveev wrote: > - Replaced direct TKit.run() calls with Test annotation. > - Increased timeout for SigningPackageTest from default to 360 due to > timeout. This is regression from JDK-8248904 due to changes done in signing > and --remove-signature

Re: RFR: 8266603: jpackage: Add missing copyright file in Java runtime .deb installers [v2]

2021-05-07 Thread Alexey Semenyuk
installation directory of Java runtime. However when installing Java runtime > or app in /usr tree, copyright file will be placed outside of installation > directory. Thus copyright file should be always created if package > installation directory is inside of /usr tree. Alexey Semenyu

RFR: 8266603: jpackage: Add missing copyright file in Java runtime .deb installers

2021-05-07 Thread Alexey Semenyuk
jpackage should create copyright file in /usr/share/doc directory tree when building .deb package for Java runtime with installation directory in /usr directory tree. jpackage creates share/doc/copyright file in installation directory for apps installed outside of /usr tree. jpackage creates

Re: RFR: 8266456: Replace direct TKit.run() calls with jdk.jpackage.test.Annotations.Test annotation [v3]

2021-05-11 Thread Alexey Semenyuk
On Tue, 11 May 2021 03:40:19 GMT, Alexander Matveev wrote: >> - Replaced direct TKit.run() calls with Test annotation. >> - Increased timeout for SigningPackageTest from default to 360 due to >> timeout. This is regression from JDK-8248904 due to changes done in signing >> and

RFR: 8266162: Remove JPackage duplicate tests

2021-05-12 Thread Alexey Semenyuk
8266162: Remove JPackage duplicate tests - Commit messages: - 8266162: Remove JPackage duplicate tests Changes: https://git.openjdk.java.net/jdk/pull/4003/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=4003=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8266162

Re: RFR: 8267056: tools/jpackage/share/RuntimePackageTest.java fails with NoSuchFileException [v2]

2021-05-20 Thread Alexey Semenyuk
On Thu, 20 May 2021 22:25:10 GMT, Alexander Matveev wrote: >> For debug build on macOS, runtime which used for test fill be located in >> /path/jdk-17/fastdebug and /path/jdk-17 will not contain any other files >> except fastdebug and in this case our check to decide if we need to copy app

Re: RFR: 8249395: (macos) jpackage tests timeout on MacPro5_1 systems

2021-05-14 Thread Alexey Semenyuk
On Fri, 14 May 2021 04:24:42 GMT, Alexander Matveev wrote: > Looks like it is similar to JDK-8236282, except for "hdiutil create". Based > on call stack from failed test we launched "hdiutil create" and were waiting > for it to terminate while reading output from it. However, based on process

Integrated: 8266603: jpackage: Add missing copyright file in Java runtime .deb installers

2021-05-10 Thread Alexey Semenyuk
On Thu, 6 May 2021 01:22:41 GMT, Alexey Semenyuk wrote: > jpackage should create copyright file in /usr/share/doc directory tree when > building .deb package for Java runtime with installation directory in /usr > directory tree. > > jpackage creates share/doc/copyright file

Re: RFR: 8267056: tools/jpackage/share/RuntimePackageTest.java fails with NoSuchFileException

2021-05-20 Thread Alexey Semenyuk
On Wed, 19 May 2021 21:00:07 GMT, Alexander Matveev wrote: > For debug build on macOS, runtime which used for test fill be located in > /path/jdk-17/fastdebug and /path/jdk-17 will not contain any other files > except fastdebug and in this case our check to decide if we need to copy app > or

Re: RFR: 8267403: tools/jpackage/share/FileAssociationsTest.java#id0 failed with "Error: Bundler "Mac PKG Package" (pkg) failed to produce a package"

2021-05-25 Thread Alexey Semenyuk
On Fri, 21 May 2021 04:34:22 GMT, Alexander Matveev wrote: > Looks like another issue similar to hdiutil (JDK-8249395) when process is > gone, but we still waiting for it. I was not able to reproduce this issue by > running test or pkgbuild separately and conclusion was made based on logs. >

Re: RFR: JDK-8267764: jpackage cannot handle window screensaver files when EXE renamed as SCR

2021-06-02 Thread Alexey Semenyuk
On Wed, 2 Jun 2021 13:48:47 GMT, Andy Herrick wrote: > JDK-8267764: jpackage cannot handle window screensaver files when EXE renamed > as SCR Marked as reviewed by asemenyuk (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/4306

Re: RFR: JDK-8267598: jpackage removes system libraries from java.library.path [v2]

2021-06-03 Thread Alexey Semenyuk
On Thu, 3 Jun 2021 16:07:13 GMT, Andy Herrick wrote: >> JDK-8267598: jpackage removes system libraries from java.library.path > > Andy Herrick has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev excludes the unrelated changes > brought in by

Re: RFR: JDK-8267423: Fix copyrights in jpackage tests

2021-05-24 Thread Alexey Semenyuk
On Fri, 21 May 2021 12:22:16 GMT, Andy Herrick wrote: > JDK-8267423: Fix copyrights in jpackage tests Marked as reviewed by asemenyuk (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/4144

Re: RFR: 8268974: GetJREPath() JLI function fails to locate libjava.so if not standard Java launcher is used

2021-07-07 Thread Alexey Semenyuk
On Sat, 3 Jul 2021 07:41:40 GMT, Alan Bateman wrote: >> Is it possible to add a test for this that is completely independent of >> jpackage? I think there are a few existing tests that copy the run-time >> image to a new location for testing purposes. >> >> We may need to rename the JBS

Re: RFR: 8268974: GetJREPath() JLI function fails to locate libjava.so if not standard Java launcher is used [v3]

2021-07-07 Thread Alexey Semenyuk
looks for "/lib/". But this is wrong order as it should > look for "/lib/" first. I.e. TruncatePath() should look for "/bin/" and then > for "/lib/" if called from GetApplicationHome() and for "/lib/" first and > then for "/bin/" i

[jdk17] Integrated: 8269185: Directories in /opt/runtimepackagetest and /path/to/jdk-17 are different

2021-07-07 Thread Alexey Semenyuk
On Fri, 2 Jul 2021 22:17:09 GMT, Alexey Semenyuk wrote: > Disable stripping to prevent rpmbuild from modifying executables This pull request has now been integrated. Changeset: 6000950b Author: Alexey Semenyuk URL: https://git.openjdk.java.net/jdk17/com

Re: RFR: 8268974: GetJREPath() JLI function fails to locate libjava.so if not standard Java launcher is used [v4]

2021-07-07 Thread Alexey Semenyuk
looks for "/lib/". But this is wrong order as it should > look for "/lib/" first. I.e. TruncatePath() should look for "/bin/" and then > for "/lib/" if called from GetApplicationHome() and for "/lib/" first and > then for "/bin/" i

[jdk17] RFR: 8269185: Directories in /opt/runtimepackagetest and /path/to/jdk-17 are different

2021-07-02 Thread Alexey Semenyuk
Disable stripping to prevent rpmbuild from modifying executables - Commit messages: - Disable executables stripping Changes: https://git.openjdk.java.net/jdk17/pull/207/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk17=207=00 Issue:

Re: RFR: 8268974: GetJREPath() JLI function fails to locate libjava.so if not standard Java launcher is used

2021-07-02 Thread Alexey Semenyuk
On Sun, 20 Jun 2021 07:25:54 GMT, Alan Bateman wrote: >> GetApplicationHomeFromDll() fails if the path to libjli.so contains "bin" >> component (/tmp/bin/HelloWorld/lib/runtime/lib/libjli.so). TruncatePath() >> looks for "/bin/" first in "/tmp/bin/HelloWorld/lib/runtime/lib/libjli.so" >>

[jdk17] RFR: 8269403: Fix jpackage tests to gracefully handle jpackage app launcher crashes

2021-06-25 Thread Alexey Semenyuk
jpackage app launcher randomly crashes JVM at termination in EMS enabled JDK builds. Until the root cause of the issue is understood and fixed let's add a workaround to jpackage tests to run test app few more times if the first attempt resulted in app crash. - Commit messages: -

[jdk17] Integrated: 8269403: Fix jpackage tests to gracefully handle jpackage app launcher crashes

2021-06-28 Thread Alexey Semenyuk
On Fri, 25 Jun 2021 22:41:10 GMT, Alexey Semenyuk wrote: > jpackage app launcher randomly crashes JVM at termination in EMS enabled JDK > builds. > Until the root cause of the issue is understood and fixed let's add a > workaround to jpackage tests to run test app few more times

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

2021-07-08 Thread Alexey Semenyuk
On Thu, 8 Jul 2021 19:25:33 GMT, Andy Herrick wrote: > JDK-8269387: jpackage --add-launcher should have option to not create > shortcuts for additional launchers src/jdk.jpackage/share/classes/jdk/jpackage/internal/AddLauncherArguments.java line 39: > 37: > 38: /* > 39: *

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

2021-07-09 Thread Alexey Semenyuk
On Fri, 9 Jul 2021 17:59:35 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-8269387: jpackage --add-launcher should have option to not create shortcuts for additional launchers

2021-07-08 Thread Alexey Semenyuk
On Thu, 8 Jul 2021 19:25:33 GMT, Andy Herrick wrote: > JDK-8269387: jpackage --add-launcher should have option to not create > shortcuts for additional launchers src/jdk.jpackage/share/classes/jdk/jpackage/internal/AppImageFile.java line 259: > 257: for (var name : names) { > 258:

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

2021-07-08 Thread Alexey Semenyuk
On Thu, 8 Jul 2021 19:25:33 GMT, Andy Herrick wrote: > JDK-8269387: jpackage --add-launcher should have option to not create > shortcuts for additional launchers test/jdk/tools/jpackage/share/AddLShortcutTest.java line 2: > 1: /* > 2: * Copyright (c) 2018, 2020, Oracle and/or its affiliates.

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

2021-07-08 Thread Alexey Semenyuk
On Thu, 8 Jul 2021 19:25:33 GMT, Andy Herrick wrote: > JDK-8269387: jpackage --add-launcher should have option to not create > shortcuts for additional launchers Review test/jdk/tools/jpackage/share/AddLShortcutTest.java line 46: > 44: */ > 45: > 46: /* Why do you need two jtreg @test-s

Re: RFR: 8266179: [macos] jpackage should specify architecture for produced pkg files

2021-04-30 Thread Alexey Semenyuk
On Fri, 30 Apr 2021 04:22:37 GMT, Alexander Matveev wrote: > jpackage should specify architecture for produced PKG files via > hostArchitectures="x86_x64 or arm64". aarch64 installer will be installable > on x64 without specifying hostArchitectures which is not correct and if > install on

Re: RFR: JDK-8266129: tools/jpackage/windows/WinInstallerIconTest.java hangs with fastdebug

2021-04-30 Thread Alexey Semenyuk
On Fri, 30 Apr 2021 20:19:25 GMT, Andy Herrick wrote: > JDK-8266129: tools/jpackage/windows/WinInstallerIconTest.java hangs with > fastdebug > > just disabling this test when vm.debug Marked as reviewed by asemenyuk (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/3827

Re: RFR: JDK-8266227: Fix help text for --mac-signing-keychain

2021-05-03 Thread Alexey Semenyuk
On Fri, 30 Apr 2021 15:25:13 GMT, Andy Herrick wrote: > JDK-8266227: Fix help text for --mac-signing-keychain Marked as reviewed by asemenyuk (Reviewer). The changes seems to be covering more than what the title of the CR states. Would it make sense to update the title accordingly?

Re: RFR: 8266179: [macos] jpackage should specify architecture for produced pkg files [v2]

2021-05-03 Thread Alexey Semenyuk
On Mon, 3 May 2021 17:24:16 GMT, Alexander Matveev wrote: >> test/jdk/tools/jpackage/macosx/HostArchPkgTest.java line 84: >> >>> 82: } >>> 83: >>> 84: public static void main(String[] args) throws Exception { >> >> Please don't use direct TKit.run() call. Use >>

Re: RFR: 8266179: [macos] jpackage should specify architecture for produced pkg files [v2]

2021-05-03 Thread Alexey Semenyuk
On Mon, 3 May 2021 17:30:15 GMT, Alexander Matveev wrote: >> test/jdk/tools/jpackage/macosx/HostArchPkgTest.java line 57: >> >>> 55: Path distributionFile = cmd.unpackedPackageDirectory() >>> 56: .toAbsolutePath() >>> 57: .getParent() >> >> Why

Re: RFR: 8266179: [macos] jpackage should specify architecture for produced pkg files [v3]

2021-05-03 Thread Alexey Semenyuk
On Mon, 3 May 2021 20:20:52 GMT, Alexey Semenyuk wrote: >> Alexander Matveev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8266179: [macos] jpackage should specify architecture for produced pkg >>

Re: RFR: 8266179: [macos] jpackage should specify architecture for produced pkg files [v3]

2021-05-03 Thread Alexey Semenyuk
On Mon, 3 May 2021 19:26:19 GMT, Alexander Matveev wrote: >> jpackage should specify architecture for produced PKG files via >> hostArchitectures="x86_x64 or arm64". aarch64 installer will be installable >> on x64 without specifying hostArchitectures which is not correct and if >> install on

Re: RFR: 8266179: [macos] jpackage should specify architecture for produced pkg files [v2]

2021-05-03 Thread Alexey Semenyuk
On Sat, 1 May 2021 04:04:17 GMT, Alexander Matveev wrote: >> jpackage should specify architecture for produced PKG files via >> hostArchitectures="x86_x64 or arm64". aarch64 installer will be installable >> on x64 without specifying hostArchitectures which is not correct and if >> install on

Re: RFR: 8266179: [macos] jpackage should specify architecture for produced pkg files [v2]

2021-05-03 Thread Alexey Semenyuk
On Sat, 1 May 2021 04:04:17 GMT, Alexander Matveev wrote: >> jpackage should specify architecture for produced PKG files via >> hostArchitectures="x86_x64 or arm64". aarch64 installer will be installable >> on x64 without specifying hostArchitectures which is not correct and if >> install on

Re: RFR: 8266179: [macos] jpackage should specify architecture for produced pkg files [v4]

2021-05-03 Thread Alexey Semenyuk
On Mon, 3 May 2021 22:52:21 GMT, Alexander Matveev wrote: >> jpackage should specify architecture for produced PKG files via >> hostArchitectures="x86_x64 or arm64". aarch64 installer will be installable >> on x64 without specifying hostArchitectures which is not correct and if >> install on

Re: RFR: JDK-8266227: Fix help text for --mac-signing-keychain

2021-05-04 Thread Alexey Semenyuk
On Fri, 30 Apr 2021 15:25:13 GMT, Andy Herrick wrote: > JDK-8266227: Fix help text for --mac-signing-keychain Marked as reviewed by asemenyuk (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/3819

Re: RFR: 8261300: jpackage: rewrite while(0)/while(false) to proper blocks [v2]

2021-02-08 Thread Alexey Semenyuk
On Mon, 8 Feb 2021 15:25:02 GMT, Aleksey Shipilev wrote: >> After JDK-8254702, SonarCloud instance complains about blocks like these: >> "Change this loop body so that it can be executed more than once." >> >> int initJvmlLauncherData(JvmlLauncherData* ptr) const { >> // Store path

Re: RFR: 8261298: LinuxPackage.c, getJvmLauncherLibPath RPM->DEB typo

2021-02-08 Thread Alexey Semenyuk
On Mon, 8 Feb 2021 08:49:17 GMT, Aleksey Shipilev wrote: > SonarCloud instance reports as new warning after JDK-8254702: > > This branch can not be reached because the condition duplicates a previous > condition in the same sequence of "if/else if" statements. > > char*

RFR: 8223188: Consider rewriting in C++ or moving to ".c" files

2021-02-08 Thread Alexey Semenyuk
Remove needless `#ifdef __cplusplus` from .cpp sources - Commit messages: - 8223188: Consider rewriting in C++ or moving to ".c" files Changes: https://git.openjdk.java.net/jdk/pull/2466/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=2466=00 Issue:

Re: RFR: 8254702: jpackage app launcher crashes on CentOS [v5]

2021-02-05 Thread Alexey Semenyuk
launcher code on other > platforms. > - `applaunchercommon`: code shared between launcher and launcher lib on Linux > and launcher code on other platforms. Alexey Semenyuk has updated the pull request incrementally with one additional commit since the last revision: 8254702: jpac

Re: RFR: 8254702: jpackage app launcher crashes on CentOS [v6]

2021-02-05 Thread Alexey Semenyuk
launcher code on other > platforms. > - `applaunchercommon`: code shared between launcher and launcher lib on Linux > and launcher code on other platforms. Alexey Semenyuk has updated the pull request incrementally with one additional commit since the last revision: 8254702: jpac

Re: RFR: 8254702: jpackage app launcher crashes on CentOS [v2]

2021-02-05 Thread Alexey Semenyuk
On Mon, 1 Feb 2021 18:35:54 GMT, Alexey Semenyuk wrote: >>> "common" was perfectly enough until this change. Unfortunately we cant just >>> drop new C sources in "common" dir because we don't want them to be >>> compiled

Re: RFR: 8223188: Removed unnecessary #ifdef __cplusplus from .cpp sources [v2]

2021-02-09 Thread Alexey Semenyuk
On Tue, 9 Feb 2021 04:03:14 GMT, Ioi Lam wrote: > The Bug Synopsis of 'Consider rewriting in C++ or moving to ".c" files' is > unclear. > I would request that the Bug Synopsis to be changed to "removed unnecessary > #ifdef __cplusplus" before integrating this PR. Bug Synopsis updated as

Re: RFR: 8223188: Removed unnecessary #ifdef __cplusplus from .cpp sources [v2]

2021-02-09 Thread Alexey Semenyuk
> Remove needless `#ifdef __cplusplus` from .cpp sources Alexey Semenyuk has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new com

Re: RFR: 8223188: Removed unnecessary #ifdef __cplusplus from .cpp sources

2021-02-09 Thread Alexey Semenyuk
On Mon, 8 Feb 2021 22:26:22 GMT, Alexey Semenyuk wrote: > Remove needless `#ifdef __cplusplus` from .cpp sources > @alexeysemenyukoracle FYI, there was no need to force-push (or even push at > all) to your branch. It doesn't matter at all what the commit message(s) of >

Integrated: 8223188: Removed unnecessary #ifdef __cplusplus from .cpp sources

2021-02-09 Thread Alexey Semenyuk
On Mon, 8 Feb 2021 22:26:22 GMT, Alexey Semenyuk wrote: > Remove needless `#ifdef __cplusplus` from .cpp sources This pull request has now been integrated. Changeset: 699a3cde Author: Alexey Semenyuk URL: https://git.openjdk.java.net/jdk/commit/699a3cde Stats: 10 lines in 2 fi

Re: RFR: 8261300: jpackage: rewrite while(0)/while(false) to proper blocks [v3]

2021-02-09 Thread Alexey Semenyuk
On Tue, 9 Feb 2021 08:58:23 GMT, Aleksey Shipilev wrote: >> After JDK-8254702, SonarCloud instance complains about blocks like these: >> "Change this loop body so that it can be executed more than once." >> >> int initJvmlLauncherData(JvmlLauncherData* ptr) const { >> // Store path

Re: RFR: 8260592: jpackage tests fail when Desktop is not supported [v2]

2021-01-29 Thread Alexey Semenyuk
On Fri, 29 Jan 2021 07:31:55 GMT, Aleksey Shipilev wrote: >> If you run x86 configuration (cross-compiled on x86_64), then many tests >> would fail with: >> >> $ CONF=linux-x86-server-fastdebug make images run-test TEST=tools/jpackage >> ... >> can not load libgnomevfs-2.so >> Exception in

Re: RFR: 8254702: jpackage app launcher crashes on CentOS [v2]

2021-02-01 Thread Alexey Semenyuk
On Mon, 1 Feb 2021 18:24:23 GMT, Erik Joelsson wrote: >> "common" was perfectly enough until this change. Unfortunately we cant just >> drop new C sources in "common" dir because we don't want them to be compiled >> with g++. That is why need new common directory (applauncherlibcommon) for C

Re: RFR: JDK-8263887: Re-create default icons

2021-03-22 Thread Alexey Semenyuk
On Mon, 22 Mar 2021 13:03:10 GMT, Andy Herrick wrote: > JDK-8263887: Re-create default icons Marked as reviewed by asemenyuk (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/3118

Re: RFR: JDK-8259926: Error in jpackage sample usage in the help text

2021-03-22 Thread Alexey Semenyuk
On Mon, 22 Mar 2021 20:40:09 GMT, Andy Herrick wrote: > JDK-8259926: Error in jpackage sample usage in the help text Marked as reviewed by asemenyuk (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/3132

Re: RFR: JDK-8264057: [redo] JDK-8248904: Add support to jpackage for the Mac App Store.

2021-03-24 Thread Alexey Semenyuk
On Wed, 24 Mar 2021 11:00:53 GMT, Andy Herrick wrote: > Restoring fix to JDK-8248904: Add support to jpackage for the Mac App Store. Marked as reviewed by asemenyuk (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/3172

Re: Production use of "jpackage.app-path" property

2021-03-24 Thread Alexey Semenyuk
Hi Zlatin, There are no plans to deprecate "jpackage.app-path" system property. It will be available in the next LTS release. - Alexey On 3/20/2021 1:25 PM, Zlatin Balevsky wrote: Hi, In my application I need to know the absolute path of the executable generated by jpackage at runtime. I

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

2021-03-24 Thread Alexey Semenyuk
Add support for --about-url, --win-update-url and --win-help-url parameters - Commit messages: - Remove unused import-s - 8220266: add support for additional metadata in add/remove programs Changes: https://git.openjdk.java.net/jdk/pull/3178/files Webrev:

RFR: 8264165: jpackage BasicTest fails after JDK-8220266: Check help text contains plaform specific parameters

2021-03-25 Thread Alexey Semenyuk
Add missing escape single quote (') and typo fix - Commit messages: - 8264165: jpackage BasicTest fails after JDK-8220266: Check help text contains plaform specific parameters Changes: https://git.openjdk.java.net/jdk/pull/3193/files Webrev:

Integrated: 8264165: jpackage BasicTest fails after JDK-8220266: Check help text contains plaform specific parameters

2021-03-25 Thread Alexey Semenyuk
On Thu, 25 Mar 2021 13:27:45 GMT, Alexey Semenyuk wrote: > Add missing escape single quote (') and typo fix This pull request has now been integrated. Changeset: 8307aa6d Author: Alexey Semenyuk URL: https://git.openjdk.java.net/jdk/commit/8307aa6d Stats: 4 lines in 4 fi

Re: RFR: JDK-8264403: [macos]: App names containing '.' characters results in an error message when launching [v3]

2021-04-01 Thread Alexey Semenyuk
On Thu, 1 Apr 2021 12:19:21 GMT, Andy Herrick wrote: >> Deriving the cfg file name is broken on mac and linux when the application >> name has a "." in it. > > Andy Herrick has updated the pull request incrementally with one additional > commit since the last revision: > > Fix trailing

RFR: 8236127: Use value of --icon CLI option to set icon for exe installers

2021-04-05 Thread Alexey Semenyuk
Use value of `--icon` jpackage CLI option to set icon for exe installers. - Commit messages: - 8236127: Use value of --icon CLI option to set icon for exe installers Changes: https://git.openjdk.java.net/jdk/pull/3347/files Webrev:

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

2021-04-06 Thread Alexey Semenyuk
> 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 AppImagePackageTest.testEmpty(true) failure - Changes: - all: ht

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

2021-03-29 Thread Alexey Semenyuk
Remove the test for failing jpackage use case scenario that we don't intent to fix - Commit messages: - 8248418: jpackage fails to extract main class and version from app module linked in external runtime Changes: https://git.openjdk.java.net/jdk/pull/3249/files Webrev:

Re: RFR: JDK-8264403: [macos]: App names containing '.' characters results in an error message when launching

2021-03-31 Thread Alexey Semenyuk
On Wed, 31 Mar 2021 19:30:10 GMT, Andy Herrick wrote: > Deriving the cfg file name is broken on mac and linux when the application > name has a "." in it. Changes requested by asemenyuk (Reviewer). src/jdk.jpackage/share/native/common/FileUtils.h line 76: > 74: > 75: // extract the name

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

2021-03-29 Thread Alexey Semenyuk
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 This pull request has now been integrated. Changeset: 128c0c97 Author:Alexey Semenyuk URL: https://git.openjdk.java.net/jdk/

Integrated: 8220266: add support for additional metadata in add/remove programs

2021-03-24 Thread Alexey Semenyuk
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. > https://bugs.o

Re: RFR: JDK-8264055: backout JDK-8248904 in order to resubmit with additional attribution.

2021-03-23 Thread Alexey Semenyuk
On Tue, 23 Mar 2021 17:09:20 GMT, Andy Herrick wrote: > We are backing out the fix to "JDK-8248904 Add support to jpackage for the > Mac App Store " in order to resubmit with added contributor attribution for > Erwin Morrhey > This is the backout. Marked as reviewed by asemenyuk (Reviewer).

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

2021-04-06 Thread Alexey Semenyuk
On Tue, 6 Apr 2021 14:45:26 GMT, Andy Herrick wrote: >> Marked as reviewed by herrick (Reviewer). > > One question - > ICON is only a valid option for creating app launcher. If creating an > installer using --app-image option it would not be a vlid option. > Does this change imply we should

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

2021-04-06 Thread Alexey Semenyuk
> 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: Make --icon option applicable in installer mode - Changes: - all: ht

Integrated: 8241716: Jpackage functionality to let users choose whether to create shortcuts

2021-03-11 Thread Alexey Semenyuk
On Mon, 8 Mar 2021 22:54:50 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: > ![shortcut_prompt](https://user-images.githubuserconte

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

2021-03-12 Thread Alexey Semenyuk
8263536: Add missing @compile tags to jpackage tests - Commit messages: - 8263536: Add missing @compile tags to jpackage tests Changes: https://git.openjdk.java.net/jdk/pull/2975/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=2975=00 Issue:

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

2021-03-12 Thread Alexey Semenyuk
> 8263536: Add missing @compile tags to jpackage tests Alexey Semenyuk has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit: 8263536: Add missing @compile tags to jpackage tests - Changes: ht

Re: RFR: 8263536: Add @compile tags to jpackage tests [v2]

2021-03-16 Thread Alexey Semenyuk
On Mon, 15 Mar 2021 20:02:02 GMT, Ioi Lam wrote: > @compile will always run javac every time you run the test Didn't know that. Thank you for explanation. Would this be OK to leave `@compile` tags? - PR: https://git.openjdk.java.net/jdk/pull/2975

Integrated: 8263536: Add @build tags to jpackage tests

2021-03-16 Thread Alexey Semenyuk
On Fri, 12 Mar 2021 17:52:30 GMT, Alexey Semenyuk wrote: > 8263536: Add @build tags to jpackage tests This pull request has now been integrated. Changeset: 422eba81 Author: Alexey Semenyuk URL: https://git.openjdk.java.net/jdk/commit/422eba81 Stats: 15 lines in 14 files chan

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

2021-03-18 Thread Alexey Semenyuk
On Tue, 16 Mar 2021 04:55:23 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: 8241716: Jpackage functionality to let users choose whether to create shortcuts [v2]

2021-03-09 Thread Alexey Semenyuk
names as placed in resource directory. > Moving source code of installer UI from main.wxs into ui.wxf, > InstallDirNotEmptyDlg.wxs and ShortcutPromptDlg.wxs files supposed to > simplify overriding of the default UI provided by jpackage. Alexey Semenyuk has updated the pull request with a new tar

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

2021-03-15 Thread Alexey Semenyuk
On Sun, 14 Mar 2021 18:22:50 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-8263545: Convert jpackage to use Stream.toList() [v2]

2021-03-15 Thread Alexey Semenyuk
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: 8263536: Add missing @compile tags to jpackage tests [v2]

2021-03-15 Thread Alexey Semenyuk
On Sat, 13 Mar 2021 08:38:16 GMT, Ioi Lam wrote: >> Alexey Semenyuk has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains one commit: >> >> 8263536: Add missing @compile tags to jpackage tests > > Change

Re: RFR: 8263536: Add @build tags to jpackage tests [v3]

2021-03-16 Thread Alexey Semenyuk
> 8263536: Add @build tags to jpackage tests Alexey Semenyuk has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the l

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

2021-03-03 Thread Alexey Semenyuk
On Wed, 3 Mar 2021 14:50:07 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. > > Andy Herrick has updated the

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

2021-03-03 Thread Alexey Semenyuk
On Wed, 3 Mar 2021 13:56:01 GMT, Andy Herrick wrote: >> test/jdk/tools/jpackage/share/jdk/jpackage/tests/NoMPathRuntimeTest.java >> line 125: >> >>> 123: .addArguments("-cvf", "junk.jar", >>> 124: "-C", tmpdir.toString(), "Hello.class") >>> 125:

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

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

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

2021-02-25 Thread Alexey Semenyuk
On Thu, 25 Feb 2021 21:36:48 GMT, Alexey Semenyuk wrote: >> …age-identifier > > Marked as reviewed by asemenyuk (Committer). Nice to have runtime packaging implemented on all supported platforms! - PR: https://git.openjdk.java.net/jdk/pull/2730

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

2021-03-04 Thread Alexey Semenyuk
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 This pull request has now been integrated. Changeset: ee09bada Author: Alexey Semenyuk URL: https://git.open

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

2021-03-02 Thread Alexey Semenyuk
On Tue, 2 Mar 2021 19:47:04 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. > > Andy Herrick has updated the

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

2021-03-02 Thread Alexey Semenyuk
On Wed, 3 Mar 2021 00:40:09 GMT, Alexey Semenyuk wrote: >> Andy Herrick has updated the pull request incrementally with one additional >> commit since the last revision: >> >> JDK-8261518: jpackage looks for main module in current dir when there is >> no mod

<    1   2   3   4   5   6   7   >