On Wed, 2011-08-10 at 17:52 +0200, David Sterba wrote: > Hi, > > On Mon, Aug 01, 2011 at 12:31:19PM +0200, Stefan Behrens wrote: > > Use _scratch_mkfs / _scratch_mkfs_sized instead of _scratch_mkfs_xfs > > where possible. > > Execute 015, 062, 083, 117, 120 and 192 for all filesystems, these > > tests used to be XFS specific. > > this patch is now in xfstests-dev and somehow breaks btrfs testing. . . . > there's a hardcoded path for mkfs.btrfs in common.rc:_scratch_mkfs_sized() > > 335 btrfs) > 336 /sbin/mkfs.$FSTYP $MKFS_OPTIONS $SCRATCH_DEV -b $fssize > 337 ;; > > I have a /usr/local/ installation of btrfsprogs from git, > _scratch_mkfs_sized() was not called for btrfs before. > > Possible ways to fix this: > 1) use /sbin/mkfs -t $FSTYP like _scratch_mkfs, or > 2) set $MKFS_BTRFS_PROG like for some other filesystems > > > What is preferred? Whatever works. My personal preference would be option (1) if it works, since it's does the generic thing directly rather than suggesting something special is needed for btrfs. But if that doesn't work, consistency rules so MKFS_BTRFS_PROG "like for some other filesystems" is good. -Alex -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
