svn commit: r184193 - in head/sys: arm/conf conf

Sam Leffler sam at freebsd.org
Fri Oct 24 16:33:36 UTC 2008


Marcel Moolenaar wrote:
>
> On Oct 24, 2008, at 9:14 AM, Sam Leffler wrote:
>
>>>> This is just wrong and clearly Marvel never discussed it with 
>>>> anyone working on arm systems or he'd have gotten an earful.
>>>
>>> Get your facts straight.
>>>
>> I checked with folks and you never asked for review of this change.
>
> What you mean to say is: "Oops, sorry!"

So long as "you" == marcel that is correct.  I don't know if it's the 
phase of the moon but we seem to have a spate of commits that affect 
many folks but were not peer-reviewed.  This is bad for the health of 
the project and I will complain when I see it happen.

>
>> As to not changing behaviour; the existing use of mbr+bsdlabels is 
>> stopgap and I believe everyone actively working on these platforms 
>> would like to ditch 'em.
>
> I am actively working on ARM and I need to ditch them myself.
> For that I chose to make them optional first so that people
> can keep the old behaviour if they want to. Stopgap or not,
> as ARM was before my change, those two were non-optional.
>
Glad to hear you're working on ARM.  Please talk to others actively 
involved so we can share the work.

    Sam



More information about the svn-src-head mailing list