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

Offer reboot after update #1382

Closed
vanowm opened this issue Feb 18, 2014 · 4 comments
Closed

Offer reboot after update #1382

vanowm opened this issue Feb 18, 2014 · 4 comments

Comments

@vanowm
Copy link

vanowm commented Feb 18, 2014

Right now user must manually reboot the phone, and if they don't they might get this:
xprivacy_after_update

Would be nice if XPrivacy would offer reboot when clicked on "Device restart required" in notification.

@M66B
Copy link
Owner

M66B commented Feb 18, 2014

Unfortunately Android has no standard way to reboot your device. It is easy to reboot the device through Linux (when having root permissions), but what is often forgotten, is that you might lose data this way and even your storage devices (internal, SD card, etc) might get corrupt using this reboot method. For this reason I don't include such an option.

@M66B
Copy link
Owner

M66B commented Feb 18, 2014

To illustrate: I reboot my device often this way and once in a while I have to do a clean install, because the internal storage was corrupted. This will happen to others too, often not thinking this is the problem, or resulting in complaints.

@vanowm
Copy link
Author

vanowm commented Feb 18, 2014

I see.
Do you kill all processes before the reboot?
Perhaps there is a way invoke a power menu, like by simulating power button press?

This is offtopic, but perhaps to fix the mess as per screenshot, XP should disable ondemand after update and block by default until rebooted?

@M66B
Copy link
Owner

M66B commented Feb 18, 2014

Actually I think I have found a clean way to reboot through the power manager.
Formerly this was not possible, because only system applications can do that.
Since the new privacy service runs within Android itself, this is not an obstacle anymore.
With a little bit of trickery I have already a working proof of concept :-)

The on demand stuff doesn't know the app was update ...

@M66B M66B reopened this Feb 18, 2014
@M66B M66B added lowprio and removed wontfix labels Feb 18, 2014
@M66B M66B closed this as completed in 343bbb7 Feb 18, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants