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

Support FreeBSD and fixed macOS build failure #10

Merged
merged 2 commits into from
Nov 10, 2020

Conversation

michael-yuji
Copy link

@michael-yuji michael-yuji commented Nov 9, 2020

This PR is based on #9 by @donileo contains patches to rebar.config.script such that it will be aware of the {"regex", "flag", "value"} style port_env configuration. This PR will unbreak compilation failure on macOS while have the FreeBSD support added.

@ioolkos
Copy link

ioolkos commented Nov 9, 2020

Thanks! Looks good to me, any remarks on MacOS @codeadict ?

@isadon
Copy link

isadon commented Nov 9, 2020

Thanks @michael-yuji, sorry I dropped off the radar after the initial PR :); got super busy with work. Seems like the remaining changes were Erlang code which I definitely wasn't going to be able to do in any short time frame..

@ioolkos
Copy link

ioolkos commented Nov 9, 2020

@donileo and thanks to you too! :)

@codeadict
Copy link

I'll compile tonight on OSX @ioolkos and will give my seal of approval

@codeadict
Copy link

codeadict commented Nov 10, 2020

All looks great on OSX @ioolkos @michael-yuji. Great stuff! :shipit:

@ioolkos
Copy link

ioolkos commented Nov 10, 2020

@codeadict thanks for testing! merging in a second.

@ioolkos ioolkos merged commit d9393e1 into vernemq:develop Nov 10, 2020
@ackratos
Copy link

ackratos commented Dec 1, 2020

Hi i am getting

===> Linking /Users/ackratos/Developer/vernemq/_build/default/lib/eleveldb/priv/eleveldb.so
make: *** [rel] Error 1

these error on macos 11.0.1 when i compile master branch of vernemq/vernemq, would this PR fix the issue and how can I workaround the problem?

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

Successfully merging this pull request may close these issues.

5 participants