Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 19 Aug 2010 21:06:08 +0200
From:      Andre Oppermann <andre@freebsd.org>
To:        "M. Warner Losh" <imp@bsdimp.com>
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:  <4C6D80A0.4080901@freebsd.org>
In-Reply-To: <20100819.124258.885796242215807555.imp@bsdimp.com>
References:  <4C6D2933.9020306@freebsd.org>	<20100819.112040.914206331691540228.imp@bsdimp.com>	<4C6D6FD7.7060106@freebsd.org> <20100819.124258.885796242215807555.imp@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 19.08.2010 20:42, M. Warner Losh wrote:
> 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.

Didn't know that branching and merging isn't that easy with SVN after
all.  This was one of the supposed benefits for switching from CVS.
If there is no risk of head breakage I don't mind at all.

-- 
Andre



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4C6D80A0.4080901>