Re: [PATCH 2.6.2] Documentation/SubmittingPatches

[Riley Williams]

Hi Bryan.

 > I've been trying to get my feet wet by submitting trivial
 > patches to various maintainers and the responses have been,
 > "you're not submitting you patches correctly". It seems most
 > developers/maintainers want a diff done like this:
 >
 > cd /source-tree
 > diff -u linux-2.6.2/FileToPatch.orig linux-2.6.2/FileToPatch
 >
 > instead of the "SubmitingPatches" document way:
 >
 > cd /source-tree/linux-2.6.2
 > diff -u FileToPatch.orig FileToPatch
 >
 > It would be _great_ if the Documentation was more accurate to
 > the taste of developers/maintainers...

Obviously, I can't speak for the rest of the developers, but my
personal stance has always been this: If I can identify both the
kernel version the patch is against and where in the source tree
is being patched, I will apply the patch. If not, I won't.

I suspect the change you note is due to the number of people who
submit patches within a directory in the source tree that don't
indicate either of the above facts. If the answers aren't obvious
from the email, I run `find -name ${FILE}` from the root directory
of the current source tree and if there's not more than three
possibilities for the file, I will try patching each of them to
see whether the patch will apply to any of them. Beyond that, I
want further information from the submitter.

One other thing I've found useful is to increase the context given
in the diff by using -u5 rather than just -u as that considerably
increases the chances of a successful patch when other patches have
already been applied. However, that's very much personal preference
and probably not shared by other developers.

As an example, if somebody sends me a patch that tweaks a file
called Makefile with no indication of either of the above - well,
let's just say I've better things to do with my time than to
search through the various files of that name in the various
kernel source trees to identify the correct one...

Best wishes from Riley.
---
 * Nothing as pretty as a smile, nothing as ugly as a frown.

---
Outgoing mail is certified Virus Free.
Checked by AVG anti-virus system (http://www.grisoft.com).
Version: 6.0.576 / Virus Database: 365 - Release Date: 30-Jan-2004

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/