[prev in list] [next in list] [prev in thread] [next in thread] 

List:       lyx-devel
Subject:    [patch for branch] fix bug #8477
From:       Uwe Stöhr <uwestoehr () web ! de>
Date:       2012-12-31 20:50:16
Message-ID: 50E1FA88.9080709 () web ! de
[Download RAW message or body]

The attached patch fixes the problem that configure.py hide the output of chkconfig.lty. But this 
output is very important to give the user a feedback what happens, that something happens and why 
LyX or the installation of it takes so long.

This fixes a regression introduced with LyX 2.0.0.
I tested this a lot and are forced to use it for the Windows installer because I got some user 
complaints about this.

OK to go in?

regards Uwe

["bug8477.diff" (text/x-patch)]

diff --git "a/C:\\DOCUME~1\\usti\\LOCALS~1\\Temp\\con478.tmp\\configure-60bca5b-left.py" \
"b/D:\\LyXGit\\2.0.x\\lib\\configure.py" index f02a5ec..62d4cf2 100644
--- "a/C:\\DOCUME~1\\usti\\LOCALS~1\\Temp\\con478.tmp\\configure-60bca5b-left.py"
+++ "b/D:\\LyXGit\\2.0.x\\lib\\configure.py"
@@ -1198,8 +1198,11 @@ def checkLatexConfig(check_config, bool_docbook):
     #
     # we have chklayouts.tex, then process it
     ret = 1
-    latex_out = cmdOutput(LATEX + ' wrap_chkconfig.ltx')
-    for line in latex_out.splitlines():
+    latex_out = os.popen(LATEX + ' wrap_chkconfig.ltx')
+    while True:
+        line = latex_out.readline()
+        if not line:
+            break;
         if re.match('^\+', line):
             logger.info(line.strip())
             # return None if the command succeeds



[prev in list] [next in list] [prev in thread] [next in thread]