[PATCH] travis: add checkpatch analysis

Roman Stratiienko roman.stratiienko at globallogic.com
Tue Mar 19 15:59:01 CET 2019


Currently I see a lot of checkpatch warnings, that can not be fixed in
xenomai project.
But checkpatch errors usually could be fixed, so we should try I think.


On Tue, Mar 19, 2019 at 4:45 PM Jan Kiszka <jan.kiszka at siemens.com> wrote:

> On 19.03.19 15:28, roman.stratiienko--- via Xenomai wrote:
> > From: Roman Stratiienko <roman.stratiienko at globallogic.com>
> >
> > Check last 10 commits with checkpatch.pl before build
> >
> > Signed-off-by: Roman Stratiienko <roman.stratiienko at globallogic.com>
> > ---
> >   .travis.yml | 1 +
> >   1 file changed, 1 insertion(+)
> >
> > diff --git a/.travis.yml b/.travis.yml
> > index 0cd9e571f..6fb604bf2 100644
> > --- a/.travis.yml
> > +++ b/.travis.yml
> > @@ -78,6 +78,7 @@ before_script:
> >     - popd
> >
> >   script:
> > +  - ${KDIR}/scripts/checkpatch.pl --git HEAD~10..HEAD
> >     - scripts/prepare-kernel.sh --ipipe=/tmp/ipipe.patch --arch=${ARCH}
> --linux=${KDIR}
> >     - pushd ${KDIR}
> >     - make -j $(nproc) olddefconfig
> >
>
> Hmm, not convinced about enforced checking. That will fail the build, even
> if
> there might be a legitimate reason for some exception, no?
>






>
> Jan
>
> --
> Siemens AG, Corporate Technology, CT RDA IOT SES-DE
> Corporate Competence Center Embedded Linux
>


More information about the Xenomai mailing list