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

Re: [PATCH] v2 Fix svnversion message as follow-up to r922176

From: Daniel Näslund <daniel_at_longitudo.com>
Date: Fri, 2 Apr 2010 10:59:15 +0200

Mispelled Uncommitted with one 't'. This patch corrects that.

On Fri, Apr 02, 2010 at 10:36:53AM +0200, Daniel Näslund wrote:
> On Tue, Mar 30, 2010 at 11:04:13PM +0200, Stefan Sperling wrote:
> > On Tue, Mar 30, 2010 at 10:14:42PM +0200, Daniel Näslund wrote:
> > > Ping! This patch has not been reviewed!
> > >
> > > On Sun, Mar 14, 2010 at 09:38:15PM +0100, Daniel Näslund wrote:
> > > > Hi!
> > > >
> > > > The 1.6 svnversion message was "'path' not versioned, and not exported".
> > > > But on trunk more than one message has been changed. My first thought
> > > > was that we should be backward compat in our output but if changes of
> > > > those messages are ok I'm supplying one.
> > > >
> > > > In case we will use new messages, the help text must be updated. It
> > > > talks of 'exported' but those are not used in the new messages.
> > > >
> > > > [[[
> > > > After the changes in r922176, versioned but not yet committed files were
> > > > not properly detected. Fixed now!
> > > >
> > > > * subversion/svnversion/main.c
> > > > (main): Check for invalid rev nr for files and dirs.
> > > >
> > > > * subversion/tests/cmdline/svnversion_tests.py
> > > > (structural_changes): New.
> > > > (tests_list): Add new test.
> > > > ]]]
> > >
> > > > Index: subversion/svnversion/main.c
> > > > ===================================================================
> > > > --- subversion/svnversion/main.c (revision 922931)
> > > > +++ subversion/svnversion/main.c (arbetskopia)
> > > > @@ -290,6 +290,17 @@
> > > > return EXIT_FAILURE;
> > > > }
> > > >
> > > > + if (res->min_rev == -1)
> >
> > s/-1/SVN_INVALID_REVNUM/
>
> Fixed!
>
> > > > + {
> > > > + /* Local uncommited modifications, no revision info was found. */
> > > > + SVN_INT_ERR(svn_cmdline_printf(pool, _("Local uncommitted "
> > > > + "modifications, no revision "
> > > > + "information found%s"),
> > > > + no_newline ? "" : "\n"));
> >
> > "No revision information found" sounds a bit like something went
> > wrong, as in, Subversion was looking for information that was
> > expected to be present but it didn't find the information.
> > What about "Uncommitted local addition, copy, or move%s" instead?
>
> I agree with you (user messages is my weak spot). The only cases when we
> can have no revision number but the path is under version control is for
> locally added, copied or moved paths. So we tell the user about it.
>
> Can I have your +1 for commiting?

Received on 2010-04-02 11:00:31 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.