On 2/5/21 4:51 AM, Magnus Ihse Bursie wrote:
On Tue, 2 Feb 2021 21:20:52 GMT, Daniel D. Daugherty <dcu...@openjdk.org> wrote:
Anton Kozlov has updated the pull request incrementally with one additional
commit since the last revision:
support macos_aarch64 in hsdis
make/autoconf/flags.m4 line 140:
138: else
139: MACOSX_VERSION_MIN=10.12.0
140: fi
Not something that needs to be addressed here, but these changes
illustrate that our collective use of macOSX/MACOSX/MacOSX names
are tied to the fact that the macOS major version number was at 10
for a very long time.
@magicus - Do we have an RFE to rename MACOSX or are we sticking
with it and evolving our interpretation of the 'X' from '10' to */splat/asterik?
@dcubed-ojdk There is no RFE to renaming "macosx" to "macos". I'm not sure it should be done. We
can't follow all marketing trends (Apple recently renamed iOS to iPadOS for the iPad; we can't keep adapting to such
schemes). Personally, I like the new name without the "x", but we had already spent some time trying to find
and fix all (or at least, most) instances of "osx" in the code, that I don't really think it's worth the
effort.
If you can drill up enough enthusiasm for such a project, and get any
objections down to minimum, I can help implementing it. But I won't be
spearheading it.
make/common/NativeCompilation.gmk line 1178:
1176: endif
1177: # This only works if the openjdk_codesign identity is
present on the system. Let
1178: # silently fail otherwise.
Might want to add a comment here:
# The '-f' option will replace an existing signature if one exists.
We're not really in the habit of adding comments for various command line options. Normally, you
can check these with "man" if you are uncertain. If they do something surprising, sure,
but here it's more of a "it's needed on aarch64 to work at all", so I don't think a
comment will be anything but added clutter.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2200
@magicus - I'm good with both of these answers. I personally like 'macosx'.
Dan
