Bug 1183 – ./waf and gcov/lcov integration broken

Bug 1183 - ./waf and gcov/lcov integration broken


Summary:	./waf and gcov/lcov integration broken

Status:	RESOLVED FIXED

Product:	ns-3
Classification:	Unclassified
Component:	build system
Version:	pre-release
Hardware:	All All

Importance:	P5 normal
Assigned To:	Mitch Watrous

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2011-06-07 11:03 UTC by Tom Henderson
Modified:	2011-09-30 17:56 UTC (History)
CC List:	2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Tom Henderson 2011-06-07 11:03:36 UTC

First part of Kirill's post on 27 May 2011:
http://mailman.isi.edu/pipermail/ns-developers/2011-May/009048.html

I was able to confirm this; also, I found that --enable-lcov seems to disable tests even if --enable-tests is also passed to configure.

Comment 1 Tom Henderson 2011-06-07 11:05:06 UTC

we need to add this to the buildbot script before closing this bug

Comment 2 Gustavo J. A. M. Carneiro 2011-09-23 06:46:53 UTC

I think this is fixed in ns-3-dev.  Buildbot I have no idea.

Comment 3 Tom Henderson 2011-09-23 17:23:05 UTC

I am not sure it is completely fixed.  I see bindings coverage and .h coverage but no .cc coverage results.

Comment 4 Gustavo J. A. M. Carneiro 2011-09-23 20:37:11 UTC

Well, ns-3 builds with the cov cflags, and the cov command is run when requested.  Not sure what else is needed.  I think the commands used are unchanged from the very first ns-3, so it probably isnt' a regression.

Comment 5 Tom Henderson 2011-09-24 11:36:15 UTC

(In reply to comment #4)
> Well, ns-3 builds with the cov cflags, and the cov command is run when
> requested.  Not sure what else is needed.  I think the commands used are
> unchanged from the very first ns-3, so it probably isnt' a regression.

I will look into it further at this point, by looking at older releases, and testing this outside of waf.

Comment 6 Tom Henderson 2011-09-30 17:56:45 UTC

fixed in changesets 5d13d02dae0f and 97606ed04882