Skip to content

fix(sync): log native panics to logcat (#220) - #235

Draft
0xbrayo wants to merge 1 commit into
ActivityWatch:masterfrom
0xbrayo:fix/220-log-rust-panics-to-logcat
Draft

fix(sync): log native panics to logcat (#220)#235
0xbrayo wants to merge 1 commit into
ActivityWatch:masterfrom
0xbrayo:fix/220-log-rust-panics-to-logcat

Conversation

@0xbrayo

@0xbrayo 0xbrayo commented Aug 24, 2026

Copy link
Copy Markdown
Member

Draft until ActivityWatch/aw-server-rust#654 is merged.

First follow-up from #220: bump aw-server-rust so libaw_sync.so installs a panic hook that writes via android_log.

The default Rust panic hook writes to stderr, which Android discards, so the SIGABRT in #220 currently has no panic message in logcat. After this lands, panics should show under tags aw-sync / aw-server-rust.

This does not fix the abort itself; it makes the panic diagnosable.

Once #654 is merged, update the submodule pointer to the merge commit (especially if squashed) before marking this ready.

Bump aw-server-rust so libaw_sync.so installs a panic hook that writes
via android_log. The SIGABRT in issue 220 currently has no panic
message in logcat because Rust's default hook uses stderr, which
Android discards.
@greptile-apps

greptile-apps Bot commented Aug 24, 2026

Copy link
Copy Markdown

Greptile Summary

Updates the aw-server-rust submodule revision so native synchronization panics can be written to Android logcat for diagnosis.

  • Advances the native server dependency from e8e6e90 to 34814ce.
  • Targets panic diagnostics for libaw_sync.so without changing the underlying abort behavior.

Confidence Score: 5/5

No actionable defect is established in the available submodule-pointer change.

The parent repository changes only the native dependency revision, and the available evidence does not demonstrate a broken build, runtime contract, or security boundary.

Important Files Changed

Filename Overview
aw-server-rust Advances the submodule pointer for Android-native panic logging; no concrete defect is established from the available gitlink-only change.

Reviews (1): Last reviewed commit: "fix(sync): log native panics to logcat (..." | Re-trigger Greptile

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.

1 participant