From 1f7f1c68d7abf170388495796743c6a01cc52370 Mon Sep 17 00:00:00 2001 From: Boris Kolpackov Date: Sun, 29 May 2022 10:26:52 +0200 Subject: New model: reset non-user package configuration to defaults Also, further rework of package skeleton in preparation for negotiation. --- bpkg/package-skeleton.cxx | 525 ++++++++++++++++++++++++++++----------------- bpkg/package-skeleton.hxx | 99 ++++++--- bpkg/pkg-build.cxx | 2 +- bpkg/pkg-configure.cxx | 29 ++- tests/pkg-build.testscript | 3 +- 5 files changed, 428 insertions(+), 230 deletions(-) diff --git a/bpkg/package-skeleton.cxx b/bpkg/package-skeleton.cxx index aca34aa..a452d4a 100644 --- a/bpkg/package-skeleton.cxx +++ b/bpkg/package-skeleton.cxx @@ -72,10 +72,12 @@ namespace bpkg src_root_ = move (v.src_root_); out_root_ = move (v.out_root_); created_ = v.created_; + verified_ = v.verified_; ctx_ = move (v.ctx_); rs_ = v.rs_; cmd_vars_ = move (v.cmd_vars_); - reflect_names_ = move (v.reflect_names_); + cmd_vars_cache_ = v.cmd_vars_cache_; + dependent_vars_ = move (v.dependent_vars_); reflect_vars_ = move (v.reflect_vars_); reflect_frag_ = move (v.reflect_frag_); @@ -97,8 +99,10 @@ namespace bpkg src_root_ (v.src_root_), out_root_ (v.out_root_), created_ (v.created_), + verified_ (v.verified_), cmd_vars_ (v.cmd_vars_), - reflect_names_ (v.reflect_names_), + cmd_vars_cache_ (v.cmd_vars_cache_), + dependent_vars_ (v.dependent_vars_), reflect_vars_ (v.reflect_vars_), reflect_frag_ (v.reflect_frag_) { @@ -150,19 +154,19 @@ namespace bpkg } void package_skeleton:: - load_defaults () + load_defaults (const strings& dependent_vars) { assert (ctx_ == nullptr); // Should only be called before evaluate_*(). + if (config_srcs_ != nullptr) + load_old_config (); + try { using namespace build2; - using build2::fail; - using build2::endf; - // @@ TODO: c.c.{disfigure|unload}. - - scope& rs (*bootstrap (*this, merge_cmd_vars ())->second.front ()); + scope& rs ( + *bootstrap (*this, merge_cmd_vars (dependent_vars))->second.front ()); // Load project's root.build. // @@ -221,6 +225,9 @@ namespace bpkg } } } + + verified_ = true; // Managed to load without errors. + ctx_ = nullptr; } catch (const build2::failed&) { @@ -228,6 +235,77 @@ namespace bpkg } } + pair package_skeleton:: + verify_sensible (const strings& dependent_vars) + { + assert (ctx_ == nullptr); // Should only be called before evaluate_*(). + + if (config_srcs_ != nullptr) + load_old_config (); + + try + { + using namespace build2; + + // For now we treat any failure to load root.build as bad configuration, + // which is not very precise. One idea to make this more precise would + // be to invent some kind of tagging for "bad configuration" diagnostics + // (e.g., either via an attribute or via special config.assert directive + // or some such). + // + // For now we rely on load_defaults() and load_old_config() to "flush" + // out any unrelated errors (e.g., one of the modules configuration is + // bad, etc). However, if that did not happen naturally, then we must do + // it ourselves. + // + if (!verified_) + { + scope& rs ( + *bootstrap (*this, merge_cmd_vars (strings {}))->second.front ()); + load_root (rs); + + verified_ = true; + ctx_ = nullptr; + } + + scope& rs ( + *bootstrap (*this, merge_cmd_vars (dependent_vars))->second.front ()); + + // Load project's root.build while redirecting the diagnostics stream. + // + ostringstream ds; + auto dg (make_guard ([ods = diag_stream] () {diag_stream = ods;})); + diag_stream = &ds; + + pair r; + try + { + load_root (rs); + r.first = true; + } + catch (const build2::failed&) + { + r.first = false; + r.second = ds.str (); + } + + ctx_ = nullptr; + return r; + } + catch (const build2::failed&) + { + throw failed (); // Assume the diagnostics has already been issued. + } + } + + void package_skeleton:: + dependent_config (strings&& vars) + { + assert (dependent_vars_.empty ()); // Must be called at most once. + + dependent_vars_ = move (vars); + } + // Print the location of a depends value in the specified manifest file. // // Note that currently we only use this function for the external packages. @@ -397,6 +475,9 @@ namespace bpkg // also argue we should diagnose this case and fail not to cause more // confusion. // + // @@ They could also clash with dependent configuration. Probably should + // be handled in the same way (it's just another type of "user"). + // // It seems like the most straightforward way to achieve the desired // semantics with the mechanisms that we have (in other words, without // inventing another "level" of overrides) is to evaluate the reflect @@ -441,12 +522,6 @@ namespace bpkg using build2::fail; using build2::endf; - // Merge old configuration variables into config_vars since otherwise - // they may end up being overridden by reflects. - // - if (config_srcs_ != nullptr) - merge_old_config_vars (); - scope& rs (load ()); istringstream is (refl); @@ -541,10 +616,19 @@ namespace bpkg process (false); - // Add to the map the reflect variables collected previously. + // Add to the map the reflect variables collected previously. Note that + // we can re-purpose the override since we re-populate it. // - for (string& n: reflect_names_) + for (string& n: reflect_vars_) { + // Transform `name=value` to just `name`. + // + { + size_t p (n.find ('=')); + assert (p != string::npos); // We've serialized it so must be there. + n.resize (p); + } + auto p (vars.emplace (&vp.insert (move (n)), value_data {nullptr, 0})); if (p.second) @@ -558,16 +642,8 @@ namespace bpkg // Re-populate everything from the map. // - reflect_names_.clear (); - reflect_frag_.clear (); - -#if 0 - // NOTE: see also collect_config() if enabling this. - // reflect_vars_.clear (); -#else - reflect_vars_ = config_vars_; -#endif + reflect_frag_.clear (); // Collect the config..* variables that were changed by this // and previous reflect clauses. @@ -586,13 +662,13 @@ namespace bpkg const variable& var (*p.first); const value& val (*p.second.val); - reflect_names_.push_back (var.name); - // For the accumulated fragment we always save the original and let // the standard overriding take its course. // serialize_buildfile (reflect_frag_, var.name, val, storage); + // Note: this is currently disabled and is likely very outdated. + // // For the accumulated overrides we have to merge user config_vars_ // with reflect values. Essentially, we have three possibilities: // @@ -617,7 +693,7 @@ namespace bpkg ovr = org; // Case #2. else { - // NOTE: see also above and below if enabling this. + // NOTE: see also below if enabling this. // #if 0 // Case #3. @@ -677,6 +753,8 @@ namespace bpkg // ovr = rs.lookup_override (var, org); #else + // @@ TODO: probably also depends, not just user. + fail << "command line override of reflect clause variable " << var << endf; #endif @@ -706,114 +784,69 @@ namespace bpkg { assert (db_ != nullptr); // Must be called only once. + if (config_srcs_ != nullptr) + load_old_config (); + + // Merge all the variables into a single list in the correct order + // and assign their sources while at it. + // strings vars; vector srcs; - // Check whether the user-specified configuration variable is a project - // variables (i.e., its name start with config.). - // - // Note that some user-specified variables may have qualifications - // (global, scope, etc) but there is no reason to expect any project - // configuration variables to use such qualifications (since they can only - // apply to one project). So we ignore all qualified variables. - // - auto prj_var = [this, p = optional ()] (const string& v) mutable + if (size_t n = (config_vars_.size () + + dependent_vars_.size () + + reflect_vars_.size ())) { - if (!p) - p = "config." + name ().variable (); + // For vars we will steal the first non-empty *_vars_. But for sources + // reserve the space. + // + srcs.reserve (n); // At most that many. - size_t n (p->size ()); + // Check whether the user-specified configuration variable override has + // a project variables (i.e., its name start with config.). + // + // Note that some user-specified variables may have qualifications + // (global, scope, etc) but there is no reason to expect any project + // configuration variables to use such qualifications (since they can + // only apply to one project). So we ignore all qualified variables. + // + auto prj_var = [this, p = optional ()] (const string& v) mutable + { + if (!p) + p = "config." + name ().variable (); - return v.compare (0, n, *p) == 0 && strchr (".=+ \t", v[n]) != nullptr; - }; + size_t n (p->size ()); - if (!reflect_vars_.empty ()) - { - assert (config_srcs_ == nullptr); // Should have been merged. + return v.compare (0, n, *p) == 0 && strchr (".=+ \t", v[n]) != nullptr; + }; - vars = move (reflect_vars_); + // Return the variable name given the variable override. + // + auto var_name = [] (const string& v) + { + size_t p (v.find_first_of ("=+ \t")); + assert (p != string::npos); + return string (v, 0, p); + }; - // Note that if the reflect variables list is not empty, then it also - // contains the user-specified configuration variables, which all come - // first (see above). + // Note that for now we assume the three sets of variables do not clash. + // + // @@ TODO: make sure it's the case for dependent. // - size_t nc (config_vars_.size ()); - if (!vars.empty ()) + // First comes the user configuration. + // + if (!config_vars_.empty ()) { - srcs.reserve (vars.size ()); // At most that many. - // Assign the user source only to user-specified configuration // variables which are project variables (i.e., names start with - // config.). Assign the reflect source to all variables that - // follow the user-specified configuration variables (which can only - // be project variables). + // config.). // - for (size_t j (0); j != vars.size (); ++j) - { - const string& v (vars[j]); - - config_source s; - - if (j < nc) - { - if (prj_var (v)) - s = config_source::user; - else - continue; - } - else - s = config_source::reflect; - - size_t p (v.find_first_of ("=+ \t")); - assert (p != string::npos); - - string n (v, 0, p); - - // Check for a duplicate. - // - auto i (find_if (srcs.begin (), srcs.end (), - [&n] (const config_variable& cv) - { - return cv.name == n; - })); - - if (i != srcs.end ()) - assert (i->source == s); // See evaluate_reflect() for details. - else - srcs.push_back (config_variable {move (n), s}); - } - } - } - else - { - vars = move (config_vars_); - - // If we don't have any reflect variables, then we don't really need to - // load user variables from config.build (or equivalent) and add them to - // config_vars since there is nothing for them to possibly override. So - // all we need to do is to add user variables from config_vars. - // - if (!vars.empty () || config_srcs_ != nullptr) - { - srcs.reserve ((config_srcs_ != nullptr ? config_srcs_->size () : 0) - + vars.size ()); // At most that many. - - if (config_srcs_ != nullptr) - for (const config_variable& v: *config_srcs_) - if (v.source == config_source::user) - srcs.push_back (v); - - for (const string& v: vars) + for (const string& v: config_vars_) { - // Similar logic to the above case. - // if (prj_var (v)) { - size_t p (v.find_first_of ("=+ \t")); - assert (p != string::npos); - - string n (v, 0, p); + string n (var_name (v)); // Check for a duplicate. // @@ -827,10 +860,56 @@ namespace bpkg srcs.push_back (config_variable {move (n), config_source::user}); } } + + vars = move (config_vars_); + } + + // Next dependent configuration. + // + if (!dependent_vars_.empty ()) + { + // These are all project variables. There should also be no duplicates + // by construction (see @@). + // + for (const string& v: dependent_vars_) + srcs.push_back ( + config_variable {var_name (v), config_source::dependent}); + + if (vars.empty ()) + vars = move (dependent_vars_); + else + { + vars.reserve (n); + vars.insert (vars.end (), + make_move_iterator (dependent_vars_.begin ()), + make_move_iterator (dependent_vars_.end ())); + } + } + + // Finally reflect. + // + if (!reflect_vars_.empty ()) + { + // These are all project variables. There should also be no duplicates + // by construction (see evaluate_reflect()). + // + for (const string& v: reflect_vars_) + srcs.push_back ( + config_variable {var_name (v), config_source::reflect}); + + if (vars.empty ()) + vars = move (reflect_vars_); + else + { + vars.reserve (n); + vars.insert (vars.end (), + make_move_iterator (reflect_vars_.begin ()), + make_move_iterator (reflect_vars_.end ())); + } } } - ctx_ = nullptr; // In case we only had conditions. + ctx_ = nullptr; // Free. db_ = nullptr; available_ = nullptr; @@ -838,114 +917,97 @@ namespace bpkg } const strings& package_skeleton:: - merge_cmd_vars () + merge_cmd_vars (const strings& dependent_vars, bool cache) { - // Merge variable overrides (note that the order is important). - // - // We can reasonably assume reflect cannot have global or absolute scope - // variable overrides so we don't need to pass them to context. + // Merge variable overrides (note that the order is important). See also a + // custom/optimized version in load_old_config(). // - const strings* r; - - const strings& v1 (build2_cmd_vars); - const strings& v2 (config_vars_); + if (!cache || !cmd_vars_cache_) + { + const strings& vs1 (build2_cmd_vars); + const strings& vs2 (config_vars_); + const strings& vs3 (dependent_vars); // Should not override. - r = (v2.empty () ? &v1 : v1.empty () ? &v2 : nullptr); + // Try to reuse both vector and string buffers. + // + cmd_vars_.resize (1 + vs1.size () + vs2.size () + vs3.size ()); - if (r == nullptr) - { - if (cmd_vars_.empty ()) // Cached. + size_t i (0); { - cmd_vars_.reserve (v1.size () + v2.size ()); - cmd_vars_.assign (v1.begin (), v1.end ()); - cmd_vars_.insert (cmd_vars_.end (), v2.begin (), v2.end ()); + string& v (cmd_vars_[i++]); + + // If the package is being disfigured, then don't load config.build at + // all. Otherwise, disfigure all package variables (config.**). + // + // Note that this semantics must be consistent with how we actually + // configure the package in pkg_configure(). + // + if (disfigure_) + v = "config.config.unload=true"; + else + { + // Note: must be quoted to preserve the pattern. + // + v = "config.config.disfigure='config."; + v += name ().variable (); + v += "**'"; + } } - r = &cmd_vars_; + for (const string& v: vs1) cmd_vars_[i++] = v; + for (const string& v: vs2) cmd_vars_[i++] = v; + for (const string& v: vs3) cmd_vars_[i++] = v; + + cmd_vars_cache_ = cache; } - return *r; + return cmd_vars_; } - build2::scope& package_skeleton:: - load () + void package_skeleton:: + load_old_config () { - if (ctx_ != nullptr) - return *rs_; + assert (config_srcs_ != nullptr && ctx_ == nullptr); try { using namespace build2; - auto rsi (bootstrap (*this, merge_cmd_vars ())); - scope& rs (*rsi->second.front ()); - - // Load project's root.build as well as potentially accumulated reflect - // fragment. - // - // If we have the accumulated reflect fragment, wedge it just before - // loading root.build (but after initializing config which may load - // config.build and which we wish to override). + // This load that must be done without config.config.disfigure. Also, it + // would be nice to optimize for the common case where the only load is + // to get the old configuration (e.g., config.*.develop) as part of + // collect_config(). So instead of calling merge_cmd_vars() we will do + // own (but consistent) thing. // - // Note that the plan for non-external packages is to extract the - // configuration and then load it with config.config.load and this - // approach should work for that case too. - // - function pre; - - if (!reflect_frag_.empty ()) + const strings* cmd_vars; { - pre = [this, &rs] (parser& p) - { - istringstream is (reflect_frag_); - is.exceptions (istringstream::failbit | istringstream::badbit); - - // Note that the fragment is just a bunch of variable assignments - // and thus unlikely to cause any errors. - // - path_name in (""); - p.parse_buildfile (is, in, &rs, rs); - }; - } + assert (!cmd_vars_cache_); // Sanity check (we are always first). - load_root (rs, pre); + const strings& vs1 (build2_cmd_vars); + const strings& vs2 (config_vars_); - setup_base (rsi, - out_root_.empty () ? src_root_ : out_root_, - src_root_); + cmd_vars = (vs2.empty () ? &vs1 : vs1.empty () ? &vs2 : nullptr); - rs_ = &rs; - return rs; - } - catch (const build2::failed&) - { - throw failed (); // Assume the diagnostics has already been issued. - } - } - - void package_skeleton:: - merge_old_config_vars () - { - if (config_srcs_ == nullptr) - return; - - assert (reflect_frag_.empty ()); // Too late. - - ctx_ = nullptr; // Reload. + if (cmd_vars == nullptr) + { + // Note: the order is important (see merge_cmd_vars()). + // + cmd_vars_.reserve (vs1.size () + vs2.size ()); + cmd_vars_.assign (vs1.begin (), vs1.end ()); + cmd_vars_.insert (cmd_vars_.end (), vs2.begin (), vs2.end ()); - try - { - using namespace build2; + cmd_vars = &cmd_vars_; + } + } - scope& rs (*bootstrap (*this, merge_cmd_vars ())->second.front ()); + scope& rs (*bootstrap (*this, *cmd_vars)->second.front ()); // Load project's root.build. // load_root (rs); // Extract and merge old user configuration variables from config.build - // (or equivalent) into config_vars. Then invalidate loaded state in - // order to make them overrides. + // (or equivalent) into config_vars. // auto i (config_vars_.begin ()); // Insert position, see below. @@ -998,9 +1060,72 @@ namespace bpkg } } - config_srcs_ = nullptr; // Merged. - cmd_vars_.clear (); // Invalidated. - ctx_ = nullptr; // Drop. + config_srcs_ = nullptr; + verified_ = true; // Managed to load without errors. + ctx_ = nullptr; + } + catch (const build2::failed&) + { + throw failed (); // Assume the diagnostics has already been issued. + } + } + + build2::scope& package_skeleton:: + load () + { + if (ctx_ != nullptr) + return *rs_; + + if (config_srcs_ != nullptr) + load_old_config (); + + try + { + using namespace build2; + + // We can reuse already merged cmd_vars if this has already been done + // (they don't change during evaluate_*() calls). + // + auto rsi ( + bootstrap (*this, merge_cmd_vars (dependent_vars_, true /* cache */))); + scope& rs (*rsi->second.front ()); + + // Load project's root.build as well as potentially accumulated reflect + // fragment. + // + // If we have the accumulated reflect fragment, wedge it just before + // loading root.build (but after initializing config which may load + // config.build and which we wish to override). + // + // Note that the plan for non-external packages is to extract the + // configuration and then load it with config.config.load and this + // approach should work for that case too. + // + function pre; + + if (!reflect_frag_.empty ()) + { + pre = [this, &rs] (parser& p) + { + istringstream is (reflect_frag_); + is.exceptions (istringstream::failbit | istringstream::badbit); + + // Note that the fragment is just a bunch of variable assignments + // and thus unlikely to cause any errors. + // + path_name in (""); + p.parse_buildfile (is, in, &rs, rs); + }; + } + + load_root (rs, pre); + + setup_base (rsi, + out_root_.empty () ? src_root_ : out_root_, + src_root_); + + rs_ = &rs; + return rs; } catch (const build2::failed&) { diff --git a/bpkg/package-skeleton.hxx b/bpkg/package-skeleton.hxx index dafb979..1b7e019 100644 --- a/bpkg/package-skeleton.hxx +++ b/bpkg/package-skeleton.hxx @@ -62,18 +62,53 @@ namespace bpkg optional src_root, optional out_root); + const package_name& + name () const {return available_->id.name;} + + // The following functions should be called in the following sequence + // (* -- zero or more, ? -- zero or one): + // + // * load_defaults() + // * verify_sensible() + // ? dependent_config() + // * evaluate_enable() | evaluate_reflect() + // collect_config() + // + // Note that a copy of the skeleton is expected to continue with the + // sequence rather than starting from scratch. + // + public: // Load the default values and type information for configuration - // variables of the package. + // variables of the package given a "tentative" dependent configuration. // - // Note: must be called before any evaluate_*() functions. + // @@ TODO: if it will be more convenient to pass it as something other + // than strings, then this can be accomodated. // void - load_defaults (); + load_defaults (const strings& dependent_vars); + + // Verify the specified "tentative" dependents configuration is sensible, + // that is, acceptable to the package. If it is not, then the second half + // of the result contains the diagnostics. + // + // @@ TODO: if it will be more convenient to pass it as something other + // than strings, then this can be accomodated. + // + pair + verify_sensible (const strings& dependent_vars); + + // Incorporate the "final" dependent configuration into subsequent + // evaluations. Dependent configuration variables are expected not to + // clash with user. + // + void + dependent_config (strings&&); // For the following evaluate_*() functions assume that the clause belongs // to the specified (by index) depends value (used to print its location // on failure for an external package). // + // Evaluate the enable clause. // bool @@ -85,22 +120,21 @@ namespace bpkg evaluate_reflect (const string&, size_t depends_index); // Return the accumulated configuration variables (first) and project - // configuration variable sources (second). Note that the arrays are - // not necessarily parallel. + // configuration variable sources (second). Note that the arrays are not + // necessarily parallel (config_vars may contain non-project variables). // - // Note that the reflect variables are merged with config_vars/config_srcs - // and should be used instead rather than in addition to config_vars. + // Note that the dependent and reflect variables are merged with + // config_vars/config_srcs and should be used instead rather than in + // addition to config_vars. // // Note also that this should be the final call on this object. // pair> collect_config () &&; - const package_name& - name () const {return available_->id.name;} - // Implementation details. // + public: // We have to define these because context is forward-declared. Also, copy // constructor has some special logic. // @@ -112,28 +146,32 @@ namespace bpkg package_skeleton& operator= (const package_skeleton&) = delete; private: - // Create the skeleton if necessary and (re)load the build system state. + // Load old user configuration variables from config.build (or equivalent) + // and merge them into config_vars_. // - // Call this function before evaluating every clause. + // This should be done before any attempt to load the configuration with + // config.config.disfigure and, if this did not happen, inside + // collect_config() (since the package will be reconfigured with + // config.config.disfigure). // - build2::scope& - load (); + void + load_old_config (); - // Extract old user configuration variables from config.build (or - // equivalent) and merge them into config_vars_. This is only necessary if - // something (e.g., reflect) could override their values in config.build. + // (Re)load the build system state. // - // @@ Isn't the plan now to reset all configs to defaul which means we - // will probably always have to extract and merge. + // Call this function before evaluating every clause. // - void - merge_old_config_vars (); + build2::scope& + load (); - // Merge command line variable overrides in one list (normally to be + // Merge command line variable overrides into one list (normally to be // passed to bootstrap()). // + // If cache is true, then assume the result can be reused on subsequent + // calls. + // const strings& - merge_cmd_vars (); + merge_cmd_vars (const strings& dependent_vars, bool cache = false); // Implementation details (public for bootstrap()). // @@ -153,13 +191,20 @@ namespace bpkg dir_path out_root_; // If empty, the same as src_root_. bool created_ = false; + bool verified_ = false; unique_ptr ctx_; build2::scope* rs_ = nullptr; - strings cmd_vars_; // Storage for merged build2_cmd_vars and config_vars_. - strings reflect_names_; // Reflect configuration variable names. - strings reflect_vars_; // Reflect configuration variable overrides. - string reflect_frag_; // Reflect configuration variable fragment. + // Storage for merged build2_cmd_vars and config_vars_ and extra overrides + // (like config.config.disfigure). If cache is true, then the existing + // content can be reused. + // + strings cmd_vars_; + bool cmd_vars_cache_ = false; + + strings dependent_vars_; // Dependent configuration variable overrides. + strings reflect_vars_; // Reflect configuration variable overrides. + string reflect_frag_; // Reflect configuration variables fragment. }; } diff --git a/bpkg/pkg-build.cxx b/bpkg/pkg-build.cxx index 4f1207e..68e5c34 100644 --- a/bpkg/pkg-build.cxx +++ b/bpkg/pkg-build.cxx @@ -3041,7 +3041,7 @@ namespace bpkg options, pdb, *ap, - pkg.config_vars, + pkg.config_vars, // @@ Maybe make optional and move? pkg.disfigure, (sp != nullptr ? &sp->config_variables : nullptr), move (src_root), diff --git a/bpkg/pkg-configure.cxx b/bpkg/pkg-configure.cxx index 23ec279..7c0f4bc 100644 --- a/bpkg/pkg-configure.cxx +++ b/bpkg/pkg-configure.cxx @@ -348,6 +348,8 @@ namespace bpkg l4 ([&]{trace << "src_root: " << src_root << ", " << "out_root: " << out_root;}); + bool disfigured (ps.disfigure_); // @@ TMP pass explicitly. + // Verify all our prerequisites are configured and populate the // prerequisites list. // @@ -383,10 +385,34 @@ namespace bpkg l4 ([&]{trace << "buildspec: " << bspec;}); + // Unless this package has been completely disfigured, disfigure all the + // package configuration variables to reset all the old values to + // defaults (all the new user/dependent/reflec values, including old + // user, are returned by collect_config() and specified as overrides). + // Note that this semantics must be consistent with how we load things + // in the package skeleton during configuration negotiation. + // + // Note also that this means we don't really use the dependent and + // reflect sources that we save in the database. But let's keep them + // for the completeness of information (maybe could be useful during + // configuration reset or some such). + // + string dvar; + if (!disfigured) + { + // Note: must be quoted to preserve the pattern. + // + dvar = "config.config.disfigure='config."; + dvar += p->name.variable (); + dvar += "**'"; + } + + // @@ TMP remove when settled done. + // +#if 0 // Deduce the configuration variables which are not reflected anymore // and disfigure them. // - string dvar; for (const config_variable& cv: p->config_variables) { if (cv.source == config_source::reflect) @@ -409,6 +435,7 @@ namespace bpkg } } } +#endif // Configure. // diff --git a/tests/pkg-build.testscript b/tests/pkg-build.testscript index 3ed04a6..2b95adc 100644 --- a/tests/pkg-build.testscript +++ b/tests/pkg-build.testscript @@ -11477,7 +11477,8 @@ test.options += --no-progress +cp -r $src/libhello-1.0.0 ./libhello +cat <+libhello/build/root.build config [bool] config.libhello.develop ?= false - text "develop=$config.libhello.develop" + if ($build.mode != 'skeleton') + text "develop=$config.libhello.develop" EOI +$rep_add libhello --type dir +$rep_fetch -- cgit v1.1