Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use newer freedesktop platform 23.08 #400

Merged
merged 2 commits into from
Sep 29, 2023
Merged

Use newer freedesktop platform 23.08 #400

merged 2 commits into from
Sep 29, 2023

Conversation

der-eismann
Copy link
Contributor

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

@flathubbot
Copy link
Contributor

Started test build 66894

@flathubbot
Copy link
Contributor

Build 66894 failed

@der-eismann
Copy link
Contributor Author

Probably needs to wait for flathub/org.electronjs.Electron2.BaseApp#43

@der-eismann
Copy link
Contributor Author

bot, build

@flathubbot
Copy link
Contributor

Queued test build for im.riot.Riot.

@flathubbot
Copy link
Contributor

Started test build 68601

@flathubbot
Copy link
Contributor

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

@der-eismann der-eismann marked this pull request as ready for review September 25, 2023 12:17
@bbhtt
Copy link
Contributor

bbhtt commented Sep 25, 2023

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
Copy link
Contributor Author

der-eismann commented Sep 25, 2023

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
Copy link
Contributor

bbhtt commented Sep 26, 2023

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

@flathubbot
Copy link
Contributor

Started test build 68939

@bbhtt
Copy link
Contributor

bbhtt commented Sep 26, 2023

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

@flathubbot
Copy link
Contributor

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
Copy link
Collaborator

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

@flathubbot
Copy link
Contributor

Started test build 69746

@der-eismann
Copy link
Contributor Author

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

Done!

@flathubbot
Copy link
Contributor

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

@SISheogorath SISheogorath merged commit 259f518 into flathub:master Sep 29, 2023
1 check passed
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.

None yet

4 participants