Skip to content

Commit 7aa2455

Browse files
author
Paul Hohensee
committed
8294310: compare.sh fails on macos after JDK-8293550
Backport-of: b9eeec2b6b8f97d2ba33462b27d091f918067f78
1 parent 27db036 commit 7aa2455

File tree

3 files changed

+14
-4
lines changed

3 files changed

+14
-4
lines changed

make/autoconf/compare.sh.in

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ export AWK="@AWK@"
4040
export BASH="@BASH@"
4141
export CAT="@CAT@"
4242
export CMP="@CMP@"
43+
export CODESIGN="@CODESIGN@"
4344
export CP="@CP@"
4445
export CUT="@CUT@"
4546
export DIFF="@DIFF@"

make/conf/jib-profiles.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -564,7 +564,7 @@ var getJibProfilesProfiles = function (input, common, data) {
564564
"ANT_HOME": input.get("ant", "home_path")
565565
}
566566
};
567-
[ "linux-x64", "macosx-x64", "windows-x64", "linux-aarch64"]
567+
[ "linux-x64", "macosx-aarch64", "macosx-x64", "windows-x64", "linux-aarch64"]
568568
.forEach(function (name) {
569569
var maketestName = name + "-testmake";
570570
profiles[maketestName] = concatObjects(profiles[name], testmakeBase);

make/scripts/compare.sh

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ if [ "$OPENJDK_TARGET_OS" = "macosx" ]; then
4242
LDD_CMD="$OTOOL -L"
4343
DIS_CMD="$OTOOL -v -V -t"
4444
STAT_PRINT_SIZE="-f %z"
45+
STRIP="$STRIP -no_code_signature_warning"
4546
elif [ "$OPENJDK_TARGET_OS" = "windows" ]; then
4647
FULLDUMP_CMD="$DUMPBIN -all"
4748
LDD_CMD="$DUMPBIN -dependents"
@@ -674,14 +675,22 @@ compare_bin_file() {
674675
ORIG_THIS_FILE="$THIS_FILE"
675676
ORIG_OTHER_FILE="$OTHER_FILE"
676677

677-
if [ "$STRIP_ALL" = "true" ] || [[ "$STRIP_BEFORE_COMPARE" = *"$BIN_FILE"* ]]; then
678+
if [ "$STRIP_ALL" = "true" ] || [[ "$STRIP_BEFORE_COMPARE" = *"$BIN_FILE"* ]] \
679+
|| [ "$OPENJDK_TARGET_OS" = "macosx" ]; then
678680
THIS_STRIPPED_FILE=$FILE_WORK_DIR/this/$NAME
679681
OTHER_STRIPPED_FILE=$FILE_WORK_DIR/other/$NAME
680682
$MKDIR -p $FILE_WORK_DIR/this $FILE_WORK_DIR/other
681683
$CP $THIS_FILE $THIS_STRIPPED_FILE
682684
$CP $OTHER_FILE $OTHER_STRIPPED_FILE
683-
$STRIP $THIS_STRIPPED_FILE
684-
$STRIP $OTHER_STRIPPED_FILE
685+
if [ "$STRIP_ALL" = "true" ] || [[ "$STRIP_BEFORE_COMPARE" = *"$BIN_FILE"* ]]; then
686+
$STRIP $THIS_STRIPPED_FILE
687+
$STRIP $OTHER_STRIPPED_FILE
688+
fi
689+
# On macosx, always remove any signature before comparing
690+
if [ "$OPENJDK_TARGET_OS" = "macosx" ]; then
691+
$CODESIGN --remove-signature $THIS_STRIPPED_FILE
692+
$CODESIGN --remove-signature $OTHER_STRIPPED_FILE
693+
fi
685694
THIS_FILE="$THIS_STRIPPED_FILE"
686695
OTHER_FILE="$OTHER_STRIPPED_FILE"
687696
fi

0 commit comments

Comments
 (0)