Bug 2459 – Add ./waf --check-config option to reprint the summary of optional features from the configuration

Bug 2459 - Add ./waf --check-config option to reprint the summary of optional features from the configuration


Summary:	Add ./waf --check-config option to reprint the summary of optional features f...

Status:	RESOLVED FIXED

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

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

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2016-08-04 16:01 UTC by Peter Barnes
Modified:	2016-09-20 02:20 UTC (History)
CC List:	2 users (show)

See Also:

Attachments
proposed patch (9.40 KB, patch) 2016-08-04 16:01 UTC, Peter Barnes	Details \| Diff
Complete patch (9.40 KB, patch) 2016-08-04 16:03 UTC, Peter Barnes	Details \| Diff
3rd try (9.40 KB, patch) 2016-08-04 16:07 UTC, Peter Barnes	Details \| Diff
proposed patch (9.40 KB, patch) 2016-08-04 16:08 UTC, Peter Barnes	Details \| Diff
Show Obsolete (3) 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 Peter Barnes 2016-08-04 16:01:58 UTC

Created attachment 2509 [details]
proposed patch

This patch adds a waf build option to reprint the summary of optional features which ends the configure step.

$ ./waf --help
...
  --check-profile       print out current build profile
  --check-config        Print the current configuration.
...

$ ./waf --check-config
Waf: Entering directory `./build'
---- Summary of optional NS-3 features:
Build profile                 : debug
...

I had to refactor the printing of the build profile to get it to work reliably during both configure and build phases.

Comment 1 Peter Barnes 2016-08-04 16:03:42 UTC

Created attachment 2510 [details]
Complete patch

Missed a diff in the original patch

Comment 2 Peter Barnes 2016-08-04 16:07:07 UTC

Created attachment 2511 [details]
3rd try

Finally think I have the bug number correct in RELEASE_NOTES and CHANGES.html

Comment 3 Peter Barnes 2016-08-04 16:08:42 UTC

Created attachment 2512 [details]
proposed patch

Comment 4 Tom Henderson 2016-08-04 16:10:28 UTC

+1 from me to merge shortly; putting in LAST CALL state

Comment 5 Tom Henderson 2016-09-20 02:20:24 UTC

pushed in changeset 12329:e0d961707c93