Use /usr/bin/env instead of /usr/bin/perl5/perl

mharsch / arcstat

uses Perl to extract, format, and display kstats from the ZFS ARC

67 stars 31 forks source link

Use /usr/bin/env instead of /usr/bin/perl5/perl #9

Closed agxs closed 11 years ago

agxs commented 11 years ago

Update to use /usr/bin/env instead of a hardcoded /usr/perl5/bin/perl so the script will work unedited on Linux, and be more portable.

mharsch commented 11 years ago

This looks fine. Let's comment out the previous first line rather than remove it (as a hint in case someone on a solaris-derived system has multiple perls). Which platforms have you tested this on?

agxs commented 11 years ago

Tried on Ubuntu 12.04 LTS, Solaris 10 and Solaris 11. You want me to update the pull request, or do you want to to the comment in the merge?

mharsch commented 11 years ago

Merged. Thanks @agxs for the contribution.