-
-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Spawned apps detaching from Clifm #262
Comments
Hi @ErinVoid. Edit Lira's config file (via X:^video/.*=xterm -e mpv %f & This instructs clifm to open video files in the background using As to the docs, thanks for the observation; I will take a look at it. Please feel free to edit it as you see it fits.
You're welcome! |
Thank for the feedback. The above doesn't work with Sakura which does spawn a new term but opens some random file from another folder completely with and without wrapping the %f in quotes. Trying st works correctly but complains "couldn't read from shell: I/O error" when exiting. What would be perfect would be a variable that could set to detach apps without the need to launch another terminal if possible either globally or on a per app basis. |
Tried with both sakura (3.8.7, AUR) and st (0.8.2) and they both work fine. Have you tried with another terminal, say xterm?
This is pretty weird, specially if it works fine when using st (which means that the file to be opened is properly passed to the opening application). Maybe some issue with your sakura version?
I'll bear this in mind. |
Thank you |
But the thing is still not working, isn't it? Could you provide more context info (OS, DE/WM, term, shell, clifm version)? I need to reproduce the issue. |
Alpine Linux 3.18 up to date (musl libc) In a folder; type "2" at the prompt, it recognises it being "Show S13E01.mp4", st opens correctly "/my/file/path/show name/Show S13E01" [st -e %f &] whereas Sakura immediately opens "/my/file/path/show/Show S00E01.mp4". It also occurs whatever file is chosen in the selected folder. Bit of debugging; the requested URI is ""/my/file/path/show name/Show" so the whole folder is enqueued. There are other posts about Sakura and it not coping with spaces. Wrapping the %f in "" makes no difference. st is fine! |
Hey @ErinVoid! I'll try to reproduce the issue, but it is good to know that it seems to be a Sakura bug. However, maybe we can think of some workaround. For the time being, I guess it's better not to rely on Sakura, at least for this specific use case. Btw, clifm current version is 1.15 (available in Alpine's Edge branch). |
Issue reproduced, and yes, the So, if there's nothing else to add regarding our original issue, I guess we can close it. |
If possible, change to a feature request for spawning apps and focus returning to Clifm? Happy the -e bug is Sakura's. |
What do you mean exactly? Isn't this request already fulfilled by my first comment? True, once an app is spawned in a different window (detached), the focus will change to it, but this is not controlled by clifm, but by the window manager, in which case there's nothing we can do as far as I know. |
The request was for Clifm to open GUI applications as a separate process to allow Clifm to continue to be used without opening additional terminal windows. |
When it comes to GUI apps, all you need to do is to specify the desired application. For example, if you want to open video files with X:^video/.*=vlc This will open X:^video/.*=vlc %f & In the case of terminal apps, like X:^video/.*=xterm -e mpv %f & If this is not what you want, please provide a more concrete example. |
@ErinVoid, is it ok to close this issue then? |
Other CLI FMs manage to open mpv without a terminal. Going to close anyway. |
You can try silencing both STDERR and STDOUT, as follows: X:^video/.*=mpv %f !EO & As an alternative, X:^video/.*=mpv --no-terminal %f & Please give it a shot and tell me if it's ok now. |
@ErinVoid, have you tried this? Does it work for you? |
@ErinVoid, does the above solution work as expected? |
Bug fixed upstream. |
I have this issue. I tried: |
@LinArcX , I cannot reproduce this issue (tried this rule with Openbox/i3 and alacritty/xterm/lxterminal, no issue). Could you provide more context information (clifm version, terminal, desktop environment/window manager)? |
OS: VoidLinux |
Tried with wezterm. Still nothing. The only way I've found to reproduce this issue (or at least something similar) is having the current terminal window maximized, in which case the video is certainly reproduced, but in a window hidden by the currently used one. Please try with a different terminal (to discard a terminal-specific issue) and with a different video application (to discard an mpv-specific issue). |
@LinArcX, could you try the latest git version (1.18.6)? We added a new value (
|
Is your feature request related to a problem? Please describe.
The documentation is not clear but would like to spawn an application, i.e. mpv, and the clifm instance return to the prompt rather than await the application quitting. Am using Lira as the opener.
Additional context
Clifm is good but not easy for mortals to understand from the documentation. Coming from nnn (as it doesn't [plugins especially nuke] play well with Alpine's standard Busybox install). Maybe the documentation could do with better structure and certainly be a little more consistent such as whenever a keypress is required, quote the necessary keys. However, getting into Clifm with previews for images so thank you.
The text was updated successfully, but these errors were encountered: