On Sun, 2009-11-15 at 20:27 +1100, Brad Hards wrote: > Hi, > > I'd like to do some refactoring, but that always makes me a bit nervous.... > > I thought I'd try to get coverage testing going. I've not had a lot of > success > with gcov and shared libraries, but it looks like it may be working now. > > Anyway, a patch against trunk is attached. > > The idea is that you pass the --enable-coverage configure time option, with > something like: > ./configure --prefix=/usr/local/samba --enable-coverage > > Do a full, clean build: > make > (note, you may need to disable ccache) > > You install (or otherwise make sure you're using the libraries you've just > built) > > You run whatever tests you're trying to evaluate. > ./bin/openchangeclient -m > ./bin/mapitest > ./bin/mapiprofile --list > > You run "make coverage": > make coverage > > The nice HTML results are in ./covresults: > konqueror covresults/index.html > > This is still a bit rough, but I'd like to get some feedback and suggestions > on it from anyone who is interested. > > Fixes for the patch, and suggestions for a set of tests to run (e.g. with the > buildbot) would also be appreciated.
Hi Brad, I've given a look at your coverage patch. I must confess I'm not very familiar with gcov, so I prefer not to comment or discuss buildbot integration or similar integration. I'd rather give you the lead on this, since you're been the ONE that implemented buildbot and you've provided significant effort on our mapitest tool ;-) Few remarks anyway - hope you'll find them useful: 1. Apply the patch! Using gcov sounds to me like a reliable way to improve openchange code and I follow you a 100% on this. I don't want this patch to end its life on the devel ml. 2. make coverage fails with libmapiadmin, probably missing reference within the build system on this. Cheers, Julien Julien Kerihuel [email protected] OpenChange Project Manager GPG Fingerprint: 0B55 783D A781 6329 108A B609 7EF6 FE11 A35F 1F79
signature.asc
Description: This is a digitally signed message part
_______________________________________________ devel mailing list [email protected] http://mailman.openchange.org/listinfo/devel
