Merge pull request #318 from ROCm-Developer-Tools/emankov

[HIPIFY][#311][fix] Get rid of socat in run_test.sh

hipify-clang/CMakeLists.txt

@@ -78,9 +78,6 @@ if (HIPIFY_CLANG_TESTS)
 
     require_program(lit)
     require_program(FileCheck)
-    if(NOT WIN32)
-        require_program(socat)
-    endif()
 
     # Populates CUDA_TOOLKIT_ROOT_DIR, which is then applied to the lit config to give the
     # value of --cuda-path for the test runs.

tests/hipify-clang/run_test.sh

@@ -13,16 +13,5 @@ shift 3
 
 # Remaining args are the ones to forward to clang proper.
 
-# Time for the classic insane little trick for making colour output work.
-# A self-deleting shell-script that does the thing we want to do...
-TMP_SCRIPT=$(mktemp)
-cat << EOF > $TMP_SCRIPT
-set -o errexit
-set -o xtrace
-rm $TMP_SCRIPT
 $HIPIFY -o=$TMP_FILE $IN_FILE -- $@ && cat $TMP_FILE | sed -Ee 's|//.+|// |g' | FileCheck $IN_FILE
-EOF
-chmod a+x $TMP_SCRIPT
 
-# Run the script via socat, spawning a virtual terminal and propagating exit code, and hence failure.
-socat -du EXEC:$TMP_SCRIPT,pty,stderr STDOUT