Re: [dwm] [patch] spawn without shell

2008-06-12 Thread Henrik Holst
On Wed, Jun 11, 2008 at 12:00:02PM +0200, [EMAIL PROTECTED] wrote: Date: Wed, 11 Jun 2008 10:17:03 +0200 From: Anselm R. Garbe [EMAIL PROTECTED] Subject: Re: [dwm] [patch] spawn without shell To: dynamic window manager dwm@suckless.org Message-ID: [EMAIL PROTECTED] Content-Type: text/plain

Re: [dwm] [patch] spawn without shell

2008-06-12 Thread Anselm R. Garbe
On Thu, Jun 12, 2008 at 10:18:07AM +0200, Henrik Holst wrote: On Wed, Jun 11, 2008 at 12:00:02PM +0200, [EMAIL PROTECTED] wrote: Date: Wed, 11 Jun 2008 10:17:03 +0200 From: Anselm R. Garbe [EMAIL PROTECTED] Subject: Re: [dwm] [patch] spawn without shell To: dynamic window manager dwm

Re: [dwm] [patch] spawn without shell

2008-06-11 Thread Anselm R. Garbe
On Mon, Jun 09, 2008 at 02:57:30PM +0200, Peter Hartlich wrote: I've attached a patch against hg tip that makes spawn() use a NULL-terminated char pointer array for the argument, which is execvp'd directly instead of going through the shell. I consider integrating your patch as well before 5.0

Re: [dwm] [patch] spawn without shell

2008-06-11 Thread Anselm R. Garbe
On Wed, Jun 11, 2008 at 10:17:03AM +0200, Anselm R. Garbe wrote: On Mon, Jun 09, 2008 at 02:57:30PM +0200, Peter Hartlich wrote: I've attached a patch against hg tip that makes spawn() use a NULL-terminated char pointer array for the argument, which is execvp'd directly instead of going

Re: [dwm] [patch] spawn without shell

2008-06-09 Thread Anselm R. Garbe
On Mon, Jun 09, 2008 at 02:57:30PM +0200, Peter Hartlich wrote: I've attached a patch against hg tip that makes spawn() use a NULL-terminated char pointer array for the argument, which is execvp'd directly instead of going through the shell. The reason for going through the shell is simply

Re: [dwm] [patch] spawn without shell

2008-06-09 Thread Peter Hartlich
Hi Anselm, The reason for going through the shell is simply because users demanded to have their shell profile evaluated if a command is invoked. Ah, I thought is was a relic from the time when every argument was a char pointer. Though if it is really sh that is used (and not bash etc.),

Re: [dwm] [patch] spawn without shell

2008-06-09 Thread Szabolcs Nagy
On 6/9/08, Peter Hartlich [EMAIL PROTECTED] wrote: Though if it is really sh that is used (and not bash etc.), there no currently getenv(SHELL) is used

Re: [dwm] [patch] spawn without shell

2008-06-09 Thread Peter Hartlich
Hi, Though if it is really sh that is used (and not bash etc.), there no currently getenv(SHELL) is used Yes, I meant that there will be no profile loaded for users with SHELL unset or set to /bin/sh. My reason for wanting to remove the shell invocation, by the way, was that on this system,