Hi Jan,
>>>>
>>>
>>> Now this is (to me at least) absurd: I'm removing a file, just to find the
>>> pipeline
>>> fails because cppcheck doesn't like docs/misra/exclude-list.json containing
>>> a
>>> reference to a non-existing file.
>>>
>>> I'll amend the commit with
>>>
>>> --- a/docs/misra/exclude-list.json
>>> +++ b/docs/misra/exclude-list.json
>>> @@ -170,10 +170,6 @@
>>> "comment": "Imported from Linux, ignore for now"
>>> },
>>> {
>>> - "rel_path": "common/symbols-dummy.c",
>>> - "comment": "The resulting code is not included in the final
>>> Xen binary, ignore for now"
>>> - },
>>> - {
>>> "rel_path": "crypto/*",
>>> "comment": "Origin is external and documented in
>>> crypto/README.source"
>>> },
>>>
>>> but I think such tidying should be optional.
>>
>> Can you share the error?
>
> + xen/scripts/xen-analysis.py --run-cppcheck --cppcheck-misra -- -j16
> ERROR: Issue with reading file
> /builds/xen-project/hardware/xen-staging/docs/misra/exclude-list.json:
> Malformed path: common/symbols-dummy.c refers to
> /builds/xen-project/hardware/xen-staging/xen/common/symbols-dummy.c that does
> not exists
>
Oh ok I see, seems to me a good feedback from the tool to keep everything
consistent.
Cheers,
Luca
