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

makefile build failed! #58

Open
johnlee175 opened this issue Jul 13, 2017 · 4 comments
Open

makefile build failed! #58

johnlee175 opened this issue Jul 13, 2017 · 4 comments
Assignees

Comments

@johnlee175
Copy link

cc -g3 -ggdb -Wall -Wstrict-prototypes -Wold-style-definition -Wundef -Wmissing-prototypes -Wmissing-declarations -Wpointer-arith -Wwrite-strings -DCCAN_STR_DEBUG=1 -I. -MMD -MP -MFccan/coroutine/coroutine.o.d -MTccan/coroutine/coroutine.o -c ccan/coroutine/coroutine.c -o ccan/coroutine/coroutine.o
ccan/coroutine/coroutine.c:63:2: error: array size is negative
BUILD_ASSERT(COROUTINE_MIN_STKSZ >= MINSIGSTKSZ);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
./ccan/build_assert/build_assert.h:23:27: note: expanded from macro 'BUILD_ASSERT'
do { (void) sizeof(char [1 - 2*!(cond)]); } while(0)
^~~~~~~~~~~~~
1 error generated.
make: *** [ccan/coroutine/coroutine.o] Error 1

@rustyrussell
Copy link
Owner

What platform?

@rustyrussell
Copy link
Owner

rustyrussell commented Jul 15, 2017 via email

@johnlee175
Copy link
Author

mac os x 10.12

@dgibson
Copy link
Collaborator

dgibson commented Aug 29, 2017

So, I've actually already fixed that particular problem with coroutine on MacOS. However, it still doesn't really work, because ucontext (which coroutine relies on) isn't supported on MacOS.

Fixing this isn't trivial - I need to work out a different way of doing coroutines that works for MacOS. Since I don't have access to a MacOS machine, and know very little about MacOS that's not particularly feasible. Patches welcome.

I would like to fix the build/test scripts so that modules not supported on that platform won't break the whole build. I just haven't had time to work it out though.

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

No branches or pull requests

3 participants