From: George Spelvin <lkml@SDF.ORG>
To: Dan Carpenter <dan.carpenter@oracle.com>
Cc: devel@driverdev.osuosl.org, Ajay.Kathat@microchip.com,
gregkh@linuxfoundation.org, linux-wireless@vger.kernel.org,
Adham.Abozaeid@microchip.com, johannes@sipsolutions.net,
lkml@sdf.org
Subject: Re: [PATCH v3] staging: wilc1000: Use crc7 in lib/ rather than a private copy
Date: Sat, 4 Apr 2020 18:15:41 +0000 [thread overview]
Message-ID: <20200404181541.GC11944@SDF.ORG> (raw)
In-Reply-To: <20200404172537.GI2066@kadam>
On Sat, Apr 04, 2020 at 08:25:37PM +0300, Dan Carpenter wrote:
> On Fri, Apr 03, 2020 at 11:40:28PM +0000, George Spelvin wrote:
>> I understand that it's addressed more to patch authors than
>> maintainers forwarding them, but I've read that thing a dozen times,
>> and the description of S-o-b always seemed to be about copyright.
>
> It's to say that you didn't add anything which you shouldn't have, for
> example, secret SCO UnixWare stuff.
Yes, I'm familiar with the (irritating) history. Which is why I
had the idea stuck in my head that that it was all about copyright
and if you didn't add anything copyrightable, an S-o-b wasn't
required. No more than I'd ask for one from the administrator
of the e-mail system which delivered it.
submitting-patches.rst says "sign your work". It didn't occur to
me to sign something that wasn't my work.
>> So I had assumed that edits which were below the de minimus standard
>> of copyright didn't need a separate S-o-b.
>>
>> Am I right that there should be an S-o-b from everyone from the
>> patch author to the patch committer (as recorded in git)? And the
>> one exception is that we don't need S-o-b for git pulls after that,
>> because the merge commits record the information?
>
> Yes. Also if people added their S-o-b for git merges it would change
> the git hash for the patch which would suck.
I understand the technical difficulties, but lawyers aren't always
deterred by such things. :-) Seriously, it's clear there has to
be an exception; the question was about the scope of the exception.
Thank you for your patience clarifying this stuff for the nth time.
prev parent reply other threads:[~2020-04-04 18:16 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-26 15:23 [PATCH v3] staging: wilc1000: Use crc7 in lib/ rather than a private copy Ajay.Kathat
2020-04-02 8:27 ` Dan Carpenter
2020-04-02 13:36 ` Ajay.Kathat
2020-04-02 15:30 ` George Spelvin
2020-04-03 9:10 ` Dan Carpenter
2020-04-03 23:40 ` George Spelvin
2020-04-04 10:05 ` Kalle Valo
2020-04-04 17:25 ` Dan Carpenter
2020-04-04 18:15 ` George Spelvin [this message]
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=20200404181541.GC11944@SDF.ORG \
--to=lkml@sdf.org \
--cc=Adham.Abozaeid@microchip.com \
--cc=Ajay.Kathat@microchip.com \
--cc=dan.carpenter@oracle.com \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.