Subject: Re: sed and pkg_rolling-replace on Solaris/SPARC
To: Tobias Nygren <tnn@NetBSD.org>
From: Mehul N. Sanghvi <mehul.sanghvi@gmail.com>
List: pkgsrc-users
Date: 06/29/2007 17:04:43
Tobias Nygren said the following on 6/29/2007 1:28 PM:
> On Fri, 29 Jun 2007 13:16:10 -0400
> "Mehul N. Sanghvi" <mehul.sanghvi@gmail.com> wrote:
> 
>> There was another issue also besides the fact that "-E" is not supported by all 
>> implementations of sed.
> 
> I rewrote the regex to not need -E, please try pkg_rolling-replace.sh
> rev 1.10.
> 
>> The script is a /bin/sh script, but uses constructs like $(...) instead of 
>> `...`, which is not supported by /bin/sh on Solaris, and I believe AIX and HPUX 
>> as well.
>>
>> I suspect that it works on NetBSD and probably on Linux as well since, at least 
>> on Debian Linux anyway, because /bin/sh is a hard link to /bin/bash or is a 
>> restricted version of bash which supports the $(...) construct.
>>
>> My suggestion is to either change the shell that is uses, or use the `...` 
>> construct to make the script portable.
> 
> Does the attached diff work for you?
> 
> -T

The patch and the update both seem to be working.

cheers,

     mehul

-- 
Mehul N. Sanghvi
email: mehul.sanghvi@gmail.com