Header '/usr/include/CommonCrypto/CommonCrypto.h' not found

[iandundas]

Trying the change introduced by #7 (instead of the unmerged #2), I now get the following when compiling:

Is there possibly an extra step required to get this to work? (Not super sure what the path there refers to, as it doesn't exist on my system.)

[lstomberg]

This is broken for me as well but works on my coworkers computer. I do not think we can assume /usr/include/CommonCrypto exists. The previous implementation just assumed /Applications/Xcode.app exist, which is far more likely. I think Pull Request #7 needs to be reverted.

[brennanMKE]

Important details for Mojave which is a breaking change.

https://developer.apple.com/documentation/xcode_release_notes/xcode_10_release_notes#3035624

https://github.com/pyenv/pyenv/wiki#suggested-build-environment