From nobody Wed Oct 20 11:42:37 2021 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 359BC17FA8E6; Wed, 20 Oct 2021 11:42:46 +0000 (UTC) (envelope-from manu@bidouilliste.com) Received: from mx.blih.net (mail.blih.net [212.83.155.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "mx.blih.net", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4HZ7yY5JWFz3Ly8; Wed, 20 Oct 2021 11:42:45 +0000 (UTC) (envelope-from manu@bidouilliste.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bidouilliste.com; s=mx; t=1634730158; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=hccGPi8jlaK8bMaxtX/0phPhC6vwEudJU0OpRlsraus=; b=CPZnfx0tUvoVdlPOPfDK3KRL2BRNPeX15V2hYxnFur7epZIUbJOksqjC2BobBJ+gSYFf/k Ai8GDYrDHv+BvlVweC4IcSUeZLKfdvMiwOr+2n6yxtRmyFWVLQAGPqNrHjyPEw1WlF3/8t QIrS9rcsVrSkr2ZMgDVXLqOSHwCbgEM= Received: from amy (lfbn-idf2-1-644-191.w86-247.abo.wanadoo.fr [86.247.100.191]) by mx.blih.net (OpenSMTPD) with ESMTPSA id 185098ab (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO); Wed, 20 Oct 2021 11:42:37 +0000 (UTC) Date: Wed, 20 Oct 2021 13:42:37 +0200 From: Emmanuel Vadot To: Slawa Olhovchenkov Cc: 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: <20211020134237.17596440e1fabda5d40f6985@bidouilliste.com> In-Reply-To: <20211020095206.GA77338@zxy.spb.ru> References: <202110190738.19J7cFtw067994@gitrepo.freebsd.org> <20211019112132.GC80160@zxy.spb.ru> <20211019133700.77c0ca7ac488eb99df50551b@bidouilliste.com> <20211020095206.GA77338@zxy.spb.ru> X-Mailer: Sylpheed 3.7.0 (GTK+ 2.24.33; amd64-portbld-freebsd14.0) List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-main@freebsd.org X-BeenThere: dev-commits-src-main@freebsd.org Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: 4HZ7yY5JWFz3Ly8 X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-ThisMailContainsUnwantedMimeParts: N On Wed, 20 Oct 2021 12:52:06 +0300 Slawa Olhovchenkov wrote: > On Tue, Oct 19, 2021 at 01:37:00PM +0200, Emmanuel Vadot wrote: > > > On Tue, 19 Oct 2021 14:21:33 +0300 > > Slawa Olhovchenkov wrote: > > > > > On Tue, Oct 19, 2021 at 07:38:15AM +0000, Emmanuel Vadot wrote: > > > > > > > The branch main has been updated by manu: > > > > > > > > URL: https://cgit.FreeBSD.org/src/commit/?id=225639e7db685a4047e384abdbc296c0e02bd147 > > > > > > > > commit 225639e7db685a4047e384abdbc296c0e02bd147 > > > > Author: Emmanuel Vadot > > > > AuthorDate: 2021-10-18 08:16:41 +0000 > > > > Commit: Emmanuel Vadot > > > > CommitDate: 2021-10-19 07:37:28 +0000 > > > > > > > > vt: Disable bell by default > > > > > > > > Bell is either useless if you're working on remote servers or really annoying > > > > when you're working with a local machine that have a loud buzzer. > > > > Switch the default to have it disable. > > > > > > For you, not for me, for example. Also, this is violate POLA. > > > > Just set kern.vt.enable_bell=1 in /etc/sysctl.conf and you will be > > happy again. I'll do some UPDATING note. > > And POLA doesn't means that we can't change a thing. > > Modify bsdinstall to set kern.vt.enable_bell=0 in /etc/sysctl.conf, > this is don't break POLA for updating existing install and be happy you. Stop using POLA for "This is a change that I don't agree with". In the past weeks we had commits that changed mixer(8) cli in a non-backward compatible way and changed the root shell. For both you haven't complain so it means that either you don't care or you agree with those changes. I agree that the bell is clearly a user preference and also I could have left the review opened for more time (but I've been there and usually you have review/feedback the first day and after it's silent). I wish there was an easy way for developer (and community member) to create a poll for changes like that but we don't have one right now. Now based on the number of people who reviewed and the people who sent me private message to thanks me for this change I think that more people are annoyed by the bell than people who wants it and we want sane default that most users agrees on in base FreeBSD. If you don't agree with the defaults we have a way to change them. Cheers, -- Emmanuel Vadot