Ticket #179 (closed defect: fixed)

Opened 2 years ago

Last modified 17 months ago

can't use rxvt, others as terminal emulators; terminal should use configured shell

Reported by: https://www.google.com/accounts/o8/id?id=aitoawkfizjo1vqnauqe0xsneuqi-v2wlwhuglm Owned by: -
Priority: major Milestone: 1
Keywords: Cc:

Description

In general, "x-terminal-emulator -e" expects an executable + arguments, NOT a shell command, as daemon/loader.c provides. The attached (if trivial) patch fixes that.

Even for xterm users, this patch has the benefit of telling xterm which shell to use (the one configured in ladish), as opposed to letting it decide on its own.

Attachments

1020-Fix-in-terminal-execution-for-non-xterm.patch Download (0.7 KB) - added by https://www.google.com/accounts/o8/id?id=aitoawkfizjo1vqnauqe0xsneuqi-v2wlwhuglm 2 years ago.

Change History

Changed 2 years ago by https://www.google.com/accounts/o8/id?id=aitoawkfizjo1vqnauqe0xsneuqi-v2wlwhuglm

Changed 2 years ago by http://nedko.arnaudov.name/

  • milestone changed from undecided to 0.4 (preview 4)

Changed 20 months ago by http://nedko.arnaudov.name/

  • status changed from new to closed
  • resolution set to fixed

Changed 17 months ago by http://nedko.arnaudov.name/

  • milestone changed from 0.4 (preview 4) to 1
Note: See TracTickets for help on using tickets.