From b7e93fde78cad6c2ed90e9bf22b4ea72df94eb09 Mon Sep 17 00:00:00 2001 From: Boris Kolpackov Date: Mon, 21 May 2018 14:52:15 +0200 Subject: Implement config-set subcommand --- bdep/config.cli | 20 +++++--- bdep/config.cxx | 156 ++++++++++++++++++++++++++++++++++++++++---------------- 2 files changed, 125 insertions(+), 51 deletions(-) diff --git a/bdep/config.cli b/bdep/config.cli index 8c355db..cf87020 100644 --- a/bdep/config.cli +++ b/bdep/config.cli @@ -23,7 +23,7 @@ namespace bdep \b{bdep config list} \ \ [] [] [...]\n \b{bdep config remove} [] [] ... | \b{--all}|\b{-a}\n \b{bdep config rename} [] [] \n - \b{bdep config set} \ \ \ [] [] \n + \b{bdep config set} \ \ \ [] [] ... | \b{--all}|\b{-a}\n \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ [\b{--}[\b{no-}]\b{default}]\n \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ [\b{--}[\b{no-}]\b{forward}]\n \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ [\b{--}[\b{no-}]\b{auto-sync}]} @@ -114,13 +114,17 @@ namespace bdep \li|\cb{set} - The \cb{set} subcommand sets various properties of the specified - build configuration. These include the - default (\c{\b{--}[\b{no-}]\b{default}}), - forward (\c{\b{--}[\b{no-}]\b{forward}}), and - auto-synchronization (\c{\b{--}[\b{no-}]\b{auto-sync}}) flags. - Note that changing any of these flags requires an explicit - \l{bdep-sync(1)} command to take effect.||" + The \cb{set} subcommand modifies various properties of one or more + build configurations associated with the project. See + \l{bdep-projects-configs(1)} for various ways to specify build + configurations. + + The properties that can be modified include the default + (\c{\b{--}[\b{no-}]\b{default}}), forward + (\c{\b{--}[\b{no-}]\b{forward}}), and auto-synchronization + (\c{\b{--}[\b{no-}]\b{auto-sync}}) flags. Note that changing any of + these flags requires an explicit \l{bdep-sync(1)} command to take + effect. ||" bool add; bool create; diff --git a/bdep/config.cxx b/bdep/config.cxx index 84aa77d..5a3268d 100644 --- a/bdep/config.cxx +++ b/bdep/config.cxx @@ -15,7 +15,7 @@ using namespace std; namespace bdep { template - void + static void print_configuration (O& o, const shared_ptr& c) { char s (' '); @@ -27,6 +27,59 @@ namespace bdep if (c->auto_sync) {o << s << "auto-synchronized"; s = ',';} } + const char* + cmd_config_validate_add (const configuration_add_options& o) + { + // --[no-]default + // + if (o.default_ () && o.no_default ()) + fail << "both --default and --no-default specified"; + + // --[no-]forward + // + if (o.forward () && o.no_forward ()) + fail << "both --forward and --no-forward specified"; + + // --[no-]auto-sync + // + if (o.auto_sync () && o.no_auto_sync ()) + fail << "both --auto-sync and --no-auto-sync specified"; + + return (o.default_ () ? "--default" : + o.no_default () ? "--no-default" : + o.forward () ? "--forward" : + o.no_forward () ? "--no-forward" : + o.auto_sync () ? "--auto-sync" : + o.no_auto_sync () ? "--no-auto-sync" : + o.wipe () ? "--wipe" : nullptr); + } + + void + cmd_config_validate_add (const configuration_name_options& o, + const char* what, + optional& name, + optional id) + { + name = nullopt; + id = nullopt; + + if (size_t n = o.config_name ().size ()) + { + if (n > 1) + fail << "multiple configuration names specified for " << what; + + name = o.config_name ()[0]; + } + + if (size_t n = o.config_id ().size ()) + { + if (n > 1) + fail << "multiple configuration ids specified for " << what; + + id = o.config_id ()[0]; + } + } + // Translate the configuration directory that is actually a name (@foo or // -@foo) to the real directory (prj-foo) and name (@foo). // @@ -390,62 +443,79 @@ namespace bdep } static int - cmd_config_set (const cmd_config_options&, cli::scanner&) + cmd_config_set (const cmd_config_options& o, cli::scanner&) { - fail << "@@ TODO" << endf; - } + tracer trace ("config_set"); - const char* - cmd_config_validate_add (const configuration_add_options& o) - { - // --[no-]default + // Note that these have been validate by cmd_config_validate_add(). // - if (o.default_ () && o.no_default ()) - fail << "both --default and --no-default specified"; + optional d, f, s; + if (o.default_ () || o.no_default ()) d = o.default_ (); + if (o.forward () || o.no_forward ()) f = o.forward (); + if (o.auto_sync () || o.no_auto_sync ()) s = o.auto_sync (); - // --[no-]forward - // - if (o.forward () && o.no_forward ()) - fail << "both --forward and --no-forward specified"; + if (!d && !f && !s) + fail << "nothing to set"; - // --[no-]auto-sync - // - if (o.auto_sync () && o.no_auto_sync ()) - fail << "both --auto-sync and --no-auto-sync specified"; + dir_path prj (find_project (o)); + database db (open (prj, trace)); - return (o.default_ () ? "--default" : - o.no_default () ? "--no-default" : - o.forward () ? "--forward" : - o.no_forward () ? "--no-forward" : - o.auto_sync () ? "--auto-sync" : - o.no_auto_sync () ? "--no-auto-sync" : - o.wipe () ? "--wipe" : nullptr); - } + transaction t (db.begin ()); - void - cmd_config_validate_add (const configuration_name_options& o, - const char* what, - optional& name, - optional id) - { - name = nullopt; - id = nullopt; + configurations cfgs ( + find_configurations (o, + prj, + t, + false /* fallback_default */, + false /* validate */)); - if (size_t n = o.config_name ().size ()) + for (const shared_ptr& c: cfgs) { - if (n > 1) - fail << "multiple configuration names specified for " << what; + using query = bdep::query; - name = o.config_name ()[0]; + if (d) + { + if (*d && !c->default_) + { + if (auto p = db.query_one (query::default_)) + fail << "configuration " << *p << " is already the default" << + info << "while updating configuration " << *c; + } + + c->default_ = *d; + } + + if (f) + { + if (*f && !c->forward) + { + if (auto p = db.query_one (query::forward)) + fail << "configuration " << *p << " is already forwarded" << + info << "while updating configuration " << *c; + } + + c->forward = *f; + } + + if (s) + c->auto_sync = *s; + + db.update (c); } - if (size_t n = o.config_id ().size ()) - { - if (n > 1) - fail << "multiple configuration ids specified for " << what; + t.commit (); - id = o.config_id ()[0]; + if (verb) + { + for (const shared_ptr& c: cfgs) + { + diag_record dr (text); + dr << "updated configuration "; + print_configuration (dr, c); + } } + + return 0; } int -- cgit v1.1