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

Re: [PATCH 4/4] Replace fork_with_pipe in bundle with run_command

From: Shawn O. Pearce <hidden>
Date: 2016-06-15 22:42:59

Junio C Hamano [off-list ref] wrote:
"Shawn O. Pearce" [off-list ref] writes:
quoted
 	return status;
As the variable 'status' is not used anymore, I think this should return 0.
Indeed.  I replaced that patch with another (same subject) in my
latest series.  Because leaving that 'status' variable in caused
a failure in the test suite when trying to create a bundle.

  fixed:  http://article.gmane.org/gmane.comp.version-control.git/42087
  borked: http://article.gmane.org/gmane.comp.version-control.git/42056

You quoted the borked one.  Please drop it and apply the fixed one.

-- 
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