Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make zfs test easier to run in local install #4495

Closed
wants to merge 1 commit into from

Conversation

jxiong
Copy link
Contributor

@jxiong jxiong commented Apr 6, 2016

When ZFS is installed by 'make install', programs will be installed
into '/usr/local'. ZFS test scripts can't locate programs 'zpool'
that caused tests failure.

Signed-off-by: Jinshan Xiong [email protected]

ZINJECT=${ZINJECT:-$sbindir/zinject}
ZHACK=${ZHACK:-$sbindir/zhack}
ZPOOL=${ZPOOL:-$sbindir/zpool}
ZTEST=${ZTEST:-$sbindir/ztest}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps we could drop this block entirely if the sourced files reordered in default.cfg which also defines these.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

When ZFS is installed by 'make install', programs will be installed
into '/usr/local'. ZFS test scripts can't locate programs 'zpool'
that caused tests failure.

Signed-off-by: Jinshan Xiong <[email protected]>
@behlendorf
Copy link
Contributor

Thanks, LGTM.

ryao pushed a commit to ClusterHQ/zfs that referenced this pull request Jun 7, 2016
When ZFS is installed by 'make install', programs will be installed
into '/usr/local'. ZFS test scripts can't locate programs 'zpool'
that caused tests failure.

Fix typo in help message.

Add sanity check to for ksh and generate a useful error message.

Signed-off-by: Jinshan Xiong <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#4495
nedbass pushed a commit to nedbass/zfs that referenced this pull request Sep 3, 2016
When ZFS is installed by 'make install', programs will be installed
into '/usr/local'. ZFS test scripts can't locate programs 'zpool'
that caused tests failure.

Fix typo in help message.

Add sanity check to for ksh and generate a useful error message.

Signed-off-by: Jinshan Xiong <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes openzfs#4495
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants