From 9342bc50505746d44731ee1357566e6b973d3ae5 Mon Sep 17 00:00:00 2001 From: Lucas De Marchi Date: Tue, 27 Mar 2018 00:21:33 -0700 Subject: [PATCH] build: invert default for lttng Since it's a debug option, particularly for the scheduler, leave it disabled by default and let the user enable on demand. --- Tools/ardupilotwaf/cxx_checks.py | 6 +++--- wscript | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/Tools/ardupilotwaf/cxx_checks.py b/Tools/ardupilotwaf/cxx_checks.py index b4e06cf0e8..7444896880 100644 --- a/Tools/ardupilotwaf/cxx_checks.py +++ b/Tools/ardupilotwaf/cxx_checks.py @@ -191,13 +191,13 @@ def check_package(cfg, env, libname): @conf def check_lttng(cfg, env): + if not cfg.options.enable_lttng: + cfg.msg("Checking for 'lttng-ust':", 'disabled', color='YELLOW') + return False if cfg.env.STATIC_LINKING: # lttng-ust depends on libdl which means it can't be used in a static build cfg.msg("Checking for 'lttng-ust':", 'disabled for static build', color='YELLOW') return False - if cfg.options.disable_lttng: - cfg.msg("Checking for 'lttng-ust':", 'disabled', color='YELLOW') - return False return check_package(cfg, env, 'lttng-ust') diff --git a/wscript b/wscript index 0f28702e8e..e3f52cf492 100644 --- a/wscript +++ b/wscript @@ -106,9 +106,9 @@ order to save typing. default=False, help='Enable benchmarks.') - g.add_option('--disable-lttng', action='store_true', + g.add_option('--enable-lttng', action='store_true', default=False, - help="Don't use lttng even if supported by board and dependencies available") + help="Enable lttng integration") g.add_option('--disable-libiio', action='store_true', default=False,