From patchwork Fri Oct 19 21:09:12 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [11/11] CheckInTests: Add a sparse requirement Date: Fri, 19 Oct 2012 11:09:12 -0000 From: Chuck Lever X-Patchwork-Id: 192816 Message-Id: <20121019210912.53119.16757.stgit@seurat.1015granger.net> To: fedfs-utils-devel@oss.oracle.com Sparse provides some level of sanity checking for new commits, so make it a check-in test requirement. Signed-off-by: Chuck Lever --- doc/CheckInTests | 11 ++++++----- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/doc/CheckInTests b/doc/CheckInTests index d8f2c17..37bc2ef 100644 --- a/doc/CheckInTests +++ b/doc/CheckInTests @@ -87,24 +87,25 @@ This is basic stuff that's easy to run, and should be working for bisecting to be useful. 1. "autogen.sh" and "configure" must run without error - 2. "./configure --enable-silent-rules; make -s" must complete without - error, and without new compiler warnings + 2. "CC=cgcc ./configure --enable-silent-rules; make -s" must complete + without error, and without new compiler or sparse warnings 3. "make doxy" must complete with no errors or warnings 4. "make distcheck" must complete without error 5. "git status" must produce no output on a clean tree 6. "git log -p" shows no new white space damage 7. "git clean -d -f -x" does not break anything -Note that to preserve bisectability, the tree MUST build after each -commit. +To preserve bisectability, the tree MUST build after each commit. Try to maintain documentation coherency with code. Check code against man pages and the INSTALL and README documents. At some point, we'd like: + 1. Automated checking of coding style and white space - 2. fortify, sparse, or splint testing + 2. Automated fortify or splint testing during check-in 3. Automated unit tests + 4. Regular runs against coverity Per-release