summaryrefslogtreecommitdiffstats
path: root/src/main.rs
diff options
context:
space:
mode:
authorMatthias Beyer <mail@beyermatthias.de>2020-11-07 16:04:45 +0100
committerMatthias Beyer <mail@beyermatthias.de>2020-11-07 17:34:00 +0100
commite4512d60e989f7d31febabb3934fd028af48522b (patch)
tree977a91820c85b09cf058711e14d8aec900ac539a /src/main.rs
parenta3c4b3900b6aff923e9c1f0fafbf6cb1069def83 (diff)
Degrade ProgressBars to a simple progress-bar-generation helper
This patch removes the idea of having a "MultiProgress" object from indicativ available in the top level of butido. It turned out that the progress bars were not updated until the MultiProgress::join() method was called, which basically yields the progress bars useless. Thus, we remove the multi-progress-bar idea from top level. Later, we might implement a multi-progress bar viewing in the orchestrator, so we can see which job is progressed how far. Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Diffstat (limited to 'src/main.rs')
-rw-r--r--src/main.rs4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/main.rs b/src/main.rs
index fbb7dea..a16208e 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -63,7 +63,7 @@ async fn main() -> Result<()> {
let repo_path = PathBuf::from(config.repository());
let _ = crate::ui::package_repo_cleanness_check(&repo_path)?;
let max_packages = count_pkg_files(&repo_path);
- let mut progressbars = ProgressBars::setup(config.progress_format().clone());
+ let progressbars = ProgressBars::setup(config.progress_format().clone());
let mut load_repo = || -> Result<Repository> {
let bar = progressbars.repo_loading();
@@ -108,7 +108,7 @@ async fn main() -> Result<()> {
(other, _) => return Err(anyhow!("Unknown subcommand: {}", other)),
}
- progressbars.into_inner().join().map_err(Error::from)
+ Ok(())
}
async fn build<'a>(matches: &ArgMatches,