simplesnapwrap not working on MacOS

[tomhoover]

1. When installing OpenZFS and the gnu utilities (gdate, ghead, ggrep & gsed) via Homebrew, all executables are placed in the /usr/local/bin directory. As the default PATH used when executing simplesnapwrap doesn't include the /usr/local/bin directory, none of the executables are found by the script.
2. The script doesn't use ghead (if available). The default head command in MacOS is not gnu compatible; therefore, the command head -n -1 returns an error.

Pull request is nearly complete, and should be uploaded tomorrow.

[clarkema]

I don't run ZFS on macos, so can't test directly, but this PR looks like it will only work on Intel macs. Homebrew uses /opt/homebrew on Apple silicon (see https://docs.brew.sh/FAQ#why-is-the-default-installation-prefix-opthomebrew-on-apple-silicon)

If you're running in a Homebrew-aware shell, you can check the environment variable $HOMEBREW_PREFIX to find the root of the Homebrew installation.

[tomhoover]

Good point. When I submitted this PR, there was no such thing as a Mac running Apple silicon. :-) @jgoerzen, although I no longer run simplesnapwrap, I'd be happy to make the appropriate change and test sometime this weekend.

…

-- Tom Hoover ***@***.***

On Tue, Jan 7, 2025, at 7:23 AM, Mike Clarke wrote: I don't run ZFS on macos, so can't test directly, but this PR looks like it will only work on Intel macs. Homebrew uses `/opt/homebrew` on Apple silicon (see https://docs.brew.sh/FAQ#why-is-the-default-installation-prefix-opthomebrew-on-apple-silicon) If you're running in a Homebrew-aware shell, you can check the environment variable `$HOMEBREW_PREFIX` to find the root of the Homebrew installation. — Reply to this email directly, view it on GitHub <#22 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABG2QCMJBRB6QKTDRSATKT2JPINZAVCNFSM6AAAAABUUEHLM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZVGI4DQMRRGI>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[tomhoover]

@clarkema , I just submitted #31 which works with both Apple Silicon and Intel.