mirror of
https://git.proxmox.com/git/mirror_frr
synced 2025-07-09 20:01:10 +00:00
Merge pull request #13801 from LabNConsulting/chopps/latest-checkpatch
Import latest checkpatch.pl from linux
This commit is contained in:
commit
2be9ce6cd8
1249
doc/developer/checkpatch.rst
Normal file
1249
doc/developer/checkpatch.rst
Normal file
File diff suppressed because it is too large
Load Diff
@ -27,6 +27,7 @@ dev_RSTFILES = \
|
|||||||
doc/developer/building-libunwind-note.rst \
|
doc/developer/building-libunwind-note.rst \
|
||||||
doc/developer/building-libyang.rst \
|
doc/developer/building-libyang.rst \
|
||||||
doc/developer/building.rst \
|
doc/developer/building.rst \
|
||||||
|
doc/developer/checkpatch.rst \
|
||||||
doc/developer/cli.rst \
|
doc/developer/cli.rst \
|
||||||
doc/developer/conf.py \
|
doc/developer/conf.py \
|
||||||
doc/developer/cross-compiling.rst \
|
doc/developer/cross-compiling.rst \
|
||||||
|
@ -868,14 +868,17 @@ clang-format
|
|||||||
https://clang.llvm.org/docs/ClangFormat.html
|
https://clang.llvm.org/docs/ClangFormat.html
|
||||||
|
|
||||||
checkpatch.sh
|
checkpatch.sh
|
||||||
|
checkpatch.pl
|
||||||
|
|
||||||
In the Linux kernel source tree there is a Perl script used to check
|
In the Linux kernel source tree there is a Perl script used to check
|
||||||
incoming patches for style errors. FRR uses an adapted version of this
|
incoming patches for style errors. FRR uses a shell script front end and an
|
||||||
script for the same purpose. It can be found at
|
adapted version of the perl script for the same purpose. These scripts can
|
||||||
:file:`tools/checkpatch.sh`. This script takes a git-formatted diff or
|
be found at :file:`tools/checkpatch.sh` and :file:`tools/checkpatch.pl`.
|
||||||
patch file, applies it to a clean FRR tree, and inspects the result to catch
|
This script takes a git-formatted diff or patch file, applies it to a clean
|
||||||
potential style errors. Running this script on your patches before
|
FRR tree, and inspects the result to catch potential style errors. Running
|
||||||
submission is highly recommended. The CI system runs this script as well and
|
this script on your patches before submission is highly recommended. The CI
|
||||||
will comment on the PR with the results if style errors are found.
|
system runs this script as well and will comment on the PR with the results
|
||||||
|
if style errors are found.
|
||||||
|
|
||||||
It is run like this::
|
It is run like this::
|
||||||
|
|
||||||
@ -916,6 +919,10 @@ checkpatch.sh
|
|||||||
If the script finds one or more WARNINGs it will exit with 1. If it finds
|
If the script finds one or more WARNINGs it will exit with 1. If it finds
|
||||||
one or more ERRORs it will exit with 2.
|
one or more ERRORs it will exit with 2.
|
||||||
|
|
||||||
|
For convenience the Linux documentation for the :file:`tools/checkpatch.pl`
|
||||||
|
script has been included unmodified (i.e., it has not been updated to
|
||||||
|
reflect local changes) :doc:`here <checkpatch>`
|
||||||
|
|
||||||
|
|
||||||
Please remember that while FRR provides these tools for your convenience,
|
Please remember that while FRR provides these tools for your convenience,
|
||||||
responsibility for properly formatting your code ultimately lies on the
|
responsibility for properly formatting your code ultimately lies on the
|
||||||
|
2300
tools/checkpatch.pl
2300
tools/checkpatch.pl
File diff suppressed because it is too large
Load Diff
Loading…
Reference in New Issue
Block a user