Paul Menzel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/38677 )
Change subject: util/lint/checkpatch: Update to 96 char line length limit ......................................................................
util/lint/checkpatch: Update to 96 char line length limit
Follow up on commit bcbf2deb42 (Documentation/coding_style.md: Update line length limit) and update the script `checkpatch.pl`.
Change-Id: I2b5c5d4cf9f0534298d00e76173cd448e8e13281 Signed-off-by: Paul Menzel pmenzel@molgen.mpg.de --- M util/lint/checkpatch.pl 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/38677/1
diff --git a/util/lint/checkpatch.pl b/util/lint/checkpatch.pl index 1affdb7..c6cbe69 100755 --- a/util/lint/checkpatch.pl +++ b/util/lint/checkpatch.pl @@ -51,7 +51,7 @@ my @exclude = (); #coreboot my $help = 0; my $configuration_file = ".checkpatch.conf"; -my $max_line_length = 80; +my $max_line_length = 96; my $ignore_perl_version = 0; my $minimum_perl_version = 5.10.0; my $min_conf_desc_length = 4;
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38677 )
Change subject: util/lint/checkpatch: Update to 96 char line length limit ......................................................................
Patch Set 1: Code-Review-1
Duplicate of CB:34386
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38677 )
Change subject: util/lint/checkpatch: Update to 96 char line length limit ......................................................................
Patch Set 1: Code-Review-1
See https://review.coreboot.org/cgit/coreboot.git/tree/util/lint/lint-007-checkp...: opts="--max-line-length 96"
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38677 )
Change subject: util/lint/checkpatch: Update to 96 char line length limit ......................................................................
Patch Set 1:
So why am I still getting this sometimes? Is it an old hooks setup?
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38677 )
Change subject: util/lint/checkpatch: Update to 96 char line length limit ......................................................................
Patch Set 1:
Patch Set 1:
So why am I still getting this sometimes? Is it an old hooks setup?
Very likely. I am not getting it at all...
Martin L Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/38677?usp=email )
Change subject: util/lint/checkpatch: Update to 96 char line length limit ......................................................................
Abandoned
This patch has not been touched in over 12 months. Anyone who wants to take over work on this patch, please feel free to restore it and do any work needed to get it merged. If you create a new patch based on this work, please credit the original author.