Openembedded Core Discussions
 help / color / mirror / Atom feed
* Re: [PATCH v3 1/3] arch-arm64.inc: include arch-armv7ve instead of arch-armv7a
From: Khem Raj @ 2016-11-21 16:36 UTC (permalink / raw)
  To: Herve Jourdain; +Cc: Patches and discussions about the oe-core layer
In-Reply-To: <000601d243b5$e526ea00$af74be00$@neuf.fr>

On Sun, Nov 20, 2016 at 9:12 PM, Herve Jourdain <herve.jourdain@neuf.fr> wrote:
> Hi Khem,
>
> Do you want me to resubmit the patch with a better commit message?
> Or do you want to submit yours instead?
> I'm fine either way.

Dont worry I will submit the one in my tree and add your sign off. if
you test that out.

>
> Herve
>
> -----Original Message-----
> From: Khem Raj [mailto:raj.khem@gmail.com]
> Sent: dimanche 20 novembre 2016 20:35
> To: Herve Jourdain <herve.jourdain@neuf.fr>
> Cc: Patches and discussions about the oe-core layer <openembedded-core@lists.openembedded.org>
> Subject: Re: [OE-core] [oe-core][PATCH v3 1/3] arch-arm64.inc: include arch-armv7ve instead of arch-armv7a
>
> On Sun, Nov 20, 2016 at 4:10 AM, Herve Jourdain <herve.jourdain@neuf.fr> wrote:
>> Signed-off-by: Herve Jourdain <herve.jourdain@neuf.fr>
>> ---
>>  meta/conf/machine/include/arm/arch-arm64.inc | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/meta/conf/machine/include/arm/arch-arm64.inc b/meta/conf/machine/include/arm/arch-arm64.inc
>> index 9440698..9eeffac 100644
>> --- a/meta/conf/machine/include/arm/arch-arm64.inc
>> +++ b/meta/conf/machine/include/arm/arch-arm64.inc
>> @@ -1,6 +1,6 @@
>>  DEFAULTTUNE ?= "aarch64"
>>
>> -require conf/machine/include/arm/arch-armv7a.inc
>> +require conf/machine/include/arm/arch-armv7ve.inc
>
> This is ok. However a better commit message would be justified.
> I have similar patch see
>
> https://github.com/kraj/openembedded-core/commit/8ece89ac164062601d3418576d51434c69654e28
>
>>
>>  TUNEVALID[aarch64] = "Enable instructions for aarch64"
>>
>> --
>> 2.7.4
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>


^ permalink raw reply

* Re: [PATCH 2/5] taglib: Update to 1.11.1
From: Burton, Ross @ 2016-11-21 17:57 UTC (permalink / raw)
  To: Khem Raj; +Cc: OE-core
In-Reply-To: <20161116174748.26994-2-raj.khem@gmail.com>

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

On 16 November 2016 at 17:47, Khem Raj <raj.khem@gmail.com> wrote:

> -DEPENDS = "zlib"
> +DEPENDS = "zlib boost"
>

This appears to be an optional dependency and pulls in all of boost for a
byte swapping macro, which whilst I appreciate is in the spirit of C++
isn't great considering GCC has its own macros that it also checks for.

Can we just fix the tests so they don't error if boost isn't present and
force it to off?

Ross

[-- Attachment #2: Type: text/html, Size: 939 bytes --]

^ permalink raw reply

* [PATCH 1/1] libxml-parser-perl: add expat/Makefile dependency
From: Joe Slater @ 2016-11-21 19:04 UTC (permalink / raw)
  To: openembedded-core

The file Expat.bs (which is empty!) is created by BOOTSTRAP,
but is deleted by INST_DYNAMIC.  Make sure we are not creating
and deleting it at the same time.

Signed-off-by: Joe Slater <jslater@windriver.com>
---
 .../perl/libxml-parser-perl_2.44.bb                |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-devtools/perl/libxml-parser-perl_2.44.bb b/meta/recipes-devtools/perl/libxml-parser-perl_2.44.bb
index 3acd826..ab9e5fa 100644
--- a/meta/recipes-devtools/perl/libxml-parser-perl_2.44.bb
+++ b/meta/recipes-devtools/perl/libxml-parser-perl_2.44.bb
@@ -20,6 +20,8 @@ inherit cpan
 do_configure_append() {
 	sed 's:--sysroot=.*\(\s\|$\):--sysroot=${STAGING_DIR_TARGET} :g' -i Makefile Expat/Makefile
 	sed 's:^FULL_AR = .*:FULL_AR = ${AR}:g' -i Expat/Makefile
+	# make sure these two do not build in parallel
+	sed 's!^$(INST_DYNAMIC):!$(INST_DYNAMIC): $(BOOTSTRAP)!' -i Expat/Makefile
 }
 
 do_compile() {
-- 
1.7.9.5



^ permalink raw reply related

* [PATCH] pigz: Update to version 2.3.4
From: Fabio Berton @ 2016-11-21 19:58 UTC (permalink / raw)
  To: openembedded-core

  - Change SRC_URI to fetch sources from Github archives. Pigz official
    site maintains only the latest tarball version and Github have all
    versions avaiable to download.

  - Remove patch link-order.patch. The following commit fix this issue:
    https://github.com/madler/pigz/commit/c3f91d1a12a6f

Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
---
 meta/recipes-extended/pigz/pigz/link-order.patch | 38 ------------------------
 meta/recipes-extended/pigz/pigz_2.3.3.bb         | 14 ---------
 meta/recipes-extended/pigz/pigz_2.3.4.bb         | 12 ++++++++
 3 files changed, 12 insertions(+), 52 deletions(-)
 delete mode 100644 meta/recipes-extended/pigz/pigz/link-order.patch
 delete mode 100644 meta/recipes-extended/pigz/pigz_2.3.3.bb
 create mode 100644 meta/recipes-extended/pigz/pigz_2.3.4.bb

diff --git a/meta/recipes-extended/pigz/pigz/link-order.patch b/meta/recipes-extended/pigz/pigz/link-order.patch
deleted file mode 100644
index 4becc0e..0000000
--- a/meta/recipes-extended/pigz/pigz/link-order.patch
+++ /dev/null
@@ -1,38 +0,0 @@
-This patch avoids underlinking issues since we pass -lz via LDFLAGS but it appears
-before pigz.o which needs symbols from libz however due to order linker happily discards libz
-
-Upstream-Status: Pending
-
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
-
-Index: pigz-2.3.3/Makefile
-===================================================================
---- pigz-2.3.3.orig/Makefile	2015-01-19 20:12:31.000000000 -0800
-+++ pigz-2.3.3/Makefile	2015-01-28 09:11:50.266669184 -0800
-@@ -5,7 +5,7 @@
- # use gcc and gmake on Solaris
- 
- pigz: pigz.o yarn.o try.o ${ZOPFLI}deflate.o ${ZOPFLI}blocksplitter.o ${ZOPFLI}tree.o ${ZOPFLI}lz77.o ${ZOPFLI}cache.o ${ZOPFLI}hash.o ${ZOPFLI}util.o ${ZOPFLI}squeeze.o ${ZOPFLI}katajainen.o
--	$(CC) $(LDFLAGS) -o pigz $^ -lpthread -lm
-+	$(CC) -o pigz $^ $(LDFLAGS) -lz -lpthread -lm
- 	ln -f pigz unpigz
- 
- pigz.o: pigz.c yarn.h try.h ${ZOPFLI}deflate.h ${ZOPFLI}util.h
-@@ -35,7 +35,7 @@
- dev: pigz pigzt pigzn
- 
- pigzt: pigzt.o yarnt.o try.o ${ZOPFLI}deflate.o ${ZOPFLI}blocksplitter.o ${ZOPFLI}tree.o ${ZOPFLI}lz77.o ${ZOPFLI}cache.o ${ZOPFLI}hash.o ${ZOPFLI}util.o ${ZOPFLI}squeeze.o ${ZOPFLI}katajainen.o
--	$(CC) $(LDFLAGS) -o pigzt $^ -lpthread -lm
-+	$(CC) -o pigzt $^ $(LDFLAGS) -lz -lpthread -lm
- 
- pigzt.o: pigz.c yarn.h try.h
- 	$(CC) $(CFLAGS) -DDEBUG -g -c -o pigzt.o pigz.c
-@@ -44,7 +44,7 @@
- 	$(CC) $(CFLAGS) -DDEBUG -g -c -o yarnt.o yarn.c
- 
- pigzn: pigzn.o tryn.o ${ZOPFLI}deflate.o ${ZOPFLI}blocksplitter.o ${ZOPFLI}tree.o ${ZOPFLI}lz77.o ${ZOPFLI}cache.o ${ZOPFLI}hash.o ${ZOPFLI}util.o ${ZOPFLI}squeeze.o ${ZOPFLI}katajainen.o
--	$(CC) $(LDFLAGS) -o pigzn $^ -lm
-+	$(CC) -o pigzn $^ $(LDFLAGS) -lz -lm
- 
- pigzn.o: pigz.c try.h
- 	$(CC) $(CFLAGS) -DDEBUG -DNOTHREAD -g -c -o pigzn.o pigz.c
diff --git a/meta/recipes-extended/pigz/pigz_2.3.3.bb b/meta/recipes-extended/pigz/pigz_2.3.3.bb
deleted file mode 100644
index b4fe831..0000000
--- a/meta/recipes-extended/pigz/pigz_2.3.3.bb
+++ /dev/null
@@ -1,14 +0,0 @@
-require pigz.inc
-LIC_FILES_CHKSUM = "file://pigz.c;beginline=7;endline=21;md5=a21d4075cb00ab4ca17fce5e7534ca95"
-
-UPSTREAM_CHECK_URI = "http://zlib.net/${BPN}/"
-SRC_URI = "http://downloads.yoctoproject.org/mirror/sources/${BP}.tar.gz"
-SRC_URI += "file://link-order.patch"
-
-SRC_URI[md5sum] = "01d7a16cce77929cc1a78aa1bdfb68cb"
-SRC_URI[sha256sum] = "4e8b67b432ce7907575a549f3e1cac4709781ba0f6b48afea9f59369846b509c"
-
-NATIVE_PACKAGE_PATH_SUFFIX = "/${PN}"
-
-BBCLASSEXTEND = "native nativesdk"
-
diff --git a/meta/recipes-extended/pigz/pigz_2.3.4.bb b/meta/recipes-extended/pigz/pigz_2.3.4.bb
new file mode 100644
index 0000000..49d9d78
--- /dev/null
+++ b/meta/recipes-extended/pigz/pigz_2.3.4.bb
@@ -0,0 +1,12 @@
+require pigz.inc
+LIC_FILES_CHKSUM = "file://pigz.c;beginline=7;endline=21;md5=a21d4075cb00ab4ca17fce5e7534ca95"
+
+UPSTREAM_CHECK_URI = "http://zlib.net/${BPN}/"
+SRC_URI = "https://github.com/madler/pigz/archive/v${PV}.tar.gz"
+SRC_URI[md5sum] = "c109057050b15edf3eb9bb4d0805235e"
+SRC_URI[sha256sum] = "763f2fdb203aa0b7b640e63385e38e5dd4e5aaa041bc8e42aa96f2ef156b06e8"
+
+NATIVE_PACKAGE_PATH_SUFFIX = "/${PN}"
+
+BBCLASSEXTEND = "native nativesdk"
+
-- 
2.1.4



^ permalink raw reply related

* [PATCH] nettle: Update to version 3.3
From: Fabio Berton @ 2016-11-21 20:17 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
---
 meta/recipes-support/nettle/{nettle_3.2.bb => nettle_3.3.bb} | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-support/nettle/{nettle_3.2.bb => nettle_3.3.bb} (75%)

diff --git a/meta/recipes-support/nettle/nettle_3.2.bb b/meta/recipes-support/nettle/nettle_3.3.bb
similarity index 75%
rename from meta/recipes-support/nettle/nettle_3.2.bb
rename to meta/recipes-support/nettle/nettle_3.3.bb
index 2b79779..c723e27 100644
--- a/meta/recipes-support/nettle/nettle_3.2.bb
+++ b/meta/recipes-support/nettle/nettle_3.3.bb
@@ -7,5 +7,5 @@ LIC_FILES_CHKSUM = "file://COPYING.LESSERv3;md5=6a6a8e020838b23406c81b19c1d46df6
                     file://serpent-decrypt.c;beginline=14;endline=36;md5=ca0d220bc413e1842ecc507690ce416e \
                     file://serpent-set-key.c;beginline=14;endline=36;md5=ca0d220bc413e1842ecc507690ce416e"
 
-SRC_URI[md5sum] = "afb15b4764ebf1b4e6d06c62bd4d29e4"
-SRC_URI[sha256sum] = "ea4283def236413edab5a4cf9cf32adf540c8df1b9b67641cfc2302fca849d97"
+SRC_URI[md5sum] = "10f969f78a463704ae73529978148dbe"
+SRC_URI[sha256sum] = "46942627d5d0ca11720fec18d81fc38f7ef837ea4197c1f630e71ce0d470b11e"
-- 
2.1.4



^ permalink raw reply related

* [PATCH] libgpg-error: Update to version 1.25
From: Fabio Berton @ 2016-11-21 20:46 UTC (permalink / raw)
  To: openembedded-core

License checksum changes are not related to license changes.

Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
---
 .../libgpg-error/{libgpg-error_1.24.bb => libgpg-error_1.25.bb}    | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)
 rename meta/recipes-support/libgpg-error/{libgpg-error_1.24.bb => libgpg-error_1.25.bb} (86%)

diff --git a/meta/recipes-support/libgpg-error/libgpg-error_1.24.bb b/meta/recipes-support/libgpg-error/libgpg-error_1.25.bb
similarity index 86%
rename from meta/recipes-support/libgpg-error/libgpg-error_1.24.bb
rename to meta/recipes-support/libgpg-error/libgpg-error_1.25.bb
index 84a882a..9d402ff 100644
--- a/meta/recipes-support/libgpg-error/libgpg-error_1.24.bb
+++ b/meta/recipes-support/libgpg-error/libgpg-error_1.25.bb
@@ -5,7 +5,7 @@ BUGTRACKER = "https://bugs.g10code.com/gnupg/index"
 LICENSE = "GPLv2+ & LGPLv2.1+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552 \
                     file://COPYING.LIB;md5=2d5025d4aa3495befef8f17206a5b0a1 \
-                    file://src/gpg-error.h.in;endline=23;md5=cf562f60b9dba7df20d6ee3f97ea1d5a \
+                    file://src/gpg-error.h.in;endline=23;md5=64af9846baaf852793fd3a5af393acbf \
                     file://src/init.c;endline=20;md5=872b2389fe9bae7ffb80d2b91225afbc"
 
 
@@ -15,8 +15,9 @@ UPSTREAM_CHECK_URI = "https://gnupg.org/download/index.html"
 SRC_URI = "${GNUPG_MIRROR}/libgpg-error/libgpg-error-${PV}.tar.bz2 \
            file://pkgconfig.patch \
 	  "
-SRC_URI[md5sum] = "feb42198c0aaf3b28eabe8f41a34b983"
-SRC_URI[sha256sum] = "9268e1cc487de5e6e4460fca612a06e4f383072ac43ae90603e5e46783d3e540"
+
+SRC_URI[md5sum] = "d9fa545922a5060cbfbd87464bc31686"
+SRC_URI[sha256sum] = "f628f75843433b38b05af248121beb7db5bd54bb2106f384edac39934261320c"
 
 BINCONFIG = "${bindir}/gpg-error-config"
 
-- 
2.1.4



^ permalink raw reply related

* [PATCHv2] apt-package: Include maintenance scripts
From: Linus Wallgren @ 2016-11-21 20:57 UTC (permalink / raw)
  To: openembedded-core; +Cc: Linus Wallgren
In-Reply-To: <68b087d8-d920-eb83-8b53-7e561fd419cb@opendreambox.org>

Apt can run multiple tasks daily, such as for example clean, update,
autoclean, unattended-upgrades etc.

[YOCTO #10669]

Signed-off-by: Linus Wallgren <linus.wallgren@scypho.com>
---
 meta/recipes-devtools/apt/apt-package.inc | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/apt/apt-package.inc b/meta/recipes-devtools/apt/apt-package.inc
index a553aa2..c17048a 100644
--- a/meta/recipes-devtools/apt/apt-package.inc
+++ b/meta/recipes-devtools/apt/apt-package.inc
@@ -32,13 +32,19 @@ FILES_${PN} = "${bindir}/apt-cdrom ${bindir}/apt-get \
 	       ${bindir}/apt-config ${bindir}/apt-cache \
 	       ${libdir}/apt ${libdir}/libapt*.so.* \
 	       ${localstatedir} ${sysconfdir} \
-	       ${libdir}/dpkg"
+	       ${libdir}/dpkg \
+	       ${systemd_unitdir}/system \
+           "
 FILES_${PN}-utils = "${bindir}/apt-sortpkgs ${bindir}/apt-extracttemplates"
 FILES_${PN}-doc = "${@get_files_apt_doc(d, bb, d.getVar('apt-manpages', True))} \
 		   ${docdir}/apt"
 FILES_${PN}-utils-doc = "${@get_files_apt_doc(d, bb, d.getVar('apt-utils-manpages', True))}"
 FILES_${PN}-dev = "${libdir}/libapt*.so ${includedir}"
 
+inherit systemd
+
+SYSTEMD_SERVICE_${PN} = "apt-daily.timer"
+
 do_install () {
 	set -x
 	install -d ${D}${bindir}
@@ -75,4 +81,11 @@ do_install () {
 
 	install -d ${D}${includedir}/apt-pkg/
 	install -m 0644 include/apt-pkg/*.h ${D}${includedir}/apt-pkg/
+
+	install -d ${D}${systemd_unitdir}/system/
+	install -m 0644 ${S}/debian/apt.systemd.daily ${D}/usr/lib/apt/
+	install -m 0644 ${S}/debian/apt-daily.service ${D}${systemd_unitdir}/system/
+	install -m 0644 ${S}/debian/apt-daily.timer ${D}${systemd_unitdir}/system/
+	install -d ${D}${sysconfdir}/cron.daily/
+	install -m 0755 ${S}/debian/apt.apt-compat.cron.daily ${D}${sysconfdir}/cron.daily/
 }
-- 
2.10.2



^ permalink raw reply related

* [PATCH] buildhistory-collect-srcrevs: write an srcrev when named, not None
From: Christopher Larson @ 2016-11-21 22:06 UTC (permalink / raw)
  To: openembedded-core; +Cc: Paul Eggleton, Christopher Larson

From: Christopher Larson <chris_larson@mentor.com>

The script was writing the main 'SRCREV' value for SRCREV_name, not the value
of the latter. In the case of recipes without 'SRCREV', like linux-yocto, it
was writing an srcrev of 'None' for all of them.

Cc: Paul Eggleton <paul.eggleton@linux.intel.com>
Signed-off-by: Christopher Larson <chris_larson@mentor.com>
---
 scripts/buildhistory-collect-srcrevs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/buildhistory-collect-srcrevs b/scripts/buildhistory-collect-srcrevs
index 8a03580..d375b04 100755
--- a/scripts/buildhistory-collect-srcrevs
+++ b/scripts/buildhistory-collect-srcrevs
@@ -101,7 +101,7 @@ def main():
                 for name, value in srcrevs.items():
                     orig = orig_srcrevs.get(name, orig_srcrev)
                     if options.reportall or value != orig:
-                        all_srcrevs[curdir].append((pn, name, srcrev))
+                        all_srcrevs[curdir].append((pn, name, value))
 
     for curdir, srcrevs in sorted(all_srcrevs.items()):
         if srcrevs:
-- 
2.8.0



^ permalink raw reply related

* Re: [PATCH] sanity: removed broken compiler check
From: Christopher Larson @ 2016-11-22  3:22 UTC (permalink / raw)
  To: Enrico Scholz
  Cc: Enrico Scholz, Patches and discussions about the oe-core layer
In-Reply-To: <1479296396-724-1-git-send-email-enrico.scholz@sigma-chemnitz.de>

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

On Wed, Nov 16, 2016 at 4:39 AM, Enrico Scholz <
enrico.scholz@sigma-chemnitz.de> wrote:

> From: Enrico Scholz <enrico.scholz@ensc.de>
>
> The compiler check is broken because it checks for the existance of
> ${CC} as a file.  This fails with ccache with
>
> |    Please install the following missing utilities: C Compiler (ccache
> gcc ),C++ Compiler (ccache g++ )
>
> Remove this check for now.
>
> Signed-off-by: Enrico Scholz <enrico.scholz@ensc.de>
> ---
>  meta/classes/sanity.bbclass | 6 ------
>  1 file changed, 6 deletions(-)
>
> diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
> index 7388da6..46f54df 100644
> --- a/meta/classes/sanity.bbclass
> +++ b/meta/classes/sanity.bbclass
> @@ -669,12 +669,6 @@ def check_sanity_version_change(status, d):
>      if not check_app_exists("${MAKE}", d):
>          missing = missing + "GNU make,"
>
> -    if not check_app_exists('${BUILD_CC}', d):
> -        missing = missing + "C Compiler (%s)," % d.getVar("BUILD_CC",
> True)
> -
> -    if not check_app_exists('${BUILD_CXX}', d):
> -        missing = missing + "C++ Compiler (%s)," % d.getVar("BUILD_CXX",
> True)
> -
>

Is there a reason check_app_exists() wasn’t fixed to handle symlinks
instead?
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics

[-- Attachment #2: Type: text/html, Size: 2130 bytes --]

^ permalink raw reply

* Re: [PATCH 2/5] taglib: Update to 1.11.1
From: Khem Raj @ 2016-11-22  3:46 UTC (permalink / raw)
  To: Burton, Ross; +Cc: OE-core
In-Reply-To: <CAJTo0Laq+KdjYoNvTM3DV+X8aGGdfWyJM=1z830=8T1orbqaJQ@mail.gmail.com>


[-- Attachment #1.1: Type: text/plain, Size: 695 bytes --]



On 11/21/16 9:57 AM, Burton, Ross wrote:
> On 16 November 2016 at 17:47, Khem Raj <raj.khem@gmail.com
> <mailto:raj.khem@gmail.com>> wrote:
> 
>     -DEPENDS = "zlib"
>     +DEPENDS = "zlib boost"
> 
> 
> This appears to be an optional dependency and pulls in all of boost for a byte
> swapping macro, which whilst I appreciate is in the spirit of C++ isn't great
> considering GCC has its own macros that it also checks for.
> 
> Can we just fix the tests so they don't error if boost isn't present and force
> it to off?

its a build time dependency. Do we have a usecase/images where taglib is used
and boost is just pulled into build because of taglib ?

> 
> Ross


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

^ permalink raw reply

* Re: [PATCH 1/5] systemd: Upgrade to 232
From: Khem Raj @ 2016-11-22  3:51 UTC (permalink / raw)
  To: Burton, Ross; +Cc: OE-core
In-Reply-To: <CAJTo0LYgH6_T5Y-YO1Px5eyYFLiud9SQvG1vHGRoebwkhVhdKg@mail.gmail.com>


[-- Attachment #1.1: Type: text/plain, Size: 1071 bytes --]



On 11/18/16 4:38 AM, Burton, Ross wrote:
> 
> On 16 November 2016 at 17:47, Khem Raj <raj.khem@gmail.com
> <mailto:raj.khem@gmail.com>> wrote:
> 
>     * Drop support for rcS.d SysV init scripts.
>       These are prone to cause dependency loops, and almost all packages with
>       rcS scripts now ship a native systemd service.
> 
>     * Drop mount propagation patch, it only happens with libseccomp, OE doesnt
>       enable it
> 
>     * kdbus option has disappeared from configure
> 
>     * Ignore dev-so for PN now since systemd introduced private .so see
>       https://github.com/systemd/systemd/issues/3810
>     <https://github.com/systemd/systemd/issues/3810>
> 
>     Signed-off-by: Khem Raj <raj.khem@gmail.com <mailto:raj.khem@gmail.com>>
> 
> 
> Because systemd-boot basically include the systemd recipe, this breaks
> systemd-boot.
> 
> Can we think up a way to handle systemd/systemd-boot without breaking every
> upgrade?

Is it a runtime break ? I just built systemd-boot for turbot/64bit successfully
> 
> Ross


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

^ permalink raw reply

* Contents of non-rootfs partitions
From: Kristian Amlie @ 2016-11-22  9:05 UTC (permalink / raw)
  To: openembedded-core

Hey

In Mender we are using a dual rootfs partition layout (A/B layout) with a persistent data partition on the side. We're using wic to do the actual image building.

However, putting files into this data partition is not a straightforward operation ATM. All recipes in OE put files into the rootfs only, and we'd like there to be a way to specify that files should go into the data partition.

I haven't seen any straightforward way to do this in OE. Some suggestions for how it could be done:

* A "DATA_FILES" variable (like "IMAGE_BOOT_FILES"), where you can list files you'd like to be included. This is very straightforward, but has the disadvantage of being a bit hairy when paths are involved, and it doesn't allow for much logic.

* Keep a special "data" directory, something similar to sysroot, where recipes could install files into in order to have it included on the data partition. We could potentially have a bbclass for use in recipes to make this more streamlined.

* Have a special recipe that users can bbappend, which specifically install to the data partition (this might also need a "data" directory, like above).

It's also possible that this should be generalized further so that it doesn't only apply to a data partition, but to any non-rootfs partition.

I'm willing to invest some time in this, but I thought I'd gather some opinions first.

Thoughts?

-- 
Kristian


^ permalink raw reply

* [PATCH] grub-efi: Force no-pie build
From: Ricardo Ribalda Delgado @ 2016-11-22  9:49 UTC (permalink / raw)
  To: openembedded-core, oss.burton, william.c.randle

PIE is enabled by default on the latest Debian LD. Disable it for
grub-efi. Otherwise:

| /usr/bin/ld: -r and -pie may not be used together
| collect2: error: ld returned 1 exit status
| Makefile:61382: recipe for target 'trig.module' failed
| make[3]: *** [trig.module] Error 1

Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>
---
 meta/recipes-bsp/grub/grub-efi_2.00.bb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-bsp/grub/grub-efi_2.00.bb b/meta/recipes-bsp/grub/grub-efi_2.00.bb
index 5a0dc954a3bf..9d07a31de7ce 100644
--- a/meta/recipes-bsp/grub/grub-efi_2.00.bb
+++ b/meta/recipes-bsp/grub/grub-efi_2.00.bb
@@ -2,12 +2,14 @@ require grub2.inc
 
 DEPENDS_class-target = "grub-efi-native"
 RDEPENDS_${PN}_class-target = "diffutils freetype"
-PR = "r3"
+PE = "1"
 
 SRC_URI += " \
            file://cfg \
           "
 
+BUILD_LDFLAGS += "-no-pie"
+
 S = "${WORKDIR}/grub-${PV}"
 
 # Determine the target arch for the grub modules
-- 
2.10.2



^ permalink raw reply related

* Re: Contents of non-rootfs partitions
From: Kristian Amlie @ 2016-11-22  9:54 UTC (permalink / raw)
  To: Yann Dirson; +Cc: openembedded-core
In-Reply-To: <CA+4=imZLMh7di712mejxQ6qgsDSXiCOJJDbUY99YC9DoCrqrmQ@mail.gmail.com>

On 22/11/16 10:27, Yann Dirson wrote:
> Since usually such a data partition needs to be written at times, you'll anyway have to handle the event of it getting corrupted, and possibly have to reformat it.
> With this in mind, one option is to include default contents in your rootfs, and populate the persistent one on first boot.

That's true, and a neat solution, but I consider this somewhat orthogonal. It solves the problem at run time, whereas my proposal tries to solve it at build time. With your solution, in a dual rootfs layout situation, you're paying the space cost of the initial data partition contents three times over, once in each of the two rootfs filesystems and once in the data filesystem. Depending on whether you have logic to detect a corrupt data partition or not, this price may not be worth paying if you can have a prepopulated data partition.

It also has higher complexity, particularly integrating the population step in the boot sequence. I'm not opposed to this solution, but maybe it makes sense to start smaller?

-- 
Kristian


^ permalink raw reply

* [PATCH 0/4] Persistent /var/log support
From: Chen Qi @ 2016-11-22 10:10 UTC (permalink / raw)
  To: openembedded-core

The following changes since commit a675b2c89e477af088faee9b3be96eae19a85f0b:

  sanity.bbclass: fix logging of an error (2016-11-15 15:18:50 +0000)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib ChenQi/persistent-var-log
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=ChenQi/persistent-var-log

Chen Qi (4):
  bitbake.conf: add VOLATILE_LOG_DIR variable
  base-files: respect VOLATILE_LOG_DIR
  initscripts: support persistent /var/log
  package.bbclass: support persistent /var/log

 meta/classes/package.bbclass                       |  2 +-
 meta/conf/bitbake.conf                             |  3 +
 meta/files/fs-perms-persistent-log.txt             | 69 ++++++++++++++++++++++
 meta/recipes-core/base-files/base-files_3.0.14.bb  |  4 +-
 .../initscripts/initscripts-1.0/volatiles          |  1 -
 meta/recipes-core/initscripts/initscripts_1.0.bb   |  3 +
 6 files changed, 78 insertions(+), 4 deletions(-)
 create mode 100644 meta/files/fs-perms-persistent-log.txt

-- 
1.9.1



^ permalink raw reply

* [PATCH 1/4] bitbake.conf: add VOLATILE_LOG_DIR variable
From: Chen Qi @ 2016-11-22 10:10 UTC (permalink / raw)
  To: openembedded-core
In-Reply-To: <cover.1479809328.git.Qi.Chen@windriver.com>

The default value is "yes" which results in the /var/log being a link
pointing to /var/volatile/log which is on tmpfs.

The user could override this value to "no" which causes /var/log to be
a directory on persistent storage.

[YOCTO #6132]

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 meta/conf/bitbake.conf | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 1472e8f..794f422 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -83,6 +83,9 @@ USRBINPATH_class-nativesdk = "/usr/bin"
 # Root home directory
 ROOT_HOME ??= "/home/root"
 
+# If set to "yes", /var/log links to /var/volatile/log; otherwise, /var/log is on persistent storage.
+VOLATILE_LOG_DIR ?= "yes"
+
 ##################################################################
 # Architecture-dependent build variables.
 ##################################################################
-- 
1.9.1



^ permalink raw reply related

* [PATCH 2/4] base-files: respect VOLATILE_LOG_DIR
From: Chen Qi @ 2016-11-22 10:10 UTC (permalink / raw)
  To: openembedded-core
In-Reply-To: <cover.1479809328.git.Qi.Chen@windriver.com>

Respect VOLATILE_LOG_DIR variable. In this way, if the user overrides
this variable to be "no", /var/log on the final image would reside on
persistent storage.

[YOCTO #6132]

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 meta/recipes-core/base-files/base-files_3.0.14.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/base-files/base-files_3.0.14.bb b/meta/recipes-core/base-files/base-files_3.0.14.bb
index 5333110..7cc8e8c 100644
--- a/meta/recipes-core/base-files/base-files_3.0.14.bb
+++ b/meta/recipes-core/base-files/base-files_3.0.14.bb
@@ -41,7 +41,7 @@ dirs755 = "/bin /boot /dev ${sysconfdir} ${sysconfdir}/default \
            ${localstatedir}/backups ${localstatedir}/lib \
            /sys ${localstatedir}/lib/misc ${localstatedir}/spool \
            ${localstatedir}/volatile \
-           ${localstatedir}/volatile/log \
+           ${localstatedir}/${@['', 'volatile/'][d.getVar('VOLATILE_LOG_DIR', True) == 'yes']}log \
            /home ${prefix}/src ${localstatedir}/local \
            /media"
 
@@ -52,7 +52,7 @@ dirs755-lsb = "/srv  \
                ${prefix}/lib/locale"
 dirs2775-lsb = "/var/mail"
 
-volatiles = "log tmp"
+volatiles = "${@['', 'log'][d.getVar('VOLATILE_LOG_DIR', True) == 'yes']} tmp"
 conffiles = "${sysconfdir}/debian_version ${sysconfdir}/host.conf \
              ${sysconfdir}/issue /${sysconfdir}/issue.net \
              ${sysconfdir}/nsswitch.conf ${sysconfdir}/profile \
-- 
1.9.1



^ permalink raw reply related

* [PATCH 3/4] initscripts: support persistent /var/log
From: Chen Qi @ 2016-11-22 10:10 UTC (permalink / raw)
  To: openembedded-core
In-Reply-To: <cover.1479809328.git.Qi.Chen@windriver.com>

Respect VOLATILE_VAR_LOG variable so that if it's set to "no", we could
have persistent /var/log on the final image.

[YOCTO #6132]

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 meta/recipes-core/initscripts/initscripts-1.0/volatiles | 1 -
 meta/recipes-core/initscripts/initscripts_1.0.bb        | 3 +++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/initscripts/initscripts-1.0/volatiles b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
index 297245d..6cccab7 100644
--- a/meta/recipes-core/initscripts/initscripts-1.0/volatiles
+++ b/meta/recipes-core/initscripts/initscripts-1.0/volatiles
@@ -25,7 +25,6 @@ d root root 1777 /run/lock none
 d root root 0755 /var/volatile/log none
 d root root 1777 /var/volatile/tmp none
 l root root 1777 /var/lock /run/lock
-l root root 0755 /var/log /var/volatile/log
 l root root 0755 /var/run /run
 l root root 1777 /var/tmp /var/volatile/tmp
 l root root 1777 /tmp /var/tmp
diff --git a/meta/recipes-core/initscripts/initscripts_1.0.bb b/meta/recipes-core/initscripts/initscripts_1.0.bb
index 8f110b0..14cdf6a 100644
--- a/meta/recipes-core/initscripts/initscripts_1.0.bb
+++ b/meta/recipes-core/initscripts/initscripts_1.0.bb
@@ -102,6 +102,9 @@ do_install () {
 	install -m 0755    ${WORKDIR}/read-only-rootfs-hook.sh ${D}${sysconfdir}/init.d
 	install -m 0755    ${WORKDIR}/save-rtc.sh	${D}${sysconfdir}/init.d
 	install -m 0644    ${WORKDIR}/volatiles		${D}${sysconfdir}/default/volatiles/00_core
+	if [ "${VOLATILE_LOG_DIR}" = "log" ]; then
+		echo "l root root 0755 /var/log /var/volatile/log" >> ${D}${sysconfdir}/default/volatiles/00_core
+	fi
 	install -m 0755    ${WORKDIR}/dmesg.sh		${D}${sysconfdir}/init.d
 	install -m 0644    ${WORKDIR}/logrotate-dmesg.conf ${D}${sysconfdir}/
 
-- 
1.9.1



^ permalink raw reply related

* [PATCH 4/4] package.bbclass: support persistent /var/log
From: Chen Qi @ 2016-11-22 10:10 UTC (permalink / raw)
  To: openembedded-core
In-Reply-To: <cover.1479809328.git.Qi.Chen@windriver.com>

Add a new file, fs-perms-persistent-log.txt, which treats /var/log
as a directory instead of a link.

Modify package.bbclass to use this file if VOLATILE_LOG_DIR is not set to "yes".

[YOCTO #6132]

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 meta/classes/package.bbclass           |  2 +-
 meta/files/fs-perms-persistent-log.txt | 69 ++++++++++++++++++++++++++++++++++
 2 files changed, 70 insertions(+), 1 deletion(-)
 create mode 100644 meta/files/fs-perms-persistent-log.txt

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index a6f0a7a..9a423f8 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -733,7 +733,7 @@ python fixup_perms () {
         bbpath = d.getVar('BBPATH', True)
         fs_perms_tables = d.getVar('FILESYSTEM_PERMS_TABLES', True)
         if not fs_perms_tables:
-            fs_perms_tables = 'files/fs-perms.txt'
+            fs_perms_tables = ['files/fs-perms-persistent-log.txt', 'files/fs-perms.txt'][d.getVar('VOLATILE_LOG_DIR', True) == 'yes']
         for conf_file in fs_perms_tables.split():
             str += " %s" % bb.utils.which(bbpath, conf_file)
         return str
diff --git a/meta/files/fs-perms-persistent-log.txt b/meta/files/fs-perms-persistent-log.txt
new file mode 100644
index 0000000..2487a68
--- /dev/null
+++ b/meta/files/fs-perms-persistent-log.txt
@@ -0,0 +1,69 @@
+# This file contains a list of files and directories with known permissions.
+# It is used by the packaging class to ensure that the permissions, owners and
+# group of listed files and directories are in sync across the system.
+#
+# The format of this file 
+#
+#<path>	<mode>	<uid>	<gid>	<walk>	<fmode>	<fuid>	<fgid>
+#
+# or
+#
+#<path> link <target>
+#
+# <path>: directory path
+# <mode>: mode for directory
+# <uid>:  uid for directory
+# <gid>:  gid for directory
+# <walk>: recursively walk the directory?  true or false
+# <fmode>: if walking, new mode for files
+# <fuid>:  if walking, new uid for files
+# <fgid>:  if walking, new gid for files
+# <target>: turn the directory into a symlink point to target
+#
+# in mode, uid or gid, a "-" means don't change any existing values
+#
+# /usr/src		0755	root	root	false	-	-	-
+# /usr/share/man	0755	root	root	true	0644	root	root
+
+# Note: all standard config directories are automatically assigned "0755 root root false - - -"
+
+# Documentation should always be corrected
+${mandir}		0755	root	root	true	0644	root	root
+${infodir}		0755	root	root	true	0644	root	root
+${docdir}		0755	root	root	true	0644	root	root
+${datadir}/gtk-doc	0755	root	root	true	0644	root	root
+
+# Fixup locales
+${datadir}/locale	0755	root	root	true	0644	root	root
+
+# Cleanup headers
+${includedir}		0755	root	root	true	0644	root	root
+${oldincludedir}	0755	root	root	true	0644	root	root
+
+# Cleanup debug src
+/usr/src/debug		0755	root	root	true	-	root	root
+
+# Items from base-files
+# Links
+${localstatedir}/run	link	/run
+${localstatedir}/lock	link	/run/lock
+${localstatedir}/tmp	link	volatile/tmp
+
+/home				0755	root	root	false - - -
+/srv				0755	root	root	false - - -
+${prefix}/src			0755	root	root	false - - -
+${localstatedir}/local		0755	root	root	false - - -
+
+# Special permissions from base-files
+# Set 1777
+/tmp				01777	root	root	false - - -
+${localstatedir}/volatile/tmp	01777	root	root	false - - -
+
+# Set 0700
+${ROOT_HOME}			0700	root	root	false - - -
+
+# Set 755-lsb
+/srv				0755	root	root	false - - -
+
+# Set 2775-lsb
+/var/mail			02775	root	mail	false - - -
-- 
1.9.1



^ permalink raw reply related

* Re: [PATCH 2/5] taglib: Update to 1.11.1
From: Burton, Ross @ 2016-11-22 10:44 UTC (permalink / raw)
  To: Khem Raj; +Cc: OE-core
In-Reply-To: <eb6aeb51-c0c5-fe39-f3e1-ac1dfede129b@gmail.com>

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

On 22 November 2016 at 03:46, Khem Raj <raj.khem@gmail.com> wrote:

> its a build time dependency. Do we have a usecase/images where taglib is
> used
> and boost is just pulled into build because of taglib ?
>

No but I've an example where boost doesnt build at all and this breaks
core-image-sato (IA x32)...

Building all of boost just for a macro seems rather unfortunate.

Ross

[-- Attachment #2: Type: text/html, Size: 864 bytes --]

^ permalink raw reply

* Re: Contents of non-rootfs partitions
From: Patrick Ohly @ 2016-11-22 11:10 UTC (permalink / raw)
  To: Kristian Amlie, Eduard Bartosh; +Cc: openembedded-core
In-Reply-To: <a90e8685-f087-5ee2-888b-aa537ebd94c4@mender.io>

On Tue, 2016-11-22 at 10:05 +0100, Kristian Amlie wrote:
> Hey
> 
> In Mender we are using a dual rootfs partition layout (A/B layout)
> with a persistent data partition on the side. We're using wic to do
> the actual image building.
> 
> However, putting files into this data partition is not a
> straightforward operation ATM. All recipes in OE put files into the
> rootfs only, and we'd like there to be a way to specify that files
> should go into the data partition.
> 
> I haven't seen any straightforward way to do this in OE. Some
> suggestions for how it could be done:
> 
> * A "DATA_FILES" variable (like "IMAGE_BOOT_FILES"), where you can
> list files you'd like to be included. This is very straightforward,
> but has the disadvantage of being a bit hairy when paths are involved,
> and it doesn't allow for much logic.
> 
> * Keep a special "data" directory, something similar to sysroot, where
> recipes could install files into in order to have it included on the
> data partition. We could potentially have a bbclass for use in recipes
> to make this more streamlined.
> 
> * Have a special recipe that users can bbappend, which specifically
> install to the data partition (this might also need a "data"
> directory, like above).

All of these introduce some special mechanism. Let me propose something
that might integrate better with the existing tooling:

The "rootfs" directory gets redefined as representing the entire virtual
file system. When creating a disk image, it gets split up into different
partitions based on the image configuration.

For example, the /home or /data directories in the rootfs could hold the
content that in some image configurations goes into separate partitions.

The advantage of this approach is that the tooling for staging content
for image creation does not need to be changed. The same staged content
then can be used to create different images, potentially even using
different partition layouts.

To implement this approach with wic, wic needs to be taught how to
exclude directories from the main rootfs. Ideally, the mkfs.* tools
should also support that without having to make an intermediate copy of
the files for a certain partition, but initially wic could create
temporary directory trees.

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.





^ permalink raw reply

* Re: [PATCH] sanity: removed broken compiler check
From: Enrico Scholz @ 2016-11-22 11:11 UTC (permalink / raw)
  To: Christopher Larson; +Cc: Patches and discussions about the oe-core layer
In-Reply-To: <CABcZANmqH9gEhO67-9_0fEinU6FhMxRsgaC0hcVrkxd9_cH1HA@mail.gmail.com>

Christopher Larson <clarson@kergoth.com> writes:

>> |    Please install the following missing utilities: C Compiler (ccache gcc ),C++ Compiler (ccache g++ )
>>
>> Remove this check for now.
>> [...]
>> +++ b/meta/classes/sanity.bbclass
>> [...]
>> -    if not check_app_exists('${BUILD_CC}', d):
>> -        missing = missing + "C Compiler (%s)," % d.getVar("BUILD_CC", True)
>> -
>> -    if not check_app_exists('${BUILD_CXX}', d):
>> -        missing = missing + "C++ Compiler (%s)," % d.getVar("BUILD_CXX", True)
>> -
>>
>
> Is there a reason check_app_exists() wasn’t fixed to handle symlinks
> instead?

This will not help. 'check_app_exist()' checks for the existence of a
file (e.g. like 'test -x "${BUILD_CC}"' with iterating $PATH).  But
'ccache gcc' is not a single file but a chain of files.

A more reliable method would be to call the program with '--version' or
so and check the exit code.

But gcc and g++ are elementary tools which are existing very likely on
every developers machine and which will not introduce silent and difficulty
to detect breakage.  So I removed the check for now instead to implement
the method above.


Enrico


^ permalink raw reply

* [PATCH] taglib: Update to 1.11.1
From: Ross Burton @ 2016-11-22 11:19 UTC (permalink / raw)
  To: openembedded-core

Forcibly disable Boost as the macros it looks for are also present in our C++
standard library.

Enable c++11 in cmake as well as in CXXFLAGS [Khem Raj]

CXXFLAGS is required due to a cmake bug where it does not honor CMAKE_CXX_FLAGS
during configure time checks [Khem Raj]

Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 .../taglib/{taglib_1.11.bb => taglib_1.11.1.bb}           | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)
 rename meta/recipes-support/taglib/{taglib_1.11.bb => taglib_1.11.1.bb} (68%)

diff --git a/meta/recipes-support/taglib/taglib_1.11.bb b/meta/recipes-support/taglib/taglib_1.11.1.bb
similarity index 68%
rename from meta/recipes-support/taglib/taglib_1.11.bb
rename to meta/recipes-support/taglib/taglib_1.11.1.bb
index c29bda9..ee1b523 100644
--- a/meta/recipes-support/taglib/taglib_1.11.bb
+++ b/meta/recipes-support/taglib/taglib_1.11.1.bb
@@ -9,8 +9,8 @@ LIC_FILES_CHKSUM = "file://COPYING.LGPL;md5=4fbd65380cdd255951079008b364516c \
 DEPENDS = "zlib"
 
 SRC_URI = "http://taglib.github.io/releases/${BP}.tar.gz"
-SRC_URI[md5sum] = "be39fa2054df40664cb557126ad7cf7c"
-SRC_URI[sha256sum] = "ed4cabb3d970ff9a30b2620071c2b054c4347f44fc63546dbe06f97980ece288"
+SRC_URI[md5sum] = "cee7be0ccfc892fa433d6c837df9522a"
+SRC_URI[sha256sum] = "b6d1a5a610aae6ff39d93de5efd0fdc787aa9e9dc1e7026fa4c961b26563526b"
 
 UPSTREAM_CHECK_URI = "http://github.com/taglib/taglib/releases/"
 
@@ -21,13 +21,18 @@ inherit cmake pkgconfig binconfig-disabled
 PACKAGES =+ "${PN}-c"
 FILES_${PN}-c = "${libdir}/libtag_c.so.*"
 
-EXTRA_OECMAKE = "-DBUILD_SHARED_LIBS=ON -DLIB_SUFFIX=${@d.getVar('baselib', True).replace('lib', '')}"
+EXTRA_OECMAKE = "-DBUILD_SHARED_LIBS=ON \
+                 -DCMAKE_DISABLE_FIND_PACKAGE_Boost=TRUE \
+                 -DHAVE_BOOST_BYTESWAP=FALSE \
+                 -DCMAKE_CXX_STANDARD=11 \
+                 -DCMAKE_CXX_STANDARD_REQUIRED=OFF \
+                 -DLIB_SUFFIX=${@d.getVar('baselib', True).replace('lib', '')} \
+"
+CXXFLAGS += "-std=c++11"
 
 do_configure_prepend () {
 	rm -f ${S}/admin/ltmain.sh
 	rm -f ${S}/admin/libtool.m4.in
-	# Don't have a floating dependeny on boost
-	sed -i -e "s/atomic.hpp/atomic-not-exist.hpp/" ${S}/ConfigureChecks.cmake ${S}/taglib/toolkit/trefcounter.cpp
 }
 
 # without -fPIC depending packages failed with many error like:
-- 
2.8.1



^ permalink raw reply related

* Re: [PATCH 2/5] taglib: Update to 1.11.1
From: Burton, Ross @ 2016-11-22 11:20 UTC (permalink / raw)
  To: Khem Raj; +Cc: OE-core
In-Reply-To: <CAJTo0Lb7jorMUq-8D-tkBTK0bP45AOtP_tBS2PTGGxPrUWbc8g@mail.gmail.com>

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

On 22 November 2016 at 10:44, Burton, Ross <ross.burton@intel.com> wrote:

> No but I've an example where boost doesnt build at all and this breaks
> core-image-sato (IA x32)...
>
> Building all of boost just for a macro seems rather unfortunate.
>

Found the magic -D options to convince taglib that boost definitely isn't
present and verified that it does the same thing with and without boost in
the sysroot so I just sent a v2.

Ross

[-- Attachment #2: Type: text/html, Size: 879 bytes --]

^ permalink raw reply

* Re: [PATCH] sanity: removed broken compiler check
From: Burton, Ross @ 2016-11-22 11:42 UTC (permalink / raw)
  To: Christopher Larson
  Cc: Enrico Scholz, Enrico Scholz,
	Patches and discussions about the oe-core layer
In-Reply-To: <CABcZANmqH9gEhO67-9_0fEinU6FhMxRsgaC0hcVrkxd9_cH1HA@mail.gmail.com>

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

On 22 November 2016 at 03:22, Christopher Larson <clarson@kergoth.com>
wrote:

> Is there a reason check_app_exists() wasn’t fixed to handle symlinks
> instead?
>

Agreed, the check should handle splitting the argument and links, instead
of removing a key sanity check.

Ross

[-- Attachment #2: Type: text/html, Size: 721 bytes --]

^ permalink raw reply


This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox