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

2022-04-26 Thread Victor Dyakov
On Thu, 14 Apr 2022 21:04:52 GMT, Alexey Semenyuk wrote: >> Implementation of [JDK-8275062: "Allow jpackage create installers for >> services"](https://bugs.openjdk.java.net/browse/JDK-8275062) >> CSR > > Alexey Semenyuk has updated the pull request with a new target base due to a > merge or a

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

2022-01-03 Thread Victor Dyakov
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: 8276084: Linux DEB Bundler: release number in outputted .deb file should be optional

2021-11-12 Thread Victor Dyakov
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 @andyherrick please review - PR: https://git.openjdk.java.net/jdk/pull/6345

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

2021-11-12 Thread Victor Dyakov
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. @andyherrick please review

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

2021-11-12 Thread Victor Dyakov
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 @sashamatveev please review - PR: https://git.openjdk.java.net/jdk/pull/6345

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

2021-07-06 Thread Victor Dyakov
On Sat, 3 Jul 2021 00:03:55 GMT, Alexander Matveev wrote: >> Disable stripping to prevent rpmbuild from modifying executables > > Marked as reviewed by almatvee (Reviewer). @sashamatveev @herrick please review - PR: https://git.openjdk.java.net/jdk17/pull/207

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

2021-06-24 Thread Victor Dyakov
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 > "h

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

2021-03-23 Thread Victor Dyakov
On Tue, 23 Mar 2021 19:40:38 GMT, Kevin Rushforth wrote: >>> Looks good, although i see the following wasn't backed out: >>> >>> ``` >>> src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/resources/JavaApp.icns >>> ``` >> >> yes - look again - it was renamed back to java.icns (the 12'th or 2

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

2021-03-23 Thread Victor Dyakov
On Tue, 23 Mar 2021 20:42:22 GMT, Victor Dyakov wrote: >> As long as there are no transient build/test failures, it will be fine. I >> was just comparing the results with what `git revert >> 11c8c78c47f21fcd87a5969a859b5c4fced5e47d` generated and noticed this >> diffe

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

2021-03-12 Thread Victor Dyakov
On Sat, 13 Mar 2021 00:52:44 GMT, Kevin Rushforth wrote: >> Marked as reviewed by almatvee (Committer). > > Since you are removing the problem listing, would it be better to use the > parent bug ID -- 8263474 -- rather than a sub-task? How do you plan to > resolve the parent bug? Manually as "D