Skip to content

Fixed negated permission string for allow and share run options - #6684

Merged
swick merged 1 commit into
flatpak:mainfrom
asbytes:fix-spawn
Jun 11, 2026
Merged

swick merged 1 commit into
flatpak:mainfrom
asbytes:fix-spawn

Conversation

@asbytes

@asbytes asbytes commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

It was solved partially in e0e1b20, but not for completely disallowed code path.
Steps to reproduce:

  1. Run any flatpak with explicit --unshare option.
    [user@user-standardpc ~]$ flatpak run --unshare=network --command=bash io.gitlab.librewolf-community
  2. Inside flatpak shell spawn any executable
    [📦 io.gitlab.librewolf-community ~]$ flatpak-spawn ls /
    error: Unknown option --noshare=network

@swick

swick commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Yikes. Could you put the PR description in the commit message please?

It was solved partially in e0e1b20, but not for completely disallowed code path.
Steps to reproduce:
1. Run any flatpak with explicit --unshare option, e.g:
    [user@user-standardpc ~]$ flatpak run --unshare=network --command=bash io.gitlab.librewolf-community
2. Inside flatpak shell spawn any executable:
    [io.gitlab.librewolf-community ~]$ flatpak-spawn ls /
Result is:
error: Unknown option --noshare=network
@asbytes

asbytes commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

Commit message updated.

@swick
swick added this pull request to the merge queue Jun 11, 2026
@swick

swick commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Thanks

Merged via the queue into flatpak:main with commit db70882 Jun 11, 2026
11 checks passed
@bbhtt

bbhtt commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

This would benefit from a small unit test to match the expected output. I may add one.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants