flathub / org.scummvm.ScummVM

https://flathub.org/apps/details/org.scummvm.ScummVM
3 stars 5 forks source link

Add permissions to store screenshots #32

Closed prodrigestivill closed 1 year ago

prodrigestivill commented 1 year ago

This PR fixes the issue #24 by allowing access to XDG Pictures folder.

flathubbot commented 1 year ago

Started test build 31812

flathubbot commented 1 year ago

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

flatpak install --user https://dl.flathub.org/build-repo/14464/org.scummvm.ScummVM.flatpakref
AsciiWolf commented 1 year ago

Thanks for the PRs! :-) @lotharsm Could you please take a look at them?

lotharsm commented 1 year ago

Hi there!

This seems to be a very useful fix. Indeed ScummVM stores the screenshots in the "Pictures" folder unless configured otherwise. Thus, access to this directory is required as pointed out in the bug report.

Looks good to me - thank you :-)