On Wed, Feb 21, 2007 at 06:57:24PM +0000, Adam D. Barratt wrote:
> tags 409028 + patch
> thanks
>
> Hi,
>
> On Tue, 2007-01-30 at 11:52 +0000, martin f krafft wrote:
> > also sprach Alexander Gattin <[EMAIL PROTECTED]> [2007.01.30.1125 +0000]:
> > > OK, looks like only ${!P*} and [EMAIL PROTECTED] need to be
> > > considered by checkbashisms. Do you agree?
> >
> > Yes.
>
> Attached is a patch adding support for detecting the above forms to
> checkbashisms.
>
> Regards,
>
> Adam
Thanks!
Julian
> --- devscripts-2.9.27.orig/checkbashisms.pl 2006-11-29 18:39:43.000000000
> +0000
> +++ devscripts-2.9.27/checkbashisms.pl 2007-02-20 23:50:28.000000000
> +0000
> @@ -114,6 +114,8 @@
> '\s\|\&' => q<pipelining is not POSIX>,
> '\$\[\w+\]' => q<arithmetic not allowed>,
> '\$\{\w+\:\d+(?::\d+)?\}' => q<${foo:3[:1]}>,
> + '[EMAIL PROTECTED]' => q<${!prefix[*|@]>,
> + '\$\{!\w+\}' => q<${!name}>,
> '\$\{\w+(/.+?){1,2}\}' => q<${parm/?/pat[/str]}>,
> '[^\\\]\{([^\s]+?,)+[^\\\}\s]+\}' =>
> q<brace expansion>,
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
