Re: [PATCH 1/2] fstests: get rid of set_prog_path

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On Sun 03-06-18 22:21:17, Eryu Guan wrote:
> On Tue, May 29, 2018 at 06:51:18PM +0200, Jan Kara wrote:
> > -        export MKFS_XFS_PROG="`set_prog_path mkfs.xfs`"
> > -        export MKFS_EXT4_PROG="`set_prog_path mkfs.ext4`"
> > -        export MKFS_UDF_PROG="`set_prog_path mkudffs`"
> > -        export MKFS_BTRFS_PROG="`set_mkfs_prog_path_with_opts btrfs`"
> > -        export MKFS_F2FS_PROG="`set_mkfs_prog_path_with_opts f2fs`"
> 
> We should keep above two setups, set_mkfs_prog_path_with_opts does extra
> work than set_prog_path.
>
> > diff --git a/tests/xfs/446 b/tests/xfs/446
> > index 752c6a7de827..ac74723f471f 100755
> > --- a/tests/xfs/446
> > +++ b/tests/xfs/446
> > @@ -37,7 +37,6 @@ trap "rm -f $tmp.*; exit \$status" 0 1 2 3 15
> >  # real QA test starts here
> >  _supported_fs xfs
> >  _supported_os Linux
> > -export CHECKBASHISMS_PROG="`set_prog_path checkbashisms`"
> 
> This is removed but not added in common/config file.
> 
> I fixed both issues on commit.

Good spottings. Thanks for fixing them up!

								Honza
-- 
Jan Kara <jack@xxxxxxxx>
SUSE Labs, CR
--
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



[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux