* checkpatch: false positive "does MAINTAINERS need updating?" warning
@ 2019-10-07 4:44 Masahiro Yamada
2019-10-07 6:52 ` Joe Perches
0 siblings, 1 reply; 2+ messages in thread
From: Masahiro Yamada @ 2019-10-07 4:44 UTC (permalink / raw)
To: Joe Perches; +Cc: Linux Kernel Mailing List
Hi Joe,
I ran checkpatch.pl against the following:
https://lore.kernel.org/patchwork/patch/1136334/
I did update MAINTAINERS, but I still get
"does MAINTAINERS need updating?" warning.
Why?
$ scripts/checkpatch.pl
0001-doc-move-namespaces.rst-out-of-kbuild-directory.patch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#18:
Documentation/{kbuild => admin-guide}/namespaces.rst | 0
total: 0 errors, 1 warnings, 14 lines checked
NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.
0001-doc-move-namespaces.rst-out-of-kbuild-directory.patch has style
problems, please review.
NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: checkpatch: false positive "does MAINTAINERS need updating?" warning
2019-10-07 4:44 checkpatch: false positive "does MAINTAINERS need updating?" warning Masahiro Yamada
@ 2019-10-07 6:52 ` Joe Perches
0 siblings, 0 replies; 2+ messages in thread
From: Joe Perches @ 2019-10-07 6:52 UTC (permalink / raw)
To: Masahiro Yamada; +Cc: Linux Kernel Mailing List
On Mon, 2019-10-07 at 13:44 +0900, Masahiro Yamada wrote:
> Hi Joe,
>
>
> I ran checkpatch.pl against the following:
> https://lore.kernel.org/patchwork/patch/1136334/
>
>
> I did update MAINTAINERS, but I still get
> "does MAINTAINERS need updating?" warning.
> Why?
Because checkpatch is not able to determine
that you updated MAINTAINERS correctly.
Ignore checkpatch output when you know better.
>
>
> $ scripts/checkpatch.pl
> 0001-doc-move-namespaces.rst-out-of-kbuild-directory.patch
> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> #18:
> Documentation/{kbuild => admin-guide}/namespaces.rst | 0
>
> total: 0 errors, 1 warnings, 14 lines checked
>
> NOTE: For some of the reported defects, checkpatch may be able to
> mechanically convert to the typical style using --fix or --fix-inplace.
>
> 0001-doc-move-namespaces.rst-out-of-kbuild-directory.patch has style
> problems, please review.
>
> NOTE: If any of the errors are false positives, please report
> them to the maintainer, see CHECKPATCH in MAINTAINERS.
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-10-07 6:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-07 4:44 checkpatch: false positive "does MAINTAINERS need updating?" warning Masahiro Yamada
2019-10-07 6:52 ` Joe Perches
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.