On Fri, Jan 11, 2013 at 6:29 AM, Vadim Bendebury <vben...@chromium.org> wrote: > On Thu, Jan 10, 2013 at 9:05 AM, Simon Glass <s...@chromium.org> wrote: >> Hi Vadim, >> >> On Wed, Jan 9, 2013 at 6:00 PM, Vadim Bendebury <vben...@chromium.org> wrote: >>> To make it usable in git trees not providing a patch checker >>> implementation, add a command line option, allowing to suppress patch >>> check. While we are at it, sort debug options alphabetically. >>> >>> Also, do not raise an exception if checkpatch.pl is not found - just >>> print an error message suggesting to use the new option, and return >>> nonzero status. >>> >>> . unit test passes: >>> $ ./patman -t >>> <unittest.result.TestResult run=7 errors=0 failures=0> >>> . successfully used patman in the autotest tree to generate a patch >>> email (with --no-check option) >>> . successfully used patman in the u-boot tree to generate a patch >>> email >>> . `patman --help' now shows command line options ordered >>> alphabetically >>> >>> Signed-off-by: Vadim Bendebury <vben...@chromium.org> >> >> Acked-by: Simon Glass <s...@chromium.org>
Applied to u-boot-x86, thanks. >> >> BTW what is this patch against - it doesn't seem to apply to mainline for me. >> > > Right, it depends on the previous patches which still have not been > applied to the upstream ToT. > > cheers, > /vb > >>> --- >>> Changes in v4: >>> . now properly corrected typo in the error message >>> >>> Changes in v3: >>> . corrected typo in the error message >>> >>> Changes in v2: >>> . addressed comments WRT use of double negative >>> . added code to gracefully handle absence of checkpatch.cl >>> >>> tools/patman/checkpatch.py | 10 +++++----- >>> tools/patman/patman.py | 14 ++++++++++---- >>> 2 files changed, 15 insertions(+), 9 deletions(-) >> [snip] >> >> Regards, >> Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot