Skip to content

Conversation

@agx
Copy link
Collaborator

@agx agx commented Oct 1, 2025

This keeps OSK startup working with gnome-session >= 49 which removed the RequiredComponents key. We can keep that around in our session file for a bit longer so we can work with older gnome-session as well.

This needs https://gitlab.gnome.org/World/Phosh/stevia/-/merge_requests/242 hence draft

@github-actions

This comment has been minimized.

agx added 3 commits October 4, 2025 14:57
Signed-off-by: Guido Günther <agx@sigxcpu.org>
Avoids warnings during build

Signed-off-by: Guido Günther <agx@sigxcpu.org>
This keeps OSK startup working with gnome-session >= 49 which
removed the `RequiredComponents` key. We can keep that around
in our session file for a bit longer so we can work with older
gnome-session as well.

Signed-off-by: Guido Günther <agx@sigxcpu.org>
@agx
Copy link
Collaborator Author

agx commented Oct 4, 2025

Turns out we need a bit more to work with gnome-session 49. This fixes it for me.

@agx agx marked this pull request as ready for review October 4, 2025 13:15
@github-actions

This comment has been minimized.

Since RequiredComponents is no longer handled by gnome-session
we need to start phrog via systemd. The unit files match what
phosh does.

This allows phrog to start with gnome-session >= 49.

Signed-off-by: Guido Günther <agx@sigxcpu.org>
@agx
Copy link
Collaborator Author

agx commented Oct 16, 2025

v1.1

  • Fix a typo in a comment in one of the unit files

@github-actions
Copy link

The CI build recorded and generated some videos

Demo video (shown on README and release notes)

accent-colours

emergency-calls

first-run

simple-flow

trivial-flow

@agx agx merged commit a0466a2 into samcday:main Oct 16, 2025
3 checks passed
@agx
Copy link
Collaborator Author

agx commented Oct 16, 2025

As this prevents the session from starting I'll merge this. @samcday please let me know if I should let things sit longer for feedback.

@samcday
Copy link
Owner

samcday commented Nov 2, 2025

As this prevents the session from starting I'll merge this. @samcday please let me know if I should let things sit longer for feedback.

No complaints here (especially since I've been quite inactive lately!) :)

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