On Tue, Jan 21, 2014 at 12:40:48PM +0100, Koen De Wit wrote: > +btrfs subvol delete $SUBVOL1 >/dev/null 2>&1 > +btrfs subvol delete $SUBVOL2 >/dev/null 2>&1
Please use $BTRFS_UTIL_PROG instead of 'btrfs' and don't shorten the command names, ie 'subvolume'. > +cp --reflink $TESTDIR1/file1 $SUBVOL1 > +cp --reflink $TESTDIR1/file1 $SUBVOL2 > +cp --reflink $SUBVOL1/file2 $TESTDIR1/ > +cp --reflink $SUBVOL1/file2 $SUBVOL2 > +cp --reflink $SUBVOL2/file3 $TESTDIR1/ > +cp --reflink $SUBVOL2/file3 $SUBVOL1 --reflink without any parameter means 'always', that's what we want, but can we possibly make it explicit? 'cp' is an external tool and if the default changes, the test would not work as expected. Otherwise ok, Reviewed-by: David Sterba <dste...@suse.cz> -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
