Re: Scheduling lcov code coverage script

Hi,

On Wed, Dec 10, 2014 at 10:35 AM, Norbert Thiebaud <nthiebaud@gmail.com>
wrote:
> On Wed, Dec 10, 2014 at 3:16 AM, Maarten Hoes <hoes.maarten@gmail.com>
wrote:
> > Hrm.
> >
> >
> > It appears I get the openldap build error when I do the build like this
> > (like it happens with the tb script) :
> >
> > $ LDFLAGS+='-fprofile-arcs'
> > $ CFLAGS+='-fprofile-arcs -ftest-coverage'
> > $ CXXFLAGS+='-fprofile-arcs -ftest-coverage'
> > $ CPPFLAGS+='-fprofile-arcs -ftest-coverage'
> > $ export LDFLAGS CFLAGS CXXFLAGS CPPFLAGS
> > $ ./autogen.sh --enable-python=internal --disable-online-update
> > --without-system-libs --without-system-headers
> > $ make build-nocheck
> >
> >
> >
> > But not when I do the build like this (like I do on the cmdline, or is
done
> > in the lcov-report.sh script) :
> >
> > $ LDFLAGS+='-fprofile-arcs' CFLAGS+='-fprofile-arcs -ftest-coverage'
> > CXXFLAGS+='-fprofile-arcs -ftest-coverage' \
> > CPPFLAGS+='-fprofile-arcs -ftest-coverage' ./autogen.sh
> > --enable-python=internal --disable-online-update \
> > --without-system-libs --without-system-headers
> > $ make build-nocheck
>
> then the easiest work around is to define a custom phase for that
> profile that overide the configure (aka autogen) step
> so that it does it the second way (and remove the *FLAGS from the
> profile's config
>
> Norbert


Ive taken another look at your suggestion, and I dont know where it went
wrong with my earlier attempts, but now it seems that at least Im getting
past the stage of building internal openldap correctly. Of course, this
workaround does mean that now there is a custom do_autogen() in phases.sh
that contains hardcoded FLAGS and options for autogen.sh, instead of
setting them in 'config' and autogen.lastrun. Im now doing a full build and
testrun to see if the rest of phase.sh does what I intended it to do.

If still interested (and my tests completes as expected), I could submit
the contents of my .tb directory somewhere, but im not too sure if that's
still needed or relevant anymore with your current modifications/version of
the script: it lets tb do a make build-nocheck, lcov-report -b, make check,
lcov-report -a, in separate steps (inside of a custom do_make in phases.sh)
instead of your 'lcov-report -f' everything in a single run.


- Maarten