Issue #17794 has been updated by Josh Cooper. Status changed from Unreviewed to In Topic Branch Pending Review Branch set to https://github.com/puppetlabs/facter/pull/358
Marc, thanks for the contribution. Yeah, confining based on OS is silly, when it's the command we care about. ---------------------------------------- Bug #17794: zpool_version and zfs_version should not be restricted to only sun and bsd https://projects.puppetlabs.com/issues/17794#change-77540 Author: Marc Fournier Status: In Topic Branch Pending Review Priority: Normal Assignee: Category: library Target version: 1.7.0 Keywords: zfs_version zpool_version Branch: https://github.com/puppetlabs/facter/pull/358 Affected Facter version: development #13047 and #13049 added support for a couple more platforms to `$::zpool_version` and `$::zfs_version`. But nowadays, ZFS runs on more or less every unix platform. So it seems better to confine on the availability of the `zpool` and `zfs` commands in the `$PATH`, instead of maintaining a list of supported operating systems. This is exactly what has been done a couple of months ago to the zfs/zpool providers in puppet. Have a look at #9416 and https://github.com/puppetlabs/puppet/pull/1011 for the details. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
