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

2023-03-17 Thread Matthias Baesken
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 Alexander Matveev
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: 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 Matthias Baesken
> 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 > like AIX) the actual env variable has the expected string

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

2023-03-16 Thread Matthias Baesken
On Wed, 15 Mar 2023 18:37:51 GMT, Alexey Semenyuk wrote: >> Matthias Baesken has updated the pull request incrementally with one >> additional commit since the last revision: >> >> redo check > > test/jdk/tools/jpackage/share/AppLauncherEnvTest.java line 93: > >> 91: } else { >> 92: