Skip to content

pkgs/portail-tray-icon: fix startup race - #261

Open
rlahfa-dinum wants to merge 1 commit into
mainfrom
portail-trayicon-race
Open

rlahfa-dinum wants to merge 1 commit into
mainfrom
portail-trayicon-race

Conversation

@rlahfa-dinum

Copy link
Copy Markdown
Contributor

On startup, Portail crashes because it try to use Qt APIs before some things are ready.

Apparently, this sort of problem is well-known, albeit not easy to fix, many applications resort to busy polling to detect when things are ready.

We resort to similar workarounds.

On startup, Portail crashes because it try to use Qt APIs before some
things are ready.

Apparently, this sort of problem is well-known, albeit not easy to fix,
many applications resort to busy polling to detect when things are
ready.

We resort to similar workarounds.

Signed-off-by: Ryan Lahfa <ryan.lahfa.ext@numerique.gouv.fr>
@rlahfa-dinum rlahfa-dinum added status: awaiting smoke testing Awaiting basic smoke testing on the Sécurix team side. A/modules Usecases abstractions: high-level NixOS modules for a need status: ready-to-merge Approved + CI green (or accepted breakage), maintainer will merge once they have a time status: awaiting-maintainers This is blocked on a maintainer's review bandwidth and removed status: awaiting smoke testing Awaiting basic smoke testing on the Sécurix team side. status: ready-to-merge Approved + CI green (or accepted breakage), maintainer will merge once they have a time labels Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A/modules Usecases abstractions: high-level NixOS modules for a need status: awaiting-maintainers This is blocked on a maintainer's review bandwidth

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant