On Tue, Mar 4, 2008 at 11:01 PM, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: > Hi, > > > On Tue, 4 Mar 2008, Ping Yin wrote: > > > When handling multiple modules, init/update/status command will exit > > when it fails for one submodule. This patch makes the command continue > > bypassing the failure. > > I think that you should not do this. The failure should not be ignored. > Maybe continuing, but not forgetting about the error. So: > > > > - name=$(module_name "$path") || exit > > + name=$(module_name "$path") || continue > > I do not like this. It should remember that there was something foul, and > eventually exit with that status. > The error output is not lost and is in module_name > Ciao, > Dscho > > -- Ping Yin -- 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
