Integrated: 8316563: test tools/jpackage/linux/LinuxResourceTest.java fails on CentOS Linux release 8.5.2111 and Fedora 27

2023-10-20 Thread yaqsun
On Wed, 20 Sep 2023 03:32:55 GMT, yaqsun wrote: > on some RHEL Linux 8.X and Fedora 27 machines , we run into errors in test > tools/jpackage/linux/LinuxResourceTest.java. > It's the same problem as https://bugs.openjdk.org/browse/JDK-8314121. > But after 8314121, too

Re: RFR: 8316563: test tools/jpackage/linux/LinuxResourceTest.java fails on CentOS Linux release 8.5.2111 and Fedora 27 [v5]

2023-10-19 Thread yaqsun
ears. tools/jpackage/linux/LinuxResourceTest.java does not use file > src/jdk.jpackage/linux/classes/jdk/jpackage/internal/resources/template.spec. yaqsun has updated the pull request incrementally with one additional commit since the last revision: After tests passed, rest

Re: RFR: 8316563: test tools/jpackage/linux/LinuxResourceTest.java fails on CentOS Linux release 8.5.2111 and Fedora 27 [v4]

2023-10-19 Thread yaqsun
ears. tools/jpackage/linux/LinuxResourceTest.java does not use file > src/jdk.jpackage/linux/classes/jdk/jpackage/internal/resources/template.spec. yaqsun has updated the pull request incrementally with one additional commit since the last revision: generate a temporary pa

Re: RFR: 8316563: test tools/jpackage/linux/LinuxResourceTest.java fails on CentOS Linux release 8.5.2111 and Fedora 27

2023-10-19 Thread yaqsun
On Tue, 17 Oct 2023 07:03:50 GMT, yaqsun wrote: >> `%define _build_id_links none` must be added to a spec file in >> LinuxResourceTest.java file. not on `rpmbuild` command line > >> `%define _build_id_links none` must be added to a spec file in >> LinuxResourceTest.

Re: RFR: 8316563: test tools/jpackage/linux/LinuxResourceTest.java fails on CentOS Linux release 8.5.2111 and Fedora 27 [v3]

2023-10-19 Thread yaqsun
ears. tools/jpackage/linux/LinuxResourceTest.java does not use file > src/jdk.jpackage/linux/classes/jdk/jpackage/internal/resources/template.spec. yaqsun has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes b

Re: RFR: 8316563: test tools/jpackage/linux/LinuxResourceTest.java fails on CentOS Linux release 8.5.2111 and Fedora 27

2023-10-17 Thread yaqsun
On Mon, 16 Oct 2023 17:24:01 GMT, Alexey Semenyuk wrote: > `%define _build_id_links none` must be added to a spec file in > LinuxResourceTest.java file. not on `rpmbuild` command line `%define _build_id_links none` has been added to a spec file in LinuxResourceTest.java file. Please review. -

Re: RFR: 8316563: test tools/jpackage/linux/LinuxResourceTest.java fails on CentOS Linux release 8.5.2111 and Fedora 27 [v2]

2023-10-17 Thread yaqsun
ears. tools/jpackage/linux/LinuxResourceTest.java does not use file > src/jdk.jpackage/linux/classes/jdk/jpackage/internal/resources/template.spec. yaqsun has updated the pull request incrementally with two additional commits since the last revision: - fixed Whitespace error: del tab - 83165

RFR: 8316563: test tools/jpackage/linux/LinuxResourceTest.java fails on CentOS Linux release 8.5.2111 and Fedora 27

2023-10-06 Thread yaqsun
on some RHEL Linux 8.X and Fedora 27 machines , we run into errors in test tools/jpackage/linux/LinuxResourceTest.java. It's the same problem as https://bugs.openjdk.org/browse/JDK-8314121. But after 8314121, tools/jpackage/linux/LinuxResourceTest.java still reappears. tools/jpackage/linux/Linux

Integrated: 8311631: When multiple users run tools/jpackage/share/LicenseTest.java, Permission denied for writing /var/tmp/*.files

2023-07-24 Thread yaqsun
On Fri, 7 Jul 2023 09:51:23 GMT, yaqsun wrote: > The prerequisite is to install the rpmbuild command, when multiple users > switch to write /var/tmp/*.files will have the permission forbidden. This pull request has now been integrated. Changeset: fac9f88c Author:sunyaqi Committer:

Re: RFR: 8311631: When multiple users run tools/jpackage/share/LicenseTest.java, Permission denied for writing /var/tmp/*.files [v4]

2023-07-22 Thread yaqsun
> The prerequisite is to install the rpmbuild command, when multiple users > switch to write /var/tmp/*.files will have the permission forbidden. yaqsun has updated the pull request incrementally with one additional commit since the last revision: 8311631: restore to final patch and t

Re: RFR: 8311631: When multiple users run tools/jpackage/share/LicenseTest.java, Permission denied for writing /var/tmp/*.files [v3]

2023-07-22 Thread yaqsun
> The prerequisite is to install the rpmbuild command, when multiple users > switch to write /var/tmp/*.files will have the permission forbidden. yaqsun has updated the pull request incrementally with one additional commit since the last revision: 8311631: When multiple users run

Re: RFR: 8311631: When multiple users run tools/jpackage/share/LicenseTest.java, Permission denied for writing /var/tmp/*.files [v2]

2023-07-16 Thread yaqsun
On Fri, 14 Jul 2023 21:58:33 GMT, Alexey Semenyuk wrote: > The fix with `%{_builddir}` looks better. Did it resolve the issue? Yes, it solved. The root of intermediate files changed to /tmp/jdk.jpackage16701381473798822/BUILD/commonlicensetest.filesystem.files. - PR Comment: https

Re: RFR: 8311631: When multiple users run tools/jpackage/share/LicenseTest.java, Permission denied for writing /var/tmp/*.files [v2]

2023-07-13 Thread yaqsun
> The prerequisite is to install the rpmbuild command, when multiple users > switch to write /var/tmp/*.files will have the permission forbidden. yaqsun has updated the pull request incrementally with one additional commit since the last revision: 8311631: When multiple users run

Re: RFR: 8311631: When multiple users run tools/jpackage/share/LicenseTest.java, Permission denied for writing /var/tmp/*.files

2023-07-11 Thread yaqsun
On Fri, 7 Jul 2023 09:51:23 GMT, yaqsun wrote: > The prerequisite is to install the rpmbuild command, when multiple users > switch to write /var/tmp/*.files will have the permission forbidden. How about JTwork(if achievable)? Files created in JTwork do not need to be cleaned wh

Re: RFR: 8311631: When multiple users run tools/jpackage/share/LicenseTest.java, Permission denied for writing /var/tmp/*.files

2023-07-11 Thread yaqsun
On Tue, 11 Jul 2023 22:15:49 GMT, Alexey Semenyuk wrote: > The change affects jpackage rpm spec template file. It is misleading to state > that this change is `Low risk, only test changes.` My mistake, I will update the initial comment. - PR Comment: https://git.openjdk.org/jdk/pu

RFR: 8311631: When multiple users run tools/jpackage/share/LicenseTest.java, Permission denied for writing /var/tmp/*.files

2023-07-08 Thread yaqsun
Low risk, only test changes. The prerequisite is to install the rpmbuild command, when multiple users switch to write /var/tmp/*.files will have the permission forbidden. - Commit messages: - 8311631: When multiple users run tools/jpackage/share/LicenseTest.java, Permission denied