Skip to content
This repository has been archived by the owner. It is now read-only.

Try to fix dumpsys command #3

Closed
wants to merge 1 commit into from
Closed

Try to fix dumpsys command #3

wants to merge 1 commit into from

Conversation

@vvb2060
Copy link

@vvb2060 vvb2060 commented May 21, 2017

The dumpsys command does not work with root.
Device: Nexus 6P Stock ROM 7.1.2-N2G47O
default

@liudongmiao
Copy link

@liudongmiao liudongmiao commented Jul 12, 2017

Suggest to squash to one commit.

@aviraxp
Copy link

@aviraxp aviraxp commented Jul 12, 2017

@topjohnwu Sorry to pin you here, but we need to get this work. Any status on this?

@liudongmiao
Copy link

@liudongmiao liudongmiao commented Jul 12, 2017

@topjohnwu It's a problem like this.

Brevent requires some result for dumpsys.
For example, dumpsys notification, dumpsys activity activities.

However, in MagiskSu, dumpsys doesn't work.
You can check it with dumpsys battery (maybe the shortest output if success).

Brevent requires this, and I think other apps may need this

@liudongmiao
Copy link

@liudongmiao liudongmiao commented Jul 24, 2017

@topjohnwu

Are you so busy, without any response in 2 month?
Fix, or not fix, without any word?

xcjiang
xcjiang approved these changes Sep 7, 2017
@xcjiang
Copy link

@xcjiang xcjiang commented Sep 7, 2017

@topjohnwu
please merge this fix ASAP, thanks.

@vvb2060 vvb2060 closed this Dec 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
5 participants