Re: [PATCH] Add an option for message logs at commit time

["Hyrum K. Wright" <hy...@mail.utexas.edu>]

Demange_2, Philippe wrote:
> The default commit message is not precise enough and does not really reflects the context of the import operation.
> 
> Here is a patch allowing to do it by the mean of a new option -m

Thanks for the patch!  Unfortunately, this patch is difficult to review
for a couple of reasons:  it isn't in unified diff format (which means
it probably wasn't generated using 'svn diff', the preferred way to
produced patches), and it doesn't have a log message.

Please regenerate your patch, following the patch submission guidelines
found here:
http://subversion.tigris.org/hacking.html#patches

Thanks!

-Hyrum

RE: [PATCH] Add an option for message logs at commit time

["Demange_2, Philippe" <ph...@philips.com>]

Log message:

   * contrib/client-side/svn_load_dirs/svn_load_dirs.pl.in
     (GetOptions, usage): Add the message log parameter as an option.
     (main, commint_renames): Replace the default message log by the user's one if needed, when committing in.


Hope this is enough.

Philippe.

-----Original Message-----
From: Hyrum K. Wright [mailto:hyrum_wright@mail.utexas.edu]
Sent: mardi 11 décembre 2007 00:02
To: Demange_2, Philippe
Cc: 'dev@subversion.tigris.org'
Subject: Re: [PATCH] Add an option for message logs at commit time

Demange_2, Philippe wrote:
> The default commit message is not precise enough and does not really reflects the context of the import operation.
>
> Here is a patch allowing to do it by the mean of a new option -m

Thanks for the patch!  Unfortunately, this patch is difficult to review for a couple of reasons:  it isn't in unified diff format (which means it probably wasn't generated using 'svn diff', the preferred way to produced patches), and it doesn't have a log message.

Please regenerate your patch, following the patch submission guidelines found here:
http://subversion.tigris.org/hacking.html#patches

Thanks!

-Hyrum