X-Git-Url: https://git.ralfj.de/lilass.git/blobdiff_plain/f8b82a2309bc55f7a216669038dccacf650c6878..3b6568d8ba3cbd8278171e048ae9d92426c018ba:/dsl.py?ds=sidebyside diff --git a/dsl.py b/dsl.py index b4f77bf..9bbf45f 100755 --- a/dsl.py +++ b/dsl.py @@ -1,6 +1,6 @@ -#!/usr/bin/python +#!/usr/bin/python3 # DSL - easy Display Setup for Laptops -# Copyright (C) 2012 Ralf Jung +# Copyright (C) 2012-2014 Ralf Jung # # This program is free software; you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -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): @@ -118,7 +117,7 @@ def getXrandrInformation(): # unknown line # not fatal as my xrandr shows strange stuff when a display is enabled, but not connected #raise Exception("Unknown line in xrandr output:\n"+line) - print "Warning: Unknown xrandr line %s" % line + print("Warning: Unknown xrandr line %s" % line) return connectors # convert a (width, height) pair into a string accepted by xrandr as argument for --mode @@ -171,7 +170,7 @@ def classifyConnectors(allConnectors): if connector == internalConnector: raise Exception("%s is both internal and external, that doesn't make sense." % connector) else: - externalConnectors = allConnectors.keys() + externalConnectors = list(allConnectors.keys()) externalConnectors.remove(internalConnector) if not externalConnectors: raise Exception("No external connector found - either your config is wrong, or your machine has only one connector.") @@ -183,12 +182,15 @@ if __name__ == "__main__": try: # parse command-line arguments parser = argparse.ArgumentParser(description='easy Display Setup for Laptops') - parser.add_argument("-f, --frontend", + parser.add_argument("-f", "--frontend", dest="frontend", help="The frontend to be used for user interaction") - parser.add_argument("-r, --relative-position", + parser.add_argument("-r", "--relative-position", dest="rel_position", choices=('left', 'right', 'external-only'), help="Position of external screen relative to internal one") + parser.add_argument("-i", "--internal-only", + dest="internal_only", action='store_true', + help="Enable internal screen, disable all the others (as if no external screen was connected") cmdArgs = parser.parse_args() # load frontend @@ -206,7 +208,8 @@ if __name__ == "__main__": # check whether we got an external screen or not # Check what to do usedExternalConnector = findAvailableConnector(externalConnectors, connectors) # *the* external connector which is actually used - if usedExternalConnector is not None: + hasExternal = not cmdArgs.internal_only and usedExternalConnector is not None + if hasExternal: # there's an external screen connected, we need to get a setup if cmdArgs.rel_position is not None: # use command-line arguments (can we do this relPosition stuff more elegant?) @@ -232,8 +235,14 @@ if __name__ == "__main__": call = ["xrandr"] for name in connectorArgs: call += ["--output", name] + connectorArgs[name] - print "Call that will be made:",call + print("Call that will be made:",call) subprocess.check_call(call) + + # make sure the internal screen is really, *really* turned on if there is no external screen + if not hasExternal: + backlight = float(subprocess.check_output(["xbacklight", "-get"]).strip()) + if backlight == 0: # it's completely turned off, we better enable it + subprocess.check_call(["xbacklight", "-set", "100"]) except Exception as e: frontend.error(str(e)) raise