From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40078) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a9vJ2-0005IQ-MB for qemu-devel@nongnu.org; Fri, 18 Dec 2015 08:42:50 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a9vIy-0001Vh-Vj for qemu-devel@nongnu.org; Fri, 18 Dec 2015 08:42:48 -0500 Received: from mx1.redhat.com ([209.132.183.28]:46367) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a9vIy-0001Va-QE for qemu-devel@nongnu.org; Fri, 18 Dec 2015 08:42:44 -0500 Date: Fri, 18 Dec 2015 13:42:40 +0000 From: "Dr. David Alan Gilbert" Message-ID: <20151218134240.GE2459@work-vm> References: <1450438575-21613-1-git-send-email-berrange@redhat.com> <5673FDB0.6040900@redhat.com> <20151218130555.GB7228@redhat.com> <56740537.9010106@redhat.com> <20151218133513.GD2459@work-vm> <20151218133745.GC7228@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20151218133745.GC7228@redhat.com> Subject: Re: [Qemu-devel] [PATCH] scripts: provide a script for checking glib symbol usage List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Daniel P. Berrange" Cc: Peter Maydell , Paolo Bonzini , qemu-devel@nongnu.org, =?iso-8859-1?Q?Marc-Andr=E9?= Lureau * Daniel P. Berrange (berrange@redhat.com) wrote: > On Fri, Dec 18, 2015 at 01:35:13PM +0000, Dr. David Alan Gilbert wrote: > > * Paolo Bonzini (pbonzini@redhat.com) wrote: > > > > > > > > > On 18/12/2015 14:05, Daniel P. Berrange wrote: > > > > > > + > > > > > > +cs-glib-syms: > > > > > > + @perl scripts/glib-syms.pl $(GLIB_SYMS_LIST) $(C_CODE_FILES) > > > > > > > > > > > > > > > Does this need to be included, or could it be a separate Makefile > > > > > invoked with e.g. make -f scripts/Makefile.style? > > > > > > > > Any particular reason to favour that over include ? I did it this > > > > way because QEMU in general seems to be biased towards includes > > > > and not recursive make > > > > > > That would not be recursive make, but rather a completely separate > > > Makefile to be manually invoked with -f. > > > > Hmm but wouldn't this Makefile also be a good place for small-fast > > style check scripts that could be included in make check ? > > Nothing about "make check" is fast, so we could probably just wire it > all into make check by default, as qtest & the block tests take some > considerable time to run. IOW people who want speed will be running > "make check-unit" or individual tests already. "make check" takes under 2m on my laptop, so I take that to be reasonably fast, i.e. I'd be happy to add other small (few second) tests to it. e.g. with your Makefile included like you had done, the scripts to check ordering of some files could go in it. Dave > > Regards, > Daniel > -- > |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| > |: http://libvirt.org -o- http://virt-manager.org :| > |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| > |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK