Re: [PATCH v2 0/2] man-pages: add documentation for statmount/listmount

Josef Bacik
2024-06-26T21:50:36Z

On Wed, Jun 26, 2024 at 05:44:07PM -0400, Josef Bacik wrote:
> On Wed, Jun 26, 2024 at 08:41:06PM +0200, Alejandro Colomar wrote:
> > Hi Josef,
> >
> > On Wed, Jun 26, 2024 at 02:04:34PM GMT, Josef Bacik wrote:
> > > On Wed, Jun 26, 2024 at 07:02:26PM +0200, Alejandro Colomar wrote:
> > > > You can
> > > >
> > > > $ make lint build check -j8 -k
> > > > $ make lint build check
> > > >
> > > > to see the full list of failures.
> > >
> > > I captured the output of
> > >
> > > make lint build check -j8 -k > out.txt 2>&1
> >
> > Hmmm, please do the following steps to have a cleaner log:
> >
> > ## Let's see if the build system itself complains:
> > $ make nothing >out0.txt
> >
> > ## Skip checkpatch stuff:
> > $ make -t lint-c-checkpatch
> >
> > ## Make fast stuff that doesn't break:
> > $ make lint build check -j8 -k >/dev/null 2>/dev/null
> >
> > ## Now log the remaining errors:
> > $ make lint build check >out.txt 2>&1
> >
> > > and pasted it here
> > >
> > > https://paste.centos.org/view/ed3387a9
> >
> > BTW, you seem to also be missing cppcheck(1), which at least in Debian
> > is provided in the cppcheck package. It also seems to be available in
> > Fedora, but I don't know if your system will have it.
> >
> > > Clang messes up a bunch for a variety of different pages.
> >
> > I suspect the Clang errors to be due to missing libbsd-dev.
> >
>
> Ok well those two things fixed most of the errors, now I'm down to just this
>
> https://paste.centos.org/view/acd71eb7
>

Err that just shows the one error, this is all of them

https://paste.centos.org/view/b68c2fb1

Thanks,

Josef