diff --git a/britney2/policies/sourceppa.py b/britney2/policies/sourceppa.py index 55a0e2d..88bb385 100644 --- a/britney2/policies/sourceppa.py +++ b/britney2/policies/sourceppa.py @@ -93,6 +93,7 @@ class SourcePPAPolicy(BasePolicy, Rest): for friend in self.pkgs_by_source_ppa[sourceppa]: sourceppa_info[friend] = shortppa if not britney_excuses[friend].is_valid: + self.log ("sourceppa: processing %s, found invalid grouped package %s, will invalidate set" % (source_name, britney_excuses[friend].name)) accept = False self.pkgs_by_source_ppa[sourceppa].add(source_name) @@ -104,7 +105,7 @@ class SourcePPAPolicy(BasePolicy, Rest): friend_exc.is_valid = False friend_exc.addreason('source-ppa') friend_exc.policy_info['source-ppa'] = sourceppa_info - self.log("Grouping %s with PPA %s" % (friend, shortppa)) + self.log ("sourceppa: ... invalidating %s due to the above (ppa: %s)" % (friend_exc.name, shortppa)) friend_exc.addhtml("Grouped with PPA %s" % shortppa) return PolicyVerdict.REJECTED_PERMANENTLY return PolicyVerdict.PASS diff --git a/tests/test_sourceppa.py b/tests/test_sourceppa.py index 180c024..0b39fff 100755 --- a/tests/test_sourceppa.py +++ b/tests/test_sourceppa.py @@ -32,6 +32,7 @@ class FakeExcuse: ver = ('1.0', '2.0') is_valid = True policy_info = {} + name = "foo" def addreason(self, reason): """Ignore reasons."""