github-starred/firejail

Fork 0

mirror of https://github.com/netblue30/firejail.git synced 2026-05-15 14:16:14 -06:00

[GH-ISSUE #676] Closing Xephyr window when exiting? #464

New issue

Closed

opened 2026-05-05 05:54:52 -06:00 by gitea-mirror · 4 comments

gitea-mirror commented

2026-05-05 05:54:52 -06:00

Owner

Originally created by @vn971 on GitHub (Aug 1, 2016).
Original GitHub issue: https://github.com/netblue30/firejail/issues/676

Should firejail close its Xephyr window when exiting? The current version of firejail does NOT close the window (for me).

Behavior in question:

start firejail with firejail --noprofile --x11=xephyr xterm
close xterm by typing Ctrl+D (exit)
check out firejail processes. There should be none corresponding to the test
observe the Xephyr window still active on the desktop, though it does not allocate anything in /tmp/.X11-unix anymore.

Originally created by @vn971 on GitHub (Aug 1, 2016). Original GitHub issue: https://github.com/netblue30/firejail/issues/676 Should firejail close its Xephyr window when exiting? The current version of firejail does NOT close the window (for me). Behavior in question: - start firejail with `firejail --noprofile --x11=xephyr xterm` - close xterm by typing Ctrl+D (`exit`) - check out firejail processes. There should be none corresponding to the test - observe the Xephyr window still active on the desktop, though it does not allocate anything in `/tmp/.X11-unix` anymore.

gitea-mirror closed this issue

2026-05-05 05:54:53 -06:00

gitea-mirror commented

2026-05-05 05:54:57 -06:00

Author

Owner

@manevich commented on GitHub (Aug 1, 2016):

-terminate option of Xephyr solves this problem.
But, for some reason -noreset (opposite of it) was added by default.
Here is quick fix.
I don't see negative consequences of this patch.
Please check it, and I will send pull request to upstream.

@manevich commented on GitHub (Aug 1, 2016): `-terminate` option of `Xephyr` solves this problem. But, for some reason `-noreset` (opposite of it) was added by default. [Here](https://github.com/manevich/firejail/commit/5ef59391e909ca4bdc1794b97bb9c829e30276e8) is quick fix. I don't see negative consequences of this patch. Please check it, and I will send pull request to upstream.

gitea-mirror commented

2026-05-05 05:54:59 -06:00

Author

Owner

@vn971 commented on GitHub (Aug 1, 2016):

BTW, since we already raised a question on Xephyr parameters. May it make sense to add a possibility to override firejail Xephyr parameters? Like xephyr-params-override -fullscreen -terminate in firejail.config.

@vn971 commented on GitHub (Aug 1, 2016): BTW, since we already raised a question on Xephyr parameters. May it make sense to add a possibility to override firejail Xephyr parameters? Like `xephyr-params-override -fullscreen -terminate` in `firejail.config`.

gitea-mirror commented

2026-05-05 05:55:02 -06:00

Author

Owner

@manevich commented on GitHub (Aug 1, 2016):

We can just change order of arguments to make xephyr-extra-params be last arg before display number, and they will override hardcoded. See 531c3401e4.

@manevich commented on GitHub (Aug 1, 2016): We can just change order of arguments to make `xephyr-extra-params` be last arg before display number, and they will override hardcoded. See https://github.com/manevich/firejail/commit/531c3401e4ead91118e33610c97128efdc786a53.

gitea-mirror commented

2026-05-05 05:55:04 -06:00

Author

Owner

@netblue30 commented on GitHub (Aug 2, 2016):

all merged in, thanks.

@netblue30 commented on GitHub (Aug 2, 2016): all merged in, thanks.

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/firejail#464

No description provided.

Rows
Columns