macOS: endian.h include problem in crypto/os_int.h about axtls HOT 7 CLOSED

So, where's the patch?

from axtls.

The patch is delete the include.

    inreplace "lib/axtls/crypto/os_int.h", "#include <endian.h>", ""

from axtls.

(It should be restricted to darwin, of course)

from axtls.

@pfalcon Homebrew PR now refreshed: https://github.com/Homebrew/homebrew-core/pull/2607/files#diff-8ea455c1c60c376aa6f749bfba8dc8cbR23

from axtls.

(It should be restricted to darwin, of course)

So, there's productive way to resolve it: please submit pull request with change you tested to fix macosx build and not regress other builds.

Even more productive way is to contribute to micropython a .travis.yml which builds it for macosx (I heard TravisCI supports macosx builds).

Otherwise, it will wait for someone to decrypt/wade thru all that.

from axtls.

This really needs to be fixed by Cameron upstream in svn however he sees fit, and the rest will take care of itself.

from axtls.

@pfalcon upstream has fixed this

https://sourceforge.net/p/axtls/code/256/tree//trunk/crypto/os_int.h?diff=511c3f4934309d74f9522c0d:255

and has tagged a new version

https://sourceforge.net/p/axtls/code/259/tree/tags/release-1.5.4/

It would be great if you could refresh your fork with the latest.

Ah, I see it's in 9776795 but not the 1.8.2 release.

I heard TravisCI supports macosx builds

It does. Here's an example from Homebrew/science, which tests both macOS El Capitan (10.11) and Linux (for the sake of Linuxbrew):

https://github.com/Homebrew/homebrew-science/blob/master/.travis.yml

(Separately, they also test 10.11, 10.10, and 10.9 via Jenkins)

from axtls.