Comment 1 for bug 1574342

Revision history for this message
Richard Laager (rlaager) wrote :

I think it is safe to "assume" python for the purposes of "is this pulling in more code?". It's not safe to "assume" python for the purposes of the actual dependencies in debian/control. That is, python will have to be listed as a dependency, but I don't think adding this dependency to zfsutils-linux is a problem.

A few questions/comments:

1) Does arc_summary.py actually use these binaries, as listed in the comment at the top of the file? I couldn't find any direct calls to them.
# Binaries used are:
#
# dc(1), kldstat(8), sed(1), sysctl(8) & vmstat(8)

2) We need man pages for these commands. (Debian/Ubuntu Policy 12.1). I don't think upstream ships them, so we'll have to write them first.

3) The commands should have the .py extensions dropped, at least when they're installed in the PATH on Ubuntu. (Debian/Ubuntu policy 10.4.)

4) Upstream also has dbufstat. Is that worth shipping too?