All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] WebKit fixes
@ 2012-07-17 18:44 Ross Burton
  2012-07-17 18:44 ` [PATCH 1/2] webkit-gtk: remove no-parallel bodge Ross Burton
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Ross Burton @ 2012-07-17 18:44 UTC (permalink / raw)
  To: openembedded-core

Fix up a warning, and make compilation fast again.

This may cause failures when doing a parallel build due to a GNU Make bug,
but it's harder to replicate than originally thought (I can't replicate it
anymore).  If we get failures again hopefully we'll be able to work out what
the real cause is.

The following changes since commit 219a855f99bc45fd273f62df9507be7223eff1b4:

  useradd.bbclass: Fix for multilib builds (2012-07-17 10:56:26 +0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib ross/webkit

for you to fetch changes up to 66df0bb01e0162d51bb20b03d11157d42a057563:

  webkit-gtk: remove spurious install_prepend hack (2012-07-17 19:41:43 +0100)

----------------------------------------------------------------
Ross Burton (2):
      webkit-gtk: remove no-parallel bodge
      webkit-gtk: remove spurious install_prepend hack

 meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb |    9 ---------
 1 file changed, 9 deletions(-)

Ross Burton (2):
  webkit-gtk: remove no-parallel bodge
  webkit-gtk: remove spurious install_prepend hack

 meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb |    9 ---------
 1 file changed, 9 deletions(-)

-- 
1.7.10




^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] webkit-gtk: remove no-parallel bodge
  2012-07-17 18:44 [PATCH 0/2] WebKit fixes Ross Burton
@ 2012-07-17 18:44 ` Ross Burton
  2012-07-17 18:44 ` [PATCH 2/2] webkit-gtk: remove spurious install_prepend hack Ross Burton
  2012-07-19  7:43 ` [PATCH 0/2] WebKit fixes Saul Wold
  2 siblings, 0 replies; 4+ messages in thread
From: Ross Burton @ 2012-07-17 18:44 UTC (permalink / raw)
  To: openembedded-core

This workaround was originally for GNU Make 3.82 which upstream was blaming
for a parallel build failure.  Since then I've changed numerous variables
and now parallel builds are working with 3.82 (and succeed in 20 minutes
instead of 2 hours).

Revert the change until we've more data about what causes the breakage, then
we can bring it back with more finesse.

Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb |    5 -----
 1 file changed, 5 deletions(-)

diff --git a/meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb b/meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb
index b2bfd24..3cef0e4 100644
--- a/meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb
+++ b/meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb
@@ -29,11 +29,6 @@ inherit autotools lib_package pkgconfig
 
 S = "${WORKDIR}/webkit-${PV}/"
 
-# GNU Make 3.82 has a bug where it incorrectly prunes intermediate targets in
-# parallel builds.  See https://bugs.webkit.org/show_bug.cgi?id=79498 for the
-# WebKitGTK bug, and savannah.gnu.org/bugs/?30653 for the upstream bug.
-PARALLEL_MAKE = ""
-
 EXTRA_OECONF = "\
                 --enable-debug=no \
                 --enable-svg \
-- 
1.7.10




^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] webkit-gtk: remove spurious install_prepend hack
  2012-07-17 18:44 [PATCH 0/2] WebKit fixes Ross Burton
  2012-07-17 18:44 ` [PATCH 1/2] webkit-gtk: remove no-parallel bodge Ross Burton
@ 2012-07-17 18:44 ` Ross Burton
  2012-07-19  7:43 ` [PATCH 0/2] WebKit fixes Saul Wold
  2 siblings, 0 replies; 4+ messages in thread
From: Ross Burton @ 2012-07-17 18:44 UTC (permalink / raw)
  To: openembedded-core

This "fix" causes an (ignored) error now, so at some point it was fixed.

Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb |    4 ----
 1 file changed, 4 deletions(-)

diff --git a/meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb b/meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb
index 3cef0e4..180861e 100644
--- a/meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb
+++ b/meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb
@@ -78,10 +78,6 @@ do_configure_append() {
 	done
 }
 
-do_install_prepend() {
-	cp ${S}/Programs/.libs/jsc ${S}/Programs/jsc-1 || true
-}
-
 do_install_append() {
 	rmdir ${D}${libexecdir}
 }
-- 
1.7.10




^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 0/2] WebKit fixes
  2012-07-17 18:44 [PATCH 0/2] WebKit fixes Ross Burton
  2012-07-17 18:44 ` [PATCH 1/2] webkit-gtk: remove no-parallel bodge Ross Burton
  2012-07-17 18:44 ` [PATCH 2/2] webkit-gtk: remove spurious install_prepend hack Ross Burton
@ 2012-07-19  7:43 ` Saul Wold
  2 siblings, 0 replies; 4+ messages in thread
From: Saul Wold @ 2012-07-19  7:43 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 07/17/2012 11:44 AM, Ross Burton wrote:
> Fix up a warning, and make compilation fast again.
>
> This may cause failures when doing a parallel build due to a GNU Make bug,
> but it's harder to replicate than originally thought (I can't replicate it
> anymore).  If we get failures again hopefully we'll be able to work out what
> the real cause is.
>
> The following changes since commit 219a855f99bc45fd273f62df9507be7223eff1b4:
>
>    useradd.bbclass: Fix for multilib builds (2012-07-17 10:56:26 +0100)
>
> are available in the git repository at:
>
>    git://git.yoctoproject.org/poky-contrib ross/webkit
>
> for you to fetch changes up to 66df0bb01e0162d51bb20b03d11157d42a057563:
>
>    webkit-gtk: remove spurious install_prepend hack (2012-07-17 19:41:43 +0100)
>
> ----------------------------------------------------------------
> Ross Burton (2):
>        webkit-gtk: remove no-parallel bodge
>        webkit-gtk: remove spurious install_prepend hack
>
>   meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb |    9 ---------
>   1 file changed, 9 deletions(-)

> Ross Burton (2):
>    webkit-gtk: remove no-parallel bodge
>    webkit-gtk: remove spurious install_prepend hack
>
>   meta/recipes-sato/webkit/webkit-gtk_1.8.1.bb |    9 ---------
>   1 file changed, 9 deletions(-)
>

Merged into OE-Core

Thanks
	Sau!



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-07-19  7:54 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-17 18:44 [PATCH 0/2] WebKit fixes Ross Burton
2012-07-17 18:44 ` [PATCH 1/2] webkit-gtk: remove no-parallel bodge Ross Burton
2012-07-17 18:44 ` [PATCH 2/2] webkit-gtk: remove spurious install_prepend hack Ross Burton
2012-07-19  7:43 ` [PATCH 0/2] WebKit fixes Saul Wold

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.