RFR: 8341939: SigningOptionsTest fails without Xcode with command line developer tools after JDK-8341443 [v2]
Alexander Matveev
almatvee at openjdk.org
Tue Oct 22 22:17:57 UTC 2024
On Tue, 22 Oct 2024 19:35:55 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:
>> Alexander Matveev has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8341939: SigningOptionsTest fails without Xcode with command line developer tools after JDK-8341443 [v2]
>
> test/jdk/tools/jpackage/macosx/base/SigningCheck.java line 104:
>
>> 102: } catch (IOException | InterruptedException ex) {
>> 103: throw new RuntimeException(ex);
>> 104: }
>
> Can be reduced to:
>
>
> public static void isXcodeDevToolsInstalled() {
> if (Executor.of("/usr/bin/xcrun", "--help").execute().getExitCode() != 0) {
> TKit.throwSkippedException("Missing Xcode with command line developer tools");
> }
> }
>
>
> Also will add a log record to help evaluate potential test failure.
Fixed, but I am using `executeWithoutExitCodeCheck()` instead of `execute()`, `execute()` throws exception if command returns non-zero value.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21628#discussion_r1811505327
More information about the core-libs-dev
mailing list