Skip to content
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

Detect systemless root #142

Closed
wants to merge 1 commit into from
Closed

Detect systemless root #142

wants to merge 1 commit into from

Conversation

d4rken
Copy link
Contributor

@d4rken d4rken commented Feb 11, 2017

/su/bin should only exist on rooted device.

@d4rken
Copy link
Contributor Author

d4rken commented Feb 12, 2017

Merge #140 to fix the build issue.

@kattrali kattrali closed this in 663589b Feb 21, 2017
@kattrali
Copy link
Contributor

Thanks, @d4rken! I merged manually to pull out the unrelated changes, but this looks good. 👍

@d4rken
Copy link
Contributor Author

d4rken commented Feb 21, 2017

@kattrali Are the other changes against the prefered code style or what's the issue with them? I made those on purpose (wasn't an auto formatter accident).

@kattrali
Copy link
Contributor

@d4rken Ah, gotcha. I assumed it was from a formatter. From a maintenance perspective, its hard to look back through commits when a single change can have different purposes. I spend a fair amount of time looking back through rationale for decisions in git history, so generally try to not mix formatting changes in with hot fixes. Outside of that,the changes were a bit different from the existing style, so I the purpose of the change wasn't obvious during review.

@d4rken d4rken deleted the pr-rootdetection branch February 21, 2017 22:23
lemnik pushed a commit that referenced this pull request Jun 2, 2021
rich-bugsnag pushed a commit that referenced this pull request Sep 3, 2021
feat: disable ANR detection in Unity notifier
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants