From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) by finch.gentoo.org (Postfix) with ESMTP id 75BE513888F for ; Tue, 6 Oct 2015 06:19:23 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id EF199E07E2; Tue, 6 Oct 2015 06:19:22 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 86B56E07E2 for ; Tue, 6 Oct 2015 06:19:22 +0000 (UTC) Received: from professor-x (S010634bdfa9ecf80.vc.shawcable.net [96.49.31.57]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: dolsen) by smtp.gentoo.org (Postfix) with ESMTPSA id 7E0EF340988 for ; Tue, 6 Oct 2015 06:19:20 +0000 (UTC) Date: Mon, 5 Oct 2015 23:18:31 -0700 From: Brian Dolbec To: gentoo-catalyst@lists.gentoo.org Subject: Re: [gentoo-catalyst] [PATCH] lint: drop unnecessary semicolon Message-ID: <20151005231831.4c9f0ebe.dolsen@gentoo.org> In-Reply-To: <1444101551-13348-1-git-send-email-vapier@gentoo.org> References: <1444101551-13348-1-git-send-email-vapier@gentoo.org> Organization: Gentoo Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-catalyst@lists.gentoo.org Reply-to: gentoo-catalyst@lists.gentoo.org MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Archives-Salt: ed6f361f-31d7-40d9-8aef-6a080d47e231 X-Archives-Hash: 7b302897edce65b1534eeb3778c5751d On Mon, 5 Oct 2015 23:19:11 -0400 Mike Frysinger wrote: > --- > catalyst/arch/powerpc.py | 2 +- > catalyst/arch/sparc.py | 2 +- > catalyst/arch/x86.py | 2 +- > targets/stage1/build.py | 6 +++--- > 4 files changed, 6 insertions(+), 6 deletions(-) > > diff --git a/catalyst/arch/powerpc.py b/catalyst/arch/powerpc.py > index 8e0f540..fb98316 100644 > --- a/catalyst/arch/powerpc.py > +++ b/catalyst/arch/powerpc.py > @@ -14,7 +14,7 @@ class generic_ppc(builder.generic): > raise CatalystError("required > executable linux32 not found " "(\"emerge setarch\" to fix.)", > print_traceback=True) self.settings["CHROOT"]="linux32 chroot" > - self.settings["crosscompile"] = False; > + self.settings["crosscompile"] = False > else: > self.settings["CHROOT"]="chroot" > > diff --git a/catalyst/arch/sparc.py b/catalyst/arch/sparc.py > index da2ad35..ec69386 100644 > --- a/catalyst/arch/sparc.py > +++ b/catalyst/arch/sparc.py > @@ -13,7 +13,7 @@ class generic_sparc(builder.generic): > raise CatalystError("required > executable linux32 not found " "(\"emerge setarch\" to fix.)", > print_traceback=True) self.settings["CHROOT"]="linux32 chroot" > - self.settings["crosscompile"] = False; > + self.settings["crosscompile"] = False > else: > self.settings["CHROOT"]="chroot" > > diff --git a/catalyst/arch/x86.py b/catalyst/arch/x86.py > index 44a6a49..49500b9 100644 > --- a/catalyst/arch/x86.py > +++ b/catalyst/arch/x86.py > @@ -13,7 +13,7 @@ class generic_x86(builder.generic): > raise > CatalystError("required executable linux32 not found " "(\"emerge > setarch\" to fix.)", print_traceback=True) > self.settings["CHROOT"]="linux32 chroot" > - self.settings["crosscompile"] = False; > + self.settings["crosscompile"] = False > else: > self.settings["CHROOT"]="chroot" > > diff --git a/targets/stage1/build.py b/targets/stage1/build.py > index b2c8b0f..db6a93f 100755 > --- a/targets/stage1/build.py > +++ b/targets/stage1/build.py > @@ -10,12 +10,12 @@ import portage > # last case is for portage-2.1_pre* > def scan_profile(file): > if "grab_stacked" in dir(portage): > - return portage.grab_stacked(file, > portage.settings.profiles, portage.grabfile, incremental_lines=1); > + return portage.grab_stacked(file, > portage.settings.profiles, portage.grabfile, incremental_lines=1) > else: if "grab_multiple" in dir(portage): > - return > portage.stack_lists( portage.grab_multiple(file, > portage.settings.profiles, portage.grabfile), incremental=1); > + return > portage.stack_lists( portage.grab_multiple(file, > portage.settings.profiles, portage.grabfile), incremental=1) else: > - return > portage.stack_lists( [portage.grabfile_package(os.path.join(x, file)) > for x in portage.settings.profiles], incremental=1); > + return > portage.stack_lists( [portage.grabfile_package(os.path.join(x, file)) > for x in portage.settings.profiles], incremental=1) # loaded the > stacked packages / packages.build files pkgs = > scan_profile("packages") looks good :) -- Brian Dolbec