Skip to content
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

Fix macOS build #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Hoishin
Copy link

@Hoishin Hoishin commented Apr 15, 2019

and added CI build on osx environment

Not sure if the CI build works or not yet

closes #13

@Hoishin
Copy link
Author

Hoishin commented Apr 15, 2019

Any reason why the travis build doesn't start?

@jleclanche
Copy link
Member

No idea, sorry :/

@Hoishin Hoishin force-pushed the fix-macos-build branch 8 times, most recently from 4e3c6a5 to 526548b Compare April 30, 2019 10:19
@beheh
Copy link
Member

beheh commented Apr 30, 2019

@Hoishin Take a look at Travis' Request Log. Looks like the YML is invalid: Could not parse HearthSim/decrunch/.travis.yml@599828925be4cd5e

@Hoishin
Copy link
Author

Hoishin commented Apr 30, 2019

@beheh I didn't know that page, thank you. Unfortunately that doesn't help without more detailed error, which I couldn't find in the log.

@Hoishin
Copy link
Author

Hoishin commented Apr 30, 2019

Turned out travis can't handle this syntax 😕

foo:
  - |
    foo
    bar

@Hoishin Hoishin force-pushed the fix-macos-build branch 3 times, most recently from 66840dc to 5ec4c9d Compare April 30, 2019 10:56
@beheh
Copy link
Member

beheh commented Apr 30, 2019

Yeah, the iteration loop isn't great for working on travis.yml files and it's always a bit painful. I think there's an offline tool you can use to validate the schema, if that's any help: https://support.travis-ci.com/hc/en-us/articles/115002904174-Validating-travis-yml-files

@Hoishin Hoishin force-pushed the fix-macos-build branch 3 times, most recently from dc1435d to 6e78a21 Compare April 30, 2019 17:50
@MegaByte
Copy link

MegaByte commented Sep 5, 2019

Any progress here?

@michaelsmoody
Copy link

May I ask, why use malloc.h on MACH vs stdlib?

@loeffel-io
Copy link

Lets go

@Kirkman
Copy link

Kirkman commented May 20, 2020

Any hope of getting this to compile on Mac?

@loeffel-io
Copy link

loeffel-io commented May 20, 2020

@Kirkman you can fix and build it locally for now

@stevenjoezhang
Copy link

It seems that #include <malloc.h> is not working on macOS. I changed it to #include <malloc/malloc.h> according to this Pull Request and it was successfully compiled.
See also #13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Problem installing on MacOS
8 participants