Announce root permissions requirement

Faerbit / android-crond

Simple Android app which implements crond using AlarmManager

MIT License

26 stars 6 forks source link

Announce root permissions requirement #1

Closed TPS closed 7 years ago

TPS commented 7 years ago

@Faerbit Tried this from F-droid, but, when run, it says it needs root (& I'm not gonna just for this app).

[ ] Please make sure app manifest requests root
[ ] Please update F-droid description w/ that, too.

Thanks!

Faerbit commented 7 years ago

Thanks for checking out my app.

As far as I am aware this commit accomplished that. I am unsure, of how to check if that metadata made it into the repos, and what to do about it if it didn't.

As for the app manifest, it seems the permission is deprecated.

TPS commented 7 years ago

If it's depreciated, then it can't hurt to include it. I think F-droid still checks for it to show such details before install.

Faerbit commented 7 years ago

Actually, yes it can: 1) It is not a standard. Some super user implementation might not know how to handle this. 2) Even with the SuperSU implementation by Chainfire this can cause problems.