Re: [PATCH] svnmerge-migrate-history-remotely.py: move from getopt to optparse

--
anatoly t.
On Wed, Jun 8, 2011 at 9:34 AM, anatoly techtonik <techtonik_at_gmail.com> wrote:
> On Mon, Jun 6, 2011 at 6:32 PM, Stefan Sperling <stsp_at_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.