[OE-core] [PATCH v3] mdadm: 3.3.2 -> 3.3.3

[Li xin]

Upgrade mdadm from 3.3.2 to 3.3.3

1) Remove backported patch:inline.patch

2) update context of mdadm-3.3.2_x32_abi_time_t.patch

3) Add systemd support:
add systemd service file mdmonitor.service and install conf example file.

Signed-off-by: Li Xin 
---
 meta/recipes-extended/mdadm/files/inline.patch | 39 --
 .../mdadm/files/mdadm-3.3.2_x32_abi_time_t.patch   | 26 +--
 .../mdadm/{mdadm_3.3.2.bb => mdadm_3.3.3.bb}   | 28 
 3 files changed, 37 insertions(+), 56 deletions(-)
 delete mode 100644 meta/recipes-extended/mdadm/files/inline.patch
 rename meta/recipes-extended/mdadm/{mdadm_3.3.2.bb => mdadm_3.3.3.bb} (66%)

diff --git a/meta/recipes-extended/mdadm/files/inline.patch 
b/meta/recipes-extended/mdadm/files/inline.patch
deleted file mode 100644
index 9009f06..000
--- a/meta/recipes-extended/mdadm/files/inline.patch
+++ /dev/null
@@ -1,39 +0,0 @@
-with gnu11 turning on a default with gcc-5 we have to abide by new inline 
semantics
-using 'inline' and using is in same translation unit means an external 
definition has to exist
-same like "external inline" in gnu89, we simply make the functions static 
inline since they
-are just meant to be used in this file
-
-Upstream-Status: Pending
-Signed-off-by: Khem Raj 
-
-Index: mdadm-3.3.2/bitmap.c
-===
 mdadm-3.3.2.orig/bitmap.c
-+++ mdadm-3.3.2/bitmap.c
-@@ -20,7 +20,7 @@
- 
- #include "mdadm.h"
- 
--inline void sb_le_to_cpu(bitmap_super_t *sb)
-+static inline void sb_le_to_cpu(bitmap_super_t *sb)
- {
-   sb->magic = __le32_to_cpu(sb->magic);
-   sb->version = __le32_to_cpu(sb->version);
-@@ -34,7 +34,7 @@ inline void sb_le_to_cpu(bitmap_super_t
-   sb->write_behind = __le32_to_cpu(sb->write_behind);
- }
- 
--inline void sb_cpu_to_le(bitmap_super_t *sb)
-+static inline void sb_cpu_to_le(bitmap_super_t *sb)
- {
-   sb_le_to_cpu(sb); /* these are really the same thing */
- }
-@@ -74,7 +74,7 @@ typedef struct bitmap_info_s {
- } bitmap_info_t;
- 
- /* count the dirty bits in the first num_bits of byte */
--inline int count_dirty_bits_byte(char byte, int num_bits)
-+static inline int count_dirty_bits_byte(char byte, int num_bits)
- {
-   int num = 0;
- 
diff --git a/meta/recipes-extended/mdadm/files/mdadm-3.3.2_x32_abi_time_t.patch 
b/meta/recipes-extended/mdadm/files/mdadm-3.3.2_x32_abi_time_t.patch
index 143a8d9..99e5ed4 100644
--- a/meta/recipes-extended/mdadm/files/mdadm-3.3.2_x32_abi_time_t.patch
+++ b/meta/recipes-extended/mdadm/files/mdadm-3.3.2_x32_abi_time_t.patch
@@ -1,26 +1,32 @@
-When try to print time_t values as a long int it causes an error because time_t
-data type in x32 ABI is long long int.
+From 72ea9bc213c0fcbc3a11bad6478645a68692af38 Mon Sep 17 00:00:00 2001
+From: Li xin 
+Date: Mon, 27 Jul 2015 00:27:12 +0900
+Subject: [PATCH] When try to print time_t values as a long int it causes an
+ error because time_t data type in x32 ABI is long long int.
 
 Upstream-Status: Pending
 
-Signed-off-by: Aníbal Limón 
+Signed-off-by: Li Xin 
+---
+ monitor.c | 4 
+ 1 file changed, 4 insertions(+)
 
 diff --git a/monitor.c b/monitor.c
-index f81e707..19ea041 100644
+index 870cc1a..068aab4 100644
 --- a/monitor.c
 +++ b/monitor.c
-@@ -260,8 +260,13 @@ static int read_and_act(struct active_array *a)
+@@ -259,7 +259,11 @@ static int read_and_act(struct active_array *a)
}
  
gettimeofday(&tv, NULL);
 +#if defined(__x86_64__) && defined(__ILP32__)
 +  dprintf("%s(%d): %lld.%06lld state:%s prev:%s action:%s prev: %s 
start:%llu\n",
 +#else
-   dprintf("%s(%d): %ld.%06ld state:%s prev:%s action:%s prev: %s 
start:%llu\n",
--  __func__, a->info.container_member,
+   dprintf("(%d): %ld.%06ld state:%s prev:%s action:%s prev: %s 
start:%llu\n",
 +#endif
-+  __func__,
-+  a->info.container_member,
+   a->info.container_member,
tv.tv_sec, tv.tv_usec,
array_states[a->curr_state],
-   array_states[a->prev_state],
+-- 
+1.8.4.2
+
diff --git a/meta/recipes-extended/mdadm/mdadm_3.3.2.bb 
b/meta/recipes-extended/mdadm/mdadm_3.3.3.bb
similarity index 66%
rename from meta/recipes-extended/mdadm/mdadm_3.3.2.bb
rename to meta/recipes-extended/mdadm/mdadm_3.3.3.bb
index b2fbe4a..5600bd9 100644
--- a/meta/recipes-extended/mdadm/mdadm_3.3.2.bb
+++ b/meta/recipes-extended/mdadm/mdadm_3.3.3.bb
@@ -10,18 +10,17 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
 
 SRC_URI = "${KERNELORG_MIRROR}/linux/utils/raid/mdadm/${BPN}-${PV}.tar.xz \
file://mdadm-3.2.2_fix_for_x32.patch \
-   file://gcc-4.9.patch \
file://mdadm-3.3.2_x32_abi_time_t.patch \
-   file://inline.patch \
+   file://gcc-4.9.patch \
file://run-ptest \
+   file://mdmonitor.service \
  "
-
-SRC_URI[md5sum] = "44698d351501cac6a89072dc877eb220"
-SRC_URI[

[OE-core] [PATCH 0/1] Fix dead lock issue when use smart with pygtk

[Kai Kang]

Hi Ross,

I sent this patch before, but no comment and has not been merged yet.
So I re-send it for review.

Thanks.

The following changes since commit 2a1573841e2b5dbfce93aaaf27ad7177c71f45ab:

  sshcontrol: Use os.environ.copy() instead of copy.copy() (2015-07-29 22:54:12 
+0100)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib kangkai/smartpm
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=kangkai/smartpm

Kai Kang (1):
  smartpm: set noprogress for pycurl

 .../smart-set-noprogress-for-pycurl.patch| 20 
 meta/recipes-devtools/python/python-smartpm_git.bb   |  1 +
 2 files changed, 21 insertions(+)
 create mode 100644 
meta/recipes-devtools/python/python-smartpm/smart-set-noprogress-for-pycurl.patch

-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 1/1] smartpm: set noprogress for pycurl

[Kai Kang]

Set NOPROGRESS for pycurl just as same as default operation in pycurl
module itself. If set NOPROGRESS with 0 for pycurl, it causes dead lock
issue of Python GIL when call smart library by python gui just like
pygtk.

Signed-off-by: Kai Kang 
---
 .../smart-set-noprogress-for-pycurl.patch| 20 
 meta/recipes-devtools/python/python-smartpm_git.bb   |  1 +
 2 files changed, 21 insertions(+)
 create mode 100644 
meta/recipes-devtools/python/python-smartpm/smart-set-noprogress-for-pycurl.patch

diff --git 
a/meta/recipes-devtools/python/python-smartpm/smart-set-noprogress-for-pycurl.patch
 
b/meta/recipes-devtools/python/python-smartpm/smart-set-noprogress-for-pycurl.patch
new file mode 100644
index 000..2885998
--- /dev/null
+++ 
b/meta/recipes-devtools/python/python-smartpm/smart-set-noprogress-for-pycurl.patch
@@ -0,0 +1,20 @@
+Set NOPROGRESS for pycurl just as same as default operation in pycurl module 
itself.
+If set NOPROGRESS with 0 for pycurl, it causes dead lock issue of Python GIL 
when
+call smart library by python gui just like pygtk.
+
+Upstream-Status: Pending
+
+Signed-off-by: Kai Kang 
+---
+diff -u smart-1.4.1/smart.orig/fetcher.py smart-1.4.1/smart/fetcher.py
+--- smart-1.4.1/smart.orig/fetcher.py  2014-07-15 16:42:19.240437080 +0800
 smart-1.4.1/smart/fetcher.py   2014-07-15 17:02:37.812470289 +0800
+@@ -1720,7 +1720,7 @@
+ handle.setopt(pycurl.OPT_FILETIME, 1)
+ handle.setopt(pycurl.LOW_SPEED_LIMIT, 1)
+ handle.setopt(pycurl.LOW_SPEED_TIME, SOCKETTIMEOUT)
+-handle.setopt(pycurl.NOPROGRESS, 0)
++handle.setopt(pycurl.NOPROGRESS, 1)
+ handle.setopt(pycurl.PROGRESSFUNCTION, progress)
+ handle.setopt(pycurl.WRITEDATA, local)
+ handle.setopt(pycurl.FOLLOWLOCATION, 1)
diff --git a/meta/recipes-devtools/python/python-smartpm_git.bb 
b/meta/recipes-devtools/python/python-smartpm_git.bb
index 163461e..8b974b0 100644
--- a/meta/recipes-devtools/python/python-smartpm_git.bb
+++ b/meta/recipes-devtools/python/python-smartpm_git.bb
@@ -22,6 +22,7 @@ SRC_URI = "\
   file://smart-rpm4-fixes.patch \
   file://smart-add-for-rpm-ignoresize-check.patch \
   file://smart-already-installed-message.patch \
+  file://smart-set-noprogress-for-pycurl.patch \
  "
 
 SRCREV = "407a7eca766431257dcd1da15175cc36a1bb22d0"
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] u-boot-mkimage: fix a building failure on OpenSus

[Rongqing Li]

On 2015年07月28日 19:47, Burton, Ross wrote:


On 28 July 2015 at 03:00, mailto:rongqing...@windriver.com>> wrote:

+config.mk  will be included only if auto.conf is
newer than .config
+but in some system, the HPET is not enabled, the smallest unit of
+time is second, and can not decise which file is newer, even if
+the correct dependency has been created.
+
+The below shows unit of time:
+
+under SUSE Linux Enterprise Desktop 11 SP2  (i586):
+$ls --full-time include/config/auto.conf .config
+2015-07-27 03:46:20.0 -0400 .config
+2015-07-27 03:46:20.0 -0400 include/config/auto.conf
+$
+
+under Ubuntu 14.04 LTS:
+$ ls --full-time include/config/auto.conf .config
+2015-07-27 13:40:14.008703027 +0800 .config
+2015-07-27 13:40:15.020703054 +0800 include/config/auto.conf
+$
+
+The rule of including config.mk  in Makefile as below
+autoconf_is_current := $(if $(wildcard
$(KCONFIG_CONFIG)),$(shell find . \
+-path ./include/config/auto.conf -newer
$(KCONFIG_CONFIG)))
+ifneq ($(autoconf_is_current),)
+include $(srctree)/config.mk 
+include $(srctree)/arch/$(ARCH)/Makefile
+endif
+
+The compilation will be failed if config.mk  is
not included
+so delay 1 second to create auto.conf after creating of .config


Adding a sleep seems pretty ugly, wouldn't a neater fix be to change the
logic so that instead of the test being "is newer" you use "is not
older", to handle identical timestamps as being current.

Ross


u-boot-mkimage just be upgraded, and has the similar your suggestive fix:





commit cffcd2861310855130db52c93f7bf4d9b511741d
Author: Masahiro Yamada 
Date:   Fri Apr 3 12:30:25 2015 +0900

kbuild: include config.mk when auto.conf is not older than .config

Since the Kconfig conversion, config.mk has been included only when
include/config/auto.conf is newer than the .config file.

It causes build error if both files have the same time-stamps.
It is actually possible because EXT* file systems have a 1s time-stamp
resolution.

The config.mk should be included when include/config/auto.conf is
*not older* than the .config file.

Signed-off-by: Masahiro Yamada 
Reported-by: Tom Rini 
Reported-by: York Sun 
Reported-by: Stephen Warren 
Reported-by: Matthew Gerlach 
Tested-by: Stephen Warren 

diff --git a/Makefile b/Makefile
index f1bce44..53ad450 100644
--- a/Makefile
+++ b/Makefile
@@ -513,12 +513,16 @@ include/config/%.conf: $(KCONFIG_CONFIG) 
include/config/auto.conf.cmd
 # is up-to-date. When we switch to a different board configuration, 
old CONFIG
 # macros are still remaining in include/config/auto.conf. Without the 
following
 # gimmick, wrong config.mk would be included leading nasty 
warnings/errors.

-autoconf_is_current := $(if $(wildcard $(KCONFIG_CONFIG)),$(shell find . \
-   -path ./include/config/auto.conf -newer $(KCONFIG_CONFIG)))
-ifneq ($(autoconf_is_current),)
+ifneq ($(wildcard $(KCONFIG_CONFIG)),)
+ifneq ($(wildcard include/config/auto.conf),)
+autoconf_is_old := $(shell find . -path ./$(KCONFIG_CONFIG) -newer \
+   include/config/auto.conf)
+ifeq ($(autoconf_is_old),)
 include $(srctree)/config.mk
 include $(srctree)/arch/$(ARCH)/Makefile
 endif
+endif
+endif

 # If board code explicitly specified LDSCRIPT or CONFIG_SYS_LDSCRIPT, use
 # that (or fail if absent).  Otherwise, search for a linker script in a



--
Best Reagrds,
Roy | RongQing Li
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH v2] tzdata: Add marking for config files in recipe

[Li Zhou]

The tzdata recipe does not mark the /etc/timezone file and
/etc/localtime link as configuration files. An on target update would
then overwite the user modified versions of those files.
Add those files in CONFFILES_${PN}.

Signed-off-by: Li Zhou 
---
 meta/recipes-extended/tzdata/tzdata_2015e.bb |2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-extended/tzdata/tzdata_2015e.bb 
b/meta/recipes-extended/tzdata/tzdata_2015e.bb
index 9179d7b..92389a9 100644
--- a/meta/recipes-extended/tzdata/tzdata_2015e.bb
+++ b/meta/recipes-extended/tzdata/tzdata_2015e.bb
@@ -203,3 +203,5 @@ FILES_${PN} += "${datadir}/zoneinfo/Pacific/Honolulu \
 ${datadir}/zoneinfo/zone.tab \
 ${datadir}/zoneinfo/iso3166.tab  \
 ${datadir}/zoneinfo/Etc/*"
+
+CONFFILES_${PN} += "${sysconfdir}/timezone ${sysconfdir}/localtime"
-- 
1.7.9.5

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [yocto] glibc 2.22

[Khem Raj]

On Tue, Jul 28, 2015 at 12:48 AM, Lei, Maohui  wrote:
> Hi Khem,
>
>
>> glibc 2.22 will release end of this month. For some time I have put
>> together the update here
>>
>> http://git.openembedded.org/openembedded-core-contrib/commit/?h=kraj/m
>> aster&id=0
>> 2e73ea526d94f21c7ef82eb96b062eff8ebb825
>>
>> It works pretty well for machines I have tested it on ( arm/x86 ) but
>> would be interested if more wider trial could be carried out and
>> issues reported between now and release can be addressed.
>>
>> Please give it a try if you have spare cycles
>
> I tested it on powerpc(e500v2),and did a ltp test. It does work well.
>

thanks for the report


>
>
> Best Regards
> Lei Maohui
>
>
>
>
>> -Original Message-
>> From: openembedded-core-boun...@lists.openembedded.org
>> [mailto:openembedded-core-boun...@lists.openembedded.org] On Behalf Of
>> Khem Raj
>> Sent: Tuesday, July 14, 2015 1:37 AM
>> To: Patches and discussions about the oe-core layer; OpenEmbedded
>> Devel List; Yocto discussion list
>> Subject: [OE-core] glibc 2.22
>>
>> Hi All
>>
>> glibc 2.22 will release end of this month. For some time I have put
>> together the update here
>>
>> http://git.openembedded.org/openembedded-core-contrib/commit/?h=kraj/m
>> aster&id=0
>> 2e73ea526d94f21c7ef82eb96b062eff8ebb825
>>
>> It works pretty well for machines I have tested it on ( arm/x86 ) but
>> would be interested if more wider trial could be carried out and
>> issues reported between now and release can be addressed.
>>
>> Please give it a try if you have spare cycles
>>
>> Thanks
>> -Khem
>>
>
> --
> ___
> yocto mailing list
> yo...@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] alsa-utils: assume the alsa storing is success if machine has no sound card

[rongqing.li]

From: Roy Li 

Signed-off-by: Roy Li 
---
 ...oring-is-success-if-not-sound-card-device.patch | 34 ++
 meta/recipes-multimedia/alsa/alsa-utils_1.0.29.bb  |  1 +
 2 files changed, 35 insertions(+)
 create mode 100644 
meta/recipes-multimedia/alsa/alsa-utils/assume-storing-is-success-if-not-sound-card-device.patch

diff --git 
a/meta/recipes-multimedia/alsa/alsa-utils/assume-storing-is-success-if-not-sound-card-device.patch
 
b/meta/recipes-multimedia/alsa/alsa-utils/assume-storing-is-success-if-not-sound-card-device.patch
new file mode 100644
index 000..f67283d
--- /dev/null
+++ 
b/meta/recipes-multimedia/alsa/alsa-utils/assume-storing-is-success-if-not-sound-card-device.patch
@@ -0,0 +1,34 @@
+[PATCH] assume storing is success if not sound card device
+
+Upstream-Statue: Pending
+
+Systemd will report failure when run alsa-*, if the machine has not the
+sound card. To void this annoyed message, alsa-restore/alsa-state ignore
+all the exit codes by prefixing "-" in ExecStart, like:
+
+   alsa-utils-1.0.29$ grep "=-" ./ -r|grep service.in
+   ./alsactl/alsa-restore.service.in:ExecStart=-@sbindir@/alsactl restore
+   ./alsactl/alsa-state.service.in:ExecStart=-@sbindir@/alsactl -s -n 19 -c 
rdaemon
+   ./alsactl/alsa-state.service.in:ExecStop=-@sbindir@/alsactl -s kill 
save_and_quit
+   lsa-utils-1.0.29$
+
+But alsa-store.service.in is missing, and better solution is to ignore
+the exit code 19 which means not sound card device, not all exit code
+
+Signed-off-by: Roy Li 
+---
+ alsactl/alsa-store.service.in | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/alsactl/alsa-store.service.in b/alsactl/alsa-store.service.in
+index f1a56bb..68ca529 100644
+--- a/alsactl/alsa-store.service.in
 b/alsactl/alsa-store.service.in
+@@ -13,3 +13,4 @@ Before=shutdown.target
+ Type=oneshot
+ ExecStart=@sbindir@/alsactl store
+ StandardOutput=syslog
++SuccessExitStatus=0 19
+-- 
+1.9.1
+
diff --git a/meta/recipes-multimedia/alsa/alsa-utils_1.0.29.bb 
b/meta/recipes-multimedia/alsa/alsa-utils_1.0.29.bb
index 65cae0b..263f38a 100644
--- a/meta/recipes-multimedia/alsa/alsa-utils_1.0.29.bb
+++ b/meta/recipes-multimedia/alsa/alsa-utils_1.0.29.bb
@@ -14,6 +14,7 @@ PACKAGECONFIG[xmlto] = "--enable-xmlto, --disable-xmlto, 
xmlto-native docbook-xm
 SRC_URI = "ftp://ftp.alsa-project.org/pub/utils/alsa-utils-${PV}.tar.bz2 \

file://0001-alsactl-don-t-let-systemd-unit-restore-the-volume-wh.patch \
file://alsa-utils-aplay-interrupt-signal-handling.patch \
+   file://assume-storing-is-success-if-not-sound-card-device.patch \
   "
 
 SRC_URI[md5sum] = "6b289bf874c4c9a63f4b3973093dd404"
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] tzdata: Add marking for config files in recipe

[Zhou, Li]

I have setup up poky as below doc described:
http://www.yoctoproject.org/docs/1.8/yocto-project-qs/yocto-project-qs.html

 $ git clone http://git.yoctoproject.org/git/poky
 $ cd poky
 $ git checkout -b fido origin/fido
 $ source oe-init-build-env

Do you mean that I should use branch master instead of fido? Has fido 
been abandoned?

Thanks.

On 07/29/2015 08:26 PM, Burton, Ross wrote:


On 29 July 2015 at 02:49, Li Zhou > wrote:


 meta/recipes-extended/tzdata/tzdata.inc |2 ++


This inc file was removed last month, please rebase to master.

Ross


--
Best Regards!
Zhou Li
Phone number: 86-10-84778511

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/2] bitbake.conf: add GITHUB_GIT

[Robert Yang]

On 07/30/2015 12:27 AM, Otavio Salvador wrote:

On Wed, Jul 29, 2015 at 8:48 AM, Olof Johansson  wrote:

Excerpts from Robert Yang's message of 2015-07-29 12:19:06 +0200:

It is just like what GNOME_GIT, GNU_MIRROR and others did.


Tbh, I don't understand them either, but maybe that's just me. What is
the reason? I think it makes it harder to see what is happening.


It allows for people to add local mirrors for example and even remap
fetching without touching the recipes. It is a good addition IMO.


Thanks, I updated the commit message in the repo:

  git://git.openembedded.org/openembedded-core-contrib rbt/github

commit 8b6d701aa812536dc5d4350d8a7646f54942ebac
Author: Robert Yang 
Date:   Tue Jul 28 03:00:37 2015 -0700

meta: git://github.com -> ${GITHUB_GIT}

It allows for people to add local mirrors for example and even remap
fetching without touching the recipes.

Signed-off-by: Robert Yang 

// Robert
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [oe-core][PATCH 1/1] nss: advance to version 3.19.1

[Slater, Joseph]

ok.  I think I just sent a new version of the patch (but for some reason I did 
not get a copy). Joe

From: Burton, Ross [mailto:ross.bur...@intel.com]
Sent: Tuesday, July 28, 2015 12:46 PM
To: Slater, Joseph
Cc: OE-core
Subject: Re: [OE-core] [oe-core][PATCH 1/1] nss: advance to version 3.19.1


On 28 July 2015 at 20:11, Joe Slater 
mailto:jsla...@windriver.com>> wrote:
+# override value in nss.inc
+#
+LIC_FILES_CHKSUM = "file://nss/COPYING;md5=3b1e88e1b9c0b5a4b2881d46cce06a18 \
+
file://nss/lib/freebl/mpi/doc/LICENSE;md5=491f158d09d948466afce85d6f1fe18f \
+
file://nss/lib/freebl/mpi/doc/LICENSE-MPL;md5=5d425c8f3157dbf212db2ec53d9e5132"

Having a separate inc file really helps with clear recipes doesn't it...


According to the layer index there's only one recipe for NSS, so how about we 
just ditch the nss.inc?

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/8] busybox: merge login-utilities.cfg into defconfig

[Richard Purdie]

On Tue, 2015-07-28 at 16:20 -0700, Andre McCurdy wrote:
> The login-utilities.cfg busybox config fragment was added during the
> transition of the default login manager from tinylogin to busybox [1].
> The tinylogin recipe was removed from oe-core prior to the 1.5 (dora)
> release [2].
> 
> Merging the config fragment into defconfig simplifies the process of
> keeping the oe-core busybox defconfig aligned with new releases of
> busybox.
> 
>   [1] https://bugzilla.yoctoproject.org/show_bug.cgi?id=4207
>   [2] 
> http://git.openembedded.org/openembedded-core/commit/?id=2762ff976a3473be4259889029e048ab8b3be5ab
> 
> Signed-off-by: Andre McCurdy 

I'm kind of torn on this. The original intent was to split up the
busybox config into more granular fragments so that rather that having
to provide a defconfig for it, you could just select the components you
wanted and your custom config on top of that.

If we do go down that route, it does imply a different kind of
maintenance of the config fragments is needed as we move forward. In
some ways I'd hope the fragment tools could help with that.

Obviously any more split up hasn't happened as yet and I know that
personally, given my workload I'm unlikely to find time soon :(. I'd be
interested to see what others think?

Cheers,

Richard




-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [oe-core][PATCH 1/1][v2] nss: advance to version 3.19.1

[Joe Slater]

Picks up fixes for CVE-2015-2721 and CVE-2015-2730.  Specify previously
overlooked license file COPYING.  Fold nss.inc into recipe.

Signed-off-by: Joe Slater 
---
 meta/recipes-support/nss/nss_3.17.3.bb |7 ---
 .../recipes-support/nss/{nss.inc => nss_3.19.1.bb} |   14 ++
 2 files changed, 10 insertions(+), 11 deletions(-)
 delete mode 100644 meta/recipes-support/nss/nss_3.17.3.bb
 rename meta/recipes-support/nss/{nss.inc => nss_3.19.1.bb} (93%)

diff --git a/meta/recipes-support/nss/nss_3.17.3.bb 
b/meta/recipes-support/nss/nss_3.17.3.bb
deleted file mode 100644
index e7d0780..000
--- a/meta/recipes-support/nss/nss_3.17.3.bb
+++ /dev/null
@@ -1,7 +0,0 @@
-require nss.inc
-
-SRC_URI += "\
-
http://ftp.mozilla.org/pub/mozilla.org/security/nss/releases/NSS_3_17_3_RTM/src/${BP}.tar.gz
 \
-"
-SRC_URI[md5sum] = "fba7489e1b26f2a0bfe5527430fd61e1"
-SRC_URI[sha256sum] = 
"f4d5e9035a2f84f25f35c283de3b0ff60d72e918748de25eaf017ed201fa21d5"
diff --git a/meta/recipes-support/nss/nss.inc 
b/meta/recipes-support/nss/nss_3.19.1.bb
similarity index 93%
rename from meta/recipes-support/nss/nss.inc
rename to meta/recipes-support/nss/nss_3.19.1.bb
index 665e4e6..66b9d60 100644
--- a/meta/recipes-support/nss/nss.inc
+++ b/meta/recipes-support/nss/nss_3.19.1.bb
@@ -10,18 +10,23 @@ SECTION = "libs"
 
 LICENSE = "MPL-2.0 | (MPL-2.0 & GPL-2.0+) | (MPL-2.0 & LGPL-2.1+)"
 
-LIC_FILES_CHKSUM = 
"file://nss/lib/freebl/mpi/doc/LICENSE;md5=491f158d09d948466afce85d6f1fe18f \
-
file://nss/lib/freebl/mpi/doc/LICENSE-MPL;md5=6bf96825e3d7ce4de25621ae886cc859"
+LIC_FILES_CHKSUM = "file://nss/COPYING;md5=3b1e88e1b9c0b5a4b2881d46cce06a18 \
+
file://nss/lib/freebl/mpi/doc/LICENSE;md5=491f158d09d948466afce85d6f1fe18f \
+
file://nss/lib/freebl/mpi/doc/LICENSE-MPL;md5=5d425c8f3157dbf212db2ec53d9e5132"
+
 SRC_URI = "\
+
http://ftp.mozilla.org/pub/mozilla.org/security/nss/releases/NSS_3_19_1_RTM/src/${BP}.tar.gz
 \
 file://nss-fix-support-cross-compiling.patch \
 file://nss-no-rpath-for-cross-compiling.patch \
 file://nss-fix-incorrect-shebang-of-perl.patch \
 file://nss-fix-nsinstall-build.patch \
-"
-SRC_URI_append = "\
 file://nss.pc.in \
 file://signlibs.sh \
 "
+
+SRC_URI[md5sum] = "68a9c01c987b9bd92066b4e0041f3e58"
+SRC_URI[sha256sum] = 
"b7be709551ec13206d8e3e8c065b894fa981c11573115e9478fa051029c52fff"
+
 inherit siteinfo
 
 DEPENDS = "sqlite3 nspr zlib nss-native"
@@ -215,3 +220,4 @@ FILES_${PN}-dev = "\
 # FILES_${PN}-dbg is populated automatically
 
 BBCLASSEXTEND = "native nativesdk"
+
-- 
1.7.9.5

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 1/3] populate_sdk_ext: use lnr, not ln -sr, for portability

[Christopher Larson]

From: Christopher Larson 

Not all hosts are running sufficiently new coreutils.

Signed-off-by: Christopher Larson 
---
 meta/classes/populate_sdk_ext.bbclass | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/classes/populate_sdk_ext.bbclass 
b/meta/classes/populate_sdk_ext.bbclass
index b28413b..0151468 100644
--- a/meta/classes/populate_sdk_ext.bbclass
+++ b/meta/classes/populate_sdk_ext.bbclass
@@ -164,8 +164,8 @@ python copy_buildsystem () {
 
 install_tools() {
install -d ${SDK_OUTPUT}/${SDKPATHNATIVE}${bindir_nativesdk}
-   ln -sr ${SDK_OUTPUT}/${SDKPATH}/${scriptrelpath}/devtool 
${SDK_OUTPUT}/${SDKPATHNATIVE}${bindir_nativesdk}/devtool
-   ln -sr ${SDK_OUTPUT}/${SDKPATH}/${scriptrelpath}/recipetool 
${SDK_OUTPUT}/${SDKPATHNATIVE}${bindir_nativesdk}/recipetool
+   lnr ${SDK_OUTPUT}/${SDKPATH}/${scriptrelpath}/devtool 
${SDK_OUTPUT}/${SDKPATHNATIVE}${bindir_nativesdk}/devtool
+   lnr ${SDK_OUTPUT}/${SDKPATH}/${scriptrelpath}/recipetool 
${SDK_OUTPUT}/${SDKPATHNATIVE}${bindir_nativesdk}/recipetool
touch ${SDK_OUTPUT}/${SDKPATH}/.devtoolbase
 
install 
${SDK_DEPLOY}/${DISTRO}-${TCLIBC}-${SDK_ARCH}-buildtools-tarball-${TUNE_PKGARCH}-buildtools-nativesdk-standalone-${DISTRO_VERSION}.sh
 ${SDK_OUTPUT}/${SDKPATH}
-- 
2.2.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 3/3] populate_sdk_ext: pass BBPATH to devtool --bbpath

[Christopher Larson]

From: Christopher Larson 

Signed-off-by: Christopher Larson 
---
 meta/classes/populate_sdk_ext.bbclass | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/populate_sdk_ext.bbclass 
b/meta/classes/populate_sdk_ext.bbclass
index 0151468..a36bf16 100644
--- a/meta/classes/populate_sdk_ext.bbclass
+++ b/meta/classes/populate_sdk_ext.bbclass
@@ -88,7 +88,8 @@ python copy_buildsystem () {
 config.write(f)
 
 # Create a layer for new recipes / appends
-bb.process.run("devtool --basepath %s create-workspace --create-only %s" % 
(baseoutpath, os.path.join(baseoutpath, 'workspace')))
+bbpath = d.getVar('BBPATH', True)
+bb.process.run(['devtool', '--bbpath', bbpath, '--basepath', baseoutpath, 
'create-workspace', '--create-only', os.path.join(baseoutpath, 'workspace')])
 
 # Create bblayers.conf
 bb.utils.mkdirhier(baseoutpath + '/conf')
-- 
2.2.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 2/3] devtool: add --bbpath argument

[Christopher Larson]

From: Christopher Larson 

Signed-off-by: Christopher Larson 
---
 scripts/devtool | 17 -
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/scripts/devtool b/scripts/devtool
index 557a830..4a1f2d6 100755
--- a/scripts/devtool
+++ b/scripts/devtool
@@ -189,6 +189,7 @@ def main():
  add_help=False,
  epilog="Use %(prog)s  --help 
to get help on a specific command")
 parser.add_argument('--basepath', help='Base directory of SDK / build 
directory')
+parser.add_argument('--bbpath', help='Explicitly specify the BBPATH, 
rather than getting it from the metadata')
 parser.add_argument('-d', '--debug', help='Enable debug output', 
action='store_true')
 parser.add_argument('-q', '--quiet', help='Print only errors', 
action='store_true')
 parser.add_argument('--color', choices=['auto', 'always', 'never'], 
default='auto', help='Colorize output (where %(metavar)s is %(choices)s)', 
metavar='COLOR')
@@ -239,13 +240,19 @@ def main():
 
 scriptutils.logger_setup_color(logger, global_args.color)
 
-tinfoil = setup_tinfoil(config_only=True)
-for path in ([scripts_path] +
- tinfoil.config_data.getVar('BBPATH', True).split(':')):
+if global_args.bbpath is None:
+tinfoil = setup_tinfoil(config_only=True)
+global_args.bbpath = tinfoil.config_data.getVar('BBPATH', True)
+else:
+tinfoil = None
+
+for path in [scripts_path] + global_args.bbpath.split(':'):
 pluginpath = os.path.join(path, 'lib', 'devtool')
 scriptutils.load_plugins(logger, plugins, pluginpath)
-tinfoil.cooker.shutdown(force=True)
-tinfoil.cooker.unlockBitbake()
+
+if tinfoil:
+tinfoil.cooker.shutdown(force=True)
+tinfoil.cooker.unlockBitbake()
 
 subparsers = parser.add_subparsers(dest="subparser_name", 
title='subcommands', metavar='')
 
-- 
2.2.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 0/3] Fix a couple populate_sdk_ext issues

[Christopher Larson]

From: Christopher Larson 

- Explicitly pass BBPATH to devtool as Paul Eggleton suggests, to avoid
  needing to acquire the bitbake lock, to resolve the issue in
  populate_sdk_ext.
- Use the oe-core lnr script rather than ln -sr to improve host portability
  (not all hosts have recent coreutils).

The following changes since commit 27d068d05239c26a3848eb101571acab54635e37:

  harfbuzz: upgrade to 1.0.1 (2015-07-27 23:28:23 +0100)

are available in the git repository at:

  g...@github.com:kergoth/openembedded-core populate_sdk_ext-fixes

for you to fetch changes up to a66090ba23452e0dca05e5811ebe88ea1bfc84de:

  populate_sdk_ext: pass BBPATH to devtool --bbpath (2015-07-29 14:48:05 -0700)


Christopher Larson (3):
  populate_sdk_ext: use lnr, not ln -sr, for portability
  devtool: add --bbpath argument
  populate_sdk_ext: pass BBPATH to devtool --bbpath

 meta/classes/populate_sdk_ext.bbclass |  7 ---
 scripts/devtool   | 17 -
 2 files changed, 16 insertions(+), 8 deletions(-)

-- 
2.2.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4 2/5] gstreamer1.0-plugins-base: handle audio/video decoder error

[Carlos Rafael Giani]

This one is quite difficult. On one hand, I see the benefits. If for 
example an audio track is broken but the video track is fine, it should 
not stop the entire playback. On the other hand, the error message about 
no valid frames found is quite significant, and if a player suddenly 
stops because of missing valid frames, this player will not get any 
message from the pipeline. The reason for the stop will remain a mystery 
unless somebody looks at the logs.


I agree with the conclusions in Bugzilla that the appropriate way to 
handle this would be to somehow aggregate the error messages and only 
stop if all tracks report an error. However, this is a nontrivial 
change, and if it is implemente, it won't make it in before GStreamer 
1.8 (perhaps even later).


But, Zhu, the bugzilla entry also contains a patch for baseparse. What 
about that one?


Am 2015-07-29 um 20:25 schrieb Yuqing Zhu:

When there is input data and no output data to the end of the stream, it will
send GST_ELEMENT_ERROR and quit from playing.
The patch comments the GST_ELEMENT_ERROR() and just add GST_ERROR_OBJECT()
information instead.

Signed-off-by: Yuqing Zhu
---
  .../handle-audio-video-decoder-error.patch | 64 ++
  .../gstreamer/gstreamer1.0-plugins-base_1.4.5.bb   |  1 +
  2 files changed, 65 insertions(+)
  create mode 100755 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/handle-audio-video-decoder-error.patch

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/handle-audio-video-decoder-error.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/handle-audio-video-decoder-error.patch
new file mode 100755
index 000..833ce72
--- /dev/null
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/handle-audio-video-decoder-error.patch
@@ -0,0 +1,64 @@
+From bcb2b8b6f49e7c66124a4f5e07dea829d5ebfe59 Mon Sep 17 00:00:00 2001
+From: Lyon Wang
+Date: Mon, 15 Dec 2014 16:52:07 +0800
+Subject: [PATCH] handle audio/video decoder error
+
+When there is input data and no output data to the end of the stream, it will
+send GST_ELEMENT_ERROR, So the clips playing will quit.
+However, if only one of the tracks is corrupt, there is no need to quit other
+tracks playing.
+
+The patch comments the GST_ELEMENT_ERROR() and just add GST_ERROR_OBJECT()
+information instead.
+
+Upstream-Status: Submitted [https://bugzilla.gnome.org/show_bug.cgi?id=741542]
+
+Signed-off-by: Lyon Wang
+---
+ gst-libs/gst/audio/gstaudiodecoder.c |5 +++--
+ gst-libs/gst/video/gstvideodecoder.c |5 +++--
+ 2 files changed, 6 insertions(+), 4 deletions(-)
+ mode change 100644 => 100755 gst-libs/gst/audio/gstaudiodecoder.c
+ mode change 100644 => 100755 gst-libs/gst/video/gstvideodecoder.c
+
+diff --git a/gst-libs/gst/audio/gstaudiodecoder.c 
b/gst-libs/gst/audio/gstaudiodecoder.c
+old mode 100644
+new mode 100755
+index c2e7a28..891df0a
+--- a/gst-libs/gst/audio/gstaudiodecoder.c
 b/gst-libs/gst/audio/gstaudiodecoder.c
+@@ -2123,9 +2123,10 @@ gst_audio_decoder_sink_eventfunc (GstAudioDecoder * 
dec, GstEvent * event)
+   GST_AUDIO_DECODER_STREAM_UNLOCK (dec);
+
+   if (dec->priv->ctx.had_input_data && !dec->priv->ctx.had_output_data) {
+-GST_ELEMENT_ERROR (dec, STREAM, DECODE,
++/* GST_ELEMENT_ERROR (dec, STREAM, DECODE,
+ ("No valid frames decoded before end of stream"),
+-("no valid frames found"));
++("no valid frames found")); */
++GST_ERROR_OBJECT(dec, "No valid frames decoded before end of stream");
+   }
+
+   /* send taglist if no valid frame is decoded util EOS */
+diff --git a/gst-libs/gst/video/gstvideodecoder.c 
b/gst-libs/gst/video/gstvideodecoder.c
+old mode 100644
+new mode 100755
+index ac581e1..4278bcd
+--- a/gst-libs/gst/video/gstvideodecoder.c
 b/gst-libs/gst/video/gstvideodecoder.c
+@@ -1068,9 +1068,10 @@ gst_video_decoder_sink_event_default (GstVideoDecoder * 
decoder,
+
+   /* Error out even if EOS was ok when we had input, but no output */
+   if (ret && priv->had_input_data && !priv->had_output_data) {
+-GST_ELEMENT_ERROR (decoder, STREAM, DECODE,
++/* GST_ELEMENT_ERROR (decoder, STREAM, DECODE,
+ ("No valid frames decoded before end of stream"),
+-("no valid frames found"));
++("no valid frames found")); */
++GST_ERROR_OBJECT(decoder, "No valid frames decoded before end of 
stream");
+   }
+
+   /* Forward EOS immediately. This is required because no
+--
+1.7.9.5
+
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
index aaa1ecb..aa90179 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
@@ -9,6 +9,7 @@ SRC_URI +="file://do-not-change-eos-event-to-gap-event-if.patch 
\
  

Re: [OE-core] [PATCH 4/6] devtool: also load plugins from BBPATH

[Christopher Larson]

On Wed, Jul 29, 2015 at 9:40 AM, Christopher Larson 
wrote:

> On Wed, Jul 29, 2015 at 2:56 AM, Paul Eggleton <
> paul.eggle...@linux.intel.com> wrote:
>
>> Hmm, I guess we will have to add a command-line parameter to specify
>> BBPATH
>> (or the full plugin search path) so that tinfoil doesn't have to be
>> instantiated when it's specified.
>>
>
> Indeed, sorry about that, I didn’t even realize populate_sdk_ext used
> devtool from a bbclass :)
>

I'll submit patches for this and one other populate_sdk_ext issue shortly.
-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [for-fido] Backport for perf - support for 4.1 kernels

[Otavio Salvador]

Hello,

Is it possible for you to cherry-pick OE-Core:46f8420 for Fido? We got
one board using 4.1 kernel and it fails badly.

It is in use in master since April and had no change since its
inclusion so it is fairly safe.

Regards,

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7

[Otavio Salvador]

On Wed, Jul 29, 2015 at 5:49 PM, Tanu Kaskinen
 wrote:
> This is safe to apply, but I'd like to understand why you think this is
> needed. The commit message says that without this patch, "pulseaudio
> can't be opened in yocto". PulseAudio works fine in core-image-sato at
> least, so what is different in your setup? Have you disabled
> autospawning for some reason?

I prefer to not divert from upstream without a reasoning for it. This
is what needs to be clarified.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7

[Tanu Kaskinen]

On Thu, 2015-07-30 at 07:49 +0800, Yuqing Zhu wrote:
> Revert "launch: Avoid specifically starting PA and rely on autospawn/socket 
> activation"
> or the pulseaudio can't be opened in yocto.
> 
> Signed-off-by: Yuqing Zhu 
> ---
>  ...ch-Avoid-specifically-starting-PA-and-rel.patch | 31 
> ++
>  .../pulseaudio/pulseaudio_6.0.bb   |  1 +
>  2 files changed, 32 insertions(+)
>  create mode 100644 
> meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
> 
> diff --git 
> a/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
>  
> b/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
> new file mode 100644
> index 000..669c355
> --- /dev/null
> +++ 
> b/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
> @@ -0,0 +1,31 @@
> +From 5760c6b1a3c9b6dd8e70c9d1ea78fcf192250b4a Mon Sep 17 00:00:00 2001
> +From: Shengjiu Wang 
> +Date: Mon, 16 Mar 2015 15:30:11 +0800
> +Subject: [PATCH] Revert "launch: Avoid specifically starting PA and rely on
> + autospawn/socket activation"
> +
> +This reverts commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7.
> +
> +Upstream-Status: Pending 
> +
> +Signed-off-by: shengjiu.wang 
> +---
> + src/daemon/start-pulseaudio-x11.in |2 ++
> + 1 file changed, 2 insertions(+)
> +
> +diff --git a/src/daemon/start-pulseaudio-x11.in 
> b/src/daemon/start-pulseaudio-x11.in
> +index 16e46b1..e108645 100755
> +--- a/src/daemon/start-pulseaudio-x11.in
>  b/src/daemon/start-pulseaudio-x11.in
> +@@ -17,6 +17,8 @@
> + 
> + set -e
> + 
> ++@PA_BINARY@ --start "$@"
> ++
> + if [ x"$DISPLAY" != x ] ; then
> + 
> + @PACTL_BINARY@ load-module module-x11-publish "display=$DISPLAY" > 
> /dev/null
> +-- 
> +1.7.9.5
> +
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb 
> b/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
> index 31e9096..10c3d7a 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
> @@ -6,6 +6,7 @@ SRC_URI = 
> "http://freedesktop.org/software/pulseaudio/releases/${BP}.tar.xz \
> file://0001-conf-parser-add-support-for-.d-directories.patch \
> file://fix-git-version-gen.patch \
> file://volatiles.04_pulse \
> +   
> file://0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch \
>  "
>  SRC_URI[md5sum] = "b691e83b7434c678dffacfa3a027750e"
>  SRC_URI[sha256sum] = 
> "b50640e0b80b1607600accfad2e45aabb79d379bf6354c9671efa2065477f6f6"
> -- 
> 1.9.1

This is safe to apply, but I'd like to understand why you think this is
needed. The commit message says that without this patch, "pulseaudio
can't be opened in yocto". PulseAudio works fine in core-image-sato at
least, so what is different in your setup? Have you disabled
autospawning for some reason?

-- 
Tanu

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] watchdog: kill update-rc.d when not using sysvinit

[Martin Jansa]

On Wed, Jul 29, 2015 at 12:27:15PM -0700, Christopher Larson wrote:
> On Wed, Jul 29, 2015 at 11:35 AM, Burton, Ross 
> wrote:
> 
> > On 29 July 2015 at 19:05, Rehman, Abdur  wrote:
> >
> >> watchdog_5.14.bb does not inherit systemd, and so
> >> INHIBIT_UPDATERCD_BBCLASS does
> >> not get set.
> >>
> >> While trying to build watchdog with systemd in DISTRO_FEATURES but not
> >> sysvinit,
> >> following warning is observed:
> >> WARNING: QA Issue: watchdog rdepends on initscripts-functions, but
> >> it isn't
> >> a build dependency? [build-deps]
> >>
> >> Since sysvinit is not in DISTRO_FEATURES and INHIBIT_UPDATERCD_BBCLASS is
> >> not
> >> set, the logic in update-rc.d.bbclass goes on to add an rdep for watchdog
> >> on
> >> initscripts-functions, inside update_rcd_package(). And this results in
> >> the
> >> QA warning.
> >>
> >
> > When a class inherits update-rcd but not systemd, and the distro is
> > systemd only, update-rcd still runs (and has still run) as otherwise you
> > don't get any init scripts.  Setting inhibit will remove the init scripts,
> > and the watchdog daemon won't start.  I'm pretty sure this isn't what you
> > intended.
> >
> > Are you overriding VIRTUAL-RUNTIME_initscripts in your distro?  update-rcd
> > adds a dependency on that when it is inherited, which by default is
> > initscripts, so you get your build dependency there.  I suspect your distro
> > is overriding that variable when perhaps it shouldn't be.
> >
> 
> I don’t think we really want initscripts installed in a systemd-only
> distro, particularly if systemd is built with sysvcompat enabled, as we
> don’t want to run the sysvinit scripts for services which are already
> covered by systemd proper, unless I’m missing something?

I agree and as work-around for this and similar issues I'm setting empty
VIRTUAL-RUNTIME_initscripts in systemd-only distros.

> -- 
> Christopher Larson
> kergoth at gmail dot com
> Founder - BitBake, OpenEmbedded, OpenZaurus
> Maintainer - Tslib
> Senior Software Engineer, Mentor Graphics

> -- 
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core


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


signature.asc
Description: Digital signature
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] watchdog: kill update-rc.d when not using sysvinit

[Christopher Larson]

On Wed, Jul 29, 2015 at 12:48 PM, Burton, Ross 
wrote:

> On 29 July 2015 at 20:27, Christopher Larson 
> wrote:
>
>> I don’t think we really want initscripts installed in a systemd-only
>> distro, particularly if systemd is built with sysvcompat enabled, as we
>> don’t want to run the sysvinit scripts for services which are already
>> covered by systemd proper, unless I’m missing something?
>>
>
> The dependency on initscripts via VIRTUAL-RUNTIME_initscripts is a
> DEPENDS, so it's build time.  The runtime dependency is on
> initscripts-functions.
>
> As far as I'm aware the sysv-generator used the rcX.d symlinks generated
> by update-rcd to decide what to start when, so you can't just disable the
> class and expect it to still work.
>

VIRTUAL-RUNTIME_initscripts is installed at runtime (as you can tell by its
name) in packagegroup-core-boot and packagegroup-core-full-cmdline both. If
I don’t want it in my images, wiping the variable is the only method that
makes any sense.
-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [oe-commits] Ross Burton : gamin: remove

[Burton, Ross]

On 29 July 2015 at 16:01, Martin Jansa  wrote:

> Someone interested in fixing tracer recipe?
>
> NOTE: Resolving any missing task queue dependencies
> ERROR: Nothing PROVIDES 'gamin' (but
> /home/jenkins/oe/world/shr-core/meta-openembedded/meta-gnome/recipes-support/tracker/
> tracker_0.14.2.bb DEPENDS on or otherwise requires it)
>

That build dependency disappeared in 2008, so you can safely delete it.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [scripts][PATCH] yocto-layer: Stops duplication of "meta-" prefix

[humberto . ibarra . lopez]

From: Humberto Ibarra 

The yocto-layer script puts an extra "meta-" prefix to the given layer
name even when the prefix is already there. This fix avoids
duplicating the prefix in these situations.

[YOCTO #8050]

Signed-off-by: Humberto Ibarra 
---
 scripts/yocto-layer | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/scripts/yocto-layer b/scripts/yocto-layer
index 53d2aab..0dbbf3b 100755
--- a/scripts/yocto-layer
+++ b/scripts/yocto-layer
@@ -74,6 +74,8 @@ def yocto_layer_create_subcommand(args, usage_str):
 
 if options.outdir:
 layer_output_dir = options.outdir
+elif layer_name.startswith("meta-"):
+layer_output_dir = layer_name
 else:
 layer_output_dir = "meta-" + layer_name
 
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] watchdog: kill update-rc.d when not using sysvinit

[Burton, Ross]

On 29 July 2015 at 20:27, Christopher Larson 
wrote:

> I don’t think we really want initscripts installed in a systemd-only
> distro, particularly if systemd is built with sysvcompat enabled, as we
> don’t want to run the sysvinit scripts for services which are already
> covered by systemd proper, unless I’m missing something?
>

The dependency on initscripts via VIRTUAL-RUNTIME_initscripts is a DEPENDS,
so it's build time.  The runtime dependency is on initscripts-functions.

As far as I'm aware the sysv-generator used the rcX.d symlinks generated by
update-rcd to decide what to start when, so you can't just disable the
class and expect it to still work.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] watchdog: kill update-rc.d when not using sysvinit

[Christopher Larson]

On Wed, Jul 29, 2015 at 11:35 AM, Burton, Ross 
wrote:

> On 29 July 2015 at 19:05, Rehman, Abdur  wrote:
>
>> watchdog_5.14.bb does not inherit systemd, and so
>> INHIBIT_UPDATERCD_BBCLASS does
>> not get set.
>>
>> While trying to build watchdog with systemd in DISTRO_FEATURES but not
>> sysvinit,
>> following warning is observed:
>> WARNING: QA Issue: watchdog rdepends on initscripts-functions, but
>> it isn't
>> a build dependency? [build-deps]
>>
>> Since sysvinit is not in DISTRO_FEATURES and INHIBIT_UPDATERCD_BBCLASS is
>> not
>> set, the logic in update-rc.d.bbclass goes on to add an rdep for watchdog
>> on
>> initscripts-functions, inside update_rcd_package(). And this results in
>> the
>> QA warning.
>>
>
> When a class inherits update-rcd but not systemd, and the distro is
> systemd only, update-rcd still runs (and has still run) as otherwise you
> don't get any init scripts.  Setting inhibit will remove the init scripts,
> and the watchdog daemon won't start.  I'm pretty sure this isn't what you
> intended.
>
> Are you overriding VIRTUAL-RUNTIME_initscripts in your distro?  update-rcd
> adds a dependency on that when it is inherited, which by default is
> initscripts, so you get your build dependency there.  I suspect your distro
> is overriding that variable when perhaps it shouldn't be.
>

I don’t think we really want initscripts installed in a systemd-only
distro, particularly if systemd is built with sysvcompat enabled, as we
don’t want to run the sysvinit scripts for services which are already
covered by systemd proper, unless I’m missing something?
-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 8/8] busybox: disable volumeid support for exFAT, F2FS and NILFS

[Otavio Salvador]

On Wed, Jul 29, 2015 at 3:49 PM, Andre McCurdy  wrote:
> On Wed, Jul 29, 2015 at 9:33 AM, Otavio Salvador
>  wrote:
>> On Tue, Jul 28, 2015 at 8:21 PM, Andre McCurdy  wrote:
>>> These are less common filesystem formats, so disable volumeid support
>>> for them by default.
>>>
>>> Signed-off-by: Andre McCurdy 
>>
>> F2FS could be of use. It is being more adopted now that Android is
>> starting to be deployed to some devices with it.
>
> The current list of filesystems with volume ID support is below.
>
> If support for F2FS should be enabled, how about enabling support for
> btrfs, luks and squashfs too?

I agree.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 8/8] busybox: disable volumeid support for exFAT, F2FS and NILFS

[Andre McCurdy]

On Wed, Jul 29, 2015 at 9:33 AM, Otavio Salvador
 wrote:
> On Tue, Jul 28, 2015 at 8:21 PM, Andre McCurdy  wrote:
>> These are less common filesystem formats, so disable volumeid support
>> for them by default.
>>
>> Signed-off-by: Andre McCurdy 
>
> F2FS could be of use. It is being more adopted now that Android is
> starting to be deployed to some devices with it.

The current list of filesystems with volume ID support is below.

If support for F2FS should be enabled, how about enabling support for
btrfs, luks and squashfs too?

#
# Filesystem/Volume identification
#
# CONFIG_FEATURE_VOLUMEID_BTRFS is not set
# CONFIG_FEATURE_VOLUMEID_CRAMFS is not set
CONFIG_FEATURE_VOLUMEID_EXFAT=y
CONFIG_FEATURE_VOLUMEID_EXT=y
CONFIG_FEATURE_VOLUMEID_F2FS=y
CONFIG_FEATURE_VOLUMEID_FAT=y
# CONFIG_FEATURE_VOLUMEID_HFS is not set
# CONFIG_FEATURE_VOLUMEID_ISO9660 is not set
# CONFIG_FEATURE_VOLUMEID_JFS is not set
# CONFIG_FEATURE_VOLUMEID_LINUXRAID is not set
# CONFIG_FEATURE_VOLUMEID_LINUXSWAP is not set
# CONFIG_FEATURE_VOLUMEID_LUKS is not set
CONFIG_FEATURE_VOLUMEID_NILFS=y
# CONFIG_FEATURE_VOLUMEID_NTFS is not set
# CONFIG_FEATURE_VOLUMEID_OCFS2 is not set
# CONFIG_FEATURE_VOLUMEID_REISERFS is not set
# CONFIG_FEATURE_VOLUMEID_ROMFS is not set
# CONFIG_FEATURE_VOLUMEID_SQUASHFS is not set
# CONFIG_FEATURE_VOLUMEID_SYSV is not set
# CONFIG_FEATURE_VOLUMEID_UDF is not set
# CONFIG_FEATURE_VOLUMEID_XFS is not set


> --
> Otavio Salvador O.S. Systems
> http://www.ossystems.com.brhttp://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 7/8] busybox: disable fstrim in defconfig

[Andre McCurdy]

On Wed, Jul 29, 2015 at 9:28 AM, Otavio Salvador
 wrote:
> On Wed, Jul 29, 2015 at 11:22 AM, Burton, Ross  wrote:
>>
>> On 29 July 2015 at 00:21, Andre McCurdy  wrote:
>>>
>>> The fstrim applet (to discard unused blocks on a mounted filesystem)
>>> doesn't seem like core functionality, so disable by default.
>>
>>
>> With SSDs being more common, is this really true?  What's the size impact of
>> this?
>
> I agree with Ross here and I think it should stay enabled.

I was thinking most people with an SSD would "mount -o discard" and
let the filesystem take care of issuing trim commands. Trimming
manually from a shell script seems like more of a niche requirement.

The fstrim applet adds less than 200 bytes to the busybox binary
though (x86). I'll send an updated patch series which leaves it
enabled.

> --
> Otavio Salvador O.S. Systems
> http://www.ossystems.com.brhttp://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] watchdog: kill update-rc.d when not using sysvinit

[Burton, Ross]

On 29 July 2015 at 19:05, Rehman, Abdur  wrote:

> watchdog_5.14.bb does not inherit systemd, and so
> INHIBIT_UPDATERCD_BBCLASS does
> not get set.
>
> While trying to build watchdog with systemd in DISTRO_FEATURES but not
> sysvinit,
> following warning is observed:
> WARNING: QA Issue: watchdog rdepends on initscripts-functions, but it
> isn't
> a build dependency? [build-deps]
>
> Since sysvinit is not in DISTRO_FEATURES and INHIBIT_UPDATERCD_BBCLASS is
> not
> set, the logic in update-rc.d.bbclass goes on to add an rdep for watchdog
> on
> initscripts-functions, inside update_rcd_package(). And this results in the
> QA warning.
>

When a class inherits update-rcd but not systemd, and the distro is systemd
only, update-rcd still runs (and has still run) as otherwise you don't get
any init scripts.  Setting inhibit will remove the init scripts, and the
watchdog daemon won't start.  I'm pretty sure this isn't what you intended.

Are you overriding VIRTUAL-RUNTIME_initscripts in your distro?  update-rcd
adds a dependency on that when it is inherited, which by default is
initscripts, so you get your build dependency there.  I suspect your distro
is overriding that variable when perhaps it shouldn't be.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4 4/5] gstreamer1.0-plugins-base: Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF related patch

[Carlos Rafael Giani]

Hi Zhu,

This backport seems useful. If I understand it correctly, the main 
benefit is that buffers that come from the video decoder won't be copied 
when make_writable() is called, which otherwise would happen with this 
patch (because the video decoder still has a reference to the buffer). 
The result is that it gets rid of unnecessary buffer copies. This is 
your main reason for these patches, right? Can you describe a use case 
where these unnecessary copies caused you trouble? It would be helpful 
for further understanding.


cheers,
  Carlos

Am 2015-07-29 um 20:25 schrieb Yuqing Zhu:

-Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF
  This makes sure that the buffer is not reffed another time when
  storing it in the GstVideoFrame, keeping it writable if it was writable.

-Use new GST_VIDEO_FRAME_MAP_FLAG_NO_REF to replace the old one because it's 
kind of ugly.

-Don't ref buffers twice when mapping

Signed-off-by: Yuqing Zhu 
---
  ...rame-Don-t-ref-buffers-twice-when-mapping.patch | 26 +++
  ...frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch | 87 ++
  ...r-Use-new-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch | 69 +
  .../gstreamer/gstreamer1.0-plugins-base_1.4.5.bb   |  3 +
  4 files changed, 185 insertions(+)
  create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-video-frame-Don-t-ref-buffers-twice-when-mapping.patch
  create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-video-frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch
  create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0003-videofilter-Use-new-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-video-frame-Don-t-ref-buffers-twice-when-mapping.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-video-frame-Don-t-ref-buffers-twice-when-mapping.patch
new file mode 100644
index 000..3db4724
--- /dev/null
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-video-frame-Don-t-ref-buffers-twice-when-mapping.patch
@@ -0,0 +1,26 @@
+From 269f642c45d85cfd630ed490478e6bd6b71a767f Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= 
+Date: Tue, 16 Sep 2014 01:07:18 +0300
+Subject: [PATCH] video-frame: Don't ref buffers twice when mapping
+
+Upstream-Status: Backport [1.5.1]
+---
+ gst-libs/gst/video/video-frame.c |2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/gst-libs/gst/video/video-frame.c 
b/gst-libs/gst/video/video-frame.c
+index 01f23c0..8a9ae96 100644
+--- a/gst-libs/gst/video/video-frame.c
 b/gst-libs/gst/video/video-frame.c
+@@ -105,7 +105,7 @@ gst_video_frame_map_id (GstVideoFrame * frame, 
GstVideoInfo * info,
+   frame->data[i] = frame->map[0].data + info->offset[i];
+ }
+   }
+-  frame->buffer = gst_buffer_ref (buffer);
++  frame->buffer = buffer;
+   if ((flags & GST_VIDEO_FRAME_MAP_FLAG_NO_REF) == 0)
+ gst_buffer_ref (frame->buffer);
+
+--
+1.7.9.5
+
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-video-frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-video-frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch
new file mode 100644
index 000..c465b5c
--- /dev/null
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-video-frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch
@@ -0,0 +1,87 @@
+From 40a293d44d1aeccf5eb8e86f23a0b13666111c5c Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= 
+Date: Fri, 12 Sep 2014 14:39:16 +0300
+Subject: [PATCH 2/3] video-frame: Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF
+
+This makes sure that the buffer is not reffed another time when
+storing it in the GstVideoFrame, keeping it writable if it was
+writable.
+
+Upstream-Status: Backport [1.5.1]
+https://bugzilla.gnome.org/show_bug.cgi?id=736118
+---
+ gst-libs/gst/video/video-frame.c |9 -
+ gst-libs/gst/video/video-frame.h |   18 ++
+ 2 files changed, 26 insertions(+), 1 deletion(-)
+
+diff --git a/gst-libs/gst/video/video-frame.c 
b/gst-libs/gst/video/video-frame.c
+index 537cf70..01f23c0 100644
+--- a/gst-libs/gst/video/video-frame.c
 b/gst-libs/gst/video/video-frame.c
+@@ -106,6 +106,9 @@ gst_video_frame_map_id (GstVideoFrame * frame, 
GstVideoInfo * info,
+ }
+   }
+   frame->buffer = gst_buffer_ref (buffer);
++  if ((flags & GST_VIDEO_FRAME_MAP_FLAG_NO_REF) == 0)
++gst_buffer_ref (frame->buffer);
++
+   frame->meta = meta;
+
+   /* buffer flags enhance the frame flags */
+@@ -189,11 +192,13 @@ gst_video_frame_unmap (GstVideoFrame * frame)
+   GstBuffer *buffer;
+   GstVideoMeta *meta;
+   gint i;
++  GstMapFlags flags;
+
+   g_return_if_fail (frame != NULL);
+
+   buffer = frame->buffer;
+   meta = frame->meta;
++  flags = frame->map[0].flags;
+
+   if (meta) {
+ for (i = 0; i < frame->info.finfo->n_planes; 

Re: [OE-core] [PATCH] watchdog: kill update-rc.d when not using sysvinit

[Rehman, Abdur]

watchdog_5.14.bb does not inherit systemd, and so INHIBIT_UPDATERCD_BBCLASS does
not get set.

While trying to build watchdog with systemd in DISTRO_FEATURES but not sysvinit,
following warning is observed:
WARNING: QA Issue: watchdog rdepends on initscripts-functions, but it isn't
a build dependency? [build-deps]

Since sysvinit is not in DISTRO_FEATURES and INHIBIT_UPDATERCD_BBCLASS is not
set, the logic in update-rc.d.bbclass goes on to add an rdep for watchdog on
initscripts-functions, inside update_rcd_package(). And this results in the
QA warning.

The patch sets INHIBIT_UPDATERCD_BBCLASS in the recipe to force
populate_packages_updatercd() not to proceed with update_rcd_package().

Kindly suggest if there is a better way to tackle this issue.

Abdur Rehman

From: Burton, Ross [ross.bur...@intel.com]
Sent: Wednesday, July 29, 2015 1:46 AM
To: Rehman, Abdur
Cc: OE-core; Larson, Chris
Subject: Re: [OE-core] [PATCH] watchdog: kill update-rc.d when not using 
sysvinit


On 28 July 2015 at 18:11, Abdur Rehman 
mailto:abdur_reh...@mentor.com>> wrote:
+python () {
+if not bb.utils.contains('DISTRO_FEATURES', 'sysvinit', True, False, d):
+d.setVar("INHIBIT_UPDATERCD_BBCLASS", "1")
+}

Isn't this what the update-rcd logic does already:

# Check that this class isn't being inhibited (generally, by
# systemd.bbclass) before doing any work.
if bb.utils.contains('DISTRO_FEATURES', 'sysvinit', True, False, d) or \
   not d.getVar("INHIBIT_UPDATERCD_BBCLASS", True):

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 4/6] devtool: also load plugins from BBPATH

[Christopher Larson]

On Wed, Jul 29, 2015 at 2:56 AM, Paul Eggleton <
paul.eggle...@linux.intel.com> wrote:

> Hmm, I guess we will have to add a command-line parameter to specify BBPATH
> (or the full plugin search path) so that tinfoil doesn't have to be
> instantiated when it's specified.
>

Indeed, sorry about that, I didn’t even realize populate_sdk_ext used
devtool from a bbclass :)
-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 8/8] busybox: disable volumeid support for exFAT, F2FS and NILFS

[Otavio Salvador]

On Tue, Jul 28, 2015 at 8:21 PM, Andre McCurdy  wrote:
> These are less common filesystem formats, so disable volumeid support
> for them by default.
>
> Signed-off-by: Andre McCurdy 

F2FS could be of use. It is being more adopted now that Android is
starting to be deployed to some devices with it.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 7/8] busybox: disable fstrim in defconfig

[Otavio Salvador]

On Wed, Jul 29, 2015 at 11:22 AM, Burton, Ross  wrote:
>
> On 29 July 2015 at 00:21, Andre McCurdy  wrote:
>>
>> The fstrim applet (to discard unused blocks on a mounted filesystem)
>> doesn't seem like core functionality, so disable by default.
>
>
> With SSDs being more common, is this really true?  What's the size impact of
> this?

I agree with Ross here and I think it should stay enabled.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/2] bitbake.conf: add GITHUB_GIT

[Otavio Salvador]

On Wed, Jul 29, 2015 at 8:48 AM, Olof Johansson  wrote:
> Excerpts from Robert Yang's message of 2015-07-29 12:19:06 +0200:
>> It is just like what GNOME_GIT, GNU_MIRROR and others did.
>
> Tbh, I don't understand them either, but maybe that's just me. What is
> the reason? I think it makes it harder to see what is happening.

It allows for people to add local mirrors for example and even remap
fetching without touching the recipes. It is a good addition IMO.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7

[Otavio Salvador]

On Wed, Jul 29, 2015 at 8:49 PM, Yuqing Zhu  wrote:
> Revert "launch: Avoid specifically starting PA and rely on autospawn/socket 
> activation"
> or the pulseaudio can't be opened in yocto.
>
> Signed-off-by: Yuqing Zhu 

Can you explain what this revert tries to fix?

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [poky][PATCH v4] pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7

[Yuqing Zhu]

Revert "launch: Avoid specifically starting PA and rely on autospawn/socket 
activation"
or the pulseaudio can't be opened in yocto.

Signed-off-by: Yuqing Zhu 
---
 ...ch-Avoid-specifically-starting-PA-and-rel.patch | 31 ++
 .../pulseaudio/pulseaudio_6.0.bb   |  1 +
 2 files changed, 32 insertions(+)
 create mode 100644 
meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch

diff --git 
a/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
 
b/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
new file mode 100644
index 000..669c355
--- /dev/null
+++ 
b/meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch
@@ -0,0 +1,31 @@
+From 5760c6b1a3c9b6dd8e70c9d1ea78fcf192250b4a Mon Sep 17 00:00:00 2001
+From: Shengjiu Wang 
+Date: Mon, 16 Mar 2015 15:30:11 +0800
+Subject: [PATCH] Revert "launch: Avoid specifically starting PA and rely on
+ autospawn/socket activation"
+
+This reverts commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7.
+
+Upstream-Status: Pending 
+
+Signed-off-by: shengjiu.wang 
+---
+ src/daemon/start-pulseaudio-x11.in |2 ++
+ 1 file changed, 2 insertions(+)
+
+diff --git a/src/daemon/start-pulseaudio-x11.in 
b/src/daemon/start-pulseaudio-x11.in
+index 16e46b1..e108645 100755
+--- a/src/daemon/start-pulseaudio-x11.in
 b/src/daemon/start-pulseaudio-x11.in
+@@ -17,6 +17,8 @@
+ 
+ set -e
+ 
++@PA_BINARY@ --start "$@"
++
+ if [ x"$DISPLAY" != x ] ; then
+ 
+ @PACTL_BINARY@ load-module module-x11-publish "display=$DISPLAY" > 
/dev/null
+-- 
+1.7.9.5
+
diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb 
b/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
index 31e9096..10c3d7a 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_6.0.bb
@@ -6,6 +6,7 @@ SRC_URI = 
"http://freedesktop.org/software/pulseaudio/releases/${BP}.tar.xz \
file://0001-conf-parser-add-support-for-.d-directories.patch \
file://fix-git-version-gen.patch \
file://volatiles.04_pulse \
+   
file://0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch \
 "
 SRC_URI[md5sum] = "b691e83b7434c678dffacfa3a027750e"
 SRC_URI[sha256sum] = 
"b50640e0b80b1607600accfad2e45aabb79d379bf6354c9671efa2065477f6f6"
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [poky][PATCH v4] pulseaudio

[Yuqing Zhu]

Fix no process to open pulseaudio in yocto. 

Yuqing Zhu (1):
  pulseaudio: revert commit 42156d2b5ac797e5f28f8d0d38b691053f3f6fc7

 ...ch-Avoid-specifically-starting-PA-and-rel.patch | 31 ++
 .../pulseaudio/pulseaudio_6.0.bb   |  1 +
 2 files changed, 32 insertions(+)
 create mode 100644 
meta/recipes-multimedia/pulseaudio/pulseaudio/0002-Revert-launch-Avoid-specifically-starting-PA-and-rel.patch

-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [oe-commits] Ross Burton : gamin: remove

[Martin Jansa]

On Mon, Jul 20, 2015 at 09:41:46AM +, g...@git.openembedded.org wrote:
> Module: openembedded-core.git
> Branch: master
> Commit: d29595925b699827fbd3279ee5368e32e0a380f3
> URL:
> http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=d29595925b699827fbd3279ee5368e32e0a380f3
> 
> Author: Ross Burton 
> Date:   Thu Jul 16 23:24:06 2015 +0100
> 
> gamin: remove
> 
> Gamin hasn't been required for many years now, maintenance upstream stopped in
> 2008 and the addition of inotify to the kernel made it redundant.
> 
> Remove the recipe from oe-core, and references to it in pacakgegroup-core-lsb
> (LSB doesn't require gamin) and packagegroup-self-hosted.
> 
> Signed-off-by: Ross Burton 

Someone interested in fixing tracer recipe?

NOTE: Resolving any missing task queue dependencies
ERROR: Nothing PROVIDES 'gamin' (but 
/home/jenkins/oe/world/shr-core/meta-openembedded/meta-gnome/recipes-support/tracker/tracker_0.14.2.bb
 DEPENDS on or otherwise requires it)

> 
> ---
> 
>  .../packagegroups/packagegroup-self-hosted.bb  |  1 -
>  .../gamin-0.1.10/obsolete_automake_macros.patch| 23 -
>  .../gamin/gamin/no-abstract-sockets.patch  | 56 
>  meta/recipes-extended/gamin/gamin/noconst.patch| 59 
> --
>  meta/recipes-extended/gamin/gamin_0.1.10.bb| 36 -
>  .../packagegroups/packagegroup-core-lsb.bb |  1 -
>  6 files changed, 176 deletions(-)
> 
> diff --git a/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb 
> b/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
> index 531f729..7d5a41a 100644
> --- a/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
> +++ b/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
> @@ -118,7 +118,6 @@ RDEPENDS_packagegroup-self-hosted-extended = "\
>  diffutils \
>  elfutils \
>  expat \
> -gamin \
>  gawk \
>  gdbm \
>  gettext \
> diff --git 
> a/meta/recipes-extended/gamin/gamin-0.1.10/obsolete_automake_macros.patch 
> b/meta/recipes-extended/gamin/gamin-0.1.10/obsolete_automake_macros.patch
> deleted file mode 100644
> index c4300ca..000
> --- a/meta/recipes-extended/gamin/gamin-0.1.10/obsolete_automake_macros.patch
> +++ /dev/null
> @@ -1,23 +0,0 @@
> -Upstream-Status: Submitted 
> [https://bugzilla.gnome.org/show_bug.cgi?id=693449]
> -
> -Signed-off-by: Marko Lindqvist 
> -diff -Nurd gamin-0.1.10/configure.in gamin-0.1.10/configure.in
>  gamin-0.1.10/configure.in2008-11-24 17:23:06.0 +0200
> -+++ gamin-0.1.10/configure.in2013-02-08 23:11:59.656279992 +0200
> -@@ -1,7 +1,7 @@
> - dnl Process this file with autoconf to produce a configure script.
> - AC_PREREQ(2.52)
> - AC_INIT(libgamin)
> --AM_CONFIG_HEADER(config.h)
> -+AC_CONFIG_HEADERS(config.h)
> - AC_CANONICAL_SYSTEM
> -
> - # get any external flags setting before we start playing with the CFLAGS 
> variable
> -@@ -31,7 +31,6 @@
> -
> - AC_USE_SYSTEM_EXTENSIONS
> - AC_PROG_CC
> --AM_PROG_CC_STDC
> - AC_HEADER_STDC
> - AC_PROG_INSTALL
> - AC_PROG_MAKE_SET
> diff --git a/meta/recipes-extended/gamin/gamin/no-abstract-sockets.patch 
> b/meta/recipes-extended/gamin/gamin/no-abstract-sockets.patch
> deleted file mode 100644
> index cddf2dc..000
> --- a/meta/recipes-extended/gamin/gamin/no-abstract-sockets.patch
> +++ /dev/null
> @@ -1,56 +0,0 @@
> -Upstream-Status: Inappropriate [configuration]
> -
>  /tmp/configure.in2007-04-30 13:08:49.0 +0200
> -+++ gamin-0.1.8/configure.in 2007-04-30 13:10:53.285251000 +0200
> -@@ -354,51 +354,6 @@
> - AC_DEFINE(HAVE_CMSGCRED,1,[Have cmsgcred structure])
> - fi
> - 
> -- Abstract sockets
> --
> --AC_MSG_CHECKING(abstract socket namespace)
> --AC_LANG_PUSH(C)
> --AC_RUN_IFELSE([AC_LANG_PROGRAM(
> --[[
> --#include 
> --#include 
> --#include 
> --#include 
> --#include 
> --#include 
> --#include 
> --]],
> --[[
> --  int listen_fd;
> --  struct sockaddr_un addr;
> --  
> --  listen_fd = socket (PF_UNIX, SOCK_STREAM, 0);
> --  
> --  if (listen_fd < 0)
> --{
> --  fprintf (stderr, "socket() failed: %s\n", strerror (errno));
> --  exit (1);
> --}
> --
> --  memset (&addr, '\0', sizeof (addr));
> --  addr.sun_family = AF_UNIX;
> --  strcpy (addr.sun_path, 
> "X/tmp/dbus-fake-socket-path-used-in-configure-test");
> --  addr.sun_path[0] = '\0'; /* this is what makes it abstract */
> --  
> --  if (bind (listen_fd, (struct sockaddr*) &addr, SUN_LEN (&addr)) < 0)
> --{
> --   fprintf (stderr, "Abstract socket namespace bind() failed: %s\n", 
> --strerror (errno));
> --   exit (1);
> --}
> --  else 
> --exit (0);
> --]])],
> --  [have_abstract_sockets=yes],
> --  [have_abstract_sockets=no])
> --AC_LANG_POP(C)
> --AC_MSG_RESULT($have_abstract_sockets)
> --
> - if test x$enable_abstract_sockets = xyes; then
> - if test x$have_abstract_sockets = xno; then
> - AC_MSG_ERROR([Abstract sockets

Re: [OE-core] [PATCH 0/4] Fix builds for MIPS64 N32

[Dmitry Eremin-Solenikov]

2015-07-29 17:58 GMT+03:00 Mark Hatle :
> On 7/28/15 12:26 PM, Dmitry Eremin-Solenikov wrote:
>> 2015-07-21 18:26 GMT+03:00 Mark Hatle :
>>> On 7/21/15 9:31 AM, Mark Hatle wrote:
 On 7/21/15 3:23 AM, Dmitry Eremin-Solenikov wrote:
> Hello,
>
> 2015-07-20 18:47 GMT+03:00 Mark Hatle :
>> Between customer escalations, other work and vacation.. I didn't happen.
>
> No problem, really. I just wanted to be sure that the patches aren't going
> to be lost.
>
>>
>> I've started testing it this morning and should have results tomorrow.. 
>> (US time).
>
> Thank you very much!

 With master and the four patches I got an error trying to construct the 
 SDK.

 I'm still investigating the error:

 Collected errors:
  * satisfy_dependencies_for: Cannot satisfy the following dependencies for
 packagegroup-cross-canadian-qemumips64:
  *  gcc-cross-canadian-mips *   binutils-cross-canadian-mips *
 gdb-cross-canadian-mips *
  * opkg_install_cmd: Cannot install package 
 packagegroup-cross-canadian-qemumips64.


 My configuration for this particular test was:

 (oe-init-build-env stock w/ the following changes)
>>>
>>> I've narrowed down the reproducer to just:
>>>
>>> MACHINE = "qemumips64"
>>> require conf/multilib.conf
>>> DEFAULTTUNE = "mips64-n32"
>>> MULTILIBS = ""
>>>
>>> (If you remove the require conf/multilib.conf then things work.)
>>>
>>> I'm going to continue to investigate.
>>
>> Any progress on your side? As I failed to reproduce the issue on my side,
>> could you please specify:
>> 1) what is the exact bitbake command that failed?
>> 2) what additional layers do you have enabled in your build?
>> 3) is it a cleanroom build, or you have an sstate cache populated with 
>> previous
>>build results?
>> 4) any additional information that I can use to reproduce the issue on my 
>> side?
>
> Recent check-ins have changed some things.  The compile side was working for a
> while, but not the compilation side (using it).
>
> I've been working with RP on IRC, and he's come up with some additional 
> patches.
>  (including this set of 4..)
>
> So I think the set is fine, it's up to the other pieces that need to mesh with
> this to produce the working SDK.

Ok. So what would be the plan then? If you need any additional tests
from my side,
I can provide them.

>
> --Mark
>
>>>
>>> --Mark
>>>
 MACHINE = "qemumips64"

 require conf/multilib.conf
 DEFAULTTUNE = "mips64-n32"
 MULTILIBS = "multilib:lib64 multilib:lib32"
 DEFAULTTUNE_virtclass-multilib-lib64 = "mips64"
 DEFAULTTUNE_virtclass-multilib-lib32 = "mips32r2"

 # Set RPM_PREFER_ELF_ARCH to configure preferred ABI when using rpm 
 packaging
 # backend to generate a rootfs, choices are:
 # 1: ELF32 wins
 # 2: ELF64 wins
 # 4: ELF64 N32 wins (for mips64 or mips64el only)
 RPM_PREFER_ELF_ARCH = "4"


>>
>> --Mark
>>
>> On 7/20/15 7:11 AM, Dmitry Eremin-Solenikov wrote:
>>> 2015-07-08 15:42 GMT+03:00 Mark Hatle :
 On 7/8/15 7:00 AM, Dmitry Eremin-Solenikov wrote:
> Hello,
>
> 2015-07-01 17:02 GMT+03:00 Dmitry Eremin-Solenikov 
> :
>> Quite simple configuration (MIPS64 N32 image) fails to build (see 
>> below).
>> I have verified these patches in both single lib and in multilib 
>> cases.
>> Tri-lib images and SDK for MIPS build fine with these patches.
>
> Any comments on this patchset? It would be nice to have N32 ubroken in
> the current
> master tree.

 Sorry, I'm traveling right now and haven't had time to review this.  I 
 should be
 able to get back to the office and test it by the end of the week.

 I visually reviewed the patches and I don't object to them going in -- 
 but I'm
 still not sure they are all correct.
>>>
>>> Re-ping. I'd like to receive a confirmation from you (and thus get the
>>> patches merged)
>>> or to discuss, what seems wrong to you.
>>>

 --Mark

>>
>> Current situation:
>>
>> lumag@nexs:~/OE$ MACHINE=qemumips64n32 bitbake core-image-base
>> NOTE: Started PRServer with DBfile:
>> /home/lumag/OE/cache/prserv.sqlite3, IP: 127.0.0.1, PORT: 46391, PID:
>> 15895
>> Loading cache: 100%
>> ||
>> ETA:  00:00:00
>> Loaded 1302 entries from dependency cache.
>> NOTE: Resolving any missing task queue dependencies
>> ERROR: Nothing RPROVIDES 'binutils-cross-canadian-mips64' (but
>> /home/lumag/OE/sources/openembedded-core/meta/recipes-core/packagegroups/packagegroup-cross-canadian.bb
>

Re: [OE-core] [PATCH 0/4] Fix builds for MIPS64 N32

[Mark Hatle]

On 7/28/15 12:26 PM, Dmitry Eremin-Solenikov wrote:
> 2015-07-21 18:26 GMT+03:00 Mark Hatle :
>> On 7/21/15 9:31 AM, Mark Hatle wrote:
>>> On 7/21/15 3:23 AM, Dmitry Eremin-Solenikov wrote:
 Hello,

 2015-07-20 18:47 GMT+03:00 Mark Hatle :
> Between customer escalations, other work and vacation.. I didn't happen.

 No problem, really. I just wanted to be sure that the patches aren't going
 to be lost.

>
> I've started testing it this morning and should have results tomorrow.. 
> (US time).

 Thank you very much!
>>>
>>> With master and the four patches I got an error trying to construct the SDK.
>>>
>>> I'm still investigating the error:
>>>
>>> Collected errors:
>>>  * satisfy_dependencies_for: Cannot satisfy the following dependencies for
>>> packagegroup-cross-canadian-qemumips64:
>>>  *  gcc-cross-canadian-mips *   binutils-cross-canadian-mips *
>>> gdb-cross-canadian-mips *
>>>  * opkg_install_cmd: Cannot install package 
>>> packagegroup-cross-canadian-qemumips64.
>>>
>>>
>>> My configuration for this particular test was:
>>>
>>> (oe-init-build-env stock w/ the following changes)
>>
>> I've narrowed down the reproducer to just:
>>
>> MACHINE = "qemumips64"
>> require conf/multilib.conf
>> DEFAULTTUNE = "mips64-n32"
>> MULTILIBS = ""
>>
>> (If you remove the require conf/multilib.conf then things work.)
>>
>> I'm going to continue to investigate.
> 
> Any progress on your side? As I failed to reproduce the issue on my side,
> could you please specify:
> 1) what is the exact bitbake command that failed?
> 2) what additional layers do you have enabled in your build?
> 3) is it a cleanroom build, or you have an sstate cache populated with 
> previous
>build results?
> 4) any additional information that I can use to reproduce the issue on my 
> side?

Recent check-ins have changed some things.  The compile side was working for a
while, but not the compilation side (using it).

I've been working with RP on IRC, and he's come up with some additional patches.
 (including this set of 4..)

So I think the set is fine, it's up to the other pieces that need to mesh with
this to produce the working SDK.

--Mark

>>
>> --Mark
>>
>>> MACHINE = "qemumips64"
>>>
>>> require conf/multilib.conf
>>> DEFAULTTUNE = "mips64-n32"
>>> MULTILIBS = "multilib:lib64 multilib:lib32"
>>> DEFAULTTUNE_virtclass-multilib-lib64 = "mips64"
>>> DEFAULTTUNE_virtclass-multilib-lib32 = "mips32r2"
>>>
>>> # Set RPM_PREFER_ELF_ARCH to configure preferred ABI when using rpm 
>>> packaging
>>> # backend to generate a rootfs, choices are:
>>> # 1: ELF32 wins
>>> # 2: ELF64 wins
>>> # 4: ELF64 N32 wins (for mips64 or mips64el only)
>>> RPM_PREFER_ELF_ARCH = "4"
>>>
>>>
>
> --Mark
>
> On 7/20/15 7:11 AM, Dmitry Eremin-Solenikov wrote:
>> 2015-07-08 15:42 GMT+03:00 Mark Hatle :
>>> On 7/8/15 7:00 AM, Dmitry Eremin-Solenikov wrote:
 Hello,

 2015-07-01 17:02 GMT+03:00 Dmitry Eremin-Solenikov 
 :
> Quite simple configuration (MIPS64 N32 image) fails to build (see 
> below).
> I have verified these patches in both single lib and in multilib 
> cases.
> Tri-lib images and SDK for MIPS build fine with these patches.

 Any comments on this patchset? It would be nice to have N32 ubroken in
 the current
 master tree.
>>>
>>> Sorry, I'm traveling right now and haven't had time to review this.  I 
>>> should be
>>> able to get back to the office and test it by the end of the week.
>>>
>>> I visually reviewed the patches and I don't object to them going in -- 
>>> but I'm
>>> still not sure they are all correct.
>>
>> Re-ping. I'd like to receive a confirmation from you (and thus get the
>> patches merged)
>> or to discuss, what seems wrong to you.
>>
>>>
>>> --Mark
>>>
>
> Current situation:
>
> lumag@nexs:~/OE$ MACHINE=qemumips64n32 bitbake core-image-base
> NOTE: Started PRServer with DBfile:
> /home/lumag/OE/cache/prserv.sqlite3, IP: 127.0.0.1, PORT: 46391, PID:
> 15895
> Loading cache: 100%
> ||
> ETA:  00:00:00
> Loaded 1302 entries from dependency cache.
> NOTE: Resolving any missing task queue dependencies
> ERROR: Nothing RPROVIDES 'binutils-cross-canadian-mips64' (but
> /home/lumag/OE/sources/openembedded-core/meta/recipes-core/packagegroups/packagegroup-cross-canadian.bb
> RDEPENDS on or otherwise requires it)
> NOTE: Runtime target 'binutils-cross-canadian-mips64' is unbuildable,
> removing...
> Missing or unbuildable dependency chain was: 
> ['binutils-cross-canadian-mips64']
> NOTE: Runtime target 'packagegroup-cross-

Re: [OE-core] [PATCH 7/8] busybox: disable fstrim in defconfig

[Burton, Ross]

On 29 July 2015 at 00:21, Andre McCurdy  wrote:

> The fstrim applet (to discard unused blocks on a mounted filesystem)
> doesn't seem like core functionality, so disable by default.
>

With SSDs being more common, is this really true?  What's the size impact
of this?

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH][resend] perl: fix build failure if building dir has the "blib" string

[Burton, Ross]

On 29 July 2015 at 04:08, Rongqing Li  wrote:

> I have update the perl bug report
>
> https://rt.perl.org/Public/Bug/Display.html?id=125603
>

Upstream now appears to agree with you - merging to mut. :)

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH v2 0/2] Yocto Bug #6945

[Bruce Ashfield]

On 15-07-29 03:32 AM, He Zhe wrote:

Ping.

Is there any comments on this?


I'm obviously ok with the patch in its current RFC status. It works
and I've tested it in my tree (hence why I've been quiet on this
one).

If it meets the style for merging into core, and that it doesn't
break other layers we can't see .. I was hoping to hear from others.

Bruce



Zhe

On 07/28/2015 11:17 AM, He Zhe wrote:

Ping.

On 07/27/2015 10:51 AM, He Zhe wrote:

Ping.

On 07/23/2015 03:48 PM, He Zhe wrote:

Ping.

On 07/21/2015 03:23 PM, zhe...@windriver.com wrote:

From: He Zhe 

  - To support building packaging and installing multi types of kernel
images, such as zImage uImage, at one time define KERNEL_IMAGETYPE
as a list.
  - Modify wherever reference KERNEL_IMAGETYPE accordingly.
  - Pass mkimage in sysroot to kernel makefile by NATIVE_MKIMAGE to avoid
depending on build machine's, when KEEPUIMAGE is "yes".
  - v2: update with the latest oe-core

He Zhe (2):
   kernel: Define KERNEL_IMAGETYPE as a list
   kernel: Pass sysroot mkimage to kernel makefile

  meta/classes/image_types.bbclass |   6 +-
  meta/classes/kernel-fitimage.bbclass |  22 +++---
  meta/classes/kernel-grub.bbclass |  47 
  meta/classes/kernel-uimage.bbclass   |  31 +---
  meta/classes/kernel.bbclass  | 127 ++-
  meta/conf/documentation.conf |   2 +-
  meta/lib/oeqa/controllers/masterimage.py |   2 +-
  meta/lib/oeqa/targetcontrol.py   |   2 +-
  meta/recipes-kernel/linux/linux-dtb.inc  |  15 ++--
  scripts/test-remote-image|   2 +-
  10 files changed, 175 insertions(+), 81 deletions(-)





--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] bitbake.conf: Remove replace call from MACHINE_ARCH

[Ioan-Adrian Ratiu]

This replace() hack was added to fix Yocto bug #946 [1] with commit id
69b3a11. It is time to fix this properly. It should not be an issue
anymore with libzypp after 4.5 years, but if the initial bug still
exists, then a more intelligent solution should be used to fix it.

If a package build fails because it expects _ instead of - inside
MACHINE_ARCH, then the problem is with the package build, not with the
global variable used by all machines.

*IF* the offending package build cannot be easily fixed, then there
are better workarounds than globally replacing. For example overwriting
MACHINE_ARCH in the .conf which is causing the offending
package to fail (thus not screwing all), or using conditional overrides
MACHINE_ARCH_qemux86-64 = "qemux86_64"

Mangling the variable globally for all machines causes other components
that expect the sane MACHINE_ARCH value to fail.

[1] https://bugzilla.yoctoproject.org/show_bug.cgi?id=946

Signed-off-by: Ioan-Adrian Ratiu 
---
 meta/conf/bitbake.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 730ab80..19e7ddf 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -136,7 +136,7 @@ SDK_AS_ARCH = "${BUILD_AS_ARCH}"
 
 TUNE_PKGARCH ??= ""
 PACKAGE_ARCH ??= "${TUNE_PKGARCH}"
-MACHINE_ARCH = "${@[d.getVar('TUNE_PKGARCH', True), d.getVar('MACHINE', 
True)][bool(d.getVar('MACHINE', True))].replace('-', '_')}"
+MACHINE_ARCH = "${@[d.getVar('TUNE_PKGARCH', True), d.getVar('MACHINE', 
True)][bool(d.getVar('MACHINE', True))]}"
 PACKAGE_EXTRA_ARCHS ??= "${PACKAGE_EXTRA_ARCHS_tune-${DEFAULTTUNE}}"
 PACKAGE_ARCHS = "all any noarch ${PACKAGE_EXTRA_ARCHS} ${MACHINE_ARCH}"
 # MACHINE_ARCH shouldn't be included here as a variable dependency
-- 
2.1.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 2/9] file: 5.23 -> 5.24

[Richard Purdie]

On Tue, 2015-07-28 at 15:48 +0200, Martin Jansa wrote:
> On Tue, Jul 28, 2015 at 02:08:20PM +0100, Burton, Ross wrote:
> > On 28 July 2015 at 03:12, Robert Yang  wrote:
> > 
> > > * Use git repo rather than tarball since the original SRC_URI is not
> > >   stable, it is not reachable sometimes.
> > >
> > 
> > Personally speaking I see this as a recipe for a release of file which just
> > happens to use git to fetch the tarball (and presumably that hash is the
> > 5.24 tag).  In situations like this I prefer to see the recipe called
> > file_5.24.bb as there's no point having a giant PV including git SHAs when
> > it's just the release.
> 
> until someone sets SRCREV_pn-file = "${AUTOREV}" somewhere and git SHAs
> in PV suddenly gets very useful.
> 
> Personally speaking I prefer recipes with git SCM in SRC_URI named
> _git.bb and PV set to "A.B.C+gitr${SRCPV}".

Having thought a bit about this, I have to admit I have a preference for
not doing this, just so we can know it is meant to be file 5.24 and not
that version plus some random git commits. If you override SRCREV, you
can override PV too to match the SRCREV.

Perhaps more importantly, its also more useful for the version
comparisons in the update tracking tooling. That might tip the balance.

Cheers,

Richard

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4 2/5] gstreamer1.0-plugins-base: handle audio/video decoder error

[Otavio Salvador]

On Wed, Jul 29, 2015 at 3:25 PM, Yuqing Zhu  wrote:
> When there is input data and no output data to the end of the stream, it will
> send GST_ELEMENT_ERROR and quit from playing.
> The patch comments the GST_ELEMENT_ERROR() and just add GST_ERROR_OBJECT()
> information instead.
>
> Signed-off-by: Yuqing Zhu 

Carlos?

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4 4/5] gstreamer1.0-plugins-base: Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF related patch

[Otavio Salvador]

On Wed, Jul 29, 2015 at 9:42 AM, Otavio Salvador
 wrote:
> On Wed, Jul 29, 2015 at 3:25 PM, Yuqing Zhu  wrote:
>> -Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF
>>  This makes sure that the buffer is not reffed another time when
>>  storing it in the GstVideoFrame, keeping it writable if it was writable.
>>
>> -Use new GST_VIDEO_FRAME_MAP_FLAG_NO_REF to replace the old one because it's 
>> kind of ugly.
>>
>> -Don't ref buffers twice when mapping
>>
>> Signed-off-by: Yuqing Zhu 
>
>
> Acked-by: Otavio Salvador 

Carlos, please check this one. This seems to be fine but ...

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH v2] mdadm: 3.3.2 -> 3.3.3

[Burton, Ross]

On 29 July 2015 at 05:13, Li xin  wrote:

> +++ b/meta/recipes-extended/mdadm/files/mdadm.conf
> @@ -0,0 +1 @@
> +MAILADDR r...@mydomain.com
>

Can this line be commented out or at least a better domain used.
 mydomain.comm is a domain registrar so r...@mydomain.com is probably a
real person...  If this *needs* a value and it cant be inferred by mdadm at
runtime then example.com has been reserved for this purpose by the IETF for
16 years now.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4 5/5] gstreamer1.0-plugins-base: Add videoencoder related patch

[Otavio Salvador]

On Wed, Jul 29, 2015 at 3:25 PM, Yuqing Zhu  wrote:
> Keep sticky events around when doing a soft reset.
> The current code will first discard all frames, and then tries to copy
> all sticky events from the (now discarded) frames. So change the order.
>
> Signed-off-by: Yuqing Zhu 


Acked-by: Otavio Salvador 

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4 4/5] gstreamer1.0-plugins-base: Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF related patch

[Otavio Salvador]

On Wed, Jul 29, 2015 at 3:25 PM, Yuqing Zhu  wrote:
> -Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF
>  This makes sure that the buffer is not reffed another time when
>  storing it in the GstVideoFrame, keeping it writable if it was writable.
>
> -Use new GST_VIDEO_FRAME_MAP_FLAG_NO_REF to replace the old one because it's 
> kind of ugly.
>
> -Don't ref buffers twice when mapping
>
> Signed-off-by: Yuqing Zhu 


Acked-by: Otavio Salvador 

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4 3/5] gstreamer1.0-plugins-base: update video alignment after video alignment

[Otavio Salvador]

On Wed, Jul 29, 2015 at 3:25 PM, Yuqing Zhu  wrote:
> Video buffer pool will update video alignment to respect stride alignment
> requirement. But haven't update it to video alignment in configure.
> Which will cause user get wrong video alignment.
>
> Signed-off-by: Yuqing Zhu 


Acked-by: Otavio Salvador 

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v4 1/5] gstreamer1.0-plugins-base: Bug fix for id3demux issue

[Otavio Salvador]

On Wed, Jul 29, 2015 at 3:25 PM, Yuqing Zhu  wrote:
> Use g_utf16_to_utf8() instead of g_convert to fix the issue that
> id3 tags utf16 charaters cannot be extreacted in id3demux when try
> to get the id3v2 tag such as TIT2, TALB etc.
>
> Signed-off-by: Yuqing Zhu 

Acked-by: Otavio Salvador 

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] tzdata: Add marking for config files in recipe

[Burton, Ross]

On 29 July 2015 at 02:49, Li Zhou  wrote:

>  meta/recipes-extended/tzdata/tzdata.inc |2 ++
>

This inc file was removed last month, please rebase to master.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v2 4/6] gstreamer1.0-plugins-base: Add video-frame related patch

[Otavio Salvador]

On Wed, Jul 29, 2015 at 6:22 AM, Zhu Carol  wrote:
>The 1.4.2 fix is ugly, so [OE-core] [poky][PATCH v3 5/6] 
> gstreamer1.0-plugins-base: Add videofilter related patch is used to revert it.
>Actually these patches are all back port from 1.5.1, I will change the 
> Upstream-Status and merge these two patches together.

I am sure Sebastian had a reason to not backport 1.5.1 fix but I am
not in a position to make a call here... Carlos know the code better
and should be the final word on this.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/2] bitbake.conf: add GITHUB_GIT

[Olof Johansson]

Excerpts from Robert Yang's message of 2015-07-29 12:19:06 +0200:
> It is just like what GNOME_GIT, GNU_MIRROR and others did.

Tbh, I don't understand them either, but maybe that's just me. What is
the reason? I think it makes it harder to see what is happening.

-- 
olof johansson
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [poky][PATCH v4 3/5] gstreamer1.0-plugins-base: update video alignment after video alignment

[Yuqing Zhu]

Video buffer pool will update video alignment to respect stride alignment
requirement. But haven't update it to video alignment in configure.
Which will cause user get wrong video alignment.

Signed-off-by: Yuqing Zhu 
---
 .../videobuffer_updata_alignment_update.patch  | 53 ++
 .../gstreamer/gstreamer1.0-plugins-base_1.4.5.bb   |  1 +
 2 files changed, 54 insertions(+)
 create mode 100755 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/videobuffer_updata_alignment_update.patch

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/videobuffer_updata_alignment_update.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/videobuffer_updata_alignment_update.patch
new file mode 100755
index 000..8cfda95
--- /dev/null
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/videobuffer_updata_alignment_update.patch
@@ -0,0 +1,53 @@
+commit 88d253ea23b06289df40401160b606323f16c910
+Author: Song Bing 
+Date:   Mon Dec 15 09:34:35 2014 +0800
+
+videopool: update video alignment after video alignment
+
+Video buffer pool will update video alignment to respect stride alignment
+requirement. But haven't update it to video alignment in configure.
+Which will cause user get wrong video alignment.
+
+https://bugzilla.gnome.org/show_bug.cgi?id=741501
+
+Upstream-Status: Backport [1.5.1]
+
+Signed-off-by: Song Bing 
+diff --git a/gst-libs/gst/video/gstvideopool.c 
b/gst-libs/gst/video/gstvideopool.c
+index 4475f45..acef594 100644
+--- a/gst-libs/gst/video/gstvideopool.c
 b/gst-libs/gst/video/gstvideopool.c
+@@ -167,6 +167,7 @@ video_buffer_pool_set_config (GstBufferPool * pool, 
GstStructure * config)
+ /* get an apply the alignment to the info */
+ gst_buffer_pool_config_get_video_alignment (config, &priv->video_align);
+ gst_video_info_align (&info, &priv->video_align);
++gst_buffer_pool_config_set_video_alignment (config, &priv->video_align);
+   }
+   priv->info = info;
+
+diff --git a/sys/ximage/ximagepool.c b/sys/ximage/ximagepool.c
+index 6cc2cfa..6a1cbc9 100644
+--- a/sys/ximage/ximagepool.c
 b/sys/ximage/ximagepool.c
+@@ -597,6 +597,8 @@ ximage_buffer_pool_set_config (GstBufferPool * pool, 
GstStructure * config)
+ /* do padding and alignment */
+ gst_video_info_align (&info, &priv->align);
+
++gst_buffer_pool_config_set_video_alignment (config, &priv->align);
++
+ /* we need the video metadata too now */
+ priv->add_metavideo = TRUE;
+   } else {
+diff --git a/sys/xvimage/xvimagepool.c b/sys/xvimage/xvimagepool.c
+index 244a51a..34b1ab2 100644
+--- a/sys/xvimage/xvimagepool.c
 b/sys/xvimage/xvimagepool.c
+@@ -124,6 +124,8 @@ xvimage_buffer_pool_set_config (GstBufferPool * pool, 
GstStructure * config)
+ /* do padding and alignment */
+ gst_video_info_align (&info, &priv->align);
+
++gst_buffer_pool_config_set_video_alignment (config, &priv->align);
++
+ /* we need the video metadata too now */
+ priv->add_metavideo = TRUE;
+   } else {
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
index aa90179..88c3254 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
@@ -10,6 +10,7 @@ SRC_URI += 
"file://do-not-change-eos-event-to-gap-event-if.patch \
 file://taglist-not-send-to-down-stream-if-all-the-frame-cor.patch \
 file://fix-id3demux-utf16-to-utf8-issue.patch \
 file://handle-audio-video-decoder-error.patch \
+file://videobuffer_updata_alignment_update.patch \
 "
 
 SRC_URI[md5sum] = "357165af625c0ca353ab47c5d843920e"
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [poky][PATCH v4 5/5] gstreamer1.0-plugins-base: Add videoencoder related patch

[Yuqing Zhu]

Keep sticky events around when doing a soft reset.
The current code will first discard all frames, and then tries to copy
all sticky events from the (now discarded) frames. So change the order.

Signed-off-by: Yuqing Zhu 
---
 ...p-sticky-events-around-when-doing-a-soft-.patch | 46 ++
 .../gstreamer/gstreamer1.0-plugins-base_1.4.5.bb   |  1 +
 2 files changed, 47 insertions(+)
 create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/videoencoder-Keep-sticky-events-around-when-doing-a-soft-.patch

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/videoencoder-Keep-sticky-events-around-when-doing-a-soft-.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/videoencoder-Keep-sticky-events-around-when-doing-a-soft-.patch
new file mode 100644
index 000..b476969
--- /dev/null
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/videoencoder-Keep-sticky-events-around-when-doing-a-soft-.patch
@@ -0,0 +1,46 @@
+From c3f7d36b992a3cbcee3386dea85720f3cb04e1ff Mon Sep 17 00:00:00 2001
+From: Song Bing 
+Date: Fri, 27 Mar 2015 13:39:43 +0800
+Subject: [PATCH] videoencoder: Keep sticky events around when doing a soft
+ reset
+
+The current code will first discard all frames, and then tries to copy
+all sticky events from the (now discarded) frames. Let's change the order.
+
+https://bugzilla.gnome.org/show_bug.cgi?id=746865
+
+Upstream-Status: Backport [1.5.1] 
+
+---
+ gst-libs/gst/video/gstvideoencoder.c |8 
+ 1 file changed, 4 insertions(+), 4 deletions(-)
+
+diff --git a/gst-libs/gst/video/gstvideoencoder.c 
b/gst-libs/gst/video/gstvideoencoder.c
+index 614ba2d..4c5b111 100644
+--- a/gst-libs/gst/video/gstvideoencoder.c
 b/gst-libs/gst/video/gstvideoencoder.c
+@@ -340,10 +340,6 @@ gst_video_encoder_reset (GstVideoEncoder * encoder, 
gboolean hard)
+ 
+   priv->drained = TRUE;
+ 
+-  g_list_foreach (priv->frames, (GFunc) gst_video_codec_frame_unref, NULL);
+-  g_list_free (priv->frames);
+-  priv->frames = NULL;
+-
+   priv->bytes = 0;
+   priv->time = 0;
+ 
+@@ -392,6 +388,10 @@ gst_video_encoder_reset (GstVideoEncoder * encoder, 
gboolean hard)
+ encoder->priv->current_frame_events);
+   }
+ 
++  g_list_foreach (priv->frames, (GFunc) gst_video_codec_frame_unref, NULL);
++  g_list_free (priv->frames);
++  priv->frames = NULL;
++
+   GST_VIDEO_ENCODER_STREAM_UNLOCK (encoder);
+ 
+   return ret;
+-- 
+1.7.9.5
+
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
index 9c31391..8daf5af 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
@@ -14,6 +14,7 @@ SRC_URI += 
"file://do-not-change-eos-event-to-gap-event-if.patch \
 file://0002-video-frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch \
 file://0001-video-frame-Don-t-ref-buffers-twice-when-mapping.patch 
\
 
file://0003-videofilter-Use-new-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch \
+
file://videoencoder-Keep-sticky-events-around-when-doing-a-soft-.patch \
 "
 
 SRC_URI[md5sum] = "357165af625c0ca353ab47c5d843920e"
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [poky][PATCH v4 0/5] gstreamer1.0-plugins-base: Add patches for Gstreamer 1.4.5 base

[Yuqing Zhu]

Fix id3demux issue

Handle audio/video decoder error

Update video alignment after video alignment

Gstvideofilter use new GST_VIDEO_FRAME_MAP_FLAG_NO_REF
Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF, keeping buffer writable
Don't ref buffers twice when mapping

Keep sticky events around when doing a soft reset

Yuqing Zhu (5):
  gstreamer1.0-plugins-base: Bug fix for id3demux issue
  gstreamer1.0-plugins-base: handle audio/video decoder error
  gstreamer1.0-plugins-base: update video alignment after video
alignment
  gstreamer1.0-plugins-base: Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF related
patch
  gstreamer1.0-plugins-base: Add videoencoder related patch

 ...rame-Don-t-ref-buffers-twice-when-mapping.patch | 26 +++
 ...frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch | 87 ++
 ...r-Use-new-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch | 69 +
 .../fix-id3demux-utf16-to-utf8-issue.patch | 54 ++
 .../handle-audio-video-decoder-error.patch | 64 
 .../videobuffer_updata_alignment_update.patch  | 53 +
 ...p-sticky-events-around-when-doing-a-soft-.patch | 46 
 .../gstreamer/gstreamer1.0-plugins-base_1.4.5.bb   |  7 ++
 8 files changed, 406 insertions(+)
 create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-video-frame-Don-t-ref-buffers-twice-when-mapping.patch
 create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-video-frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch
 create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0003-videofilter-Use-new-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch
 create mode 100755 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/fix-id3demux-utf16-to-utf8-issue.patch
 create mode 100755 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/handle-audio-video-decoder-error.patch
 create mode 100755 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/videobuffer_updata_alignment_update.patch
 create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/videoencoder-Keep-sticky-events-around-when-doing-a-soft-.patch

-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [poky][PATCH v4 1/5] gstreamer1.0-plugins-base: Bug fix for id3demux issue

[Yuqing Zhu]

Use g_utf16_to_utf8() instead of g_convert to fix the issue that
id3 tags utf16 charaters cannot be extreacted in id3demux when try
to get the id3v2 tag such as TIT2, TALB etc.

Signed-off-by: Yuqing Zhu 
---
 .../fix-id3demux-utf16-to-utf8-issue.patch | 54 ++
 .../gstreamer/gstreamer1.0-plugins-base_1.4.5.bb   |  1 +
 2 files changed, 55 insertions(+)
 create mode 100755 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/fix-id3demux-utf16-to-utf8-issue.patch

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/fix-id3demux-utf16-to-utf8-issue.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/fix-id3demux-utf16-to-utf8-issue.patch
new file mode 100755
index 000..ef3f75f
--- /dev/null
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/fix-id3demux-utf16-to-utf8-issue.patch
@@ -0,0 +1,54 @@
+Author: Lyon Wang 
+Date:   Thu Oct 9 17:37:43 2014 +0800
+
+[id3v2frames] Bug fix for id3demux issue
+
+Fix the issue that id3 tags utf16 charaters cannot be extreacted in id3demux
+when I tried to get the id3v2 tag such as TIT2, TALB etc. it will return extrac
+failed.
+
+Checked in id3v2frame.c,  When parse the UTF-16 streams, it used g_convert() to
+convert the buffer from UTF-16 to UTF-8, however it will return err that this
+conversion is not supported which cause the extraction failed with these UTF-16
+characters.
+
+In the patch, use g_utf16_to_utf8() instead of g_convert, which can convert the
+character format successfully.
+
+https://bugzilla.gnome.org/show_bug.cgi?id=741144
+
+Upstream-Status: Backport [1.5.1]
+
+Signed-off-by: Lyon Wang 
+
+diff --git a/gst-libs/gst/tag/id3v2frames.c b/gst-libs/gst/tag/id3v2frames.c
+old mode 100644
+new mode 100755
+index 3785c2a..7b9d8ac
+--- a/gst-libs/gst/tag/id3v2frames.c
 b/gst-libs/gst/tag/id3v2frames.c
+@@ -1057,14 +1057,17 @@ parse_insert_string_field (guint8 encoding, gchar * 
data, gint data_size,
+ data_size -= 2;
+   }
+ 
+-  field = g_convert (data, data_size, "UTF-8", in_encode, NULL, NULL, 
NULL);
+-
+-  if (field == NULL || g_utf8_validate (field, -1, NULL) == FALSE) {
+-/* As a fallback, try interpreting UTF-16 in the other endianness */
+-if (in_encode == utf16beenc)
+-  field = g_convert (data, data_size, "UTF-8", utf16leenc,
+-  NULL, NULL, NULL);
++  if (in_encode == utf16beenc) {
++   gunichar2 *data_utf16;
++   guint i;
++   data_utf16 =  (gunichar2 *) data;
++  for (i=0; i<(data_size>>1); i++) {
++data_utf16[i] = GUINT16_TO_LE (data_utf16[i]);
++  }
+   }
++  //field = g_convert (data, data_size, "UTF-8", in_encode, NULL, NULL, 
NULL);
++   field = g_utf16_to_utf8((gunichar2 *)data, (glong)(data_size>>1), 
NULL, NULL, NULL);
++
+ }
+ 
+   break;
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
index 11cf8b4..aaa1ecb 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
@@ -8,6 +8,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=c54ce9345727175ff66d17b67ff51f58 \
 SRC_URI += "file://do-not-change-eos-event-to-gap-event-if.patch \
 file://get-caps-from-src-pad-when-query-caps.patch \
 file://taglist-not-send-to-down-stream-if-all-the-frame-cor.patch \
+file://fix-id3demux-utf16-to-utf8-issue.patch \
 "
 
 SRC_URI[md5sum] = "357165af625c0ca353ab47c5d843920e"
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [poky][PATCH v4 4/5] gstreamer1.0-plugins-base: Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF related patch

[Yuqing Zhu]

-Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF
 This makes sure that the buffer is not reffed another time when
 storing it in the GstVideoFrame, keeping it writable if it was writable.

-Use new GST_VIDEO_FRAME_MAP_FLAG_NO_REF to replace the old one because it's 
kind of ugly.

-Don't ref buffers twice when mapping

Signed-off-by: Yuqing Zhu 
---
 ...rame-Don-t-ref-buffers-twice-when-mapping.patch | 26 +++
 ...frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch | 87 ++
 ...r-Use-new-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch | 69 +
 .../gstreamer/gstreamer1.0-plugins-base_1.4.5.bb   |  3 +
 4 files changed, 185 insertions(+)
 create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-video-frame-Don-t-ref-buffers-twice-when-mapping.patch
 create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-video-frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch
 create mode 100644 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0003-videofilter-Use-new-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-video-frame-Don-t-ref-buffers-twice-when-mapping.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-video-frame-Don-t-ref-buffers-twice-when-mapping.patch
new file mode 100644
index 000..3db4724
--- /dev/null
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0001-video-frame-Don-t-ref-buffers-twice-when-mapping.patch
@@ -0,0 +1,26 @@
+From 269f642c45d85cfd630ed490478e6bd6b71a767f Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= 
+Date: Tue, 16 Sep 2014 01:07:18 +0300
+Subject: [PATCH] video-frame: Don't ref buffers twice when mapping
+
+Upstream-Status: Backport [1.5.1]
+---
+ gst-libs/gst/video/video-frame.c |2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/gst-libs/gst/video/video-frame.c 
b/gst-libs/gst/video/video-frame.c
+index 01f23c0..8a9ae96 100644
+--- a/gst-libs/gst/video/video-frame.c
 b/gst-libs/gst/video/video-frame.c
+@@ -105,7 +105,7 @@ gst_video_frame_map_id (GstVideoFrame * frame, 
GstVideoInfo * info,
+   frame->data[i] = frame->map[0].data + info->offset[i];
+ }
+   }
+-  frame->buffer = gst_buffer_ref (buffer);
++  frame->buffer = buffer;
+   if ((flags & GST_VIDEO_FRAME_MAP_FLAG_NO_REF) == 0)
+ gst_buffer_ref (frame->buffer);
+ 
+-- 
+1.7.9.5
+
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-video-frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-video-frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch
new file mode 100644
index 000..c465b5c
--- /dev/null
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/0002-video-frame-Add-GST_VIDEO_FRAME_MAP_FLAG_NO_REF.patch
@@ -0,0 +1,87 @@
+From 40a293d44d1aeccf5eb8e86f23a0b13666111c5c Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= 
+Date: Fri, 12 Sep 2014 14:39:16 +0300
+Subject: [PATCH 2/3] video-frame: Add GST_VIDEO_FRAME_MAP_FLAG_NO_REF
+
+This makes sure that the buffer is not reffed another time when
+storing it in the GstVideoFrame, keeping it writable if it was
+writable.
+
+Upstream-Status: Backport [1.5.1]
+https://bugzilla.gnome.org/show_bug.cgi?id=736118
+---
+ gst-libs/gst/video/video-frame.c |9 -
+ gst-libs/gst/video/video-frame.h |   18 ++
+ 2 files changed, 26 insertions(+), 1 deletion(-)
+
+diff --git a/gst-libs/gst/video/video-frame.c 
b/gst-libs/gst/video/video-frame.c
+index 537cf70..01f23c0 100644
+--- a/gst-libs/gst/video/video-frame.c
 b/gst-libs/gst/video/video-frame.c
+@@ -106,6 +106,9 @@ gst_video_frame_map_id (GstVideoFrame * frame, 
GstVideoInfo * info,
+ }
+   }
+   frame->buffer = gst_buffer_ref (buffer);
++  if ((flags & GST_VIDEO_FRAME_MAP_FLAG_NO_REF) == 0)
++gst_buffer_ref (frame->buffer);
++
+   frame->meta = meta;
+ 
+   /* buffer flags enhance the frame flags */
+@@ -189,11 +192,13 @@ gst_video_frame_unmap (GstVideoFrame * frame)
+   GstBuffer *buffer;
+   GstVideoMeta *meta;
+   gint i;
++  GstMapFlags flags;
+ 
+   g_return_if_fail (frame != NULL);
+ 
+   buffer = frame->buffer;
+   meta = frame->meta;
++  flags = frame->map[0].flags;
+ 
+   if (meta) {
+ for (i = 0; i < frame->info.finfo->n_planes; i++) {
+@@ -202,7 +207,9 @@ gst_video_frame_unmap (GstVideoFrame * frame)
+   } else {
+ gst_buffer_unmap (buffer, &frame->map[0]);
+   }
+-  gst_buffer_unref (buffer);
++
++  if ((flags & GST_VIDEO_FRAME_MAP_FLAG_NO_REF) == 0)
++gst_buffer_unref (frame->buffer);
+ }
+ 
+ /**
+diff --git a/gst-libs/gst/video/video-frame.h 
b/gst-libs/gst/video/video-frame.h
+index 627fab0..f8e6304 100644
+--- a/gst-libs/gst/video/video-frame.h
 b/gst-libs/gst/video/video-frame.h
+@@ -149,6 +149,24 @@ typedef enum {
+   GST_VIDEO_BUFFER_FLAG_LAST= (GST_BUFFER_FLAG_LAST << 8)
+ } GstVideoBufferFla

[OE-core] [poky][PATCH v4 2/5] gstreamer1.0-plugins-base: handle audio/video decoder error

[Yuqing Zhu]

When there is input data and no output data to the end of the stream, it will
send GST_ELEMENT_ERROR and quit from playing.
The patch comments the GST_ELEMENT_ERROR() and just add GST_ERROR_OBJECT()
information instead.

Signed-off-by: Yuqing Zhu 
---
 .../handle-audio-video-decoder-error.patch | 64 ++
 .../gstreamer/gstreamer1.0-plugins-base_1.4.5.bb   |  1 +
 2 files changed, 65 insertions(+)
 create mode 100755 
meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/handle-audio-video-decoder-error.patch

diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/handle-audio-video-decoder-error.patch
 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/handle-audio-video-decoder-error.patch
new file mode 100755
index 000..833ce72
--- /dev/null
+++ 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/handle-audio-video-decoder-error.patch
@@ -0,0 +1,64 @@
+From bcb2b8b6f49e7c66124a4f5e07dea829d5ebfe59 Mon Sep 17 00:00:00 2001
+From: Lyon Wang 
+Date: Mon, 15 Dec 2014 16:52:07 +0800
+Subject: [PATCH] handle audio/video decoder error
+
+When there is input data and no output data to the end of the stream, it will
+send GST_ELEMENT_ERROR, So the clips playing will quit.
+However, if only one of the tracks is corrupt, there is no need to quit other
+tracks playing.
+
+The patch comments the GST_ELEMENT_ERROR() and just add GST_ERROR_OBJECT()
+information instead.
+
+Upstream-Status: Submitted [https://bugzilla.gnome.org/show_bug.cgi?id=741542]
+
+Signed-off-by: Lyon Wang 
+---
+ gst-libs/gst/audio/gstaudiodecoder.c |5 +++--
+ gst-libs/gst/video/gstvideodecoder.c |5 +++--
+ 2 files changed, 6 insertions(+), 4 deletions(-)
+ mode change 100644 => 100755 gst-libs/gst/audio/gstaudiodecoder.c
+ mode change 100644 => 100755 gst-libs/gst/video/gstvideodecoder.c
+
+diff --git a/gst-libs/gst/audio/gstaudiodecoder.c 
b/gst-libs/gst/audio/gstaudiodecoder.c
+old mode 100644
+new mode 100755
+index c2e7a28..891df0a
+--- a/gst-libs/gst/audio/gstaudiodecoder.c
 b/gst-libs/gst/audio/gstaudiodecoder.c
+@@ -2123,9 +2123,10 @@ gst_audio_decoder_sink_eventfunc (GstAudioDecoder * 
dec, GstEvent * event)
+   GST_AUDIO_DECODER_STREAM_UNLOCK (dec);
+ 
+   if (dec->priv->ctx.had_input_data && !dec->priv->ctx.had_output_data) {
+-GST_ELEMENT_ERROR (dec, STREAM, DECODE,
++/* GST_ELEMENT_ERROR (dec, STREAM, DECODE,
+ ("No valid frames decoded before end of stream"),
+-("no valid frames found"));
++("no valid frames found")); */
++GST_ERROR_OBJECT(dec, "No valid frames decoded before end of stream");
+   }
+ 
+   /* send taglist if no valid frame is decoded util EOS */
+diff --git a/gst-libs/gst/video/gstvideodecoder.c 
b/gst-libs/gst/video/gstvideodecoder.c
+old mode 100644
+new mode 100755
+index ac581e1..4278bcd
+--- a/gst-libs/gst/video/gstvideodecoder.c
 b/gst-libs/gst/video/gstvideodecoder.c
+@@ -1068,9 +1068,10 @@ gst_video_decoder_sink_event_default (GstVideoDecoder * 
decoder,
+ 
+   /* Error out even if EOS was ok when we had input, but no output */
+   if (ret && priv->had_input_data && !priv->had_output_data) {
+-GST_ELEMENT_ERROR (decoder, STREAM, DECODE,
++/* GST_ELEMENT_ERROR (decoder, STREAM, DECODE,
+ ("No valid frames decoded before end of stream"),
+-("no valid frames found"));
++("no valid frames found")); */
++GST_ERROR_OBJECT(decoder, "No valid frames decoded before end of 
stream");
+   }
+ 
+   /* Forward EOS immediately. This is required because no
+-- 
+1.7.9.5
+
diff --git 
a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
index aaa1ecb..aa90179 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.4.5.bb
@@ -9,6 +9,7 @@ SRC_URI += 
"file://do-not-change-eos-event-to-gap-event-if.patch \
 file://get-caps-from-src-pad-when-query-caps.patch \
 file://taglist-not-send-to-down-stream-if-all-the-frame-cor.patch \
 file://fix-id3demux-utf16-to-utf8-issue.patch \
+file://handle-audio-video-decoder-error.patch \
 "
 
 SRC_URI[md5sum] = "357165af625c0ca353ab47c5d843920e"
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] bjam-native: inhibit sysroot strip

[Burton, Ross]

On 29 July 2015 at 08:34,  wrote:

> +# bjam is already stripped, don't strip it again
> +INHIBIT_SYSROOT_STRIP = "1"
>

In case it's even needed to debug bjam-native, how hard would it be to stop
bjam from being stripped in the first place?

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/2] bitbake.conf: add GITHUB_GIT

[Robert Yang]

On 07/29/2015 06:11 PM, Paul Eggleton wrote:

Hi Robert,

On Wednesday 29 July 2015 02:11:53 Robert Yang wrote:

The following changes since commit 27d068d05239c26a3848eb101571acab54635e37:

   harfbuzz: upgrade to 1.0.1 (2015-07-27 23:28:23 +0100)

are available in the git repository at:

   git://git.openembedded.org/openembedded-core-contrib rbt/github

http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/
github

Robert Yang (2):
   bitbake.conf: add GITHUB_GIT
   meta: git://github.com -> ${GITHUB_GIT}


Interesting, but you don't explain why this change is needed in the commit
messages.


Hi Paul,

It is just like what GNOME_GIT, GNU_MIRROR and others did.

// Robert



Cheers,
Paul


--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/2] bitbake.conf: add GITHUB_GIT

[Paul Eggleton]

Hi Robert,

On Wednesday 29 July 2015 02:11:53 Robert Yang wrote:
> The following changes since commit 27d068d05239c26a3848eb101571acab54635e37:
> 
>   harfbuzz: upgrade to 1.0.1 (2015-07-27 23:28:23 +0100)
> 
> are available in the git repository at:
> 
>   git://git.openembedded.org/openembedded-core-contrib rbt/github
>  
> http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/
> github
> 
> Robert Yang (2):
>   bitbake.conf: add GITHUB_GIT
>   meta: git://github.com -> ${GITHUB_GIT}

Interesting, but you don't explain why this change is needed in the commit 
messages.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 4/6] devtool: also load plugins from BBPATH

[Paul Eggleton]

Hmm, I guess we will have to add a command-line parameter to specify BBPATH 
(or the full plugin search path) so that tinfoil doesn't have to be 
instantiated when it's specified.

Cheers,
Paul

On Wednesday 29 July 2015 14:58:37 ChenQi wrote:
> Hi Christopher,
> 
> I suspect this patch is causing failure for 'populate_sdk_ext' task.
> Could you please take a look at it?
> 
> See error message below.
> 
> Best Regards,
> Chen Qi
> 
> ===
> 
> ERROR: Error executing a python function in
> /buildarea2/chenqi/poky/meta/recipes-core/images/core-image-minimal.bb:
> 
> The stack trace of python calls that resulted in this exception/failure was:
> File: 'copy_buildsystem', lineno: 127, function: 
>   0123:with open(baseoutpath + '/conf/work-config.inc', 'w') as f:
>   0124:pass
>   0125:
>   0126:
>   *** 0127:copy_buildsystem(d)
>   0128:
> File: 'copy_buildsystem', lineno: 53, function: copy_buildsystem
>   0049:with open(os.path.join(baseoutpath, 'conf',
> 'devtool.conf'), 'w') as f:
>   0050:config.write(f)
>   0051:
>   0052:# Create a layer for new recipes / appends
>   *** 0053:bb.process.run("devtool --basepath %s create-workspace
> --create-only %s" % (baseoutpath, os.path.join(baseoutpath, 'workspace')))
>   0054:
>   0055:# Create bblayers.conf
>   0056:bb.utils.mkdirhier(baseoutpath + '/conf')
>   0057:with open(baseoutpath + '/conf/bblayers.conf', 'w') as f:
> File: '/buildarea2/chenqi/poky/bitbake/lib/bb/process.py', lineno: 152,
> function: run
>   0148:else:
>   0149:stdout, stderr = pipe.communicate(input)
>   0150:
>   0151:if pipe.returncode != 0:
>   *** 0152:raise ExecutionError(cmd, pipe.returncode, stdout,
> stderr)
>   0153:return stdout, stderr
> Exception: ExecutionError: Execution of 'devtool --basepath
> /buildarea2/chenqi/poky/build-systemd/tmp-2/work/qemuarm-poky-linux-gnueabi/
> core-image-minimal/1.0-r0/sdk-ext/image//opt/poky/1.8+snapshot
> create-workspace --create-only
> /buildarea2/chenqi/poky/build-systemd/tmp-2/work/qemuarm-poky-linux-gnueabi/
> core-image-minimal/1.0-r0/sdk-ext/image//opt/poky/1.8+snapshot/workspace'
> failed with exit code 1:
> ERROR: Only one copy of bitbake should be run against a build directory
> Traceback (most recent call last):
>File "/buildarea2/chenqi/poky/scripts/devtool", line 281, in 
>  ret = main()
>File "/buildarea2/chenqi/poky/scripts/devtool", line 242, in main
>  tinfoil = setup_tinfoil(config_only=True)
>File "/buildarea2/chenqi/poky/scripts/lib/devtool/__init__.py", line
> 108, in setup_tinfoil
>  tinfoil = bb.tinfoil.Tinfoil()
>File
> "/buildarea2/chenqi/poky/build-systemd/tmp-2/work/qemuarm-poky-linux-gnueabi
> /core-image-minimal/1.0-r0/sdk-ext/image//opt/poky/1.8+snapshot/layers/poky/
> bitbake/lib/bb/tinfoil.py", line 54, in __init__
>  self.cooker = BBCooker(self.config, features)
>File
> "/buildarea2/chenqi/poky/build-systemd/tmp-2/work/qemuarm-poky-linux-gnueabi
> /core-image-minimal/1.0-r0/sdk-ext/image//opt/poky/1.8+snapshot/layers/poky/
> bitbake/lib/bb/cooker.py", line 155, in __init__
>  bb.fatal("Only one copy of bitbake should be run against a build
> directory")
> BBHandledException
> 
> 
> ERROR: Function failed: copy_buildsystem
> ERROR: Logfile of failure stored in:
> /buildarea2/chenqi/poky/build-systemd/tmp-2/work/qemuarm-poky-linux-gnueabi/
> core-image-minimal/1.0-r0/temp/log.do_populate_sdk_ext.3616 ERROR: Task 12
> (/buildarea2/chenqi/poky/meta/recipes-core/images/core-image-minimal.bb,
> do_populate_sdk_ext) failed with exit code '1'
> 
> On 07/14/2015 11:57 PM, Christopher Larson wrote:
> > From: Christopher Larson 
> > 
> > This makes it easier to extend, as a layer can add its own sub-commands.
> > 
> > tinfoil is now passed into the commands, as we needed to parse the
> > configuration metadata to get BBPATH, and we don't want to construct
> > tinfoil more than once, otherwise we have to deal with startup and
> > shutdown of cooker.
> > 
> > [YOCTO #7625]
> > 
> > Signed-off-by: Christopher Larson 
> > ---
> > 
> >   scripts/devtool | 57
> >   +
> >   scripts/lib/devtool/__init__.py |  4 +--
> >   scripts/lib/devtool/deploy.py   |  6 ++---
> >   scripts/lib/devtool/standard.py | 15 +--
> >   4 files changed, 46 insertions(+), 36 deletions(-)
> > 
> > diff --git a/scripts/devtool b/scripts/devtool
> > index fa799f6..01bb412 100755
> > --- a/scripts/devtool
> > +++ b/scripts/devtool
> > @@ -35,7 +35,7 @@ context = None
> > 
> >   scripts_path = os.path.dirname(os.path.realpath(__file__))
> >   lib_path = scripts_path + '/lib'
> >   sys.path = sys.path + [lib_path]
> > 
> > -from devtool import DevtoolError
> > +from devtool import DevtoolError, setup_tinfoil
> > 
> >   import scriptutils
> >   logger = scriptutils.logger_create('devtool')
> > 
> >

[OE-core] [PATCH] sshcontrol: Use os.environ.copy() instead of copy.copy()

[Richard Purdie]

os.environ is special and copy.copy() doesn't do what we'd expect,
changes in the child object change the parent. copy.deepcopy() is
also known to have issues with it.

Use the dedicated .copy() method which will not influence the 
parent. This fixes selftest failures where the DISPLAY variable 
disappears.

Signed-off-by: Richard Purdie 

diff --git a/meta/lib/oeqa/utils/sshcontrol.py 
b/meta/lib/oeqa/utils/sshcontrol.py
index 4f8d3d2..6ed48ba 100644
--- a/meta/lib/oeqa/utils/sshcontrol.py
+++ b/meta/lib/oeqa/utils/sshcontrol.py
@@ -10,7 +10,6 @@ import subprocess
 import time
 import os
 import select
-import copy
 
 
 class SSHProcess(object):
@@ -33,7 +32,7 @@ class SSHProcess(object):
 self.logfile = None
 
 # Unset DISPLAY which means we won't trigger SSH_ASKPASS
-env = copy.copy(os.environ)
+env = os.environ.copy()
 if "DISPLAY" in env:
 del env['DISPLAY']
 self.options['env'] = env


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [poky][PATCH v2 4/6] gstreamer1.0-plugins-base: Add video-frame related patch

[Zhu Carol]

Hi Otavio,
   The 1.4.2 fix is ugly, so [OE-core] [poky][PATCH v3 5/6] 
gstreamer1.0-plugins-base: Add videofilter related patch is used to revert it.
   Actually these patches are all back port from 1.5.1, I will change the 
Upstream-Status and merge these two patches together.

B.R.
Carol

-Original Message-
From: Otavio Salvador [mailto:otavio.salva...@ossystems.com.br] 
Sent: Tuesday, July 28, 2015 7:19 PM
To: Zhu Yuqing-B54851
Cc: Patches and discussions about the oe-core layer
Subject: Re: [OE-core] [poky][PATCH v2 4/6] gstreamer1.0-plugins-base: Add 
video-frame related patch

On Tue, Jul 28, 2015 at 2:23 AM, Zhu Carol  wrote:
> Hi Otavio,
>Really sorry for the Upstream-Status format error, we just kept the same 
> with some patches on fido branch, maybe they are old version. And I still 
> have a little confusion about the third comment. My question are as followed.

No problem; thanks by being responsive with the feedback. Comments below...

...
>> +Upstream Status: Accepted
>> +https://bugzilla.gnome.org/show_bug.cgi?id=736118
>
> The field is:
>
> Upstream-Status: Submitted [https://...]
>
> [Yuqing Zhu-b54851]: the Upstream-Status of this patch is "Accepted", I am 
> not very clear about your comments "Submitted", do you mean I should add the 
> web site in the same line with Upstream-Status?
> Just like this:
> Upstream-Status:  Accepted 
> [https://bugzilla.gnome.org/show_bug.cgi?id=736118]

Accepted would be when upstream says it will apply it for next release but you 
don't know when.

Looking at the bug, it seemed it was not been applied yet so it would be 
submitted.

> I've read the guide line, here is my understanding:
> Upstream-Status:  Pending
> Upstream-Status:  Submitted [where]
> Upstream-Status:  Accepted
> Upstream-Status:  Backport [from where]
> Upstream-Status:  Denied
> Upstream-Status:  Inappropriate [reason]
>
> Just obey this, right?

Yes.

> So that is
> Upstream-Status:  Accepted
> That's it ?

To be honest, this specific case seems to have been addressed in 1.4.2 in a 
different way;

http://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?h=1.4&id=f711288c7c86c33aed18f52ed6d7787d19b7741d

So this could be dropped.

-- 
Otavio Salvador O.S. Systems
http://www.ossystems.com.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9981-7854Mobile: +1 (347) 903-9750
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 0/2] bitbake.conf: add GITHUB_GIT

[Robert Yang]

The following changes since commit 27d068d05239c26a3848eb101571acab54635e37:

  harfbuzz: upgrade to 1.0.1 (2015-07-27 23:28:23 +0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib rbt/github
  
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/github

Robert Yang (2):
  bitbake.conf: add GITHUB_GIT
  meta: git://github.com -> ${GITHUB_GIT}

 meta/conf/bitbake.conf |1 +
 .../connman/connman-gnome_0.7.bb   |2 +-
 meta/recipes-core/update-rc.d/update-rc.d_0.7.bb   |2 +-
 meta/recipes-devtools/bootchart2/bootchart2_git.bb |2 +-
 .../build-compare/build-compare_git.bb |2 +-
 meta/recipes-devtools/remake/remake.inc|2 +-
 .../recipes-devtools/sgmlspl/sgmlspl-native_git.bb |2 +-
 .../squashfs-tools/squashfs-tools_git.bb   |2 +-
 .../chkconfig-alternatives-native_1.3.59.bb|2 +-
 meta/recipes-extended/ltp/ltp_20150420.bb  |2 +-
 meta/recipes-extended/xinetd/xinetd_2.3.15.bb  |2 +-
 meta/recipes-graphics/eglinfo/eglinfo.inc  |2 +-
 meta/recipes-graphics/mx/mx-1.0_1.4.7.bb   |2 +-
 .../xinput-calibrator/xinput-calibrator_git.bb |2 +-
 .../recipes-multimedia/gstreamer/gst-player_git.bb |2 +-
 meta/recipes-multimedia/libav/libpostproc_git.bb   |2 +-
 meta/recipes-support/lz4/lz4.bb|2 +-
 17 files changed, 17 insertions(+), 16 deletions(-)

-- 
1.7.9.5

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 2/2] meta: git://github.com -> ${GITHUB_GIT}

[Robert Yang]

Signed-off-by: Robert Yang 
---
 .../connman/connman-gnome_0.7.bb   |2 +-
 meta/recipes-core/update-rc.d/update-rc.d_0.7.bb   |2 +-
 meta/recipes-devtools/bootchart2/bootchart2_git.bb |2 +-
 .../build-compare/build-compare_git.bb |2 +-
 meta/recipes-devtools/remake/remake.inc|2 +-
 .../recipes-devtools/sgmlspl/sgmlspl-native_git.bb |2 +-
 .../squashfs-tools/squashfs-tools_git.bb   |2 +-
 .../chkconfig-alternatives-native_1.3.59.bb|2 +-
 meta/recipes-extended/ltp/ltp_20150420.bb  |2 +-
 meta/recipes-extended/xinetd/xinetd_2.3.15.bb  |2 +-
 meta/recipes-graphics/eglinfo/eglinfo.inc  |2 +-
 meta/recipes-graphics/mx/mx-1.0_1.4.7.bb   |2 +-
 .../xinput-calibrator/xinput-calibrator_git.bb |2 +-
 .../recipes-multimedia/gstreamer/gst-player_git.bb |2 +-
 meta/recipes-multimedia/libav/libpostproc_git.bb   |2 +-
 meta/recipes-support/lz4/lz4.bb|2 +-
 16 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/meta/recipes-connectivity/connman/connman-gnome_0.7.bb 
b/meta/recipes-connectivity/connman/connman-gnome_0.7.bb
index f5575d2..5a0e37a 100644
--- a/meta/recipes-connectivity/connman/connman-gnome_0.7.bb
+++ b/meta/recipes-connectivity/connman/connman-gnome_0.7.bb
@@ -10,7 +10,7 @@ DEPENDS = "gtk+ dbus-glib intltool-native"
 
 # 0.7 tag
 SRCREV = "cf3c325b23dae843c5499a113591cfbc98acb143"
-SRC_URI = "git://github.com/connectivity/connman-gnome.git \
+SRC_URI = "${GITHUB_GIT}/connectivity/connman-gnome.git \
file://0001-Removed-icon-from-connman-gnome-about-applet.patch \
file://null_check_for_ipv4_config.patch \
file://images/* \
diff --git a/meta/recipes-core/update-rc.d/update-rc.d_0.7.bb 
b/meta/recipes-core/update-rc.d/update-rc.d_0.7.bb
index c15cb78..b5a1727 100644
--- a/meta/recipes-core/update-rc.d/update-rc.d_0.7.bb
+++ b/meta/recipes-core/update-rc.d/update-rc.d_0.7.bb
@@ -10,7 +10,7 @@ PR = "r5"
 # Revision corresponding to tag update-rc.d_0.7
 SRCREV = "eca680ddf28d024954895f59a241a622dd575c11"
 
-SRC_URI = "git://github.com/philb/update-rc.d.git \
+SRC_URI = "${GITHUB_GIT}/philb/update-rc.d.git \
file://add-verbose.patch \
file://check-if-symlinks-are-valid.patch \
file://fix-to-handle-priority-numbers-correctly.patch \
diff --git a/meta/recipes-devtools/bootchart2/bootchart2_git.bb 
b/meta/recipes-devtools/bootchart2/bootchart2_git.bb
index ea9f179..9018bbe 100644
--- a/meta/recipes-devtools/bootchart2/bootchart2_git.bb
+++ b/meta/recipes-devtools/bootchart2/bootchart2_git.bb
@@ -91,7 +91,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=44ac4678311254db62edf8fd39cb8124"
 # one commit beyond 1.14.6 for a systemd-related bugfix
 PV = "0.14.6+git${SRCPV}"
 
-SRC_URI = "git://github.com/mmeeks/bootchart.git \
+SRC_URI = "${GITHUB_GIT}/mmeeks/bootchart.git \
file://bootchartd_stop.sh \
file://bootchartd-no-bashism.patch \
   "
diff --git a/meta/recipes-devtools/build-compare/build-compare_git.bb 
b/meta/recipes-devtools/build-compare/build-compare_git.bb
index 418aee0..036f906 100644
--- a/meta/recipes-devtools/build-compare/build-compare_git.bb
+++ b/meta/recipes-devtools/build-compare/build-compare_git.bb
@@ -5,7 +5,7 @@ HOMEPAGE = "https://github.com/openSUSE/build-compare";
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
 
-SRC_URI = "git://github.com/openSUSE/build-compare.git"
+SRC_URI = "${GITHUB_GIT}/openSUSE/build-compare.git"
 SRC_URI += "file://Rename-rpm-check.sh-to-pkg-diff.sh.patch;striplevel=1"
 SRC_URI += "file://Ignore-DWARF-sections.patch;striplevel=1"
 PATCHTOOL = "git"
diff --git a/meta/recipes-devtools/remake/remake.inc 
b/meta/recipes-devtools/remake/remake.inc
index 6ac833e..04605cf 100644
--- a/meta/recipes-devtools/remake/remake.inc
+++ b/meta/recipes-devtools/remake/remake.inc
@@ -6,7 +6,7 @@ execution in a comprehensible way, and a debugger."
 HOMEPAGE = "http://bashdb.sourceforge.net/remake/";
 SECTION = "devel"
 
-SRC_URI = "git://github.com/rocky/remake.git"
+SRC_URI = "${GITHUB_GIT}/rocky/remake.git"
 
 inherit autotools gettext update-alternatives
 
diff --git a/meta/recipes-devtools/sgmlspl/sgmlspl-native_git.bb 
b/meta/recipes-devtools/sgmlspl/sgmlspl-native_git.bb
index 5e3460f..e402c59 100644
--- a/meta/recipes-devtools/sgmlspl/sgmlspl-native_git.bb
+++ b/meta/recipes-devtools/sgmlspl/sgmlspl-native_git.bb
@@ -4,7 +4,7 @@ SECTION = "libs"
 LICENSE = "GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=18810669f13b87348459e611d31ab760"
 
-SRC_URI = "git://github.com/gitpan/SGMLSpm \
+SRC_URI = "${GITHUB_GIT}/gitpan/SGMLSpm \
   "
 
 SRCREV = "71595b9b5e36bfc00046995e058926bd27793fef"
diff --git a/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb 
b/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
index 497b282..944ca13 100644
--

[OE-core] [PATCH 1/2] bitbake.conf: add GITHUB_GIT

[Robert Yang]

Signed-off-by: Robert Yang 
---
 meta/conf/bitbake.conf |1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index 730ab80..6775393 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -577,6 +577,7 @@ XORG_MIRROR = "http://xorg.freedesktop.org/releases";
 SAVANNAH_GNU_MIRROR = "http://download.savannah.gnu.org/releases";
 SAVANNAH_NONGNU_MIRROR = "http://download.savannah.nongnu.org/releases";
 CPAN_MIRROR = "http://search.cpan.org/CPAN";
+GITHUB_GIT = "git://github.com"
 
 # You can use the mirror of your country to get faster downloads by putting
 #  export DEBIAN_MIRROR = "ftp://ftp.de.debian.org/debian/pool";
-- 
1.7.9.5

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] image: Fix a build error by adding kernel dependence to do_rootfs

[Firo Yang]

On 07/29/2015 04:50 PM, Paul Eggleton wrote:

Hi Firo,

On Wednesday 29 July 2015 15:48:28 Firo Yang wrote:

Issue: LIN7-4290

do_rootfs needs a file named kernel-abiversion generated by
virtual/kernel:do_packagedata to run depmod.

If do_rootfs scheduled before virtual/kernel:do_packagedata that will
make do_roofs failed because of leaking the file kernel-abiversion.
So add the dependence of virtual/kernel:do_packagedata to do_rootfs
will make it happier.

Signed-off-by: Firo Yang 
---
  meta/classes/image.bbclass | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
index ed12abb..b6128da 100644
--- a/meta/classes/image.bbclass
+++ b/meta/classes/image.bbclass
@@ -77,7 +77,7 @@ LDCONFIGDEPEND_libc-uclibc = ""
  LDCONFIGDEPEND_libc-musl = ""

  do_rootfs[depends] += "makedevs-native:do_populate_sysroot
virtual/fakeroot-native:do_populate_sysroot ${LDCONFIGDEPEND}"
-do_rootfs[depends] +=
"virtual/update-alternatives-native:do_populate_sysroot
update-rc.d-native:do_populate_sysroot" +do_rootfs[depends] +=
"virtual/update-alternatives-native:do_populate_sysroot
update-rc.d-native:do_populate_sysroot virtual/kernel:do_packagedata"
do_rootfs[recrdeptask] += "do_packagedata"

  def command_variables(d):

This was already fixed some time ago:

   
http://cgit.openembedded.org/openembedded-core/commit/?id=41f0f86ec0a3e0b6f6c9bb4ef71a4215c00bf66c

And modified slightly since then:

   
http://cgit.openembedded.org/openembedded-core/commit/?id=2e3c8fd70694dcf9553b5e4c50a57b617a9130b6

If you could please ensure future patches you send are against the
master branch, that would be much appreciated.

Hi Paul

Thanks for your kind reply.
I will do more check before sending patches next time.

Best,
Firo


Thanks,
Paul



--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] image: Fix a build error by adding kernel dependence to do_rootfs

[Paul Eggleton]

Hi Firo,

On Wednesday 29 July 2015 15:48:28 Firo Yang wrote:
> Issue: LIN7-4290
> 
> do_rootfs needs a file named kernel-abiversion generated by
> virtual/kernel:do_packagedata to run depmod.
> 
> If do_rootfs scheduled before virtual/kernel:do_packagedata that will
> make do_roofs failed because of leaking the file kernel-abiversion.
> So add the dependence of virtual/kernel:do_packagedata to do_rootfs
> will make it happier.
> 
> Signed-off-by: Firo Yang 
> ---
>  meta/classes/image.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
> index ed12abb..b6128da 100644
> --- a/meta/classes/image.bbclass
> +++ b/meta/classes/image.bbclass
> @@ -77,7 +77,7 @@ LDCONFIGDEPEND_libc-uclibc = ""
>  LDCONFIGDEPEND_libc-musl = ""
> 
>  do_rootfs[depends] += "makedevs-native:do_populate_sysroot
> virtual/fakeroot-native:do_populate_sysroot ${LDCONFIGDEPEND}"
> -do_rootfs[depends] +=
> "virtual/update-alternatives-native:do_populate_sysroot
> update-rc.d-native:do_populate_sysroot" +do_rootfs[depends] +=
> "virtual/update-alternatives-native:do_populate_sysroot
> update-rc.d-native:do_populate_sysroot virtual/kernel:do_packagedata"
> do_rootfs[recrdeptask] += "do_packagedata"
> 
>  def command_variables(d):

This was already fixed some time ago:

  
http://cgit.openembedded.org/openembedded-core/commit/?id=41f0f86ec0a3e0b6f6c9bb4ef71a4215c00bf66c

And modified slightly since then:

  
http://cgit.openembedded.org/openembedded-core/commit/?id=2e3c8fd70694dcf9553b5e4c50a57b617a9130b6

If you could please ensure future patches you send are against the
master branch, that would be much appreciated.

Thanks,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] guile: Add earlyclobber constraint to the SMULL outputs.

[jackie.huang]

From: Jackie Huang 

backport a patch to fix issue:
{standard input}: Assembler messages:
{standard input}:16966: rdhi, rdlo and rm must all be different

Signed-off-by: Jackie Huang 
---
 ...guile-VM-ASM_MUL-for-ARM-Add-earlyclobber.patch | 43 ++
 meta/recipes-devtools/guile/guile_2.0.11.bb|  1 +
 2 files changed, 44 insertions(+)
 create mode 100644 
meta/recipes-devtools/guile/files/libguile-VM-ASM_MUL-for-ARM-Add-earlyclobber.patch

diff --git 
a/meta/recipes-devtools/guile/files/libguile-VM-ASM_MUL-for-ARM-Add-earlyclobber.patch
 
b/meta/recipes-devtools/guile/files/libguile-VM-ASM_MUL-for-ARM-Add-earlyclobber.patch
new file mode 100644
index 000..e50059a
--- /dev/null
+++ 
b/meta/recipes-devtools/guile/files/libguile-VM-ASM_MUL-for-ARM-Add-earlyclobber.patch
@@ -0,0 +1,43 @@
+From a85c78ea1393985fdb6e6678dea19135c553d341 Mon Sep 17 00:00:00 2001
+From: Mark H Weaver 
+Date: Fri, 19 Sep 2014 21:18:09 -0400
+Subject: [PATCH] VM: ASM_MUL for ARM: Add earlyclobber constraint to the SMULL
+ outputs.
+
+Reported by Rob Browning .
+
+* libguile/vm-i-scheme.c (ASM_MUL)[ARM]: Add earlyclobber (&) constraint
+  to the SMULL output registers.
+
+Author: Mark H Weaver 
+
+Upstream-Status: Backport
+
+Signed-off-by: Jackie Huang 
+---
+ libguile/vm-i-scheme.c | 5 ++---
+ 1 file changed, 2 insertions(+), 3 deletions(-)
+
+diff --git a/libguile/vm-i-scheme.c b/libguile/vm-i-scheme.c
+index 587aa95..162efab 100644
+--- a/libguile/vm-i-scheme.c
 b/libguile/vm-i-scheme.c
+@@ -1,5 +1,4 @@
+-/* Copyright (C) 2001, 2009, 2010, 2011, 2012, 2013,
+- *   2014 Free Software Foundation, Inc.
++/* Copyright (C) 2001, 2009-2014 Free Software Foundation, Inc.
+  * 
+  * This library is free software; you can redistribute it and/or
+  * modify it under the terms of the GNU Lesser General Public License
+@@ -363,7 +362,7 @@ VM_DEFINE_FUNCTION (149, ge, "ge?", 2)
+   {   
\
+   scm_t_signed_bits rlo, rhi; \
+   asm ("smull %0, %1, %2, %3\n"   \
+-   : "=r" (rlo), "=r" (rhi)   \
++   : "=&r" (rlo), "=&r" (rhi) \
+: "r" (SCM_UNPACK (x) - scm_tc2_int),  \
+  "r" (SCM_I_INUM (y)));   \
+   if (SCM_LIKELY (SCM_SRS (rlo, 31) == rhi))  \
+-- 
+2.3.5
+
diff --git a/meta/recipes-devtools/guile/guile_2.0.11.bb 
b/meta/recipes-devtools/guile/guile_2.0.11.bb
index 621fa0f..ea9017b 100644
--- a/meta/recipes-devtools/guile/guile_2.0.11.bb
+++ b/meta/recipes-devtools/guile/guile_2.0.11.bb
@@ -22,6 +22,7 @@ SRC_URI = "${GNU_MIRROR}/guile/guile-${PV}.tar.xz \
file://arm_aarch64.patch \
file://workaround-ice-ssa-corruption.patch \
file://libguile-Makefile.am-hook.patch \
+   file://libguile-VM-ASM_MUL-for-ARM-Add-earlyclobber.patch \
"
 
 #   file://debian/0001-Change-guile-to-guile-X.Y-for-info-pages.patch
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] image: Fix a build error by adding kernel dependence to do_rootfs

[Firo Yang]

Issue: LIN7-4290

do_rootfs needs a file named kernel-abiversion generated by
virtual/kernel:do_packagedata to run depmod.

If do_rootfs scheduled before virtual/kernel:do_packagedata that will
make do_roofs failed because of leaking the file kernel-abiversion.
So add the dependence of virtual/kernel:do_packagedata to do_rootfs
will make it happier.

Signed-off-by: Firo Yang 
---
 meta/classes/image.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
index ed12abb..b6128da 100644
--- a/meta/classes/image.bbclass
+++ b/meta/classes/image.bbclass
@@ -77,7 +77,7 @@ LDCONFIGDEPEND_libc-uclibc = ""
 LDCONFIGDEPEND_libc-musl = ""
 
 do_rootfs[depends] += "makedevs-native:do_populate_sysroot 
virtual/fakeroot-native:do_populate_sysroot ${LDCONFIGDEPEND}"
-do_rootfs[depends] += "virtual/update-alternatives-native:do_populate_sysroot 
update-rc.d-native:do_populate_sysroot"
+do_rootfs[depends] += "virtual/update-alternatives-native:do_populate_sysroot 
update-rc.d-native:do_populate_sysroot virtual/kernel:do_packagedata"
 do_rootfs[recrdeptask] += "do_packagedata"
 
 def command_variables(d):
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] bjam-native: inhibit sysroot strip

[wenzong.fan]

From: Wenzong Fan 

The bjam is already stripped by default. Fix warning:

  WARNING: File '.../tmp/sysroots/x86_64-linux/usr/bin/bjam' from \
  bjam-native was already stripped, this will prevent future debugging!

Signed-off-by: Wenzong Fan 
---
 meta/recipes-support/boost/bjam-native_1.58.0.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/meta/recipes-support/boost/bjam-native_1.58.0.bb 
b/meta/recipes-support/boost/bjam-native_1.58.0.bb
index d85d1a9..e2e0611 100644
--- a/meta/recipes-support/boost/bjam-native_1.58.0.bb
+++ b/meta/recipes-support/boost/bjam-native_1.58.0.bb
@@ -5,6 +5,9 @@ SECTION = "devel"
 
 inherit native
 
+# bjam is already stripped, don't strip it again
+INHIBIT_SYSROOT_STRIP = "1"
+
 do_compile() {
 ./bootstrap.sh --with-toolset=gcc
 }
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH v2 0/2] Yocto Bug #6945

[He Zhe]

Ping.

Is there any comments on this?

Zhe

On 07/28/2015 11:17 AM, He Zhe wrote:
> Ping.
>
> On 07/27/2015 10:51 AM, He Zhe wrote:
>> Ping.
>>
>> On 07/23/2015 03:48 PM, He Zhe wrote:
>>> Ping.
>>>
>>> On 07/21/2015 03:23 PM, zhe...@windriver.com wrote:
 From: He Zhe 

  - To support building packaging and installing multi types of kernel
images, such as zImage uImage, at one time define KERNEL_IMAGETYPE
as a list.
  - Modify wherever reference KERNEL_IMAGETYPE accordingly.
  - Pass mkimage in sysroot to kernel makefile by NATIVE_MKIMAGE to avoid
depending on build machine's, when KEEPUIMAGE is "yes".
  - v2: update with the latest oe-core

 He Zhe (2):
   kernel: Define KERNEL_IMAGETYPE as a list
   kernel: Pass sysroot mkimage to kernel makefile

  meta/classes/image_types.bbclass |   6 +-
  meta/classes/kernel-fitimage.bbclass |  22 +++---
  meta/classes/kernel-grub.bbclass |  47 
  meta/classes/kernel-uimage.bbclass   |  31 +---
  meta/classes/kernel.bbclass  | 127 
 ++-
  meta/conf/documentation.conf |   2 +-
  meta/lib/oeqa/controllers/masterimage.py |   2 +-
  meta/lib/oeqa/targetcontrol.py   |   2 +-
  meta/recipes-kernel/linux/linux-dtb.inc  |  15 ++--
  scripts/test-remote-image|   2 +-
  10 files changed, 175 insertions(+), 81 deletions(-)


-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core