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

portals: Support GameMode #258

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

portals: Support GameMode #258

wants to merge 1 commit into from

Conversation

tchx84
Copy link
Owner

@tchx84 tchx84 commented Apr 12, 2021

No description provided.

@tchx84
Copy link
Owner Author

tchx84 commented Apr 13, 2021

Hey @gicmo,

According to this you're the right person to ask :)

Not sure if you have tried Flatseal before but is basically a GUI for managing Flatpak permissions (overrides, initially). I am currently working on adding support for managing dynamic permissions (portals) as well. I am close to be done, but I have some questions regarding GameMode:

  1. I see here that you're reading PermissionStore to determine if a particular application can go into GameMode, but I can't find where is being set. Is this just to make it future proof?

  2. Is well known among Flatseal users that permissions changes require restarting the application. It is safe to assume that next time the application (or game, in this case) runs, the PermissionStore values will be honered? Or is it there another mechanism that Flatseal would need to interact with?

Thanks in advance!

@gicmo
Copy link

gicmo commented Apr 13, 2021

Hey, ... all good questions. Let me refresh my memory and get back to you.

@tchx84
Copy link
Owner Author

tchx84 commented May 12, 2021

Hey, ... all good questions. Let me refresh my memory and get back to you.

hey @gicmo , did you get a chance to look back on this? Mind giving me any pointers to study, beyond gamemode code? 🙏

@otuva
Copy link

otuva commented Dec 14, 2022

Hi @gicmo and @tchx84, what's happened so far regarding this pr?

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.

3 participants