From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 8BF552095DBBD for ; Mon, 24 Jul 2017 11:13:46 -0700 (PDT) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 24 Jul 2017 11:15:46 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.40,408,1496127600"; d="scan'208";a="1175969808" Received: from orsmsx104.amr.corp.intel.com ([10.22.225.131]) by fmsmga001.fm.intel.com with ESMTP; 24 Jul 2017 11:15:34 -0700 Received: from orsmsx113.amr.corp.intel.com ([169.254.9.9]) by ORSMSX104.amr.corp.intel.com ([169.254.4.205]) with mapi id 14.03.0319.002; Mon, 24 Jul 2017 11:13:52 -0700 From: "Kinney, Michael D" To: Leif Lindholm , "Justen, Jordan L" , "Kinney, Michael D" CC: "edk2-devel@lists.01.org" , Andrew Fish Thread-Topic: [edk2] [Patch V3 3/8] edk2: Remove commit message details from Contributions.txt Thread-Index: AQHTARtIXU4sCpZTakCYLYseC4bL0aJfQHwAgAAP/4CAA+cwgA== Date: Mon, 24 Jul 2017 18:13:52 +0000 Message-ID: References: <20170720054411.18928-1-michael.d.kinney@intel.com> <20170720054411.18928-4-michael.d.kinney@intel.com> <150067181439.20068.8395013529501693336@jljusten-skl> <20170721221409.GF1501@bivouac.eciton.net> In-Reply-To: <20170721221409.GF1501@bivouac.eciton.net> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 10.0.102.7 dlp-reaction: no-action x-originating-ip: [10.22.254.138] MIME-Version: 1.0 Subject: Re: [Patch V3 3/8] edk2: Remove commit message details from Contributions.txt X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 24 Jul 2017 18:13:46 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Leif, I was attempting to keep the number of top level files to a minimum and also attempt to use the same Contributions.txt file for both source code repositories and document repositories. The set of files I am proposing be at the root of repositories are: * Readme.md * License.txt * Contributions.txt My first attempt at this was to move some of the commit message details that are slightly different between source code and documentation from Contributions.txt to the Readme.md and the Readme.md also provides links to the wiki pages with even more details. I am seeing feedback that there is a desire to retain the commit message format details in Contributions.txt. How about the following approach for updates to Contributions.txt: 1) Update Contribution Agreement section from 1.0 to 1.1 to add=20 support for TianoCore Documentation. 2) Update the 'Code Contributions' section to add the FreeBSD=20 Documentation License as one of the accepted licenses for documentation. Of course additional licenses can be considered=20 at a later date if a community member requests to add documentation under a different license. 3) Update the 'Change Description / Commit Message / Patch Email' section so it applies equally well to both source code and documentation. The following proposal provides guidelines that can work for patches to patches to edk2/master, other branches in edk2, other code repositories such as edk2-staging and edk2-non-osi, and document repositories in tianocore-docs. ---- Begin ---- =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D =3D Change Description / Commit Message / Patch Email =3D =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D Your change description should use the standard format for a commit message, and must include your "Signed-off-by" signature and the "Contributed-under" message. =3D=3D Sample Change Description / Commit Message =3D =3D=3D=3D Start of sample patch email message =3D=3D=3D From: Contributor Name Subject: [Repository/Branch PATCH] CodeModule: Brief-single-line-summary Full-commit-message Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Contributor Name --- An extra message for the patch email which will not be considered part of the commit message can be added here. Patch content inline or attached =3D=3D=3D End of sample patch email message =3D=3D=3D =3D=3D=3D Notes for sample patch email =3D=3D=3D * The first line of commit message is taken from the email's subject line following [Repository/Branch PATCH]. The remaining portion of the commit message is the email's content until the '---' line. * git format-patch is one way to create this format =3D=3D=3D Definitions for sample patch email =3D=3D=3D * "Repository" is the identifier of the repository the patch applies. This identifier should only be provided for repositories other than 'edk2'. For example 'edk2-BuildSpecification' or 'staging'. * "Branch" is the identifier of the branch the patch applies. This identifier should only be provided for branches other than 'edk2/master'. For example 'edk2/UDK2015', 'edk2-BuildSpecification/release/1.27', or=20 'staging/edk2-test'. * "CodeModule" is a short identifier for the affected code. This identifier should not be provided for documentation patches. For example 'MdePkg',=20 or 'MdeModulePkg/UsbBusDxe'. * "Brief-single-line-summary" is a short summary of the change. * The entire first line should be less than ~70 characters. * "Full-commit-message" a verbose multiple line comment describing the change. Each line should be less than ~70 characters. * "Contributed-under" explicitly states that the contribution is made under the terms of the contribution agreement. This agreement is included below in this document. * "Signed-off-by" is the contributor's signature identifying them by their real/legal name and their email address. ---- End ---- If this approach is acceptable, then I will update the patch series to V4. Best regards, Mike > -----Original Message----- > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On > Behalf Of Leif Lindholm > Sent: Friday, July 21, 2017 3:14 PM > To: Justen, Jordan L > Cc: Kinney, Michael D ; edk2- > devel@lists.01.org; Andrew Fish > Subject: Re: [edk2] [Patch V3 3/8] edk2: Remove commit message > details from Contributions.txt >=20 > On Fri, Jul 21, 2017 at 02:16:54PM -0700, Jordan Justen wrote: > > On 2017-07-19 22:44:06, Michael D Kinney wrote: > > > https://bugzilla.tianocore.org/show_bug.cgi?id=3D629 > > > > > > Remove the change description, commit message, and > > > patch email section from Contributions.txt. This > > > section is not part of the TianoCore Contribution > > > Agreement. > > > > Neither are the ~35 remaining lines before this content, which > you are > > not removing. > > > > > The information is very important and is > > > provided on the TianoCore web site and wiki and will > > > be added to Readme.md in the root of the edk2 > > > repository. > > > > Yes. It is important for making contributions, as is the ~35 > lines > > before the agreement that you didn't remove. So, why don't we > just > > keep them in Contributions.txt since they document how to make > a > > contribution? > > > > If you want a single file that only has the contribution > agreement > > text, then how about moving it to a ContributionAgreement.txt > file, > > and making Contributions.txt refer to it? I'm not sure why this > would > > be needed though. >=20 > I think separating the mechanics of what a developer is supposed > to do > in order to contribute code from legalese of more interest to > their > legal department is a useful thing. >=20 > This was why I was suggesting a SubmittingPatches.txt. Renaming > Contributions.txt ContributionAgreement.txt would however > indicate it > contained only the contribution agreement, which (as you allude > to) is > not currently the case. The list of acceptable licenses would > also > need to move into the "howto" in this case. >=20 > Regards, >=20 > Leif >=20 > > -Jordan > > > > > > > > Cc: Leif Lindholm > > > Cc: Andrew Fish > > > Cc: Jordan Justen > > > Contributed-under: TianoCore Contribution Agreement 1.1 > > > Signed-off-by: Michael D Kinney > > > --- > > > Contributions.txt | 49 ------------------------------------- > ------------ > > > 1 file changed, 49 deletions(-) > > > > > > diff --git a/Contributions.txt b/Contributions.txt > > > index f87cbd73c6..06c60d8e17 100644 > > > --- a/Contributions.txt > > > +++ b/Contributions.txt > > > @@ -34,55 +34,6 @@ To make a contribution to a TianoCore > project, follow these steps. > > > Contributions using other licenses might be accepted, but > further > > > review will be required. > > > > > > -=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D > > > -=3D Change Description / Commit Message / Patch Email =3D > > > -=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D > > > - > > > -Your change description should use the standard format for a > > > -commit message, and must include your "Signed-off-by" > signature > > > -and the "Contributed-under" message. > > > - > > > -=3D=3D Sample Change Description / Commit Message =3D > > > - > > > -=3D=3D=3D Start of sample patch email message =3D=3D=3D > > > - > > > -From: Contributor Name > > > -Subject: [PATCH] CodeModule: Brief-single-line-summary > > > - > > > -Full-commit-message > > > - > > > -Contributed-under: TianoCore Contribution Agreement 1.0 > > > -Signed-off-by: Contributor Name > > > ---- > > > - > > > -An extra message for the patch email which will not be > considered part > > > -of the commit message can be added here. > > > - > > > -Patch content inline or attached > > > - > > > -=3D=3D=3D End of sample patch email message =3D=3D=3D > > > - > > > -=3D=3D=3D Notes for sample patch email =3D=3D=3D > > > - > > > -* The first line of commit message is taken from the email's > subject > > > - line following [PATCH]. The remaining portion of the > commit message > > > - is the email's content until the '---' line. > > > -* git format-patch is one way to create this format > > > - > > > -=3D=3D=3D Definitions for sample patch email =3D=3D=3D > > > - > > > -* "CodeModule" is a short idenfier for the affected code. > For > > > - example MdePkg, or MdeModulePkg UsbBusDxe. > > > -* "Brief-single-line-summary" is a short summary of the > change. > > > -* The entire first line should be less than ~70 characters. > > > -* "Full-commit-message" a verbose multiple line comment > describing > > > - the change. Each line should be less than ~70 characters. > > > -* "Contributed-under" explicitely states that the > contribution is > > > - made under the terms of the contribtion agreement. This > > > - agreement is included below in this document. > > > -* "Signed-off-by" is the contributor's signature identifying > them > > > - by their real/legal name and their email address. > > > - > > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > > > =3D TianoCore Contribution Agreement 1.0 =3D > > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > > > -- > > > 2.13.1.windows.2 > > > > > > _______________________________________________ > > > edk2-devel mailing list > > > edk2-devel@lists.01.org > > > https://lists.01.org/mailman/listinfo/edk2-devel > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel