Thread (3 messages) flat view 3 messages, 2 authors, 2016-06-15

Re: [RFC PATCH v4 26/26] test smart http fetch and push

From: Shawn O. Pearce <hidden>
Date: 2016-06-15 22:47:38

"Shawn O. Pearce" [off-list ref] wrote:
Junio C Hamano [off-list ref] wrote:
quoted
quoted
+		s/^Content-Length: .*$/Content-Length: xxxx/
This chomped line is so unlike you---what happened?
I was getting different Content-Lengths on different runs of the
test.  I don't know why.  Here the Content-Length is of the gzip'd
request, it shouldn't be varying with each run, but it seemed to be.
FWIW, I found the variablity here.  It was triggered by running the
test with -v or without -v, which caused clone to change its first
"want" line to include or exclude progress messages, based on the
tty-ness of stderr.

Adding a --quiet to the clone command means I can avoid this munging
and hard code the length in the test vector.  But I'm still against
doing that here in case the protocol capabilities ever change in
the future.
  
-- 
Shawn.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help