[gentoo-portage-dev] [PATCH 2/3] portage/sync/syncbase.py: Change has_bin to an @property function

[Brian Dolbec]

This avoids that self.logger is None error in the __init__().
---
 pym/portage/sync/syncbase.py | 24 +++++++++++++++---------
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/pym/portage/sync/syncbase.py b/pym/portage/sync/syncbase.py
index c820bcf..973d545 100644
--- a/pym/portage/sync/syncbase.py
+++ b/pym/portage/sync/syncbase.py
@@ -35,17 +35,23 @@ class SyncBase(object):
 		self.xterm_titles = None
 		self.spawn_kwargs = None
 		self.bin_command = None
-		self.has_bin = False
+		self._bin_command = bin_command
+		self.bin_pkg = bin_pkg
 		if bin_command:
 			self.bin_command = portage.process.find_binary(bin_command)
-			if self.bin_command is None:
-				msg = ["Command not found: %s" % bin_command,
-				"Type \"emerge %s\" to enable %s support." % (bin_pkg, bin_command)]
-				for l in msg:
-					writemsg_level("!!! %s\n" % l,
-						level=self.logger.ERROR, noiselevel=-1)
-			else:
-				self.has_bin = True
+
+
+	@property
+	def has_bin(self):
+		if self.bin_command is None:
+			msg = ["Command not found: %s" % self._bin_command,
+			"Type \"emerge %s\" to enable %s support."
+			% (self.bin_pkg, self._bin_command)]
+			for l in msg:
+				writemsg_level("!!! %s\n" % l,
+					level=self.logger.ERROR, noiselevel=-1)
+			return False
+		return True
 
 
 	def _kwargs(self, kwargs):
-- 
2.2.2