diff options
author | Zac Medico <zmedico@gentoo.org> | 2011-12-01 22:14:13 -0800 |
---|---|---|
committer | Zac Medico <zmedico@gentoo.org> | 2011-12-01 22:14:13 -0800 |
commit | c6c71eefb9532af0d4505de70e58f45ad8e8fe0a (patch) | |
tree | f157124695ebd1a818c5ce3ed07fc343ff827a35 /pym/_emerge/PipeReader.py | |
parent | caea9ee807eba03118564030a166f9856d4439de (diff) | |
download | portage-c6c71eefb9532af0d4505de70e58f45ad8e8fe0a.tar.gz portage-c6c71eefb9532af0d4505de70e58f45ad8e8fe0a.tar.bz2 portage-c6c71eefb9532af0d4505de70e58f45ad8e8fe0a.zip |
Handle OSError from os.read and loop if needed.
Looping fixes EbuildMetadataPhase failures for ebuilds that produce
more than 4096 bytes of metadata, broken since commit
b432a1b3051d91546649e8f3190675767461d8e8.
Diffstat (limited to 'pym/_emerge/PipeReader.py')
-rw-r--r-- | pym/_emerge/PipeReader.py | 25 |
1 files changed, 14 insertions, 11 deletions
diff --git a/pym/_emerge/PipeReader.py b/pym/_emerge/PipeReader.py index 67144c393..af8cfdd4e 100644 --- a/pym/_emerge/PipeReader.py +++ b/pym/_emerge/PipeReader.py @@ -59,18 +59,21 @@ class PipeReader(AbstractPollTask): if event & PollConstants.POLLIN: - data = None - try: - data = os.read(fd, self._bufsize) - except IOError as e: - if e.errno not in (errno.EAGAIN,): - raise - else: - if data: - self._read_data.append(data) + while True: + data = None + try: + data = os.read(fd, self._bufsize) + except OSError as e: + if e.errno not in (errno.EAGAIN,): + raise + break else: - self._unregister() - self.wait() + if data: + self._read_data.append(data) + else: + self._unregister() + self.wait() + break self._unregister_if_appropriate(event) |