From: Andrew Dunstan <andrew@dunslane.net>
Date: Thu, 1 Apr 2010 14:44:39 +0000 (+0000)
Subject: Use a file of patterns of filenames to exclude from pgindent runs, instead if using... 
X-Git-Tag: REL9_0_BETA1~125
X-Git-Url: http://git.postgresql.org/gitweb/?a=commitdiff_plain;h=799c0d3f65082591583086949bba2be159a36e32;p=postgresql.git

Use a file of patterns of filenames to exclude from pgindent runs, instead if using multiple invocations of egrep. Add perl ppport.h to the current list.
---

diff --git a/src/tools/pgindent/README b/src/tools/pgindent/README
index fa774583547..d1719f7e1d2 100644
--- a/src/tools/pgindent/README
+++ b/src/tools/pgindent/README
@@ -1,4 +1,4 @@
-$PostgreSQL: pgsql/src/tools/pgindent/README,v 1.39 2010/02/26 18:00:15 momjian Exp $
+$PostgreSQL: pgsql/src/tools/pgindent/README,v 1.40 2010/04/01 14:44:39 adunstan Exp $
 
 pgindent
 ========
@@ -8,7 +8,7 @@ This can format all PostgreSQL *.c and *.h files, but excludes *.y, and
 
 1) Change directory to the top of the build tree.
 
-2) Download the typedef file from the builfarm:
+2) Download the typedef file from the buildfarm:
 
 	wget -O src/tools/pgindent/typedefs.list http://www.pgbuildfarm.org/cgi-bin/typedefs.pl
 
@@ -18,10 +18,8 @@ This can format all PostgreSQL *.c and *.h files, but excludes *.y, and
 
 4) Run pgindent:
 
-	find . -name '*.[ch]' -type f -print |
-	egrep -v '/s_lock.h|/ecpg/test/expected/|/snowball/libstemmer/' |
-	egrep -v '/ecpg/include/(sqlda|sqltypes)\.h$' |
-	egrep -v '/ecpg/include/preproc/struct\.h$' |
+	find . -name '*.[ch]' -type f -print | \
+	egrep -v -f src/tools/pgindent/exclude_file_patterns | \
 	xargs -n100 pgindent src/tools/pgindent/typedefs.list
 
 5) Remove any files that generate errors.
diff --git a/src/tools/pgindent/exclude_file_patterns b/src/tools/pgindent/exclude_file_patterns
new file mode 100644
index 00000000000..49d61ac1631
--- /dev/null
+++ b/src/tools/pgindent/exclude_file_patterns
@@ -0,0 +1,7 @@
+#list of file patterns to exclude from pg_indent runs
+/s_lock\.h$
+/ecpg/test/expected/
+/snowball/libstemmer/
+/ecpg/include/(sqlda|sqltypes)\.h$
+/ecpg/include/preproc/struct\.h$
+/pl/plperl/ppport\.h$