* [PATCH 0/2] Update libsamplerate0
From: Tanu Kaskinen @ 2016-11-02 15:39 UTC (permalink / raw)
To: openembedded-core
The following changes since commit 98c6ebf1e05158c689e01b785d32757847cdb10c:
oeqa/selftest/kernel.py: Add new file destined for kernel related tests (2016-11-01 10:05:40 +0000)
are available in the git repository at:
git://git.openembedded.org/openembedded-core-contrib tanuk/updates
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=tanuk/updates
Tanu Kaskinen (2):
libsamplerate0: 0.1.8 -> 0.1.9
libsamplerate0: clean up dependencies
.../0001-configure.ac-improve-alsa-handling.patch | 62 ++++++++++++++++++++++
.../libsamplerate/libsamplerate0_0.1.8.bb | 21 --------
.../libsamplerate/libsamplerate0_0.1.9.bb | 26 +++++++++
3 files changed, 88 insertions(+), 21 deletions(-)
create mode 100644 meta/recipes-multimedia/libsamplerate/libsamplerate0/0001-configure.ac-improve-alsa-handling.patch
delete mode 100644 meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.8.bb
create mode 100644 meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb
--
2.9.3
^ permalink raw reply
* [PATCH 2/2] libsamplerate0: clean up dependencies
From: Tanu Kaskinen @ 2016-11-02 15:39 UTC (permalink / raw)
To: openembedded-core
In-Reply-To: <cover.1478101052.git.tanuk@iki.fi>
The flac dependency was completely bogus. Flac isn't used at all.
FFTW is only used by tests, so we don't need to provide a packageconfig
for that.
ALSA is only used by example code that isn't part of the packaged files,
so even if ALSA is enabled, it doesn't affect the build result.
Nevertheless, I prefer to disable it explicitly to be extra sure.
--disable-alsa resulted in a warning about an unsupported configure
option, although by some magic it seemed to actually work as expected.
A patch is added to get rid of that warning.
Signed-off-by: Tanu Kaskinen <tanuk@iki.fi>
---
.../0001-configure.ac-improve-alsa-handling.patch | 62 ++++++++++++++++++++++
.../libsamplerate/libsamplerate0_0.1.9.bb | 11 ++--
2 files changed, 70 insertions(+), 3 deletions(-)
create mode 100644 meta/recipes-multimedia/libsamplerate/libsamplerate0/0001-configure.ac-improve-alsa-handling.patch
diff --git a/meta/recipes-multimedia/libsamplerate/libsamplerate0/0001-configure.ac-improve-alsa-handling.patch b/meta/recipes-multimedia/libsamplerate/libsamplerate0/0001-configure.ac-improve-alsa-handling.patch
new file mode 100644
index 0000000..d19b514
--- /dev/null
+++ b/meta/recipes-multimedia/libsamplerate/libsamplerate0/0001-configure.ac-improve-alsa-handling.patch
@@ -0,0 +1,62 @@
+From 957ebce3837588f71016e37ffaf9aad0a9d41cec Mon Sep 17 00:00:00 2001
+From: Tanu Kaskinen <tanuk@iki.fi>
+Date: Fri, 23 Sep 2016 12:02:06 +0300
+Subject: [PATCH] configure.ac: improve alsa handling
+
+Three improvements:
+
+1) "--enable-alsa" or "--disable-alsa" caused a warning about an
+unsupported configure option, because AC_ARG_ENABLE was not used.
+
+2) If alsa was disabled, the "Have ALSA" item in the summary would print
+an empty string instead of "no".
+
+3) If "--enable-alsa" was passed to configure, but the headers were not
+found, configure would still succeed (with alsa disabled). It's better
+to fail and abort configure if a feature that was explicitly requested
+can't be enabled.
+
+Upstream-Status: Submitted [sent to src@mega-nerd.com]
+
+Signed-off-by: Tanu Kaskinen <tanuk@iki.fi>
+---
+ configure.ac | 9 ++++++++-
+ 1 file changed, 8 insertions(+), 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index 474c6ab..bad597e 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -101,6 +101,9 @@ AC_ARG_ENABLE(cpu-clip,
+ AC_HELP_STRING([--disable-cpu-clip], [disable tricky cpu specific clipper]),
+ ac_arg_cpu_clip="N", ac_arg_cpu_clip="Y")
+
++AC_ARG_ENABLE(alsa,
++ AC_HELP_STRING([--disable-alsa], [disable ALSA support in the varispeed-play example program]))
++
+ #====================================================================================
+ # Check types and their sizes.
+
+@@ -194,6 +197,10 @@ if test x$enable_alsa != xno ; then
+ if test x$ac_cv_header_alsa_asoundlib_h = xyes ; then
+ ALSA_LIBS="-lasound"
+ enable_alsa=yes
++ elif test x$enable_alsa = xyes ; then
++ AC_MSG_ERROR(["ALSA headers not found"])
++ else
++ enable_alsa=no
+ fi
+ fi
+
+@@ -340,7 +347,7 @@ AC_MSG_RESULT([
+ Have FFTW : ................... ${ac_cv_fftw3}])
+
+ AC_MSG_RESULT([ Have libsndfile : ............. ${ac_cv_sndfile}])
+-AC_MSG_RESULT([ Have ALSA : ................... ${ac_cv_header_alsa_asoundlib_h}
++AC_MSG_RESULT([ Have ALSA : ................... ${enable_alsa}
+ ])
+
+ AC_MSG_RESULT([ Installation directories :
+--
+2.9.3
+
diff --git a/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb b/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb
index a24b8c7..5473bff 100644
--- a/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb
+++ b/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb
@@ -4,10 +4,12 @@ SECTION = "libs"
LICENSE = "BSD-2-Clause"
LIC_FILES_CHKSUM = "file://COPYING;md5=336d6faf40fb600bafb0061f4052f1f4 \
file://src/samplerate.c;beginline=1;endline=7;md5=5b6982a8c2811c7312c13cccbf55f55e"
-DEPENDS = "flac libsndfile1"
+DEPENDS = "libsndfile1"
PR = "r1"
-SRC_URI = "http://www.mega-nerd.com/SRC/libsamplerate-${PV}.tar.gz"
+SRC_URI = "http://www.mega-nerd.com/SRC/libsamplerate-${PV}.tar.gz \
+ file://0001-configure.ac-improve-alsa-handling.patch \
+"
SRC_URI[md5sum] = "2b78ae9fe63b36b9fbb6267fad93f259"
SRC_URI[sha256sum] = "0a7eb168e2f21353fb6d84da152e4512126f7dc48ccb0be80578c565413444c1"
@@ -18,4 +20,7 @@ S = "${WORKDIR}/libsamplerate-${PV}"
inherit autotools pkgconfig
-PACKAGECONFIG[fftw] = ",--disable-fftw,fftw"
+# FFTW and ALSA are only used in tests and examples, so they don't affect
+# normal builds. It should be safe to ignore these, but explicitly disabling
+# them adds some extra certainty that builds are deterministic.
+EXTRA_OECONF = "--disable-fftw --disable-alsa"
--
2.9.3
^ permalink raw reply related
* [PATCH 1/2] libsamplerate0: 0.1.8 -> 0.1.9
From: Tanu Kaskinen @ 2016-11-02 15:39 UTC (permalink / raw)
To: openembedded-core
In-Reply-To: <cover.1478101052.git.tanuk@iki.fi>
The license has changed to BSD as explained here:
http://www.mega-nerd.com/SRC/license.html
Signed-off-by: Tanu Kaskinen <tanuk@iki.fi>
---
.../{libsamplerate0_0.1.8.bb => libsamplerate0_0.1.9.bb} | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
rename meta/recipes-multimedia/libsamplerate/{libsamplerate0_0.1.8.bb => libsamplerate0_0.1.9.bb} (62%)
diff --git a/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.8.bb b/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb
similarity index 62%
rename from meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.8.bb
rename to meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb
index 1c37681..a24b8c7 100644
--- a/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.8.bb
+++ b/meta/recipes-multimedia/libsamplerate/libsamplerate0_0.1.9.bb
@@ -1,16 +1,16 @@
SUMMARY = "Audio Sample Rate Conversion library"
HOMEPAGE = "http://www.mega-nerd.com/SRC/"
SECTION = "libs"
-LICENSE = "GPLv2+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f \
- file://src/samplerate.c;beginline=1;endline=17;md5=d0807c35fc906466d24a50463534815a"
+LICENSE = "BSD-2-Clause"
+LIC_FILES_CHKSUM = "file://COPYING;md5=336d6faf40fb600bafb0061f4052f1f4 \
+ file://src/samplerate.c;beginline=1;endline=7;md5=5b6982a8c2811c7312c13cccbf55f55e"
DEPENDS = "flac libsndfile1"
PR = "r1"
SRC_URI = "http://www.mega-nerd.com/SRC/libsamplerate-${PV}.tar.gz"
-SRC_URI[md5sum] = "1c7fb25191b4e6e3628d198a66a84f47"
-SRC_URI[sha256sum] = "93b54bdf46d5e6d2354b7034395fe329c222a966790de34520702bb9642f1c06"
+SRC_URI[md5sum] = "2b78ae9fe63b36b9fbb6267fad93f259"
+SRC_URI[sha256sum] = "0a7eb168e2f21353fb6d84da152e4512126f7dc48ccb0be80578c565413444c1"
UPSTREAM_CHECK_URI = "http://www.mega-nerd.com/SRC/download.html"
--
2.9.3
^ permalink raw reply related
* Re: [PATCH 4/8] serf: Update to version 1.3.9
From: Alexander Kanavin @ 2016-11-02 15:15 UTC (permalink / raw)
To: openembedded-core
In-Reply-To: <CAAPZna0XP4GOEg1gKCv2sdbw7H4FKkWTcTFWhVNh_rHAVL5FMA@mail.gmail.com>
On 11/02/2016 03:00 PM, Fabio Berton wrote:
> I'll send a v2 using https://archive.apache.org/dist/serf/
Please update the UPSTREAM_CHECK_URI too, or maybe you can drop it
altogether.
Alex
^ permalink raw reply
* [PATCH] classes/license: copy licenses even if there are no checksummed files
From: Ross Burton @ 2016-11-02 15:07 UTC (permalink / raw)
To: openembedded-core
Previously do_populate_lic would bail writing the license files (such as the MIT
license text) to deploy/licenses/${PN}/ if there were no files listed in
LIC_FILES_CHKSUM.
However this means that recipes that generate their content (such as os-release)
or are otherwise "interesting" (such as perf) don't have their license files
copied over, resulting in warnings from do_rootfs.
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/classes/license.bbclass | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/meta/classes/license.bbclass b/meta/classes/license.bbclass
index 660b85f..afcfbfc 100644
--- a/meta/classes/license.bbclass
+++ b/meta/classes/license.bbclass
@@ -390,7 +390,7 @@ def find_license_files(d):
from collections import defaultdict, OrderedDict
# All the license files for the package
- lic_files = d.getVar('LIC_FILES_CHKSUM', True)
+ lic_files = d.getVar('LIC_FILES_CHKSUM', True) or ""
pn = d.getVar('PN', True)
# The license files are located in S/LIC_FILE_CHECKSUM.
srcdir = d.getVar('S', True)
@@ -469,12 +469,6 @@ def find_license_files(d):
if not generic_directory:
bb.fatal("COMMON_LICENSE_DIR is unset. Please set this in your distro config")
- if not lic_files:
- # No recipe should have an invalid license file. This is checked else
- # where, but let's be pedantic
- bb.note(pn + ": Recipe file does not have license file information.")
- return lic_files_paths
-
for url in lic_files.split():
try:
(type, host, path, user, pswd, parm) = bb.fetch.decodeurl(url)
--
2.8.1
^ permalink raw reply related
* [PATCH] rm_work: Ensure we don't remove sigbasedata files
From: Richard Purdie @ 2016-11-02 15:05 UTC (permalink / raw)
To: openembedded-core
We don't remove sigdata files, we also shouldn't remove sigbasedata files
as this hinders debugging.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
diff --git a/meta/classes/rm_work.bbclass b/meta/classes/rm_work.bbclass
index b71a9d1..64b6981 100644
--- a/meta/classes/rm_work.bbclass
+++ b/meta/classes/rm_work.bbclass
@@ -58,7 +58,7 @@ do_rm_work () {
*do_setscene*)
break
;;
- *sigdata*)
+ *sigdata*|*sigbasedata*)
i=dummy
break
;;
^ permalink raw reply related
* [PATCH] sstate: Ensure we don't remove sigbasedata files
From: Richard Purdie @ 2016-11-02 15:04 UTC (permalink / raw)
To: openembedded-core
We don't remove sigdata files, we also shouldn't remove sigbasedata files
as this hinders debugging.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index 172384b..8643f3d 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -457,7 +457,7 @@ def sstate_clean(ss, d):
rm_nohash = ".do_%s" % ss['task']
for stfile in glob.glob(wildcard_stfile):
# Keep the sigdata
- if ".sigdata." in stfile:
+ if ".sigdata." in stfile or ".sigbasedata." in stfile:
continue
# Preserve taint files in the stamps directory
if stfile.endswith('.taint'):
^ permalink raw reply related
* Re: [PATCH] u-boot: mkimage: Fix build of u-boot-mkimage
From: Burton, Ross @ 2016-11-02 14:48 UTC (permalink / raw)
To: Marek Vasut; +Cc: OE-core
In-Reply-To: <20161102120724.30063-1-marex@denx.de>
[-- Attachment #1: Type: text/plain, Size: 315 bytes --]
On 2 November 2016 at 12:07, Marek Vasut <marex@denx.de> wrote:
> Drop the incorrect EXTRA_OEMAKE and just use the one
> from u-boot-common*.inc . Moreover, since U-Boot buildsystem
> already strips the tools, add INSANE_SKIP = "already-stripped" .
>
But u-boot-common doesn't set EXTRA_OEMAKE...
Ross
[-- Attachment #2: Type: text/html, Size: 825 bytes --]
^ permalink raw reply
* Re: [PATCH] u-boot: mkimage: Fix build of u-boot-mkimage
From: André Draszik @ 2016-11-02 14:44 UTC (permalink / raw)
To: openembedded-core
In-Reply-To: <20161102120724.30063-1-marex@denx.de>
On Wed, 2016-11-02 at 13:07 +0100, Marek Vasut wrote:
> The build failed on qemux86-64 because it couldn't execute
> tools/bin2header on a host due to it being compiled with target
> toolchain. Drop the incorrect EXTRA_OEMAKE and just use the one
> from u-boot-common*.inc . Moreover, since U-Boot buildsystem
> already strips the tools, add INSANE_SKIP = "already-stripped" .
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Ross Burton <ross.burton@intel.com>
> ---
> meta/recipes-bsp/u-boot/u-boot-mkimage_2016.09.01.bb | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.09.01.bb
> b/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.09.01.bb
> index 5025961..76b303e 100644
> --- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.09.01.bb
> +++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.09.01.bb
> @@ -3,8 +3,6 @@ require u-boot-common_${PV}.inc
> SUMMARY = "U-Boot bootloader image creation tool"
> DEPENDS = "openssl"
>
> -EXTRA_OEMAKE = 'CROSS_COMPILE="${TARGET_PREFIX}" CC="${CC} ${CFLAGS}
> ${LDFLAGS}" STRIP=true V=1'
Can we keep V=1, please? (Unless it's default anyway).
Cheers,
Andre'
> -
> do_compile () {
> oe_runmake sandbox_defconfig
> oe_runmake cross_tools NO_SDL=1
> @@ -17,3 +15,4 @@ do_install () {
> }
>
> BBCLASSEXTEND = "native nativesdk"
> +INSANE_SKIP_${PN} = "already-stripped"
> --
> 2.9.3
>
^ permalink raw reply
* Re: [PATCH 7/8] gawk: Update to version 4.1.4
From: Fabio Berton @ 2016-11-02 13:01 UTC (permalink / raw)
To: Burton, Ross; +Cc: OE-core
In-Reply-To: <CAJTo0LYzaYCx6S1BP=FF_D6d20wzdGh7uZcypr+yPhAMWWQjLQ@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2461 bytes --]
Hi Ross,
I'll look this issue.
Thanks,
Fabio
On Wed, Nov 2, 2016 at 9:34 AM, Burton, Ross <ross.burton@intel.com> wrote:
> This causes QA warnings:
>
> WARNING: gawk-4.1.4-r0 do_package_qa: QA Issue: /usr/lib/gawk/ptest/test/arrayind1.awk
> contained in package gawk-ptest requires /usr/local/bin/awk, but no
> providers found in RDEPENDS_gawk-ptest? [file-rdeps]
>
> We should patch out the hashbang line (and submit that upstream as none of
> the other awk scripts in the test suite have a hashbang).
>
> Ross
>
> On 1 November 2016 at 19:53, Fabio Berton <fabio.berton@ossystems.com.br>
> wrote:
>
>> Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
>> ---
>> meta/recipes-extended/gawk/{gawk-4.1.3 => gawk-4.1.4}/run-ptest | 0
>> meta/recipes-extended/gawk/{gawk_4.1.3.bb => gawk_4.1.4.bb} | 4 ++--
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>> rename meta/recipes-extended/gawk/{gawk-4.1.3 => gawk-4.1.4}/run-ptest
>> (100%)
>> rename meta/recipes-extended/gawk/{gawk_4.1.3.bb => gawk_4.1.4.bb} (89%)
>>
>> diff --git a/meta/recipes-extended/gawk/gawk-4.1.3/run-ptest
>> b/meta/recipes-extended/gawk/gawk-4.1.4/run-ptest
>> similarity index 100%
>> rename from meta/recipes-extended/gawk/gawk-4.1.3/run-ptest
>> rename to meta/recipes-extended/gawk/gawk-4.1.4/run-ptest
>> diff --git a/meta/recipes-extended/gawk/gawk_4.1.3.bb
>> b/meta/recipes-extended/gawk/gawk_4.1.4.bb
>> similarity index 89%
>> rename from meta/recipes-extended/gawk/gawk_4.1.3.bb
>> rename to meta/recipes-extended/gawk/gawk_4.1.4.bb
>> index 6ca7f3e..ad18237 100644
>> --- a/meta/recipes-extended/gawk/gawk_4.1.3.bb
>> +++ b/meta/recipes-extended/gawk/gawk_4.1.4.bb
>> @@ -19,8 +19,8 @@ SRC_URI = "${GNU_MIRROR}/gawk/gawk-${PV}.tar.gz \
>> file://run-ptest \
>> "
>>
>> -SRC_URI[md5sum] = "55d37f4069502677f25d1340df8eec97"
>> -SRC_URI[sha256sum] = "524effa5b9ecd4ed940f2581c5d3c
>> 1df4e4bd7e6f768aa033c1916f47dfc6e29"
>> +SRC_URI[md5sum] = "f20c94ca51b6ebfc9bffb90f95c8ffbb"
>> +SRC_URI[sha256sum] = "8c03080e2b5a56263e8783f6f1f30
>> 6398d4591be18254041f3f547efef944d35"
>>
>> inherit autotools gettext texinfo update-alternatives
>>
>> --
>> 2.1.4
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>
>
[-- Attachment #2: Type: text/html, Size: 4845 bytes --]
^ permalink raw reply
* Re: [PATCH 4/8] serf: Update to version 1.3.9
From: Fabio Berton @ 2016-11-02 13:00 UTC (permalink / raw)
To: Burton, Ross; +Cc: OE-core
In-Reply-To: <CAJTo0LbayQcAY1qGtiJWjfJ2X+kNHZAeC4cf43d-5rwWx-EEjA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 510 bytes --]
Ok,
I'll send a v2 using https://archive.apache.org/dist/serf/
Thanks,
Fabio
On Wed, Nov 2, 2016 at 10:21 AM, Burton, Ross <ross.burton@intel.com> wrote:
>
> On 2 November 2016 at 11:39, Bas Mevissen <abuse@basmevissen.nl> wrote:
>
>> It looks like https://archive.apache.org/dist/serf contains all
>> releases, including the current one.
>>
>
> Perfect, can you send a v2?
>
> (just checked the history and it moved to snapshot.debian.org from
> googlecode)
>
> Thanks,
> Ross
>
[-- Attachment #2: Type: text/html, Size: 1825 bytes --]
^ permalink raw reply
* [PATCH] guilt: remove
From: Ross Burton @ 2016-11-02 12:31 UTC (permalink / raw)
To: openembedded-core
This was used by the kernel tooling but has since been replaced by kgit-s2q. As
there are no other users in oe-core, and this was just a native recipe, remove
it.
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/recipes-devtools/guilt/files/guilt-bash.patch | 288 ---------------------
meta/recipes-devtools/guilt/guilt-native_git.bb | 23 --
2 files changed, 311 deletions(-)
delete mode 100644 meta/recipes-devtools/guilt/files/guilt-bash.patch
delete mode 100644 meta/recipes-devtools/guilt/guilt-native_git.bb
diff --git a/meta/recipes-devtools/guilt/files/guilt-bash.patch b/meta/recipes-devtools/guilt/files/guilt-bash.patch
deleted file mode 100644
index 70a439e..0000000
--- a/meta/recipes-devtools/guilt/files/guilt-bash.patch
+++ /dev/null
@@ -1,288 +0,0 @@
-guilt: explicitly call for bash
-
-Running complex guilt stacks of patches/commits can cause interesting
-failures after ~20m of processing, with errors like "Bad substitution".
-
-These have been traced back to having /bin/sh --> /bin/dash on Ubuntu
-systems. Putting a shell that actually *works* (such as /bin/bash)
-in as /bin/sh makes the problem go away. So here we change the guilt
-scripts to explicitly call for bash to ensure we don't have a similar
-issue after deployment.
-
-Upstream-Status: Inappropriate [oe-specific]
-
-Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
-
----
- guilt | 2 +-
- guilt-add | 2 +-
- guilt-applied | 2 +-
- guilt-branch | 2 +-
- guilt-delete | 2 +-
- guilt-diff | 2 +-
- guilt-export | 2 +-
- guilt-files | 2 +-
- guilt-fold | 2 +-
- guilt-fork | 2 +-
- guilt-graph | 2 +-
- guilt-header | 2 +-
- guilt-help | 2 +-
- guilt-import | 2 +-
- guilt-import-commit | 2 +-
- guilt-init | 4 ++--
- guilt-new | 2 +-
- guilt-next | 2 +-
- guilt-patchbomb | 2 +-
- guilt-pop | 2 +-
- guilt-prev | 2 +-
- guilt-push | 2 +-
- guilt-rebase | 2 +-
- guilt-refresh | 2 +-
- guilt-rm | 2 +-
- guilt-series | 2 +-
- guilt-status | 2 +-
- guilt-top | 2 +-
- guilt-unapplied | 2 +-
- 29 files changed, 30 insertions(+), 30 deletions(-)
-
---- a/guilt
-+++ b/guilt
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006-2010
- #
---- a/guilt-add
-+++ b/guilt-add
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-applied
-+++ b/guilt-applied
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-branch
-+++ b/guilt-branch
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2007-2008
- #
---- a/guilt-delete
-+++ b/guilt-delete
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-diff
-+++ b/guilt-diff
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (C) 2007 Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
- #
---- a/guilt-export
-+++ b/guilt-export
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Pierre Habouzit, 2007
- #
---- a/guilt-files
-+++ b/guilt-files
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (C) 2007 Yasushi SHOJI <yashi@atmark-techno.com>
- #
---- a/guilt-fold
-+++ b/guilt-fold
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-fork
-+++ b/guilt-fork
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2007
- #
---- a/guilt-graph
-+++ b/guilt-graph
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2007
- #
---- a/guilt-header
-+++ b/guilt-header
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006-2010
- #
---- a/guilt-help
-+++ b/guilt-help
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2007
- #
---- a/guilt-import
-+++ b/guilt-import
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2007
- #
---- a/guilt-import-commit
-+++ b/guilt-import-commit
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2007
- #
---- a/guilt-init
-+++ b/guilt-init
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
-@@ -31,7 +31,7 @@ touch "$GUILT_DIR/$branch/status"
-
- mkdir -p "$GIT_DIR/hooks/guilt"
- cat > "$GIT_DIR/hooks/guilt/delete" <<EOF
--#!/bin/sh
-+#!/bin/bash
- # Usage: <script> <patch being removed>
-
- echo "Removing patch '\$1'..."
---- a/guilt-new
-+++ b/guilt-new
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-next
-+++ b/guilt-next
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-patchbomb
-+++ b/guilt-patchbomb
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2007
- #
---- a/guilt-pop
-+++ b/guilt-pop
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-prev
-+++ b/guilt-prev
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-push
-+++ b/guilt-push
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-rebase
-+++ b/guilt-rebase
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2007
- #
---- a/guilt-refresh
-+++ b/guilt-refresh
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-rm
-+++ b/guilt-rm
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-series
-+++ b/guilt-series
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-status
-+++ b/guilt-status
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-top
-+++ b/guilt-top
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
---- a/guilt-unapplied
-+++ b/guilt-unapplied
-@@ -1,4 +1,4 @@
--#!/bin/sh
-+#!/bin/bash
- #
- # Copyright (c) Josef "Jeff" Sipek, 2006, 2007
- #
diff --git a/meta/recipes-devtools/guilt/guilt-native_git.bb b/meta/recipes-devtools/guilt/guilt-native_git.bb
deleted file mode 100644
index b63c68c..0000000
--- a/meta/recipes-devtools/guilt/guilt-native_git.bb
+++ /dev/null
@@ -1,23 +0,0 @@
-SUMMARY = "quilt-like tool for Git"
-LICENSE = "GPLv2"
-
-LIC_FILES_CHKSUM = "file://COPYING;md5=b6f3400dc1a01cebafe8a52b3f344135"
-
-inherit native
-
-SRC_URI = "git://repo.or.cz/guilt.git \
- file://guilt-bash.patch \
- "
-PV = "0.35+git${SRCPV}"
-SRCREV = "c2a5bae511c6d5354aa4e1cb59069c31df2b8eeb"
-
-S = "${WORKDIR}/git"
-
-# we don't compile, we just install
-do_compile() {
- :
-}
-
-do_install() {
- oe_runmake PREFIX=${D}/${prefix} install
-}
--
2.8.1
^ permalink raw reply related
* Re: [PATCH 8/8] guilt-native: Update to tag v0.36
From: Burton, Ross @ 2016-11-02 12:30 UTC (permalink / raw)
To: Fabio Berton; +Cc: OE-core
In-Reply-To: <1478030040-20670-8-git-send-email-fabio.berton@ossystems.com.br>
[-- Attachment #1: Type: text/plain, Size: 243 bytes --]
On 1 November 2016 at 19:54, Fabio Berton <fabio.berton@ossystems.com.br>
wrote:
> Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
>
The kernel tooling doesn't use this anymore, so I propose we remove it
instead.
Ross
[-- Attachment #2: Type: text/html, Size: 723 bytes --]
^ permalink raw reply
* Re: [PATCH 4/8] serf: Update to version 1.3.9
From: Burton, Ross @ 2016-11-02 12:21 UTC (permalink / raw)
To: Bas Mevissen; +Cc: OE-core
In-Reply-To: <1BF9A377-D1BB-467B-90F2-0E205B7CEF49@basmevissen.nl>
[-- Attachment #1: Type: text/plain, Size: 316 bytes --]
On 2 November 2016 at 11:39, Bas Mevissen <abuse@basmevissen.nl> wrote:
> It looks like https://archive.apache.org/dist/serf contains all releases,
> including the current one.
>
Perfect, can you send a v2?
(just checked the history and it moved to snapshot.debian.org from
googlecode)
Thanks,
Ross
[-- Attachment #2: Type: text/html, Size: 898 bytes --]
^ permalink raw reply
* Re: [PATCH 09/13] icu: update to 58.1
From: Burton, Ross @ 2016-11-02 12:16 UTC (permalink / raw)
To: Alexander Kanavin; +Cc: OE-core
In-Reply-To: <fced9d20130046ffaf6daa1fe7d619098fdf91c6.1478013702.git.alexander.kanavin@linux.intel.com>
[-- Attachment #1: Type: text/plain, Size: 1167 bytes --]
On 1 November 2016 at 15:25, Alexander Kanavin <
alexander.kanavin@linux.intel.com> wrote:
> The copyright for the software has been transferred to Unicode Inc from
> IBM,
> but the terms are same.
>
> libiculx and libicule are no longer produced as they depend on an external
> package icu-le-hb (previous versions had an option of using an internal
> implementation which now has been dropped). I have verified that icu
> dependencies in oe-core and meta-oe still build.
>
> Signed-off-by: Alexander Kanavin <alexander.kanavin@linux.intel.com>
>
This breaks under musl:
../../icu/source/i18n/digitlst.cpp:67:24: fatal error: xlocale.h: No such
file or directory
# include <xlocale.h>
musl doesn't have xlocale.h but does have a crippled strtod_l (doesn't
actually support a passed in locale, which is the entire point of the _l
functions).
Upstreams such as pulseaudio have done
https://bugs.freedesktop.org/attachment.cgi?id=118644 to guard the include,
but I see that digitlist.cpp has an alternative branch where strtod_l isn't
supported: as that function on musl is crippled maybe ICU should be told
not to use it?
Ross
[-- Attachment #2: Type: text/html, Size: 1888 bytes --]
^ permalink raw reply
* Re: [PATCH 1/4] u-boot: Update to 2016.09.01 release
From: Marek Vasut @ 2016-11-02 12:08 UTC (permalink / raw)
To: Burton, Ross; +Cc: OE-core
In-Reply-To: <CAJTo0LZoN5QRB3pg0+kP0_tQqJ_WTzZuaH_FA-ijDtoY+mYHPg@mail.gmail.com>
On 10/31/2016 03:59 PM, Burton, Ross wrote:
>
> On 30 October 2016 at 18:18, Marek Vasut <marex@denx.de
> <mailto:marex@denx.de>> wrote:
>
> So the simplest test would be to do the build of native u-boot-mkimage
> on, say, centos/fedora ?
>
>
> No, a build on Debian (I've just replicated this on my own debian build
> machine). Literally just "bitbake u-boot-mkimage" fails to build.
Got it, replicated, thanks.
I sent a patch, see if that works for you.
--
Best regards,
Marek Vasut
^ permalink raw reply
* [PATCH] u-boot: mkimage: Fix build of u-boot-mkimage
From: Marek Vasut @ 2016-11-02 12:07 UTC (permalink / raw)
To: openembedded-core; +Cc: Marek Vasut
The build failed on qemux86-64 because it couldn't execute
tools/bin2header on a host due to it being compiled with target
toolchain. Drop the incorrect EXTRA_OEMAKE and just use the one
from u-boot-common*.inc . Moreover, since U-Boot buildsystem
already strips the tools, add INSANE_SKIP = "already-stripped" .
Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Ross Burton <ross.burton@intel.com>
---
meta/recipes-bsp/u-boot/u-boot-mkimage_2016.09.01.bb | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.09.01.bb b/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.09.01.bb
index 5025961..76b303e 100644
--- a/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.09.01.bb
+++ b/meta/recipes-bsp/u-boot/u-boot-mkimage_2016.09.01.bb
@@ -3,8 +3,6 @@ require u-boot-common_${PV}.inc
SUMMARY = "U-Boot bootloader image creation tool"
DEPENDS = "openssl"
-EXTRA_OEMAKE = 'CROSS_COMPILE="${TARGET_PREFIX}" CC="${CC} ${CFLAGS} ${LDFLAGS}" STRIP=true V=1'
-
do_compile () {
oe_runmake sandbox_defconfig
oe_runmake cross_tools NO_SDL=1
@@ -17,3 +15,4 @@ do_install () {
}
BBCLASSEXTEND = "native nativesdk"
+INSANE_SKIP_${PN} = "already-stripped"
--
2.9.3
^ permalink raw reply related
* Re: [PATCH 4/8] serf: Update to version 1.3.9
From: Bas Mevissen @ 2016-11-02 11:39 UTC (permalink / raw)
To: Burton, Ross; +Cc: OE-core
In-Reply-To: <CAJTo0LbZrrrcvMmqpb943oarTKY6B=UkmszpdFLv=3cWqCV9SA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1076 bytes --]
> On 2 Nov 2016, at 12:04, Burton, Ross <ross.burton@intel.com> wrote:
>
>
>> On 1 November 2016 at 19:53, Fabio Berton <fabio.berton@ossystems.com.br> wrote:
>> -SRC_URI = "http://snapshot.debian.org/archive/debian/20160728T043443Z/pool/main/s/${BPN}/${BPN}_${PV}.orig.tar.gz \
>> +SRC_URI = "https://www.apache.org/dist/${PN}/${PN}-${PV}.tar.bz2 \
>
> The reason we download from snapshot.debian.org is that apache.org/dist/ only contains the latest release (see http://apache.org/dist/serf/) so the moment 1.4.0 is released the tarball will disappear.
>
> If apache maintains an archive of *all* releases with a stable URL then that would be preferable to snapshot.debian.org, but relying on apache.org/dist/ isn't possible.
>
It looks like https://archive.apache.org/dist/serf contains all releases, including the current one.
Bas.
> Ross
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
[-- Attachment #2: Type: text/html, Size: 2779 bytes --]
^ permalink raw reply
* Re: [PATCH 7/8] gawk: Update to version 4.1.4
From: Burton, Ross @ 2016-11-02 11:34 UTC (permalink / raw)
To: Fabio Berton; +Cc: OE-core
In-Reply-To: <1478030040-20670-7-git-send-email-fabio.berton@ossystems.com.br>
[-- Attachment #1: Type: text/plain, Size: 2253 bytes --]
This causes QA warnings:
WARNING: gawk-4.1.4-r0 do_package_qa: QA Issue:
/usr/lib/gawk/ptest/test/arrayind1.awk contained in package gawk-ptest
requires /usr/local/bin/awk, but no providers found in RDEPENDS_gawk-ptest?
[file-rdeps]
We should patch out the hashbang line (and submit that upstream as none of
the other awk scripts in the test suite have a hashbang).
Ross
On 1 November 2016 at 19:53, Fabio Berton <fabio.berton@ossystems.com.br>
wrote:
> Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
> ---
> meta/recipes-extended/gawk/{gawk-4.1.3 => gawk-4.1.4}/run-ptest | 0
> meta/recipes-extended/gawk/{gawk_4.1.3.bb => gawk_4.1.4.bb} | 4 ++--
> 2 files changed, 2 insertions(+), 2 deletions(-)
> rename meta/recipes-extended/gawk/{gawk-4.1.3 => gawk-4.1.4}/run-ptest
> (100%)
> rename meta/recipes-extended/gawk/{gawk_4.1.3.bb => gawk_4.1.4.bb} (89%)
>
> diff --git a/meta/recipes-extended/gawk/gawk-4.1.3/run-ptest
> b/meta/recipes-extended/gawk/gawk-4.1.4/run-ptest
> similarity index 100%
> rename from meta/recipes-extended/gawk/gawk-4.1.3/run-ptest
> rename to meta/recipes-extended/gawk/gawk-4.1.4/run-ptest
> diff --git a/meta/recipes-extended/gawk/gawk_4.1.3.bb
> b/meta/recipes-extended/gawk/gawk_4.1.4.bb
> similarity index 89%
> rename from meta/recipes-extended/gawk/gawk_4.1.3.bb
> rename to meta/recipes-extended/gawk/gawk_4.1.4.bb
> index 6ca7f3e..ad18237 100644
> --- a/meta/recipes-extended/gawk/gawk_4.1.3.bb
> +++ b/meta/recipes-extended/gawk/gawk_4.1.4.bb
> @@ -19,8 +19,8 @@ SRC_URI = "${GNU_MIRROR}/gawk/gawk-${PV}.tar.gz \
> file://run-ptest \
> "
>
> -SRC_URI[md5sum] = "55d37f4069502677f25d1340df8eec97"
> -SRC_URI[sha256sum] = "524effa5b9ecd4ed940f2581c5d3c1
> df4e4bd7e6f768aa033c1916f47dfc6e29"
> +SRC_URI[md5sum] = "f20c94ca51b6ebfc9bffb90f95c8ffbb"
> +SRC_URI[sha256sum] = "8c03080e2b5a56263e8783f6f1f306
> 398d4591be18254041f3f547efef944d35"
>
> inherit autotools gettext texinfo update-alternatives
>
> --
> 2.1.4
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
[-- Attachment #2: Type: text/html, Size: 3875 bytes --]
^ permalink raw reply
* Re: [PATCH 4/8] serf: Update to version 1.3.9
From: Burton, Ross @ 2016-11-02 11:04 UTC (permalink / raw)
To: Fabio Berton; +Cc: OE-core
In-Reply-To: <1478030040-20670-4-git-send-email-fabio.berton@ossystems.com.br>
[-- Attachment #1: Type: text/plain, Size: 671 bytes --]
On 1 November 2016 at 19:53, Fabio Berton <fabio.berton@ossystems.com.br>
wrote:
> -SRC_URI = "http://snapshot.debian.org/archive/debian/
> 20160728T043443Z/pool/main/s/${BPN}/${BPN}_${PV}.orig.tar.gz \
> +SRC_URI = "https://www.apache.org/dist/${PN}/${PN}-${PV}.tar.bz2 \
>
The reason we download from snapshot.debian.org is that apache.org/dist/
only contains the latest release (see http://apache.org/dist/serf/) so the
moment 1.4.0 is released the tarball will disappear.
If apache maintains an archive of *all* releases with a stable URL then
that would be preferable to snapshot.debian.org, but relying on
apache.org/dist/ isn't possible.
Ross
[-- Attachment #2: Type: text/html, Size: 1709 bytes --]
^ permalink raw reply
* Re: [PATCH 2/2] shadow: add nologin.8 to alternatives
From: Burton, Ross @ 2016-11-02 10:58 UTC (permalink / raw)
To: Paul Eggleton; +Cc: OE-core
In-Reply-To: <11200184.2dbvXmKaFJ@peggleto-mobl.ger.corp.intel.com>
[-- Attachment #1: Type: text/plain, Size: 300 bytes --]
On 2 November 2016 at 02:50, Paul Eggleton <paul.eggleton@linux.intel.com>
wrote:
> Missing commit message. Probably the most important thing to include would
> be
> which other recipe packages the same named binary.
>
Fixed in git, thanks. I meant to go back and fill this in...
Ross
[-- Attachment #2: Type: text/html, Size: 714 bytes --]
^ permalink raw reply
* Re: [PATCH] connman: fix bad file descriptor initialisation
From: Anders Darander @ 2016-11-02 8:10 UTC (permalink / raw)
To: openembedded-core
In-Reply-To: <a7611154-80b8-3a80-131a-9200ba4ccb34@gmail.com>
* Lukasz Nowak <lnowak.tyco@gmail.com> [161101 13:57]:
> It is a re-send. Apologies, but I have not received either of my two
> e-mails back from the mailing list, so I assumed the first one got
> stuck somewhere. I can see both of them in the web archive, though. Is
> that normal?
Are you receiving any email you send to an list? GMail is pretty good at
removing the e-mails from a list that you sent yourself... At least,
I've never received any of them myself.
There's quite a few old threads about this on GMails support forum.
/Anders
--
Anders Darander, Senior System Architect
ChargeStorm AB / eStorm AB
^ permalink raw reply
* [PATCH 1/1] fix "svnadmin create" fail on x86
From: Dengke Du @ 2016-11-02 7:12 UTC (permalink / raw)
To: openembedded-core
In-Reply-To: <cover.1478070580.git.dengke.du@windriver.com>
When run the following command on x86:
svnadmin create /var/test_repo
It cause segmentation fault error like the following:
[16499.751837] svnadmin[21117]: segfault at 83 ip 00000000f74bf7f6 sp 00000000ffdd9b34 error 4 in libc-2.24.so[f7441000+1af000]
Segmentation fault (core dumped)
This is because in source code ./subversion/libsvn_fs_fs/low_level.c,
function svn_fs_fs__unparse_footer, when:
target arch: x86
apr_off_t: 4 bytes
if the "APR_OFF_T_FMT" is "lld", it still use type "apr_off_t" to pass
data to apr, but in apr source code file apr_snprintf.c the function
apr_vformatter meet "lld", it would use the:
i_quad = va_arg(ap, apr_int64_t);
It uses the apr_int64_t to deal data, it read 8 bytes, so the follow-up
data may be error.
Signed-off-by: Dengke Du <dengke.du@windriver.com>
---
.../0001-fix-svnadmin-create-fail-on-x86.patch | 56 ++++++++++++++++++++++
.../subversion/subversion_1.9.4.bb | 1 +
2 files changed, 57 insertions(+)
create mode 100644 meta/recipes-devtools/subversion/subversion/0001-fix-svnadmin-create-fail-on-x86.patch
diff --git a/meta/recipes-devtools/subversion/subversion/0001-fix-svnadmin-create-fail-on-x86.patch b/meta/recipes-devtools/subversion/subversion/0001-fix-svnadmin-create-fail-on-x86.patch
new file mode 100644
index 0000000..d440528
--- /dev/null
+++ b/meta/recipes-devtools/subversion/subversion/0001-fix-svnadmin-create-fail-on-x86.patch
@@ -0,0 +1,56 @@
+From 09475e0befca8d120c957177ce8568fa2209a1a9 Mon Sep 17 00:00:00 2001
+From: Dengke Du <dengke.du@windriver.com>
+Date: Wed, 2 Nov 2016 11:09:44 +0800
+Subject: [PATCH] fix "svnadmin create" fail on x86
+
+When run the following command on x86:
+
+ svnadmin create /var/test_repo
+
+It cause segmentation fault error like the following:
+
+ [16499.751837] svnadmin[21117]: segfault at 83 ip 00000000f74bf7f6 sp 00000000ffdd9b34 error 4 in libc-2.24.so[f7441000+1af000]
+ Segmentation fault (core dumped)
+
+This is because in source code ./subversion/libsvn_fs_fs/low_level.c,
+function svn_fs_fs__unparse_footer, when:
+
+ target arch: x86
+ apr_off_t: 4 bytes
+
+if the "APR_OFF_T_FMT" is "lld", it still use type "apr_off_t" to pass
+data to apr, but in apr source code file apr_snprintf.c the function
+apr_vformatter meet "lld", it would use the:
+
+ i_quad = va_arg(ap, apr_int64_t);
+
+It uses the apr_int64_t to deal data, it read 8 bytes, so the follow-up
+data may be error.
+
+Upstream-Status: Pending
+
+Signed-off-by: Dengke Du <dengke.du@windriver.com>
+---
+ subversion/libsvn_fs_fs/low_level.c | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/subversion/libsvn_fs_fs/low_level.c b/subversion/libsvn_fs_fs/low_level.c
+index a27bbcc..6ddbe28 100644
+--- a/subversion/libsvn_fs_fs/low_level.c
++++ b/subversion/libsvn_fs_fs/low_level.c
+@@ -250,10 +250,10 @@ svn_fs_fs__unparse_footer(apr_off_t l2p_offset,
+ {
+ return svn_stringbuf_createf(result_pool,
+ "%" APR_OFF_T_FMT " %s %" APR_OFF_T_FMT " %s",
+- l2p_offset,
++ (APR_OFF_T_FMT=="lld") ? (apr_int64_t)l2p_offset : l2p_offset,
+ svn_checksum_to_cstring(l2p_checksum,
+ scratch_pool),
+- p2l_offset,
++ (APR_OFF_T_FMT=="lld") ? (apr_int64_t)p2l_offset : p2l_offset,
+ svn_checksum_to_cstring(p2l_checksum,
+ scratch_pool));
+ }
+--
+2.7.4
+
diff --git a/meta/recipes-devtools/subversion/subversion_1.9.4.bb b/meta/recipes-devtools/subversion/subversion_1.9.4.bb
index 3ce83c1..fa6a7f2 100644
--- a/meta/recipes-devtools/subversion/subversion_1.9.4.bb
+++ b/meta/recipes-devtools/subversion/subversion_1.9.4.bb
@@ -14,6 +14,7 @@ SRC_URI = "${APACHE_MIRROR}/${BPN}/${BPN}-${PV}.tar.bz2 \
file://disable_macos.patch \
file://serf.m4-Regex-modified-to-allow-D-in-paths.patch \
file://0001-Fix-libtool-name-in-configure.ac.patch \
+ file://0001-fix-svnadmin-create-fail-on-x86.patch \
"
SRC_URI[md5sum] = "29121a038f87641055a8183f49e9739f"
--
2.7.4
^ permalink raw reply related
* [PATCH 0/1] fix "svnadmin create" fail on x86
From: Dengke Du @ 2016-11-02 7:12 UTC (permalink / raw)
To: openembedded-core
The following changes since commit c3d2df883a9d6d5036277114339673656d89a728:
oeqa/selftest/kernel.py: Add new file destined for kernel related tests (2016-11-01 10:05:46 +0000)
are available in the git repository at:
git://git.openembedded.org/openembedded-core-contrib dengke/fix-svnadmin-create-fail
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=dengke/fix-svnadmin-create-fail
Dengke Du (1):
fix "svnadmin create" fail on x86
.../0001-fix-svnadmin-create-fail-on-x86.patch | 56 ++++++++++++++++++++++
.../subversion/subversion_1.9.4.bb | 1 +
2 files changed, 57 insertions(+)
create mode 100644 meta/recipes-devtools/subversion/subversion/0001-fix-svnadmin-create-fail-on-x86.patch
--
2.7.4
^ permalink raw reply
* [PATCH 1/1] classes/populate_sdk_ext: require uninative
From: Paul Eggleton @ 2016-11-02 3:16 UTC (permalink / raw)
To: openembedded-core
In-Reply-To: <cover.1478056518.git.paul.eggleton@linux.intel.com>
It seems that possibly due to OE-Core commit
ac59063bee0e32d0737340974f657341717a6abe, binaries produced without
uninative aren't compatible with the uninative glibc. I did try earlier
to ensure that the eSDK could work without uninative since the default
configuration in OE-Core does not enable it, but it seems like I didn't
go far enough. Given the practical considerations, just give up and
require uninative to be enabled in order to build the eSDK. I'm not
particularly happy about this, but I don't seem much of an alternative.
Fixes [YOCTO #10566].
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
meta/classes/populate_sdk_ext.bbclass | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/meta/classes/populate_sdk_ext.bbclass b/meta/classes/populate_sdk_ext.bbclass
index 0f0525d..61e00af 100644
--- a/meta/classes/populate_sdk_ext.bbclass
+++ b/meta/classes/populate_sdk_ext.bbclass
@@ -114,10 +114,6 @@ def create_filtered_tasklist(d, sdkbasepath, tasklistfile, conf_initpath):
f.write('SSTATE_MIRRORS_forcevariable = ""\n')
# Ensure TMPDIR is the default so that clean_esdk_builddir() can delete it
f.write('TMPDIR_forcevariable = "${TOPDIR}/tmp"\n')
- # Drop uninative if the build isn't using it (or else NATIVELSBSTRING will
- # be different and we won't be able to find our native sstate)
- if not bb.data.inherits_class('uninative', d):
- f.write('INHERIT_remove = "uninative"\n')
# Unfortunately the default SDKPATH (or even a custom value) may contain characters that bitbake
# will not allow in its COREBASE path, so we need to rename the directory temporarily
@@ -608,6 +604,9 @@ fakeroot python do_populate_sdk_ext() {
if d.getVar('SDK_ARCH', True) != d.getVar('BUILD_ARCH', True):
bb.fatal('The extensible SDK can currently only be built for the same architecture as the machine being built on - SDK_ARCH is set to %s (likely via setting SDKMACHINE) which is different from the architecture of the build machine (%s). Unable to continue.' % (d.getVar('SDK_ARCH', True), d.getVar('BUILD_ARCH', True)))
+ if not bb.data.inherits_class('uninative', d):
+ bb.fatal('The extensible SDK requires uninative to be enabled. Enabling this is straightforward - just add the following to your configuration:\n\nrequire meta/conf/distro/include/yocto-uninative.inc\nINHERIT += "uninative"\n')
+
d.setVar('SDK_INSTALL_TARGETS', get_sdk_install_targets(d))
buildtools_fn = get_current_buildtools(d)
d.setVar('SDK_REQUIRED_UTILITIES', get_sdk_required_utilities(buildtools_fn, d))
--
2.5.5
^ permalink raw reply related
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox