public inbox for developer@lists.illumos.org (since 2011-08)
 help / color / mirror / Atom feed
* [REVIEW] 16437 workaround gerrit misclassifying awk test T.misc as binary
@ 2024-04-02  0:46 Bill Sommerfeld
  0 siblings, 0 replies; only message in thread
From: Bill Sommerfeld @ 2024-04-02  0:46 UTC (permalink / raw)
  To: developer

[-- Attachment #1: Type: text/plain, Size: 1078 bytes --]

Issue: https://www.illumos.org/issues/16437
CR: https://code.illumos.org/c/illumos-gate/+/3395
Diff: attached

While responding to code review feedback on #16435 I found that
Gerrit's implementation of git viewed the awk test
usr/src/test/util-tests/tests/awk/tests/T.misc as a non-diffable
binary file.

Some digging around determined that the trigger for problem was an
ASCII carriage return in the middle of a line; with that changed in
the script to the backslash-r equivalent, Gerrit no longer has a
problem creating diffs in subsequent reviews.

Unfortunately this means that Gerrit will not be able to directly 
display diffs for this change; if you want to see what I've uploaded to 
the change, your best bet is to:

	git fetch ssh://code.illumos.org/illumos-gate \
		refs/changes/95/3395/1 && git show FETCH_HEAD

I've attached a diff I've generated - it's perhaps best read by feeding 
it through "cat -vt" as it involves a change to a line that contains a 
litany of embedded ASCII carriage-control codes.

Thanks in advance for your review.

[-- Attachment #2: bug16437.diff --]
[-- Type: text/x-patch, Size: 801 bytes --]

commit f105fa0bf0467b20bfc3f3ef5e816aba2b40c38c
Author: Bill Sommerfeld <sommerfeld@hamachi.org>
Date:   Mon Apr 1 16:07:41 2024 -0700

    16437 workaround gerrit misclassifying awk test T.misc as binary
    
    Change-Id: Ia52a48a68bd02734e0b8bd39c4957c3e4179a301

diff --git a/usr/src/test/util-tests/tests/awk/tests/T.misc b/usr/src/test/util-tests/tests/awk/tests/T.misc
index 04e9794de9..062f93b19b 100755
--- a/usr/src/test/util-tests/tests/awk/tests/T.misc
+++ b/usr/src/test/util-tests/tests/awk/tests/T.misc
@@ -307,7 +307,7 @@ $AWK '#
 ' >/dev/null 2> $TEMP0
 grep 'line [45]' $TEMP0 >/dev/null || fail '\aBAD: T.misc lineno'
 
-echo 'x\f\r\b\v\a\y' > $TEMP1
+echo 'x\f\r\b\v\a\y' > $TEMP1
 $AWK 'BEGIN { print "x\f\r\b\v\a\\y" }' > $TEMP2
 cmp -s $TEMP1 $TEMP2 || fail '\aBAD: T.misc weird chars'
 

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-04-02  0:46 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-02  0:46 [REVIEW] 16437 workaround gerrit misclassifying awk test T.misc as binary Bill Sommerfeld

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).