Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 21 Oct 2021 06:13:25 +0000 (UTC)
From:      Baptiste Daroussin <bapt@etoilebsd.net>
To:        Gleb Smirnoff <glebius@freebsd.org>
Cc:        Emmanuel Vadot <manu@bidouilliste.com>, Slawa Olhovchenkov <slw@zxy.spb.ru>, src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: 225639e7db68 - main - vt: Disable bell by default
Message-ID:  <d6f060a0-91a5-4f4f-a0fe-c3b67ce0f08a@etoilebsd.net>
In-Reply-To: <YXBNqdMe2y1bJj43@FreeBSD.org>
References:  <202110190738.19J7cFtw067994@gitrepo.freebsd.org> <20211019112132.GC80160@zxy.spb.ru> <20211019133700.77c0ca7ac488eb99df50551b@bidouilliste.com> <20211020095206.GA77338@zxy.spb.ru> <20211020134237.17596440e1fabda5d40f6985@bidouilliste.com> <YXBNqdMe2y1bJj43@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help

20 oct. 2021 19:11:15 Gleb Smirnoff <glebius@freebsd.org>:

> =C2=A0 Emmanuel,
>
> On Wed, Oct 20, 2021 at 01:42:37PM +0200, Emmanuel Vadot wrote:
> E>=C2=A0 Stop using POLA for "This is a change that I don't agree with".
>
> A change that just changes the system behaviour and nothing else,
> is a POLA violation.
>
> E>=C2=A0 In the past weeks we had commits that changed mixer(8) cli in a
> E> non-backward compatible way and changed the root shell. For both you
> E> haven't complain so it means that either you don't care or you agree
> E> with those changes.
>
> Mixer and root shell weren't like your change. Mixer is a complete
> rewrite which has some technical background to justify it. Root shell
> is also different to what you did. It is not flip of a binary switch.
>
> E>=C2=A0 I agree that the bell is clearly a user preference and also I co=
uld
> E> have left the review opened for more time (but I've been there and
> E> usually you have review/feedback the first day and after it's silent).
> E> I wish there was an easy way for developer (and community member) to
> E> create a poll for changes like that but we don't have one right now.
> E>=C2=A0 Now based on the number of people who reviewed and the people wh=
o sent
> E> me private message to thanks me for this change I think that more
> E> people are annoyed by the bell than people who wants it and we want
> E> sane default that most users agrees on in base FreeBSD. If you don't
> E> agree with the defaults we have a way to change them.
>
> To me this looks like abuse of committer priveleges. I got commit bit,
> so I can change the defaults.
>
> Speaking of the bell itself. It is of course very annoying on some
> laptops, especially if the laptop is used by someone else, not you.
> But it is extremely useful when working in a loud server room on
> a console, reviving a crashed machine, usually in a very stressful
> environment. Gives feeling of machine being responsive or not being
> responsive.
>
> So it is more a question of server vs desktop default. Okay, let's
> ignore the biased opinion that FreeBSD is a server OS. But I would
> assert that vt(4) is a way more used on servers, rather than on
> laptops. If somebody is using vt(4) instead of X on a laptop, then
> this person is special. Either really impared, as Shawn suggested,
> or just a geek who hates GUI. In the latter case, they should tweak
> the configuration.
>
> --
> Gleb Smirnoff

Interrestingly my experience in datacenters is the exact opposite, the firs=
t thing everyone I know is doing when installing freebsd in datacenter is d=
isabling the bell, because the last thing you want is add noise on top of t=
he noise. And use the bell only for very specific purpose.

So your claim that it is server usage vs laptop is not true.

It seems that there are very different habbits their and no way to really p=
oll what the majority would like.

Note that I am for both laptop/desktop and servers part of the people consi=
dering the default should be off as well.

Best regards,
Bapt






Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?d6f060a0-91a5-4f4f-a0fe-c3b67ce0f08a>