Home | History | Annotate | Download | only in conf
History log of /src/sys/conf/lint.mk
RevisionDateAuthorComments
 1.5  27-Aug-2022  rillig sys/conf/link.mk: clean up

The modifier ':C' did not need the modifier 'g', as there couldn't ever
be more than one match per word. Using the modifier ':from=to' is easier
to read.

Align the variable assignments. There was no point in having 3 different
alignment styles in the same file.

Remove underscore from .for iteration variable, as it is not needed.

No functional change.
 1.4  02-May-2021  rillig sys/conf/lint.mk: fix order of command line options

The option -o must come before the first filename.

Now it is possible to lint kern_exec.c at least. The next failure is in
netbsd32_machdep.c:

netbsd32_machdep.c(395): error: illegal bit-field size: 255 [36]

That line in netbsd32_process_read_fpregs reads:

__CTASSERT(sizeof(*regs) == sizeof(struct save87));

This is probably a bug in lint. The struct save87 contains 3 uint16_t,
followed by a union containing a uint64_t, followed by a packed struct
with size 10. The combination of packed and padding is suspicious.
 1.3  02-May-2021  rillig sys/conf/lint.mk: allow GCC extensions in the kernel

The syntax errors due to the __asm statements are now gone. The file
kern_exec.c only produces a few warnings now. But after running that
file through lint1, the main lint complains about wrong usage. This is
due to this call:

${LINT} ... -i $< -o $@

The main lint has never supported -o after the first filename, so it
complains. It would have been helpful if lint had given any hint as to
which option was invalid. Anyway, the next step is to reorder the
arguments. As it is now, the code can never have worked.
 1.2  02-May-2021  rillig sys/conf/lint.mk: a small step for linting the kernel

Due to the missing path, the following commands had failed:

$ cd src/sys/arch/amd64/compile/GENERIC
$ nbmake-amd64-lint kern_exec.ln
nbmake: don't know how to make kern/kern_exec.c. Stop

After fixing the path, "make kern_exec.ln" fails with:

sys/kern/kern_exec.c(65): error: syntax error '"' [249]

The affected line contains:

__KERNEL_RCSID(0, "...");

The macro __KERNEL_RCSID expands to __asm("some strings"). Since
KERNLINTFLAGS is missing the -g, lint does not recognize __asm as
keyword and tries to parse it as an identifier instead, expecting a
variable or function declaration.
 1.1  29-Aug-2015  uebayasi branches: 1.1.2; 1.1.18; 1.1.38;
Move less important part out of Makefile.kern.inc.
 1.1.38.1  13-May-2021  thorpej Sync with HEAD.
 1.1.18.2  03-Dec-2017  jdolecek update from HEAD
 1.1.18.1  29-Aug-2015  jdolecek file lint.mk was added on branch tls-maxphys on 2017-12-03 11:36:57 +0000
 1.1.2.2  22-Sep-2015  skrll Sync with HEAD
 1.1.2.1  29-Aug-2015  skrll file lint.mk was added on branch nick-nhusb on 2015-09-22 12:05:56 +0000

RSS XML Feed