From nobody Tue Sep 15 22:08:54 2026 X-Original-To: dev-commits-src-all@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 4hkx4h0f8Lz6rdkW for ; Tue, 15 Sep 2026 22:09:08 +0000 (UTC) (envelope-from jrtc27@jrtc27.com) Received: from mail-wr2-f12.google.com (mail-wr2-f12.google.com [74.125.225.76]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange x25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "WR4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4hkx4g286fz4Vbf for ; Tue, 15 Sep 2026 22:09:07 +0000 (UTC) (envelope-from jrtc27@jrtc27.com) Authentication-Results: mx1.freebsd.org; dkim=none; spf=pass (mx1.freebsd.org: domain of jrtc27@jrtc27.com designates 74.125.225.76 as permitted sender) smtp.mailfrom=jrtc27@jrtc27.com; dmarc=fail reason="SPF not aligned (relaxed), No valid DKIM" header.from=freebsd.org (policy=none) Received: by mail-wr2-f12.google.com with SMTP id ffacd0b85a97d-482f6351831so105611f8f.1 for ; Tue, 15 Sep 2026 15:09:07 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1789510146; x=1790114946; h=to:references:message-id:content-transfer-encoding:cc:date :in-reply-to:from:subject:mime-version:content-type:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=PKlgiRqxvIfc84L6ePuVxD1/xb25hxqF6X1TCaTa1mw=; b=yvOU3ay2gIl3MyqAPrGfBPf/jRN1ute0eOTUzgtmh1NfPq0HkfhPGaHEBnJyeGNnUZ DvqGuRV/xW+qKtL/Vcm28kj8qFswRImGMe6eiMb2G38ryaV84ssvHXOj5W+pcYdaa3YZ 6OiN3OHt2Uk/MqO3qJgyI/DprOzuWXxVWXv5f6+89QX6ZvbEecTgT4Tb3KqNxf3r7ecf j20bxwLX8yRgVbGeO9A7LxfcM8aGRUWhLfuNWpnuLjXAhA3/LvWSiiy2AEywfm/rwl0P w6NhLwbc0MsT22yvYJufLQFS6zH6DOFvSRCWDxCInpZLaoks5O/1toW7wp28zYRuootK wdZA== X-Forwarded-Encrypted: i=1; AKwUvBze4PeXfvBl1hrURjYQU+NftM0b6Gv3acniBHMVt49LQgEmfL53TDZ19r80qEho1FetDzKkcoSjZCWbQ9hAdUX79+rn@freebsd.org X-Gm-Message-State: AFuF++nE1+U89OvBg2DqLCbgNt4jY1RmShEG9wjmZoY6BbuEtR0RAiYw lEOZtklpiiCZkY+Ie/TDF3GuZFlowCBF4GxAwKcF1LcdlAU0JcNO7qg8xHKZLUQ3/1U= X-Gm-Gg: AYBFou06HgqMsqlHb/QsyzgpuOUXiOGIuzQH5y861qufrlvRa3y0PZG4v6KRhLyijSt gE12O5+5diKX0uzolOj/nWiydujCPzfO6vnCz+AkBUArENS0V1PryfSoxEUhV12nvSODbapcXiM 5ozI8YygE4Hsmy00E/q6+UOSbe37vO2kAteoKgmdVB5RnCjJWI1//Qsvoj6QJAEG2A83trVnP4y w/HzZg4ck92Kn/gDNIcqQQIKzkGLccAfOd3Gl4UAEKHLvdiuYLWahTAu7bNSEuQ/8iRL5QjkBJF GKORb8fmK50tEyZO0P7Ei2Sxg7gVN+gdS8BO2O0AXvSZo0s4TG6BrHPGC4uXnTWcL2PpPfvUkHB UFRyw6SGYGyuJRwb1sl4G3MXYLkSo14xYrohC7oMj4zlaEpIjHTD+phQDVO+eY+iVbrwI4nvP/6 BOHA7H6NkoCU/a3TUu2t+HflJHPaf4hpnmksbayZSyVognY307XV7wQitCsqkd+viS2Rttja1PJ +ZuzQGwLmCMkaxpx4ow1wY4innfHQ== X-Received: by 2002:a05:600c:3547:b0:49c:f4e1:4c2d with SMTP id 5b1f17b1804b1-49eb732633bmr391375e9.16.1789510145636; Tue, 15 Sep 2026 15:09:05 -0700 (PDT) Received: from smtpclient.apple ([2a00:23c8:d60b:1d01:f97b:ed14:3f2a:b746]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-4870bf438fbsm2098202f8f.36.2026.09.15.15.09.04 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 15 Sep 2026 15:09:05 -0700 (PDT) Content-Type: text/plain; charset=utf-8 List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3901.100.1.1.11\)) Subject: Re: git: 09711ccb09f4 - main - Remove source_rc_confs, inlined from defaults/rc.conf From: Jessica Clarke In-Reply-To: Date: Tue, 15 Sep 2026 23:08:54 +0100 Cc: "src-committers@freebsd.org" , "dev-commits-src-all@freebsd.org" , "dev-commits-src-main@freebsd.org" Content-Transfer-Encoding: quoted-printable Message-Id: References: <6aa9b038.338f3.46192136@gitrepo.freebsd.org> <9467A34A-BDA8-4AC0-9883-1A5401EFF7F3@freebsd.org> <96890F48-7939-459C-92D6-026174AF5281@freebsd.org> <8192F29B-C42C-4579-AF0D-109F483846D5@freebsd.org> <6651F051-4227-4ABA-A4A7-06DBB855FE93@freebsd.org> To: Devin Teske X-Mailer: Apple Mail (2.3901.100.1.1.11) X-Spamd-Bar: / X-Spamd-Result: default: False [0.60 / 15.00]; MV_CASE(0.50)[]; FORGED_SENDER(0.30)[jrtc27@freebsd.org,jrtc27@jrtc27.com]; R_SPF_ALLOW(-0.20)[+ip4:74.125.0.0/16]; DMARC_POLICY_SOFTFAIL(0.10)[freebsd.org : SPF not aligned (relaxed), No valid DKIM,none]; MIME_GOOD(-0.10)[text/plain]; RCVD_VIA_SMTP_AUTH(0.00)[]; ARC_NA(0.00)[]; TO_DN_EQ_ADDR_SOME(0.00)[]; FREEFALL_USER(0.00)[jrtc27]; ASN(0.00)[asn:15169, ipnet:74.125.225.0/24, country:US]; TO_DN_SOME(0.00)[]; MIME_TRACE(0.00)[0:+]; RWL_MAILSPIKE_POSSIBLE(0.00)[74.125.225.76:from]; R_DKIM_NA(0.00)[]; RCVD_IN_DNSWL_NONE(0.00)[74.125.225.76:from]; RCVD_COUNT_TWO(0.00)[2]; FROM_NEQ_ENVFROM(0.00)[jrtc27@freebsd.org,jrtc27@jrtc27.com]; FROM_HAS_DN(0.00)[]; MID_RHS_MATCH_FROM(0.00)[]; TO_MATCH_ENVRCPT_SOME(0.00)[]; RCVD_TLS_LAST(0.00)[]; PREVIOUSLY_DELIVERED(0.00)[dev-commits-src-all@freebsd.org]; MLMMJ_DEST(0.00)[dev-commits-src-all@freebsd.org]; RCPT_COUNT_THREE(0.00)[4] X-Rspamd-Queue-Id: 4hkx4g286fz4Vbf On 15 Sep 2026, at 22:55, Devin Teske wrote: >=20 >=20 >> On Sep 15, 2026, at 2:31=E2=80=AFPM, Jessica Clarke = wrote: >>=20 >> just because you=E2=80=99re listed as a maintainer doesn=E2=80=99t = mean you >> shouldn=E2=80=99t seek review from others for 100+ LoC commits. >=20 >=20 > LoC is not the handbook=E2=80=99s "non-trivial=E2=80=9D test; most of = the diff is comments and deletion of the mistaken inline. >=20 > The committer's guide encourages review for non-trivial work. It does = not require a Differential for a trivial follow-up on your own files. = eaeb5f29bc6f inlined source_rc_confs from defaults, broke A_flag, and = left a load path that could not work. I took that out and the direct = debris. That is trivial in the handbook sense. >=20 > https://reviews.freebsd.org/D59658 >=20 >=20 >> We all make >> mistakes even for code we wrote that we know like the back of our >> hands, so more eyes from peers can help us all reach a better end >> result. >>=20 >=20 > I=E2=80=99m correcting mistakes I failed to catch in bapt=E2=80=99s = review; siva later caught them in the tests. >=20 >=20 >> just because each of these changes is fixing something in the >> same commit doesn=E2=80=99t mean they all need to be bundled into one = commit. >> Splitting them up into multiple commits makes it easier to review = (both >> pre-commit and post-commit), and makes it clearer which source = changes >> correspond to what part of the commit message (even if maybe in this >> case it=E2=80=99s relatively apparent when you look at the diff, = modern source >> control software makes it easy to break changes into multiple = commits). >>=20 >=20 > Splitting leaves landmines for bisection. >=20 > The bundled pieces have to land together so main is not left = half-broken for bisect. =46rom what the commit message says I find it hard to believe that it would not be possible to split it into multiple commits without introducing new, temporary regressions. Is that what you are trying to say here? Also, can you please not be so aggressive to any negative feedback? I really don=E2=80=99t think it should be this controversial for me to = have pointed out that a commit listing what sounded like at least five separate things should have been broken up. Jessica