<div dir="ltr">Should this check .c, instead of .cpp ?</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 14, 2017 at 11:57 AM, Sandro Santilli <span dir="ltr"><<a href="mailto:strk@kbt.io" target="_blank">strk@kbt.io</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: strk<br>
Date: 2017-09-14 08:57:04 -0700 (Thu, 14 Sep 2017)<br>
New Revision: 15732<br>
<br>
Modified:<br>
   trunk/GNUmakefile.in<br>
Log:<br>
Add check-no-trailing-blanks Makefile rule and run on `make check`<br>
<br>
Only checks .h and .c files, and skip those in .gitignore<br>
<br>
Modified: trunk/GNUmakefile.in<br>
==============================<wbr>==============================<wbr>=======<br>
--- trunk/GNUmakefile.in        2017-09-14 15:56:58 UTC (rev 15731)<br>
+++ trunk/GNUmakefile.in        2017-09-14 15:57:04 UTC (rev 15732)<br>
@@ -1,6 +1,6 @@<br>
 #-----------------------------<wbr>------------------------<br>
 #<br>
-# Configuration directives are in postgis_config.h<br>
+# Configuration directives are in postgis_config.h<br>
 #<br>
 #-----------------------------<wbr>------------------------<br>
<br>
@@ -35,8 +35,13 @@<br>
 # TODO: drop 'test' target..<br>
 test: check<br>
<br>
-check: all docs-check<br>
+check: all docs-check check-no-trailing-blanks<br>
<br>
+check-no-trailing-blanks:<br>
+       ! find . -name '*.cpp' -o -name '*.h' | \<br>
+               grep -vf .gitignore | \<br>
+               xargs grep -Pn '[\t ]$$'<br>
+<br>
 installcheck:<br>
        RUNTESTFLAGS="$(RUNTESTFLAGS) --extension" $(MAKE) check<br>
<br>
@@ -45,7 +50,7 @@<br>
 distclean-local: clean-local<br>
        rm -Rf autom4te.cache<br>
        rm -f GNUmakefile<br>
-       rm -f config.log config.cache config.status<br>
+       rm -f config.log config.cache config.status<br>
        rm -f postgis_config.h<br>
<br>
 maintainer-clean-local: distclean-local<br>
@@ -53,7 +58,7 @@<br>
        rm -f config.guess config.rpath config.sub<br>
        rm -f libtool install-sh ltmain.sh<br>
<br>
-maintainer-clean:<br>
+maintainer-clean:<br>
        @echo '-----------------------------<wbr>-------------------------'<br>
        @echo 'This command is intended for maintainers to use; it'<br>
        @echo 'deletes files that may need special tools to rebuild.'<br>
@@ -85,7 +90,7 @@<br>
 templategis-uninstall:<br>
        $(MAKE) -C extras/template_gis uninstall<br>
<br>
-docs:<br>
+docs:<br>
        $(MAKE) -C doc html<br>
<br>
 docs-clean:<br>
@@ -96,7 +101,7 @@<br>
<br>
 comments:<br>
        $(MAKE) -C doc comments<br>
-<br>
+<br>
 cheatsheets:<br>
        $(MAKE) -C doc cheatsheets<br>
<br>
@@ -132,7 +137,7 @@<br>
<br>
 astyle:<br>
        ./astyle.sh<br>
-<br>
+<br>
 commit:<br>
        $(MAKE) astyle && $(MAKE) clean && $(MAKE) check && svn commit<br>
<br>
@@ -142,7 +147,7 @@<br>
 svnrebase: authors.git<br>
        git svn rebase --authors-file authors.git<br>
<br>
-postgis_svn_revision.h:<br>
+postgis_svn_revision.h:<br>
        $(PERL) utils/<a href="http://svn_repo_revision.pl" rel="noreferrer" target="_blank">svn_repo_revision.pl</a><br>
<br>
 .PHONY: utils liblwgeom ChangeLog raster postgis_svn_revision.h<br>
<br>
______________________________<wbr>_________________<br>
postgis-tickets mailing list<br>
<a href="mailto:postgis-tickets@lists.osgeo.org">postgis-tickets@lists.osgeo.<wbr>org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/postgis-tickets" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/postgis-<wbr>tickets</a></blockquote></div><br></div>