Re: [dev] [st][PATCH] Add support for utmp in st

From: <k0ga_AT_shike2.com>
Date: Fri, 10 Oct 2014 16:14:00 +0200

>> - setenv("SHELL", sh, 1);
>> + setenv("SHELL", args[0], 1);
>
> I'm not sure whether this is a good idea? The user asked to execute
> args[0] instead of the shell, but this doesn't imply that args[0] is
> his shell.

Yes, you are right, it should be setenv("SHELL", sh, 1);

Can you send a patch for it?


Regards,
Received on Fri Oct 10 2014 - 16:14:00 CEST