On Tue, Oct 23, 2012 at 04:04:13PM +0200, Michael Hanselmann wrote:
> 2012/10/23 Iustin Pop <ius...@google.com>:
> > + raise errors.OpExecError("Internal assertion error: please report"
> > + " this as a bug. Error message: '%s';"
> > + " location: %s" % (str(err),
> > err_info[-1]))
>
> Thanks a lot for this patch! Now please just remove the period from
> the error message (it shouldn't contain “sentences”), then LGTM.
It will anyway contain formatting - tracebacks have embedded new lines,
etc. Hence I thought that for this specific case, we should try to
format it.
I'm fine either way, let me know how to proceed.
iustin
