On Wed, May 11, 2022 at 02:55:37PM +1000, Dave Chinner wrote: > On Tue, May 10, 2022 at 08:25:20PM -0400, Gabriel Krisman Bertazi wrote: > > seqres is not properly expanded on these error paths, causing a file > > called seqres.full to be created instead. > > > > Signed-off-by: Gabriel Krisman Bertazi <krisman@xxxxxxxxxxxxx> > > --- > > common/casefold | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/common/casefold b/common/casefold > > index 9172d818125a..7e8f941211db 100644 > > --- a/common/casefold > > +++ b/common/casefold > > @@ -25,7 +25,7 @@ _require_scratch_casefold() > > _notrun "$FSTYP does not support casefold feature" > > fi > > > > - if ! _scratch_mkfs_casefold &>>seqres.full; then > > + if ! _scratch_mkfs_casefold &>>$seqres.full; then By running `grep -rsn seqres|grep -v \$seqres`, I think this's only one file we missed '$' for 'seqres'. So this fix is good to me. And... > > PLease add a space between the &>> and $seqres.full while you are > fixing this. It's unreadable without proper whitespace separation... I'll merge your V2 patch, after you change it as Dave suggested above. Thanks, Zorro > > Cheers, > > Dave. > -- > Dave Chinner > david@xxxxxxxxxxxxx >
