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

Re: svn checkpoint: r27139 - checkpoints

From: Karl Fogel <kfogel_at_red-bean.com>
Date: 2007-10-14 15:55:04 CEST

"Justin Erenkrantz" <justin@erenkrantz.com> writes:
> On Oct 13, 2007 12:55 AM, Karl Fogel <kfogel@red-bean.com> wrote:
>> How about: "anything you would have formerly attached to an issue as a
>> patch", for starters? Now we can just say a revision number instead.
>
> But, then, it's impossible to review it via email because we don't have diffs.

No objection here to turning diffs back on for /checkpoints.

> All I'm suggesting is that we turn back on diffs and socially tell
> folks, "don't be a turd" about giving nit-pick comments to anything in
> /checkpoints. Yet, if the intent of /checkpoints is that
> occassionally folks will be asked to look in it, then that implies we
> need easy ways to review what is there. -- justin

Agreed. (Sorry, I didn't grok the core of your complaint before.)

Now, one usage I forsee is

   $ svn cp -m "checkpointing current code" . http://.../checkpoints/

In that case, will it show just the diff against the files' baselines
(what I hope), or will it show the entire tree (what I fear)?

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Oct 14 15:55:21 2007

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