[Libvir] 0/8 add syntax checks

Daniel P. Berrange berrange at redhat.com
Tue Jan 29 14:52:52 UTC 2008


On Fri, Jan 25, 2008 at 11:16:15PM +0100, Jim Meyering wrote:
> The first patch simply removes the useless if tests.
> Following it, I introduce a "make syntax-check" framework
> similar to what I use in coreutils.  In it, most tests
> are disabled, except the new one that tests for the unnecessary
> "if"-before-free.

We should add a call to 'make syntax-check' into the autobuild.sh script
so that these checks get on the automated builds each night.

> I deliberately keep these tests in a separate Makefile.maint
> file, and pull them in via GNUmakefile so that they are not
> subject to the burdensome portability constraints associated
> with regular Makefile.am contents.  I.e., if you run "make
> syntax-check", you are expected to have GNU make as well
> as a reasonable set of working tools.  Thus, we don't have to
> worry about whether egrep works or whether e.g., perl is present.

Sounds like a good strategy

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 




More information about the libvir-list mailing list