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

Re: Add -c option to merge

From: Daniel Berlin <dberlin_at_dberlin.org>
Date: 2005-10-25 23:23:34 CEST

On Tue, 2005-10-25 at 17:08 -0400, Greg Hudson wrote:
> On Tue, 2005-10-25 at 16:23 -0400, Daniel Berlin wrote:
> > > > + return svn_cmdline_handle_exit_error (err, pool, "svn: ");
> > > > + }
> > > > + opt_state.end_revision.value.number = strtol (opt_arg, &end, 10);
> > > > + opt_state.end_revision.kind = svn_opt_revision_number;
> (I see a tab here. Tabs must die!)
> > > > + if (end == opt_arg || *end != '\0')
> > > > + {
> > > > + err = svn_error_create (SVN_ERR_CL_ARG_PARSING_ERROR, NULL,
> > > > + _("Non-numeric change argument given"));
> > >
> > > So the argument must be a number? You haven't explicitly mentioned that.
> >
> > Yes, how would we know what date - 1 is?
>
> -c {DATE} would mean the same thing as -r X-1:X where X is what {DATE}
> resolves to.
>
> Admittedly, "show me or merge in the change I made most recently as of
> foo time" is probably an edge case, but it does have a straightforward
> meaning in my mind.

I can accept the full revision format if that's what we want. It just
seemed weird to talk about "change 2005-06-19", when 2005-06-19 is not a
revision number.

>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Oct 25 23:29:33 2005

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