(trafficserver) branch master updated: tools/cripts/compiler.sh: sanitize and allow for extensibility (#12018)

[zwoop]

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 1fdb602af3 tools/cripts/compiler.sh: sanitize and allow for 
extensibility (#12018)
1fdb602af3 is described below

commit 1fdb602af323edfd244fac160cf52ecb2bb2ad84
Author: Yann Kerhervé <40152+ya...@users.noreply.github.com>
AuthorDate: Fri Feb 7 13:30:41 2025 -0800

    tools/cripts/compiler.sh: sanitize and allow for extensibility (#12018)
    
    - Avoid potential issues with globbing/space in filenames
    - Make this script usable by other scripts calling into it
---
 tools/cripts/compiler.sh | 26 +++++++++++++-------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/tools/cripts/compiler.sh b/tools/cripts/compiler.sh
index 74e6d3dd64..c2e4b4f840 100755
--- a/tools/cripts/compiler.sh
+++ b/tools/cripts/compiler.sh
@@ -1,4 +1,4 @@
-#!/usr/bin/env bash
+#!/bin/sh
 
 #  Licensed to the Apache Software Foundation (ASF) under one
 #  or more contributor license agreements.  See the NOTICE file
@@ -24,24 +24,24 @@
 ##############################################################################
 
 # Configurable parts
-ATS_ROOT=/opt/ats
-CXX=clang++
-CXXFLAGS="-std=c++20 -I/opt/homebrew/include -undefined dynamic_lookup"
+: ${ATS_ROOT:="/opt/ats"}
+: ${CXX:="clang++"}
+: ${CXXFLAGS:="-std=c++20 -I/opt/homebrew/include -undefined dynamic_lookup"}
 
 # Probably don't need to change these ?
 STDFLAGS="-shared -fPIC -Wall -Werror -I${ATS_ROOT}/include -L${ATS_ROOT}/lib 
-lcripts"
 
 # This is optional, but if set, the script will cache the compiled shared 
objects for faster restarts/reloads
-CACHE_DIR=/tmp/ats-cache
+: ${CACHE_DIR:="/tmp/ats-cache"}
 
 # Extract the arguments, and do some sanity checks
-SOURCE=$1
-DEST=$2
+SOURCE="$1"
+DEST="$2"
 
-SOURCE_DIR=$(dirname $SOURCE)
-DEST_DIR=$(dirname $DEST)
-SOURCE_FILE=$(basename $SOURCE)
-DEST_FILE=$(basename $DEST)
+SOURCE_DIR=$(dirname "$SOURCE")
+DEST_DIR=$(dirname "$DEST")
+SOURCE_FILE=$(basename "$SOURCE")
+DEST_FILE=$(basename "$DEST")
 
 cd "$SOURCE_DIR"
 if [ $(pwd) != "$SOURCE_DIR" ]; then
@@ -81,10 +81,10 @@ if [ -d "$CACHE_DIR" ]; then
 fi
 
 # Compile the plugin
-${CXX} ${CXXFLAGS} ${STDFLAGS} -o $DEST $SOURCE
+${CXX} ${CXXFLAGS} ${STDFLAGS} -o "$DEST" "$SOURCE"
 exit_code=$?
 
-if [ $exit_code -ne 0 ]; then
+if [ "$exit_code" -ne 0 ]; then
     echo "Compilation failed with exit code $exit_code"
     exit $exit_code
 fi