flathub / im.riot.Riot

https://flathub.org/apps/details/im.riot.Riot
32 stars 43 forks source link

Use newer freedesktop platform 23.08 #400

Closed der-eismann closed 9 months ago

der-eismann commented 9 months ago

A new version of the runtime was recently released, see here. Brings some updated packages, but they probably won't affect Element.

flathubbot commented 9 months ago

Started test build 66894

flathubbot commented 9 months ago

Build 66894 failed

der-eismann commented 9 months ago

Probably needs to wait for https://github.com/flathub/org.electronjs.Electron2.BaseApp/issues/43

der-eismann commented 9 months ago

bot, build

flathubbot commented 9 months ago

Queued test build for im.riot.Riot.

flathubbot commented 9 months ago

Started test build 68601

flathubbot commented 9 months ago

Build 68601 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/51248/im.riot.Riot.flatpakref
bbhtt commented 9 months ago

Can you check if desktop notifications work without --talk-name=org.freedesktop.Notifications?

The 23.08 baseapp includes libnotify 0.8.2 which should not require that hole.

der-eismann commented 9 months ago

I just tested it on Fedora 38 with Gnome 44, notifications actually worked without --talk-name=org.freedesktop.Notifications. Want me to remove it?

bbhtt commented 9 months ago

Yes, please. Add this link in the commit message https://gitlab.gnome.org/GNOME/libnotify/-/blob/69aff6e5fa2842e00b409c348bd73188548828b3/NEWS#L25

flathubbot commented 9 months ago

Started test build 68939

bbhtt commented 9 months ago

If someone using KDE, can test it too, would be great.

flathubbot commented 9 months ago

Build 68939 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/51588/im.riot.Riot.flatpakref
SISheogorath commented 9 months ago

If we get this rebased, I would merge it this weekend.

flathubbot commented 9 months ago

Started test build 69746

der-eismann commented 9 months ago

If we get this rebased, I would merge it this weekend.

Done!

flathubbot commented 9 months ago

Build 69746 successful To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/52397/im.riot.Riot.flatpakref