Re: [PATCH 01/13] fast-import: add the 'done' command
- From
Sverre Rabbelier <srabbelier@gmail.com>
- Date
- Aug 30, 2010, 00:30 UTC
- Message-ID
- <AANLkTik3H6hVgViAX5ur9Tq4tFQ9mJEPuTmAwcrLStvU@mail.gmail.com>
- In-Reply-To
- <20100829223218.GL1890@burratino>
Heya,
On Sun, Aug 29, 2010 at 17:32, Jonathan Nieder <jrnieder@gmail.com> wrote:
> where an error might go undiagnosed (and any error message drowned out > by the summary fast-import writes at the end). > > Will think more.
As far as I'm concerned that should be the responsibility of the importer. If there is an error it should make sure not to drown the error message with it's summary. I think it does a pretty good job at that already though, doesn't it? It even saves a log file to try and help you diagnose what went wrong.
-- Cheers, Sverre Rabbelier