[Pkg-shadow-devel] Re: [Pkg-shadow-commits] r397 - trunk/debian

[Alexander Gattin]

Hi!

On Sun, Jul 24, 2005 at 09:10:40AM +0200, Christian Perrier wrote:
> > Replaced `shadowconfig on/off >/dev/null` with `shadowconfig on/off` to ease
> > error diagnostics. Also, use `` instead of "$()".
> 
> What is the rationale of replacing $() by `` ?

Actually, to make uniform use of either `` or "$()"
throughout passwd.config

`` is old style of command substitution while $() is
newer one.

As we changed to /bin/sh, I decided that `` is more
appropriate (and also because vim shell syntax
plugin suggests so).

But $() is not less POSIX or whatever, except that
together with double quotes it's 3 chars longer.

> In several debian installer scripts I first created, Colin Watson
> replaced some `` by $(), so I kept thinking this is probably more
> portable or whatever. Am I wrong?

You pose an interesting question... I should consider
this deeper.

BTW I installed several POSIX-like shells (even busybox
ash ;)) specifically to test such corner cases, so
sooner or later we will have an answer.

-- 
WBR,
xrgtn