Ticket #3574 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

MC_XDG_OPEN="/bin/false" doesn't work since 4.8.15

Reported by: asy Owned by: andrew_b
Priority: major Milestone: 4.8.16
Component: mc-core Version: 4.8.15
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description (last modified by andrew_b) (diff)

The change

-    "${MC_XDG_OPEN}" "${MC_EXT_FILENAME}" 2>/dev/null || \
+    ("${MC_XDG_OPEN}" "${MC_EXT_FILENAME}" >/dev/null 2>&1 &) || \

is broke to start do_open_action "${filetype}"

Change History

comment:1 follow-up: ↓ 3 Changed 4 years ago by az_ua

Last "&" in "2>&1 &)" is useless.
My fix:

sed 's#&) ||#) ||#' -i /usr/lib/mc/ext.d/*.sh

I'm using busybox ash (and busybox sed).

comment:2 Changed 4 years ago by andrew_b

  • Summary changed from MC_XDG_OPEN="/bin/false" is not works since 4.8.15 to MC_XDG_OPEN="/bin/false" doesn't work since 4.8.15

comment:3 in reply to: ↑ 1 Changed 4 years ago by asy

Replying to az_ua:

sed 's#&) ||#) ||#' -i /usr/lib/mc/ext.d/*.sh

I'm using busybox ash (and busybox sed).


It works for GNU bash 3.2.54 also.

Last edited 4 years ago by asy (previous) (diff)

comment:4 Changed 4 years ago by andrew_b

  • Description modified (diff)

comment:5 Changed 4 years ago by andrew_b

  • Status changed from new to accepted
  • Owner set to andrew_b
  • Branch state changed from no branch to on review
  • Milestone changed from Future Releases to 4.8.16

comment:6 Changed 4 years ago by andrew_b

  • Votes for changeset set to andrew_b
  • Branch state changed from on review to approved

comment:7 Changed 4 years ago by andrew_b

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b to committed-master
  • Resolution set to fixed
  • Branch state changed from approved to merged

comment:8 Changed 4 years ago by andrew_b

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.