you can use 'print' instead of 'echo' - it is applicable for portability.
zfs-tests was developed on illumos based platform and for more
universal/portable way we can update it.
for example: with DilOS i have GNU coreutils in primary place instead of
illumos tools and i have the same issues what can found others non-illumos
based platforms.
i did some tricks to use illumos tools from /system/bin/ location for ZFS
tests, but right now my env with zfs tests is broken by latest big update by
delphix - where they wants to use easy way with PATH.
for my opinion - ZFS tests should be more universal for others platforms and we
have to use universal tools - not depend on illumos versions of tools.
more universal will be:
to use GNU coreutils - more platforms have it, but only illumos based platforms
have it at /usr/gnu/
to use GNU find - all platforms are using it
GNU SED, GREP/EGREP, DD, etc.
it is why it was more important to me to have environment variables file where
we can redefine tools locations if needed like:
export ECHO=/usr/gnu/bin/echo
and use ${ECHO} where we need it, instead of to use PATH with links to tools.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/124#issuecomment-336012316
------------------------------------------
openzfs-developer
Archives:
https://openzfs.topicbox.com/groups/developer/discussions/Ta17fb0edb29e7895-Mc5556a605f072b32898ae0f6
Powered by Topicbox: https://topicbox.com
