Merge

eventlet/greenio.py

@@ -336,7 +336,7 @@ class GreenSocket(object):
 class GreenPipe(object):
     """ GreenPipe is a cooperatively-yielding wrapper around OS pipes.
     """
-    newlines = '\r\n'
+    newlines = '\n'
     def __init__(self, fd):
         set_nonblocking(fd)
         self.fd = fd
@@ -416,7 +416,7 @@ class GreenPipe(object):
                     chunk, self.recvbuffer = buf[:found], buf[found:]
                     return chunk
                 checked = max(0, len(buf) - (len(terminator) - 1))
-                d = self.read(BUFFER_SIZE)
+                d = self._recv(BUFFER_SIZE)
                 if not d:
                     break
                 buf += d
@@ -428,7 +428,7 @@ class GreenPipe(object):
                 chunk, self.recvbuffer = buf[:found], buf[found:]
                 return chunk
             checked = len(buf)
-            d = self.read(BUFFER_SIZE)
+            d = self._recv(BUFFER_SIZE)
             if not d:
                 break
             buf += d

tests/greenio_test.py

@@ -260,6 +260,38 @@ class TestGreenIo(LimitedTestCase):
         server.close()
         client.close()
 
+    def test_pipe_read(self):
+        # ensure that 'readline' works properly on GreenPipes when data is not
+        # immediately available (fd is nonblocking, was raising EAGAIN)
+        # also ensures that readline() terminates on '\n' and '\r\n'
+        r, w = os.pipe()
+
+        r = os.fdopen(r)
+        w = os.fdopen(w, 'w')
+
+        r = greenio.GreenPipe(r)
+        w = greenio.GreenPipe(w)
+
+        def writer():
+            eventlet.sleep(.1)
+
+            w.write('line\n')
+            w.flush()
+
+            w.write('line\r\n')
+            w.flush()
+
+        gt = eventlet.spawn(writer)
+
+        eventlet.sleep(0)
+
+        line = r.readline()
+        self.assertEquals(line, 'line\n')
+
+        line = r.readline()
+        self.assertEquals(line, 'line\r\n')
+
+        gt.wait()
 
 class TestGreenIoLong(LimitedTestCase):
     TEST_TIMEOUT=10  # the test here might take a while depending on the OS