-- anatoly t.
On Wed, Jun 8, 2011 at 9:34 AM, anatoly techtonik <techto...@gmail.com> wrote: > On Mon, Jun 6, 2011 at 6:32 PM, Stefan Sperling <s...@elego.de> wrote: >> On Thu, Jun 02, 2011 at 07:06:05PM +0300, anatoly techtonik wrote: >>> Attached patch fixes svnmerge history conversion script to use >>> optparse library instead of getopt. This allows further addition of >>> new options, such as --username and --password. >> >> Hi, >> >> Can you explain why you need to convert this script to optparse >> to add new options? Is this a cosmetic fix or is there a technical >> problem with adding more options via getopt? Thanks! > > 1. It makes code more clear > 2. It saves me time on learning what a getopt is The previous patch had wrong handling of naive parameter. I attach new one that also adds config-dir option handling, which allows to deal with repository permissions by specifying config that allows to cache them. Please, CC.
