Commit 799c0d3f authored by Andrew Dunstan's avatar Andrew Dunstan

Use a file of patterns of filenames to exclude from pgindent runs, instead if...

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.
parent 399ea9e7
$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 pgindent
======== ========
...@@ -8,7 +8,7 @@ This can format all PostgreSQL *.c and *.h files, but excludes *.y, and ...@@ -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. 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 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 ...@@ -18,10 +18,8 @@ This can format all PostgreSQL *.c and *.h files, but excludes *.y, and
4) Run pgindent: 4) Run pgindent:
find . -name '*.[ch]' -type f -print | find . -name '*.[ch]' -type f -print | \
egrep -v '/s_lock.h|/ecpg/test/expected/|/snowball/libstemmer/' | egrep -v -f src/tools/pgindent/exclude_file_patterns | \
egrep -v '/ecpg/include/(sqlda|sqltypes)\.h$' |
egrep -v '/ecpg/include/preproc/struct\.h$' |
xargs -n100 pgindent src/tools/pgindent/typedefs.list xargs -n100 pgindent src/tools/pgindent/typedefs.list
5) Remove any files that generate errors. 5) Remove any files that generate errors.
......
#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$
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment