Bug 1101 – Sqlite stats disappeared from configuration file src/wscript

Bug 1101 - Sqlite stats disappeared from configuration file src/wscript


Summary:	Sqlite stats disappeared from configuration file src/wscript

Status:	RESOLVED FIXED

Product:	ns-3
Classification:	Unclassified
Component:	build system
Version:	ns-3-dev
Hardware:	PC Linux

Importance:	P5 trivial
Assigned To:	Gustavo J. A. M. Carneiro

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2011-04-16 12:09 UTC by julien.mineraud
Modified:	2011-05-18 17:39 UTC (History)
CC List:	2 users (show)

See Also:

Attachments
Add the line to src/wscript to enable Sqlite stats (21.91 KB, text/plain) 2011-04-16 12:11 UTC, julien.mineraud	Details
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 julien.mineraud 2011-04-16 12:09:38 UTC

Comment 1 julien.mineraud 2011-04-16 12:11:54 UTC

Created attachment 1073 [details]
Add the line to src/wscript to enable Sqlite stats

Comment 2 Gustavo J. A. M. Carneiro 2011-05-18 13:34:33 UTC

Comment on attachment 1073 [details]
Add the line to src/wscript to enable Sqlite stats

this is not a patch

Comment 3 Gustavo J. A. M. Carneiro 2011-05-18 13:40:29 UTC

src/stats/wscript is being included, and it conditionally enables sqlite stuff.  Are you sure this is a problem in ns-3-dev?  A lot of module reorganization happened lately...

Comment 4 julien.mineraud 2011-05-18 13:57:15 UTC

(In reply to comment #3)
> src/stats/wscript is being included, and it conditionally enables sqlite stuff.
>  Are you sure this is a problem in ns-3-dev?  A lot of module reorganization
> happened lately...

I cloned the ns-3-dev repository from the Internet to be sure the complication was appearing from the pull I did in the first place.

Then I compare the two script files (one from an old repository without the reorganisation and the new one), the diff told me that the line that includes the stats module disappeared.

I added this line back and the SQLite enable is back.

Comment 5 Tommaso Pecorella 2011-05-18 17:37:27 UTC

You're right, the line missing is:

    conf.sub_config('stats')

lost in space... I'll re-add it.