8000 README: Add permissions rationale by travier · Pull Request #128 · flathub/org.inkscape.Inkscape · GitHub
[go: up one dir, main page]

Skip to content

README: Add permissions rationale - #128

Closed
travier wants to merge 1 commit into
flathub:masterfrom
travier:readme
Closed

travier wants to merge 1 commit into
flathub:masterfrom
travier:readme

Conversation

@travier
@travier travier commented Jul 8, 2022
Copy link
Copy Markdown
Member

JSON does not support comments well thus move those to the README.

JSON does not support comments well thus move those to the README.
@travier
travier commented Jul 8, 2022
Copy link
Copy Markdown
Member Author

Does Inkscape really need full host filesystem acces? Probably for exports?

@hfiguiere
Copy link
Copy Markdown
Contributor

I would say it doesn't but then downgrading permissions break for users.

(I'm not the original maintainer)

@travier
travier commented Jul 8, 2022
Copy link
Copy Markdown
Member Author

Note that this PR does not touch the permissions. I'm just asking to update the README with the info.

@hfiguiere
Copy link
Copy Markdown
Contributor

yeah I was just explaining my position on this. Also it should be noted why --socket=x11 is there too: #101

@flathubbot
Copy link
Copy Markdown
Contributor

Started test build 99804

@flathubbot
Copy link
Copy Markdown
Contributor

Build 99804 failed

@travier
travier commented Jul 9, 2022
Copy link
Copy Markdown
Member Author

Replaced by #129

@travier travier closed this Jul 9, 2022
@travier
travier deleted the readme branch July 9, 2022 23:00
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

0