Re: [PATCH v2] tools: use $(PYTHON) to call cppcheck tools

[Julien Grall <julien@xxxxxxx>]

Hi Bertrand,

On 03/08/2022 14:29, Bertrand Marquis wrote:
> > On 3 Aug 2022, at 14:01, Julien Grall <julien@xxxxxxx> wrote:
> >
> > Hi Bertrand,
> >
> > On 03/08/2022 13:56, Bertrand Marquis wrote:
> > > > On 3 Aug 2022, at 13:55, Bertrand Marquis <Bertrand.Marquis@xxxxxxx> wrote:
> > > >
> > > >
> > > >
> > > > > On 3 Aug 2022, at 13:10, Julien Grall <julien@xxxxxxx> wrote:
> > > > >
> > > > > Hi,
> > > > >
> > > > > On 03/08/2022 13:05, Jan Beulich wrote:
> > > > > > On 03.08.2022 13:57, Bertrand Marquis wrote:
> > > > > > > When calling python tools to convert misra documentation or merge
> > > > > > > cppcheck xml files, use $(PYTHON).
> > > > > > > While there fix misra document conversion script to be executable.
> > > > > > >
> > > > > > > Fixes: 57caa53753 (xen: Add MISRA support to cppcheck make rule)
> > > > > > > Fixes: 43aa3f6e72 xen/build: Add cppcheck and cppcheck-html make rules
> > > > > > Nit: These two titles would want expressing consistently. If I
> > > > > > end up committing this, I'll try to remember to adjust.
> > > > > There are actually a few issues with the Fixes tags. From the doc [1] :
> > > > > - the sha1 should be 12 characters (rather than 10)
> > > > > - the format should be:
> > > > > Fixes: <comidid> ("Commit title")
> > > > You want me to resend and fix that ?
> > If Jan doesn't want to fix it on commit. Then I would be happy to do it.
> Thanks a lot
I have committed with the adjustments discussed.

Cheers,

--
Julien Grall