Re: RFR: 8343610: InOutPathTest jpackage test produces invalid app image on macOS

2024-11-07 Thread Matthias Baesken
On Tue, 5 Nov 2024 16:08:20 GMT, Alexey Semenyuk wrote: > - fix `JPackageCommand.excludeAppLayoutAsserts()`; > - fix `InOutPath` test case that will make jpackage produce an invalid app > image on macOS. Hi Aleksei, with your patch added I see so far no errors any more related to InOutPathTest

Re: RFR: 8343610: InOutPathTest jpackage test produces invalid app image on macOS

2024-11-05 Thread Matthias Baesken
On Tue, 5 Nov 2024 22:42:13 GMT, Alexey Semenyuk wrote: > can you try this fix in your environment? Sure, I added the patch to our build/test queue . - PR Comment: https://git.openjdk.org/jdk/pull/21909#issuecomment-2458923465

Re: RFR: 8343610: InOutPathTest jpackage test produces invalid app image on macOS

2024-11-05 Thread Alexander Matveev
On Tue, 5 Nov 2024 16:08:20 GMT, Alexey Semenyuk wrote: > - fix `JPackageCommand.excludeAppLayoutAsserts()`; > - fix `InOutPath` test case that will make jpackage produce an invalid app > image on macOS. Looks good. - Marked as reviewed by almatvee (Reviewer). PR Review: https://

Re: RFR: 8343610: InOutPathTest jpackage test produces invalid app image on macOS

2024-11-05 Thread Alexey Semenyuk
On Tue, 5 Nov 2024 16:08:20 GMT, Alexey Semenyuk wrote: > - fix `JPackageCommand.excludeAppLayoutAsserts()`; > - fix `InOutPath` test case that will make jpackage produce an invalid app > image on macOS. @MBaesken can you try this fix in your environment? - PR Comment: https://git

Re: RFR: 8343610: InOutPathTest jpackage test produces invalid app image on macOS

2024-11-05 Thread Alexey Semenyuk
On Tue, 5 Nov 2024 16:08:20 GMT, Alexey Semenyuk wrote: > - fix `JPackageCommand.excludeAppLayoutAsserts()`; > - fix `InOutPath` test case that will make jpackage produce an invalid app > image on macOS. @sashamatveev ptal - PR Comment: https://git.openjdk.org/jdk/pull/21909#issue

RFR: 8343610: InOutPathTest jpackage test produces invalid app image on macOS

2024-11-05 Thread Alexey Semenyuk
- fix `JPackageCommand.excludeAppLayoutAsserts()`; - fix `InOutPath` test case that will make jpackage produce an invalid app image on macOS. - Commit messages: - Better wording of the comment - One more try to make it work on OSX - Bugfix for the case when `--app-content` is set