From 47181a164cbfc888d81931e6e3950c379db2f786 Mon Sep 17 00:00:00 2001 From: Florian Larysch Date: Fri, 22 Aug 2025 23:27:54 +0200 Subject: [PATCH] docs/manual: mention checkpackageignore file Currently, the manual doesn't mention the checkpackageignore file at all, leaving the user under the assumption that running "make check-package" and running utils/check-package directly on the package they have touched would be equivalent, when the former evaluates the checkpackageignore file while the latter does not. This could cause them to forget to remove stale entries from that file. Point out this difference in behavior. Signed-off-by: Florian Larysch [Arnout: add that stale lines should be removed from .checkpackageignore] Signed-off-by: Arnout Vandecappelle --- docs/manual/contribute.adoc | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/docs/manual/contribute.adoc b/docs/manual/contribute.adoc index ffaff6f857..f77aff9d6c 100644 --- a/docs/manual/contribute.adoc +++ b/docs/manual/contribute.adoc @@ -321,9 +321,12 @@ Now check the coding style for the changes you committed: $ utils/docker-run make check-package ---- -Now, you are ready to generate then submit your patch set. +This command (unlike calling +utils/check-package+ directly) will also warn +about any stale entries in the +.checkpackageignore+ file that your patches +might have resolved. If this is the case, make sure to remove those lines +from +.checkpackageignore+ as well. -To generate it, run: +Now, you are ready to generate then submit your patch set. To generate it, run: ---- $ git format-patch -M -n -s -o outgoing origin/master