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 1AFE9139737 for ; Tue, 11 Aug 2015 23:54:19 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 305161419F; Tue, 11 Aug 2015 23:54:13 +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 3F53514186 for ; Tue, 11 Aug 2015 23:54:12 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 5FE5934092E for ; Tue, 11 Aug 2015 23:54:11 +0000 (UTC) Received: from localhost.localdomain (localhost [127.0.0.1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id 6124F152 for ; Tue, 11 Aug 2015 23:54:08 +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: <1439337172.ef1361570982a70b80b84c71526da7fc802e2465.dolsen@gentoo> Subject: [gentoo-commits] proj/portage:repoman commit in: pym/repoman/ X-VCS-Repository: proj/portage X-VCS-Files: pym/repoman/main.py pym/repoman/utilities.py X-VCS-Directories: pym/repoman/ X-VCS-Committer: dolsen X-VCS-Committer-Name: Brian Dolbec X-VCS-Revision: ef1361570982a70b80b84c71526da7fc802e2465 X-VCS-Branch: repoman Date: Tue, 11 Aug 2015 23:54:08 +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: d74d1165-12b9-4dc3-9db3-dc0c2d413b16 X-Archives-Hash: aa947a35fba7fb5b76f7c043f7b1f668 commit: ef1361570982a70b80b84c71526da7fc802e2465 Author: Tom Wijsman gentoo org> AuthorDate: Tue Jun 3 11:28:44 2014 +0000 Commit: Brian Dolbec gentoo org> CommitDate: Tue Aug 11 23:52:52 2015 +0000 URL: https://gitweb.gentoo.org/proj/portage.git/commit/?id=ef136157 repoman/main.py: Refactor "RepoMan Sez" pym/repoman/main.py | 27 ++++++++++----------------- pym/repoman/utilities.py | 8 ++++++-- 2 files changed, 16 insertions(+), 19 deletions(-) diff --git a/pym/repoman/main.py b/pym/repoman/main.py index 95af4d4..c183aea 100755 --- a/pym/repoman/main.py +++ b/pym/repoman/main.py @@ -981,25 +981,21 @@ if options.mode != 'commit': if dofull: print(bold("Note: type \"repoman full\" for a complete listing.")) if dowarn and not dofail: - print( - green("RepoMan sez:"), + utilities.repoman_sez( "\"You're only giving me a partial QA payment?\n" " I'll take it this time, but I'm not happy.\"") elif not dofail: - print( - green("RepoMan sez:"), + utilities.repoman_sez( "\"If everyone were like you, I'd be out of business!\"") elif dofail: print(bad("Please fix these important QA issues first.")) - print( - green("RepoMan sez:"), + utilities.repoman_sez( "\"Make your QA payment on time" " and you'll never see the likes of me.\"\n") sys.exit(1) else: if dofail and can_force and options.force and not options.pretend: - print( - green("RepoMan sez:"), + utilities.repoman_sez( " \"You want to commit even with these QA issues?\n" " I'll take it this time, but I'm not happy.\"\n") elif dofail: @@ -1008,15 +1004,13 @@ else: "The --force option has been disabled" " due to extraordinary issues.")) print(bad("Please fix these important QA issues first.")) - print( - green("RepoMan sez:"), + utilities.repoman_sez( "\"Make your QA payment on time" " and you'll never see the likes of me.\"\n") sys.exit(1) if options.pretend: - print( - green("RepoMan sez:"), + utilities.repoman_sez( "\"So, you want to play it safe. Good call.\"\n") myunadded = [] @@ -1195,7 +1189,8 @@ else: if vcs_settings.vcs: if not (mychanged or mynew or myremoved or (vcs_settings.vcs == "hg" and mydeleted)): - print(green("RepoMan sez:"), "\"Doing nothing is not always good for QA.\"") + utilities.repoman_sez( + "\"Doing nothing is not always good for QA.\"") print() print("(Didn't find any changed files...)") print() @@ -1600,8 +1595,7 @@ else: portage.util.write_atomic(x, b''.join(mylines), mode='wb') if repolevel == 1: - print( - green("RepoMan sez:"), + utilities.repoman_sez( "\"You're rather crazy... " "doing the entire repository.\"\n") @@ -1718,7 +1712,6 @@ else: "repoman was too scared" " by not seeing any familiar version control file" " that he forgot to commit anything") - print( - green("RepoMan sez:"), + utilities.repoman_sez( "\"If everyone were like you, I'd be out of business!\"\n") sys.exit(0) diff --git a/pym/repoman/utilities.py b/pym/repoman/utilities.py index 93e8840..2a1f4d9 100644 --- a/pym/repoman/utilities.py +++ b/pym/repoman/utilities.py @@ -39,9 +39,10 @@ from portage import shutil from portage import _encodings from portage import _unicode_decode from portage import _unicode_encode +from portage import util from portage.localization import _ from portage.process import find_binary -from portage import util +from portage.output import green from repoman.copyrights import update_copyright, update_copyright_year @@ -126,7 +127,6 @@ def FindPackagesToScan(settings, startdir, reposplit): path = os.path.join(startdir, cat) if not os.path.isdir(path): continue - pkgdirs = os.listdir(path) scanlist.extend(AddPackagesInDir(path)) elif repolevel == 2: # category level, startdir = catdir # We only want 1 segment of the directory, @@ -589,3 +589,7 @@ def UpdateChangeLog( except OSError: pass return None + + +def repoman_sez(msg): + print (green("RepoMan sez:"), msg)