Hi, On Sun, 15 Feb 2009, Junio C Hamano wrote: > Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: > > >> > Or maybe even better: set a variable "ret" and at the end of > >> > cmd_config(), "return !!ret;"? > >> > >> Huh? So git commands don't return negative error values? > > > > AFAICT an exit status is supposed to be between 0 and 127. > > Are you two talking about the return value from cmd_config()? > > git.c::run_builtin() already knows what to do with status codes from the > builtins to protect you from (rare) shells that do not cope with a > negative return that come from the common pattern of doing: > > return error("it is wrong in this way") > > So "negative" is not really a problem. Ooops. I missed that. Ciao, Dscho -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html