Re: [PATCH] "svnlook diff" option to filter files revisited

"Hyrum K. Wright" <hyrum_wright@mail.utexas.edu> writes:
> Perhaps you can post your patch again, following these guidelines?

Pete, I'd like to second what Hyrum said. It may seem like we're
being incredibly picky, but I assure you the intent is not to
discourage you, but rather to efficiently distribute the work of
making changes comprehensible to reviewers. If each reviewer had to
dig up the context themselves, that's N reviewers * M amount of work;
if the original poster does it, it's 1 * M. 1*M <= N*M, because there
are never fractional reviewers :-).

All the developers, committers included, follow these guidelines. For
example, see:

   http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=125951
   http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=125940
   http://svn.collab.net/viewvc/svn?view=rev&revision=24139

For guidance, you can't go wrong here:

   http://subversion.tigris.org/hacking.html#patches
   http://subversion.tigris.org/hacking.html#log-messages
   http://subversion.tigris.org/mailing-list-guidelines.html#patches

Looking forward to the repost,
-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Apr 7 20:53:29 2007