public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Kinney, Michael D" <michael.d.kinney@intel.com>
To: "Justen, Jordan L" <jordan.l.justen@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>, Andrew Fish <afish@apple.com>
Subject: Re: [Patch v4 3/6] edk2: Update to TianoCore Contribution Agreement 1.1
Date: Tue, 25 Jul 2017 17:55:45 +0000	[thread overview]
Message-ID: <E92EE9817A31E24EB0585FDF735412F5A7D5F2BB@ORSMSX113.amr.corp.intel.com> (raw)
In-Reply-To: <150100419707.16362.3894920649442321083@jljusten-skl>

Hi Jordan,

I also considered changing CodeModule to Module.  I will
make that change.

I like the idea of providing a chapter or section reference
in a document patch when applicable.  I will update the 
tianocore-docs wiki with that recommendation as well.
 
Mike


> -----Original Message-----
> From: Justen, Jordan L
> Sent: Tuesday, July 25, 2017 10:37 AM
> To: Kinney, Michael D <michael.d.kinney@intel.com>; edk2-
> devel@lists.01.org
> Cc: Leif Lindholm <leif.lindholm@linaro.org>; Andrew Fish
> <afish@apple.com>
> Subject: Re: [Patch v4 3/6] edk2: Update to TianoCore
> Contribution Agreement 1.1
> 
> On 2017-07-24 16:45:13, Michael D Kinney wrote:
> > https://bugzilla.tianocore.org/show_bug.cgi?id=629
> >
> > Update the TianoCore Contribution Agreement from Version 1.0
> > to Version 1.1 to cover open source documentation associated
> > with the TianoCore project.
> >
> > Version 1.0 covers source code files.  Version 1.1 is a
> > backwards compatible extension that adds support for document
> > files in both source form and compiled form.
> >
> > The edk2 repository is updated so the same contribution
> > agreement is used by developers that work on TianoCore
> > related source code or TianoCore related documents.
> >
> > Links to RFC and Wiki on the GitBook documentation process
> > * https://lists.01.org/pipermail/edk2-devel/2017-
> March/008654.html
> > * https://github.com/tianocore-docs/edk2-
> TemplateSpecification/wiki
> >
> > Cc: Leif Lindholm <leif.lindholm@linaro.org>
> > Cc: Andrew Fish <afish@apple.com>
> > Cc: Jordan Justen <jordan.l.justen@intel.com>
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
> > ---
> >  Contributions.txt | 43 ++++++++++++++++++++++++++++++--------
> -----
> >  1 file changed, 30 insertions(+), 13 deletions(-)
> >
> > diff --git a/Contributions.txt b/Contributions.txt
> > index f87cbd73c6..e52fd1adcc 100644
> > --- a/Contributions.txt
> > +++ b/Contributions.txt
> > @@ -1,4 +1,3 @@
> > -
> >  ======================
> >  = Code Contributions =
> >  ======================
> > @@ -12,7 +11,7 @@ To make a contribution to a TianoCore
> project, follow these steps.
> >     contribution is made under the terms of the specified
> >     contribution agreement.  Your "Contributed-under" message
> >     must include the name of contribution agreement and
> version.
> > -   For example: Contributed-under: TianoCore Contribution
> Agreement 1.0
> > +   For example: Contributed-under: TianoCore Contribution
> Agreement 1.1
> >     The "TianoCore Contribution Agreement" is included below
> in
> >     this document.
> >  4. Submit your code to the TianoCore project using the
> process
> > @@ -28,6 +27,10 @@ To make a contribution to a TianoCore
> project, follow these steps.
> >     * Python-2.0: http://opensource.org/licenses/Python-2.0
> >     * Zlib: http://opensource.org/licenses/Zlib
> >
> > +   For documentation:
> > +   * FreeBSD Documentation License
> > +     https://www.freebsd.org/copyright/freebsd-doc-
> license.html
> > +
> >     Contributions of code put into the public domain can also
> be
> >     accepted.
> >
> > @@ -47,11 +50,11 @@ and the "Contributed-under" message.
> >  === Start of sample patch email message ===
> >
> >  From: Contributor Name <contributor@example.com>
> > -Subject: [PATCH] CodeModule: Brief-single-line-summary
> > +Subject: [Repository/Branch PATCH] CodeModule: Brief-single-
> line-summary
> >
> >  Full-commit-message
> >
> > -Contributed-under: TianoCore Contribution Agreement 1.0
> > +Contributed-under: TianoCore Contribution Agreement 1.1
> >  Signed-off-by: Contributor Name <contributor@example.com>
> >  ---
> >
> > @@ -65,29 +68,37 @@ Patch content inline or attached
> >  === Notes for sample patch email ===
> >
> >  * The first line of commit message is taken from the email's
> subject
> > -  line following [PATCH]. The remaining portion of the commit
> message
> > +  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
> >
> >  === Definitions for sample patch email ===
> >
> > -* "CodeModule" is a short idenfier for the affected code.
> For
> > -  example MdePkg, or MdeModulePkg UsbBusDxe.
> > +* "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
> > +  'staging/edk2-test'.
> > +* "CodeModule" is a short identifier for the affected code.
> This identifier
> > +  should not be provided for documentation patches. For
> example 'MdePkg',
> > +  or 'MdeModulePkg/UsbBusDxe'
> 
> Maybe we could rename this from CodeModule to Module? I think it
> could
> and should be use in documents where feasible.
> 
> For example, in the style guide:
> 
> * whitespace: Change involving whitespace style
> 
> * case: Change involving word case style
> 
> I think this should be applicable in all but the most trivial
> documents or code packages.
> 
> I also had a comment on patch 6, but otherwise series:
> 
> Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
> 
> >  * "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
> > +* "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.
> >
> >  ========================================
> > -= TianoCore Contribution Agreement 1.0 =
> > += TianoCore Contribution Agreement 1.1 =
> >  ========================================
> >
> > -INTEL CORPORATION ("INTEL") MAKES AVAILABLE SOFTWARE,
> DOCUMENTATION,
> > +INTEL CORPORATION ("INTEL") MAKES AVAILABLE SOFTWARE,
> DOCUMENTATION ("DOCUMENTATION"),
> >  INFORMATION AND/OR OTHER MATERIALS FOR USE IN THE TIANOCORE
> OPEN SOURCE
> >  PROJECT (COLLECTIVELY "CONTENT"). USE OF THE CONTENT IS
> GOVERNED BY THE
> >  TERMS AND CONDITIONS OF THIS AGREEMENT BETWEEN YOU AND INTEL
> AND/OR THE
> > @@ -100,14 +111,20 @@ AGREEMENT AND THE TERMS AND CONDITIONS
> OF ANY APPLICABLE LICENSE
> >  AGREEMENTS OR NOTICES INDICATED OR REFERENCED BELOW, THEN YOU
> MAY NOT
> >  USE THE CONTENT.
> >
> > -Unless otherwise indicated, all Content made available on the
> TianoCore
> > +Unless otherwise indicated, all Content (except
> Documentation) made available on the TianoCore
> >  site is provided to you under the terms and conditions of the
> BSD
> >  License ("BSD"). A copy of the BSD License is available at
> >  http://opensource.org/licenses/bsd-license.php
> >  or when applicable, in the associated License.txt file.
> >
> > +Unless otherwise indicated, all Documentation made available
> on the
> > +TianoCore site is provided to you under the terms and
> conditions of the
> > +FreeBSD Documentation License ("FreeBSD"). A copy of the BSD
> License is
> > +available at https://www.freebsd.org/copyright/freebsd-doc-
> license.html or,
> > +when applicable, in the associated License.txt file.
> > +
> >  Certain other content may be made available under other
> licenses as
> > -indicated in or with such Content. (For example, in a
> License.txt file.)
> > +indicated in or with such Content (for example, in a
> License.txt file).
> >
> >  You accept and agree to the following terms and conditions
> for Your
> >  present and future Contributions submitted to TianoCore site.
> Except
> > --
> > 2.13.1.windows.2
> >

  reply	other threads:[~2017-07-25 17:53 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-24 23:45 [Patch V4 0/6] Update to Tiano Contribution Agreement 1.1 Michael D Kinney
2017-07-24 23:45 ` [Patch v4 1/6] BaseTools/PatchCheck: Support " Michael D Kinney
2017-07-24 23:45 ` [Patch v4 2/6] edk2: Move TianoCore Contribution Agreement to root Michael D Kinney
2017-07-24 23:45 ` [Patch v4 3/6] edk2: Update to TianoCore Contribution Agreement 1.1 Michael D Kinney
2017-07-25 17:36   ` Jordan Justen
2017-07-25 17:55     ` Kinney, Michael D [this message]
2017-07-26 17:05       ` Jordan Justen
2017-07-26 17:45         ` Kinney, Michael D
2017-07-26 18:20           ` Jordan Justen
2017-07-26 19:13             ` Laszlo Ersek
2017-07-24 23:45 ` [Patch v4 4/6] edk2: Reformat " Michael D Kinney
2017-07-24 23:45 ` [Patch v4 5/6] edk2: Move License.txt file to root Michael D Kinney
2017-07-25  0:12   ` Kinney, Michael D
2017-07-24 23:45 ` [Patch v4 6/6] edk2: Add Readme.md to root of edk2 repository Michael D Kinney
2017-07-25 17:25   ` Jordan Justen
2017-07-25 17:43     ` Kinney, Michael D
2017-07-25 13:07 ` [Patch V4 0/6] Update to Tiano Contribution Agreement 1.1 Laszlo Ersek
2017-07-25 16:06   ` Kinney, Michael D
2017-07-26 10:39     ` Laszlo Ersek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=E92EE9817A31E24EB0585FDF735412F5A7D5F2BB@ORSMSX113.amr.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox