[dpdk-stable] patch 'devtools: fix check of symbol added as stable API' has been queued to LTS release 18.11.2

Kevin Traynor ktraynor at redhat.com
Tue May 14 16:44:01 CEST 2019


Hi,

FYI, your patch has been queued to LTS release 18.11.2

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 05/20/19. So please
shout if anyone has objections.

Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.

Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable-queue

This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable-queue/commit/920f07edb9d0c6ac65cbd683f92e0ba30e09f7ff

Thanks.

Kevin Traynor

---
>From 920f07edb9d0c6ac65cbd683f92e0ba30e09f7ff Mon Sep 17 00:00:00 2001
From: David Marchand <david.marchand at redhat.com>
Date: Fri, 3 May 2019 16:34:20 +0200
Subject: [PATCH] devtools: fix check of symbol added as stable API

[ upstream commit af14b150df78ec46de919ae4a30efbefbcd44c95 ]

The incriminated commit broke the detection of new symbols skipping the
EXPERIMENTAL step before entering a stable ABI section.
sed won't return an error, check a null output instead.

Fixes: 3630757803ab ("devtools: accept experimental symbol promotion")

Signed-off-by: David Marchand <david.marchand at redhat.com>
---
 devtools/check-symbol-change.sh | 25 +++++++++++++++++--------
 1 file changed, 17 insertions(+), 8 deletions(-)

diff --git a/devtools/check-symbol-change.sh b/devtools/check-symbol-change.sh
index cd59b8b5d..f6f79a883 100755
--- a/devtools/check-symbol-change.sh
+++ b/devtools/check-symbol-change.sh
@@ -111,13 +111,22 @@ check_for_rule_violations()
 			# A symbol can not enter a non experimental
 			# section directly
-			if [ $? -ne 0 ] && [ "$secname" != 'EXPERIMENTAL' ]
+			if [ -z "$oldsecname" ]
 			then
-				echo -n "ERROR: symbol $symname "
-				echo -n "is added in the $secname "
-				echo -n "section, but is expected to "
-				echo -n "be added in the EXPERIMENTAL "
-				echo "section of the version map"
-				ret=1
-				continue
+				if [ "$secname" = 'EXPERIMENTAL' ]
+				then
+					echo -n "INFO: symbol $symname has "
+					echo -n "been added to the "
+					echo -n "EXPERIMENTAL section of the "
+					echo "version map"
+					continue
+				else
+					echo -n "ERROR: symbol $symname "
+					echo -n "is added in the $secname "
+					echo -n "section, but is expected to "
+					echo -n "be added in the EXPERIMENTAL "
+					echo "section of the version map"
+					ret=1
+					continue
+				fi
 			fi
 
-- 
2.20.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2019-05-14 15:39:08.120438741 +0100
+++ 0017-devtools-fix-check-of-symbol-added-as-stable-API.patch	2019-05-14 15:39:07.313558811 +0100
@@ -1 +1 @@
-From af14b150df78ec46de919ae4a30efbefbcd44c95 Mon Sep 17 00:00:00 2001
+From 920f07edb9d0c6ac65cbd683f92e0ba30e09f7ff Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit af14b150df78ec46de919ae4a30efbefbcd44c95 ]
+
@@ -11 +12,0 @@
-Cc: stable at dpdk.org
@@ -19 +20 @@
-index 28f093a9f..c5434f3bb 100755
+index cd59b8b5d..f6f79a883 100755


More information about the stable mailing list