Openembedded Devel Discussions
 help / color / mirror / Atom feed
From: Gary Thomas <gary@mlbassoc.com>
To: Tom Rini <tom_rini@mentor.com>
Cc: openembedded-devel@lists.openembedded.org
Subject: Re: FireFox on PowerPC
Date: Tue, 06 Apr 2010 07:04:34 -0600	[thread overview]
Message-ID: <4BBB3162.80907@mlbassoc.com> (raw)
In-Reply-To: <4BB7EEE2.9060408@mlbassoc.com>

[-- Attachment #1: Type: text/plain, Size: 1614 bytes --]

On 04/03/2010 07:44 PM, Gary Thomas wrote:
> On 04/03/2010 05:11 PM, Tom Rini wrote:
>> On Sat, 2010-04-03 at 11:22 -0600, Gary Thomas wrote:
>>> On 04/03/2010 10:40 AM, Tom Rini wrote:
>>>> On Sat, 2010-04-03 at 09:28 -0600, Gary Thomas wrote:
>>>>
>>>>> That said, this seems to be a [Make] parallelism problem, not clear
>>>>> why/how.
>>>>> If I just rerun 'bitbake firefox' which restarts the compile step,
>>>>> it will
>>>>> run to completion. I do normally use -j4, but disabling this did
>>>>> not seem
>>>>> to make any difference.
>>>>>
>>>>> Note: the previous version I built (3.5.2) did not exhibit this
>>>>> behaviour.
>>>>
>>>> Well that's depressing since we just added the best we can tell correct
>>>> magic to build firefox with make -jN. If you put PARALLEL_MAKE = "" in
>>>> the .bb, does it complete this time?
>>>>
>>>
>>> No change, I'm afraid.
>>
>> Interesting. From log.do_compile, was make -jN taken out? I kinda fear
>> not...
>>
>
> I see only -j1 in the logs
>

I'm not getting anywhere with this.  True, it builds, but only
after a second attempt (the first always fails as reported).

Perhaps you'd like to try?  Just build firefox (3.6.2 is what
I'm using) for a PowerPC target (I use MACHINE=efika, DISTRO="angstrom-2008.1")
You'll need the attached patches to work around some of the va_list bugs.

Thanks

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------

[-- Attachment #2: 0001-Fixes-to-build-on-PowerPC.patch --]
[-- Type: text/plain, Size: 2084 bytes --]

From eedcc958ad89e17d4737f3948919b549335fd20e Mon Sep 17 00:00:00 2001
From: Gary Thomas <gary@mlbassoc.com>
Date: Tue, 6 Apr 2010 07:00:57 -0600
Subject: [PATCH] Fixes to build on PowerPC

---
 recipes/mozilla/files/powerpc/jsautocfg.h          |    2 ++
 .../mozilla/firefox-3.6.2/powerpc_va_list.patch    |   14 ++++++++++++++
 recipes/mozilla/firefox_3.6.2.bb                   |    1 +
 3 files changed, 17 insertions(+), 0 deletions(-)
 create mode 100644 recipes/mozilla/firefox-3.6.2/powerpc_va_list.patch

diff --git a/recipes/mozilla/files/powerpc/jsautocfg.h b/recipes/mozilla/files/powerpc/jsautocfg.h
index 408a5e4..bc5da80 100644
--- a/recipes/mozilla/files/powerpc/jsautocfg.h
+++ b/recipes/mozilla/files/powerpc/jsautocfg.h
@@ -51,4 +51,6 @@
 
 #define JS_HAVE_LONG_LONG
 
+#define HAVE_VA_LIST_AS_ARRAY
+
 #endif /* js_cpucfg___ */
diff --git a/recipes/mozilla/firefox-3.6.2/powerpc_va_list.patch b/recipes/mozilla/firefox-3.6.2/powerpc_va_list.patch
new file mode 100644
index 0000000..2ae56e4
--- /dev/null
+++ b/recipes/mozilla/firefox-3.6.2/powerpc_va_list.patch
@@ -0,0 +1,14 @@
+--- mozilla-1.9.1.ORIG/xpcom/glue/nsTextFormatter.cpp	2009-07-30 09:30:35.000000000 -0600
++++ mozilla-1.9.1/xpcom/glue/nsTextFormatter.cpp	2010-03-24 15:48:24.000000000 -0600
+@@ -70,7 +70,10 @@
+ ** Note: on some platforms va_list is defined as an array,
+ ** and requires array notation.
+ */
+-
++#if defined(__powerpc__)
++#define HAVE_VA_LIST_AS_ARRAY
++#endif
++
+ #ifdef HAVE_VA_COPY
+ #define VARARGS_ASSIGN(foo, bar)        VA_COPY(foo,bar)
+ #elif defined(HAVE_VA_LIST_AS_ARRAY)
diff --git a/recipes/mozilla/firefox_3.6.2.bb b/recipes/mozilla/firefox_3.6.2.bb
index 1c975e0..604ab82 100644
--- a/recipes/mozilla/firefox_3.6.2.bb
+++ b/recipes/mozilla/firefox_3.6.2.bb
@@ -14,6 +14,7 @@ SRC_URI = "ftp://ftp.mozilla.org/pub/mozilla.org/firefox/releases/${PV}/source/f
 	file://firefox-plugin.pc \
 	file://firefox-xpcom.pc \
 	file://nspr.pc \
+        file://powerpc_va_list.patch;patch=1 \
 "
 
 SRC_URI[archive.md5sum] = "a1972a2216ac7139b92b7378a328ec93"
-- 
1.6.2.5


  reply	other threads:[~2010-04-06 13:07 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-31 13:59 FireFox on PowerPC Gary Thomas
2010-03-31 14:58 ` Martyn Welch
2010-03-31 14:59   ` Gary Thomas
2010-03-31 17:06 ` Khem Raj
2010-03-31 17:13   ` Gary Thomas
2010-03-31 17:36     ` Tom Rini
2010-04-03 15:28       ` Gary Thomas
2010-04-03 16:40         ` Tom Rini
2010-04-03 17:22           ` Gary Thomas
2010-04-03 23:11             ` Tom Rini
2010-04-04  1:44               ` Gary Thomas
2010-04-06 13:04                 ` Gary Thomas [this message]
2010-04-07  2:52                   ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4BBB3162.80907@mlbassoc.com \
    --to=gary@mlbassoc.com \
    --cc=openembedded-devel@lists.openembedded.org \
    --cc=tom_rini@mentor.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox