RFR: 8333116: test/jdk/tools/jpackage/share/ServiceTest.java test fails

2024-05-28 Thread Alexey Semenyuk
Add missing "--add-opens jdk.jpackage/jdk.jpackage.internal=ALL-UNNAMED" to the test descriptor - Commit messages: - Add missing "--add-opens jdk.jpackage/jdk.jpackage.internal=ALL-UNNAMED" to test desc. It fixes the following error: Changes: https://git.openjdk.org/jdk/pull/19436

Re: RFR: 8333116: test/jdk/tools/jpackage/share/ServiceTest.java test fails

2024-05-28 Thread Alexey Semenyuk
On Tue, 28 May 2024 20:13:24 GMT, Alexey Semenyuk wrote: > Add missing "--add-opens jdk.jpackage/jdk.jpackage.internal=ALL-UNNAMED" to > the test descriptor @sashamatveev please review - PR Comment: https://git.openjdk.org/jdk/pull/19436#issuecomment-2136094953

Integrated: 8333116: test/jdk/tools/jpackage/share/ServiceTest.java test fails

2024-05-28 Thread Alexey Semenyuk
On Tue, 28 May 2024 20:13:24 GMT, Alexey Semenyuk wrote: > Add missing "--add-opens jdk.jpackage/jdk.jpackage.internal=ALL-UNNAMED" to > the test descriptor This pull request has now been integrated. Changeset: 91ab088d Author: Alexey Semenyuk URL: https://git

Re: RFR: 8332110: [macos] jpackage tries to sign added files without the --mac-sign option

2024-05-30 Thread Alexey Semenyuk
On Fri, 24 May 2024 01:08:03 GMT, Alexander Matveev wrote: > This issue is reproducible with and without `--mac-sign`. jpackage will > "_ad-hoc_" sign application bundle when `--mac-sign` is not specified by > using pseudo-identity "_-_". This is why jpackage tries to sign added files > and t

Re: RFR: 8333303: Issues with DottedVersion class

2024-05-30 Thread Alexey Semenyuk
On Thu, 30 May 2024 20:10:05 GMT, Alexey Semenyuk wrote: > - Get rid of DottedVersion#greedy field. > - Add support to save the unrecognizable remainder of the version string > (required to handle Wix4 version string). > - Implement DottedVersion#equals(). > - ad

RFR: 8333303: Issues with DottedVersion class

2024-05-30 Thread Alexey Semenyuk
- Get rid of DottedVersion#greedy field. - Add support to save the unrecognizable remainder of the version string (required to handle Wix4 version string). - Implement DottedVersion#equals(). - add DottedVersion#compareComponents(DottedVersion, DottedVersion) that compares recognized component

RFR: 8333307: Don't suppress jpackage logging in tests when it is detecting packaging tools in the system

2024-05-30 Thread Alexey Semenyuk
Change jpackage tests output from: [19:16:29.586] Create: SimplePackageTest.test [19:16:29.587] [ RUN ] SimplePackageTest.test [19:16:29.663] TRACE: Bundler rpm supported [19:16:29.674] TRACE: Actions: [[initialize], [create], [unpack], [verify-install], [finalize]] to: [19:16:29.586] Create

Re: RFR: 8333307: Don't suppress jpackage logging in tests when it is detecting packaging tools in the system

2024-05-30 Thread Alexey Semenyuk
On Thu, 30 May 2024 20:26:02 GMT, Alexey Semenyuk wrote: > Change jpackage tests output from: > > > [19:16:29.586] Create: SimplePackageTest.test > [19:16:29.587] [ RUN ] SimplePackageTest.test > [19:16:29.663] TRACE: Bundler rpm supported > [19:16:29.674] TRACE:

Integrated: 8333307: Don't suppress jpackage logging in tests when it is detecting packaging tools in the system

2024-05-30 Thread Alexey Semenyuk
On Thu, 30 May 2024 20:26:02 GMT, Alexey Semenyuk wrote: > Change jpackage tests output from: > > > [19:16:29.586] Create: SimplePackageTest.test > [19:16:29.587] [ RUN ] SimplePackageTest.test > [19:16:29.663] TRACE: Bundler rpm supported > [19:16:29.674] TRACE:

Re: RFR: 8332110: [macos] jpackage tries to sign added files without the --mac-sign option [v2]

2024-05-30 Thread Alexey Semenyuk
On Thu, 30 May 2024 22:54:12 GMT, Alexander Matveev wrote: >> This issue is reproducible with and without `--mac-sign`. jpackage will >> "_ad-hoc_" sign application bundle when `--mac-sign` is not specified by >> using pseudo-identity "_-_". This is why jpackage tries to sign added files >> a

Integrated: 8333303: Issues with DottedVersion class

2024-05-30 Thread Alexey Semenyuk
On Thu, 30 May 2024 20:10:05 GMT, Alexey Semenyuk wrote: > - Get rid of DottedVersion#greedy field. > - Add support to save the unrecognizable remainder of the version string > (required to handle Wix4 version string). > - Implement DottedVersion#equals(). > - ad

Re: RFR: 8331977: Crash: SIGSEGV in dlerror()

2024-05-31 Thread Alexey Semenyuk
On Fri, 31 May 2024 14:05:07 GMT, Alexey Semenyuk wrote: > Fix MainClassTest class to use HelloApp.AppOutputVerifier class to run app > launcher instead of raw Executor. This makes MainClassTest test run app > launchers with retries. This change addresses the primary issue.

RFR: 8331977: Crash: SIGSEGV in dlerror()

2024-05-31 Thread Alexey Semenyuk
Fix MainClassTest class to use HelloApp.AppOutputVerifier class to run app launcher instead of raw Executor. This makes MainClassTest test run app launchers with retries. This change addresses the primary issue. Fix inconsistencies in HelloApp.AppOutputVerifier class. It used to provide API all

Integrated: 8331977: Crash: SIGSEGV in dlerror()

2024-06-01 Thread Alexey Semenyuk
On Fri, 31 May 2024 14:05:07 GMT, Alexey Semenyuk wrote: > Fix MainClassTest class to use HelloApp.AppOutputVerifier class to run app > launcher instead of raw Executor. This makes MainClassTest test run app > launchers with retries. This change addresses the primary issue.

Re: RFR: 8331977: Crash: SIGSEGV in dlerror()

2024-06-03 Thread Alexey Semenyuk
On Mon, 3 Jun 2024 08:37:50 GMT, David Holmes wrote: >> Fix MainClassTest class to use HelloApp.AppOutputVerifier class to run app >> launcher instead of raw Executor. This makes MainClassTest test run app >> launchers with retries. This change addresses the primary issue. >> >> Fix inconsiste

RFR: 8333452: Make logging in jdk.jpackage.internal.OverridableResource class configurable

2024-06-03 Thread Alexey Semenyuk
8333452: Make logging in jdk.jpackage.internal.OverridableResource class configurable - Commit messages: - 8333452: Make logging in jdk.jpackage.internal.OverridableResource class configurable Changes: https://git.openjdk.org/jdk/pull/19532/files Webrev: https://webrevs.openjdk.

Re: RFR: 8333452: Make logging in jdk.jpackage.internal.OverridableResource class configurable

2024-06-03 Thread Alexey Semenyuk
On Mon, 3 Jun 2024 20:29:47 GMT, Alexey Semenyuk wrote: > 8333452: Make logging in jdk.jpackage.internal.OverridableResource class > configurable @sashamatveev, please review - PR Comment: https://git.openjdk.org/jdk/pull/19532#issuecomment-2146097131

Withdrawn: 8333452: Make logging in jdk.jpackage.internal.OverridableResource class configurable

2024-06-03 Thread Alexey Semenyuk
On Mon, 3 Jun 2024 20:29:47 GMT, Alexey Semenyuk wrote: > 8333452: Make logging in jdk.jpackage.internal.OverridableResource class > configurable This pull request has been closed without being integrated. - PR: https://git.openjdk.org/jdk/pull/19532

Re: RFR: 8333452: Make logging in jdk.jpackage.internal.OverridableResource class configurable

2024-06-03 Thread Alexey Semenyuk
On Mon, 3 Jun 2024 21:53:06 GMT, Alexander Matveev wrote: > This added functionality is not used. It is completely unclear from PR and/or > issue description why we need this and how it will be used in the future? It was supposed to be used in [WiX4 project](https://bugs.openjdk.org/browse/JDK

Re: RFR: 8319457: Update jpackage to support WiX Toolset 4 on Windows

2024-06-03 Thread Alexey Semenyuk
On Tue, 4 Jun 2024 01:24:15 GMT, Alexander Matveev wrote: > Is there plan to add support for WiX 5? This patch supports WiX5 as well. There is not much difference between WiX4 and WiX5 from jpackage perspective. I'll update the description - PR Comment: https://git.openjdk.org/jdk

Re: RFR: 8319457: Update jpackage to support WiX Toolset 4 on Windows

2024-06-04 Thread Alexey Semenyuk
On Tue, 4 Jun 2024 02:38:51 GMT, Alexander Matveev wrote: > I assume WiX5 will just work if installed instead of WiX4? Correct. > Do you think it will make sense to introduce Wix5 ToolsetType in case if we > need to do something different between 4 and 5? They claim WiX5 is backward compatibl

Re: RFR: 8331977: Crash: SIGSEGV in dlerror()

2024-06-04 Thread Alexey Semenyuk
On Fri, 31 May 2024 14:05:07 GMT, Alexey Semenyuk wrote: > Fix MainClassTest class to use HelloApp.AppOutputVerifier class to run app > launcher instead of raw Executor. This makes MainClassTest test run app > launchers with retries. This change addresses the primary issue.

Re: RFR: 8331977: Crash: SIGSEGV in dlerror()

2024-06-04 Thread Alexey Semenyuk
On Tue, 4 Jun 2024 06:43:12 GMT, Alan Bateman wrote: > If I read JDK-8269403 correctly then there is an issue somewhere that hasn't > been diagnosed. Correct. > A workaround has been put in to "re-run" when there is a crash, thus hiding > the issue. Correct. > Are there follow-up issues cre

Re: RFR: 8319457: Update jpackage to support WiX Toolset 4 on Windows [v3]

2024-06-04 Thread Alexey Semenyuk
On Tue, 4 Jun 2024 01:07:15 GMT, Alexander Matveev wrote: >> Alexey Semenyuk has updated the pull request incrementally with one >> additional commit since the last revision: >> >> EOLs restored > > src/jdk.jpackage/windows/classes/jdk/jpackage/internal/resour

Re: RFR: 8319457: Update jpackage to support WiX v4 and v5 on Windows [v4]

2024-06-05 Thread Alexey Semenyuk
On Wed, 5 Jun 2024 04:20:17 GMT, Alexander Matveev wrote: >> Alexey Semenyuk has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - WixSourceConverter#applyTo should do what >> OverridableResource#saveToFile()

Re: RFR: 8319457: Update jpackage to support WiX v4 and v5 on Windows [v4]

2024-06-05 Thread Alexey Semenyuk
On Tue, 4 Jun 2024 01:20:15 GMT, Alexander Matveev wrote: > Do we need to handle default case for unknown wix type? In same cases you > have default -> throw new IllegalArgumentException(); and in some you do not > have. Good catch. I'll update the code to make `switch(wixVersion) ...` expres

Re: RFR: 8319457: Update jpackage to support WiX v4 and v5 on Windows [v4]

2024-06-05 Thread Alexey Semenyuk
On Wed, 5 Jun 2024 03:54:46 GMT, Alexander Matveev wrote: >> Alexey Semenyuk has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - WixSourceConverter#applyTo should do what >> OverridableResource#saveToFile()

Re: RFR: 8333827: JDK 23 RDP1 L10n resource files update [v3]

2024-06-11 Thread Alexey Semenyuk
On Tue, 11 Jun 2024 19:28:27 GMT, Damon Nguyen wrote: >> This issue is responsible for updating the translations of all the >> localize(able) resources in the JDK. Primarily, the changes between JDK 22 >> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated. >> >> The tra

Re: RFR: 8325525: Create jtreg test case for JDK-8325203

2024-06-20 Thread Alexey Semenyuk
On Tue, 4 Jun 2024 07:13:15 GMT, Vanitha B P wrote: > Created jtreg test case for > [JDK-8325203](https://bugs.openjdk.org/browse/JDK-8325203) issue. > > The JpackageTest created tests that the child process started from the app > launched by jpackage launcher is not automatically terminated w

Re: RFR: 8325525: Create jtreg test case for JDK-8325203 [v2]

2024-07-03 Thread Alexey Semenyuk
On Wed, 3 Jul 2024 04:11:40 GMT, Vanitha B P wrote: >> Created jtreg test case for >> [JDK-8325203](https://bugs.openjdk.org/browse/JDK-8325203) issue. >> >> The JpackageTest created tests that the child process started from the app >> launched by jpackage launcher is not automatically termina

Re: RFR: 8325525: Create jtreg test case for JDK-8325203 [v2]

2024-07-03 Thread Alexey Semenyuk
On Wed, 3 Jul 2024 04:11:40 GMT, Vanitha B P wrote: >> Created jtreg test case for >> [JDK-8325203](https://bugs.openjdk.org/browse/JDK-8325203) issue. >> >> The JpackageTest created tests that the child process started from the app >> launched by jpackage launcher is not automatically termina

Re: RFR: 8325525: Create jtreg test case for JDK-8325203 [v3]

2024-07-09 Thread Alexey Semenyuk
On Tue, 9 Jul 2024 10:49:06 GMT, Vanitha B P wrote: >> Created jtreg test case for >> [JDK-8325203](https://bugs.openjdk.org/browse/JDK-8325203) issue. >> >> The JpackageTest created tests that the child process started from the app >> launched by jpackage launcher is not automatically termina

Re: RFR: 8325525: Create jtreg test case for JDK-8325203 [v4]

2024-07-10 Thread Alexey Semenyuk
On Tue, 9 Jul 2024 16:12:58 GMT, Vanitha B P wrote: >> Created jtreg test case for >> [JDK-8325203](https://bugs.openjdk.org/browse/JDK-8325203) issue. >> >> The JpackageTest created tests that the child process started from the app >> launched by jpackage launcher is not automatically termina

Re: RFR: 8336315: tools/jpackage/windows/WinChildProcessTest.java Failed: Check is calculator process is alive

2024-07-24 Thread Alexey Semenyuk
On Wed, 24 Jul 2024 12:20:05 GMT, Vanitha B P wrote: > tools/jpackage/windows/WinChildProcessTest.java was failing intermittently, > fixed the issue and changes are tested. Changes requested by asemenyuk (Reviewer). @sashamatveev please review test/jdk/tools/jpackage/apps/ChildProcessAppLaunc

Re: RFR: 8336315: tools/jpackage/windows/WinChildProcessTest.java Failed: Check is calculator process is alive [v2]

2024-07-25 Thread Alexey Semenyuk
On Thu, 25 Jul 2024 08:40:44 GMT, Vanitha B P wrote: >> tools/jpackage/windows/WinChildProcessTest.java was failing intermittently, >> fixed the issue and changes are tested. > > Vanitha B P has updated the pull request incrementally with one additional > commit since the last revision: > >

Re: jpackage nits

2024-07-29 Thread Alexey Semenyuk
Hi Bernd, The issue with the infinite directory tree you observe looks like a duplicate of [1]. You can add "--verbose" option to jpackage command line to get more details on what is going on with jpackage. There is no way to configure the list of directories where native launcher is looki

Re: RFR: 8293579: tools/jpackage/share/jdk/jpackage/tests/UnicodeArgsTest.java fails on Japanese Windows platform

2022-09-28 Thread Alexey Semenyuk
On Wed, 28 Sep 2022 09:40:56 GMT, KIRIYAMA Takuya wrote: > It is difficult to add all the encodings, so I will fix for Japanese and > would like people who need other languages to do the same. It would be good to have a comment in the source code describing that this works only for Japanese en

Re: RFR: 8293579: tools/jpackage/share/jdk/jpackage/tests/UnicodeArgsTest.java fails on Japanese Windows platform [v2]

2022-09-29 Thread Alexey Semenyuk
On Thu, 29 Sep 2022 08:04:07 GMT, KIRIYAMA Takuya wrote: >> test/jdk/tools/jpackage/share/jdk/jpackage/tests/UnicodeArgsTest.java line >> 55: >> >>> 53: String encoding = System.getProperty("native.encoding"); >>> 54: switch (encoding) { >>> 55: default: >> >> What I me

Re: RFR: 8293579: tools/jpackage/share/jdk/jpackage/tests/UnicodeArgsTest.java fails on Japanese Windows platform [v2]

2022-09-29 Thread Alexey Semenyuk
On Thu, 29 Sep 2022 18:17:18 GMT, Naoto Sato wrote: >> I agree skipping the test is not the best solution. Leaving a hint in the >> comments on how to fix the issue looks better alternative. > > Well, what concerns me with the current fix is that it is simply patching > only the case for the Ja

Re: RFR: 8293579: tools/jpackage/share/jdk/jpackage/tests/UnicodeArgsTest.java fails on Japanese Windows platform [v2]

2022-10-03 Thread Alexey Semenyuk
On Wed, 28 Sep 2022 09:45:32 GMT, KIRIYAMA Takuya wrote: >> Could you please review the JDK-8293579 bug fixes? >> >> tools/jpackage/share/jdk/jpackage/tests/UnicodeArgsTest.java attempts to >> give >> the launcher the character which is encoded by Windows API >> WideCharToMultiByte() >> from

Re: RFR: 8293579: tools/jpackage/share/jdk/jpackage/tests/UnicodeArgsTest.java fails on Japanese Windows platform [v2]

2022-10-03 Thread Alexey Semenyuk
On Mon, 3 Oct 2022 09:39:08 GMT, KIRIYAMA Takuya wrote: >> I don't think it will be many practical setups where this test fails. This >> test has been around for more than 2 years and this is the first accident. > > At least, the results in different encodings are the same as before the fix. > H

Re: RFR: 8295231: Move all linking of native libraries to make [v6]

2022-10-17 Thread Alexey Semenyuk
On Mon, 17 Oct 2022 14:41:06 GMT, Julian Waters wrote: >> Some external libraries required by native code are linked via linker >> comments embedded in pragmas. Searching for which libraries are linked can >> then become frustrating and confusing since they may be included in an >> obscure pla

Re: RFR: 8295231: Move all linking of native libraries to make [v6]

2022-10-17 Thread Alexey Semenyuk
On Mon, 17 Oct 2022 17:37:11 GMT, Erik Joelsson wrote: > I believe this is part of the effort for > https://bugs.openjdk.org/browse/JDK-8288293. Agree. I'd prefer to have a different description of the bug though to make it clear that this is necessary for decoupling a compiler and an OS. ---

Re: RFR: 8296154: [macos] Change /Applications to Applications in DMG image

2022-11-04 Thread Alexey Semenyuk
On Fri, 4 Nov 2022 23:26:00 GMT, Alexander Matveev wrote: > Changed names of symbolic links for drag and drop location in DMG image. See > attached images in JBS. > "/Applications" -> "Applications" for app DMG image. > "/Library/Java/JavaVirtualMachines" -> "JavaVirtualMachines" for runtime DMG

Re: RFR: 8296154: [macos] Change "/Applications" to "Applications" in DMG image [v2]

2022-11-07 Thread Alexey Semenyuk
On Sat, 5 Nov 2022 01:08:11 GMT, Alexander Matveev wrote: >> Changed names of symbolic links for drag and drop location in DMG image. See >> attached images in JBS. >> "/Applications" -> "Applications" for app DMG image. >> "/Library/Java/JavaVirtualMachines" -> "JavaVirtualMachines" for runtime

Re: RFR: 8296156: [macos] Resize DMG windows and background to fit additional DMG contents

2022-11-09 Thread Alexey Semenyuk
On Wed, 9 Nov 2022 20:02:05 GMT, Alexander Matveev wrote: > Additional DMG content will be arranged in raws with 3 items per raw. DMG > windows will be resized to show one or two raws without any scrolling > required. To see any additional content after two raws (7 or more items) user > will n

RFR: 8296489: tools/jpackage/windows/WinL10nTest.java fails with timeout

2022-12-05 Thread Alexey Semenyuk
Simply increase timeout limit to make test pass on slower/loaded hosts - Commit messages: - 8296489: tools/jpackage/windows/WinL10nTest.java fails with timeout Changes: https://git.openjdk.org/jdk/pull/11522/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=11522&range=00 Is

RFR: 8293453: tools/jpackage/share/AddLShortcutTest.java "Failed: Check the number of mismatched pixels [1024] of [1024] is < [0.100000] threshold"

2022-12-05 Thread Alexey Semenyuk
- throw an exception if icon swap fails with the corresponding l10n message; - add retries to the jpackage test routine that extracts an icon from a launcher - Commit messages: - 8293453: tools/jpackage/share/AddLShortcutTest.java "Failed: Check the number of mismatched pixels [1024

Integrated: 8293453: tools/jpackage/share/AddLShortcutTest.java "Failed: Check the number of mismatched pixels [1024] of [1024] is < [0.100000] threshold"

2022-12-05 Thread Alexey Semenyuk
On Mon, 5 Dec 2022 22:14:50 GMT, Alexey Semenyuk wrote: > - throw an exception if icon swap fails with the corresponding l10n message; > - add retries to the jpackage test routine that extracts an icon from a > launcher This pull request has now been integrated. Changeset: 884b9a

Integrated: 8296489: tools/jpackage/windows/WinL10nTest.java fails with timeout

2022-12-05 Thread Alexey Semenyuk
On Mon, 5 Dec 2022 22:20:19 GMT, Alexey Semenyuk wrote: > Simply increase timeout limit to make test pass on slower/loaded hosts This pull request has now been integrated. Changeset: 8d8a28ff Author: Alexey Semenyuk URL: https://git.openjdk.org/jdk/com

Re: [jdk20] RFR: 8298488: [macos13] tools/jpackage tests failing with "Exit code: 137" on macOS

2022-12-12 Thread Alexey Semenyuk
On Mon, 12 Dec 2022 21:23:50 GMT, Alexander Matveev wrote: > This issue is similar to JDK-8277493. Looks like macOS 13 cannot execute > unsigned code and code should be at least ad hoc signed. Same as for macOS > aarch64. Fixed by enabling ad hoc signing for macOS platform for both x64 and >

RFR: 8299278: tools/jpackage/share/AddLauncherTest.java#id1 failed AddLauncherTest.bug8230933

2023-01-05 Thread Alexey Semenyuk
8299278: tools/jpackage/share/AddLauncherTest.java#id1 failed AddLauncherTest.bug8230933 - Commit messages: - Comment fixed - 8299278: tools/jpackage/share/AddLauncherTest.java#id1 failed AddLauncherTest.bug8230933 Changes: https://git.openjdk.org/jdk/pull/11868/files Webrev: ht

Re: jpackageapplauncher linker arguments for osx

2023-01-09 Thread Alexey Semenyuk
Hi David, The request to adjust osx linker command lines looks reasonable. Please go ahead with a pull request. - Alexey On 1/9/2023 1:21 AM, David Holmes wrote: On 8/01/2023 8:39 pm, David Schumann wrote: Hello, I'm not 100% sure if this list is the correct one for this topic, feel free

RFR: 8298735: Some tools/jpackage/windows/* tests fails with jtreg test timeout

2023-01-09 Thread Alexey Semenyuk
Increase failed test timeouts. Simple tests got an x1.5 increase and parametrized tests got an x2 increase - Commit messages: - 8298735: Some tools/jpackage/windows/* tests fails with jtreg test timeout Changes: https://git.openjdk.org/jdk/pull/11914/files Webrev: https://webrevs.o

Integrated: 8298735: Some tools/jpackage/windows/* tests fails with jtreg test timeout

2023-01-10 Thread Alexey Semenyuk
On Mon, 9 Jan 2023 22:53:18 GMT, Alexey Semenyuk wrote: > Increase failed test timeouts. > Simple tests got an x1.5 increase and parametrized tests got an x2 increase This pull request has now been integrated. Changeset: 3c99e786 Author:Alexey Semenyuk URL: https://git.openj

Integrated: 8299278: tools/jpackage/share/AddLauncherTest.java#id1 failed AddLauncherTest.bug8230933

2023-01-10 Thread Alexey Semenyuk
On Thu, 5 Jan 2023 20:10:45 GMT, Alexey Semenyuk wrote: > 8299278: tools/jpackage/share/AddLauncherTest.java#id1 failed > AddLauncherTest.bug8230933 This pull request has now been integrated. Changeset: c595f965 Author:Alexey Semenyuk URL: https://git.openjdk.org/jdk/

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Alexey Semenyuk
On Tue, 24 Jan 2023 22:33:24 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v3]

2023-01-24 Thread Alexey Semenyuk
On Tue, 24 Jan 2023 22:33:24 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: [jdk20] RFR: 8300719: JDK 20 RDP2 L10n resource files update [v5]

2023-01-24 Thread Alexey Semenyuk
On Tue, 24 Jan 2023 23:56:23 GMT, Damon Nguyen wrote: >> Open l10n drop. Files have been updated with translated versions. Whitespace >> tool has been ran on files. >> All tests passed > > Damon Nguyen has updated the pull request incrementally with one additional > commit since the last revisi

Re: RFR: 8303227: JniObjWithEnv should be NullablePointer compliant

2023-02-28 Thread Alexey Semenyuk
On Mon, 27 Feb 2023 08:19:53 GMT, Julian Waters wrote: > JniObjWithEnv is a struct that is commonly managed by std::unique_ptr. > Although it can support managing objects that are not raw pointers, any such > objects have to be > [NullablePointers](https://en.cppreference.com/w/cpp/named_req/N

Re: RFR: 8299779: Test tools/jpackage/share/jdk/jpackage/tests/MainClassTest.java timed out

2023-03-10 Thread Alexey Semenyuk
On Tue, 7 Mar 2023 00:56:02 GMT, Alexander Matveev wrote: > - Fixed by increasing test timeout. Fix verified on host which reproduced > issue. Marked as reviewed by asemenyuk (Reviewer). - PR: https://git.openjdk.org/jdk/pull/12896

Re: RFR: 8299779: Test tools/jpackage/share/jdk/jpackage/tests/MainClassTest.java timed out

2023-03-10 Thread Alexey Semenyuk
On Tue, 7 Mar 2023 00:56:02 GMT, Alexander Matveev wrote: > - Fixed by increasing test timeout. Fix verified on host which reproduced > issue. Marked as reviewed by asemenyuk (Reviewer). - PR: https://git.openjdk.org/jdk/pull/12896

Re: RFR: JDK-8304063: tools/jpackage/share/AppLauncherEnvTest.java fails when checking LD_LIBRARY_PATH

2023-03-15 Thread Alexey Semenyuk
On Wed, 15 Mar 2023 11:46:22 GMT, Matthias Baesken wrote: > The test fails on Alpine Linux 3.17, when checking the environment variable > LD_LIBRARY_PATH; looks like the actual env variable is much longer than the > test expects. It turned out that at least on Linux (probably also on other OS

Re: RFR: JDK-8304063: tools/jpackage/share/AppLauncherEnvTest.java fails when checking LD_LIBRARY_PATH [v2]

2023-03-16 Thread Alexey Semenyuk
On Thu, 16 Mar 2023 09:08:08 GMT, Matthias Baesken wrote: >> The test fails on Alpine Linux 3.17, when checking the environment variable >> LD_LIBRARY_PATH; looks like the actual env variable is much longer than the >> test expects. It turned out that at least on Linux (probably also on other

Re: RFR: JDK-8304063: tools/jpackage/share/AppLauncherEnvTest.java fails when checking LD_LIBRARY_PATH [v2]

2023-03-16 Thread Alexey Semenyuk
On Thu, 16 Mar 2023 09:03:36 GMT, Matthias Baesken wrote: >> test/jdk/tools/jpackage/share/AppLauncherEnvTest.java line 93: >> >>> 91: } else { >>> 92: TKit.assertEquals(expectedEnvVarValue, actualEnvVarValue, >>> msg); >>> 93: } >> >> I'd keep the check as a single

Re: RFR: 8298995: tools/jpackage/share/AddLauncherTest.java#id1 failed "AddLauncherTest.test; checks=53"

2023-03-17 Thread Alexey Semenyuk
On Mon, 13 Mar 2023 16:50:51 GMT, Alexander Matveev wrote: > This is similar to many other issues we had with `hdiutil` failures. Fixed in > similar way by repeating `hdiutil` command. Cannot confirm that it will > resolve this issue, but similar issues got resolved by repeating command. > Al

RFR: 8294806: jpackaged-app ignores splash screen from jar file

2023-04-10 Thread Alexey Semenyuk
When `--main-jar` is specified and the name of the main class is picked from the jar's manifest, set `app.jarfile` property in launcher cfg file instead of `app.mainclass` and `app.classpath` properties to make app launcher run the app from the jar and not as a class from the classpath. ---

Re: RFR: 8294806: jpackaged-app ignores splash screen from jar file

2023-04-10 Thread Alexey Semenyuk
On Mon, 10 Apr 2023 21:16:19 GMT, Alexey Semenyuk wrote: > When `--main-jar` is specified and the name of the main class is picked from > the jar's manifest, set `app.jarfile` property in launcher cfg file instead > of `app.mainclass` and `app.classpath` properties to make ap

Integrated: 8294806: jpackaged-app ignores splash screen from jar file

2023-04-11 Thread Alexey Semenyuk
On Mon, 10 Apr 2023 21:16:19 GMT, Alexey Semenyuk wrote: > When `--main-jar` is specified and the name of the main class is picked from > the jar's manifest, set `app.jarfile` property in launcher cfg file instead > of `app.mainclass` and `app.classpath` properties to make ap

Re: RFR: 8304914: Use OperatingSystem, Architecture, and Version in jpackage [v2]

2023-05-12 Thread Alexey Semenyuk
On Fri, 5 May 2023 14:12:21 GMT, Roger Riggs wrote: >> Refactor the Platform class in jdk.jpackage to use the internal >> OperatingSystem, Architecture, and Version classes. >> The OperatingSystem.isXXX() and Architecture.isYYY() methods replace >> comparisons in the Platform class. >> The chec

Re: RFR: 8304914: Use OperatingSystem, Architecture, and Version in jpackage [v3]

2023-05-18 Thread Alexey Semenyuk
On Thu, 18 May 2023 21:25:26 GMT, Alexander Matveev wrote: > did we test jpackage on AIX system? I don't think so. What is the native packaging on this platform? - PR Comment: https://git.openjdk.org/jdk/pull/13586#issuecomment-1553675715

RFR: 8287401: jpackage tests failing on Windows due to powershell issue

2022-06-15 Thread Alexey Semenyuk
Run a script extracting icons from executables as PowerShell command line rather than a script - Commit messages: - 8287401: jpackage tests failing on Windows due to powershell issue Changes: https://git.openjdk.org/jdk19/pull/26/files Webrev: https://webrevs.openjdk.org/?repo=jdk

Integrated: 8287401: jpackage tests failing on Windows due to powershell issue

2022-06-16 Thread Alexey Semenyuk
On Thu, 16 Jun 2022 05:15:27 GMT, Alexey Semenyuk wrote: > Run a script extracting icons from executables as PowerShell command line > rather than a script This pull request has now been integrated. Changeset: c254c9d0 Author:Alexey Semenyuk URL: https://git.openjdk.org

Re: RFR: 8287971: Throw exception for missing values in .jpackage.xml [v2]

2022-06-16 Thread Alexey Semenyuk
On Thu, 16 Jun 2022 18:21:02 GMT, Alexander Matveev wrote: >> - Error will be thrown if app image is generated with another JDK version or >> has malformed .jpackage.xml. >> - Re-fixed as Alexey suggested in >> https://urldefense.com/v3/__https://github.com/openjdk/jdk/pull/9098__;!!ACWV5N9M2

Re: RFR: 8287971: Throw exception for missing values in .jpackage.xml [v2]

2022-06-16 Thread Alexey Semenyuk
On Thu, 16 Jun 2022 18:14:45 GMT, Alexander Matveev wrote: >> test/jdk/tools/jpackage/helpers/jdk/jpackage/test/TKit.java line 272: >> >>> 270: )); >>> 271: } >>> 272: >> >> 1. This is jpackage specific, so it should belong to JPackageCommand class. >> 2. Please use proper XML

Re: RFR: 8287971: Throw exception for missing values in .jpackage.xml [v3]

2022-06-17 Thread Alexey Semenyuk
On Fri, 17 Jun 2022 04:07:47 GMT, Alexander Matveev wrote: >> - Error will be thrown if app image is generated with another JDK version or >> has malformed .jpackage.xml. >> - Re-fixed as Alexey suggested in >> https://urldefense.com/v3/__https://github.com/openjdk/jdk/pull/9098__;!!ACWV5N9M2

Re: RFR: 8287971: Throw exception for missing values in .jpackage.xml [v4]

2022-06-17 Thread Alexey Semenyuk
On Fri, 17 Jun 2022 21:49:55 GMT, Alexander Matveev wrote: >> - Error will be thrown if app image is generated with another JDK version or >> has malformed .jpackage.xml. >> - Re-fixed as Alexey suggested in >> https://urldefense.com/v3/__https://github.com/openjdk/jdk/pull/9098__;!!ACWV5N9M2

Re: RFR: 8288961: jpackage: test MSI installation fix

2022-06-27 Thread Alexey Semenyuk
On Wed, 22 Jun 2022 11:53:39 GMT, Alex Kasko wrote: > Please review this minor fix to the jpackage test suite. > > When `INSTALL` and `UNINSTALL` actions are enabled for jpackage test suite > runs (disabled by default), installation of `EXE` and `MSI` packages is > performed. `EXE` package is

Re: RFR: 8288013: jpackage: test utility Windows registry enhancement

2022-06-27 Thread Alexey Semenyuk
On Wed, 8 Jun 2022 11:46:42 GMT, Alex Kasko wrote: > In jpackage test suite there is a test utility > WindowsHelper.queryRegistryValue . It is used to verify changes to Windows > registry that can be done by installer packages. It spawns `reg.exe query` > process and parses its output. This pa

Re: RFR: 8288838: jpackage: file association additional arguments

2022-06-27 Thread Alexey Semenyuk
On Tue, 21 Jun 2022 09:30:30 GMT, Alex Kasko wrote: > jpackage implementation of file association on Windows currently passes a > selected filename as an only argument to associated executable. > > It is proposed to introduce additional option in file association property > file to allow optio

Re: RFR: 8288838: jpackage: file association additional arguments

2022-06-27 Thread Alexey Semenyuk
On Tue, 21 Jun 2022 09:30:30 GMT, Alex Kasko wrote: > jpackage implementation of file association on Windows currently passes a > selected filename as an only argument to associated executable. > > It is proposed to introduce additional option in file association property > file to allow optio

Re: RFR: 8288838: jpackage: file association additional arguments

2022-06-28 Thread Alexey Semenyuk
On Tue, 28 Jun 2022 21:03:28 GMT, Alex Kasko wrote: > I assume we will need an additional similar property for a verb label > displayed in a context menu. Agree. At least we need to provide l10n for it. > Can we change the existing default behaviour? Should we? It won't break > existing usage

Re: RFR: 8288838: jpackage: file association additional arguments

2022-06-29 Thread Alexey Semenyuk
On Wed, 29 Jun 2022 12:30:05 GMT, Alex Kasko wrote: > For runtime installers it would be great to have a custom verb label, to be > able to use something like: "Open with VendorName OpenJDK". I intend to > prototype this and file in a separate PR Sounds good. > %* contains all the arguments b

Re: [jdk19] RFR: 8289030: [macos] app image signature invalid when creating DMG or PKG

2022-07-01 Thread Alexey Semenyuk
On Wed, 29 Jun 2022 03:03:15 GMT, Alexander Matveev wrote: > Fixed 3 issues which made signature invalid: > - We should not remove .jpackage.xml from signed app image when creating DMG > or PKG otherwise it invalidates signature. > - .package should be created when app image is generated, so th

Re: [jdk19] RFR: 8289030: [macos] app image signature invalid when creating DMG or PKG

2022-07-01 Thread Alexey Semenyuk
On Wed, 29 Jun 2022 03:03:15 GMT, Alexander Matveev wrote: > Fixed 3 issues which made signature invalid: > - We should not remove .jpackage.xml from signed app image when creating DMG > or PKG otherwise it invalidates signature. > - .package should be created when app image is generated, so th

Re: RFR: 8288838: jpackage: file association additional arguments [v4]

2022-07-05 Thread Alexey Semenyuk
On Mon, 4 Jul 2022 23:38:08 GMT, Alex Kasko wrote: >> jpackage implementation of file association on Windows currently passes a >> selected filename as an only argument to associated executable. >> >> It is proposed to introduce additional option in file association property >> file to allow o

Re: RFR: 8288838: jpackage: file association additional arguments [v4]

2022-07-07 Thread Alexey Semenyuk
On Mon, 4 Jul 2022 23:38:08 GMT, Alex Kasko wrote: >> jpackage implementation of file association on Windows currently passes a >> selected filename as an only argument to associated executable. >> >> It is proposed to introduce additional option in file association property >> file to allow o

Re: RFR: 8288838: jpackage: file association additional arguments [v3]

2022-07-07 Thread Alexey Semenyuk
On Wed, 29 Jun 2022 12:29:33 GMT, Alex Kasko wrote: >> jpackage implementation of file association on Windows currently passes a >> selected filename as an only argument to associated executable. >> >> It is proposed to introduce additional option in file association property >> file to allow

Re: [jdk19] RFR: 8289030: [macos] app image signature invalid when creating DMG or PKG [v2]

2022-07-08 Thread Alexey Semenyuk
On Thu, 7 Jul 2022 19:30:07 GMT, Alexander Matveev wrote: >> src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/MacBaseInstallerBundler.java >> line 171: >> >>> 169: >>> Files.deleteIfExists(AppImageFile.getPathInAppImage(appDir)); >>> 170: } >>> 171: >> >> I think ther

Re: [jdk19] RFR: 8289030: [macos] app image signature invalid when creating DMG or PKG [v2]

2022-07-08 Thread Alexey Semenyuk
On Thu, 7 Jul 2022 07:27:56 GMT, Alexander Matveev wrote: >> Fixed 3 issues which made signature invalid: >> - We should not remove .jpackage.xml from signed app image when creating DMG >> or PKG otherwise it invalidates signature. >> - .package should be created when app image is generated, so

Re: [jdk19] RFR: 8289030: [macos] app image signature invalid when creating DMG or PKG [v3]

2022-07-08 Thread Alexey Semenyuk
On Fri, 8 Jul 2022 00:14:27 GMT, Alexander Matveev wrote: >> Fixed 3 issues which made signature invalid: >> - We should not remove .jpackage.xml from signed app image when creating DMG >> or PKG otherwise it invalidates signature. >> - .package should be created when app image is generated, so

RFR: 8283707: Support version format on Windows

2022-07-16 Thread Alexey Semenyuk
8283707: Support version format on Windows - Commit messages: - Create separate tests for the new feature. - Trailing whitespaces fixed - Better test coverage - Bugfixes after manual testing - 8283707: Support version format on Windows Changes: https://git.openjdk.org/jdk/pull

RFR: 8290398: jpackage exe instellers are not installed in jtreg tests

2022-07-16 Thread Alexey Semenyuk
8290398: jpackage exe instellers are not installed in jtreg tests - Commit messages: - 8290398: jpackage exe instellers are not installed in jtreg tests Changes: https://git.openjdk.org/jdk/pull/9530/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9530&range=00 Issue: http

RFR: 8290400: Must run exe installers in jpackage jtreg tests without UI

2022-07-16 Thread Alexey Semenyuk
8290400: Must run exe installers in jpackage jtreg tests without UI - Commit messages: - 8290400: Must run exe installers in jpackage jtreg tests without UI Changes: https://git.openjdk.org/jdk/pull/9531/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9531&range=00 Issue:

RFR: 8290402: jpackage exe uninstallers don't return correct exit code in case of failure

2022-07-16 Thread Alexey Semenyuk
8290402: jpackage exe uninstallers don't return correct exit code in case of failure - Commit messages: - 8290402: jpackage exe uninstallers don't return correct exit code in case of failure Changes: https://git.openjdk.org/jdk/pull/9532/files Webrev: https://webrevs.openjdk.org/

Re: RFR: 8283707: Support version format on Windows [v2]

2022-07-16 Thread Alexey Semenyuk
> 8283707: Support version format on Windows Alexey Semenyuk has updated the pull request incrementally with one additional commit since the last revision: Comments fixed and better naming in the code. - Changes: - all: https://git.openjdk.org/jdk/pull/9507/files -

Re: RFR: 8290402: jpackage exe uninstallers don't return correct exit code in case of failure

2022-07-18 Thread Alexey Semenyuk
On Mon, 18 Jul 2022 22:01:09 GMT, Alexander Matveev wrote: >> 8290402: jpackage exe uninstallers don't return correct exit code in case of >> failure > > src/jdk.jpackage/windows/native/msiwrapper/MsiWrapper.cpp line 53: > >> 51: >> 52: // Uninstall product. >> 53: msi::Suppre

Re: RFR: 8290471: jpackage: allow to specify codepage on Windows

2022-07-18 Thread Alexey Semenyuk
On Mon, 18 Jul 2022 22:33:15 GMT, Alex Kasko wrote: > It is proposed to introduce support for `--win-codepage` argument, and > substitute the `Codepage` attribute with `--win-codepage` specified value in > primary l10n file when it is being copied to the config dir. > > Instead of copying all

Integrated: 8290402: jpackage exe uninstallers don't return correct exit code in case of failure

2022-07-19 Thread Alexey Semenyuk
On Sat, 16 Jul 2022 21:47:43 GMT, Alexey Semenyuk wrote: > 8290402: jpackage exe uninstallers don't return correct exit code in case of > failure This pull request has now been integrated. Changeset: d67e7ccd Author:Alexey Semenyuk URL: https://git.openjdk.org

Integrated: 8283707: Support version format on Windows

2022-07-19 Thread Alexey Semenyuk
On Fri, 15 Jul 2022 04:37:58 GMT, Alexey Semenyuk wrote: > 8283707: Support version format on Windows This pull request has now been integrated. Changeset: 977e0948 Author: Alexey Semenyuk URL: https://git.openjdk.org/jdk/commit/977e09489dd1f49d8f373ef7b8c5e47fedb82793 St

  1   2   3   4   5   6   7   8   >