-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch VT using dbus to fix #72 #75
base: develop
Are you sure you want to change the base?
Conversation
|
||
def __enter__(self): | ||
logger.info('running on tty: %s', self.tty_name) | ||
# really useful ? Can block if permission is not set correctly on /dev/ttyx |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I recall, we need to enable graphics. It is not enabled by default.
self.seat_methods.SwitchTo(self.tty_num) | ||
except: | ||
logger.exception('cannot switch to: %s', self.tty_name) | ||
else: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: I'd probaby just print this in init() and remove the ekese part here.
Thanks for this. Is that dbus interface already available under jessie? I am still mostly using jessie due to lack of time testing stretch :/ |
I don't know, I juste have stretch now, you can check with qdbus for instance, but if the dbus interface isn't available, it won't hurt, , the exception will be catched, self.tty_name will be equal to self.previous_tty_num and it won't try to switch, as before |
Should fix #72