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 running is > > the launched a

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

2013-04-16 Thread Chris Down
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 running is > the launched application. > --- > dmenu_run | 2 +- > 1 file changed, 1 insertion(+

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

2013-04-16 Thread Truls Becken
On 2013-04-16, at 10:17, Ross Lagerwall wrote: > -dmenu_path | dmenu "$@" | ${SHELL:-"/bin/sh"} & > +exec "$(dmenu_path | dmenu "$@")" On 2013-04-16, at 15:47, Ross Lagerwall wrote: > -dmenu_path | dmenu "$@" | ${SHELL:-"/bin/sh"} & > +eval "exec "$(dmenu_path | dmenu "$@") To not screw up the

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. > > > > AFAICT, the pipeline outputs an

[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 Alexander Sedov
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. > > AFAICT, the pipeline outputs an executable with no other arguments. I mean, previously I could write 'gimp I

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 > > the launched application.

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

2013-04-16 Thread Alexander Sedov
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 > the launched application. > > In addition, it can now handle launching applications with spaces in >

[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