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

Build failure on kFreeBSD #1044

Closed
jmallach opened this issue Jul 5, 2016 · 10 comments
Closed

Build failure on kFreeBSD #1044

jmallach opened this issue Jul 5, 2016 · 10 comments

Comments

@jmallach
Copy link
Contributor

jmallach commented Jul 5, 2016

Hello,

While building 0.175 on kFreeBSD, I encounter the following:

Compiling 3rdparty/bgfx/3rdparty/ocornut-imgui/imgui_draw.cpp...
../../../../../3rdparty/bgfx/3rdparty/ocornut-imgui/imgui_draw.cpp: In member function 'void ImDrawList::AddPolyline(const ImVec2*, int, ImU32, bool, float, bool)':
../../../../../3rdparty/bgfx/3rdparty/ocornut-imgui/imgui_draw.cpp:439:100: error: 'alloca' was not declared in this scope
 ormals = (ImVec2*)alloca(points_count * (thick_line ? 5 : 3) * sizeof(ImVec2));
                                                                              ^
../../../../../3rdparty/bgfx/3rdparty/ocornut-imgui/imgui_draw.cpp: In member function 'void ImDrawList::AddConvexPolyFilled(const ImVec2*, int, ImU32, bool)':
../../../../../3rdparty/bgfx/3rdparty/ocornut-imgui/imgui_draw.cpp:620:77: error: 'alloca' was not declared in this scope
         ImVec2* temp_normals = (ImVec2*)alloca(points_count * sizeof(ImVec2));
@ajrhacker
Copy link
Contributor

Is a1f9b62 enough to fix this?

@jmallach
Copy link
Contributor Author

Sorry, I didn't realise I commented on the actual commit, not issue. No, this commit does not fix it.

@h0tw1r3
Copy link
Contributor

h0tw1r3 commented Jul 28, 2016

What version of kFreeBSD?

@h0tw1r3
Copy link
Contributor

h0tw1r3 commented Jul 28, 2016

Following up. I was able to build mame HEAD on jessie-kFreeBSD with a number small build patches, mostly to uv.

@jmallach
Copy link
Contributor Author

With the fixes from #1093, and using system libuv, yes, the build succeeds, as can be seen here: https://buildd.debian.org/status/fetch.php?pkg=mame&arch=kfreebsd-amd64&ver=0.176-2&stamp=1469694187

This bug should have been closed a week ago.

@jmallach
Copy link
Contributor Author

These are Debian's libuv portability patches: https://sources.debian.net/src/libuv/0.10.36-3/debian/patches/ . It might be a good idea to merge a good part of these into 3rdparty.

@stilett0
Copy link
Contributor

so who wins the $20? 😃 (man, that was totally random...)

@h0tw1r3
Copy link
Contributor

h0tw1r3 commented Jul 28, 2016

@jmallach @mmicko UV upstream PR for GNU/kFreeBSD support libuv/libuv#959

@jmallach
Copy link
Contributor Author

TBH, if anyone deserves these $20, it's @stevenc99. But how on earth did these $20 appear out of nowhere?

angelosa pushed a commit that referenced this issue Sep 14, 2016
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

5 participants
@h0tw1r3 @jmallach @stilett0 @ajrhacker and others