[gentoo-commits] proj/portage:master commit in: pym/repoman/

["Brian Dolbec" <dolsen@gentoo.org>]

commit:     9f978b5c60fe40034ddc4d14abb69238fd60e3c5
Author:     Brian Dolbec <dolsen <AT> gentoo <DOT> org>
AuthorDate: Fri Sep 25 17:52:36 2015 +0000
Commit:     Brian Dolbec <dolsen <AT> gentoo <DOT> org>
CommitDate: Sat Sep 26 20:51:32 2015 +0000
URL:        https://gitweb.gentoo.org/proj/portage.git/commit/?id=9f978b5c

repoman/repos.py: Add check that we need commit signing (bug 561474)

Side effect of the code splitting, repoman full in a non -vcs repo directory was reporting the inability to sign warning.

X-Gentoo-bug: 561474
X-Gentoo-bug-url: https://bugs.gentoo.org/show_bug.cgi?id=561474
Title: repoman: spews useless warning about commit while running 'full'

 pym/repoman/repos.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pym/repoman/repos.py b/pym/repoman/repos.py
index f16bf7a..72e5735 100644
--- a/pym/repoman/repos.py
+++ b/pym/repoman/repos.py
@@ -60,7 +60,7 @@ class RepoSettings(object):
 		if self.repo_config.allow_provide_virtual:
 			qawarnings.add("virtual.oldstyle")
 
-		if self.repo_config.sign_commit:
+		if self.repo_config.sign_commit and options.mode in ("commit", "fix", "manifest"):
 			if vcs_settings.vcs:
 				func = getattr(self, '_vcs_gpg_%s' % vcs_settings.vcs)
 				func()