Ticket #3574 (closed defect: fixed)
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:2 Changed 9 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 9 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.
comment:5 Changed 9 years ago by andrew_b
- Owner set to andrew_b
- Status changed from new to accepted
- Branch state changed from no branch to on review
- Milestone changed from Future Releases to 4.8.16
Branch: 3574_open_silent
changeset:9296f0bd6d56869a9b013a5475777a612a9ef5ac
comment:6 Changed 9 years ago by andrew_b
- Votes for changeset set to andrew_b
- Branch state changed from on review to approved
comment:7 Changed 9 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
Merged to master: [9296f0bd6d56869a9b013a5475777a612a9ef5ac].
Note: See
TracTickets for help on using
tickets.
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).