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

Re: revision number output in 'svn mergeinfo' command

From: Karl Fogel <kfogel_at_red-bean.com>
Date: Sat, 05 Apr 2008 10:35:40 -0400

"Paul Burba" <ptburba_at_gmail.com> writes:
>> {
>> - svn_cmdline_printf(pool, "%ld\n", log_entry->revision);
>> + svn_cmdline_printf(pool, "r%ld\n", log_entry->revision);
>> return SVN_NO_ERROR;
>> }
>
> We've talked so much, for so long, about 'svn mergeinfo' that I don't
> recall if anyone ever had a reason not to include the 'r' prefix on
> the output. I don't see why not though, it fits with how we typically
> display revisions on the command line.

In my posts in that thread, I simply assumed the "r" would be there, but
I never actually gave a transcript-style example, so there was no way
for us to detect if someone else assumed otherwise.

Note also http://subversion.tigris.org/issues/show_bug.cgi?id=2366.

I'm going to commit this and propose for backport.

-Karl

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-04-05 16:35:53 CEST

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.