[Erp5-report] r29611 - /erp5/trunk/products/ERP5/Tool/TemplateTool.py

[nobody at svn.erp5.org]

Author: nicolas.dumazet
Date: Tue Oct 13 18:30:26 2009
New Revision: 29611

URL: http://svn.erp5.org?rev=29611&view=rev
Log:
doc-only: explain why we use a while construct here

Modified:
    erp5/trunk/products/ERP5/Tool/TemplateTool.py

Modified: erp5/trunk/products/ERP5/Tool/TemplateTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Tool/TemplateTool.py?rev=29611&r1=29610&r2=29611&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Tool/TemplateTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Tool/TemplateTool.py [utf8] Tue Oct 13 18:30:26 2009
@@ -459,6 +459,10 @@
                                  stdout=subprocess.PIPE,
                                  stderr=subprocess.STDOUT)
 
+      # "for line in process.stdout" is cleaner but is buffered,
+      # see http://bugs.python.org/issue3907
+      # We use this ugly construct to avoid waiting for test
+      # termination before printing content
       while True:
         line = process.stdout.readline()
         if not line: