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

Alejandro Colomar
2024-06-26T18:41:10Z

[-- Attachment #1: Type: text/plain, Size: 1356 bytes --]

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.

> Also I did hit some mandoc warnings on this series, I'll post a v3 in a moment
> with those things fixed. Thanks,

Thank you!

Have a lovely day!
Alex

>
> Josef

--
<https://www.alejandro-colomar.es/>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]