Re: [PATCH 3/7] Test basic quota enforcement

[Jan Kara <jack@xxxxxxx>]

On Thu 20-05-10 11:11:30, Christoph Hellwig wrote:
> The golden output contains "/local/test1/file5" which appaears
> to be in your test fs.  You need to do something like
> 
>       sed -e "s,$SCRATCH_MNT,SCRATCH_MNT,g"
> 
> or
> 
>       sed -e "s,$T�T_MNT,TEST_MNT,g"
> 
> to make the test portable.  This probably applies to the other tests
> as well.
  I have filter_scratch function for this in all tests. I just forgot to
filter output of touch(1) through it. It should be fixed now.
  BTW: What is the purpose of 'status' variable? I set it to 1 if I want
the test to fail because diff found differences between accounted and
real usage. Is this correct? Actually, diff will also print the differences
to stdout so setting status probably shouldn't be needed. So is it preferred
to set status to 1 or leave it 0?

                                                                Honza
-- 
Jan Kara <jack@xxxxxxx>
SUSE Labs, CR