Skip to content
This repository has been archived by the owner on Oct 27, 2019. It is now read-only.

[DO NOT MERGE] Update for 0.8.8 #35

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mmhobi7
Copy link

@mmhobi7 mmhobi7 commented May 19, 2018

#34

@mmhobi7
Copy link
Author

mmhobi7 commented May 19, 2018

it's a bit slow to restart, and I can't tell if it's my system or the indicator

@Motherboard
Copy link

Motherboard commented May 20, 2018

Leave the logout for versions for which it works... some systems (like mine) still work well with logout.

@andrebrait
Copy link
Owner

IMHO, we haven't investigated enough to make such an extreme solution the default.

I'll leave this PR open until I can take a better look into this.

@mmhobi7 mmhobi7 changed the title Replace Log Out with Reboot [DO NOT MERGE] Update for 0.8.8 May 20, 2018
@mmhobi7
Copy link
Author

mmhobi7 commented May 20, 2018

/sys/kernel/debug/vgaswitcheroo/switch requires root to read, and I didn't test this.

@mmhobi7
Copy link
Author

mmhobi7 commented May 20, 2018

also, the output is misleading as of right now, https://bugs.launchpad.net/ubuntu/+source/nvidia-prime/+bug/1757180

out = subprocess.getoutput("cat /proc/acpi/bbswitch")
return out.lower().endswith("on")
out = subprocess.getoutput("cat /sys/kernel/debug/vgaswitcheroo/switch")
return "DIS: :Off" in out
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be "not in", or should change "Off" to the opposite which I believe is "Pwr" but I need to confirm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants