video & images at cli (not in an X terminal)

[glennmcc]

pressing 'enter' on a video or image file does not play or view the file when outside of X but rather at a cli

In the older versions of mc (circa 2010 and before) it worked just fine.

In the newer versions of mc it only works when inside of X running mc within a terminal window.

My currently installed version is 4.8.29

[glennmcc]

Both of these now work perfectly both at the cli and inside of X

[glennmcc]

Same problem was happening with sound.sh

xdg-open works OK inside of X but not at the cli outside of X

At the cli, xdg-open is treating the file linked to within mc
just as-if it were a URL and hence sends it to the links or lynx web browser.

[andrew_b]

9	if [ -n "$DISPLAY" ]; then
10	([ -n "${MC_XDG_OPEN}" ] || MC_XDG_OPEN="xdg-open")
11	fi

Code in the parenthesis is run in the child shell and has no effect in the current one.

$ A=aaa
$ echo $A
aaa
$ (A=bbb)
$ echo $A
aaa
$ A=bbb 
$ echo $A
bbb

[glennmcc]

Unless that code is either commented-out all together
or simply only run when inside of X
the result is as I said... the file is sent to links or lynx
instead of to mplayer, or fim

xdg-open simply does not work correctly when running mc completely outside of X

It does work fine running mc in a terminal window _inside_ of X

So, do I simply need to remove the parenthesis from that line ?

[glennmcc]

Yep, that fixed it.

With the parenthesis inside of X mplayer was being used even though
xine was set as the default app for mp4 files.

Removed the parenthesis and now xine is used.

And since xdg-open is not coming into play at the cli outside of X
everything works as it should and images, videos and sound files
all get opened with the correct programs and are no-longer getting sent to links or lynx.

[zaytsev]

Reporter: can you test? Andrew: OK?

[glennmcc]

Replying to zaytsev:

Reporter: can you test? Andrew: OK?

Line #9 does not work but rather results only in...

/usr/libexec/mc/ext.d/image.sh: line 9: [: -n: binary operator expected

My original lines 9, 10 & 11 work correctly after removing the parenthesis.

[glennmcc]

All 3 scripts without parenthesis have been working perfectly for me for a week so have now been re-uploaded to here.

[zaytsev]

Branch: 4479_console_players
Initial changeset:572312bd882581fcc49b2d8e070a7f4e4179b34e

• Added the blocks to suppress the use of xdg-open to image, audio and video scripts.
• Added console image viewers.
• Removed error messages with sleep.

[andrew_b]

Why not use built-in command which is POSIX compatible:
​How can I check if a program exists from a Bash script?

[zaytsev]

Well, I didn't know about it. Thank you for the pointer, looks like a good idea. I have just replaced all occurrences that I could find.

[zaytsev]

Merged to master: 9e06a50eb8088d93d912ef643d79f761b8c4fb31