Skip to content

fix: AyatanaAppIndicator3 fallback + bounded anisette bootstrap guard - #18

Open
MisterWanted wants to merge 1 commit into
SideStore:mainfrom
MisterWanted:fix/ayatana-appindicator-bootstrap-guard
Open

fix: AyatanaAppIndicator3 fallback + bounded anisette bootstrap guard#18
MisterWanted wants to merge 1 commit into
SideStore:mainfrom
MisterWanted:fix/ayatana-appindicator-bootstrap-guard

Conversation

@MisterWanted

Copy link
Copy Markdown

What

  • Fall back to AyatanaAppIndicator3 when AppIndicator3 is unavailable (several modern distros renamed it), so the tray icon loads instead of crashing at import.
  • Replace the curl 127.0.0.1:6969 | grep "{" subshells with a dependency-free, no-shell port_open() TCP probe.
  • Guard the bootstrap worker (lol321actualfunction) with a bounded 60s wait that polls port_open() instead of spinning forever, and surface the real failure via a new OopsAnisette error window instead of freezing the splash screen.
  • Bump the anisette_server download to 2.0.0 and add requests status checks.
  • shebang -> python3.

Why

On distros without AppIndicator3 (or without curl on PATH), the app currently dies at import or hangs on the splash forever with no diagnosable error. This makes bootstrap failures visible and lets the server start cleanly when needed.

Tested

Runs under a GTK3 environment where only AyatanaAppIndicator3 is present; tray icon shows and the anisette bootstrap path executes/errors gracefully.

- Fall back to AyatanaAppIndicator3 when AppIndicator3 is unavailable (modern distros renamed it).
- Replace curl|grep subshells with a socket-based port_open() probe (no shell, no curl-on-PATH dependency).
- Guard the bootstrap worker with a bounded 60s wait and surface the real exception via a new OopsAnisette window instead of freezing the splash forever.
- Bump anisette_server download to 2.0.0.
- shebang -> python3.
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.

2 participants