devtools: increase column width to 100

Message ID 20201016071250.27471-1-rmody@marvell.com (mailing list archive)
State Superseded, archived
Delegated to: Thomas Monjalon
Headers
Series devtools: increase column width to 100 |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/iol-broadcom-Performance success Performance Testing PASS
ci/Intel-compilation success Compilation OK
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-testing fail Testing issues
ci/iol-mellanox-Performance success Performance Testing PASS

Commit Message

Rasesh Mody Oct. 16, 2020, 7:12 a.m. UTC
  In line with Linux, increase the default column width from 80 to 100
characters for patch line length check.

Linux upstream commit reference bdc48fa11e46f867ea4d75fa59ee87a7f48be144

Signed-off-by: Rasesh Mody <rmody@marvell.com>
---
 devtools/checkpatches.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

David Marchand Oct. 16, 2020, 1:11 p.m. UTC | #1
On Fri, Oct 16, 2020 at 9:13 AM Rasesh Mody <rmody@marvell.com> wrote:
>
> In line with Linux, increase the default column width from 80 to 100
> characters for patch line length check.
>
> Linux upstream commit reference bdc48fa11e46f867ea4d75fa59ee87a7f48be144

Duplicate for https://patchwork.dpdk.org/patch/70970/
  

Patch

diff --git a/devtools/checkpatches.sh b/devtools/checkpatches.sh
index 78a408ef9..4fbd6fd45 100755
--- a/devtools/checkpatches.sh
+++ b/devtools/checkpatches.sh
@@ -15,7 +15,7 @@  VALIDATE_NEW_API=$(dirname $(readlink -f $0))/check-symbol-change.sh
 # Codespell can also be enabled by setting DPDK_CHECKPATCH_CODESPELL to a valid path
 # to a dictionary.txt file if dictionary.txt is not in the default location.
 codespell=${DPDK_CHECKPATCH_CODESPELL:-enable}
-length=${DPDK_CHECKPATCH_LINE_LENGTH:-80}
+length=${DPDK_CHECKPATCH_LINE_LENGTH:-100}
 
 # override default Linux options
 options="--no-tree"