Skip to content

Update exceptions.json#946

Open
QmwJlHuSg9pa wants to merge 2 commits intoflathub-infra:masterfrom
QmwJlHuSg9pa:patch-1
Open

Update exceptions.json#946
QmwJlHuSg9pa wants to merge 2 commits intoflathub-infra:masterfrom
QmwJlHuSg9pa:patch-1

Conversation

@github-actions github-actions bot added the exceptions Request to add exceptions. Handled by automation label Mar 15, 2026
@QmwJlHuSg9pa QmwJlHuSg9pa marked this pull request as draft March 15, 2026 13:01
@QmwJlHuSg9pa QmwJlHuSg9pa marked this pull request as ready for review March 15, 2026 13:03
"finish-args-unnecessary-xdg-data-umu-create-access": "Uses shared installation for the umu runtime",
"finish-args-unnecessary-xdg-data-Steam-create-access": "Uses shared installation for Proton builds",
"finish-args-unnecessary-xdg-config-MangoHud-ro-access": "Access shared MangoHud configuration files",
"finish-args-unnecessary-xdg-cache-lutris-create-access": "Access to Lutris cache directory",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why you need to mount host cache directory instead of using flatpak default one?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lutris, non-Flatpak'ed and Flatpak'ed alike, are set to use this directory: https://github.com/lutris/lutris/blob/a5b1a4eda7361292de0239b0cf5bbc20562b7979/lutris/settings.py#L31

I actually should also add xdg-config/lutris and xdg-data/lutris (similarly defined on lines 21 and 22)...

Copy link
Contributor

@Erick555 Erick555 Mar 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unlike non-flatpak, flatpak sets XDG_*_HOME env variables though so the cache, config and data paths are different between flatpak & non-flatpak installations. lutris flatpak shouldn't use host ones right now, even with --filesystem=home. Your change would mount host paths on flatpak specific path (effectively sharing flatpak & non-flatpak installations) which would be brand new behavior and possibly unwanted.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't precisely account for the actual functioning of code that handles xdg-cache/lutris, xdg-config/lutris, or xdg-data/lutris -- but my naive understanding is that, under some parameters, Lutris intentionally bypasses Flatpak paths, in favour of host paths?

At least, in plain English, this is explicitly explained as the intent of some subdirectories that point to xdg-data/lutris: lutris/lutris#6503

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No there is no reason to bypass Flatpak's cache folder location here.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but my naive understanding is that, under some parameters, Lutris intentionally bypasses Flatpak paths, in favour of host paths?

At least, in plain English, this is explicitly explained as the intent of some subdirectories that point to xdg-data/lutris: lutris/lutris#6503

The paths mentioned in that thread aren't from lutris but from 3rd party tools like steam or umu launcher. Lutris don't control what paths those use but does control its own paths.

@bbhtt
Copy link
Contributor

bbhtt commented Mar 17, 2026

you aren't a maintainer of the lutris flatpak and no maintainer has commented there. so this is on hold. at this point we don't know if the PRs adding the permissions will be accepted by lutris maintainers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked exceptions Request to add exceptions. Handled by automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants