[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi-commits] [lmi] master da16184 2/2: Place a throwaway directory in /

From: Greg Chicares
Subject: [lmi-commits] [lmi] master da16184 2/2: Place a throwaway directory in /opt/lmi/, not in /tmp/
Date: Wed, 8 May 2019 18:31:46 -0400 (EDT)

branch: master
commit da16184f76cb221c86202ce052a5451e7b5e4da0
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Place a throwaway directory in /opt/lmi/, not in /tmp/
    To a native msw program running under cygwin, '/tmp' is something like
    whereas to cygwin itself, '/tmp' is something like
    . To avoid that unwanted contrariety, placed the throwaway directory
    under /opt/lmi/ so that the cygwin "identity mount" maps both to the
    same place.
    Full paths seem preferable because of their unambiguity. For example, if
      $PERFORM /opt/lmi/bin/lmi_cli_shared --file="$throwaway_dir"/X.ill
    is echoed into a nychthemeral log as
      wine /opt/lmi/bin/lmi_cli_shared --file=/opt/lmi/amd64/logs/tmp/X.ill
    and that command fails, then it can be copied and pasted into a terminal
    to be run manually without adjusting the terminal's $PWD.
 nychthemeral_test.sh | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/nychthemeral_test.sh b/nychthemeral_test.sh
index 71e1a61..9ec8ffb 100755
--- a/nychthemeral_test.sh
+++ b/nychthemeral_test.sh
@@ -135,7 +135,12 @@ nychthemeral_clutter='
 # Directory for test logs.
+# It seems redundant to construct yet another $prefix and $exec_prefix here;
+# perhaps that should be done OAOO in a script that selects a toolchain.
 mkdir --parents "$log_dir"
@@ -186,9 +191,9 @@ java -jar /opt/lmi/third_party/rng/jing.jar -c xrc.rnc 
./*.xrc 2>&1 \
 # Run the following tests in a throwaway directory so that the files
 # they create can be cleaned up easily.
-cd /tmp
-mkdir --parents /tmp/lmi/tmp
-cd /tmp/lmi/tmp
+mkdir --parents "$throwaway_dir"
+cd "$throwaway_dir"
 # Copy these files hither because the emission tests write some
 # output files to the input file's directory.
@@ -197,9 +202,9 @@ cp /opt/lmi/src/lmi/sample.cns .
 printf '\n# test all valid emission types\n\n'
-$PERFORM /opt/lmi/bin/lmi_cli_shared --file=/tmp/lmi/tmp/sample.ill --accept 
--ash_nazg --data_path=/opt/lmi/data 
+$PERFORM /opt/lmi/bin/lmi_cli_shared --file="$throwaway_dir"/sample.ill 
--accept --ash_nazg --data_path=/opt/lmi/data 
-$PERFORM /opt/lmi/bin/lmi_cli_shared --file=/tmp/lmi/tmp/sample.cns --accept 
--ash_nazg --data_path=/opt/lmi/data 
+$PERFORM /opt/lmi/bin/lmi_cli_shared --file="$throwaway_dir"/sample.cns 
--accept --ash_nazg --data_path=/opt/lmi/data 
 printf '\n# schema tests\n\n'
 /opt/lmi/src/lmi/test_schemata.sh 2>&1 \
@@ -207,7 +212,7 @@ printf '\n# schema tests\n\n'
 # Clean up stray output. (The zsh '(N)' glob qualifier turns on
 # null_glob for a single expansion.)
-for z in /tmp/lmi/tmp/*(N); do rm "$z"; done
+for z in "$throwaway_dir"/*(N); do rm "$z"; done
 # The automated GUI test simulates keyboard and mouse actions, so
 # no such actions must be performed manually while it is running.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]