-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sha256sum not working on MacOS #11
Comments
Yeah... macOS does not ship GNU coreutils by default, you need to install them via brew or in some other way. That makes me wonder, which make implementation is shipped by default (if any)? Could you paste the output of I am hesitant to change installation instructions as you suggested, because calling Perl to calculate a hash sum instead of using coreutils feels deeply wrong to me. I think I'll add a note about coreutils and mention your workaround too. |
Output for
By the way, may I ask why not using coreutils is bad? |
Oh, it's not bad and it's not about coreutils per se. It just feels wrong to me to call into interpreted language for a number-crunching task when there exists a compiled binary tool for the same purpose. |
Personally, I don't think the performance loss is really significant, since the file Apologise if I ask a stupid question! |
Your question is not stupid :) I'm being somewhat irrational here, you're right about performance being similar for small files. I'm being too much of a GNU fanboy :) I'll look into how ubiquitous Perl currently is. I'll check if common Docker containers ship with Perl included, since it's the only scenario I could think of that's questionable. Then I will take a 1-2 day pause to let this sink in, but yeah, the rational thing would be to agree with you. |
My intuition proved correct. On the other hand, GNU coreutils are present in each of those images. I'll add a note about macOS to README now, but the instructions should stay unchanged. |
I'm using Big Sur 11.2.
Somehow,
sha256sum
is not pre-installed on my machine. So I have to change from:to:
Wonder if it should be updated...
The text was updated successfully, but these errors were encountered: