Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/distrib/utils/sysinst

On Wed, Nov 21, 2012 at 12:52:04PM +0100, Martin Husemann wrote:
> On Tue, Nov 20, 2012 at 08:17:43PM +0000, Jonathan A. Kollasch wrote:
> > Module Name:        src
> > Committed By:       jakllsch
> > Date:               Tue Nov 20 20:17:43 UTC 2012
> > 
> > Modified Files:
> >     src/distrib/utils/sysinst: bsddisklabel.c
> > 
> > Log Message:
> > Don't enable WAPBL by default. It's far too dangerous.
> Please do not make changes like this without discussion.

I have to admit, I committed this when I was in an unreasonable frame of

>  (a) I can see no PRs to this effect

This is true, I will file one.

>  (b) There is no big enough warning in the man pages

True again.

>  (c) we now have a serious performance degration in the default install
>      compared to the softdep world

AFAIR softdeps were never enabled by default when they were around,
it was always more of a "if you want performance, enable them" note

        Jonathan Kollasch

Home | Main Index | Thread Index | Old Index