I confirmed it works as expected :
root@test01:~# dpkg -l | grep -i zfsutils-linux
ii zfsutils-linux 0.6.5.6-0ubuntu14 amd64
Native OpenZFS management utilities for Linux
root@test01:~# cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.04
DISTRIB_CODENAME=xenial
DISTRIB_DESCRIPTION="Ubuntu 16.04 LTS"
root@test01:~# ls -altr /usr/sbin/arc*
-rwxr-xr-x 1 root root 40752 Oct 5 09:05 /usr/sbin/arc_summary.py
-rwxr-xr-x 1 root root 13058 Oct 5 09:05 /usr/sbin/arcstat.py
lrwxrwxrwx 1 root root 14 Oct 5 09:05 /usr/sbin/arc_summary ->
arc_summary.py
lrwxrwxrwx 1 root root 10 Oct 5 09:05 /usr/sbin/arcstat -> arcstat.py
root@test01:~# ls -altr /usr/sbin/dbufstat
lrwxrwxrwx 1 root root 11 Oct 5 09:05 /usr/sbin/dbufstat -> dbufstat.py
Eric
** Tags added: verification-done
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to zfs-linux in Ubuntu.
https://bugs.launchpad.net/bugs/1628279
Title:
python utilities script suffix (.py) should be removed as per Policy
10.4
Status in zfs-linux package in Ubuntu:
Fix Released
Status in zfs-linux source package in Xenial:
Fix Committed
Bug description:
[Impact]
The zfs utilities scripts shipped with debian and ubuntu such as :
* arc_summary.py : Provides a summary of the statistics
* arcstat.py : Print out ZFS ARC Statistics exported via kstat(1)
* dbufstat.py : Print out statistics for all cached dmu buffers
should be renamed according to Policy 10.4 :
https://www.debian.org/doc/debian-policy/ch-files.html#s-scripts
"When scripts are installed into a directory in the system PATH, the
script name should not include an extension such as .sh or .pl that
denotes the scripting language currently used to implement it."
Ideally, this should be fixed and upstreamed to Debian (or to zfslinux
upstream) as well.
[Test Case]
* Install zfsutils-linux
* List files
$ dpkg -L zfsutils-linux | egrep "arc|dbufstat"
/usr/sbin/arc_summary.py
/usr/sbin/arcstat.py
/usr/sbin/dbufstat.py
[Regression Potential]
* none expected, this is a trivial change that simply rename the
scripts to remove the extension (.py) with a simple 'mv' before the
dh_install invocation in order to get this expected result :
$ dpkg -L zfsutils-linux | egrep -i "arc|dbuf"
/usr/sbin/arc_summary
/usr/sbin/arcstat
/usr/sbin/dbufstat
Yakkety include the renaming as shown above.
Xenial will have a symlink to not break user experience until next release
upgrade:
https://bugs.launchpad.net/ubuntu/xenial/+source/zfs-linux/+bug/1628279/comments/19
So user will be able to use both for instance : arcstat.py or arcstat
[Other Info]
* Justification/argument for Xenial
https://bugs.launchpad.net/ubuntu/+source/zfs-linux/+bug/1628279/comments/13
https://bugs.launchpad.net/ubuntu/+source/zfs-linux/+bug/1628279/comments/14
* Debian bug
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=839645
* Upstream (ZoL) is reluctant to change it at upstream code level.
https://bugs.launchpad.net/ubuntu/+source/zfs-linux/+bug/1628279/comments/2
* This has been flag by Adam Conrad
https://bugs.launchpad.net/ubuntu/+source/zfs-linux/+bug/1574342/comments/19
* Debian policy
https://www.debian.org/doc/debian-policy/ch-files.html#s-scripts
To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/zfs-linux/+bug/1628279/+subscriptions
--
Mailing list: https://launchpad.net/~kernel-packages
Post to : [email protected]
Unsubscribe : https://launchpad.net/~kernel-packages
More help : https://help.launchpad.net/ListHelp
