From owner-dev-commits-doc-all@freebsd.org Tue Apr 20 12:28:53 2021 Return-Path: Delivered-To: dev-commits-doc-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id C0EB65F471E for ; Tue, 20 Apr 2021 12:28:53 +0000 (UTC) (envelope-from mat@freebsd.org) Received: from smtp.freebsd.org (smtp.freebsd.org [96.47.72.83]) (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 (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4FPjfF550Lz4grr; Tue, 20 Apr 2021 12:28:53 +0000 (UTC) (envelope-from mat@freebsd.org) Received: from mail.j.mat.cc (owncloud.cube.mat.cc [IPv6:2a01:678:4:1::228]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mail.mat.cc", Issuer "R3" (verified OK)) (Authenticated sender: mat/mail) by smtp.freebsd.org (Postfix) with ESMTPSA id 667ABBE80; Tue, 20 Apr 2021 12:28:53 +0000 (UTC) (envelope-from mat@freebsd.org) Received: from aching.in.mat.cc (unknown [IPv6:2a01:678:ab:0:3356:ef6b:256f:d0c6]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: mat@mat.cc) by mail.j.mat.cc (Postfix) with ESMTPSA id 4720B942D80; Tue, 20 Apr 2021 12:28:50 +0000 (UTC) Date: Tue, 20 Apr 2021 14:28:49 +0200 From: Mathieu Arnold To: Mateusz Piotrowski <0mp@freebsd.org> Cc: doc-committers@freebsd.org, dev-commits-doc-all@freebsd.org Subject: Re: git: ceee1def00 - main - articles/committers-guide: Document change in MFH policy Message-ID: <20210420122849.rh5v2kohzvkxs4uu@aching.in.mat.cc> References: <202104201032.13KAWNFn016258@gitrepo.freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="z2mucrzakid4554m" Content-Disposition: inline In-Reply-To: <202104201032.13KAWNFn016258@gitrepo.freebsd.org> X-BeenThere: dev-commits-doc-all@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for all branches of the doc repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 20 Apr 2021 12:28:53 -0000 --z2mucrzakid4554m Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Apr 20, 2021 at 10:32:23AM +0000, Mateusz Piotrowski wrote: > The branch main has been updated by 0mp: >=20 > URL: https://cgit.FreeBSD.org/doc/commit/?id=3Dceee1def00ac69f06b9d86b2fb= 1ff04bb6c84d48 >=20 > commit ceee1def00ac69f06b9d86b2fb1ff04bb6c84d48 > Author: Mateusz Piotrowski <0mp@FreeBSD.org> > AuthorDate: 2021-04-20 10:14:58 +0000 > Commit: Mateusz Piotrowski <0mp@FreeBSD.org> > CommitDate: 2021-04-20 10:14:58 +0000 >=20 > articles/committers-guide: Document change in MFH policy > =20 > Merging commits into the quarterly branch no longer requires an expli= cit > approval [1]. Let's remove outdated documentation and leave a note > explaining that there is no need for an explicit approval to MFH. Note that the MFH line should still be used to note intent. > [1] https://svnweb.freebsd.org/ports?view=3Drevision&revision=3D5566= 26 > --- > .../en/articles/committers-guide/_index.adoc | 60 +---------------= ------ > 1 file changed, 1 insertion(+), 59 deletions(-) >=20 > diff --git a/documentation/content/en/articles/committers-guide/_index.ad= oc b/documentation/content/en/articles/committers-guide/_index.adoc > index 3dbc468fda..f0548fbce5 100644 > --- a/documentation/content/en/articles/committers-guide/_index.adoc > +++ b/documentation/content/en/articles/committers-guide/_index.adoc > @@ -2584,9 +2584,6 @@ Approved by: re (username) > |`MFC with:` > |If the commit should be merged together with a previous one in a single= MFC commit (for example, where this commit corrects a bug in the previous = change), specify the corresponding Git hash. > =20 > -|`MFH:` > -|A ports quarterly branch name, to request approval for merge. > - > |`Relnotes:` > |If the change is a candidate for inclusion in the release notes for the= next release from the branch, set to `yes`. > =20 > @@ -3450,62 +3447,7 @@ For more information on how to merge commits to th= e quarterly branch, see < [[ports-qa-misc-request-mfh]] > =3D=3D=3D=3D What is the procedure to request authorization for merging = a commit to the quarterly branch? > =20 > -When doing the commit, add the branch name to the `MFH:` line, for examp= le: > - > -[.programlisting] > -.... > -MFH: 2014Q1 > -.... > - > -It will automatically notify the {ports-secteam} and the {portmgr}. > -They will then decide if the commit can be merged and answer with the pr= ocedure. > - > -If the commit has already been made, send an email to the {ports-secteam= } and the {portmgr} with the revision number and a small description of why= the commit needs to be merged. > - > -[TIP] > -=3D=3D=3D=3D > -If the MFH is covered by a blanket approval, please explain why with a c= ouple of words on the `MFH` line, so that the reviewing team can skip this = commit and save time. > - > -For example: > - > -[.programlisting] > -.... > -MFH: 2014Q1 (runtime fix) > -MFH: 2014Q1 (browser blanket) > -.... > - > -The list of blanket approvals is available in <>. > -=3D=3D=3D=3D > - > -[[ports-qa-blanket]] > -=3D=3D=3D=3D Are there any changes that can be merged without asking for= approval? > - > -The following blanket approvals for merging to the quarterly branches ar= e in effect: > - > -[NOTE] > -=3D=3D=3D=3D > -This blanket approval also applies to direct commits for ports that have= been removed from `head`. > -=3D=3D=3D=3D > - > -[IMPORTANT] > -=3D=3D=3D=3D > -These fixes must be tested on the quarterly branch. > -=3D=3D=3D=3D > - > -* Fixes that do not result in a change in contents of the resulting pack= age. For example: > -** [.filename]#pkg-descr#: `WWW`: URL updates (existing 404, moved or in= correct) > -Build, runtime or packaging fixes, if the quarterly branch version is cu= rrently broken. > -Missing dependencies (detected, linked against but not registered via `*= _DEPENDS`). > -* Fixing link:{porters-handbook}#uses-shebangfix[shebangs], stripping in= stalled libraries and binaries, and plist fixes. > -* Backport of security and reliability fixes which only result in `PORTR= EVISION` bumps and no changes to enabled features. for example, adding a pa= tch fixing a buffer overflow. > -* Minor version changes that do nothing but fix security or crash-relate= d issues. > -* Adding/fixing `CONFLICTS`. > -* Web Browsers, browser plugins, and their required dependencies. > - > -[IMPORTANT] > -=3D=3D=3D=3D > -Commits that are not covered by these blanket approvals always require e= xplicit approval of either {ports-secteam} or {portmgr}. > -=3D=3D=3D=3D > +As of November 30, 2020, there is no need to seek explicit approval to c= ommit to the quarterly branch. > =20 > [[ports-qa-misc-commit-mfh]] > =3D=3D=3D=3D What is the procedure for merging commits to the quarterly = branch? >=20 --=20 Mathieu Arnold --z2mucrzakid4554m Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQKTBAABCgB9FiEEVhwchfRfuV0unqO5KesJApEdfgIFAmB+yPxfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldDU2 MUMxQzg1RjQ1RkI5NUQyRTlFQTNCOTI5RUIwOTAyOTExRDdFMDIACgkQKesJApEd fgLwShAAnVC0eR5wk+GAWm4LzlomiAlMfFlQjxDur6/il+xWhCL08lEHwPIs6Vwr vPoOteRE3x8i6ODeg4t9okOu3aEe9339ArdKlKihB8u/P9KtWjvVlzqdCAa5mvuL jLrUhkRmnyLNZ+auBo1DxHzxQL10mkZ2slLX7b3xK/DVH8Xe8g5IcOePNPsNQIFY teFArnNQi+wP56rSennaoFtO4ZABatT2x2bFs2FJY0nkZ/w4ySNrMr3CRPfbbUCw pt+dNDmpTSo4pcMw/Poq+Ph9mgVyidopRC33bw9Wr5UyL2+j/pgfrJP9g5Oy1lzE Xyj2B4EYo7T8QLuHBohDyXezLomQWMooK8ObdS9LCZbiOPLNaM61Y3dbkAVdELw5 r7gJM8I25AOOi+Gwurmm+BJxjbIr5miVUIaXA99Q0HIiDTbXOBvYrEsoiVyFnwUI qWwA/0HSN0udTmdB7+y/+l4UhndbsLl+7s722X/5WYIOXoEnFfzEBh8jvV6avYz7 Ih7ES+cpl+TAMN/JsbQUOqz4ixpWg30r04UyUxMgLpVi/PPYV4FW9NPryyGwZmqP 6WTOtiuqaaw8cAcm9WAwt6HJ+vfUSgQZXMW+4ZmLNwmaTF6as4zBf/GNizeYGFJf wIXKxUafVKaOnEfRxldaT7ohY2JBoxoIAq2N7iVr76xFcDNYkVo= =TDCH -----END PGP SIGNATURE----- --z2mucrzakid4554m--