[Svnmerge] Slow generation of commit file

[Giovanni Bajo]

Blair Zajac <blair at orcaware.com> wrote:

> Well, when I see -f - or -f-, I see the second - as a symbol for
> standard output, as that's the common meaning for a file named '-'
> from tar and other programs.  But I don't have a good idea on how to
> turn off logging.

Doh! For some reason, I convinced myself that '-' was only used as a shortcut
for stdin, and never for stdout.


> That's great that we have a larger test suite to work with.
>
>> I believe we should tweak construct_merged_log_message so to invoke
>> "svn log" only once, including the whole range of revisions we need
>> a log for. Some past experiments have shown me that "svn log" has
>> slow setup and fast execution, so it's much better to ask more data,
>> rather than calling it with more resolution. Especially for normal
>> cases, I don't expect wild ranges in revisions being merged, so it
>> looks like the best solution and the easiest to implement. Comments?
>
> Sounds like a good idea.  If we were using the Python swig bindings,
> then this would be a very safe operation, as the bindings would
> distinctly separate the log messages.  With this method, I think we
> should switch to using the --xml option to svn log, otherwise we get
> into a revision separator parsing game, which I would rather avoid.
>
> Regards,
> Blair

Giovanni Bajo