python utilities script suffix (.py) should be removed as per Policy 10.4
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
zfs-linux (Debian) |
Fix Released
|
Unknown
|
|||
zfs-linux (Ubuntu) |
Fix Released
|
Low
|
Eric Desrochers | ||
Xenial |
Fix Released
|
Low
|
Eric Desrochers |
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:/
"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/
/usr/
/usr/
[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/
/usr/sbin/arcstat
/usr/
Yakkety include the renaming as shown above.
Xenial will have a symlink to not break user experience until next release upgrade:
https:/
So user will be able to use both for instance : arcstat.py or arcstat
[Other Info]
* Justification/
https:/
https:/
* Debian bug
https:/
* Upstream (ZoL) is reluctant to change it at upstream code level.
https:/
* This has been flag by Adam Conrad
https:/
* Debian policy
https:/
Changed in zfs-linux (Ubuntu): | |
importance: | Undecided → Low |
Changed in zfs-linux (Ubuntu): | |
assignee: | nobody → Eric Desrochers (slashd) |
status: | New → Confirmed |
Changed in zfs-linux (Ubuntu): | |
importance: | Low → Medium |
Changed in zfs-linux (Ubuntu): | |
importance: | Medium → Low |
description: | updated |
description: | updated |
description: | updated |
Changed in zfs-linux (Ubuntu Xenial): | |
status: | New → In Progress |
importance: | Undecided → Low |
assignee: | nobody → Eric Desrochers (slashd) |
description: | updated |
description: | updated |
tags: | removed: sts-sru |
description: | updated |
tags: | removed: sts-sponsor |
Changed in zfs-linux (Debian): | |
status: | Unknown → Fix Committed |
Changed in zfs-linux (Debian): | |
status: | Fix Committed → Fix Released |
It seems like "dh-exec" might do the trick.
http:// manpages. ubuntu. com/manpages/ xenial/ man1/dh- exec-install. 1.html
Example : package/ start.conf
#! /usr/bin/dh-exec
debian/default.conf => /etc/my-