[pacman-dev] [PATCH 7/8] validate shell scripts with $(BASH_SHELL) on build

2012-04-12 Thread Dave Reisner
Pass $(OURSCRIPTS) through the bash parser in read only mode to validate syntax. Note that this doesn't actually catch all errors, but it might be useful for developers working on these scripts. Signed-off-by: Dave Reisner --- contrib/Makefile.am |1 + scripts/Makefile.am |1 + 2 files c

Re: [pacman-dev] [PATCH 7/8] validate shell scripts with $(BASH_SHELL) on build

2012-04-12 Thread Allan McRae
On 13/04/12 00:54, Dave Reisner wrote: > Pass $(OURSCRIPTS) through the bash parser in read only mode to validate > syntax. Note that this doesn't actually catch all errors, but it might > be useful for developers working on these scripts. > > Signed-off-by: Dave Reisner > --- > contrib/Makefile

Re: [pacman-dev] [PATCH 7/8] validate shell scripts with $(BASH_SHELL) on build

2012-04-13 Thread Florian Pritz
On 13.04.2012 05:30, Allan McRae wrote: > On 13/04/12 00:54, Dave Reisner wrote: >> Pass $(OURSCRIPTS) through the bash parser in read only mode to validate >> syntax. Note that this doesn't actually catch all errors, but it might >> be useful for developers working on these scripts. >> >> Signed-

Re: [pacman-dev] [PATCH 7/8] validate shell scripts with $(BASH_SHELL) on build

2012-04-13 Thread Allan McRae
On 13/04/12 20:49, Florian Pritz wrote: > On 13.04.2012 05:30, Allan McRae wrote: >> On 13/04/12 00:54, Dave Reisner wrote: >>> Pass $(OURSCRIPTS) through the bash parser in read only mode to validate >>> syntax. Note that this doesn't actually catch all errors, but it might >>> be useful for devel

Re: [pacman-dev] [PATCH 7/8] validate shell scripts with $(BASH_SHELL) on build

2012-04-13 Thread Dave Reisner
On Fri, Apr 13, 2012 at 6:57 AM, Allan McRae wrote: > On 13/04/12 20:49, Florian Pritz wrote: > > On 13.04.2012 05:30, Allan McRae wrote: > >> On 13/04/12 00:54, Dave Reisner wrote: > >>> Pass $(OURSCRIPTS) through the bash parser in read only mode to > validate > >>> syntax. Note that this doesn

Re: [pacman-dev] [PATCH 7/8] validate shell scripts with $(BASH_SHELL) on build

2012-04-13 Thread Allan McRae
On 13/04/12 22:57, Dave Reisner wrote: > On Fri, Apr 13, 2012 at 6:57 AM, Allan McRae wrote: > >> On 13/04/12 20:49, Florian Pritz wrote: >>> On 13.04.2012 05:30, Allan McRae wrote: On 13/04/12 00:54, Dave Reisner wrote: > Pass $(OURSCRIPTS) through the bash parser in read only mode to >