diff mbox series

license: remove exception for old pmdinfogen

Message ID 20210204101015.3111924-1-thomas@monjalon.net (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series license: remove exception for old pmdinfogen |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/travis-robot warning Travis build: failed
ci/iol-testing warning Testing issues

Commit Message

Thomas Monjalon Feb. 4, 2021, 10:10 a.m. UTC 
  The C version of pmdinfogen was replaced by a Python implementation.
The license exception can be removed.

Fixes: dd2ea14772f5 ("buildtools: remove C implementation of pmdinfogen")

Reported-by: John McNamara <john.mcnamara@intel.com>
Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 license/exceptions.txt | 1 -
 1 file changed, 1 deletion(-)

Comments

Bruce Richardson Feb. 4, 2021, 10:25 a.m. UTC | #1 
On Thu, Feb 04, 2021 at 11:10:15AM +0100, Thomas Monjalon wrote:
> The C version of pmdinfogen was replaced by a Python implementation.
> The license exception can be removed.
> 
> Fixes: dd2ea14772f5 ("buildtools: remove C implementation of pmdinfogen")
> 
> Reported-by: John McNamara <john.mcnamara@intel.com>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
John McNamara Feb. 4, 2021, 10:35 a.m. UTC | #2 
> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Thursday, February 4, 2021 10:10 AM
> To: dev@dpdk.org
> Cc: Mcnamara, John <john.mcnamara@intel.com>; Neil Horman
> <nhorman@tuxdriver.com>; Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
> Subject: [PATCH] license: remove exception for old pmdinfogen
> 
> The C version of pmdinfogen was replaced by a Python implementation.
> The license exception can be removed.
> 

Ignore my similar version.

Acked-by: John McNamara <john.mcnamara@intel.com>
Hemant Agrawal Feb. 4, 2021, 10:42 a.m. UTC | #3 
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
Thomas Monjalon Feb. 5, 2021, 5:12 p.m. UTC | #4 
04/02/2021 11:35, Mcnamara, John:
> From: Thomas Monjalon <thomas@monjalon.net>
> > 
> > The C version of pmdinfogen was replaced by a Python implementation.
> > The license exception can be removed.
> 
> Ignore my similar version.
> 
> Acked-by: John McNamara <john.mcnamara@intel.com>

Actually I prefer the wording of your version.
I keep the acks from this thread and apply to yours.
diff mbox series

Patch

diff --git a/license/exceptions.txt b/license/exceptions.txt
index 3afb996ed9..636c69b9bf 100644
--- a/license/exceptions.txt
+++ b/license/exceptions.txt
@@ -16,5 +16,4 @@  SPDX Identifier     TB Approval Date  GB Approval Date  File name
 2.BSD-2-Clause      10/23/2019        12/18/2019        lib/librte_eal/windows/include/getopt.h
 3.ISC AND
   BSD-2-Clause      10/23/2019        12/18/2019        lib/librte_eal/windows/getopt.c
-4.GPL-2.0           09/25/2019        12/18/2019        buildtools/pmdinfogen/pmdinfogen.*
 ---------------------------------------------------------------------------------------------------