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

Drop org.kde.* own-name #318

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

bbhtt
Copy link

@bbhtt bbhtt commented Sep 28, 2023

This is no longer required with any supported runtimes, the issue was fixed in Qt

https://docs.flatpak.org/en/latest/desktop-integration.html#statusnotifier

Most implementations of StatusNotifer have dropped this requirement

flathub-infra/flatpak-builder-lint#66 (comment)

Fixes #111

@flathubbot
Copy link
Contributor

Started test build 69303

@flathubbot
Copy link
Contributor

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

flatpak install --user https://dl.flathub.org/build-repo/51958/com.dropbox.Client.flatpakref

@bbhtt
Copy link
Author

bbhtt commented Sep 28, 2023

This seems to need loggin in. I don't have an account, so testing needed.

@wjt
Copy link
Member

wjt commented Oct 29, 2023

I tested this (or rather, the equivalent using Flatseal) and the tray icon does not appear.

@bbhtt
Copy link
Author

bbhtt commented Oct 29, 2023

Ok thanks. I'll check tomorrow

@bbhtt bbhtt marked this pull request as draft October 29, 2023 17:04
This is no longer required with any supported runtimes, the issue was fixed in Qt

https://docs.flatpak.org/en/latest/desktop-integration.html#statusnotifier

> Most implementations of StatusNotifer have dropped this requirement

flathub-infra/flatpak-builder-lint#66 (comment)
@flathubbot
Copy link
Contributor

Started test build 77600

@flathubbot
Copy link
Contributor

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

flatpak install --user https://dl.flathub.org/build-repo/60193/com.dropbox.Client.flatpakref

@bbhtt
Copy link
Author

bbhtt commented Oct 30, 2023

They are using a super old Qt version, 5.5.1 (compiled with GCC 4.8.4 on Ubuntu 14.04). The fix that enables to drop this permission is in Qt 5.15 LTS (only in KDE's patchsets) and Qt 6 (upstream).

bbhtt added a commit to bbhtt/flatpak-builder-lint that referenced this pull request Jan 17, 2024
requires --own-name='org.kde.*' under flatpak

Only handles maintained applications that had activity in the past year

Zoom: flathub/us.zoom.Zoom#416
Discord: flathub/com.discordapp.Discord#324
DiscordCanary: flathub/com.discordapp.DiscordCanary#78
Celeste: flathub/com.hunterwittenborn.Celeste#35
Tribler: flathub/org.tribler.Tribler#10
Dropbox: flathub/com.dropbox.Client#318
barthalion pushed a commit to flathub-infra/flatpak-builder-lint that referenced this pull request Jan 23, 2024
requires --own-name='org.kde.*' under flatpak

Only handles maintained applications that had activity in the past year

Zoom: flathub/us.zoom.Zoom#416
Discord: flathub/com.discordapp.Discord#324
DiscordCanary: flathub/com.discordapp.DiscordCanary#78
Celeste: flathub/com.hunterwittenborn.Celeste#35
Tribler: flathub/org.tribler.Tribler#10
Dropbox: flathub/com.dropbox.Client#318
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.

Remove org.kde.* permission
3 participants