From dfb51bc816cde2cb345f8a0300205e6ac95a2065 Mon Sep 17 00:00:00 2001 From: Boris Kolpackov Date: Mon, 30 Mar 2020 15:30:08 +0200 Subject: Switch to project variable visibility by default --- build2/b.cxx | 4 ++-- build2/cli/init.cxx | 10 ++++------ 2 files changed, 6 insertions(+), 8 deletions(-) (limited to 'build2') diff --git a/build2/b.cxx b/build2/b.cxx index eafa1dc..b74433d 100644 --- a/build2/b.cxx +++ b/build2/b.cxx @@ -1422,7 +1422,7 @@ main (int argc, char* argv[]) for (const variable_override& o: ctx->var_overrides) { - if (o.ovr.visibility != variable_visibility::normal) + if (o.ovr.visibility != variable_visibility::global) continue; // If we have a directory, enter the scope, similar to how we do @@ -1445,7 +1445,7 @@ main (int argc, char* argv[]) { // Ours is either project (%foo) or scope (/foo). // - if (o.ovr.visibility == variable_visibility::normal) + if (o.ovr.visibility == variable_visibility::global) continue; scope& s (o.dir diff --git a/build2/cli/init.cxx b/build2/cli/init.cxx index b0ff4f9..c68a62f 100644 --- a/build2/cli/init.cxx +++ b/build2/cli/init.cxx @@ -41,13 +41,11 @@ namespace build2 { auto& vp (rs.var_pool ()); - // Note: some overridable, some not. + // The special config.cli=false value is recognized as an explicit + // request to leave the module unconfigured. // - // The config.cli=false is recognized as an explicit request to leave - // the module unconfigured. - // - vp.insert ("config.cli", true); - vp.insert ("config.cli.options", true); + vp.insert ("config.cli"); + vp.insert ("config.cli.options"); //@@ TODO: split version into componets (it is stdver). // -- cgit v1.1