[dpdk-ci] Wrong Checkpatch failure in Intel CI
Jerin Jacob Kollanukkaran
jerinj at marvell.com
Tue Mar 31 21:51:19 CEST 2020
Hello,
It looks like "devtools/check-symbol-change.sh" cached in Intel Patchwork CI.
The commit[2] is fixed the issue reported by [1]. The commit[2] already part of
Master but some reason some this cached in CI system and it pick OLD checkpatch infra.
Could someone look into this issue from Intel side?
[1]
http://mails.dpdk.org/archives/test-report/2020-March/123095.html
[2]
commit f08885494f241b115ad66763b585fbfb486d5160
Author: Nithin Dabilpuram <ndabilpuram at marvell.com>
Date: Mon Mar 23 17:26:13 2020 +0530
devtools: fix symbol map change check
Fix check symbol change script to detect new diff file when
it is in between "--- /dev/null" to "b/lib/...".
Current awk line expects line to start with "a/..."
which is not always true for all diffs.
As a result if in_map was '1' earlier, it will not be changed
to '0' and we get check patch errors which are not true.
Fixes: 4bec48184e33 ("devtools: add checks for ABI symbol addition")
Cc: stable at dpdk.org
Signed-off-by: David Marchand <david.marchand at redhat.com>
Signed-off-by: Nithin Dabilpuram <ndabilpuram at marvell.com>
Acked-by: Neil Horman <nhorman at tuxdriver.com>
Tested-by: Jerin Jacob <jerinj at marvell.com>
More information about the ci
mailing list