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

ignore TrichromeLibrary since it doesn't extist for arm #278

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

root-intruder
Copy link

similar to issue #222 I get build errors during makefile gathering for nonexisting packets on arm:

[ 99% 13085/13114] including vendor/opengapps/build/modules/TrichromeLibrary/Android.mk ... FAILED: zipinfo: cannot find or open , .zip or .ZIP. vendor/opengapps/build/modules/GoogleWebViewOverlay/Android.mk: error: TrichromeLibraryGoogle: No source files specified build/make/core/prebuilt_internal.mk:37: error: done. 10:28:56 ckati failed with: exit status 1

...since TrichromeLibrary doesn't exist on arm only on arm64...thus it should be excluded from makefiles

@MarijnS95
Copy link
Contributor

@root-intruder Do you happen to know if Google provides a Chrome/webview variant of arm that depends on trichrome, but has simply not been added to the OpenGapps repos yet?

@root-intruder
Copy link
Author

no I haven't researched much but according to #253 (comment) there should be...but since I don't want webview etc. in the rom I didn't further research

@acr92
Copy link
Contributor

acr92 commented Oct 22, 2020

There exists a TrichromeLibrary for arm and arm64. https://www.apkmirror.com/apk/google-inc/trichrome-library/trichrome-library-86-0-4240-110-release/

I guess this should be added to OpenGapps @mfonville

@mfonville
Copy link
Member

@nezorflame and @NicholasBuse what is your take on this?

@nezorflame
Copy link
Contributor

While it does exist, it never worked for me when I was testing it. Need to take a second look at it though, maybe something changed.

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.

5 participants