Message ID | 1439268314-6980-1-git-send-email-swarren@wwwdotorg.org |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
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@gmail.com> Best Regards On Mon, Aug 10, 2015 at 9:45 PM, Stephen Warren <swarren@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@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 > >
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@gmail.com > <mailto:suriyan.r@gmail.com>> > > Best Regards > > On Mon, Aug 10, 2015 at 9:45 PM, Stephen Warren <swarren@wwwdotorg.org > <mailto:swarren@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@wwwdotorg.org > <mailto:swarren@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 > >
On Aug 12, 2015 8:08 AM, "Stephen Warren" <swarren@wwwdotorg.org> wrote: > > 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? > Now, that you point that out. I am OK with your version. Thanks! >> Acked-by: Suriyan Ramasami <suriyan.r@gmail.com >> <mailto:suriyan.r@gmail.com>> >> >> >> Best Regards >> >> On Mon, Aug 10, 2015 at 9:45 PM, Stephen Warren <swarren@wwwdotorg.org >> <mailto:swarren@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@wwwdotorg.org >> <mailto:swarren@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 >> >> >
On Mon, Aug 10, 2015 at 10:45:14PM -0600, Stephen Warren 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@wwwdotorg.org> > Acked-by: Suriyan Ramasami <suriyan.r@gmail.com> Applied to u-boot/master, thanks!
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"
- 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@wwwdotorg.org> --- test/fs/fs-test.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)