Re: 0/8 add syntax checks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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  -=| 

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]