Opened 13 years ago

Closed 13 years ago

Last modified 8 years ago

#715 closed defect (fixed)

[PATCH] Bundled SpiderMonkey fails to build on 32bit userland on 64bit kernel (Linux)

Reported by: Matěj Laitl Owned by:
Priority: Should Have Milestone: Alpha 4
Component: Non-game systems Keywords: buildsystem
Cc: Patch:

Description

Due to some silly autodetection, Spidermonkey wants to compile itself as 64bit (64bit macros are set), but the compiler still builds it as 32bit, resulting in a build failue.

On such build log can be found at ​http://bugs.gentoo.org/attachment.cgi?id=244021&action=view (more info at ​http://bugs.gentoo.org/show_bug.cgi?id=278541#c37)

Attached patch adds support for (fairly standardised) CTARGET, CHOST and CBUILD environment variables to libraries/spidermonkey-tip/build.sh. This change alone fixes build in Gentoo, whose package manager sets these variables during build.

(the echo line in the patch is just debug, you may want to strip these (or not))

Attachments (1)

0ad-spidermonkey-respect-CHOST.patch​ (1002 bytes ) - added by Matěj Laitl 13 years ago.
0ad-spidermonkey-respect-CHOST.patch

Download all attachments as: .zip

Change History (5)

by Matěj Laitl, 13 years ago

Attachment: 0ad-spidermonkey-respect-CHOST.patch​ added

0ad-spidermonkey-respect-CHOST.patch

comment:1 by Erik Johansson, 13 years ago

Keywords: review added

comment:2 by philip, 13 years ago

Resolution: → fixed
Status: new → closed

(In [8858]) Fix #715 (respect CHOST/etc when building SpiderMonkey), based on patch from strohel

comment:3 by Kieran P, 13 years ago

Milestone: Backlog → Alpha 4

comment:4 by sanderd17, 8 years ago

Keywords: review removed
Note: See TracTickets for help on using tickets.