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

[PATCH] svn diff -cHEAD

From: Charles Acknin <charlesacknin_at_gmail.com>
Date: 2007-08-09 16:16:21 CEST

Last night I bumped into this:

% svn diff -cHEAD
subversion/svn/main.c:1103: (apr_err=205000)
svn: Non-numeric change argument given to -c

It looked like a trivial fix but it turned out to be quite tricky as
Subversion doesn't have any revision keyword for 'HEAD-1'.

It doesn't break anything, i.e. passes all diff_tests.

(Warning: this is hacky-and-dirty. I'm lacking time to polish this
off as I'm late with my SoC. Additionally one can find the time to
implement -c -HEAD too.)

[[[
* subversion/svn/main.c
  (main): use svn_opt_parse_revision() instead of strtol() to read '-c''s
  revision argument (numeric or HEAD) and adapt to support HEAD argument.

* subversion/libsvn_subr/opt.c
  (parse_one_rev): add a case for negative numbers as 'diff -c N' is now using
  this function.

* subversion/libsvn_client/diff.c
  (diff_prepare_repos_repos): set revisions accordingly to our hack/hint from
  main() and save one roundtrip.
]]]

[[[
Index: subversion/svn/main.c
===================================================================
--- subversion/svn/main.c (revision 25966)
+++ subversion/svn/main.c (working copy)
@@ -1082,8 +1082,6 @@
         break;
       case 'c':
         {
- char *end;
- svn_revnum_t changeno;
           if (opt_state.start_revision.kind != svn_opt_revision_unspecified)
             {
               err = svn_error_create
@@ -1099,35 +1097,71 @@
                  _("Can't specify -c with --old"));
               return svn_cmdline_handle_exit_error(err, pool, "svn: ");
             }
- changeno = strtol(opt_arg, &end, 10);
- if (end == opt_arg || *end != '\0')
+ if (svn_opt_parse_revision(&(opt_state.start_revision),
+ &(opt_state.end_revision),
+ opt_arg, pool) != 0)
             {
- err = svn_error_create(SVN_ERR_CL_ARG_PARSING_ERROR, NULL,
- _("Non-numeric change argument
given to -c"));
+ err = svn_utf_cstring_to_utf8(&utf8_opt_arg, opt_arg, pool);
+ if (! err)
+ err = svn_error_createf
+ (SVN_ERR_CL_ARG_PARSING_ERROR, NULL,
+ _("Syntax error in revision argument '%s'"),
+ utf8_opt_arg);
               return svn_cmdline_handle_exit_error(err, pool, "svn: ");
             }
- if (changeno == 0)
+ if (opt_state.end_revision.kind != svn_opt_revision_unspecified)
             {
- err = svn_error_create(SVN_ERR_CL_ARG_PARSING_ERROR, NULL,
- _("There is no change 0"));
+ err = svn_error_create
+ (SVN_ERR_CL_ARG_PARSING_ERROR, NULL,
+ _("Multiple revision arguments encountered; "
+ "can't specify N:M range with -c, "
+ "try '-r N:M' instead"));
               return svn_cmdline_handle_exit_error(err, pool, "svn: ");
             }
- /* Figure out the range:
- -c N -> -r N-1:N
- -c -N -> -r N:N-1 */
- if (changeno > 0)
+ if (opt_state.start_revision.kind == svn_opt_revision_number)
             {
- opt_state.start_revision.value.number = changeno - 1;
- opt_state.end_revision.value.number = changeno;
+ svn_revnum_t *change_rev
+ = &opt_state.start_revision.value.number;
+
+ /* Figure out the range:
+ -c N -> -r N-1:N
+ -c -N -> -r N:N-1 */
+ if (*change_rev == 0)
+ {
+ err = svn_error_create(SVN_ERR_CL_ARG_PARSING_ERROR, NULL,
+ _("There is no change 0"));
+ return svn_cmdline_handle_exit_error(err, pool, "svn: ");
+ }
+ else if (*change_rev > 0)
+ {
+ opt_state.end_revision.value.number = *change_rev;
+ opt_state.start_revision.value.number = *change_rev - 1;
+ }
+ else
+ {
+ *change_rev = -(*change_rev);
+ opt_state.end_revision.value.number = *change_rev - 1;
+ opt_state.start_revision.value.number = *change_rev;
+ }
+ opt_state.end_revision.kind = svn_opt_revision_number;
             }
+ else if (opt_state.start_revision.kind == svn_opt_revision_head)
+ {
+ /* This is a hack for '-c HEAD': as there's no 'HEAD-1'
+ * rev-keyword, set both {start,end}_revision.kind to HEAD
+ * and their revision value to a dummy so we can
+ * differentiate from a true HEAD/HEAD repos-diff later on. */
+ opt_state.end_revision.kind = svn_opt_revision_head;
+ opt_state.end_revision.value.number = SVN_IGNORED_REVNUM;
+ opt_state.start_revision.value.number = SVN_IGNORED_REVNUM;
+ }
           else
             {
- changeno = -changeno;
- opt_state.start_revision.value.number = changeno;
- opt_state.end_revision.value.number = changeno - 1;
+ err = svn_error_create
+ (SVN_ERR_CL_ARG_PARSING_ERROR, NULL,
+ _("-c argument must either be numeric or HEAD"));
+ return svn_cmdline_handle_exit_error(err, pool, "svn: ");
             }
- opt_state.start_revision.kind = svn_opt_revision_number;
- opt_state.end_revision.kind = svn_opt_revision_number;
           used_change_arg = TRUE;
         }
         break;
Index: subversion/libsvn_client/diff.c
===================================================================
--- subversion/libsvn_client/diff.c (revision 25966)
+++ subversion/libsvn_client/diff.c (working copy)
@@ -1022,11 +1022,28 @@
        _("'%s' was not found in the repository at revision %ld"),
        drr->url2, drr->rev2);

- /* Do the same for the first target. */
   SVN_ERR(svn_ra_reparent(ra_session, drr->url1, pool));
- SVN_ERR(svn_client__get_revision_number
- (&drr->rev1, ra_session, params->revision1,
- (params->path1 == drr->url1) ? NULL : params->path1, pool));
+
+ /* When the two revisions are both HEAD, avoid a useless second
+ * roundtrip to the server and copy the value from rev2 instead. */
+ if (params->revision1->kind == svn_opt_revision_head
+ && params->revision2->kind == params->revision1->kind)
+ {
+ /* This is a hack to get 'svn diff -cHEAD' to work. */
+ if (params->revision1->value.number == SVN_IGNORED_REVNUM
+ && params->revision2->value.number == SVN_IGNORED_REVNUM)
+ drr->rev1 = drr->rev2 - 1;
+ else
+ drr->rev1 = drr->rev2;
+ }
+ else
+ {
+ /* Do the same for the first target. */
+ SVN_ERR(svn_client__get_revision_number
+ (&drr->rev1, ra_session, params->revision1,
+ (params->path1 == drr->url1) ? NULL : params->path1, pool));
+ }
+
   SVN_ERR(svn_ra_check_path(ra_session, "", drr->rev1, &kind1, pool));
   if (kind1 == svn_node_none)
     return svn_error_createf
Index: subversion/libsvn_subr/opt.c
===================================================================
--- subversion/libsvn_subr/opt.c (revision 25966)
+++ subversion/libsvn_subr/opt.c (working copy)
@@ -567,6 +567,19 @@
       *end = save;
       return end;
     }
+ else if ((*str == '-' && apr_isdigit(*(str + 1))))
+ {
+ /* It's a negative number. */
+ end = str + 2;
+ while (apr_isdigit(*end))
+ end++;
+ save = *end;
+ *end = '\0';
+ revision->kind = svn_opt_revision_number;
+ revision->value.number = SVN_STR_TO_REV(str);
+ *end = save;
+ return end;
+ }
   else if (apr_isalpha(*str))
     {
       end = str + 1;
]]]

Cheers,
Charles

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Aug 9 16:14:28 2007

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.