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

fix install android debug samples apk error. #7230

Merged

Conversation

suzp1984
Copy link
Contributor

@suzp1984 suzp1984 commented Oct 4, 2023

cp not exist debug sample apk.

./build.sh -p android -k sample-gltf-viewer -i debug
Take sample-gltf-viewer as an example, above cmd will result in :

cp: samples/sample-gltf-viewer/build/outputs/apk/debug/sample-gltf-viewer-debug-unsigned.apk: No such file or directory

The root cause is that the android will signed the debug apk with ~/.android/debug.keystore, which is the default generated by android gradle plugin globally.

remove unsigned tag from release apk don't make sense

[Cause] the unsigned apk will not installed on the android, the apk must be signed anyway. So I think remove the unsigned keyword don't make sense, at lease let people know, it's an unsigned apk.

@suzp1984 suzp1984 force-pushed the fix/install-android-samples-cp-error branch from 8a8d723 to 95133b9 Compare October 4, 2023 16:00
build.sh Outdated Show resolved Hide resolved
build.sh Outdated Show resolved Hide resolved
@suzp1984 suzp1984 force-pushed the fix/install-android-samples-cp-error branch from 95133b9 to 1633c9a Compare October 4, 2023 17:02
@suzp1984 suzp1984 force-pushed the fix/install-android-samples-cp-error branch from 1633c9a to 23ab49f Compare October 5, 2023 00:49
@pixelflinger pixelflinger added the internal Issue/PR does not affect clients label Oct 10, 2023
@pixelflinger pixelflinger self-requested a review October 26, 2023 20:14
@pixelflinger pixelflinger force-pushed the fix/install-android-samples-cp-error branch from 23ab49f to d7ec69c Compare October 26, 2023 20:14
@suzp1984 suzp1984 force-pushed the fix/install-android-samples-cp-error branch from d7ec69c to 9edc650 Compare March 23, 2024 05:13
@suzp1984
Copy link
Contributor Author

I notice this problem still exist.
cp: samples/sample-gltf-viewer/build/outputs/apk/debug/sample-gltf-viewer-debug-unsigned.apk: No such file or directory

@romainguy romainguy enabled auto-merge (squash) March 23, 2024 06:04
@romainguy romainguy merged commit 2763931 into google:main Mar 23, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Issue/PR does not affect clients
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants