On Fri, May 20, 2011 at 9:36 AM, Steven Dake <[email protected]> wrote: > On 05/19/2011 08:23 PM, Angus Salkeld wrote: >> On Thu, May 19, 2011 at 07:54:20PM -0700, Steven Dake wrote: >>> Allow a make coverity target for those developers with coverity tools >>> available to them.
You should add the coverity target to .PHONY. >> >> Gasp, no configure check for coverity? >> > > http://lolpics.se/pics/482.jpg nice ... > pretty sure nobody would be able to actually configure the code base > with a configure check... Presumably he meant an optional check. :-) Then you could do something like ... coverity: ifeq ($(HAVE_COVERITY),yes) cov-build --dir=cov make cov-analyze --dir cov cov-format-errors --dir cov else @echo "coverity not found ... no coverity for you." @echo "As a consolation prize, here is kirby dancing:" @echo "<("<) ^("^) (^")^ (>")>" @exit 1 endif -- Russell Bryant _______________________________________________ Openais mailing list [email protected] https://lists.linux-foundation.org/mailman/listinfo/openais
