Re: [External] : Re: jpackage OS/X JDK-8263157 fix regressed out

2021-07-22 Thread Alexey Semenyuk
There is no corresponding folder on other platforms, so to simplify packaging "app" directory is used on all platforms. - Alexey On 7/22/2021 2:42 PM, Alan Snyder wrote: Why is a directory named “app” used for dynamic libraries instead of the conventional directory “Frameworks”? Alan

Re: [External] : Re: jpackage OS/X JDK-8263157 fix regressed out

2021-07-22 Thread Michael Hall
> On Jul 22, 2021, at 1:46 PM, Alexey Semenyuk > wrote: > > Oh, inconsistent path separators indeed. Good catch! This is the reason it > doesn't work as it should. > Filed https://bugs.openjdk.java.net/browse/JDK-8271155 to track it. > > - Alexey > > Thank you.

Re: [External] : Re: jpackage OS/X JDK-8263157 fix regressed out

2021-07-22 Thread Alexey Semenyuk
Oh, inconsistent path separators indeed. Good catch! This is the reason it doesn't work as it should. Filed https://bugs.openjdk.java.net/browse/JDK-8271155 to track it. - Alexey On 7/22/2021 2:04 PM, Michael Hall wrote: On Jul 22, 2021, at 12:29 PM, Alexey Semenyuk wrote: The fix for JD

Re: jpackage OS/X JDK-8263157 fix regressed out

2021-07-22 Thread Alan Snyder
Why is a directory named “app” used for dynamic libraries instead of the conventional directory “Frameworks”? Alan > On Jul 22, 2021, at 11:04 AM, Michael Hall wrote: > > > >> On Jul 22, 2021, at 12:29 PM, Alexey Semenyuk >> wrote: >> >> The fix for JDK-8263157 cleared the default val

Re: jpackage OS/X JDK-8263157 fix regressed out

2021-07-22 Thread Michael Hall
> On Jul 22, 2021, at 12:29 PM, Alexey Semenyuk > wrote: > > The fix for JDK-8263157 cleared the default value of `java.library.path` > system property and resulted in JDK-8267598 regression. So the fix for > JDK-8263157 was reworked: jpackage doesn't set `java.library.path` system > prope

Re: jpackage OS/X JDK-8263157 fix regressed out

2021-07-22 Thread Alexey Semenyuk
The fix for JDK-8263157 cleared the default value of `java.library.path` system property and resulted in JDK-8267598 regression. So the fix for JDK-8263157 was reworked: jpackage doesn't set `java.library.path` system property, but it adds `app` directory to `DYLD_LIBRARY_PATH` env variable on

jpackage OS/X JDK-8263157 fix regressed out

2021-07-22 Thread Michael Hall
JDK-8263157 [macos]: java.library.path is being set incorrectly The fix for this seems to be gone in both current jdk17 and jdk18 releases. There is no ‘app’ directory included in java.library.path. outputdir/HalfPipe.app/Contents/runtime/Conte