Fix bug inside recordRootFsSize

It worked when all mod for test script works inside chroot. But not anymore since "df /" now report the host disk space, not the rootfs.

[kmixter1, 2010-03-22 16:12:53]

LGTM

Thanks for fixing...

http://codereview.chromium.org/1130005/diff/1/2
File src/scripts/mod_for_test_scripts/000recordRootFsSize (right):

http://codereview.chromium.org/1130005/diff/1/2#newcode16
src/scripts/mod_for_test_scripts/000recordRootFsSize:16: df -B1 -P 2>/dev/null |
grep ${ROOT_FS_DIR} | awk '{ print $3 }' > "${FILE}"
Could df -B1 ${ROOT_FS_DIR} be used instead?  It's ok this way though.

[ericli, 2010-03-22 16:16:35]

I still need either a grep or tail after the df command any way, so I will
keep it as it is.



On Mon, Mar 22, 2010 at 9:12 AM, <kmixter@chromium.org> wrote:

> LGTM
>
> Thanks for fixing...
>
>
> http://codereview.chromium.org/1130005/diff/1/2
> File src/scripts/mod_for_test_scripts/000recordRootFsSize (right):
>
> http://codereview.chromium.org/1130005/diff/1/2#newcode16
> src/scripts/mod_for_test_scripts/000recordRootFsSize:16: df -B1 -P
>
> 2>/dev/null | grep ${ROOT_FS_DIR} | awk '{ print $3 }' > "${FILE}"
> Could df -B1 ${ROOT_FS_DIR} be used instead?  It's ok this way though.
>
>
> http://codereview.chromium.org/1130005
>



-- 
Eric Li
李咏竹
Google Kirkland

[sosa, 2010-04-01 16:39:43]

This is still open.  Did you mean to close it?

[ericli, 2010-04-01 16:54:42]

This change list had been pushed long time ago. I am closing it for now.

On 2010/04/01 16:39:43, sosa wrote:
> This is still open.  Did you mean to close it?