On Thu, Oct 16, 2014 at 04:32:06PM +0800, Qu Wenruo wrote: > -------- Original Message -------- > Subject: Re: [PATCH] btrfs: regression test for btrfs extent merge > From: Eryu Guan <eguan@xxxxxxxxxx> > >>+ > >>+tmp=`mktemp -d` > >I'm a bit concerned about this. Some functions in common/rc use > >$tmp.some_sufix as temp file. If $tmp defined as a dir here, I'm not > >sure if it will cause any trouble. > It seems that most tests uses /tmp/$$ as tmp, only some exception > like btrfs/077 uses mktemp -d. > So I'd better use the old /tmp/$$ method. Can you please send a patch to fix btrfs/077 as well? Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
