[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: anatoly techtonik <techtonik_at_gmail.com>
Date: Wed, 8 Jun 2011 11:19:50 +0300

On Wed, Jun 8, 2011 at 10:50 AM, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
>> 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
>
> It didn't reach the list.
>
> Please try *.txt extension so it has text/plain MIME type.

Done.

--
anatoly t.

Received on 2011-06-08 10:20:43 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.