From d4101f5f57a2a2f7be7bac1e16954142ae8742f4 Mon Sep 17 00:00:00 2001 From: Zac Medico Date: Sun, 15 Aug 2010 14:34:14 -0700 Subject: Fix SubProcess instances to call _set_returncode() when appropriate, instead of setting self.returncode directly. --- pym/_emerge/BinpkgFetcher.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'pym/_emerge/BinpkgFetcher.py') diff --git a/pym/_emerge/BinpkgFetcher.py b/pym/_emerge/BinpkgFetcher.py index 5677419e7..0032e601d 100644 --- a/pym/_emerge/BinpkgFetcher.py +++ b/pym/_emerge/BinpkgFetcher.py @@ -63,7 +63,7 @@ class BinpkgFetcher(SpawnProcess): if pretend: portage.writemsg_stdout("\n%s\n" % uri, noiselevel=-1) - self.returncode = os.EX_OK + self._set_returncode((self.pid, os.EX_OK)) self.wait() return @@ -102,7 +102,7 @@ class BinpkgFetcher(SpawnProcess): def _set_returncode(self, wait_retval): SpawnProcess._set_returncode(self, wait_retval) - if self.returncode == os.EX_OK: + if not self.pretend and self.returncode == os.EX_OK: # If possible, update the mtime to match the remote package if # the fetcher didn't already do it automatically. bintree = self.pkg.root_config.trees["bintree"] -- cgit v1.2.3-1-g7c22