Re: [OE-core] [PATCH] file: remove the original magic.h

2015-03-27 Thread Burton, Ross
On 27 March 2015 at 03:31, Junling Zheng zhengjunl...@huawei.com wrote: So I think the true reason is that if the magic.h exists already, sometimes Makefile will not generate it again. In that case it's likely entirely down to timestamps. How are you modifying magic.h.in? Ross --

Re: [OE-core] [PATCH V2 0/4] systemd and dbus split into packages

2015-03-27 Thread Burton, Ross
On 18 March 2015 at 21:40, Bottazzini, Bruno bruno.bottazz...@intel.com wrote: Sorry, I didn't get why we can't consider this patchset to be included into master now. Because when Khem said that, master was freezing so that we can make the 1.8 release. We've now branched off the release

[OE-core] [PATCH 2/3] combo-layer: fix file_exclude for dest_dir = .

2015-03-27 Thread Patrick Ohly
filterdiff -x ./some/file does not remove changes for some/file. We must be more careful about constructing the path name and only add the prefix when it really means a directory. While at it, also better normalize the path in copy_selected_files() early on, to handle double slashes. Useful

[OE-core] [PATCH 1/3] combo-layer: clean up dest_dir checking

2015-03-27 Thread Patrick Ohly
Empty dest_dir is basically undocumented behavior. The sample conf only mentions using just a dot for the current directory. In practice, the empty string does not work because of code like this: def action_splitpatch(conf, args): ... if dest_dir != .: filerange_root = '%s

[OE-core] [PATCH 0/3] combo-layer + file_exclude

2015-03-27 Thread Patrick Ohly
Applying the new feature to additional repos highlighted some issue. Proposing for master, but would also be suitable for Fido. The following changes since commit 2d923d6dfe9431dbc005f8ba39838eb4519c471c: python-pygobject: Disable parallel make install in native case (2015-03-25 12:38:44

[OE-core] [PATCH 3/3] combo-layer: fix file_exclude for empty commits

2015-03-27 Thread Patrick Ohly
The code detecting empty patches after removing files with file_exclude failed for commits which were already empty before (like the initial commit in some repos): such patches are completely empty files, without a From line. Detect that case and just let the normal empty patch detection deal

Re: [OE-core] [PATCH 3/4] systemd: split modules into packages

2015-03-27 Thread Anders Darander
* Bottazzini, Bruno bruno.bottazz...@intel.com [150326 14:40]: On Qui, 2015-03-26 at 08:56 -0300, Otavio Salvador wrote: On Thu, Mar 26, 2015 at 5:29 AM, Anders Darander and...@chargestorm.se wrote: * Bruno Bottazzini bruno.bottazz...@intel.com [150325 22:50]: if one wants to launch

Re: [OE-core] [PATCH 3/4] systemd: split modules into packages

2015-03-27 Thread Bottazzini, Bruno
On Sex, 2015-03-27 at 14:11 -0300, Otavio Salvador wrote: On Fri, Mar 27, 2015 at 2:09 PM, Bottazzini, Bruno bruno.bottazz...@intel.com wrote: On Sex, 2015-03-27 at 15:57 +0100, Anders Darander wrote: * Bottazzini, Bruno bruno.bottazz...@intel.com [150326 14:40]: On Qui, 2015-03-26 at

Re: [OE-core] [PATCH v5] u-boot.inc: fix rename image error

2015-03-27 Thread Otavio Salvador
On Wed, Mar 25, 2015 at 6:51 AM, Chunrong Guo b40...@freescale.com wrote: Resolve mismatch between U-Boot configs and uboot image name. The ${S}/${config}/u-boot-${type}.${UBOOT_SUFFIX} may alway be false and repeat compile or install so we need to check if ${type} match ${config}

[OE-core] Ownership issue in package contents

2015-03-27 Thread Mario Domenech Goulart
Hi, I'm observing an issue with directory ownership in package contents. Not sure if I'm overlooking something really basic or if I hit a bug. Here's a minimal recipe to illustrate the problem: $ cat foo.bb LICENSE = CLOSED inherit useradd USERADD_PACKAGES = ${PN} USERADD_PARAM_${PN} = foo

Re: [OE-core] Ownership issue in package contents

2015-03-27 Thread Otavio Salvador
Hello Mario, On Fri, Mar 27, 2015 at 2:31 PM, Mario Domenech Goulart ma...@ossystems.com.br wrote: ... I also generated and checked the rpm package and observed the same issue. ... I tested the foo recipe on master (OE-Core:2d923d6) and I see the same behavior. -- Otavio Salvador

Re: [OE-core] [PATCH 3/4] systemd: split modules into packages

2015-03-27 Thread Otavio Salvador
On Fri, Mar 27, 2015 at 2:09 PM, Bottazzini, Bruno bruno.bottazz...@intel.com wrote: On Sex, 2015-03-27 at 15:57 +0100, Anders Darander wrote: * Bottazzini, Bruno bruno.bottazz...@intel.com [150326 14:40]: On Qui, 2015-03-26 at 08:56 -0300, Otavio Salvador wrote: On Thu, Mar 26, 2015 at

Re: [OE-core] [PATCH V2 0/4] systemd and dbus split into packages

2015-03-27 Thread Bottazzini, Bruno
On Sex, 2015-03-27 at 12:38 +, Burton, Ross wrote: On 18 March 2015 at 21:40, Bottazzini, Bruno bruno.bottazz...@intel.com wrote: Sorry, I didn't get why we can't consider this patchset to be included into master now. Because when Khem said that,

Re: [OE-core] [PATCH 3/4] systemd: split modules into packages

2015-03-27 Thread Bottazzini, Bruno
On Sex, 2015-03-27 at 15:57 +0100, Anders Darander wrote: * Bottazzini, Bruno bruno.bottazz...@intel.com [150326 14:40]: On Qui, 2015-03-26 at 08:56 -0300, Otavio Salvador wrote: On Thu, Mar 26, 2015 at 5:29 AM, Anders Darander and...@chargestorm.se wrote: * Bruno Bottazzini

Re: [OE-core] [PATCH 1/2] linux-yocto: add kern-tools-native to kernel_metadata depends

2015-03-27 Thread Bruce Ashfield
On Fri, Mar 27, 2015 at 11:21 AM, Ross Burton ross.bur...@intel.com wrote: The autobuilder failed like this: temp/run.do_kernel_metadata.25242: line 165: createme: command not found createme is provided by kern-tools-native. do_patch has a dependency on kern-tools-native, but

Re: [OE-core] [PATCH 2/2] kernel-yocto: merge duplicate kernel_configme task definitions

2015-03-27 Thread Bruce Ashfield
On Fri, Mar 27, 2015 at 11:21 AM, Ross Burton ross.bur...@intel.com wrote: The kernel_configme task was added twice (once in the .bbclass, one in a .inc) with different ordering constraints. Change this to be just one definition in the bbclass with the stronger ordering constraints.

[OE-core] [PATCH 1/2] linux-yocto: add kern-tools-native to kernel_metadata depends

2015-03-27 Thread Ross Burton
The autobuilder failed like this: temp/run.do_kernel_metadata.25242: line 165: createme: command not found createme is provided by kern-tools-native. do_patch has a dependency on kern-tools-native, but do_kernel_metadata runs before do_patch. So move the dependency from do_patch to

[OE-core] [PATCH 2/2] kernel-yocto: merge duplicate kernel_configme task definitions

2015-03-27 Thread Ross Burton
The kernel_configme task was added twice (once in the .bbclass, one in a .inc) with different ordering constraints. Change this to be just one definition in the bbclass with the stronger ordering constraints. Signed-off-by: Ross Burton ross.bur...@intel.com --- meta/classes/kernel-yocto.bbclass

Re: [OE-core] [PATCH] oe-git-proxy: Redirect error messages to STDERR

2015-03-27 Thread Burton, Ross
Hi Juro, On 27 March 2015 at 20:37, Darren Hart dvh...@linux.intel.com wrote: oe-git-proxy script needs socat. If socat is not found, an error message is issued on STDOUT. This leads to a misleading git message: fatal: protocol error: bad line length character: ERRO instead of the

Re: [OE-core] [PATCH] oe-git-proxy: Redirect error messages to STDERR

2015-03-27 Thread Darren Hart
On 3/27/15, 10:17 AM, Juro Bystricky juro.bystri...@intel.com wrote: oe-git-proxy script needs socat. If socat is not found, an error message is issued on STDOUT. This leads to a misleading git message: fatal: protocol error: bad line length character: ERRO instead of the intended message:

Re: [OE-core] Not selecting old version of package as installing it would break existing dependencies

2015-03-27 Thread Alejandro del Castillo
openembedded-core-boun...@lists.openembedded.org wrote on 03/24/2015 12:16:05 PM: From: Mike Looijmans mike.looijm...@topic.nl To: openembedded-core@lists.openembedded.org, Date: 03/24/2015 12:16 PM Subject: [OE-core] Not selecting old version of package as installing it would break

[OE-core] [PATCH] wpa-supplicant: Make SystemD D-Bus config conditional

2015-03-27 Thread Otavio Salvador
The SystemD D-Bus configuration should only to be installed when SystemD support is enabled. Signed-off-by: Otavio Salvador ota...@ossystems.com.br --- meta/recipes-connectivity/wpa-supplicant/wpa-supplicant.inc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

Re: [OE-core] [PATCH] file: remove the original magic.h

2015-03-27 Thread Junling Zheng
On 2015/3/27 17:56, Burton, Ross wrote: On 27 March 2015 at 03:31, Junling Zheng zhengjunl...@huawei.com mailto:zhengjunl...@huawei.com wrote: So I think the true reason is that if the magic.h exists already, sometimes Makefile will not generate it again. In that case it's