[PATCH 16/24] syntax-check: Remove 90 char limit for 'argv' files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Splitting lines with arguments causes in many cases a rewrap if the
arguments are modified making it harder to see what actually changed.

In upcoming patches some rewrapping of 'args' files will be removed so
remove this check first.

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
 build-aux/syntax-check.mk | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/build-aux/syntax-check.mk b/build-aux/syntax-check.mk
index a9f3aaf45f..7d13e10bae 100644
--- a/build-aux/syntax-check.mk
+++ b/build-aux/syntax-check.mk
@@ -775,14 +775,6 @@ sc_spec_indentation:
 	  echo '$(ME): skipping test $@: cppi not installed' 1>&2; \
 	fi

-# Long lines can be harder to diff; too long, and git send-email chokes.
-# For now, only enforce line length on files where we have intentionally
-# fixed things and don't want to regress.
-sc_prohibit_long_lines:
-	@prohibit='.{90}' \
-	in_vc_files='\.arg[sv]' \
-	halt='Wrap long lines in expected output files' \
-	  $(_sc_search_regexp)

 # Prefer the new URL listing over the old street address listing when
 # calling out where to get a copy of the [L]GPL.  Also, while we have
-- 
2.30.2




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux