All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.