Re: [hackers] [dmenu][PATCH] Replace dmenu_run shell with executed application

From: Silvan Jegen <s.jegen_AT_gmail.com>
Date: Sun, 10 Feb 2019 10:13:39 +0000

On February 9, 2019 7:48:34 PM GMT+00:00, Nick <suckless-hackers_AT_njw.name> wrote:
>Quoth Hiltjo Posthuma:
>> On Sat, Feb 09, 2019 at 01:48:34PM +0100, Jan Bessai wrote:
>> > On Sun, Dec 30, 2018 at 02:59:13PM +0100, Jan Bessai wrote:
>> > > Thanks! I've attached the updated patch below.
>> >
>> > Sorry if I'm breaching any rules, but any update on
>accepting/rejecting
>> > the patch?
>> >
>>
>> Rejected
>
>Ignore if you're too busy, but why is this considered bad practise?
>Is there some case of possible shell escaping or something I'm
>failing to see? I just ask for my own education.

I would also be interested in the reason for the rejection. It seems sensible to me to not let all these shells linger around for no reason.


Cheers,

Silvan
Received on Sun Feb 10 2019 - 11:13:39 CET