Bug 1094103
Summary: | tolua++ segfault while building stratagus (x86 only, x86_64 looks good) | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Filipe Rosset <rosset.filipe> |
Component: | tolua++ | Assignee: | Hans de Goede <hdegoede> |
Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | rawhide | CC: | hdegoede, tim |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2014-06-16 17:44:31 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 993385, 1107379 |
Description
Filipe Rosset
2014-05-05 05:36:30 UTC
Seems you are building with Lua 5.2. I haven't tried that here, I'm still using lua-compat for our stuff. Would that even be an option for stratagus? Hi All, Tim is right for tolua++ using projects you must use compat-lua51, tolua++ was never ported to 5.2, there is an upstream mail-thread about it here: http://lua-users.org/lists/lua-l/2013-01/msg00352.html Note that that is about a never finished port, so for now we're stuck with using compat-lua51. As for the tolua++ segfault itself, that sounds a lot like an arm issue I fixed a while back: * Fri Aug 09 2013 Hans de Goede <hdegoede> - 1.0.93-10 - Use the pre-generated toluabind.c on non x86, as something goes wrong when re-generating it (bootstrap) on non x86. This fixes tolua++ segfaulting on arm, and hopefully also on ppc (rhbz#704372) So maybe with a recent gcc / lua we just get bad code for 32 bit platforms when re-generating the toluabind.c file and we should stop doing that all together ? Regards, Hans Hi guys, I'm building with lua5.1 (tolua++ tolua++-devel and compat-lua-devel), but still reaching this issue. Hans, there are any chance to build tolua for x86 in the same way we're doing for arm? Regards, Filipe R. Hi, (In reply to Filipe Rosset from comment #3) > Hi guys, I'm building with lua5.1 (tolua++ tolua++-devel and > compat-lua-devel), but still reaching this issue. > > Hans, there are any chance to build tolua for x86 in the same way we're > doing for arm? If you can do a local build like that, and then confirm that it helps, yes that can be done, but first I would like to know if it actually helps :) Regards, Hans OK, I can confirm tolua++ works fine in x86 if build w/ this command: gcc -o bin/%{name} src/bin/tolua.o src/bin/toluabind_default.o -Llib -l%{solib} -llua-5.1 -ldl -lm To reproduce, I put some srpms here: http://filiperosset.fedorapeople.org/rawhide/stratagus/ http://filiperosset.fedorapeople.org/rawhide/stratagus/stratagus-2.2.7-3.fc21.src.rpm (segfault w/ currently tolua++ in rawhide x86, fortunately both arm and x86_64 looks fine) http://filiperosset.fedorapeople.org/rawhide/stratagus/tolua++-1.0.93-13.fc21.src.rpm (w/ this version, above stratagus rpm builds fine, without segfault when calling tolua++) I'm not sure about the counter side effects when building tolua w/ this option (arm arch are already build that way). It's almost done, needs a fix in tolua++ to build properly in x86 (arm + x86_64 are fine). ping I've kicked of a build building tolua++ with the pre-build byte-code object on x86 (32-bit), which should fix this: http://koji.fedoraproject.org/koji/taskinfo?taskID=7048282 Please try rebuild stratagus once this is in the buildroot, to wait for it to show up in the buildroot do: koji wait-repo --target=tolua++-1.0.93-12.fc21 f21-build If this new build fixes things please close this bug once stratagus has been build successfully. The new tolua++ is in the build root now, so I've started another attempt at building stratagus-2.2.7-3.fc21 now. The stratagus build has completed successfully, closing. |