summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormorganamilo <morganamilo@archlinux.org>2021-04-18 21:59:43 +0200
committerAllan McRae <allan@archlinux.org>2021-04-23 01:43:30 +0200
commit41f9c50abf08e71aa5fd03568934ab80abc7715c (patch)
treeb005d4328b3c38d87b049d13153105cc26339145
parent63ad8b8b7639ac29fad6c4e0dcc699510c47d587 (diff)
downloadpacman-41f9c50abf08e71aa5fd03568934ab80abc7715c.tar.gz
pacman-41f9c50abf08e71aa5fd03568934ab80abc7715c.tar.xz
Always enable TotalDownload
Previously TotalDownload would switch the % download from per package to overall. Meaning you had a choice of which information to dispplay. Now with parallel downloads TotalDownload adds an extra progress bar. There's no reason to have this an off by default feature. Let's just make it always on. Signed-off-by: Allan McRae <allan@archlinux.org>
-rw-r--r--README4
-rw-r--r--doc/pacman.conf.5.asciidoc5
-rw-r--r--etc/pacman.conf.in1
-rw-r--r--scripts/completion/zsh_completion.in1
-rw-r--r--src/pacman/callback.c3
-rw-r--r--src/pacman/conf.c3
-rw-r--r--src/pacman/conf.h3
-rw-r--r--src/pacman/pacman-conf.c3
8 files changed, 4 insertions, 19 deletions
diff --git a/README b/README
index 470ccf3c..4465432a 100644
--- a/README
+++ b/README
@@ -194,8 +194,8 @@ remove.c and sync.c).
The frontend is using a configuration file, usually "/etc/pacman.conf". Some
of these options are only useful for the frontend only (mainly the ones used to
-control the output like totaldownload, or the behavior with cleanmethod and
-syncfirst). The rest is used to configure the library.
+control the output like verbosepkglist, or the behavior with cleanmethod).
+The rest is used to configure the library.
[UPGRADE/REMOVE/SYNC]
diff --git a/doc/pacman.conf.5.asciidoc b/doc/pacman.conf.5.asciidoc
index 11ca1e4f..fd765f3d 100644
--- a/doc/pacman.conf.5.asciidoc
+++ b/doc/pacman.conf.5.asciidoc
@@ -190,11 +190,6 @@ Options
Disables progress bars. This is useful for terminals which do
not support escape characters.
-*TotalDownload*::
- When downloading, display an extra progress bar with the amount downloaded,
- download rate, ETA, and completed percentage of the entire download list.
- This option won't work if XferCommand is used.
-
*CheckSpace*::
Performs an approximate check for adequate available disk space before
installing packages.
diff --git a/etc/pacman.conf.in b/etc/pacman.conf.in
index 634ecc06..1799efc7 100644
--- a/etc/pacman.conf.in
+++ b/etc/pacman.conf.in
@@ -32,7 +32,6 @@ Architecture = auto
#UseSyslog
#Color
#NoProgressBar
-#TotalDownload
CheckSpace
#VerbosePkgLists
ParallelDownloads = 5
diff --git a/scripts/completion/zsh_completion.in b/scripts/completion/zsh_completion.in
index e4bf3312..5fd8aebc 100644
--- a/scripts/completion/zsh_completion.in
+++ b/scripts/completion/zsh_completion.in
@@ -511,7 +511,6 @@ _pacman_conf_general_directives=(
'XferCommand'
'UseSyslog'
'Color'
- 'TotalDownload'
'CheckSpace'
'VerbosePkgLists'
'DisableDownloadTimeout'
diff --git a/src/pacman/callback.c b/src/pacman/callback.c
index 3f32360c..98887d9f 100644
--- a/src/pacman/callback.c
+++ b/src/pacman/callback.c
@@ -340,7 +340,8 @@ void cb_event(alpm_event_t *event)
on_progress = 1;
list_total_pkgs = event->pkg_retrieve.num;
list_total = event->pkg_retrieve.total_size;
- total_enabled = config->totaldownload && list_total && dload_progressbar_enabled();
+ total_enabled = list_total && dload_progressbar_enabled();
+
if(total_enabled) {
init_total_progressbar();
}
diff --git a/src/pacman/conf.c b/src/pacman/conf.c
index 9578daa3..0f0c2cfb 100644
--- a/src/pacman/conf.c
+++ b/src/pacman/conf.c
@@ -605,9 +605,6 @@ static int _parse_options(const char *key, char *value,
} else if(strcmp(key, "VerbosePkgLists") == 0) {
config->verbosepkglists = 1;
pm_printf(ALPM_LOG_DEBUG, "config: verbosepkglists\n");
- } else if(strcmp(key, "TotalDownload") == 0) {
- config->totaldownload = 1;
- pm_printf(ALPM_LOG_DEBUG, "config: totaldownload\n");
} else if(strcmp(key, "CheckSpace") == 0) {
config->checkspace = 1;
} else if(strcmp(key, "Color") == 0) {
diff --git a/src/pacman/conf.h b/src/pacman/conf.h
index 316c8d0f..04350d39 100644
--- a/src/pacman/conf.h
+++ b/src/pacman/conf.h
@@ -113,9 +113,6 @@ typedef struct __config_t {
unsigned short chomp;
/* format target pkg lists as table */
unsigned short verbosepkglists;
- /* When downloading, display the amount downloaded, rate, ETA, and percent
- * downloaded of the total download list */
- unsigned short totaldownload;
/* number of parallel download streams */
unsigned int parallel_downloads;
/* select -Sc behavior */
diff --git a/src/pacman/pacman-conf.c b/src/pacman/pacman-conf.c
index 6ca21119..600f1622 100644
--- a/src/pacman/pacman-conf.c
+++ b/src/pacman/pacman-conf.c
@@ -263,7 +263,6 @@ static void dump_config(void)
show_bool("UseSyslog", config->usesyslog);
show_bool("Color", config->color);
- show_bool("TotalDownload", config->totaldownload);
show_bool("CheckSpace", config->checkspace);
show_bool("VerbosePkgLists", config->verbosepkglists);
show_bool("DisableDownloadTimeout", config->disable_dl_timeout);
@@ -372,8 +371,6 @@ static int list_directives(void)
show_bool("UseSyslog", config->usesyslog);
} else if(strcasecmp(i->data, "Color") == 0) {
show_bool("Color", config->color);
- } else if(strcasecmp(i->data, "TotalDownload") == 0) {
- show_bool("TotalDownload", config->totaldownload);
} else if(strcasecmp(i->data, "CheckSpace") == 0) {
show_bool("CheckSpace", config->checkspace);
} else if(strcasecmp(i->data, "VerbosePkgLists") == 0) {