On 07/07/18 10:32, m...@dk0.us wrote:
> From: Dmitry Kudriavtsev <m...@dk0.us>
> 
> Adds a --nolist option for package transactions. This option removes the list
> display of packages to be installed or removed.
> 
> Signed-off-by: Dmitry Kudriavtsev <m...@dk0.us>
> ---

I don't think this is a good option to include.

> diff --git a/src/pacman/remove.c b/src/pacman/remove.c
> index a2269ed8..5df45c69 100644
> --- a/src/pacman/remove.c
> +++ b/src/pacman/remove.c
> @@ -164,8 +164,11 @@ int pacman_remove(alpm_list_t *targets)
>       }
>  
>       /* print targets and ask user confirmation */
> -     display_targets();
> -     printf("\n");
> +     if(!config->nolist) {
> +             display_targets();
> +             printf("\n");
> +     }
> +

Here pacman would print nothing then ask if it is OK to remove those
packages.

>       if(yesno(_("Do you want to remove these packages?")) == 0) {



>               retval = 1;
>               goto cleanup;
> diff --git a/src/pacman/sync.c b/src/pacman/sync.c
> index 3c6be89d..1e0c6c91 100644
> --- a/src/pacman/sync.c
> +++ b/src/pacman/sync.c
> @@ -793,8 +793,10 @@ int sync_prepare_execute(void)
>               goto cleanup;
>       }
>  
> -     display_targets();
> -     printf("\n");
> +     if(!config->nolist) {
> +             display_targets();
> +             printf("\n");
> +     }
>  
>       int confirm;
>       if(config->op_s_downloadonly) {
> 

Reply via email to