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

Re: svn commit: r36214 - in branches/1.6.x: . subversion/svn

From: Greg Stein <gstein_at_gmail.com>
Date: Sun, 1 Mar 2009 01:49:48 +0100

That applies to partial committers fixing "obvious" things outside
their normal allowed area. It does not apply to merging changes onto a
release branch. The description of change management for release
branches even refers to obvious fixes when talking about *voting* for
those types of changes.

And (IMO) merging changes into a release should be deferred to the RM
simply for politeness' sake.

On Sun, Mar 1, 2009 at 01:23, Arfrever Frehtes Taifersar Arahesis
<arfrever.fta_at_gmail.com> wrote:
> 2009-03-01 01:13:39 Greg Stein napisał(a):
>> You should let the RM decide what is "obvious" or not, what should be
>> merged, and to do the actual merging.
>
> http://subversion.tigris.org/hacking.html#obvious-fix
>
>> On Sat, Feb 28, 2009 at 22:28, Arfrever Frehtes Taifersar Arahesis
>> <Arfrever.FTA_at_gmail.com> wrote:
>> > Author: arfrever
>> > Date: Sat Feb 28 13:28:46 2009
>> > New Revision: 36214
>> >
>> > Log:
>> > On the '1.6.x' branch:
>> > Merge obvious fix r36213 from trunk.
>> >
>> >  * r36213
>> >   * subversion/svn/main.c
>> >     (svn_cl__cmd_table."status"): Update help message for 'svn status'.
>> >
>> > Modified:
>> >   branches/1.6.x/   (props changed)
>> >   branches/1.6.x/CHANGES   (props changed)
>> >   branches/1.6.x/subversion/svn/main.c
>> >
>> > Merged:
>> >   /trunk:r36213
>> >
>> > Modified: branches/1.6.x/subversion/svn/main.c
>> > URL: http://svn.collab.net/viewvc/svn/branches/1.6.x/subversion/svn/main.c?pathrev=36214&r1=36213&r2=36214
>> > ==============================================================================
>> > --- branches/1.6.x/subversion/svn/main.c        Sat Feb 28 13:14:42 2009        (r36213)
>> > +++ branches/1.6.x/subversion/svn/main.c        Sat Feb 28 13:28:46 2009        (r36214)
>> > @@ -2,7 +2,7 @@
>> >  * main.c:  Subversion command line client.
>> >  *
>> >  * ====================================================================
>> > - * Copyright (c) 2000-2008 CollabNet.  All rights reserved.
>> > + * Copyright (c) 2000-2009 CollabNet.  All rights reserved.
>> >  *
>> >  * This software is licensed as described in the file COPYING, which
>> >  * you should have received as part of this distribution.  The terms
>> > @@ -899,7 +899,7 @@ const svn_opt_subcommand_desc2_t svn_cl_
>> >      "    svn status\n"
>> >      "     M      wc/bar.c\n"
>> >      "    !     C wc/qaz.c\n"
>> > -     "          >   incoming edit, local missing\n"
>> > +     "          >   local missing, incoming edit upon update\n"
>> >      "    D       wc/qax.c\n"),
>> >     { 'u', 'v', 'N', opt_depth, 'q', opt_no_ignore, opt_incremental, opt_xml,
>> >       opt_ignore_externals, opt_changelist} },
>
> --
> Arfrever Frehtes Taifersar Arahesis
>

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1247676
Received on 2009-03-01 01:50:03 CET

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.