Re: [PATCH v3] tests: _fail on _scratch_mkfs_sized failure

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On Tue, 30 Apr 2024 03:02:44 +0800, Zorro Lang wrote:

> On Mon, Apr 29, 2024 at 09:05:16AM -0700, Darrick J. Wong wrote:
...
> > > +_scratch_mkfs_sized()
> > > +{
> > > +	_try_scratch_mkfs_sized $* || _fail "_scratch_mkfs_sized (size=${1}) failed"  
> > 
> > I wonder if you ought to dump all the arguments, but the maintainer did
> > say to print the *size*. :)  
> 
> Sorry I didn't say that clearly, I mean the arguments related with size :-D

No worries :) I'm fine with it either way.

> 
> If both of you don't have more suggestions, I can help to change this line to:
> 
> _try_scratch_mkfs_sized $* || \
> 	_fail "_scratch_mkfs_sized failed with ($*)"
> 
> when I merge it.

Sure, works for me.

Cheers, David




[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux