From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH] pakfire: Fix small bug: wrong pak count on list
Date: Fri, 16 Jul 2021 11:50:43 +0100
Message-ID: <07C4E405-0F31-434D-8D36-F805C6644553@ipfire.org>
In-Reply-To: <20210624220700.14161-1-robin.roevens@disroot.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4502105456300217738=="
List-Id: <development.lists.ipfire.org>

--===============4502105456300217738==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Thanks!

Reviewed-by: Michael Tremer <michael.tremer(a)ipfire.org>

> On 24 Jun 2021, at 23:07, Robin Roevens <robin.roevens(a)disroot.org> wrote:
> 
> Fixes pakfire list installed/notinstalled command displaying the
> count of all available paks instead of the count of actual installed
> respectively not installed paks.
> 
> Signed-off-by: Robin Roevens <robin.roevens(a)disroot.org>
> ---
> src/pakfire/lib/functions.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/pakfire/lib/functions.pl b/src/pakfire/lib/functions.pl
> index f9a19b60d..c15e54f94 100644
> --- a/src/pakfire/lib/functions.pl
> +++ b/src/pakfire/lib/functions.pl
> @@ -512,13 +512,13 @@ sub dblist {
> 		foreach $line (sort @db) {
> 			next unless ($line =~ /.*;.*;.*;/ );
> 			$use_color = "";
> -			$count++;
> 			@templine = split(/\;/,$line);
> 			if ("$filter" eq "notinstalled") {
> 				next if ( -e "$Conf::dbdir/installed/meta-$templine[0]" );
> 			} elsif ("$filter" eq "installed") {
> 				next unless ( -e "$Conf::dbdir/installed/meta-$templine[0]" );
> 			}
> +			$count++;
> 			if ("$forweb" eq "forweb")
> 			 {
> 				if ("$filter" eq "notinstalled") {
> -- 
> 2.31.1
> 
> 
> -- 
> Dit bericht is gescanned op virussen en andere gevaarlijke
> inhoud door MailScanner en lijkt schoon te zijn.
> 


--===============4502105456300217738==--