Linux-SPDX Archive on lore.kernel.org
 help / color / Atom feed
From: Allison Randal <allison@lohutok.net>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: Richard Fontana <rfontana@redhat.com>, linux-spdx@vger.kernel.org
Subject: Re: [patch 05/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 80
Date: Fri, 24 May 2019 12:29:10 -0400
Message-ID: <14f371ad-ad4b-92c6-e5b1-c3f4aa2abb4d@lohutok.net> (raw)
In-Reply-To: <20190524055532.GA2428@kroah.com>

On 5/24/19 1:55 AM, Greg KH wrote:
> On Thu, May 23, 2019 at 10:21:57PM -0400, Richard Fontana wrote:
>>
>> I assume it's okay to put the "Reviewed-by:" at the top of the reply though?
> 
> It's not the nicest, as you should trim the email and not quote the
> whole large patch, but we can live with it for now.

I've been a developer on so many different open source projects, that
I've adopted the "When in Rome" Rule, following whatever formal or
informal patterns the project prefers. Here, I'd expect that to be:

https://github.com/torvalds/linux/blob/master/Documentation/process/submitting-patches.rst

But, that doc doesn't actually say where to put the "Reviewed-by" line
in a reply email, so the tools will pick it up. As far as I can tell
from lkml, the r-b line can go anywhere in the reply, so I'll experiment
with that on today's batch. But, really, I am totally happy to precisely
follow whatever pattern you prefer, so just let us know. We are guests
in your development workflow, so it makes more sense for us to adapt to
you, than for you to adapt to us. :)

Allison

  reply index

Thread overview: 94+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-22  7:51 [patch 00/25] SPDX cleanup: batch 4 Thomas Gleixner
2019-05-22  7:51 ` [patch 01/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 76 Thomas Gleixner
2019-05-22 12:10   ` Armijn Hemel - Tjaldur Software Governance Solutions
2019-05-22 12:47   ` Richard Fontana
2019-05-22 12:48   ` Kate Stewart
2019-05-23 21:04   ` Allison Randal
2019-05-22  7:51 ` [patch 02/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 77 Thomas Gleixner
2019-05-22 13:14   ` Richard Fontana
2019-05-22 14:35   ` Armijn Hemel - Tjaldur Software Governance Solutions
2019-05-23 21:04   ` Allison Randal
2019-05-22  7:51 ` [patch 03/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 78 Thomas Gleixner
2019-05-22 13:15   ` Richard Fontana
2019-05-22 15:22   ` Armijn Hemel - Tjaldur Software Governance Solutions
2019-05-23 21:06   ` Allison Randal
2019-05-22  7:51 ` [patch 04/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 79 Thomas Gleixner
2019-05-22 13:08   ` Richard Fontana
2019-05-22 15:36   ` Armijn Hemel - Tjaldur Software Governance Solutions
2019-05-23 21:06   ` Allison Randal
2019-05-22  7:51 ` [patch 05/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 80 Thomas Gleixner
2019-05-22 13:07   ` Richard Fontana
2019-05-22 13:18     ` Greg KH
2019-05-23 21:20       ` Allison Randal
2019-05-24  2:40         ` Richard Fontana
2019-05-24  2:21       ` Richard Fontana
2019-05-24  5:54         ` Thomas Gleixner
2019-05-24  5:55         ` Greg KH
2019-05-24 16:29           ` Allison Randal [this message]
2019-05-22  7:51 ` [patch 06/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 81 Thomas Gleixner
2019-05-22 13:13   ` Richard Fontana
2019-05-22 16:42   ` Armijn Hemel - Tjaldur Software Governance Solutions
2019-05-23  0:43   ` Kate Stewart
2019-05-23 21:10   ` Allison Randal
2019-05-22  7:51 ` [patch 07/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 82 Thomas Gleixner
2019-05-22 13:15   ` Richard Fontana
2019-05-22 16:43   ` Armijn Hemel - Tjaldur Software Governance Solutions
2019-05-23  0:43   ` Kate Stewart
2019-05-23 21:11   ` Allison Randal
2019-05-22  7:51 ` [patch 08/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 83 Thomas Gleixner
2019-05-22 13:13   ` Richard Fontana
2019-05-22 21:58   ` Armijn Hemel - Tjaldur Software Governance Solutions
2019-05-23  0:43   ` Kate Stewart
2019-05-23 21:12   ` Allison Randal
2019-05-22  7:51 ` [patch 09/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 84 Thomas Gleixner
2019-05-22 13:12   ` Richard Fontana
2019-05-22 21:58   ` Armijn Hemel - Tjaldur Software Governance Solutions
2019-05-23 21:22   ` Allison Randal
2019-05-22  7:51 ` [patch 10/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 85 Thomas Gleixner
2019-05-22 13:03   ` Richard Fontana
2019-05-22  7:51 ` [patch 11/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 86 Thomas Gleixner
2019-05-22 13:14   ` Richard Fontana
2019-05-23 21:41   ` Allison Randal
2019-05-22  7:51 ` [patch 12/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 87 Thomas Gleixner
2019-05-22 13:11   ` Richard Fontana
2019-05-22 13:46     ` Thomas Gleixner
2019-05-22  7:51 ` [patch 13/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 88 Thomas Gleixner
2019-05-22 13:11   ` Richard Fontana
2019-05-23 21:43   ` Allison Randal
2019-05-22  7:51 ` [patch 14/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 89 Thomas Gleixner
2019-05-22 13:12   ` Richard Fontana
2019-05-23 21:44   ` Allison Randal
2019-05-22  7:51 ` [patch 15/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 90 Thomas Gleixner
2019-05-22 13:06   ` Richard Fontana
2019-05-23 21:45   ` Allison Randal
2019-05-22  7:51 ` [patch 16/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 91 Thomas Gleixner
2019-05-22 12:48   ` Richard Fontana
2019-05-23 21:45   ` Allison Randal
2019-05-22  7:51 ` [patch 17/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 92 Thomas Gleixner
2019-05-22 13:02   ` Richard Fontana
2019-05-22 18:07     ` Thomas Gleixner
2019-05-22  7:51 ` [patch 18/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 93 Thomas Gleixner
2019-05-22 12:48   ` Richard Fontana
2019-05-23 21:48   ` Allison Randal
2019-05-22  7:51 ` [patch 19/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 94 Thomas Gleixner
2019-05-22 13:06   ` Richard Fontana
2019-05-23 21:48   ` Allison Randal
2019-05-22  7:51 ` [patch 20/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 95 Thomas Gleixner
2019-05-22 13:00   ` Richard Fontana
2019-05-23 21:49   ` Allison Randal
2019-05-22  7:51 ` [patch 21/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 96 Thomas Gleixner
2019-05-22 12:59   ` Richard Fontana
2019-05-23 21:50   ` Allison Randal
2019-05-22  7:51 ` [patch 22/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 97 Thomas Gleixner
2019-05-22 13:05   ` Richard Fontana
2019-05-23 21:50   ` Allison Randal
2019-05-22  7:51 ` [patch 23/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 98 Thomas Gleixner
2019-05-22 13:06   ` Richard Fontana
2019-05-23 21:51   ` Allison Randal
2019-05-22  7:51 ` [patch 24/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 99 Thomas Gleixner
2019-05-22 13:04   ` Richard Fontana
2019-05-22  7:51 ` [patch 25/25] treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 100 Thomas Gleixner
2019-05-22 12:59   ` Richard Fontana
2019-05-22 13:42     ` Thomas Gleixner
2019-05-22 15:30     ` J Lovejoy
2019-05-23 22:05     ` Allison Randal

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-all 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=14f371ad-ad4b-92c6-e5b1-c3f4aa2abb4d@lohutok.net \
    --to=allison@lohutok.net \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-spdx@vger.kernel.org \
    --cc=rfontana@redhat.com \
    /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

Linux-SPDX Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-spdx/0 linux-spdx/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-spdx linux-spdx/ https://lore.kernel.org/linux-spdx \
		linux-spdx@vger.kernel.org
	public-inbox-index linux-spdx

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-spdx


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git