flathub / org.opencpn.OpenCPN

https://flathub.org/apps/details/org.opencpn.OpenCPN
0 stars 5 forks source link

Change in build 127382 rejected #47

Closed flathubbot closed 2 months ago

flathubbot commented 2 months ago

A change in build 127382 has been reviewed by the Flathub team, and rejected for the following reason:

Please mention why host-etc:ro is required now. The commit has no information why this new permission is needed https://github.com/flathub/org.opencpn.OpenCPN/commit/bebce24e83c1dbb1f7d2bee2b3fae73a36c84f3f

Changes

Field Old value New value
filesystems ['home'] ['home', 'host-etc:ro']
leamas commented 2 months ago

This is basically about https://github.com/OpenCPN/OpenCPN/issues/4119.

I'm not able to disclose any details here. That said, access to /etc is required in order to make the same information which is present in the native package visible also in the sandbox to create a license key which works in both cases.

bbhtt commented 2 months ago

approved