Re: [dev] [PATCH v3] dmenu_run: Don't leave a shell running

2013-04-29 Thread Ross Lagerwall
au:~ $ cat >x2 > #!/bin/sh > eval exec "$(./x1)" > tg@blau:~ $ chmod +x x? > tg@blau:~ $ ./x2 > 48 65 6c 6c 6f 09 57 6f 72 6c 64 21 0a |Hello.World!.| > 000d > > So the tab is kept when you put Ross’ suggestion into double quotes. > Would the following work in all the cases that we care about? eval exec "$(dmenu_path | dmenu "$@")" -- Ross Lagerwall

Re: [dev] [PATCH v3] dmenu_run: Don't leave a shell running

2013-04-27 Thread Ross Lagerwall
On Sat, Apr 27, 2013 at 09:41:22PM +, Thorsten Glaser wrote: > Ross Lagerwall dixit: > > >+eval exec $(dmenu_path | dmenu "$@") > > You might need double-quotes around that (the eval argument). > Best to check with something containing a tab, e.g. > 「a\↹b &q

[dev] [PATCH v3] dmenu_run: Don't leave a shell running

2013-04-27 Thread Ross Lagerwall
The previous logic leaves a shell running for the duration that the launched application runs. This changes it so that the only application that is left running is the launched application. --- OK, so here is another attempt. It should hopefully behave like the current master branch (without the s

Re: [dev] [PATCH v2] dmenu_run: Don't leave a shell running

2013-04-16 Thread Ross Lagerwall
On Tue, Apr 16, 2013 at 10:31:24PM +0800, Chris Down wrote: > On 2013-04-16 14:47, Ross Lagerwall wrote: > > The previous logic leaves a shell running for the duration that the > > launched application runs. > > This changes it so that the only application that is left runnin

Re: [dev] [PATCH] dmenu_run: Don't leave a shell running

2013-04-16 Thread Ross Lagerwall
On Tue, Apr 16, 2013 at 04:15:16PM +0400, Alexander Sedov wrote: > 2013/4/16 Ross Lagerwall : > > How did it handle command-line arguments before? > > Command-line arguments are passed to dmenu as before. > > Running "dmenu_run -b" would pass the -b to dmenu. >

[dev] [PATCH v2] dmenu_run: Don't leave a shell running

2013-04-16 Thread Ross Lagerwall
The previous logic leaves a shell running for the duration that the launched application runs. This changes it so that the only application that is left running is the launched application. --- dmenu_run | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dmenu_run b/dmenu_run inde

Re: [dev] [PATCH] dmenu_run: Don't leave a shell running

2013-04-16 Thread Ross Lagerwall
On Tue, Apr 16, 2013 at 01:35:44PM +0400, Alexander Sedov wrote: > 2013/4/16 Ross Lagerwall : > > The previous logic leaves a shell running for the duration that the > > launched application runs. > > This changes it so that the only application that is left running is > &g

[dev] [PATCH] dmenu_run: Don't leave a shell running

2013-04-16 Thread Ross Lagerwall
The previous logic leaves a shell running for the duration that the launched application runs. This changes it so that the only application that is left running is the launched application. In addition, it can now handle launching applications with spaces in the filename. --- dmenu_run | 2 +- 1