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 0401213888F for ; Thu, 8 Oct 2015 20:02:39 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id C666DE07DF; Thu, 8 Oct 2015 20:02:37 +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 5D772E07DF for ; Thu, 8 Oct 2015 20:02:37 +0000 (UTC) Received: from oystercatcher.gentoo.org (unknown [IPv6:2a01:4f8:202:4333:225:90ff:fed9:fc84]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 3F00234072E for ; Thu, 8 Oct 2015 20:02:36 +0000 (UTC) Received: from localhost.localdomain (localhost [127.0.0.1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id C9D1AC60 for ; Thu, 8 Oct 2015 20:02:33 +0000 (UTC) From: "Brian Dolbec" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Brian Dolbec" Message-ID: <1444332847.080eab6b951ada8e6b982a71a4ff701241d7caee.dolsen@gentoo> Subject: [gentoo-commits] proj/catalyst:master commit in: catalyst/base/, catalyst/ X-VCS-Repository: proj/catalyst X-VCS-Files: catalyst/base/genbase.py catalyst/lock.py X-VCS-Directories: catalyst/ catalyst/base/ X-VCS-Committer: dolsen X-VCS-Committer-Name: Brian Dolbec X-VCS-Revision: 080eab6b951ada8e6b982a71a4ff701241d7caee X-VCS-Branch: master Date: Thu, 8 Oct 2015 20:02:33 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Archives-Salt: 76abb55b-6d22-4dd8-970b-d68c3784707c X-Archives-Hash: 7ab8979c8c9625f053d8808794d68280 commit: 080eab6b951ada8e6b982a71a4ff701241d7caee Author: Brian Dolbec gentoo org> AuthorDate: Thu Oct 8 19:33:17 2015 +0000 Commit: Brian Dolbec gentoo org> CommitDate: Thu Oct 8 19:34:07 2015 +0000 URL: https://gitweb.gentoo.org/proj/catalyst.git/commit/?id=080eab6b Fix argparse conversion breakage for verbose and debug settings Argparse always defines the target variable. Convert the use of them to normal assignment rather than a member inclusion test. catalyst/base/genbase.py | 6 +++--- catalyst/lock.py | 12 ++++++------ 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/catalyst/base/genbase.py b/catalyst/base/genbase.py index a163638..32459b4 100644 --- a/catalyst/base/genbase.py +++ b/catalyst/base/genbase.py @@ -26,7 +26,7 @@ class GenBase(object): array.sort() for j in array: contents = contents_map.contents(path, j, - verbose="VERBOSE" in self.settings) + verbose=self.settings["VERBOSE"]) if contents: myf.write(contents) myf.close() @@ -49,12 +49,12 @@ class GenBase(object): if "all" in array: for k in list(hash_map.hash_map): digest = hash_map.generate_hash(f,hash_=k, - verbose = "VERBOSE" in self.settings) + verbose=self.settings["VERBOSE"]) myf.write(digest) else: for j in array: digest = hash_map.generate_hash(f,hash_=j, - verbose = "VERBOSE" in self.settings) + verbose=self.settings["VERBOSE"]) myf.write(digest) myf.close() diff --git a/catalyst/lock.py b/catalyst/lock.py index 3d50c06..8095a82 100644 --- a/catalyst/lock.py +++ b/catalyst/lock.py @@ -71,7 +71,7 @@ class LockDir(object): def set_gid(self,gid): if not self.islocked(): -# if "DEBUG" in self.settings: +# if self.settings["DEBUG"]: # print "setting gid to", gid self.gid=gid @@ -83,7 +83,7 @@ class LockDir(object): if lockdir[-1] == "/": lockdir=lockdir[:-1] self.lockdir=normpath(lockdir) -# if "DEBUG" in self.settings: +# if self.settings["DEBUG"]: # print "setting lockdir to", self.lockdir else: raise "the lock object needs a path to a dir" @@ -91,13 +91,13 @@ class LockDir(object): def set_lockfilename(self,lockfilename): if not self.islocked(): self.lockfilename=lockfilename -# if "DEBUG" in self.settings: +# if self.settings["DEBUG"]: # print "setting lockfilename to", self.lockfilename def set_lockfile(self): if not self.islocked(): self.lockfile=normpath(self.lockdir+'/'+self.lockfilename) -# if "DEBUG" in self.settings: +# if self.settings["DEBUG"]: # print "setting lockfile to", self.lockfile def read_lock(self): @@ -222,7 +222,7 @@ class LockDir(object): os.unlink(self.lockfile) os.close(self.myfd) self.myfd=None -# if "DEBUG" in self.settings: +# if self.settings["DEBUG"]: # print "Unlinked lockfile..." except Exception, e: # We really don't care... Someone else has the lock. @@ -269,7 +269,7 @@ class LockDir(object): try: os.link(self.myhardlock, self.lockfile) except Exception: -# if "DEBUG" in self.settings: +# if self.settings["DEBUG"]: # print "lockfile(): Hardlink: Link failed." # print "Exception: ",e pass