2013-02-15 3 views
1

저는 bash를 사용하는 데 상당히 익숙하며 일부 테스트 사례를 실행하기 위해 자동 학습 스크립트를 만들려고했습니다. 현재 내 bash 스크립트가 이상하게 작동하는 것 같습니다. -e 플래그가 설정되면 diff가 양수 크기 일 때 bash가 종료되고 -e 플래그가 설정되지 않으면 스크립트는 diff 파일의 차이를 무시하고 모든 테스트가 통과되었다고 말합니다.diff를 사용하는 중 오류없이 종료하는 Bash 스크립트

"write_diff_out = ...."명령 바로 다음에 스크립트가 종료되고 다음 줄이 인쇄되지 않습니다. 나는 그 밖의 모든 것들이 잘 실행되도록 스크립트의 diffing 부분만을 포함 시켰습니다 (파일들은 모두 존재합니다).

# Validate outputs and print results 
echo "> Comparing current build's final memory output with golden memory output..."; 
for file in `ls test_progs`; 
do 
    file=$(echo $file | cut -d '.' -f1); 
    echo "$file"; 
    write_diff_out=$(diff ./log/$file.writeback.out ./log/$file.writeback.gold.out > ./diff/$file.writeback.diff); 
    echo "Finished write_diff"; 
    program_diff_out=$(diff -u <(grep -E '@@@' ./log/$file.program.out) <(grep -E '@@@' ./log/$file.program.gold.out) > ./diff/$file.program.diff); 
    echo "Finished program diff"; 

    if [ -z "$write_diff_out" ] && [ -z "$program_diff_out" ]; then 
     printf "%20s:\e[0;32mPASSED\e[0m\n" "$file"; 
    else 
     printf "%20s:\e[0;31mFAILED\e[0m\n" "$file"; 
    fi 
done 
echo "> Done comparing test outputs."; 

diff 명령의 형식을 지정하는 더 좋은 방법을 제안해도 좋지만 다른 방법으로 작성하는 것이 좋습니다.

+0

팁 : [ls을 구문 분석하지 마십시오.] (http://mywiki.wooledge.org/ParsingLs) 대신'for file in test_progs/*'를 실행하십시오. 또한, 줄 끝에서';'를 사용하지 않아도됩니다. (같은 행의 명령을 구분하는 데 사용됩니다. '예 : this') – darque

+0

'-e flag'의 의미를 이해하지 못합니다. 'bash -e' 또는'diff -e'를 의미합니까? 나는 또한 당신이'write_diff_out = $ (diff ... ...> diff/$ file.writeback.diff)'로 성취하고자하는 것을 이해할 수 없다. 아마도'diff/$ file.writeback.diff'가 비어 있는지 테스트해야합니다. – darque

+0

-e 플래그는 내가 생각하는 bash -e 플래그입니다 (bash 스크립트 맨 위에 "set -e"라고 부르는 플래그). write_diff_out 변수는 diff의 내용을 포함하고 diff를 지정된 파일에 기록하기위한 것입니다. 당신이 아래에 게시 한 방법은 더 깨끗해 보이지만 그렇게 할 것입니다. – Bill

답변

0

정확히 무엇이 문제인지는 모르지만 필자는 스크립트를 모범 사례에 맞게 다시 작성했습니다. 아마도 더 잘 작동 할 것입니다.

#!/bin/bash 

# Debugging mode: prints every command as executed, remove when uneeded 
set -x 

# Validate outputs and print results 
echo "> Comparing current build's final memory output with golden memory output..." 

cd test_progs 

for file in *; do 
    file="$(echo "$file" | sed 's/\.[^.]*$//')" 
    echo "$file" 

    # will PASS when both diffs return non-zero 

    if ! diff "log/$file.writeback.out" \ 
       "log/$file.writeback.gold.out" > \ 
       "diff/$file.writeback.diff" && \ 
     ! diff -u <(grep -E @@@ "log/$file.program.out") \ 
       <(grep -E @@@ "log/$file.program.gold.out") > \ 
       "diff/$file.program.diff"; then 
     printf '%20s:\e[0;32mPASSED\e[0m\n' "$file" 
    else 
     printf '%20s:\e[0;31mFAILED\e[0m\n' "$file" 
    fi 
done 

echo "> Done comparing test outputs." 

그것은 구문 분석 LS를 방지, use quotes where it is due 대신 [의 [[을 사용 (당신의 내부 변수를 인용 할 필요가 없습니다 [[), 및 기록 된 파일 대신 변수에 무언가를 저장하는 빈 상태 (empty)의 경우는 테스트 . 당신이 정말로 변수에 DIFF의 출력을 저장하고 싶다면

, 당신은이 작업을 수행 할 것입니다 :

write_diff_out="$(diff "log/$file.writeback.out" "log/$file.writeback.gold.out" | tee "diff/$file.writeback.diff")" 

그런 다음 $write_diff_outdiff/$file.writeback.diff 파일이 동일한 데이터를 포함한다.

EDIT : 내 대답을 편집하여 의견에 몇 가지 사항을 구현하십시오.

+0

$ file에 공백이 포함되지 않는다고 보장하면 파일 이름을 따옴표로 묶을 수 있습니다. 아마도 파일을 직접 만들어서 지독한 파일 이름에 대해 경계해야하기 때문입니다. 그러나 diff의 출력에는 공백과 다른 구분 기호가 포함될 수 있으므로 * $ (diff ...)를 따옴표로 묶을 수는 없습니다. 또한 파일 이름에 여러개의 점이 있으면'cut'은 예상했던대로 작동하지 않을 수 있습니다 (첫 번째 점 다음에는'filey.name.c'가'filey'가 될 것입니다). – darque

+0

고마워요! 나중에 사용하기 위해이 중 일부를 시도해보고 사용 방법을 알려줍니다. 파일 이름 구조를 잘랐다면, 적절하게 이름을 지정해야합니다 ... 단지 memtest와 같은 파일 이름을 사용하는 것입니다.확장명없이 파일 이름을 가져옵니다. 아마도 그냥 마지막을 제거. 바람직 할 것이다. – Bill

+0

나는 당신의 상처를 거의 교정했는데, 아마도이 행동은 의도 된 것이지만 잘못 생각한 것일 수 있습니다. 다음은 sed로 올바른 버전입니다 :'file = "$ (echo"$ file "| sed '///.[^.]*$//')" – darque

관련 문제