]> git.99rst.org Git - openwrt-packages.git/commit
CONTRIBUTING: document what a patch has to look like
authorJosef Schlehofer <redacted>
Fri, 7 Aug 2026 21:16:33 +0000 (23:16 +0200)
committerJosef Schlehofer <redacted>
Thu, 17 Sep 2026 08:03:50 +0000 (10:03 +0200)
commit9c7f2f0429396f7d057020afc45d5997629ce7f9
tree0a7b1c093933f7fa9adbbe385e8329352d517de3
parent8c29bd78acca44d2815f465d6208033e5114dcfe
CONTRIBUTING: document what a patch has to look like

Three things are expected of a patch under <package>/patches/ and none of
them were written down for contributors.

The header has to be git am compatible, with the From/From:/Date:/Subject:
lines, which the formality check enforces. The body has to say where the
change stands with respect to upstream, so a later maintainer can tell a
backport that a version bump makes redundant from a cross-compile fix that
has to stay. And the diff has to come from the buildroot's refresh target,
which invokes quilt with -p ab --no-index --no-timestamps and
QUILT_DIFF_OPTS=-p; a git diff carries index lines and timestamps and lacks
the function name in the hunk headers, so it gets sent back even when it
applies.

Write all three down, and say what each quilt option is for rather than
just naming the command, so the reason survives the next time someone
wonders whether the timestamps matter.

llm-review-rules.md already tells the review bot about the refresh target
and the upstream reference. That file is the bot's instructions and it
points contributors here, so this is where the requirement belongs.

Signed-off-by: Josef Schlehofer <redacted>
CONTRIBUTING.md
git clone https://git.99rst.org/PROJECT