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

From: Ross Lagerwall <rosslagerwall_AT_gmail.com>
Date: Tue, 16 Apr 2013 11:30:44 +0100

On Tue, Apr 16, 2013 at 01:35:44PM +0400, Alexander Sedov wrote:
> 2013/4/16 Ross Lagerwall <rosslagerwall_AT_gmail.com>:
> > 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 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/dmenu_run b/dmenu_run
> > index 834ede5..191d4db 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_")"
> > --
> > 1.8.2.1
> >
> ...and cannot handle command-line arguments anymore.
>

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.

-- 
Ross Lagerwall
Received on Tue Apr 16 2013 - 12:30:44 CEST

This archive was generated by hypermail 2.3.0 : Tue Apr 16 2013 - 12:36:04 CEST