diff options
| author | duritong <peter.meier+github@immerda.ch> | 2013-08-20 01:43:06 -0700 |
|---|---|---|
| committer | duritong <peter.meier+github@immerda.ch> | 2013-08-20 01:43:06 -0700 |
| commit | f79c556357cdd4e0a06596b32661e3f156e2fc4f (patch) | |
| tree | 091f9a4a2738cac8f9b874f50c7bbaeff4583d40 | |
| parent | 04fad243b3585a6da4ba3d363ae352c2fc2dee91 (diff) | |
| parent | 41be4c9b05b95fc0326ec19ebae0fdffea151f7c (diff) | |
Merge pull request #16 from badgerious/which_fix
Use Facter::Util::Resolution.which instead of `which ...`
| -rw-r--r-- | lib/facter/acpi_available.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/facter/acpi_available.rb b/lib/facter/acpi_available.rb index 9a0474f..dae057f 100644 --- a/lib/facter/acpi_available.rb +++ b/lib/facter/acpi_available.rb @@ -1,7 +1,7 @@ # return whether acpi is available -- used for deciding whether to install the munin plugin Facter.add("acpi_available") do setcode do - if not File.exist? `which acpi 2>/dev/null`.chomp or `acpi -t -B -A 2>/dev/null`.match(/\d/).nil? + if not Facter::Util::Resolution.which('acpi') or `acpi -t -B -A 2>/dev/null`.match(/\d/).nil? "absent" else "present" |