7
4%
master: 4%

Ran 27 May 2020 10:09AM UTC

Files 114

Run time 11s

Badge

Committed 27 May 2020 09:35AM UTC coverage: 39.231%. Remained the same

Job # CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES=makemanuals

Build Type

push

travis-ci

Committed by

fingolfin

Commit Message

buildsys: fix two regressions and add more tests

That first change modified $DEPFLAGS to ensure that .d files contain
targets for .lo files and not for the .d files -- the bug here was use
of the $@ variable which can have two values: the name of the .lo file
or the name of the .d file, depending on why the combined build rule for
the .lo/.d pair was triggered. This is easily fixed by using $OBJFILE
instead.

This bug meant that e.g. `make print-VAR` which should never lead to
anything being compiled could trigger compilation of stuff anyway.

Secondly, we want all *.lo files to depend on config.h; but the old way
to do that, which was to just let $(OBJS) depend on config.h, does not
work reliably anymore due to our combined .lo/.d rule: if the
compilation is triggered by the .d file being regenerated, we end up
calling the C compiler before config.h was regenerated. We could have
fixed this by adding a dependency from $(DEPFILES) to config.h, but it
seemed more natural to add config.h as a dependency to the rules which
actually invoke the compiler.

Finally we add a bunch of tests to `testbuildsys`, which ought to have
caught the two issues fixed above, plus some more.

Run Details

18249 of 46517 relevant lines covered (39.23%)

22315939.94 hits per line

Source Files on job 12025.7 (CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES=makemanuals)

gap-system / gap / 12025 / 7
4%
master: 4%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 12025.7 (CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES=makemanuals)

gap-system / gap / 12025 / 7 4% master: 4%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 12025.7 (CFLAGS="--coverage -O2 -g" CXXFLAGS="--coverage -O2 -g" LDFLAGS="--coverage" CONFIGFLAGS="--enable-debug" COVERALLS_PARALLEL=true TEST_SUITES=makemanuals)

gap-system / gap / 12025 / 7
4%
master: 4%

README BADGES
x