RFR: 8352480: Don't follow symlinks in additional content for app images
Alexander Matveev
almatvee at openjdk.org
Sat May 3 03:37:46 UTC 2025
On Thu, 1 May 2025 21:23:38 GMT, Mikhail Yankelevich <myankelevich at openjdk.org> wrote:
>> - Symbolic links will not be followed for `--app-content` on all platforms.
>> - Added test to cover this case.
>> - `MacHelper` updated to use "cp -R" instead of "cp -r" when unpacking DMG, since "cp -r" on macOS is not documented option and when used `cp` will follow symbolic links which breaks test. "cp -R" does not follow symbolic links.
>
> test/jdk/tools/jpackage/share/AppContentTest.java line 57:
>
>> 55: */
>> 56: public class AppContentTest {
>> 57:
>
> Nitpick: do you think it might be worth it to add a bug id to the `@test`?
No. I do not see a need. This is a generic test and not something specific to particular JBS issue.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24974#discussion_r2072305435
More information about the core-libs-dev
mailing list