RFR: 8342858: Make target mac-jdk-bundle fails on chmod command
Lutz Schmidt
lucy at openjdk.org
Wed Oct 23 13:30:09 UTC 2024
On Tue, 22 Oct 2024 21:30:40 GMT, Erik Joelsson <erikj at openjdk.org> wrote:
> The target mac-jdk-bundle can fail randomly. MacBundles.gmk defines a large number of individual copy rules, which can execute in any order. The "install-file" (our copy) macro on macos includes a check for weird attributes using `xattr` so that we can remove them. We use the switch `-s` to make sure that xattr operates on symlinks instead of their targets. However, if we find something, we also run `chmod` to make sure we have the permissions to remove attributes in the first place, but the chmod command does not have the `-h` switch to operate on the symlink instead of the target. So if the symlink gets copied before its target, there is a chance that we try to run chmod before the target exists, which will result in a failure. My proposed fix is to add `-h` to the chmod command so that everything operates on the symlink instead of the target.
LGTM.
-------------
Marked as reviewed by lucy (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/21649#pullrequestreview-2388698027
More information about the build-dev
mailing list