Landry, following up on a hint from billm, I got your patch to work on
Linux with this addition:

diff -r e7bbcbb6c24a js/src/jscntxt.h
--- a/js/src/jscntxt.h  Tue Mar 13 17:42:33 2012 -0700
+++ b/js/src/jscntxt.h  Tue Mar 13 18:32:46 2012 -0700
@@ -64,6 +64,7 @@
 #include "js/HashTable.h"
 #include "js/Vector.h"
 #include "vm/Stack.h"
+#include "assembler/jit/ExecutableAllocator.h"
 
 #ifdef _MSC_VER
 #pragma warning(push)


I'm not quite sure what's going on, but it kind of seems like 
ENABLE_ASSEMBLER=1 is not being set for all the files in the build, and so 
different object files get different definitions of JSRuntime, which causes the 
trc field to get overwritten with junk, and then you crash. Your patch does 
modify how ENABLE_ASSEMBLER is set, so it seems vaguely plausible that could be 
the problem.

The strange thing is that it seems like js.cpp is the file that doesn't
get ENABLE_ASSEMBLER=1, but I do see it being set on the command line to
build js.o. And it also doesn't make much sense that the patch above
would solve the problem.

Bug 731110 might just make things easy: if you don't have to make
execAlloc_ conditional on ENABLE_ASSEMBLER (because you'll have the
assembler), then you won't have to modify jscntxt.h and the Makefile.in,
and it should just work.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/908508

Title:
  Firefox/Thunderbird 10 FTBFS on powerpc in js/src/yarr/pcre

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/908508/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to