diff options
author | Matthias Beyer <mail@beyermatthias.de> | 2020-11-07 16:04:45 +0100 |
---|---|---|
committer | Matthias Beyer <mail@beyermatthias.de> | 2020-11-07 17:34:00 +0100 |
commit | e4512d60e989f7d31febabb3934fd028af48522b (patch) | |
tree | 977a91820c85b09cf058711e14d8aec900ac539a /src/main.rs | |
parent | a3c4b3900b6aff923e9c1f0fafbf6cb1069def83 (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.rs | 4 |
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, |