[U-Boot] [PATCH] fs-test.sh: minor fixes

Suriyan Ramasami suriyan.r at gmail.com
Wed Aug 12 16:29:40 CEST 2015


Hi Stephen,

  Small nitpick, just to be consistent (for that particular line), we could
use OUT_FILE="${OUT}.${1}.${fs}.out"

Acked-by: Suriyan Ramasami <suriyan.r at gmail.com>

Best Regards

On Mon, Aug 10, 2015 at 9:45 PM, Stephen Warren <swarren at wwwdotorg.org>
wrote:

> - Re-direct stderr into the log files, so any errors U-Boot emits are
>   visible in the logs. This is relevant if the "reset" shell command
>   attempts to report that it's not supported on the sandbox board.
> - Fix test_fs_nonfs() to name the files it created differently for each
>   invocation. Otherwise, the logs from different tests overwrite
>   each-other.
>
> Signed-off-by: Stephen Warren <swarren at wwwdotorg.org>
> ---
>  test/fs/fs-test.sh | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/test/fs/fs-test.sh b/test/fs/fs-test.sh
> index a4fb0559625d..b88a67ca066d 100755
> --- a/test/fs/fs-test.sh
> +++ b/test/fs/fs-test.sh
> @@ -485,9 +485,9 @@ function test_fs_nonfs() {
>         echo "Creating files in $fs image if not already present."
>         create_files $IMAGE $MD5_FILE_FS
>
> -       OUT_FILE="${OUT}.fs.${fs}.out"
> +       OUT_FILE="${OUT}.$1.${fs}.out"
>         test_image $IMAGE $fs $SMALL_FILE $BIG_FILE $1 "" \
> -               > ${OUT_FILE}
> +               > ${OUT_FILE} 2>&1
>         check_results $OUT_FILE $MD5_FILE_FS $SMALL_FILE $BIG_FILE \
>                 $WRITE_FILE
>         TOTAL_FAIL=$((TOTAL_FAIL + FAIL))
> @@ -535,7 +535,7 @@ for fs in ext4 fat; do
>
>         OUT_FILE="${OUT}.sb.${fs}.out"
>         test_image $IMAGE $fs $SMALL_FILE $BIG_FILE sb `pwd`/$MOUNT_DIR \
> -               > ${OUT_FILE}
> +               > ${OUT_FILE} 2>&1
>         sudo umount "$MOUNT_DIR"
>         rmdir "$MOUNT_DIR"
>
> --
> 1.9.1
>
>


More information about the U-Boot mailing list