Date: Thu, 19 Aug 2010 12:42:58 -0600 (MDT) From: "M. Warner Losh" <imp@bsdimp.com> To: andre@freebsd.org Cc: svn-src-head@freebsd.org, adrian@freebsd.org, src-committers@freebsd.org, svn-src-all@freebsd.org Subject: Re: svn commit: r211503 - head/sys/mips/atheros Message-ID: <20100819.124258.885796242215807555.imp@bsdimp.com> In-Reply-To: <4C6D6FD7.7060106@freebsd.org> References: <4C6D2933.9020306@freebsd.org> <20100819.112040.914206331691540228.imp@bsdimp.com> <4C6D6FD7.7060106@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
In message: <4C6D6FD7.7060106@freebsd.org>
Andre Oppermann <andre@freebsd.org> writes:
: On 19.08.2010 19:20, M. Warner Losh wrote:
: > In message:<4C6D2933.9020306@freebsd.org>
: > Andre Oppermann<andre@freebsd.org> writes:
: > : On 19.08.2010 13:53, Adrian Chadd wrote:
: > :> Author: adrian
: > :> Date: Thu Aug 19 11:53:55 2010
: > :> New Revision: 211503
: > :> URL: http://svn.freebsd.org/changeset/base/211503
: > :>
: > :> Log:
: > :> Add some initial AR724X chipset support.
: > :>
: > :> This is untested but should at least allow an AR724X to boot.
: > :
: > : Isn't this something that should be done on a project branch and
: > : merged back when in a good working state?
: >
: > We don't have a branch for mips stuff these days. This stuff is OK,
: > since the AR724X is just being rolled out right now... For non AR724x
: > systems, this won't affect anything...
:
: I was more concerned about tree breakage for non-tested code. When
: developing something bleeding edge it is often useful to just commit
: some stuff and have it sorted out later. In head this is more
: dangerous. A small AR724X development branch would be ideal for
: this. Branching is cheap with SVN these days.
Merging isn't that cheap with svn. The svn:mergeinfo properties make
them a pita. Given that this code won't break anything, except
possibly the now-unsupported AR724x, I think a branch would be
overkill. We'd have to drag that branch along all the time until we
can get actual hardware to test it on, which is a high overhead.
Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100819.124258.885796242215807555.imp>