From: Ralf Jung Date: Sat, 7 Sep 2013 10:23:27 +0000 (+0200) Subject: make it work properly with python 3 X-Git-Url: https://git.ralfj.de/lilass.git/commitdiff_plain/27d7d48deea2278c3cb593805ed8c2a7f789c0d5?ds=sidebyside;hp=--cc make it work properly with python 3 --- 27d7d48deea2278c3cb593805ed8c2a7f789c0d5 diff --git a/dsl.py b/dsl.py index 9dd2223..7dc8665 100755 --- a/dsl.py +++ b/dsl.py @@ -57,15 +57,15 @@ class ScreenSetup: return args # Load a section-less config file: maps parameter names to space-separated lists of strings (with shell quotation) -def loadConfigFile(file): +def loadConfigFile(filename): import shlex result = {} - if not os.path.exists(file): + if not os.path.exists(filename): return result # no config file # read config file linenr = 0 - with open(file) as file: - for line in file: + with open(filename) as f: + for line in f: linenr += 1 line = line.strip() if not len(line) or line.startswith("#"): continue # skip empty and comment lines @@ -82,10 +82,9 @@ def loadConfigFile(file): # helper function: execute a process, return output as iterator, throw exception if there was an error # you *must* iterate to the end if you use this! def processOutputGen(*args): - p = subprocess.Popen(args, stdout=subprocess.PIPE) - for line in p.stdout: - yield line - p.wait() # wait for process to exit (it closed stdout, so it can't block anymore) + with subprocess.Popen(args, stdout=subprocess.PIPE) as p: + for line in p.stdout: + yield line.decode("utf-8") if p.returncode != 0: raise Exception("Error executing "+str(args)) def processOutputIt(*args):