The Wayback Machine - https://web.archive.org/web/20220706204055/https://github.com/opengapps/opengapps/pull/948
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

add Go edition variants #948

Merged
merged 7 commits into from Jun 11, 2022
Merged

add Go edition variants #948

merged 7 commits into from Jun 11, 2022

Conversation

tenzap
Copy link
Contributor

@tenzap tenzap commented Jan 12, 2022

Comments are welcome.
To add packages, new google certificates need to be added to the opengapps keystore #949

Go variants are suffixed with "_go" (except super which contains in itself the go packages).

For example
bash build_gapps.sh arm 31 pico_go

In some cases, if the script was run with sh or bash (instead of ./)
TOP couldn't be set correctly
@tenzap tenzap changed the title add Go edition variants (WIP) add Go edition variants Jan 12, 2022
@tenzap tenzap force-pushed the f_go_edition branch 12 times, most recently from b37ac53 to 6c7513c Compare Jan 16, 2022
tenzap added 2 commits Jan 17, 2022
On pie emulator system-image, GmsCore is stored in priv-app/PrebuiltGmsCore and not in priv-app/PrebuiltGmsCorePi
so remove this too
@mfonville
Copy link
Member

@mfonville mfonville commented Apr 29, 2022

Thanks, sorry that I didn't take a look earlier yet. But I will need some time to review

Copy link
Member

@mfonville mfonville left a comment

Thanks for your big PR to set-up the Go-packages.

Did you do extensive testing on these changes? It are quite a lot of changes, so it is good to make sure

scripts/build_gapps.sh Outdated Show resolved Hide resolved
@tenzap
Copy link
Contributor Author

@tenzap tenzap commented May 15, 2022

Did you do extensive testing on these changes? It are quite a lot of changes, so it is good to make sure

I don't know what you mean by extended, but I tested it with lineageos & google system images using emulated device with this script that works from android 8 up to android 12.
I also tested with different variants.

Tests also led to filing #951 & #952

@mfonville
Copy link
Member

@mfonville mfonville commented Jun 11, 2022

Thanks for all the testing, if you can fix that bashism I will merge it!

@mfonville
Copy link
Member

@mfonville mfonville commented Jun 11, 2022

Could you rebase on current master to resolve the conflict?

@tenzap
Copy link
Contributor Author

@tenzap tenzap commented Jun 11, 2022

I don't see any conflict. I solved it in 6c6b593

@tenzap
Copy link
Contributor Author

@tenzap tenzap commented Jun 11, 2022

After merge, you may also consider #949

@mfonville
Copy link
Member

@mfonville mfonville commented Jun 11, 2022

Github still mentions the conflict, can you rebase on 897620f ?

@tenzap
Copy link
Contributor Author

@tenzap tenzap commented Jun 11, 2022

IMG_20220611_192553
That's what I have

@mfonville
Copy link
Member

@mfonville mfonville commented Jun 11, 2022

That is so weird, I have "This branch cannot be rebased due to conflicts".
I will rebase manually

@mfonville mfonville merged commit f6d7696 into opengapps:master Jun 11, 2022
@tenzap
Copy link
Contributor Author

@tenzap tenzap commented Jun 11, 2022

Ok. Thanks again for merging.

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.

None yet

2 participants