Re: [LTP] [PATCH: test_ioctl]

[CAI Q. <ca...@cc...>]

Hi,

--- Elder Costa <eld...@te...> wrote:

> test_ioctl produces a 'Syntax error: Bad substitution: 28' under
> Ubuntu 
> (and much probably Debian).
> 
> The following patch solves this issue. I tested it with sh (aka dash)
> 
> and bash.
> 
> Regards.
> 
> Elder.
> 
> Signed-Off-By: Elder Costa <eld...@te...>
> 
> 
> > --- testcases/kernel/syscalls/ioctl/test_ioctl.ORIG	2008-10-02
> 11:38:04.000000000 -0300
> +++ testcases/kernel/syscalls/ioctl/test_ioctl	2008-10-02
> 11:45:44.000000000 -0300
> @@ -25,7 +25,7 @@
>  
>  for tttype in `ls /dev/tty*`
>  do
> -device_no=${tttype:8}

This is bash extension ${parameter:offset}. See man bash.

> +device_no="${tttype}:8"

Changed it to the following does not make sense.

Cai Qian

>  case "$device_no" in
>  [0-9]|[0-9][0-9])
>      tst_resm TINFO "Testing ioctl01 with $tttype"
> @@ -43,7 +43,7 @@
>  
>  for tttype in `ls /dev/tty*`
>  do
> -device_no=${tttype:8}
> +device_no="${tttype}:8"
>  case "$device_no" in
>  [0-9]|[0-9][0-9])
>      tst_resm TINFO "Testing ioctl02 with $tttype"
> 
> 
> 
> >
-------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge
> Build the coolest Linux based applications with Moblin SDK & win
> great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the
> world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/>
_______________________________________________
> Ltp-list mailing list
> Ltp...@li...
> https://lists.sourceforge.net/lists/listinfo/ltp-list
>