Re: [hackers] [dmenu][PATCH] Replace dmenu_run shell with executed application

From: Jan Bessai <jan.bessai_AT_tu-dortmund.de>
Date: Sun, 30 Dec 2018 14:59:13 +0100

[2018-12-30 14:45] Silvan Jegen <s.jegen_AT_gmail.com>
> To keep this portable you should refrain from using backticks (`) and
> use the POSIX-way

Thanks! I've attached the updated patch below.

---
 dmenu_run | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dmenu_run b/dmenu_run
index 834ede5..1eff6af 100755
--- a/dmenu_run
+++ b/dmenu_run
_AT_@ -1,2 +1,2 @@
 #!/bin/sh
-dmenu_path | dmenu "$_AT_" | ${SHELL:-"/bin/sh"} &
+exec $(dmenu_path | dmenu "$_AT_")
-- 
2.20.1

Received on Sun Dec 30 2018 - 14:59:13 CET