convert: svn_sink: workaround of command line size limitation

Shun-ichi GOTO shunichi.goto at gmail.com
Fri Jan 4 21:52:03 CST 2008


2008/1/5, Maxim Dounin <mdounin at mdounin.ru>:
> Shouldn't this go into commandline class instead? I think method like
> xargs would be usable.

It'd better, I think.
But I didn't make such a change for some reasons:

* User of commandline.run() or run0() is darcs.py and subversion.py
  And svn_sink class is the only one passing many argument.

* To do it in run() or run0(), they should know the list to split,
  but I can't imagine good reasonable interface for that.

* As first step, it is enough :-)
  I hope someone improve them.

-- 
Shun-ichi GOTO


More information about the Mercurial-devel mailing list