DPDK CI discussions
 help / color / mirror / Atom feed
* [dpdk-ci] Wrong Checkpatch failure in Intel CI
@ 2020-03-31 19:51 Jerin Jacob Kollanukkaran
  2020-03-31 20:03 ` Thomas Monjalon
  0 siblings, 1 reply; 6+ messages in thread
From: Jerin Jacob Kollanukkaran @ 2020-03-31 19:51 UTC (permalink / raw)
  To: zhaoyan.chen, ci, sys_stv
  Cc: Nithin Kumar Dabilpuram, Thomas Monjalon, david.marchand

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@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@dpdk.org

    Signed-off-by: David Marchand <david.marchand@redhat.com>
    Signed-off-by: Nithin Dabilpuram <ndabilpuram@marvell.com>
    Acked-by: Neil Horman <nhorman@tuxdriver.com>
    Tested-by: Jerin Jacob <jerinj@marvell.com>


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2020-04-01 12:18 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-31 19:51 [dpdk-ci] Wrong Checkpatch failure in Intel CI Jerin Jacob Kollanukkaran
2020-03-31 20:03 ` Thomas Monjalon
2020-03-31 20:12   ` [dpdk-ci] [EXT] " Jerin Jacob Kollanukkaran
2020-03-31 20:39     ` Thomas Monjalon
2020-04-01 11:22       ` Jerin Jacob Kollanukkaran
2020-04-01 12:18         ` Thomas Monjalon

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).