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

Stephen Warren swarren at wwwdotorg.org
Wed Aug 12 17:09:13 CEST 2015


On 08/12/2015 08:29 AM, Suriyan Ramasami wrote:
> Hi Stephen,
>
>    Small nitpick, just to be consistent (for that particular line), we
> could use OUT_FILE="${OUT}.${1}.${fs}.out"

I thought about that, but that would be inconsistent with plain $1 being 
used everywhere else:-) Which option do you prefer?

> Acked-by: Suriyan Ramasami <suriyan.r at gmail.com
> <mailto:suriyan.r at gmail.com>>
>
> Best Regards
>
> On Mon, Aug 10, 2015 at 9:45 PM, Stephen Warren <swarren at wwwdotorg.org
> <mailto: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
>     <mailto: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