From: Yasufumi Ogawa <usufumu@gmail.com>
To: Ferruh Yigit <ferruh.yigit@intel.com>
Cc: Yasufumi Ogawa <ogawa.yasufumi@lab.ntt.co.jp>,
Trishan de Lanerolle <tdelanerolle@linuxfoundation.org>,
Tim O'Driscoll <tim.odriscoll@intel.com>,
"St Leger, Jim" <jim.st.leger@intel.com>,
spp@dpdk.org
Subject: Re: [spp] [PATCH 2/7] spp_primary: convert license to SPDX tag
Date: Fri, 25 May 2018 13:45:49 -0700 [thread overview]
Message-ID: <2e69caeb-0fa2-3da3-8339-7efc58fee7ed@gmail.com> (raw)
In-Reply-To: <eb5e5b55-0153-c603-49b9-acff43f6143d@intel.com>
On 2018/05/24 5:29, Ferruh Yigit wrote:
> On 5/18/2018 9:03 AM, Yasufumi Ogawa wrote:
>> On 2018/05/17 23:25, Trishan de Lanerolle wrote:
>>> Can you provide additional details on what you need clarification on?
>> Sorry for ambiguously. I meant that I need to talk with my boss about your suggestion.
>>
>> I was not sure that I should add our license if I make considerable changes, or not. But I understand that I don't need to add
>> without if I will to do. As your suggestion, we decided to not replace existing license.
>>
>> Ferruh,
>> I would like to send modified patches to not include NTT beside Intel.
> Hi Yasufumi,
>
> I have seen the new version, thank you for the update, I will get them today.
>
> And I would like to acknowledge that some of the files that NTT did lots of
> contribution. I guess concern here is introduce the copyright as part of SPDX
> work. I am OK to introduce copyright to them later as the significant
> contribution done on them.
>
> Thanks,
> ferruh
Hi Ferruh,
Thank you for your comment kindly. I think it would be better to talk
with Sy Jong or others who has responsibility for. I would like to add
NTT later if they agree with it, and I hope so.
Thanks,
Yasufumi
>
>> Thanks,
>> Yasufumi
>>> Regards,
>>> Trishan
>>>
>>>
>>> On Thu, May 17, 2018 at 3:09 AM, Yasufumi Ogawa <ogawa.yasufumi@lab.ntt.co.jp <mailto:ogawa.yasufumi@lab.ntt.co.jp>> wrote:
>>>
>>> On 2018/05/16 23:06, Trishan de Lanerolle wrote:
>>>
>>> Hi,
>>> Thanks for bringing up the question. Generally, we recommend that SPDX license identifiers -- for existing files be
>>> thought as supplemental information to add. Don't replace the existing license information, add the tags on. Suggest
>>> reviewing the patch against that criteria. Remember to maintain the attribution information. You may want to speak to
>>> your in-house counsel with respect to any changes you would like to make to files over which you hold copyright.
>>>
>>> Regards,
>>> Trishan
>>>
>>> Hi Trishan,
>>>
>>> Thank you for your reply. I understand that I should not replace existing license info. I would like to have a talk to and
>>> confirm that it is acceptable.
>>>
>>> Regards,
>>> Yasufumi
>>>
>>>
>>>
>>> On Wed, May 16, 2018 at 5:08 AM, Yasufumi Ogawa <ogawa.yasufumi@lab.ntt.co.jp <mailto:ogawa.yasufumi@lab.ntt.co.jp>
>>> <mailto:ogawa.yasufumi@lab.ntt.co.jp <mailto:ogawa.yasufumi@lab.ntt.co.jp>>> wrote:
>>>
>>> On 2018/05/16 1:55, Ferruh Yigit wrote:
>>>
>>> On 5/3/2018 5:08 PM, Ferruh Yigit wrote:
>>>
>>> On 4/23/2018 7:28 AM, ogawa.yasufumi@lab.ntt.co.jp <mailto:ogawa.yasufumi@lab.ntt.co.jp>
>>> <mailto:ogawa.yasufumi@lab.ntt.co.jp <mailto:ogawa.yasufumi@lab.ntt.co.jp>> wrote:
>>>
>>> @@ -1,34 +1,6 @@
>>> -/*-
>>> - * BSD LICENSE
>>> - *
>>> - * Copyright(c) 2015-2016 Intel Corporation. All rights reserved.
>>> - * All rights reserved.
>>> - *
>>> - * Redistribution and use in source and binary forms, with or without
>>> - * modification, are permitted provided that the following conditions
>>> - * are met:
>>> - *
>>> - * * Redistributions of source code must retain the above copyright
>>> - * notice, this list of conditions and the following disclaimer.
>>> - * * Redistributions in binary form must reproduce the above copyright
>>> - * notice, this list of conditions and the following disclaimer in
>>> - * the documentation and/or other materials provided with the
>>> - * distribution.
>>> - * * Neither the name of Intel Corporation nor the names of its
>>> - * contributors may be used to endorse or promote products derived
>>> - * from this software without specific prior written permission.
>>> - *
>>> - * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
>>> - * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
>>> - * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
>>> - * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
>>> - * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
>>> - * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
>>> - * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
>>> - * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
>>> - * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
>>> - * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
>>> - * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
>>> +/* SPDX-License-Identifier: BSD-3-Clause
>>> + * Copyright(c) 2015-2016 Intel Corporation
>>> + * Copyright(c) 2017-2018 Nippon Telegraph and Telephone Corporation
>>> */
>>>
>>>
>>> Hi Ogawa-san,
>>>
>>> Thanks for taking initiative to switch spp to SPDX, appreciated.
>>>
>>> First Trishan from Linux foundation and Jim from DPDK governance board are cc'ed
>>> for input if there is any concern or more process to follow from governance
>>> point of view.
>>>
>>>
>>> The is no response, which I am taking as there is no concern and will move
>>> forward on getting patches. Please shout if there are concerns.
>>>
>>> Hi Trishan, Jim,
>>>
>>> SPP was provided originally from Intel, but currently is maintained by NTT and planed to change license to SPDX. We
>>> have
>>> made many changes on SPP and I would like to add NTT to the license only for considerably modified or newly created
>>> files by
>>> NTT. Could you give us advice if there are any concern?
>>>
>>> Thanks,
>>> Yasufumi
>>>
>>>
>>>
>>> Second these patches are both converts licensing to SPDX tag and adds ntt as
>>> copyright holder. Can we separate these two please? A patchset to convert
>>> existing licensing to SPDX as it is, and another set to add ntt.
>>>
>>>
>>> Thanks,
>>> ferruh
>>>
>>>
>>>
>>>
>>>
>>>
>>> -- Yasufumi Ogawa
>>> NTT Network Service Systems Labs
>>>
>>>
>>>
>>>
>>> --
>>> Trishan R. de Lanerolle
>>> Program Manager, Networking
>>> Linux Foundation
>>> voice: +1.203.699.6401
>>> skype: tdelanerolle
>>> email: tdelanerolle@linuxfoundation.org <mailto:tdelanerolle@linuxfoundation.org>
>>> <mailto:tdelanerolle@linuxfoundation.org <mailto:tdelanerolle@linuxfoundation.org>>
>>>
>>>
>>>
>>> --
>>> Yasufumi Ogawa
>>> NTT Network Service Systems Labs
>>>
>>>
>>>
>>>
>>> --
>>> Trishan R. de Lanerolle
>>> Program Manager, Networking
>>> Linux Foundation
>>> voice: +1.203.699.6401
>>> skype: tdelanerolle
>>> email: tdelanerolle@linuxfoundation.org <mailto:tdelanerolle@linuxfoundation.org>
>>
>
next prev parent reply other threads:[~2018-05-25 20:45 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-23 6:28 [spp] [PATCH 0/7] Update licence to SPDX ogawa.yasufumi
2018-04-23 6:28 ` [spp] [PATCH 1/7] license: add bsd-3-clause text ogawa.yasufumi
2018-04-23 6:28 ` [spp] [PATCH 2/7] spp_primary: convert license to SPDX tag ogawa.yasufumi
2018-05-03 16:08 ` Ferruh Yigit
2018-05-15 16:55 ` Ferruh Yigit
2018-05-16 9:08 ` Yasufumi Ogawa
2018-05-16 14:06 ` Trishan de Lanerolle
2018-05-17 7:09 ` Yasufumi Ogawa
2018-05-17 14:25 ` Trishan de Lanerolle
2018-05-18 8:03 ` Yasufumi Ogawa
2018-05-24 12:29 ` Ferruh Yigit
2018-05-25 20:45 ` Yasufumi Ogawa [this message]
2018-04-23 6:28 ` [spp] [PATCH 3/7] spp_nfv: " ogawa.yasufumi
2018-04-23 6:28 ` [spp] [PATCH 4/7] controller: " ogawa.yasufumi
2018-04-23 6:28 ` [spp] [PATCH 5/7] spp_vm: " ogawa.yasufumi
2018-04-23 6:28 ` [spp] [PATCH 6/7] shared: " ogawa.yasufumi
2018-04-23 6:28 ` [spp] [PATCH 7/7] spp_vf: " ogawa.yasufumi
2018-05-10 11:10 ` [spp] [PATCH v2 00/13] Update license to SPDX ogawa.yasufumi
2018-05-10 11:10 ` [spp] [PATCH v2 01/13] license: add bsd-3-clause text ogawa.yasufumi
2018-05-10 11:10 ` [spp] [PATCH v2 02/13] spp_primary: convert license to SPDX tag ogawa.yasufumi
2018-05-10 11:10 ` [spp] [PATCH v2 03/13] spp_primary: add NTT to license description ogawa.yasufumi
2018-05-10 11:10 ` [spp] [PATCH v2 04/13] spp_nfv: convert license to SPDX tag ogawa.yasufumi
2018-05-10 11:10 ` [spp] [PATCH v2 05/13] spp_nfv: add NTT to license description ogawa.yasufumi
2018-05-10 11:10 ` [spp] [PATCH v2 06/13] controller: convert license of Intel to SPDX tag ogawa.yasufumi
2018-05-10 11:10 ` [spp] [PATCH v2 07/13] controller: add NTT " ogawa.yasufumi
2018-05-10 11:10 ` [spp] [PATCH v2 08/13] controller: add NTT to license description ogawa.yasufumi
2018-05-23 20:05 ` [spp] [PATCH v3 0/8] Update license to SPDX ogawa.yasufumi
2018-05-23 20:05 ` [spp] [PATCH v3 1/8] license: add bsd-3-clause text ogawa.yasufumi
2018-05-23 20:05 ` [spp] [PATCH v3 2/8] spp_primary: convert license to SPDX tag ogawa.yasufumi
2018-05-23 20:05 ` [spp] [PATCH v3 3/8] spp_nfv: " ogawa.yasufumi
2018-05-23 20:05 ` [spp] [PATCH v3 4/8] controller: convert license of Intel " ogawa.yasufumi
2018-05-23 20:05 ` [spp] [PATCH v3 5/8] controller: add NTT " ogawa.yasufumi
2018-05-23 20:05 ` [spp] [PATCH v3 6/8] spp_vm: convert license to " ogawa.yasufumi
2018-05-23 20:05 ` [spp] [PATCH v3 7/8] shared: " ogawa.yasufumi
2018-05-23 20:06 ` [spp] [PATCH v3 8/8] spp_vf: " ogawa.yasufumi
2018-05-24 12:37 ` [spp] [PATCH v3 0/8] Update license to SPDX Ferruh Yigit
2018-05-24 12:42 ` Ferruh Yigit
2018-05-10 11:17 ` [spp] [PATCH v2 09/13] spp_vm: convert license to SPDX tag ogawa.yasufumi
2018-05-10 11:17 ` [spp] [PATCH v2 10/13] spp_vm: add NTT to license description ogawa.yasufumi
2018-05-10 11:18 ` [spp] [PATCH v2 11/13] shared: convert license to SPDX tag ogawa.yasufumi
2018-05-10 11:18 ` [spp] [PATCH v2 12/13] shared: add NTT to license description ogawa.yasufumi
2018-05-10 11:19 ` [spp] [PATCH v2 13/13] spp_vf: convert license to SPDX tag ogawa.yasufumi
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=2e69caeb-0fa2-3da3-8339-7efc58fee7ed@gmail.com \
--to=usufumu@gmail.com \
--cc=ferruh.yigit@intel.com \
--cc=jim.st.leger@intel.com \
--cc=ogawa.yasufumi@lab.ntt.co.jp \
--cc=spp@dpdk.org \
--cc=tdelanerolle@linuxfoundation.org \
--cc=tim.odriscoll@intel.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
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;
as well as URLs for NNTP newsgroup(s).