-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Move Tvserver to examples #11780
Move Tvserver to examples #11780
Conversation
copy from examples/tv-app/linux but later will be moved to jni and upto java wordconnectedhomeip/examples/tv-app/android/BUILD.gn Lines 22 to 29 in 3dbd9a1
This comment was generated by todo based on a
|
add classpath support (we likely need to add something likeconnectedhomeip/examples/tv-app/android/BUILD.gn Lines 93 to 103 in 3dbd9a1
This comment was generated by todo based on a
|
Runtime dependencies should be computed by the build system rather than hardcodedconnectedhomeip/scripts/build/builders/android.py Lines 147 to 157 in 3dbd9a1
This comment was generated by todo based on a
|
3dbd9a1
to
d750561
Compare
PR #11780: Size comparison from 5623978 to d750561 Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
d750561
to
4f6010f
Compare
PR #11780: Size comparison from 5623978 to 4f6010f Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Fast tracking, given this is a platform maintainer updating their platform. |
… to example, so that we cloud have different android tv apps for different clusters
4f6010f
to
ecab319
Compare
PR #11780: Size comparison from cbd3e95 to ecab319 Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
ecab319
to
402eccb
Compare
PR #11780: Size comparison from cbd3e95 to 402eccb Full report (13 builds for efr32, k32w, p6, qpg, telink)
|
402eccb
to
f7ddab7
Compare
PR #11780: Size comparison from cbd3e95 to f7ddab7 Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
|
||
/** | ||
* @file | ||
* Implementation of JNI bridge for Tv App on Android. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: this @file
comment does not seem to help me understand code more: it is obvious that a file named TVApp-JNI.cpp
contains an implementation of JNI for a TV APP
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, I will remove it in next PR
* move share lib and tv-common(zzz_generated/tv-app/zap-generated) link to example, so that we cloud have different android tv apps for different clusters * make it buildable under example * move CHIPTVServer to examples/tv-app/android * fix runtime issue, force load jni before using AndroidChipPlatform * change build_example.py for new app struct * fix restyled-io and ci issues
Problem
Change overview
Testing