On Mon, Oct 28, 2024 at 01:45:12AM -0700, Christoph Hellwig wrote: > > + _try_scratch_mount >/dev/null 2>&1 \ > > + || _notrun "XFS doesn't support ascii-ci feature" > > I would have normally expected the || to be placed on the previous > line. Sure, I'll merge this patch as this: _try_scratch_mount >/dev/null 2>&1 || \ _notrun "XFS doesn't support ascii-ci feature" Thanks for your reviewing. > > Nitpicking aside: > > Reviewed-by: Christoph Hellwig <hch@xxxxxx> > >