Mailing-List: contact dev-help@tomcat.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Tomcat Developers List" <dev@tomcat.apache.org>
Message-ID: <4FCD014E.9030809@apache.org>
Date: Mon, 04 Jun 2012 19:41:18 +0100
From: Mark Thomas <markt@apache.org>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7;
 rv:12.0) Gecko/20120428 Thunderbird/12.0.1
MIME-Version: 1.0
To: Tomcat Developers List <dev@tomcat.apache.org>
Subject: Re: svn commit: r1345848 - in /tomcat/tc7.0.x/trunk: ./
 java/org/apache/catalina/deploy/NamingResources.java
 java/org/apache/tomcat/util/net/AbstractEndpoint.java
References: <20120604072747.39040238896F@eris.apache.org>
 <049c01cd426b$ea407880$bec16980$@hanik.com>
In-Reply-To: <049c01cd426b$ea407880$bec16980$@hanik.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

On 04/06/2012 17:05, Filip Hanik (mailing lists) wrote:
> 
> Ok, this is back to code discipline. At some point, we'd have to
> expect that more users will adopt v7 in production (I'm still seeing
> 80%+ being on v6), at that point, commits like this do nothing except
> pollute the diffs.

It depends which diffs you are looking at. If you are checking trunk
against 7.0.x to check that the back-port of a fix hasn't been missed
then having trunk and 7.0.x as closely aligned as possible is helpful.
If you are checking the changes in 7.0.x then formatting changes don't
help. It all depends on your point of view. I would also add that
consistently formatted code is easier to read and less likely to be misread.

> Servlet 3.1 has released a draft, where I'd expect that trunk is
> headed. There is no reason for v7 to continue to be an exact mirror
> of trunk, especially from a formatting point of view.

That work hasn't started in Tomcat yet so at the moment there is no
driver for them to diverge. That will probably change over the next few
months.

> While this makes the code prettier, it makes it a lot harder to trace
> regressions.

Not just prettier (see above) but I agree regarding regressions.

> I'd suggest we start treating this as a stable branch, stable in my
> perception is that it's a branch that I use to support production
> environments, and that I can use to trace regressions and fixes.

The obvious change would be to move 7.0.x to RTC rather than CTR. That
inevitably slows down the pace of development. We as a community need to
decide if that is what we want to do. If you want to propose that, I
suggest you start a new thread on that specific topic.

Mark

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org