[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Integrated: 8287254: Clean up Xcode sysroot logic
From: Magnus Ihse Bursie <ihse () openjdk ! java ! net>
Date: 2022-05-25 8:09:00
Message-ID: p67Ecmipfqtc96YZvcIeKOv3ELWEeWBgKegbZE59EPY=.c82398f8-47c1-4b2f-8761-100d0689c639 () github ! com
[Download RAW message or body]
On Tue, 24 May 2022 17:09:10 GMT, Magnus Ihse Bursie <ihse@openjdk.org> wrote:
> The logic in BASIC_SETUP_DEVKIT for setting a correct sysroot for Xcode is hard to \
> follow. This should be straightened out. We also expose variables that are no \
> longer used. So there's a bit of related cleanup.
> The new code is more or less functionally equivalent to the old. There were some \
> corner cases which the old code did not handle well; this has now been improved. \
> I've also added yet another method of trying to get the SDK root before falling \
> back to the system library, by using `xcrun --show-sdk-path`.
> In an ideal world, the sysroot flags to `mig` should be set in configure, e.g. as \
> `MIG_FLAGS` or `MIG_SYSROOT_FLAGS`. I can't be bothered to do that for a single \
> call to `mig`, though.
> As always, changes like this that depend on the environment is tricky to test. I've \
> tried running it on a couple of different macs, with and without a devkit.
This pull request has now been integrated.
Changeset: d889319a
Author: Magnus Ihse Bursie <ihse@openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/d889319a86101e944aefd4ad7f300505abbe5b30
Stats: 200 lines in 4 files changed: 91 ins; 98 del; 11 mod
8287254: Clean up Xcode sysroot logic
Reviewed-by: erikj
-------------
PR: https://git.openjdk.java.net/jdk/pull/8872
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic