[PATCH 0/8] Subversion-1.7.0 and arm/thumb work arounds

[PATCH 0/8] Subversion-1.7.0 and arm/thumb work arounds

[Martin Jansa]

Hi,

arm mode was forced without PR bump, because if it was failing for someone he 
will notice.

Be carefull with using subversion-1.7.0 (read the commit message).

Pulseaudio-1.1 was released, so hopefully this work arounds won't be needed
there or they could be resolved better with upgrade.

Regards,

The following changes since commit 99da9a4e65f9dffb04efc3ad60125194c476d6b3:

  distro-tracking-fields: update fields for tzdata and gst-plugins-good (2011-10-20 13:07:16 +0100)

are available in the git repository at:
  git://git.openembedded.org/openembedded-core-contrib jansa/pull2
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=jansa/pull2

Martin Jansa (8):
  pulseaudio-0.9.23: inherit perlnative to work around build on host
    without XML/Parser.pm
  libatomics-ops: force ARM mode
  pulseaudio-0.9.23: force ARM mode
  webkit-gtk: force arm mode to work around binutils segfault
  apr: add native support
  neon: add native support
  apr-util: add native support
  subversion: add 1.7.0 with native support and negative D_P for now

 .../subversion/subversion-1.7.0/libtool2.patch     |   15 ++++++++
 .../subversion/subversion_1.7.0.bb                 |   37 ++++++++++++++++++++
 .../pulseaudio/libatomics-ops_1.2.bb               |    2 +
 .../pulseaudio/pulseaudio_0.9.23.bb                |    3 +-
 meta/recipes-sato/webkit/webkit-gtk_svn.bb         |   17 +++++++++
 meta/recipes-support/apr/apr-util_1.3.12.bb        |    8 ++++
 meta/recipes-support/apr/apr_1.4.5.bb              |    2 +
 meta/recipes-support/neon/neon_0.29.5.bb           |    3 ++
 8 files changed, 86 insertions(+), 1 deletions(-)
 create mode 100644 meta/recipes-devtools/subversion/subversion-1.7.0/libtool2.patch
 create mode 100644 meta/recipes-devtools/subversion/subversion_1.7.0.bb

-- 
1.7.7

[PATCH 1/8] pulseaudio-0.9.23: inherit perlnative to work around build on host without XML/Parser.pm

[Martin Jansa]

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 .../pulseaudio/pulseaudio_0.9.23.bb                |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
index 33f5e15..4ac2418 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
@@ -4,7 +4,7 @@ PR = "r5"
 
 DEPENDS += "gdbm speex"
 
-inherit gettext
+inherit gettext perlnative
 
 SRC_URI = "http://freedesktop.org/software/pulseaudio/releases/pulseaudio-${PV}.tar.gz \
   file://buildfix.patch \
-- 
1.7.7

[PATCH 2/8] libatomics-ops: force ARM mode

[Martin Jansa]

* otherwise ie spitz (armv5te) build fails with:
| make[3]: Entering directory `/OE/shr-core/tmp/work/armv5te-oe-linux-gnueabi/libatomics-ops-1.2-r5/libatomic_ops-1.2/src'
| arm-oe-linux-gnueabi-gcc  -march=armv5te  -mthumb -mthumb-interwork  -mtune=xscale --sysroot=/OE/shr-core/tmp/sysroots/spitz -DHAVE_CONFIG_H -I.    -fPIC -O
2 -pipe -g -feliminate-unused-debug-types -DNDEBUG -c atomic_ops.c
| arm-oe-linux-gnueabi-gcc  -march=armv5te  -mthumb -mthumb-interwork  -mtune=xscale --sysroot=/OE/shr-core/tmp/sysroots/spitz -DHAVE_CONFIG_H -I.    -fPIC -O
2 -pipe -g -feliminate-unused-debug-types -DNDEBUG -c atomic_ops_stack.c
| arm-oe-linux-gnueabi-gcc  -march=armv5te  -mthumb -mthumb-interwork  -mtune=xscale --sysroot=/OE/shr-core/tmp/sysroots/spitz -DHAVE_CONFIG_H -I.    -fPIC -O
2 -pipe -g -feliminate-unused-debug-types -DNDEBUG -c atomic_ops_malloc.c
| atomic_ops_malloc.c: In function 'msb':
| atomic_ops_malloc.c:223:2: warning: right shift count >= width of type [enabled by default]
| rm -f libatomic_ops_gpl.a
| ar cru libatomic_ops_gpl.a atomic_ops_stack.o atomic_ops_malloc.o
| arm-oe-linux-gnueabi-ranlib libatomic_ops_gpl.a
| {standard input}: Assembler messages:
| {standard input}:286: Error: selected processor does not support Thumb mode `swp r1,r2,[r3]'
| {standard input}:329: Error: selected processor does not support Thumb mode `swp r0,r1,[r3]'

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 .../pulseaudio/libatomics-ops_1.2.bb               |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-multimedia/pulseaudio/libatomics-ops_1.2.bb b/meta/recipes-multimedia/pulseaudio/libatomics-ops_1.2.bb
index 05b22e8..4d9ca0d 100644
--- a/meta/recipes-multimedia/pulseaudio/libatomics-ops_1.2.bb
+++ b/meta/recipes-multimedia/pulseaudio/libatomics-ops_1.2.bb
@@ -20,4 +20,6 @@ S = "${WORKDIR}/libatomic_ops-${PV}"
 
 ALLOW_EMPTY_${PN} = "1"
 
+ARM_INSTRUCTION_SET = "arm"
+
 inherit autotools pkgconfig
-- 
1.7.7

[PATCH 3/8] pulseaudio-0.9.23: force ARM mode

[Martin Jansa]

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 .../pulseaudio/pulseaudio_0.9.23.bb                |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
index 4ac2418..2f872b8 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
@@ -23,3 +23,4 @@ do_compile_prepend() {
     cp ${STAGING_LIBDIR}/libltdl* ${S}/libltdl
 }
 
+ARM_INSTRUCTION_SET = "arm"
-- 
1.7.7

[PATCH 4/8] webkit-gtk: force arm mode to work around binutils segfault

[Martin Jansa]

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-sato/webkit/webkit-gtk_svn.bb |   17 +++++++++++++++++
 1 files changed, 17 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-sato/webkit/webkit-gtk_svn.bb b/meta/recipes-sato/webkit/webkit-gtk_svn.bb
index 2862ad4..0a1f8b6 100644
--- a/meta/recipes-sato/webkit/webkit-gtk_svn.bb
+++ b/meta/recipes-sato/webkit/webkit-gtk_svn.bb
@@ -45,6 +45,23 @@ EXTRA_OECONF = "\
 
 EXTRA_AUTORECONF = " -I Source/autotools "
 
+
+#| ./Source/JavaScriptCore/heap/HandleTypes.h: In static member function 'static T* JSC::HandleTypes<T>::getFromSlot(JSC::HandleSlot) [with T = JSC::Structure, JSC::HandleTypes<T>::ExternalType = JSC::Structure*, JSC::HandleSlot = JSC::JSValue*]':
+#| ./Source/JavaScriptCore/heap/Handle.h:141:79:   instantiated from 'JSC::Handle<T>::ExternalType JSC::Handle<T>::get() const [with T = JSC::Structure, JSC::Handle<T>::ExternalType = JSC::Structure*]'
+#| ./Source/JavaScriptCore/runtime/ScopeChain.h:39:75:   instantiated from here
+#| ./Source/JavaScriptCore/heap/HandleTypes.h:38:130: warning: cast from 'JSC::JSCell*' to 'JSC::HandleTypes<JSC::Structure>::ExternalType {aka JSC::Structure*}' increases required alignment of target type [-Wcast-align]
+#| {standard input}: Assembler messages:
+#| {standard input}:28873: Error: invalid immediate: 983040 is out of range
+#| {standard input}:28873: Error: value of 983040 too large for field of 2 bytes at 15110
+#| /OE/shr-core/tmp/sysroots/x86_64-linux/usr/libexec/armv4t-oe-linux-gnueabi/gcc/arm-oe-linux-gnueabi/4.6.2/as: BFD (GNU Binutils) 2.21.1 assertion fail /OE/shr-core/tmp/work/armv4t-oe-linux-gnueabi/binutils-cross-2.21.1a-r0/binutils-2.21.1/bfd/elf.c:2819
+#| arm-oe-linux-gnueabi-g++: internal compiler error: Segmentation fault (program as)
+#| Please submit a full bug report,
+#| with preprocessed source if appropriate.
+#| See <http://gcc.gnu.org/bugs.html> for instructions.
+#| make[1]: *** [Source/JavaScriptCore/jit/libjavascriptcoregtk_1_0_la-JIT.lo] Error 1
+#| make[1]: Leaving directory `/OE/shr-core/tmp/work/armv4t-oe-linux-gnueabi/webkit-gtk-1.5.1+svnr90727-r0'
+ARM_INSTRUCTION_SET = "arm"
+
 CONFIGUREOPT_DEPTRACK = ""
 
 do_configure_append() {
-- 
1.7.7

[PATCH 5/8] apr: add native support

[Martin Jansa]

* needed for native subversion

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-support/apr/apr_1.4.5.bb |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-support/apr/apr_1.4.5.bb b/meta/recipes-support/apr/apr_1.4.5.bb
index 173402a..be465b4 100644
--- a/meta/recipes-support/apr/apr_1.4.5.bb
+++ b/meta/recipes-support/apr/apr_1.4.5.bb
@@ -6,6 +6,8 @@ LICENSE = "Apache-2.0"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=0c35ff3c4c83b89d2f076e315caac28b \
                     file://include/apr_lib.h;endline=17;md5=ee42fa7575dc40580a9e01c1b75fae96"
 
+BBCLASSEXTEND = "native"
+
 PR = "r1"
 
 SRC_URI = "${APACHE_MIRROR}/apr/${BPN}-${PV}.tar.bz2 \
-- 
1.7.7

[PATCH 6/8] neon: add native support

[Martin Jansa]

* needed for native subversion

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-support/neon/neon_0.29.5.bb |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-support/neon/neon_0.29.5.bb b/meta/recipes-support/neon/neon_0.29.5.bb
index 8201ffe..34c7253 100644
--- a/meta/recipes-support/neon/neon_0.29.5.bb
+++ b/meta/recipes-support/neon/neon_0.29.5.bb
@@ -5,9 +5,12 @@ LICENSE = "LGPLv2+"
 LIC_FILES_CHKSUM = "file://src/COPYING.LIB;md5=f30a9716ef3762e3467a2f62bf790f0a \
                     file://src/ne_utils.h;beginline=1;endline=20;md5=2caca609538eddaa6f6adf120a218037"
 DEPENDS = "zlib libxml2 expat time gnutls libproxy"
+DEPENDS_virtclass-native = "zlib-native libxml2-native expat-native gnutls-native"
 
 PR = "r1"
 
+BBCLASSEXTEND = "native"
+
 SRC_URI = "http://www.webdav.org/${BPN}/${BPN}-${PV}.tar.gz \
            file://pkgconfig.patch"
 
-- 
1.7.7

[PATCH 7/8] apr-util: add native support

[Martin Jansa]

* needed for native subversion
* do_configure_append_virtclass-native is needed, because apr_builddir points to
  /OE/shr-core/tmp/sysroots/x86_64-linux/usr/share/build-1
  and we're not installing libtool from apr there

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-support/apr/apr-util_1.3.12.bb |    8 ++++++++
 1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-support/apr/apr-util_1.3.12.bb b/meta/recipes-support/apr/apr-util_1.3.12.bb
index 800e67e..0064c51 100644
--- a/meta/recipes-support/apr/apr-util_1.3.12.bb
+++ b/meta/recipes-support/apr/apr-util_1.3.12.bb
@@ -3,6 +3,8 @@ HOMEPAGE = "http://apr.apache.org/"
 SECTION = "libs"
 DEPENDS = "apr expat gdbm"
 
+BBCLASSEXTEND = "native"
+
 LICENSE = "Apache-2.0"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=519e0a18e03f7c023070568c14b077bb \
                     file://include/apu_version.h;endline=17;md5=806685a84e71f10c80144c48eb35df42"
@@ -33,6 +35,12 @@ OE_BINCONFIG_EXTRA_MANGLE = " -e 's:location=source:location=installed:'"
 do_configure_prepend() {
 	cp ${STAGING_DATADIR}/apr/apr_rules.mk ${S}/build/rules.mk
 }
+do_configure_prepend_virtclass-native() {
+	cp ${STAGING_DATADIR_NATIVE}/apr/apr_rules.mk ${S}/build/rules.mk
+}
+do_configure_append_virtclass-native() {
+	sed -i "s#LIBTOOL=\$(SHELL) \$(apr_builddir)#LIBTOOL=\$(SHELL) ${STAGING_BINDIR_NATIVE}#" ${S}/build/rules.mk
+}
 
 FILES_${PN}     += "${libdir}/apr-util-1/apr_dbm_gdbm-1.so"
 FILES_${PN}-dev += "${libdir}/aprutil.exp ${libdir}/apr-util-1/apr_dbm_gdbm.*"
-- 
1.7.7

[PATCH 8/8] subversion: add 1.7.0 with native support and negative D_P for now

[Martin Jansa]

* be aware that checkouts from 1.7.0 are not compatible with older
  subversion clients (ie when builder populating distro PREMIRROR is
  using 1.7.0 all builders need to have also 1.7.0)
* and also 1.7.0 client needs to call svn upgrade in checkout first in
  order to use it (so if PREMIRROR has tarball from 1.6.x it won't work
  on client using 1.7.0 unless fetcher2 is improved to detect this and
  call svn upgrade)
* tested on SHR distribution
  http://wiki.shr-project.org/trac/wiki/Building%20SHR#subversion1.7inshr-chroot
* only missing part is to add subversion-native dependency, so that
  native subversion is built, before building ie elementary (because EFL
  are using svnversion from configure.ac to detect source revision and
  .svn dir needs to be from compatible version).
* read http://subversion.apache.org/docs/release-notes/1.7.html

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 .../subversion/subversion-1.7.0/libtool2.patch     |   15 ++++++++
 .../subversion/subversion_1.7.0.bb                 |   37 ++++++++++++++++++++
 2 files changed, 52 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-devtools/subversion/subversion-1.7.0/libtool2.patch
 create mode 100644 meta/recipes-devtools/subversion/subversion_1.7.0.bb

diff --git a/meta/recipes-devtools/subversion/subversion-1.7.0/libtool2.patch b/meta/recipes-devtools/subversion/subversion-1.7.0/libtool2.patch
new file mode 100644
index 0000000..5cd572b
--- /dev/null
+++ b/meta/recipes-devtools/subversion/subversion-1.7.0/libtool2.patch
@@ -0,0 +1,15 @@
+Upstream-Status: Inappropriate [embedded specific]
+
+--- a/configure.ac	2011-10-20 21:56:02.230663987 +0200
++++ b/configure.ac	2011-08-17 15:01:30.000000000 +0200
+@@ -227,8 +227,8 @@
+   LIBTOOL="$sh_libtool"
+   SVN_LIBTOOL="$sh_libtool"
+ else
+-  sh_libtool="$abs_builddir/libtool"
+-  SVN_LIBTOOL="\$(SHELL) $sh_libtool"
++  sh_libtool="$abs_builddir/$host_alias-libtool"
++  SVN_LIBTOOL="\$(SHELL) \$(abs_builddir)/$host_alias-libtool"
+ fi
+ AC_SUBST(SVN_LIBTOOL)
+ 
diff --git a/meta/recipes-devtools/subversion/subversion_1.7.0.bb b/meta/recipes-devtools/subversion/subversion_1.7.0.bb
new file mode 100644
index 0000000..396b35b
--- /dev/null
+++ b/meta/recipes-devtools/subversion/subversion_1.7.0.bb
@@ -0,0 +1,37 @@
+DESCRIPTION = "The Subversion (svn) client"
+SECTION = "console/network"
+DEPENDS = "apr-util neon"
+RDEPENDS_${PN} = "neon"
+LICENSE = "Apache-2"
+HOMEPAGE = "http://subversion.tigris.org"
+
+BBCLASSEXTEND = "native"
+
+# negative, because of new checkout format in 1.7.0 
+# and distro PREMIRRORs need to be in sync with users
+DEFAULT_PREFERENCE = "-1"
+
+SRC_URI = "${APACHE_MIRROR}/${BPN}/${BPN}-${PV}.tar.bz2 \
+           file://libtool2.patch \
+"
+SRC_URI[md5sum] = "930e6644a1b6094efd268fde6a318f04"
+SRC_URI[sha256sum] = "64fd5f263a80e609717a3ca42f1f2625606a5c4a40a85716f82c866033780978"
+
+LIC_FILES_CHKSUM = "file://LICENSE;md5=4a14fd2da3134e40a087eb4326a4ecd4"
+
+EXTRA_OECONF = " \
+                --without-berkeley-db --without-apxs --without-apache \
+                --without-swig --with-apr=${STAGING_BINDIR_CROSS} \
+                --with-apr-util=${STAGING_BINDIR_CROSS}"
+
+inherit autotools
+
+export LDFLAGS += " -L${STAGING_LIBDIR} "
+
+acpaths = "-I build/ -I build/ac-macros/"
+
+do_configure_prepend () {
+	rm -f ${S}/libtool
+	rm -f ${S}/build/libtool.m4
+	sed -i -e 's:with_sasl="/usr/local":with_sasl="${STAGING_DIR}":' ${S}/build/ac-macros/sasl.m4
+}
-- 
1.7.7

Re: [PATCH 3/8] pulseaudio-0.9.23: force ARM mode

[Koen Kooi]

Op 21 okt. 2011, om 10:17 heeft Martin Jansa het volgende geschreven:

> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
> .../pulseaudio/pulseaudio_0.9.23.bb                |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> index 4ac2418..2f872b8 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> @@ -23,3 +23,4 @@ do_compile_prepend() {
>     cp ${STAGING_LIBDIR}/libltdl* ${S}/libltdl
> }
> 
> +ARM_INSTRUCTION_SET = "arm"

Missing PR bump

Re: [PATCH 2/8] libatomics-ops: force ARM mode

[Henning Heinold]

Hi,

hm we should think of reworking this recipe now. Because since gcc 4.5
pulseaudio for arm can use the gcc internal atomicstuff and in oe-core 
and meta-oe we have 4.5 or 4.6 only. The lib is
only needed for mips and it is still the old release, on cvs
is a much better version, which supports thumb too, if
remember correctly.

Bye Henning

Re: [PATCH 2/8] libatomics-ops: force ARM mode

[Martin Jansa]

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

On Fri, Oct 21, 2011 at 10:43:46AM +0200, Henning Heinold wrote:
> Hi,
> 
> hm we should think of reworking this recipe now. Because since gcc 4.5
> pulseaudio for arm can use the gcc internal atomicstuff and in oe-core 
> and meta-oe we have 4.5 or 4.6 only. The lib is
> only needed for mips and it is still the old release, on cvs
> is a much better version, which supports thumb too, if
> remember correctly.

Agreed, I've added this only because pulseaudio is now needed for
gst-plugins and I wasn't able to build any armv[45]t images.

But because I don't know much about pulseaudio or libatomics-ops I was
hoping that someone who maintains them will fix it properly and this
work around will be needed only temporary.

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

[PATCH] base.bbclass: add subversion-native to DEPENDS if there is svn:// in SRC_URI

[Martin Jansa]

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/classes/base.bbclass |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index f539744..bced226 100644
--- a/meta/classes/base.bbclass
+++ b/meta/classes/base.bbclass
@@ -401,6 +401,13 @@ python () {
                     bb.note("SKIPPING %s because it's %s" % (pn, this_license))
                     raise bb.parse.SkipPackage("incompatible with license %s" % this_license)
 
+    # Svn packages should DEPEND on subversion-native
+    srcuri = bb.data.getVar('SRC_URI', d, 1)
+    if "svn://" in srcuri:
+        depends = bb.data.getVarFlag('do_fetch', 'depends', d) or ""
+        depends = depends + " subversion-native:do_populate_sysroot"
+        bb.data.setVarFlag('do_fetch', 'depends', depends, d)
+
     # Git packages should DEPEND on git-native
     srcuri = bb.data.getVar('SRC_URI', d, 1)
     if "git://" in srcuri:
-- 
1.7.7

Re: [PATCH] base.bbclass: add subversion-native to DEPENDS if there is svn:// in SRC_URI

[Martin Jansa]

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

On Fri, Oct 21, 2011 at 11:48:03AM +0200, Martin Jansa wrote:
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
>  meta/classes/base.bbclass |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
> index f539744..bced226 100644
> --- a/meta/classes/base.bbclass
> +++ b/meta/classes/base.bbclass
> @@ -401,6 +401,13 @@ python () {
>                      bb.note("SKIPPING %s because it's %s" % (pn, this_license))
>                      raise bb.parse.SkipPackage("incompatible with license %s" % this_license)
>  
> +    # Svn packages should DEPEND on subversion-native
> +    srcuri = bb.data.getVar('SRC_URI', d, 1)
> +    if "svn://" in srcuri:
> +        depends = bb.data.getVarFlag('do_fetch', 'depends', d) or ""
> +        depends = depends + " subversion-native:do_populate_sysroot"
> +        bb.data.setVarFlag('do_fetch', 'depends', depends, d)
> +
>      # Git packages should DEPEND on git-native
>      srcuri = bb.data.getVar('SRC_URI', d, 1)
>      if "git://" in srcuri:

For this to work correctly, we need to add BBCLASSEXTEND = "native" also
to subversion_1.6* otherwise everybody will get subversion-native-1.7.0
-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

[PATCH] subversion-1.6.15: add native support too

[Martin Jansa]

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 .../subversion/subversion_1.6.15.bb                |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-devtools/subversion/subversion_1.6.15.bb b/meta/recipes-devtools/subversion/subversion_1.6.15.bb
index 9a9484f..d3c7392 100644
--- a/meta/recipes-devtools/subversion/subversion_1.6.15.bb
+++ b/meta/recipes-devtools/subversion/subversion_1.6.15.bb
@@ -5,6 +5,8 @@ RDEPENDS_${PN} = "neon"
 LICENSE = "Apache-2"
 HOMEPAGE = "http://subversion.tigris.org"
 
+BBCLASSEXTEND = "native"
+
 PR = "r1"
 
 SRC_URI = "http://subversion.tigris.org/downloads/${BPN}-${PV}.tar.bz2 \
-- 
1.7.7

Re: [PATCH 2/8] libatomics-ops: force ARM mode

[Mark Hatle]

On 10/21/11 3:17 AM, Martin Jansa wrote:
> * otherwise ie spitz (armv5te) build fails with:
> | make[3]: Entering directory `/OE/shr-core/tmp/work/armv5te-oe-linux-gnueabi/libatomics-ops-1.2-r5/libatomic_ops-1.2/src'
> | arm-oe-linux-gnueabi-gcc  -march=armv5te  -mthumb -mthumb-interwork  -mtune=xscale --sysroot=/OE/shr-core/tmp/sysroots/spitz -DHAVE_CONFIG_H -I.    -fPIC -O
> 2 -pipe -g -feliminate-unused-debug-types -DNDEBUG -c atomic_ops.c
> | arm-oe-linux-gnueabi-gcc  -march=armv5te  -mthumb -mthumb-interwork  -mtune=xscale --sysroot=/OE/shr-core/tmp/sysroots/spitz -DHAVE_CONFIG_H -I.    -fPIC -O
> 2 -pipe -g -feliminate-unused-debug-types -DNDEBUG -c atomic_ops_stack.c
> | arm-oe-linux-gnueabi-gcc  -march=armv5te  -mthumb -mthumb-interwork  -mtune=xscale --sysroot=/OE/shr-core/tmp/sysroots/spitz -DHAVE_CONFIG_H -I.    -fPIC -O
> 2 -pipe -g -feliminate-unused-debug-types -DNDEBUG -c atomic_ops_malloc.c
> | atomic_ops_malloc.c: In function 'msb':
> | atomic_ops_malloc.c:223:2: warning: right shift count >= width of type [enabled by default]
> | rm -f libatomic_ops_gpl.a
> | ar cru libatomic_ops_gpl.a atomic_ops_stack.o atomic_ops_malloc.o
> | arm-oe-linux-gnueabi-ranlib libatomic_ops_gpl.a
> | {standard input}: Assembler messages:
> | {standard input}:286: Error: selected processor does not support Thumb mode `swp r1,r2,[r3]'
> | {standard input}:329: Error: selected processor does not support Thumb mode `swp r0,r1,[r3]'
> 
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
>  .../pulseaudio/libatomics-ops_1.2.bb               |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/meta/recipes-multimedia/pulseaudio/libatomics-ops_1.2.bb b/meta/recipes-multimedia/pulseaudio/libatomics-ops_1.2.bb
> index 05b22e8..4d9ca0d 100644
> --- a/meta/recipes-multimedia/pulseaudio/libatomics-ops_1.2.bb
> +++ b/meta/recipes-multimedia/pulseaudio/libatomics-ops_1.2.bb
> @@ -20,4 +20,6 @@ S = "${WORKDIR}/libatomic_ops-${PV}"
>  
>  ALLOW_EMPTY_${PN} = "1"
>  
> +ARM_INSTRUCTION_SET = "arm"
> +
>  inherit autotools pkgconfig

Wouldn't it be better to fix libatomic-ops to have proper thumb mode assembly,
then fall back to non-thumb assembly?

I swear I had a patch to do this in the past, but I'm currently not able to find it.

--Mark

Re: [PATCH 4/8] webkit-gtk: force arm mode to work around binutils segfault

[Khem Raj]

On Fri, Oct 21, 2011 at 1:17 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
>  meta/recipes-sato/webkit/webkit-gtk_svn.bb |   17 +++++++++++++++++
>  1 files changed, 17 insertions(+), 0 deletions(-)
>
> diff --git a/meta/recipes-sato/webkit/webkit-gtk_svn.bb b/meta/recipes-sato/webkit/webkit-gtk_svn.bb
> index 2862ad4..0a1f8b6 100644
> --- a/meta/recipes-sato/webkit/webkit-gtk_svn.bb
> +++ b/meta/recipes-sato/webkit/webkit-gtk_svn.bb
> @@ -45,6 +45,23 @@ EXTRA_OECONF = "\
>
>  EXTRA_AUTORECONF = " -I Source/autotools "
>
> +
> +#| ./Source/JavaScriptCore/heap/HandleTypes.h: In static member function 'static T* JSC::HandleTypes<T>::getFromSlot(JSC::HandleSlot) [with T = JSC::Structure, JSC::HandleTypes<T>::ExternalType = JSC::Structure*, JSC::HandleSlot = JSC::JSValue*]':
> +#| ./Source/JavaScriptCore/heap/Handle.h:141:79:   instantiated from 'JSC::Handle<T>::ExternalType JSC::Handle<T>::get() const [with T = JSC::Structure, JSC::Handle<T>::ExternalType = JSC::Structure*]'
> +#| ./Source/JavaScriptCore/runtime/ScopeChain.h:39:75:   instantiated from here
> +#| ./Source/JavaScriptCore/heap/HandleTypes.h:38:130: warning: cast from 'JSC::JSCell*' to 'JSC::HandleTypes<JSC::Structure>::ExternalType {aka JSC::Structure*}' increases required alignment of target type [-Wcast-align]
> +#| {standard input}: Assembler messages:
> +#| {standard input}:28873: Error: invalid immediate: 983040 is out of range
> +#| {standard input}:28873: Error: value of 983040 too large for field of 2 bytes at 15110
> +#| /OE/shr-core/tmp/sysroots/x86_64-linux/usr/libexec/armv4t-oe-linux-gnueabi/gcc/arm-oe-linux-gnueabi/4.6.2/as: BFD (GNU Binutils) 2.21.1 assertion fail /OE/shr-core/tmp/work/armv4t-oe-linux-gnueabi/binutils-cross-2.21.1a-r0/binutils-2.21.1/bfd/elf.c:2819
> +#| arm-oe-linux-gnueabi-g++: internal compiler error: Segmentation fault (program as)

What it seems is that tables are growing pretty large for thumb to
handle. Did you try to use -O1 or -Os with thumb mode ?

> +#| Please submit a full bug report,
> +#| with preprocessed source if appropriate.
> +#| See <http://gcc.gnu.org/bugs.html> for instructions.
> +#| make[1]: *** [Source/JavaScriptCore/jit/libjavascriptcoregtk_1_0_la-JIT.lo] Error 1
> +#| make[1]: Leaving directory `/OE/shr-core/tmp/work/armv4t-oe-linux-gnueabi/webkit-gtk-1.5.1+svnr90727-r0'
> +ARM_INSTRUCTION_SET = "arm"
> +
>  CONFIGUREOPT_DEPTRACK = ""
>
>  do_configure_append() {
> --
> 1.7.7
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

Re: [PATCH 3/8] pulseaudio-0.9.23: force ARM mode

[Khem Raj]

On Fri, Oct 21, 2011 at 1:17 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
>  .../pulseaudio/pulseaudio_0.9.23.bb                |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> index 4ac2418..2f872b8 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> @@ -23,3 +23,4 @@ do_compile_prepend() {
>     cp ${STAGING_LIBDIR}/libltdl* ${S}/libltdl
>  }
>
> +ARM_INSTRUCTION_SET = "arm"

not averse to patch but it would be nice to know what fails in thumb mode.

> --
> 1.7.7
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

Re: [PATCH 3/8] pulseaudio-0.9.23: force ARM mode

[Koen Kooi]

Op 21 okt. 2011, om 17:40 heeft Khem Raj het volgende geschreven:

> On Fri, Oct 21, 2011 at 1:17 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
>> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
>> ---
>>  .../pulseaudio/pulseaudio_0.9.23.bb                |    1 +
>>  1 files changed, 1 insertions(+), 0 deletions(-)
>> 
>> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
>> index 4ac2418..2f872b8 100644
>> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
>> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
>> @@ -23,3 +23,4 @@ do_compile_prepend() {
>>     cp ${STAGING_LIBDIR}/libltdl* ${S}/libltdl
>>  }
>> 
>> +ARM_INSTRUCTION_SET = "arm"
> 
> not averse to patch but it would be nice to know what fails in thumb mode.

And pulse is at 1.1 now, maybe upstream has fixed this issue.

Re: [PATCH 2/8] libatomics-ops: force ARM mode

[Khem Raj]

On Fri, Oct 21, 2011 at 1:48 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> On Fri, Oct 21, 2011 at 10:43:46AM +0200, Henning Heinold wrote:
>> Hi,
>>
>> hm we should think of reworking this recipe now. Because since gcc 4.5
>> pulseaudio for arm can use the gcc internal atomicstuff and in oe-core
>> and meta-oe we have 4.5 or 4.6 only. The lib is
>> only needed for mips and it is still the old release, on cvs
>> is a much better version, which supports thumb too, if
>> remember correctly.
>
> Agreed, I've added this only because pulseaudio is now needed for
> gst-plugins and I wasn't able to build any armv[45]t images.
>
> But because I don't know much about pulseaudio or libatomics-ops I was
> hoping that someone who maintains them will fix it properly and this
> work around will be needed only temporary.

Would you have cycles to create a git recipe for it the git tree is here
https://github.com/ivmai/libatomic_ops/

>
> --
> Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>

Re: [PATCH 3/8] pulseaudio-0.9.23: force ARM mode

[Martin Jansa]

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

On Fri, Oct 21, 2011 at 08:40:20AM -0700, Khem Raj wrote:
> On Fri, Oct 21, 2011 at 1:17 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> > ---
> >  .../pulseaudio/pulseaudio_0.9.23.bb                |    1 +
> >  1 files changed, 1 insertions(+), 0 deletions(-)
> >
> > diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > index 4ac2418..2f872b8 100644
> > --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > @@ -23,3 +23,4 @@ do_compile_prepend() {
> >     cp ${STAGING_LIBDIR}/libltdl* ${S}/libltdl
> >  }
> >
> > +ARM_INSTRUCTION_SET = "arm"
> 
> not averse to patch but it would be nice to know what fails in thumb mode.

selected processor does not support Thumb mode `swp r1,r2,[r3]'

Regards,

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH 2/8] libatomics-ops: force ARM mode

[Martin Jansa]

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

On Fri, Oct 21, 2011 at 09:06:34AM -0700, Khem Raj wrote:
> On Fri, Oct 21, 2011 at 1:48 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> > On Fri, Oct 21, 2011 at 10:43:46AM +0200, Henning Heinold wrote:
> >> Hi,
> >>
> >> hm we should think of reworking this recipe now. Because since gcc 4.5
> >> pulseaudio for arm can use the gcc internal atomicstuff and in oe-core
> >> and meta-oe we have 4.5 or 4.6 only. The lib is
> >> only needed for mips and it is still the old release, on cvs
> >> is a much better version, which supports thumb too, if
> >> remember correctly.
> >
> > Agreed, I've added this only because pulseaudio is now needed for
> > gst-plugins and I wasn't able to build any armv[45]t images.
> >
> > But because I don't know much about pulseaudio or libatomics-ops I was
> > hoping that someone who maintains them will fix it properly and this
> > work around will be needed only temporary.
> 
> Would you have cycles to create a git recipe for it the git tree is here
> https://github.com/ivmai/libatomic_ops/

Sorry, I'm not going to spend more of my free time on stuff I've never 
used on my phones at least as long as we have broken stuff which we're
using almost daily.

But I can keep this work around or .bbappend dropping whole pulseaudio
in SHR until someone fixes it properly or until I fix all other issues
we have.

Regards,

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH 3/8] pulseaudio-0.9.23: force ARM mode

[Saul Wold]

On 10/21/2011 09:16 AM, Martin Jansa wrote:
> On Fri, Oct 21, 2011 at 08:40:20AM -0700, Khem Raj wrote:
>> On Fri, Oct 21, 2011 at 1:17 AM, Martin Jansa<martin.jansa@gmail.com>  wrote:
>>> Signed-off-by: Martin Jansa<Martin.Jansa@gmail.com>
>>> ---
>>>   .../pulseaudio/pulseaudio_0.9.23.bb                |    1 +
>>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
>>> index 4ac2418..2f872b8 100644
>>> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
>>> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
>>> @@ -23,3 +23,4 @@ do_compile_prepend() {
>>>      cp ${STAGING_LIBDIR}/libltdl* ${S}/libltdl
>>>   }
>>>
>>> +ARM_INSTRUCTION_SET = "arm"
>>
>> not averse to patch but it would be nice to know what fails in thumb mode.
>
> selected processor does not support Thumb mode `swp r1,r2,[r3]'
>

As already asked, can you upgrade to 1.1 and verify if this problem 
still exists.  Also if it does, it would be good to have this info in 
the commit message.

Thanks
	Sau!


> Regards,
>
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Re: [PATCH 3/8] pulseaudio-0.9.23: force ARM mode

[Phil Blundell]

On Fri, 2011-10-21 at 18:16 +0200, Martin Jansa wrote:
> On Fri, Oct 21, 2011 at 08:40:20AM -0700, Khem Raj wrote:
> > On Fri, Oct 21, 2011 at 1:17 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> > > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> > > ---
> > >  .../pulseaudio/pulseaudio_0.9.23.bb                |    1 +
> > >  1 files changed, 1 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > > index 4ac2418..2f872b8 100644
> > > --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > > +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > > @@ -23,3 +23,4 @@ do_compile_prepend() {
> > >     cp ${STAGING_LIBDIR}/libltdl* ${S}/libltdl
> > >  }
> > >
> > > +ARM_INSTRUCTION_SET = "arm"
> > 
> > not averse to patch but it would be nice to know what fails in thumb mode.
> 
> selected processor does not support Thumb mode `swp r1,r2,[r3]'

If it's only that one error (presumably some kind of home-grown atomic
operation) then let's just fix the code in question rather than forcing
the whole thing to build in ARM-state.  Or if the code is hard to fix,
patch the makefile so that only the particular source files with the
assembly in are built for ARM and let the rest stay as Thumb.  Making
the whole package be ARM is a bit of a heavyweight fix for that issue.

p.

Re: [PATCH] base.bbclass: add subversion-native to DEPENDS if there is svn:// in SRC_URI

[Saul Wold]

On 10/21/2011 02:48 AM, Martin Jansa wrote:
> Signed-off-by: Martin Jansa<Martin.Jansa@gmail.com>
> ---
>   meta/classes/base.bbclass |    7 +++++++
>   1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
> index f539744..bced226 100644
> --- a/meta/classes/base.bbclass
> +++ b/meta/classes/base.bbclass
> @@ -401,6 +401,13 @@ python () {
>                       bb.note("SKIPPING %s because it's %s" % (pn, this_license))
>                       raise bb.parse.SkipPackage("incompatible with license %s" % this_license)
>
> +    # Svn packages should DEPEND on subversion-native
> +    srcuri = bb.data.getVar('SRC_URI', d, 1)
> +    if "svn://" in srcuri:
> +        depends = bb.data.getVarFlag('do_fetch', 'depends', d) or ""
> +        depends = depends + " subversion-native:do_populate_sysroot"
> +        bb.data.setVarFlag('do_fetch', 'depends', depends, d)
> +
>       # Git packages should DEPEND on git-native
>       srcuri = bb.data.getVar('SRC_URI', d, 1)
>       if "git://" in srcuri:
This change introduces a circular dependency when I tried to build.

Sau!

Re: [PATCH 3/8] pulseaudio-0.9.23: force ARM mode

[Martin Jansa]

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

On Sat, Oct 22, 2011 at 01:33:46PM +0100, Phil Blundell wrote:
> On Fri, 2011-10-21 at 18:16 +0200, Martin Jansa wrote:
> > On Fri, Oct 21, 2011 at 08:40:20AM -0700, Khem Raj wrote:
> > > On Fri, Oct 21, 2011 at 1:17 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> > > > Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> > > > ---
> > > >  .../pulseaudio/pulseaudio_0.9.23.bb                |    1 +
> > > >  1 files changed, 1 insertions(+), 0 deletions(-)
> > > >
> > > > diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > > > index 4ac2418..2f872b8 100644
> > > > --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > > > +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_0.9.23.bb
> > > > @@ -23,3 +23,4 @@ do_compile_prepend() {
> > > >     cp ${STAGING_LIBDIR}/libltdl* ${S}/libltdl
> > > >  }
> > > >
> > > > +ARM_INSTRUCTION_SET = "arm"
> > > 
> > > not averse to patch but it would be nice to know what fails in thumb mode.
> > 
> > selected processor does not support Thumb mode `swp r1,r2,[r3]'
> 
> If it's only that one error (presumably some kind of home-grown atomic
> operation) then let's just fix the code in question rather than forcing
> the whole thing to build in ARM-state.  Or if the code is hard to fix,
> patch the makefile so that only the particular source files with the
> assembly in are built for ARM and let the rest stay as Thumb.  Making
> the whole package be ARM is a bit of a heavyweight fix for that issue.

That's why I've called it work around in cover, because I think that fixing it in
old 0.9.23 is not worth it, we can try to fix it in 1.1 when we have it.

And it's not the only one error, I've updated commit message with bigger
chunk of log in 
http://git.openembedded.org/openembedded-core-contrib/log/?h=jansa/pull

and for om-gta02 (armv4t) it's about
selected processor does not support Thumb mode `mla r3,r7,r0,r3'

while swp was from spitz (armv5te).

Regards,

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH 3/8] pulseaudio-0.9.23: force ARM mode

[Phil Blundell]

On Tue, 2011-10-25 at 09:26 +0200, Martin Jansa wrote:
> That's why I've called it work around in cover, because I think that fixing it in
> old 0.9.23 is not worth it, we can try to fix it in 1.1 when we have it.

Fair enough, though as a general rule I don't think the cover letter is
the right place for this sort of information.  Each patch that you send
ought really to be able to stand on its own merits and whatever
explanation is necessary ought to be in its own checkin message. 

In particular, the cover letter does not form part of the permanent
record in the git history, and it is liable to get lost if your patch is
incorporated into any kind of consolidated pull. 

p.

Re: [PATCH 0/8] Subversion-1.7.0 and arm/thumb work arounds

[Saul Wold]

On 10/21/2011 10:17 AM, Martin Jansa wrote:
> Hi,
>
> arm mode was forced without PR bump, because if it was failing for someone he
> will notice.
>
> Be carefull with using subversion-1.7.0 (read the commit message).
>
> Pulseaudio-1.1 was released, so hopefully this work arounds won't be needed
> there or they could be resolved better with upgrade.
>
> Regards,
>
> The following changes since commit 99da9a4e65f9dffb04efc3ad60125194c476d6b3:
>
>    distro-tracking-fields: update fields for tzdata and gst-plugins-good (2011-10-20 13:07:16 +0100)
>
> are available in the git repository at:
>    git://git.openembedded.org/openembedded-core-contrib jansa/pull2
>    http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=jansa/pull2
>
> Martin Jansa (8):
>    pulseaudio-0.9.23: inherit perlnative to work around build on host
>      without XML/Parser.pm
>    libatomics-ops: force ARM mode
>    pulseaudio-0.9.23: force ARM mode
>    webkit-gtk: force arm mode to work around binutils segfault
>    apr: add native support
>    neon: add native support
>    apr-util: add native support
>    subversion: add 1.7.0 with native support and negative D_P for now
>
Of the above patches, the pulse audio and adding native support for apr, 
neon and apr-util were taken, the other are on hold pending different 
solutions.

Thanks

Sau!

>   .../subversion/subversion-1.7.0/libtool2.patch     |   15 ++++++++
>   .../subversion/subversion_1.7.0.bb                 |   37 ++++++++++++++++++++
>   .../pulseaudio/libatomics-ops_1.2.bb               |    2 +
>   .../pulseaudio/pulseaudio_0.9.23.bb                |    3 +-
>   meta/recipes-sato/webkit/webkit-gtk_svn.bb         |   17 +++++++++
>   meta/recipes-support/apr/apr-util_1.3.12.bb        |    8 ++++
>   meta/recipes-support/apr/apr_1.4.5.bb              |    2 +
>   meta/recipes-support/neon/neon_0.29.5.bb           |    3 ++
>   8 files changed, 86 insertions(+), 1 deletions(-)
>   create mode 100644 meta/recipes-devtools/subversion/subversion-1.7.0/libtool2.patch
>   create mode 100644 meta/recipes-devtools/subversion/subversion_1.7.0.bb
>

Re: [PATCH] subversion-1.6.15: add native support too

[Saul Wold]

On 10/21/2011 03:11 PM, Martin Jansa wrote:
> Signed-off-by: Martin Jansa<Martin.Jansa@gmail.com>
> ---
>   .../subversion/subversion_1.6.15.bb                |    2 ++
>   1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/meta/recipes-devtools/subversion/subversion_1.6.15.bb b/meta/recipes-devtools/subversion/subversion_1.6.15.bb
> index 9a9484f..d3c7392 100644
> --- a/meta/recipes-devtools/subversion/subversion_1.6.15.bb
> +++ b/meta/recipes-devtools/subversion/subversion_1.6.15.bb
> @@ -5,6 +5,8 @@ RDEPENDS_${PN} = "neon"
>   LICENSE = "Apache-2"
>   HOMEPAGE = "http://subversion.tigris.org"
>
> +BBCLASSEXTEND = "native"
> +
>   PR = "r1"
>
>   SRC_URI = "http://subversion.tigris.org/downloads/${BPN}-${PV}.tar.bz2 \

Pulled this one also into OE-Core

Thanks
	Sau!

Re: [PATCH 0/8] Subversion-1.7.0 and arm/thumb work arounds

[Martin Jansa]

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

On Wed, Oct 26, 2011 at 09:54:04PM +0200, Saul Wold wrote:
> On 10/21/2011 10:17 AM, Martin Jansa wrote:
> > Hi,
> >
> > arm mode was forced without PR bump, because if it was failing for someone he
> > will notice.
> >
> > Be carefull with using subversion-1.7.0 (read the commit message).
> >
> > Pulseaudio-1.1 was released, so hopefully this work arounds won't be needed
> > there or they could be resolved better with upgrade.
> >
> > Regards,
> >
> > The following changes since commit 99da9a4e65f9dffb04efc3ad60125194c476d6b3:
> >
> >    distro-tracking-fields: update fields for tzdata and gst-plugins-good (2011-10-20 13:07:16 +0100)
> >
> > are available in the git repository at:
> >    git://git.openembedded.org/openembedded-core-contrib jansa/pull2
> >    http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=jansa/pull2
> >
> > Martin Jansa (8):
> >    pulseaudio-0.9.23: inherit perlnative to work around build on host
> >      without XML/Parser.pm
> >    libatomics-ops: force ARM mode
> >    pulseaudio-0.9.23: force ARM mode
> >    webkit-gtk: force arm mode to work around binutils segfault
> >    apr: add native support
> >    neon: add native support
> >    apr-util: add native support
> >    subversion: add 1.7.0 with native support and negative D_P for now
> >
> Of the above patches, the pulse audio and adding native support for apr, 
> neon and apr-util were taken, the other are on hold pending different 
> solutions.

I've added error log to pulseaudio commit and consolidated all my
pending patches to single jansa/pull branch, but different (better)
solutions are welcome :).

Cheers,

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

Re: [PATCH 0/8] Subversion-1.7.0 and arm/thumb work arounds

[Saul Wold]

On 10/21/2011 01:17 AM, Martin Jansa wrote:
> Hi,
>
> arm mode was forced without PR bump, because if it was failing for someone he
> will notice.
>
> Be carefull with using subversion-1.7.0 (read the commit message).
>
> Pulseaudio-1.1 was released, so hopefully this work arounds won't be needed
> there or they could be resolved better with upgrade.
>
> Regards,
>
> The following changes since commit 99da9a4e65f9dffb04efc3ad60125194c476d6b3:
>
>    distro-tracking-fields: update fields for tzdata and gst-plugins-good (2011-10-20 13:07:16 +0100)
>
> are available in the git repository at:
>    git://git.openembedded.org/openembedded-core-contrib jansa/pull2
>    http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=jansa/pull2
>
> Martin Jansa (8):
>    pulseaudio-0.9.23: inherit perlnative to work around build on host
>      without XML/Parser.pm
>    libatomics-ops: force ARM mode
>    pulseaudio-0.9.23: force ARM mode
>    webkit-gtk: force arm mode to work around binutils segfault
Martin,

What is the status of these various workarounds, did you do any further 
work to determine root causes, or do we still need these patches to do in?

Thanks
	Sau!

>    apr: add native support
>    neon: add native support
>    apr-util: add native support
>    subversion: add 1.7.0 with native support and negative D_P for now
>
>   .../subversion/subversion-1.7.0/libtool2.patch     |   15 ++++++++
>   .../subversion/subversion_1.7.0.bb                 |   37 ++++++++++++++++++++
>   .../pulseaudio/libatomics-ops_1.2.bb               |    2 +
>   .../pulseaudio/pulseaudio_0.9.23.bb                |    3 +-
>   meta/recipes-sato/webkit/webkit-gtk_svn.bb         |   17 +++++++++
>   meta/recipes-support/apr/apr-util_1.3.12.bb        |    8 ++++
>   meta/recipes-support/apr/apr_1.4.5.bb              |    2 +
>   meta/recipes-support/neon/neon_0.29.5.bb           |    3 ++
>   8 files changed, 86 insertions(+), 1 deletions(-)
>   create mode 100644 meta/recipes-devtools/subversion/subversion-1.7.0/libtool2.patch
>   create mode 100644 meta/recipes-devtools/subversion/subversion_1.7.0.bb
>

Re: [PATCH 0/8] Subversion-1.7.0 and arm/thumb work arounds

[Martin Jansa]

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

On Mon, Nov 07, 2011 at 11:13:36AM -0800, Saul Wold wrote:
> On 10/21/2011 01:17 AM, Martin Jansa wrote:
> > Hi,
> >
> > arm mode was forced without PR bump, because if it was failing for someone he
> > will notice.
> >
> > Be carefull with using subversion-1.7.0 (read the commit message).
> >
> > Pulseaudio-1.1 was released, so hopefully this work arounds won't be needed
> > there or they could be resolved better with upgrade.
> >
> > Regards,
> >
> > The following changes since commit 99da9a4e65f9dffb04efc3ad60125194c476d6b3:
> >
> >    distro-tracking-fields: update fields for tzdata and gst-plugins-good (2011-10-20 13:07:16 +0100)
> >
> > are available in the git repository at:
> >    git://git.openembedded.org/openembedded-core-contrib jansa/pull2
> >    http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=jansa/pull2
> >
> > Martin Jansa (8):
> >    pulseaudio-0.9.23: inherit perlnative to work around build on host
> >      without XML/Parser.pm
> >    libatomics-ops: force ARM mode
> >    pulseaudio-0.9.23: force ARM mode
> >    webkit-gtk: force arm mode to work around binutils segfault
> Martin,
> 
> What is the status of these various workarounds, did you do any further 
> work to determine root causes, or do we still need these patches to do in?

I've only updated commit messages in branch and added the same
workaround for aspell which also fails for thumb.

All in jansa/pull

> 
> Thanks
> 	Sau!
> 
> >    apr: add native support
> >    neon: add native support
> >    apr-util: add native support
> >    subversion: add 1.7.0 with native support and negative D_P for now
> >
> >   .../subversion/subversion-1.7.0/libtool2.patch     |   15 ++++++++
> >   .../subversion/subversion_1.7.0.bb                 |   37 ++++++++++++++++++++
> >   .../pulseaudio/libatomics-ops_1.2.bb               |    2 +
> >   .../pulseaudio/pulseaudio_0.9.23.bb                |    3 +-
> >   meta/recipes-sato/webkit/webkit-gtk_svn.bb         |   17 +++++++++
> >   meta/recipes-support/apr/apr-util_1.3.12.bb        |    8 ++++
> >   meta/recipes-support/apr/apr_1.4.5.bb              |    2 +
> >   meta/recipes-support/neon/neon_0.29.5.bb           |    3 ++
> >   8 files changed, 86 insertions(+), 1 deletions(-)
> >   create mode 100644 meta/recipes-devtools/subversion/subversion-1.7.0/libtool2.patch
> >   create mode 100644 meta/recipes-devtools/subversion/subversion_1.7.0.bb
> >

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]