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

Improve Android logging #814

Closed
mhsmith opened this issue Aug 7, 2022 · 1 comment · Fixed by #821
Closed

Improve Android logging #814

mhsmith opened this issue Aug 7, 2022 · 1 comment · Fixed by #821
Labels
android The issue relates to Android mobile support. enhancement New features, or improvements to existing features.

Comments

@mhsmith
Copy link
Member

mhsmith commented Aug 7, 2022

Is your feature request related to a problem? Please describe.

When running an Android app, the logcat filtering currently displays a few specific tags from all apps. The problem is:

  • If you have multiple apps on the device, you may see logs from all of them mixed together.
  • There may be important information in other tags, such as Java or JNI crashes, which Briefcase doesn't display at all.

Describe the solution you'd like

Briefcase should log all tags, but only for the app the user is working on. This would match the default behavior of Android Studio.

Depending on what's best supported by the tools, this may involve filtering by process ID, user ID, application ID, or something else.

@mhsmith mhsmith added the enhancement New features, or improvements to existing features. label Aug 7, 2022
@freakboy3742 freakboy3742 added up-for-grabs android The issue relates to Android mobile support. labels Aug 8, 2022
@freakboy3742
Copy link
Member

+1.

An additional nice-to-have - it would be desirable to be able to configure this as an end user. No matter what we pick as a default, there will be a situation where the value we've picked isn't as helpful as it could be. The user can always involve adb logcat themselves, but setting up the runtime environment to access Briefcase's adb instance isn't trivial for end users.

It's not worth doing excessive gymnastics to make this happen; but if there's an easy pass through of arguments that we can enable (e.g., a multi-valued --adb-filter <filter to add> option), it might be worth considering.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android The issue relates to Android mobile support. enhancement New features, or improvements to existing features.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants