]> git.ipfire.org Git - thirdparty/git.git/commitdiff
Merge branch 'jx/sideband-cleanup'
authorJunio C Hamano <gitster@pobox.com>
Thu, 8 Jul 2021 20:15:01 +0000 (13:15 -0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 8 Jul 2021 20:15:01 +0000 (13:15 -0700)
The side-band demultiplexer that is used to display progress output
from the remote end did not clear the line properly when the end of
line hits at a packet boundary, which has been corrected.  Also
comes with test clean-ups.

* jx/sideband-cleanup:
  test: refactor to use "get_abbrev_oid" to get abbrev oid
  test: refactor to use "test_commit" to create commits
  test: compare raw output, not mangle tabs and spaces
  sideband: don't lose clear-to-eol at packet boundary

1  2 
t/t6020-bundle-misc.sh

index 3140ca4fdca251cc3dda35e94ad55a6fb61b30a7,cf13bcc3c8c1485d442a5aa161a01a0de37c7930..b13e8a52a93a4e96cde54c5076c19960a4d41a2c
@@@ -80,43 -80,34 +80,46 @@@ test_commit_setvar () 
        eval $var=$oid
  }
  
 +get_abbrev_oid () {
 +      oid=$1 &&
 +      suffix=${oid#???????} &&
 +      oid=${oid%$suffix} &&
 +      if test -n "$oid"
 +      then
 +              echo "$oid"
 +      else
 +              echo "undefined-oid"
 +      fi
 +}
 +
  # Format the output of git commands to make a user-friendly and stable
  # text.  We can easily prepare the expect text without having to worry
- # about future changes of the commit ID and spaces of the output.
+ # about future changes of the commit ID.
  make_user_friendly_and_stable_output () {
        sed \
 -              -e "s/${A%${A#???????}}[0-9a-f]*/<COMMIT-A>/g" \
 -              -e "s/${B%${B#???????}}[0-9a-f]*/<COMMIT-B>/g" \
 -              -e "s/${C%${C#???????}}[0-9a-f]*/<COMMIT-C>/g" \
 -              -e "s/${D%${D#???????}}[0-9a-f]*/<COMMIT-D>/g" \
 -              -e "s/${E%${E#???????}}[0-9a-f]*/<COMMIT-E>/g" \
 -              -e "s/${F%${F#???????}}[0-9a-f]*/<COMMIT-F>/g" \
 -              -e "s/${G%${G#???????}}[0-9a-f]*/<COMMIT-G>/g" \
 -              -e "s/${H%${H#???????}}[0-9a-f]*/<COMMIT-H>/g" \
 -              -e "s/${I%${I#???????}}[0-9a-f]*/<COMMIT-I>/g" \
 -              -e "s/${J%${J#???????}}[0-9a-f]*/<COMMIT-J>/g" \
 -              -e "s/${K%${K#???????}}[0-9a-f]*/<COMMIT-K>/g" \
 -              -e "s/${L%${L#???????}}[0-9a-f]*/<COMMIT-L>/g" \
 -              -e "s/${M%${M#???????}}[0-9a-f]*/<COMMIT-M>/g" \
 -              -e "s/${N%${N#???????}}[0-9a-f]*/<COMMIT-N>/g" \
 -              -e "s/${O%${O#???????}}[0-9a-f]*/<COMMIT-O>/g" \
 -              -e "s/${P%${P#???????}}[0-9a-f]*/<COMMIT-P>/g" \
 -              -e "s/${TAG1%${TAG1#???????}}[0-9a-f]*/<TAG-1>/g" \
 -              -e "s/${TAG2%${TAG2#???????}}[0-9a-f]*/<TAG-2>/g" \
 -              -e "s/${TAG3%${TAG3#???????}}[0-9a-f]*/<TAG-3>/g"
 +              -e "s/$(get_abbrev_oid $A)[0-9a-f]*/<COMMIT-A>/g" \
 +              -e "s/$(get_abbrev_oid $B)[0-9a-f]*/<COMMIT-B>/g" \
 +              -e "s/$(get_abbrev_oid $C)[0-9a-f]*/<COMMIT-C>/g" \
 +              -e "s/$(get_abbrev_oid $D)[0-9a-f]*/<COMMIT-D>/g" \
 +              -e "s/$(get_abbrev_oid $E)[0-9a-f]*/<COMMIT-E>/g" \
 +              -e "s/$(get_abbrev_oid $F)[0-9a-f]*/<COMMIT-F>/g" \
 +              -e "s/$(get_abbrev_oid $G)[0-9a-f]*/<COMMIT-G>/g" \
 +              -e "s/$(get_abbrev_oid $H)[0-9a-f]*/<COMMIT-H>/g" \
 +              -e "s/$(get_abbrev_oid $I)[0-9a-f]*/<COMMIT-I>/g" \
 +              -e "s/$(get_abbrev_oid $J)[0-9a-f]*/<COMMIT-J>/g" \
 +              -e "s/$(get_abbrev_oid $K)[0-9a-f]*/<COMMIT-K>/g" \
 +              -e "s/$(get_abbrev_oid $L)[0-9a-f]*/<COMMIT-L>/g" \
 +              -e "s/$(get_abbrev_oid $M)[0-9a-f]*/<COMMIT-M>/g" \
 +              -e "s/$(get_abbrev_oid $N)[0-9a-f]*/<COMMIT-N>/g" \
 +              -e "s/$(get_abbrev_oid $O)[0-9a-f]*/<COMMIT-O>/g" \
 +              -e "s/$(get_abbrev_oid $P)[0-9a-f]*/<COMMIT-P>/g" \
 +              -e "s/$(get_abbrev_oid $TAG1)[0-9a-f]*/<TAG-1>/g" \
 +              -e "s/$(get_abbrev_oid $TAG2)[0-9a-f]*/<TAG-2>/g" \
-               -e "s/$(get_abbrev_oid $TAG3)[0-9a-f]*/<TAG-3>/g" \
-               -e "s/ *\$//"
++              -e "s/$(get_abbrev_oid $TAG3)[0-9a-f]*/<TAG-3>/g"
+ }
+ format_and_save_expect () {
+       sed -e 's/Z$//' >expect
  }
  
  #            (C)   (D, pull/1/head, topic/1)