tools: improve macos-firewall.sh output

The output of tools/macos-firewall.sh can cause people to think it
didn't work. Update things slightly to make the output mildly more
informative.

Refs: https://github.com/nodejs/node/issues/37233#issuecomment-802201046

PR-URL: https://github.com/nodejs/node/pull/37846
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
Rich Trott 2021-03-20 11:53:47 -07:00
parent 448158e0f8
commit f2090877f1

View File

@ -21,36 +21,25 @@ CCTEST_DEBUG="$OUTDIR/Debug/cctest"
OPENSSL_CLI_RELEASE="$OUTDIR/Release/openssl-cli"
OPENSSL_CLI_DEBUG="$OUTDIR/Debug/openssl-cli"
add_and_unblock () {
if [ -e "$1" ]
then
echo Processing "$1"
$SFW --remove "$1" >/dev/null
$SFW --add "$1"
$SFW --unblock "$1"
fi
}
if [ -f $SFW ];
then
# Duplicating these commands on purpose as the symbolic link node might be
# linked to either out/Debug/node or out/Release/node depending on the
# BUILDTYPE.
$SFW --remove "$NODE_DEBUG"
$SFW --remove "$NODE_DEBUG"
$SFW --remove "$NODE_RELEASE"
$SFW --remove "$NODE_RELEASE"
$SFW --remove "$NODE_LINK"
$SFW --remove "$CCTEST_DEBUG"
$SFW --remove "$CCTEST_RELEASE"
$SFW --remove "$OPENSSL_CLI_DEBUG"
$SFW --remove "$OPENSSL_CLI_RELEASE"
$SFW --add "$NODE_DEBUG"
$SFW --add "$NODE_RELEASE"
$SFW --add "$NODE_LINK"
$SFW --add "$CCTEST_DEBUG"
$SFW --add "$CCTEST_RELEASE"
$SFW --add "$OPENSSL_CLI_DEBUG"
$SFW --add "$OPENSSL_CLI_RELEASE"
$SFW --unblock "$NODE_DEBUG"
$SFW --unblock "$NODE_RELEASE"
$SFW --unblock "$NODE_LINK"
$SFW --unblock "$CCTEST_DEBUG"
$SFW --unblock "$CCTEST_RELEASE"
$SFW --unblock "$OPENSSL_CLI_DEBUG"
$SFW --unblock "$OPENSSL_CLI_RELEASE"
add_and_unblock "$NODE_DEBUG"
add_and_unblock "$NODE_RELEASE"
add_and_unblock "$NODE_LINK"
add_and_unblock "$CCTEST_DEBUG"
add_and_unblock "$CCTEST_RELEASE"
add_and_unblock "$OPENSSL_CLI_DEBUG"
add_and_unblock "$OPENSSL_CLI_RELEASE"
else
echo "SocketFirewall not found in location: $SFW"
fi