From: Junio C Hamano <gitster@pobox•com>
To: "Torsten Bögershausen" <tboegi@web•de>
Cc: git@vger•kernel.org
Subject: Re: [PATCH] t0027: Improve test for not-normalized files
Date: Fri, 09 Oct 2015 15:12:34 -0700 [thread overview]
Message-ID: <xmqqk2qvofn1.fsf@gitster.mtv.corp.google.com> (raw)
In-Reply-To: <56172D62.9060408@web.de> ("Torsten Bögershausen"'s message of "Fri, 9 Oct 2015 04:58:42 +0200")
Torsten Bögershausen <tboegi@web•de> writes:
This patch is seriously broken and I do not know how you managed to
do so. Notice how "+create_NNO_files" is indented but no other
added lines in the same hunk, for example.
I tried to hand-munge, but gave up.
> +commit_chk_wrnNNO () {
Squashing warn into wrn or (check into chk) does not make it any
easier to read or type.
> + crlf=$1
> + attr=$2
> + lfwarn=$3
> + crlfwarn=$4
> + lfmixcrlf=$5
> + lfmixcr=$6
> + crlfnul=$7
> + pfx=NNO_${crlf}_attr_${attr}
> + #Commit files on top of existing file
> + create_gitattributes "$attr" &&
> + for f in LF CRLF CRLF_mix_LF LF_mix_CR CRLF_nul
> + do
> + fname=${pfx}_$f.txt &&
> + cp $f $fname &&
> + git -c core.autocrlf=$crlf add $fname 2>/dev/null &&
> + git -c core.autocrlf=$crlf commit -m "commit_$fname" $fname >"${pfx}_$f.err" 2>&1
> + done
> +
> + test_expect_success "commit NNO files crlf=$crlf attr=$attr LF" '
> + check_warning "$lfwarn" ${pfx}_LF.err
> + '
> + test_expect_success "commit NNO files crlf=$crlf attr=$attr CRLF" '
> + check_warning "$crlfwarn" ${pfx}_CRLF.err
> + '
> +
> + test_expect_success "commit NNO files crlf=$crlf attr=$attr CRLF_mix_LF" '
> + check_warning "$lfmixcrlf" ${pfx}_CRLF_mix_LF.err
> + '
> +
> + test_expect_success "commit NNO files crlf=$crlf attr=$attr LF_mix_cr" '
> + check_warning "$lfmixcr" ${pfx}_LF_mix_CR.err
> + '
> +
> + test_expect_success "commit NNO files crlf=$crlf attr=$attr CRLF_nul" '
> + check_warning "$crlfnul" ${pfx}_CRLF_nul.err
> + '
> +}
> +
> check_files_in_repo () {
> crlf=$1
> attr=$2
> @@ -115,6 +165,31 @@ check_files_in_repo () {
> compare_files $crlfnul ${pfx}CRLF_nul.txt
> }
> +check_in_repo_NNO () {
> + crlf=$1
> + attr=$2
> + lfname=$3
> + crlfname=$4
> + lfmixcrlf=$5
> + lfmixcr=$6
> + crlfnul=$7
> + pfx=NNO_${crlf}_attr_${attr}_
> + test_expect_success "compare_files $lfname ${pfx}LF.txt" '
> + compare_files $lfname ${pfx}LF.txt
> + '
> + test_expect_success "compare_files $crlfname ${pfx}CRLF.txt" '
> + compare_files $crlfname ${pfx}CRLF.txt
> + '
> + test_expect_success "compare_files $lfmixcrlf ${pfx}CRLF_mix_LF.txt" '
> + compare_files $lfmixcrlf ${pfx}CRLF_mix_LF.txt
> + '
> + test_expect_success "compare_files $lfmixcr ${pfx}LF_mix_CR.txt" '
> + compare_files $lfmixcr ${pfx}LF_mix_CR.txt
> + '
> + test_expect_success "compare_files $crlfnul ${pfx}CRLF_nul.txt" '
> + compare_files $crlfnul ${pfx}CRLF_nul.txt
> + '
> +}
> checkout_files () {
> eol=$1
> @@ -169,7 +244,11 @@ test_expect_success 'setup master' '
> printf "line1\nline2\rline3" >LF_mix_CR &&
> printf "line1\r\nline2\rline3" >CRLF_mix_CR &&
> printf "line1Q\r\nline2\r\nline3" | q_to_nul >CRLF_nul &&
> - printf "line1Q\nline2\nline3" | q_to_nul >LF_nul
> + printf "line1Q\nline2\nline3" | q_to_nul >LF_nul &&
> + create_NNO_files CRLF_mix_LF CRLF_mix_LF CRLF_mix_LF CRLF_mix_LF CRLF_mix_LF &&
> + git -c core.autocrlf=false add NNO_*.txt &&
> + git commit -m "mixed line endings" &&
> + test_tick
> '
> @@ -191,46 +270,72 @@ else
> WAMIX=CRLF_LF
> fi
> -# attr LF CRLF repoMIX CRLFmixLF
> LFmixCR CRLFNUL
> +# attr LF CRLF CRLFmixLF LFmixCR CRLFNUL
> test_expect_success 'commit files empty attr' '
> - commit_check_warn false "" "" "" "" "" ""
> "" &&
> - commit_check_warn true "" "LF_CRLF" "" "LF_CRLF" "LF_CRLF" ""
> "" &&
> - commit_check_warn input "" "" "CRLF_LF" "CRLF_LF" "CRLF_LF" ""
> ""
> + commit_check_warn false "" "" "" "" "" "" &&
> + commit_check_warn true "" "LF_CRLF" "" "LF_CRLF" "" "" &&
> + commit_check_warn input "" "" "CRLF_LF" "CRLF_LF" "" ""
> '
> test_expect_success 'commit files attr=auto' '
> - commit_check_warn false "auto" "$WILC" "$WICL" "$WAMIX" "$WAMIX" ""
> "" &&
> - commit_check_warn true "auto" "LF_CRLF" "" "LF_CRLF" "LF_CRLF" ""
> "" &&
> - commit_check_warn input "auto" "" "CRLF_LF" "CRLF_LF" "CRLF_LF" ""
> ""
> + commit_check_warn false "auto" "$WILC" "$WICL" "$WAMIX" "" "" &&
> + commit_check_warn true "auto" "LF_CRLF" "" "LF_CRLF" "" "" &&
> + commit_check_warn input "auto" "" "CRLF_LF" "CRLF_LF" "" ""
> '
> test_expect_success 'commit files attr=text' '
> - commit_check_warn false "text" "$WILC" "$WICL" "$WAMIX" "$WAMIX" "$WILC"
> "$WICL" &&
> - commit_check_warn true "text" "LF_CRLF" "" "LF_CRLF" "LF_CRLF"
> "LF_CRLF" "" &&
> - commit_check_warn input "text" "" "CRLF_LF" "CRLF_LF" "CRLF_LF" ""
> "CRLF_LF"
> + commit_check_warn false "text" "$WILC" "$WICL" "$WAMIX" "$WILC" "$WICL"
> &&
> + commit_check_warn true "text" "LF_CRLF" "" "LF_CRLF" "LF_CRLF" ""
> &&
> + commit_check_warn input "text" "" "CRLF_LF" "CRLF_LF" "" "CRLF_LF"
> '
> test_expect_success 'commit files attr=-text' '
> - commit_check_warn false "-text" "" "" "" "" ""
> "" &&
> - commit_check_warn true "-text" "" "" "" "" ""
> "" &&
> - commit_check_warn input "-text" "" "" "" "" ""
> ""
> + commit_check_warn false "-text" "" "" "" "" "" &&
> + commit_check_warn true "-text" "" "" "" "" "" &&
> + commit_check_warn input "-text" "" "" "" "" ""
> '
> test_expect_success 'commit files attr=lf' '
> - commit_check_warn false "lf" "" "CRLF_LF" "CRLF_LF" "CRLF_LF" ""
> "CRLF_LF" &&
> - commit_check_warn true "lf" "" "CRLF_LF" "CRLF_LF" "CRLF_LF" ""
> "CRLF_LF" &&
> - commit_check_warn input "lf" "" "CRLF_LF" "CRLF_LF" "CRLF_LF" ""
> "CRLF_LF"
> + commit_check_warn false "lf" "" "CRLF_LF" "CRLF_LF" ""
> "CRLF_LF" &&
> + commit_check_warn true "lf" "" "CRLF_LF" "CRLF_LF" ""
> "CRLF_LF" &&
> + commit_check_warn input "lf" "" "CRLF_LF" "CRLF_LF" "" "CRLF_LF"
> '
> test_expect_success 'commit files attr=crlf' '
> - commit_check_warn false "crlf" "LF_CRLF" "" "LF_CRLF" "LF_CRLF"
> "LF_CRLF" "" &&
> - commit_check_warn true "crlf" "LF_CRLF" "" "LF_CRLF" "LF_CRLF"
> "LF_CRLF" "" &&
> - commit_check_warn input "crlf" "LF_CRLF" "" "LF_CRLF" "LF_CRLF"
> "LF_CRLF" ""
> + commit_check_warn false "crlf" "LF_CRLF" "" "LF_CRLF" "LF_CRLF" "" &&
> + commit_check_warn true "crlf" "LF_CRLF" "" "LF_CRLF" "LF_CRLF" "" &&
> + commit_check_warn input "crlf" "LF_CRLF" "" "LF_CRLF" "LF_CRLF" ""
> '
> +# attr LF CRLF CRLFmixLF LF_mix_CR CRLFNUL
> +commit_chk_wrnNNO false "" "" "" "" "" ""
> +commit_chk_wrnNNO true "" "LF_CRLF" "" "" "" ""
> +commit_chk_wrnNNO input "" "" "" "" "" ""
> +
> +
> +commit_chk_wrnNNO false "auto" "$WILC" "$WICL" "$WAMIX" "" ""
> +commit_chk_wrnNNO true "auto" "LF_CRLF" "" "LF_CRLF" "" ""
> +commit_chk_wrnNNO input "auto" "" "CRLF_LF" "CRLF_LF" "" ""
> +
> +commit_chk_wrnNNO false "text" "$WILC" "$WICL" "$WAMIX" "$WILC" "$WICL"
> +commit_chk_wrnNNO true "text" "LF_CRLF" "" "LF_CRLF" "LF_CRLF" ""
> +commit_chk_wrnNNO input "text" "" "CRLF_LF" "CRLF_LF" ""
> "CRLF_LF"
> +
> +commit_chk_wrnNNO false "-text" "" "" "" "" ""
> +commit_chk_wrnNNO true "-text" "" "" "" "" ""
> +commit_chk_wrnNNO input "-text" "" "" "" "" ""
> +
> +commit_chk_wrnNNO false "lf" "" "CRLF_LF" "CRLF_LF" ""
> "CRLF_LF"
> +commit_chk_wrnNNO true "lf" "" "CRLF_LF" "CRLF_LF" ""
> "CRLF_LF"
> +commit_chk_wrnNNO input "lf" "" "CRLF_LF" "CRLF_LF" ""
> "CRLF_LF"
> +
> +commit_chk_wrnNNO false "crlf" "LF_CRLF" "" "LF_CRLF" "LF_CRLF" ""
> +commit_chk_wrnNNO true "crlf" "LF_CRLF" "" "LF_CRLF" "LF_CRLF" ""
> +commit_chk_wrnNNO input "crlf" "LF_CRLF" "" "LF_CRLF" "LF_CRLF" ""
> +
> test_expect_success 'create files cleanup' '
> rm -f *.txt &&
> - git reset --hard
> + git -c core.autocrlf=false reset --hard
> '
> test_expect_success 'commit empty gitattribues' '
> @@ -257,6 +362,24 @@ test_expect_success 'commit -text' '
> check_files_in_repo input "-text" LF CRLF CRLF_mix_LF LF_mix_CR CRLF_nul
> '
> +# attr LF CRLF CRLF_mix_LF LF_mix_CR
> CRLFNUL
> +check_in_repo_NNO false "" LF CRLF CRLF_mix_LF LF_mix_CR
> CRLF_nul
> +check_in_repo_NNO true "" LF CRLF CRLF_mix_LF LF_mix_CR
> CRLF_nul
> +check_in_repo_NNO input "" LF CRLF CRLF_mix_LF LF_mix_CR
> CRLF_nul
> +
> +check_in_repo_NNO false "auto" LF LF LF LF_mix_CR
> CRLF_nul
> +check_in_repo_NNO true "auto" LF LF LF LF_mix_CR
> CRLF_nul
> +check_in_repo_NNO input "auto" LF LF LF LF_mix_CR
> CRLF_nul
> +
> +check_in_repo_NNO false "text" LF LF LF LF_mix_CR LF_nul
> +check_in_repo_NNO true "text" LF LF LF LF_mix_CR LF_nul
> +check_in_repo_NNO input "text" LF LF LF LF_mix_CR LF_nul
> +
> +check_in_repo_NNO false "-text" LF CRLF CRLF_mix_LF LF_mix_CR
> CRLF_nul
> +check_in_repo_NNO true "-text" LF CRLF CRLF_mix_LF LF_mix_CR
> CRLF_nul
> +check_in_repo_NNO input "-text" LF CRLF CRLF_mix_LF LF_mix_CR
> CRLF_nul
> +
> +
> ################################################################################
> # Check how files in the repo are changed when they are checked out
> # How to read the table below:
prev parent reply other threads:[~2015-10-09 22:12 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-09 2:58 [PATCH] t0027: Improve test for not-normalized files Torsten Bögershausen
2015-10-09 22:12 ` Junio C Hamano [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqk2qvofn1.fsf@gitster.mtv.corp.google.com \
--to=gitster@pobox$(echo .)com \
--cc=git@vger$(echo .)kernel.org \
--cc=tboegi@web$(echo .)de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox