Skip to content

Commit 8f76bab

Browse files
committed
Fix the use of mktemp in tool scripts
The manual of `mktemp` states that "TEMPLATE must contain at least 3 consecutive 'X's in last component." Linux implementation seems to be relaxed about this and accepts and rewrites X's even inside the template, but mktemp of OSX is more strict and handles trailing X's only. This patch makes sure that mktemp templates work on both OS's. JerryScript-DCO-1.0-Signed-off-by: Akos Kiss [email protected]
1 parent a2d5acb commit 8f76bab

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tools/runners/run-test-suite.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ do
115115
if [ "$IS_SNAPSHOT" == true ]
116116
then
117117
# Testing snapshot
118-
SNAPSHOT_TEMP=`mktemp $(basename -s .js $test).XXXXXXXXXX.snapshot`
118+
SNAPSHOT_TEMP=`mktemp $(basename -s .js $test).snapshot.XXXXXXXXXX`
119119

120120
cmd_line="${ENGINE#$ROOT_DIR} $ENGINE_ARGS --save-snapshot-for-global $SNAPSHOT_TEMP ${full_test#$ROOT_DIR}"
121121
( ulimit -t $TIMEOUT; $ENGINE $ENGINE_ARGS --save-snapshot-for-global $SNAPSHOT_TEMP $full_test &> $ENGINE_TEMP )

0 commit comments

Comments
 (0)