[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