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

Volume keys working incorrectly on install. #8

Closed
l4jos opened this issue Dec 1, 2021 · 11 comments
Closed

Volume keys working incorrectly on install. #8

l4jos opened this issue Dec 1, 2021 · 11 comments
Labels
bug Something isn't working invalid This doesn't seem right stale Issue hasn't seen activity in awhile

Comments

@l4jos
Copy link

l4jos commented Dec 1, 2021

Model: star2lte
Magisk: version 23
OS: LineageOS 18.1 unofficial build for star2lte

Hi,

This is not a very big issue because I can install bromitebrowser afterwards, but I struggled with this module to let it install webview.apk and browser.apk. It does not pick up the .apk's for arm64 which I stored in /sdcard/WebviewManager/apks. Defaults are every time returned to Bromite Webview and Chromium browser. When module installer asks me to install chromium AND/OR bromite. I haven't found a way to answer the question correctly, because volume UP did not select (1.)Bromite but shows me the second option (2.chromium). And here it goes weird, because it's installing Chromium Browser which I do not want. Volume UP did not select (1.)Bromite. workaround is to install bromite manually afterwards or skip the question for user interaction or leaving it default by not responding volume up/down button.
full-2021-12-01-11_27_15.log

Kind regards,
Lajos

@github-actions
Copy link

github-actions bot commented Dec 1, 2021

Thanks for your contribution! Please keep in mind we will get to this issue when we have a chance. Logs and device information are mandatory if you do not want your issue closed without response.

@androidacy-user
Copy link
Contributor

Ah yes Samsung fuckery again. I'll talk to @Zackptg5 see what I can find out, it's his code that handles that.

@androidacy-user androidacy-user changed the title Module installer does not pick up browser.apk and webview.apk Volume keys working incorrectly on install. Dec 1, 2021
@androidacy-user
Copy link
Contributor

The browser.apk and webview.apk are no longer used so that part is intended behavior.

@androidacy-user androidacy-user added bug Something isn't working help wanted Extra attention is needed labels Dec 1, 2021
@Zackptg5
Copy link
Contributor

Zackptg5 commented Dec 1, 2021

@l4jos Can you save the log after flashing it through magisk managed? the log you attached doesn't show any mmtex output

@androidacy-user
Copy link
Contributor

androidacy-user commented Dec 1, 2021

@l4jos Can you save the log after flashing it through magisk managed? the log you attached doesn't show any mmtex output

We've modified your original logic, logging isn't sent to magisk logs.

The volume keys selector is entirely your own but we use our own fork of mmt-ex.

@l4jos create a file in the root of your internal storage called .androidacy-debug then install. Send /sdcard/WebViewManager/logs/install.log here

@l4jos
Copy link
Author

l4jos commented Dec 7, 2021

I tried again after creating .androidacy-debug in /sdcard
The first question "browser and webview?" accepts power button, the second one ignores power button (as far as I noticed)
This sequence I accidentally got degoogled chromium, was not the intention but for sharing install.log

Tbh I do not need this fixed. I just couldn't stand the situation last time and the phone by itself so I felt like sharing a log.

install.log

@Zackptg5
Copy link
Contributor

Zackptg5 commented Dec 7, 2021

I tried again after creating .androidacy-debug in /sdcard The first question "browser and webview?" accepts power button, the second one ignores power button (as far as I noticed) This sequence I accidentally got degoogled chromium, was not the intention but for sharing install.log

Tbh I do not need this fixed. I just couldn't stand the situation last time and the phone by itself (when it ignores the only button that are left on smartphones nowaday) so I felt like sharing a log.

install.log

Hmm, power button shouldn't have responded. Can you run keycheck manually in terminal and echo the output for each volume key? You can grab it from the zip or here

# Move keycheck to somewhere you can execute - ie not /sdcard
# cd to directory you moved it to
chmod +x keycheck
./keycheck
# Press volup key
echo $?
./keycheck
# Press voldown key
echo $?
./keycheck
# Press power key - should do nothing and still wait for input but your device may be weird
echo $? # If applicable

Then post the numbers for each here. Should be 42 for volup and 41 for voldown

@androidacy-user
Copy link
Contributor

What do you mean, the only button left on smartphones? Every device I know of has hardware volume keys, it's why we chose this method

@androidacy-user androidacy-user added the stale Issue hasn't seen activity in awhile label Dec 29, 2021
@androidacy-user
Copy link
Contributor

@l4jos we need the logs @Zackptg5 asked for or we'll have to close this issue.

@l4jos
Copy link
Author

l4jos commented Dec 29, 2021

What do you mean, the only button left on smartphones? Every device I know of has hardware volume keys, it's why we chose this method

I mean there were more buttons in history on telephones.

@androidacy-user
Copy link
Contributor

Closing as we didn't receive the logs requested. If you still need help, open a new issue with the appropriate logs.

@androidacy-user androidacy-user added invalid This doesn't seem right and removed help wanted Extra attention is needed labels Jan 6, 2022
@Androidacy Androidacy locked and limited conversation to collaborators Jan 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working invalid This doesn't seem right stale Issue hasn't seen activity in awhile
Projects
None yet
Development

No branches or pull requests

3 participants