Re: PATCH: help text: presentation of optionally-repeated arguments

In a message dated: Sat, 28 Jun 2003 22:33:18 +0200
Sander Roobol said:

>On Fri, Jun 27, 2003 at 10:33:22AM -0400, Paul L Lussier wrote:
>>
>> Filed as issue 1377 here:
>>
>> http://subversion.tigris.org/issues/show_bug.cgi?id=1377
>
>Whoah, I guess you live faster than most of us :)

Nahh, I decided 'Friday is patch issue day' :)

>In my timeframe less than a day has passed since Julian Foad posted the
>second version of the patch. It might be better to wait a while and give
>the developers a chance to review the patches so the issue tracker
>doesn't get cluttered unnecesary.

Yeah, I know. I only meant to enter an issue on 2 submitted patches,
but I got ahead of myself :)

>Also, the issue you filed looks fine but it seems you forgot to add the
>author of the patch to the cc list.

Oh, good idea! I'll do that as well.

>And it might be nice to state more clearly who made the patch.
>I just wrote 'patch by ...'

Will do!

>but actually I like your idea of adding the complete log message to the issue.

Well, I figured the log message was (should be) a good 'description'
of what the patch actually does :)

> please combine the best of those two ways :)

Okay.

>Apart from the nitpicking I must say thanks for doing this! I'll post my
>current list of patches tomorrow...

You're welcome, and please continue picking these nits, I seem to be
covered in them. Until they're all gone, I still have a lot to learn ;)

-- 
Seeya,
Paul
--
Key fingerprint = 1660 FECC 5D21 D286 F853  E808 BB07 9239 53F1 28EE
	It may look like I'm just sitting here doing nothing,
   but I'm really actively waiting for all my problems to go away.
	 If you're not having fun, you're not doing it right!
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org