diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2021-08-26 11:40:35 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2021-08-26 11:40:35 +0200 |
commit | 5b84b34eb1c4c49607a0d14706b15a6923f1ab0c (patch) | |
tree | a17fc8d975695fee597d71053ad4cb230a54be33 /bdep/status.cxx | |
parent | 17341bc0fdd23617e3f2e81b0228ff9ecc5cb194 (diff) |
Print diagnostics about empty configurations after "in configuration ...:"
In particular, this makes sure they are separated with blank lines.
Diffstat (limited to 'bdep/status.cxx')
-rw-r--r-- | bdep/status.cxx | 36 |
1 files changed, 17 insertions, 19 deletions
diff --git a/bdep/status.cxx b/bdep/status.cxx index fc07698..b64a9a4 100644 --- a/bdep/status.cxx +++ b/bdep/status.cxx @@ -96,15 +96,6 @@ namespace bdep bool first (true); for (const shared_ptr<configuration>& c: cfgs) { - if (c->packages.empty ()) - { - if (verb) - info << "skipping configuration " << *c << - info << "configuration is empty"; - - continue; - } - // Collect the packages to print, unless the dependency packages are // specified. // @@ -131,16 +122,6 @@ namespace bdep }) != ps.end ()) pkgs.push_back (s.name.string ()); } - - if (pkgs.empty ()) - { - if (verb) - info << "skipping configuration " << *c << - info << "none of specified packages initialized in this " - << "configuration"; - - continue; - } } // If we are printing multiple configurations, separate them with a @@ -154,6 +135,23 @@ namespace bdep first = false; } + if (c->packages.empty () || (pkgs.empty () && dep_pkgs.empty ())) + { + if (verb) + { + diag_record dr (info); + + if (c->packages.empty ()) + dr << "no packages "; + else + dr << "none of specified packages "; + + dr << "initialized in configuration " << *c << ", skipping"; + } + + continue; + } + bool fetch (o.fetch () || o.fetch_full ()); if (fetch) |