search

pimpimmi / LibreAlarm

https://plus.google.com/communities/118306966582165178782
GNU General Public License v3.0
52 stars 48 forks source link

Some rework of WearActivity.java #17

Closed jamorham closed 8 years ago

jamorham commented 8 years ago

I think this works and is an improvement but you should test it to verify my findings.

It needs root installed on the watch. I installed a wear compatible super-su binary via twrp from here:

http://forum.xda-developers.com/attachment.php?attachmentid=3342605&d=1433157678

jamorham commented 8 years ago

I've reworked this again after I found some times NFC got left on. I have it integrating with xDrip+ now so I think I will submit an updated PR with all the bits needed as soon as I've tidied that up and tested it a little more.

dabear commented 8 years ago

why does it require root?

jamorham commented 8 years ago

Because the watch activity is doing things like turning off the nfc service, adjusting the cpu clock speed and being able to disable the kernel touchscreen driver to ensure no screen presses occur.

As the watch is already being flashed with a custom kernel I didn't think the root requirement would be a big ask to give maximum flexibility for what we can do within the watch component.

jamorham commented 8 years ago

Have a look at https://github.com/pimpimmi/LibreAlarm/pull/18 instead now as it supersedes this.