Re: [PATCH] etc/ksh.kshrc - unify command substitution

2017-11-02 Thread Raf Czlonka
Hi all, So there's one OK - anyone else? Would anyone be so kind as to commit it, please? :^) Regards, Raf On Mon, Oct 23, 2017 at 06:32:03PM BST, Alexander Hall wrote: > I'm OK with this. > > /Alexander > > > On October 23, 2017 3:29:57 PM GMT+02:00, Raf Czlonka >

Re: [PATCH] etc/ksh.kshrc - unify command substitution

2017-10-23 Thread Alexander Hall
I'm OK with this. /Alexander On October 23, 2017 3:29:57 PM GMT+02:00, Raf Czlonka wrote: >What say you? > >On Tue, Aug 29, 2017 at 08:44:43PM BST, Raf Czlonka wrote: >> Ping. >> >> Anyone? >> >> On Sun, Jul 16, 2017 at 01:43:32PM BST, Raf Czlonka wrote: >> > Hi all, >>

Re: [PATCH] etc/ksh.kshrc - unify command substitution

2017-10-23 Thread Raf Czlonka
What say you? On Tue, Aug 29, 2017 at 08:44:43PM BST, Raf Czlonka wrote: > Ping. > > Anyone? > > On Sun, Jul 16, 2017 at 01:43:32PM BST, Raf Czlonka wrote: > > Hi all, > > > > Further simplification - 'ps | grep' can be replaced by pgrep(1) > > and if-then-fi by &&. > > > > > While there: > >

Re: [PATCH] etc/ksh.kshrc - unify command substitution

2017-08-29 Thread Raf Czlonka
Ping. Anyone? On Sun, Jul 16, 2017 at 01:43:32PM BST, Raf Czlonka wrote: > Hi all, > > Further simplification - 'ps | grep' can be replaced by pgrep(1) > and if-then-fi by &&. > > > While there: > > > > - remove ':' (null utility) from the very first line of the file - > > I *do* understand

Re: [PATCH] etc/ksh.kshrc - unify command substitution

2017-07-16 Thread Raf Czlonka
Hi all, Further simplification - 'ps | grep' can be replaced by pgrep(1) and if-then-fi by &&. > While there: > > - remove ':' (null utility) from the very first line of the file - > I *do* understand what it does but it doesn't seem like it's needed > at all, unless I'm missing something

Re: [PATCH] etc/ksh.kshrc - unify command substitution

2017-07-14 Thread Raf Czlonka
On Fri, Jul 07, 2017 at 05:47:46AM BST, Raf Czlonka wrote: > Hi all, > > I've noticed that etc/ksh.kshrc uses both types of command substitution > `command` and $(command). The below diff unifies it and uses > $(command) notation consistently. > > While there, [...] remove basename(1) invocation

Re: [PATCH] etc/ksh.kshrc - unify command substitution

2017-07-10 Thread Raf Czlonka
On Mon, Jul 10, 2017 at 06:48:25PM BST, Robert Peichaer wrote: > On Fri, Jul 07, 2017 at 05:47:46AM +0100, Raf Czlonka wrote: > > Hi all, > > > > I've noticed that etc/ksh.kshrc uses both types of command substitution > > `command` and $(command). The below diff unifies it and uses > > $(command)

Re: [PATCH] etc/ksh.kshrc - unify command substitution

2017-07-10 Thread Robert Peichaer
On Fri, Jul 07, 2017 at 05:47:46AM +0100, Raf Czlonka wrote: > Hi all, > > I've noticed that etc/ksh.kshrc uses both types of command substitution > `command` and $(command). The below diff unifies it and uses > $(command) notation consistently. > > While there: > > - remove ':' (null utility)