Re: [OE-core] [PATCH 3/3] uninative-tarball: Add libjis and euc-jp gconv files

2018-05-22 Thread Khem Raj
Hi Ross/Richard

We need to regenerate uninative-tarball now this patch has landed in master.

On Fri, May 11, 2018 at 2:52 AM, Khem Raj  wrote:
> packages like fontforge-native fail with mysterious errors like
>
> | ../../git/inc/gwwiconv.h:44:21: error: conflicting types for 
> ‘gww_iconv_close’
> |  #define iconv_close gww_iconv_close
> |  ^~~
> | ../../git/inc/gwwiconv.h:37:13: note: previous declaration of 
> ‘gww_iconv_close’ was here
> |  extern void gww_iconv_close( gww_iconv_t cd);
> |  ^~~
>
> The reason behind this is that a check for iconv fails during native
> configure run, the check fails because the autoconf test to check for iconv
> pokes for these gconv's in test runs before declaring iconv support 
> successful.
>
> Therefore when uninative is active the package fails to build but when
> uninative is inactive all works fine. this patch fixes that
>
> Signed-off-by: Khem Raj 
> ---
>  meta/recipes-core/meta/uninative-tarball.bb | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/meta/recipes-core/meta/uninative-tarball.bb 
> b/meta/recipes-core/meta/uninative-tarball.bb
> index 38c3a314e1..c38c2a445e 100644
> --- a/meta/recipes-core/meta/uninative-tarball.bb
> +++ b/meta/recipes-core/meta/uninative-tarball.bb
> @@ -12,6 +12,8 @@ TOOLCHAIN_HOST_TASK = "\
>  nativesdk-glibc-gconv-iso8859-1 \
>  nativesdk-glibc-gconv-utf-16 \
>  nativesdk-glibc-gconv-cp1252 \
> +nativesdk-glibc-gconv-euc-jp \
> +nativesdk-glibc-gconv-libjis \
>  nativesdk-patchelf \
>  nativesdk-libxcrypt \
>  "
> --
> 2.17.0
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] recipes-kernel/linux: Enable NUMA Kconfig from MACHINE_FEATURES

2018-05-22 Thread Bruce Ashfield

On 05/22/2018 08:14 PM, Burton, Ross wrote:

Bruce, can you review this please?


Ack'd. Give me a day or so, since I'm traveling, but will reply ASAP.

Bruce



Ross

On 23 May 2018 at 00:45, Alistair Francis  wrote:

On Wed, May 2, 2018 at 1:49 PM, Alistair Francis
 wrote:

If the user has set numa in their MACHINE_FEATURES we should enable
NUMA support in the kernel config.

Signed-off-by: Alistair Francis 


Ping!

Alistair


---
  meta/recipes-kernel/linux/files/numa.cfg  | 4 
  meta/recipes-kernel/linux/linux-yocto.inc | 3 +++
  2 files changed, 7 insertions(+)
  create mode 100644 meta/recipes-kernel/linux/files/numa.cfg

diff --git a/meta/recipes-kernel/linux/files/numa.cfg 
b/meta/recipes-kernel/linux/files/numa.cfg
new file mode 100644
index 00..bb0f9ab20c
--- /dev/null
+++ b/meta/recipes-kernel/linux/files/numa.cfg
@@ -0,0 +1,4 @@
+CONFIG_NUMA=y
+CONFIG_ACPI_NUMA=y
+CONFIG_NUMA_BALANCING=y
+CONFIG_NUMA_BALANCING_DEFAULT_ENABLED=y
diff --git a/meta/recipes-kernel/linux/linux-yocto.inc 
b/meta/recipes-kernel/linux/linux-yocto.inc
index 3bb872a32a..e8fb38b8f4 100644
--- a/meta/recipes-kernel/linux/linux-yocto.inc
+++ b/meta/recipes-kernel/linux/linux-yocto.inc
@@ -28,6 +28,9 @@ KERNEL_LD_append_nios2 = " ${TOOLCHAIN_OPTIONS}"

  KERNEL_FEATURES_append_qemuall=" features/debug/printk.scc"

+SRC_URI += "${@bb.utils.contains('MACHINE_FEATURES', 'numa', ' file://numa.cfg', 
'', d)}"
+KERNEL_FEATURES_append = " ${@bb.utils.contains('MACHINE_FEATURES', 'numa', 
'features/numa/numa.scc', '', d)}"
+
  # A KMACHINE is the mapping of a yocto $MACHINE to what is built
  # by the kernel. This is typically the branch that should be built,
  # and it can be specific to the machine or shared
--
2.17.0


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


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


[OE-core] [PATCH] qemu.inc: Change kernel provider assignment to a weaker one

2018-05-22 Thread zhe.he
From: He Zhe 

Currently we have local.conf included before qemu.inc, and ${DISTRO}.conf
after qemu.inc. They both possibly specify their expected kernel providers.
To let other config files override it in real use, this commit changes
kernel provider assignment to ??= .

Signed-off-by: He Zhe 
---
 meta/conf/machine/include/qemu.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/machine/include/qemu.inc 
b/meta/conf/machine/include/qemu.inc
index aa332fe..5050e41 100644
--- a/meta/conf/machine/include/qemu.inc
+++ b/meta/conf/machine/include/qemu.inc
@@ -19,7 +19,7 @@ IMAGE_FSTYPES += "tar.bz2 ext4"
 RDEPENDS_${KERNEL_PACKAGE_NAME}-base = ""
 
 # Use a common kernel recipe for all QEMU machines
-PREFERRED_PROVIDER_virtual/kernel ?= "linux-yocto"
+PREFERRED_PROVIDER_virtual/kernel ??= "linux-yocto"
 
 EXTRA_IMAGEDEPENDS += "qemu-native qemu-helper-native"
 
-- 
2.7.4

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


[OE-core] [v2][PATCH] bind: update to 9.12.1

2018-05-22 Thread Armin Kuster
[v2]
Remove python3 in default PACKACKECONFIG
include https://patchwork.openembedded.org/patch/148257/
Break the hard runtime dependency on python3-core by adding a
(default-on) packageconfig option for python.

Signed-off-by: Martin Hundebøll 

LIC_FILES_CHKSUM changed do to updated year
Requires python-ply package in core

removed:
dont-test-on-host.patch, no longer impliemented
drop use-python3-and-fix-install-lib-path.patch, they added the ability to pass 
in lib dir loctions
drop bind-confgen-build-unix.o-once.patch, fix included in update

Refresh other patches:
add python3 flag for PACKAGECONFIG to pull in python
add new config option --with-eddsa=

Signed-off-by: Armin Kuster 
---
 ...0001-build-use-pkg-config-to-find-libxml2.patch | 13 +++---
 ...-gen.c-extend-DIRNAMESIZE-from-256-to-512.patch | 13 +++---
 .../0001-lib-dns-gen.c-fix-too-long-error.patch| 13 +++---
 .../bind/bind/bind-confgen-build-unix.o-once.patch | 48 --
 ...-searching-for-json-headers-searches-sysr.patch | 13 +++---
 .../bind/bind/dont-test-on-host.patch  | 17 
 .../use-python3-and-fix-install-lib-path.patch | 36 
 .../bind/{bind_9.10.6.bb => bind_9.12.1.bb}| 31 --
 8 files changed, 39 insertions(+), 145 deletions(-)
 delete mode 100644 
meta/recipes-connectivity/bind/bind/bind-confgen-build-unix.o-once.patch
 delete mode 100644 meta/recipes-connectivity/bind/bind/dont-test-on-host.patch
 delete mode 100644 
meta/recipes-connectivity/bind/bind/use-python3-and-fix-install-lib-path.patch
 rename meta/recipes-connectivity/bind/{bind_9.10.6.bb => bind_9.12.1.bb} (79%)

diff --git 
a/meta/recipes-connectivity/bind/bind/0001-build-use-pkg-config-to-find-libxml2.patch
 
b/meta/recipes-connectivity/bind/bind/0001-build-use-pkg-config-to-find-libxml2.patch
index 1e23c0f..6d73e18 100644
--- 
a/meta/recipes-connectivity/bind/bind/0001-build-use-pkg-config-to-find-libxml2.patch
+++ 
b/meta/recipes-connectivity/bind/bind/0001-build-use-pkg-config-to-find-libxml2.patch
@@ -15,11 +15,11 @@ Signed-off-by: Kai Kang 
  configure.in | 23 +++
  1 file changed, 3 insertions(+), 20 deletions(-)
 
-diff --git a/configure.in b/configure.in
-index 4da73a4..6f2a754 100644
 a/configure.in
-+++ b/configure.in
-@@ -2282,26 +2282,9 @@ case "$use_libxml2" in
+Index: bind-9.12.1/configure.in
+===
+--- bind-9.12.1.orig/configure.in
 bind-9.12.1/configure.in
+@@ -2569,26 +2569,9 @@ case "$use_libxml2" in
DST_LIBXML2_INC=""
;;
auto|yes)
@@ -49,6 +49,3 @@ index 4da73a4..6f2a754 100644
fi
;;
  esac
--- 
-2.1.4
-
diff --git 
a/meta/recipes-connectivity/bind/bind/0001-gen.c-extend-DIRNAMESIZE-from-256-to-512.patch
 
b/meta/recipes-connectivity/bind/bind/0001-gen.c-extend-DIRNAMESIZE-from-256-to-512.patch
index 1215093..9d87303 100644
--- 
a/meta/recipes-connectivity/bind/bind/0001-gen.c-extend-DIRNAMESIZE-from-256-to-512.patch
+++ 
b/meta/recipes-connectivity/bind/bind/0001-gen.c-extend-DIRNAMESIZE-from-256-to-512.patch
@@ -7,11 +7,11 @@ Signed-off-by: Chen Qi 
  lib/dns/gen.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
-diff --git a/lib/dns/gen.c b/lib/dns/gen.c
-index 7a7dafb..51a0435 100644
 a/lib/dns/gen.c
-+++ b/lib/dns/gen.c
-@@ -148,7 +148,7 @@ static const char copyright[] =
+Index: bind-9.12.1/lib/dns/gen.c
+===
+--- bind-9.12.1.orig/lib/dns/gen.c
 bind-9.12.1/lib/dns/gen.c
+@@ -130,7 +130,7 @@ static const char copyright[] =
  #define TYPECLASSBUF (TYPECLASSLEN + 1)
  #define TYPECLASSFMT "%" STR(TYPECLASSLEN) "[-0-9a-z]_%d"
  #define ATTRIBUTESIZE 256
@@ -20,6 +20,3 @@ index 7a7dafb..51a0435 100644
  
  static struct cc {
struct cc *next;
--- 
-1.9.1
-
diff --git 
a/meta/recipes-connectivity/bind/bind/0001-lib-dns-gen.c-fix-too-long-error.patch
 
b/meta/recipes-connectivity/bind/bind/0001-lib-dns-gen.c-fix-too-long-error.patch
index 1ed858c..1e34746 100644
--- 
a/meta/recipes-connectivity/bind/bind/0001-lib-dns-gen.c-fix-too-long-error.patch
+++ 
b/meta/recipes-connectivity/bind/bind/0001-lib-dns-gen.c-fix-too-long-error.patch
@@ -13,11 +13,11 @@ Signed-off-by: Robert Yang 
  lib/dns/gen.c |4 
  1 file changed, 4 insertions(+)
 
-diff --git a/lib/dns/gen.c b/lib/dns/gen.c
-index 51a0435..3d7214f 100644
 a/lib/dns/gen.c
-+++ b/lib/dns/gen.c
-@@ -148,7 +148,11 @@ static const char copyright[] =
+Index: bind-9.12.1/lib/dns/gen.c
+===
+--- bind-9.12.1.orig/lib/dns/gen.c
 bind-9.12.1/lib/dns/gen.c
+@@ -130,7 +130,11 @@ static const char copyright[] =
  #define TYPECLASSBUF (TYPECLASSLEN + 1)
  #define TYPECLASSFMT "%" STR(TYPECLASSLEN) 

[OE-core] [PATCH 1/1] sysklogd: fix typo in RCONFLICTS

2018-05-22 Thread Chen Qi
It should be RCONFLICTS_${PN} instead of RCONFLICTS_${PN}-syslog.
There's no ${PN}-syslog package.

Signed-off-by: Chen Qi 
---
 meta/recipes-extended/sysklogd/sysklogd.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-extended/sysklogd/sysklogd.inc 
b/meta/recipes-extended/sysklogd/sysklogd.inc
index 1a537fa..fc4e67c 100644
--- a/meta/recipes-extended/sysklogd/sysklogd.inc
+++ b/meta/recipes-extended/sysklogd/sysklogd.inc
@@ -33,7 +33,7 @@ SYSTEMD_AUTO_ENABLE = "enable"
 
 INITSCRIPT_NAME = "syslog"
 CONFFILES_${PN} = "${sysconfdir}/syslog.conf"
-RCONFLICTS_${PN}-syslog = "rsyslog busybox-syslog syslog-ng"
+RCONFLICTS_${PN} = "rsyslog busybox-syslog syslog-ng"
 
 CFLAGS += "-DSYSV -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE"
 
-- 
1.9.1

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


[OE-core] [PATCH 0/1] sysklogd: fix typo in RCONFLICTS

2018-05-22 Thread Chen Qi
The following changes since commit 7897fb35a3420c2555ab59dfa438217ada4a9784:

  libnss-mdns: fix postinst and postrm (2018-05-22 13:13:33 +0100)

are available in the git repository at:

  git://git.pokylinux.org/poky-contrib ChenQi/sysklogd-typo
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/sysklogd-typo

Chen Qi (1):
  sysklogd: fix typo in RCONFLICTS

 meta/recipes-extended/sysklogd/sysklogd.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
1.9.1

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


Re: [OE-core] [PATCH] boost: add contract lib

2018-05-22 Thread Yu, Mingli



On 2018年05月16日 12:37, Randy MacLeod wrote:

On 2018-05-11 02:23 AM, mingli...@windriver.com wrote:

From: Mingli Yu 

Add the contract lib which implements contract
programming (a.k.a., Design by Contract or DbC) [1]
for the C++ programming language.


Please clean up the long log.
You should write something like:

Add the contract lib which implements contract programming
(a.k.a., Design by Contract) for the C++ programming language.

I don't know what the [1] was about.
Ah, it's a reference from this documentation:
   https://www.boost.org/doc/libs/develop/libs/contract/doc/html/index.html

For anyone reading, but not following links:

[1] Design by Contract (DbC) is a registered trademark of the Eiffel
Software company and it was first introduced by the Eiffel programming
language (see [Meyer97]).


Good catch!
Thanks Randy! I will update the commit message altogether with some 
other changes next time as it has been already merged.


Thanks,




../Randy



Signed-off-by: Mingli Yu 
---
  meta/recipes-support/boost/boost.inc | 1 +
  1 file changed, 1 insertion(+)

diff --git a/meta/recipes-support/boost/boost.inc
b/meta/recipes-support/boost/boost.inc
index b447897a0c..6984bee777 100644
--- a/meta/recipes-support/boost/boost.inc
+++ b/meta/recipes-support/boost/boost.inc
@@ -9,6 +9,7 @@ BOOST_LIBS = "\
  atomic \
  chrono \
  container \
+contract \
  date_time \
  exception \
  filesystem \





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


Re: [OE-core] [V2][PATCH] busybox: update to 1.28.3

2018-05-22 Thread Andre McCurdy
On Tue, May 22, 2018 at 5:48 PM, Randy MacLeod
 wrote:
> On 05/22/2018 07:06 PM, Andre McCurdy wrote:
> ...
>>
>> Building before and after Armin's patch and then diffing .config files
>> in each build directory gives:
>>
>
> I didn't know what all of these configs were so I've
> listed them below and stated a straw-man opinion as
> to whether to keep them or not.

Thanks for digging in to the details!

>> Old 1.27.2 specific config options removed:
>>
>> -CONFIG_FEATURE_ADDGROUP_LONG_OPTIONS=y
>> -CONFIG_FEATURE_ADDUSER_LONG_OPTIONS=y
>> -CONFIG_FEATURE_ENV_LONG_OPTIONS=y
>> -CONFIG_FEATURE_HWCLOCK_LONG_OPTIONS=y
>> -CONFIG_FEATURE_MKDIR_LONG_OPTIONS=y
>
> I'm okay with following upstream busybox wrt long options so
> we can drop this.

There's no real choice but to drop these (ie they disappear when you
run "make oldconfig").

>> New 1.28.3 specific options added:
>>
>> +CONFIG_BB_ARCH=y
>
>
> Adds 'arch' which is the same output as 'uname -m' but
> I always use arch and didn't know off-hand that uname -m was the same.
>
> Keep it.
>
>> +CONFIG_FEATURE_CATN=y
>
>   This adds an option to cat:
>-n numbers all output lines while -b numbers nonempty output lines.
>   The commit that added this feature says:
>  a2bdc5c55 cat: allow compiling out -n and -b
>   so since previously the feature was always available,
>
> I'm okay with keeping it.
>
>> +CONFIG_FEATURE_SH_READ_FRAC=y
>
>
>   Enables a fractional second read timeout for the shell builtin.
>   bool "read -t N.NNN support (+110 bytes)"
>   Enable support for fractional second timeout in read builtin.
>
> Potentially useful, keep it.
>
>> +CONFIG_FEATURE_SWAPONOFF_LABEL=y
>
>
>   This allows for specifying a device by label or uuid, rather than by
>   name. This feature utilizes the same functionality as blkid/findfs.
>
> Useful. Keep it.
>
>> +CONFIG_FEATURE_VOLUMEID_MINIX=y
>
>
> I don't know of a case where this would be generally useful.
>
> Omit.
>
>> +CONFIG_FEATURE_XARGS_SUPPORT_ARGS_FILE=y
>
> xargs: support -a FILE
>
> The GNU-specific option -a lets xargs read the arguments from a file
> rather than from stdin.
>
> This is particularly convenient when debugging in gdb interactively,
> and it might be of more general use.
>
> Huh... what?
> Digging through the findutils repo, I see that there's a better explaination
> in the findutils/NEWS file from 2004(!):
>
> *** xargs can now read a list of arguments from a named file, allowing
> the invoked program to use the same stdin as xargs started with
> (for example ``xargs --arg-file=todo emacs'').
>
> Keep it (I suppose).
>
>> +CONFIG_FEATURE_XARGS_SUPPORT_PARALLEL=y
>
>
> From: https://bugs.busybox.net/show_bug.cgi?id=9511
> Example:
> find samplegz | xargs -n1 -P4 sh parse.sh
>
> Costs 241 bytes.
>
> Keep it.
>
> The next group are described below but
> I've put my conclusions after each config.
>
>> +CONFIG_HEXEDIT=y
>
> Drop it.
>
>> +CONFIG_NUKE=y
>
> Drop it.
>
>> +CONFIG_RESUME=y
>
> Drop it -- maybe?
>
>> +CONFIG_RUN_INIT=y
>
> Drop it.
>
>> +CONFIG_SETFATTR=y
>
> Drop it.
>
>>
>> So new applets hexedit, nuke, etc, are going to sneak in to our builds
>> unless we refresh our defconfig to disable them.
>>
>
>
> Here is the rough estimate of the new applet sizes:
>
> $ git show 38da4c4420ea6d7b3cf120c0902458e7d8256560 | \
>grep "^+//config:.*bool"
> +//config:  bool "nuke (2.4 kb)"
> +//config:  bool "resume (3.3 kb)"
> +//config:  bool "run-init (7.5 kb)"
> +//config:  bool "hexedit (20 kb)"
> +//config:  bool "setfattr (3.6 kb)"
>
>
> and the commit that added most of the applets that includes
> a brief explanation of the applet's function:
>
> commit ab77e81a8527fa11a4f9392d97c2da037d6f4f98
> Author: Denys Vlasenko 
> Date:   Fri Aug 18 19:15:29 2017 +0200
>
> klibc-utils: new applets: resume, nuke, minips
>
> minips is a pure alias to ps,
> just in case someone needs 100% klibc-utils compat.
>
> nuke is a primitive version of "rm -rf" without
> options and error checks. ~30 bytes.
>
> resume is a tool for initramfs which resumes from
> a given block device.
>
> function old new   delta
> resume_main- 582+582
> packed_usage   31640   31712 +72
> nuke_main  -  28 +28
> xstrtoull  -  24 +24
> applet_names26462665 +19
> applet_main 15321544 +12
> applet_suid   96  97  +1
> applet_install_loc   192 193  +1
> applet_flags  96  97  +1
>
> 

Re: [OE-core] [V2][PATCH] busybox: update to 1.28.3

2018-05-22 Thread Randy MacLeod

On 05/22/2018 07:06 PM, Andre McCurdy wrote:
...

Building before and after Armin's patch and then diffing .config files
in each build directory gives:



I didn't know what all of these configs were so I've
listed them below and stated a straw-man opinion as
to whether to keep them or not.


Old 1.27.2 specific config options removed:

-CONFIG_FEATURE_ADDGROUP_LONG_OPTIONS=y
-CONFIG_FEATURE_ADDUSER_LONG_OPTIONS=y
-CONFIG_FEATURE_ENV_LONG_OPTIONS=y
-CONFIG_FEATURE_HWCLOCK_LONG_OPTIONS=y
-CONFIG_FEATURE_MKDIR_LONG_OPTIONS=y


I'm okay with following upstream busybox wrt long options so
we can drop this.



New 1.28.3 specific options added:

+CONFIG_BB_ARCH=y


Adds 'arch' which is the same output as 'uname -m' but
I always use arch and didn't know off-hand that uname -m was the same.

Keep it.


+CONFIG_FEATURE_CATN=y

  This adds an option to cat:
   -n numbers all output lines while -b numbers nonempty output lines.
  The commit that added this feature says:
 a2bdc5c55 cat: allow compiling out -n and -b
  so since previously the feature was always available,

I'm okay with keeping it.


+CONFIG_FEATURE_SH_READ_FRAC=y


  Enables a fractional second read timeout for the shell builtin.
  bool "read -t N.NNN support (+110 bytes)"
  Enable support for fractional second timeout in read builtin.

Potentially useful, keep it.


+CONFIG_FEATURE_SWAPONOFF_LABEL=y


  This allows for specifying a device by label or uuid, rather than by
  name. This feature utilizes the same functionality as blkid/findfs.

Useful. Keep it.


+CONFIG_FEATURE_VOLUMEID_MINIX=y


I don't know of a case where this would be generally useful.

Omit.


+CONFIG_FEATURE_XARGS_SUPPORT_ARGS_FILE=y

xargs: support -a FILE

The GNU-specific option -a lets xargs read the arguments from a file
rather than from stdin.

This is particularly convenient when debugging in gdb interactively,
and it might be of more general use.

Huh... what?
Digging through the findutils repo, I see that there's a better 
explaination in the findutils/NEWS file from 2004(!):


*** xargs can now read a list of arguments from a named file, allowing
the invoked program to use the same stdin as xargs started with
(for example ``xargs --arg-file=todo emacs'').

Keep it (I suppose).


+CONFIG_FEATURE_XARGS_SUPPORT_PARALLEL=y


From: https://bugs.busybox.net/show_bug.cgi?id=9511
Example:
find samplegz | xargs -n1 -P4 sh parse.sh

Costs 241 bytes.

Keep it.

The next group are described below but
I've put my conclusions after each config.


+CONFIG_HEXEDIT=y

Drop it.


+CONFIG_NUKE=y

Drop it.


+CONFIG_RESUME=y

Drop it -- maybe?


+CONFIG_RUN_INIT=y

Drop it.


+CONFIG_SETFATTR=y

Drop it.



So new applets hexedit, nuke, etc, are going to sneak in to our builds
unless we refresh our defconfig to disable them.




Here is the rough estimate of the new applet sizes:

$ git show 38da4c4420ea6d7b3cf120c0902458e7d8256560 | \
   grep "^+//config:.*bool"
+//config:  bool "nuke (2.4 kb)"
+//config:  bool "resume (3.3 kb)"
+//config:  bool "run-init (7.5 kb)"
+//config:  bool "hexedit (20 kb)"
+//config:  bool "setfattr (3.6 kb)"


and the commit that added most of the applets that includes
a brief explanation of the applet's function:

commit ab77e81a8527fa11a4f9392d97c2da037d6f4f98
Author: Denys Vlasenko 
Date:   Fri Aug 18 19:15:29 2017 +0200

klibc-utils: new applets: resume, nuke, minips

minips is a pure alias to ps,
just in case someone needs 100% klibc-utils compat.

nuke is a primitive version of "rm -rf" without
options and error checks. ~30 bytes.

resume is a tool for initramfs which resumes from
a given block device.

function old new   delta
resume_main- 582+582
packed_usage   31640   31712 +72
nuke_main  -  28 +28
xstrtoull  -  24 +24
applet_names26462665 +19
applet_main 15321544 +12
applet_suid   96  97  +1
applet_install_loc   192 193  +1
applet_flags  96  97  +1


(add/remove: 5/0 grow/shrink: 6/0 up/down: 740/0)   Total: 740 bytes

Signed-off-by: Denys Vlasenko 



While these are useful utilities,
I don't see a need for any of the new applets in the oe-core defconfig.

Armin,
If no one objects to my suggestions, please change the defconfig
in the next update or ping me and I'll do it.

--
# Randy MacLeod
# Wind River Linux
--
___

Re: [OE-core] [PATCH] recipes-kernel/linux: Enable NUMA Kconfig from MACHINE_FEATURES

2018-05-22 Thread Burton, Ross
Bruce, can you review this please?

Ross

On 23 May 2018 at 00:45, Alistair Francis  wrote:
> On Wed, May 2, 2018 at 1:49 PM, Alistair Francis
>  wrote:
>> If the user has set numa in their MACHINE_FEATURES we should enable
>> NUMA support in the kernel config.
>>
>> Signed-off-by: Alistair Francis 
>
> Ping!
>
> Alistair
>
>> ---
>>  meta/recipes-kernel/linux/files/numa.cfg  | 4 
>>  meta/recipes-kernel/linux/linux-yocto.inc | 3 +++
>>  2 files changed, 7 insertions(+)
>>  create mode 100644 meta/recipes-kernel/linux/files/numa.cfg
>>
>> diff --git a/meta/recipes-kernel/linux/files/numa.cfg 
>> b/meta/recipes-kernel/linux/files/numa.cfg
>> new file mode 100644
>> index 00..bb0f9ab20c
>> --- /dev/null
>> +++ b/meta/recipes-kernel/linux/files/numa.cfg
>> @@ -0,0 +1,4 @@
>> +CONFIG_NUMA=y
>> +CONFIG_ACPI_NUMA=y
>> +CONFIG_NUMA_BALANCING=y
>> +CONFIG_NUMA_BALANCING_DEFAULT_ENABLED=y
>> diff --git a/meta/recipes-kernel/linux/linux-yocto.inc 
>> b/meta/recipes-kernel/linux/linux-yocto.inc
>> index 3bb872a32a..e8fb38b8f4 100644
>> --- a/meta/recipes-kernel/linux/linux-yocto.inc
>> +++ b/meta/recipes-kernel/linux/linux-yocto.inc
>> @@ -28,6 +28,9 @@ KERNEL_LD_append_nios2 = " ${TOOLCHAIN_OPTIONS}"
>>
>>  KERNEL_FEATURES_append_qemuall=" features/debug/printk.scc"
>>
>> +SRC_URI += "${@bb.utils.contains('MACHINE_FEATURES', 'numa', ' 
>> file://numa.cfg', '', d)}"
>> +KERNEL_FEATURES_append = " ${@bb.utils.contains('MACHINE_FEATURES', 'numa', 
>> 'features/numa/numa.scc', '', d)}"
>> +
>>  # A KMACHINE is the mapping of a yocto $MACHINE to what is built
>>  # by the kernel. This is typically the branch that should be built,
>>  # and it can be specific to the machine or shared
>> --
>> 2.17.0
>>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] recipes-kernel/linux: Enable NUMA Kconfig from MACHINE_FEATURES

2018-05-22 Thread Alistair Francis
On Wed, May 2, 2018 at 1:49 PM, Alistair Francis
 wrote:
> If the user has set numa in their MACHINE_FEATURES we should enable
> NUMA support in the kernel config.
>
> Signed-off-by: Alistair Francis 

Ping!

Alistair

> ---
>  meta/recipes-kernel/linux/files/numa.cfg  | 4 
>  meta/recipes-kernel/linux/linux-yocto.inc | 3 +++
>  2 files changed, 7 insertions(+)
>  create mode 100644 meta/recipes-kernel/linux/files/numa.cfg
>
> diff --git a/meta/recipes-kernel/linux/files/numa.cfg 
> b/meta/recipes-kernel/linux/files/numa.cfg
> new file mode 100644
> index 00..bb0f9ab20c
> --- /dev/null
> +++ b/meta/recipes-kernel/linux/files/numa.cfg
> @@ -0,0 +1,4 @@
> +CONFIG_NUMA=y
> +CONFIG_ACPI_NUMA=y
> +CONFIG_NUMA_BALANCING=y
> +CONFIG_NUMA_BALANCING_DEFAULT_ENABLED=y
> diff --git a/meta/recipes-kernel/linux/linux-yocto.inc 
> b/meta/recipes-kernel/linux/linux-yocto.inc
> index 3bb872a32a..e8fb38b8f4 100644
> --- a/meta/recipes-kernel/linux/linux-yocto.inc
> +++ b/meta/recipes-kernel/linux/linux-yocto.inc
> @@ -28,6 +28,9 @@ KERNEL_LD_append_nios2 = " ${TOOLCHAIN_OPTIONS}"
>
>  KERNEL_FEATURES_append_qemuall=" features/debug/printk.scc"
>
> +SRC_URI += "${@bb.utils.contains('MACHINE_FEATURES', 'numa', ' 
> file://numa.cfg', '', d)}"
> +KERNEL_FEATURES_append = " ${@bb.utils.contains('MACHINE_FEATURES', 'numa', 
> 'features/numa/numa.scc', '', d)}"
> +
>  # A KMACHINE is the mapping of a yocto $MACHINE to what is built
>  # by the kernel. This is typically the branch that should be built,
>  # and it can be specific to the machine or shared
> --
> 2.17.0
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [V2][PATCH] busybox: update to 1.28.3

2018-05-22 Thread Andre McCurdy
On Mon, May 21, 2018 at 12:21 PM, Andre McCurdy  wrote:
> On Mon, May 21, 2018 at 12:13 PM, akuster808  wrote:
>> On 05/21/2018 11:18 AM, Andre McCurdy wrote:
>>> On Sun, May 20, 2018 at 7:49 AM, Armin Kuster  wrote:
 From: Armin Kuster 

 [v2]
 Add back busybox-udhcpc-no_deconfig.patch ti SRC_URI, missed earlier

 [v1]
 removed patches included in update:
 busybox/CVE-2011-5325.patch
 busybox/CVE-2017-15873.patch
 busybox/busybox-CVE-2017-16544.patch

 refactored busybox-udhcpc-no_deconfig.patch for this update
>>> Did you check the defconfig?
>>  That patch does not touch the defconfigs? It changes the dhcpd.c it self.
>
> Right, removing that patch has nothing to do with the busybox
> defconfig. I wasn't commenting that particular line in the patch.
>
>>> Often it needs a refresh, otherwise any new config options added
>>> between busybox 1.27.2 and 1.28.3 will take busybox's defaults (which
>>> may enable new applets or features which we haven't historically
>>> enabled when configuring busybox for OE).
>> am I missing some context here?
>
> Refreshing (or at least checking) the busybox defconfig is something
> that should ideally be done whenever the busybox version is updated
> and it wasn't clear from your patch that it's been done, so I just
> wanted to double check.

Building before and after Armin's patch and then diffing .config files
in each build directory gives:

Old 1.27.2 specific config options removed:

-CONFIG_FEATURE_ADDGROUP_LONG_OPTIONS=y
-CONFIG_FEATURE_ADDUSER_LONG_OPTIONS=y
-CONFIG_FEATURE_ENV_LONG_OPTIONS=y
-CONFIG_FEATURE_HWCLOCK_LONG_OPTIONS=y
-CONFIG_FEATURE_MKDIR_LONG_OPTIONS=y

New 1.28.3 specific options added:

+CONFIG_BB_ARCH=y
+CONFIG_FEATURE_CATN=y
+CONFIG_FEATURE_SH_READ_FRAC=y
+CONFIG_FEATURE_SWAPONOFF_LABEL=y
+CONFIG_FEATURE_VOLUMEID_MINIX=y
+CONFIG_FEATURE_XARGS_SUPPORT_ARGS_FILE=y
+CONFIG_FEATURE_XARGS_SUPPORT_PARALLEL=y
+CONFIG_HEXEDIT=y
+CONFIG_NUKE=y
+CONFIG_RESUME=y
+CONFIG_RUN_INIT=y
+CONFIG_SETFATTR=y

So new applets hexedit, nuke, etc, are going to sneak in to our builds
unless we refresh our defconfig to disable them.
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] meta-openembedded maintainer ship change.

2018-05-22 Thread Khem Raj
Thanks Armin, I am hoping to keep working with you on various stable
releases in active maintenance.



On Tue, May 22, 2018 at 11:40 AM, akuster808  wrote:
> Hello all,
>
> After much thought, I have decided to step away from being the
> meta-openembedded maintainer, patch wrangler and builder.  Khem Raj has
> kindly stepped up to continue these activities. Maintaining this was
> consuming too much of my free time and I need to focus on other things
> in the community.
>
> Thanks for the opportunity to have served you.
>
> Kind regards,
>
> Armin
>
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [meta-oe][PATCH] Turbostat: add a necessary recipe to meta-oe

2018-05-22 Thread Peter Kjellerstedt
> -Original Message-
> From: openembedded-core-boun...@lists.openembedded.org
> [mailto:openembedded-core-boun...@lists.openembedded.org] On Behalf Of
> Hongzhi.Song
> Sent: den 22 maj 2018 05:27
> To: openembedded-core@lists.openembedded.org
> Subject: [OE-core] [meta-oe][PATCH] Turbostat: add a necessary recipe
> to meta-oe

This should be sent to openembedded-de...@lists.openembedded.org instead 
as it relates to meta-oe and not meta.

> This is an efficient tool to reflect the status of processors.
> Turbostat reports processor topology, frequency, idle power-state
> statistics, temperature and power on processors.
> 
> Signed-off-by: Hongzhi.Song 
> ---
>  meta-oe/recipes-kernel/turbostat/turbostat.bb | 53
> +++
>  1 file changed, 53 insertions(+)
>  create mode 100644 meta-oe/recipes-kernel/turbostat/turbostat.bb
> 
> diff --git a/meta-oe/recipes-kernel/turbostat/turbostat.bb b/meta-
> oe/recipes-kernel/turbostat/turbostat.bb
> new file mode 100644
> index 0..f4614b406
> --- /dev/null
> +++ b/meta-oe/recipes-kernel/turbostat/turbostat.bb
> @@ -0,0 +1,53 @@
> +SUMMARY = "Report processor frequency and idle statistics"
> +DESCRIPTION = "turbostat  reports processor topology, frequency, idle \
> +power-state statistics, temperature and power on modern X86 processors. \
> +Either command is forked and statistics are printed upon its completion, \
> +or statistics are printed periodically."
> +
> +LICENSE = "GPLv2"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7"
> +
> +inherit kernelsrc
> +PROVIDES = "virtual/turbostat"

Why is virtual/turbostat needed? Are there likely to be alternatives?

> +do_populate_lic[depends] += "virtual/kernel:do_patch"
> +do_configure[depends] += "virtual/kernel:do_shared_workdir"
> +
> +B = "${WORKDIR}/${BPN}-${PV}"
> +
> +EXTRA_OEMAKE = '\

It is better to use " here and ' around ${CC} and ${AR} below.

> +-C ${S}/tools/power/x86/turbostat \

What about other architectures? If they are not supported, you should add: 

COMPATIBLE_MACHINE = "x86"

> +O=${B} \
> +CROSS_COMPILE=${TARGET_PREFIX} \
> +ARCH=${ARCH} \
> +CC="${CC}" \
> +AR="${AR}" \
> +'

Combine the two EXTRA_OEMAKE into one.

> +
> +EXTRA_OEMAKE += "\
> +'prefix=${prefix}' \
> +'bindir=${bindir}' \
> +'sharedir=${datadir}' \
> +'sysconfdir=${sysconfdir}' \
> +'sharedir=${@os.path.relpath(datadir, prefix)}' \
> +'mandir=${@os.path.relpath(mandir, prefix)}' \
> +'infodir=${@os.path.relpath(infodir, prefix)}' \
> +"
> +
> +do_compile() {
> + # Linux kernel build system is expected to do the right thing
> + unset CFLAGS
> + oe_runmake turbostat
> +}
> +
> +do_install() {
> + # Linux kernel build system is expected to do the right thing
> + unset CFLAGS
> + oe_runmake DESTDIR=${D} install
> +}
> +
> +python do_package_prepend() {
> +d.setVar('PKGV', d.getVar("KERNEL_VERSION", True).split("-")[0])
> +}
> +
> +TARGET_CC_ARCH += "${LDFLAGS}"
> --
> 2.11.0

//Peter

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


Re: [OE-core] meta-openembedded maintainer ship change.

2018-05-22 Thread Richard Purdie
On Tue, 2018-05-22 at 08:40 -0700, akuster808 wrote:
> After much thought, I have decided to step away from being the
> meta-openembedded maintainer, patch wrangler and builder.  Khem Raj
> has kindly stepped up to continue these activities. Maintaining this
> was consuming too much of my free time and I need to focus on other
> things in the community.
> 
> Thanks for the opportunity to have served you.

I appreciate the work involved in this, thanks Armin and thanks Khem!

There are probably some discussions we should have about various
aspects of this but now isn't the time, lets see how Khem gets on
first.

Cheers,

Richard


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


Re: [OE-core] meta-openembedded maintainer ship change.

2018-05-22 Thread Tom King
Thanks Armin and Khem!

On Tue, May 22, 2018 at 9:14 AM, Martin Jansa 
wrote:

> Thanks Armin and Khem!
>
> On Tue, May 22, 2018 at 5:41 PM akuster808  wrote:
>
>> Hello all,
>>
>> After much thought, I have decided to step away from being the
>> meta-openembedded maintainer, patch wrangler and builder.  Khem Raj has
>> kindly stepped up to continue these activities. Maintaining this was
>> consuming too much of my free time and I need to focus on other things
>> in the community.
>>
>> Thanks for the opportunity to have served you.
>>
>> Kind regards,
>>
>> Armin
>>
>>
>> --
>> ___
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>>
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] meta-openembedded maintainer ship change.

2018-05-22 Thread Martin Jansa
Thanks Armin and Khem!

On Tue, May 22, 2018 at 5:41 PM akuster808  wrote:

> Hello all,
>
> After much thought, I have decided to step away from being the
> meta-openembedded maintainer, patch wrangler and builder.  Khem Raj has
> kindly stepped up to continue these activities. Maintaining this was
> consuming too much of my free time and I need to focus on other things
> in the community.
>
> Thanks for the opportunity to have served you.
>
> Kind regards,
>
> Armin
>
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] meta-openembedded maintainer ship change.

2018-05-22 Thread Burton, Ross
Thanks Armin for all your hard work!

Ross

On 22 May 2018 at 16:40, akuster808  wrote:
> Hello all,
>
> After much thought, I have decided to step away from being the
> meta-openembedded maintainer, patch wrangler and builder.  Khem Raj has
> kindly stepped up to continue these activities. Maintaining this was
> consuming too much of my free time and I need to focus on other things
> in the community.
>
> Thanks for the opportunity to have served you.
>
> Kind regards,
>
> Armin
>
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] meta-openembedded maintainer ship change.

2018-05-22 Thread Alexander Kanavin

On 05/22/2018 06:40 PM, akuster808 wrote:

After much thought, I have decided to step away from being the
meta-openembedded maintainer, patch wrangler and builder.  Khem Raj has
kindly stepped up to continue these activities. Maintaining this was
consuming too much of my free time and I need to focus on other things
in the community.


Thanks Armin, I can only express my support and appreciation.

Being a maintainer comes with an unhealthy dynamic, and generally cannot 
be sustained indefinitely. It takes massive. yet invisible effort to 
make things 'just work'. Then that effort is taken for granted by 
everyone. However, when you mess up, or even simply overlook something, 
complaints come in thick and fast.



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


Re: [OE-core] meta-openembedded maintainer ship change.

2018-05-22 Thread Andrea Adami
On Tue, May 22, 2018 at 5:40 PM, akuster808  wrote:
> Hello all,
>
> After much thought, I have decided to step away from being the
> meta-openembedded maintainer, patch wrangler and builder.  Khem Raj has
> kindly stepped up to continue these activities. Maintaining this was
> consuming too much of my free time and I need to focus on other things
> in the community.
>
> Thanks for the opportunity to have served you.
>
> Kind regards,
>
> Armin
>

Many thanks for the time you've spent!
Andrea

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


[OE-core] meta-openembedded maintainer ship change.

2018-05-22 Thread akuster808
Hello all,

After much thought, I have decided to step away from being the
meta-openembedded maintainer, patch wrangler and builder.  Khem Raj has
kindly stepped up to continue these activities. Maintaining this was
consuming too much of my free time and I need to focus on other things
in the community.

Thanks for the opportunity to have served you.

Kind regards,

Armin


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


Re: [OE-core] [PATCH] scripts/oe-init-bashrc: add more user-friendly oe-setup utility

2018-05-22 Thread Christopher Larson
I agree, there are a ton of build dir and project management tools, without
information about the use cases satisfied and why it's superior to
alternatives, much less why you think it belongs in oe-core as opposed to
all the other tools folks use, which would imply tactit approval as the
official method, particularly going into scripts instead of contrib, I
doubt this is going anywhere.

On Mon, May 21, 2018 at 12:04 PM Alexander Kanavin <
alexander.kana...@linux.intel.com> wrote:

> On 05/21/2018 06:55 PM, Jens Rehsack wrote:
> > Move scripts/bashrc from meta-jens/scripts to oe-core to share
> user-friendly
> > oe builddir management with community. Using this script will help manage
> > multiple BSPs and targets like perl developers using perlbrew.
>
> Please explain in detail what this does, and why is it superior to
> existing tools. What are the typical usage scenarios? Where and how this
> should be documented and tested? It is not a good idea to just add
> something into scripts/, as no one will know or use it, and so it will
> just quietly bitrot.
>
> > +__oe_check_py () {
> > +# Make sure we're not using python v3.x. This check can't go into
> > +# sanity.bbclass because bitbake's source code doesn't even pass
> > +# parsing stage when used with python v3, so we catch it here so we
> > +# can offer a meaningful error message.
> > +py_v3_check=`/usr/bin/env python --version 2>&1 | grep "Python 3"`
> > +if [ "$py_v3_check" != "" ]; then
> > +echo >&2 "Bitbake is not compatible with python v3"
> > +echo >&2 "Please set up python v2 as your default python
> interpreter"
> > +return 1
> > +fi
>
> Bitbake has in fact been compatible with Python 3.x for several
> releases. The above check is not particularly useful, as /usr/bin/python
> nearly always points to a 2.x version.
>
> Alex
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>


-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Senior Software Engineer, Mentor Graphics
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 3/3] alsa-utils: 1.1.5 -> 1.1.6

2018-05-22 Thread Tanu Kaskinen
Dropped 0001-alsactl-don-t-let-systemd-unit-restore-the-volume-wh.patch,
because an equivalent patch is included in the new release.

License-Update: FSF address updated

Signed-off-by: Tanu Kaskinen 
---
 ...s_1.1.5.bb => alsa-utils-scripts_1.1.6.bb} |  0
 ...t-systemd-unit-restore-the-volume-wh.patch | 56 ---
 ...lsa-utils_1.1.5.bb => alsa-utils_1.1.6.bb} |  7 +--
 3 files changed, 3 insertions(+), 60 deletions(-)
 rename meta/recipes-multimedia/alsa/{alsa-utils-scripts_1.1.5.bb => 
alsa-utils-scripts_1.1.6.bb} (100%)
 delete mode 100644 
meta/recipes-multimedia/alsa/alsa-utils/0001-alsactl-don-t-let-systemd-unit-restore-the-volume-wh.patch
 rename meta/recipes-multimedia/alsa/{alsa-utils_1.1.5.bb => 
alsa-utils_1.1.6.bb} (94%)

diff --git a/meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.5.bb 
b/meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.6.bb
similarity index 100%
rename from meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.5.bb
rename to meta/recipes-multimedia/alsa/alsa-utils-scripts_1.1.6.bb
diff --git 
a/meta/recipes-multimedia/alsa/alsa-utils/0001-alsactl-don-t-let-systemd-unit-restore-the-volume-wh.patch
 
b/meta/recipes-multimedia/alsa/alsa-utils/0001-alsactl-don-t-let-systemd-unit-restore-the-volume-wh.patch
deleted file mode 100644
index 907ca5ce0e..00
--- 
a/meta/recipes-multimedia/alsa/alsa-utils/0001-alsactl-don-t-let-systemd-unit-restore-the-volume-wh.patch
+++ /dev/null
@@ -1,56 +0,0 @@
-From b9d41b784657152a4c5683a9458dffab557f0689 Mon Sep 17 00:00:00 2001
-From: Koen Kooi 
-Date: Fri, 29 Aug 2014 18:58:56 +0300
-Subject: [PATCH] alsactl: don't let systemd unit restore the volume when
- asound.state is missing
-
-This avoids an error on bootup
-
-Filed as https://bugtrack.alsa-project.org/alsa-bug/view.php?id=5459
-
-Upstream-Status: Pending
-
-Signed-off-by: Koen Kooi 
-Signed-off-by: Cristian Iorga 
-
-Rebased on 1.1.0.
-
-Signed-off-by: Tanu Kaskinen 

- alsactl/Makefile.am | 7 ---
- alsactl/alsa-restore.service.in | 1 +
- 2 files changed, 5 insertions(+), 3 deletions(-)
-
-diff --git a/alsactl/Makefile.am b/alsactl/Makefile.am
-index 90fab9d..190525a 100644
 a/alsactl/Makefile.am
-+++ b/alsactl/Makefile.am
-@@ -38,9 +38,10 @@ install-data-hook:
- endif
- 
- edit = \
--  $(SED) -r -e 's,@sbindir\@,$(sbindir),g' \
---e 's,@mydatadir\@,$(mydatadir),g' \
---e 's,@daemonswitch\@,$(ALSACTL_DAEMONSWITCH),g' \
-+  $(SED) -e 's,@localstatedir\@,$(localstatedir),g' \
-+ -e 's,@sbindir\@,$(sbindir),g' \
-+ -e 's,@mydatadir\@,$(mydatadir),g' \
-+ -e 's,@daemonswitch\@,$(ALSACTL_DAEMONSWITCH),g' \
-   < $< > $@ || rm $@
- 
- alsa-state.service: alsa-state.service.in
-diff --git a/alsactl/alsa-restore.service.in b/alsactl/alsa-restore.service.in
-index 80fd5fd..6beb36d 100644
 a/alsactl/alsa-restore.service.in
-+++ b/alsactl/alsa-restore.service.in
-@@ -7,6 +7,7 @@
- Description=Save/Restore Sound Card State
- ConditionPathExists=!@daemonswitch@
- ConditionPathExistsGlob=/dev/snd/control*
-+ConditionPathExists=@localstatedir@/lib/alsa/asound.state
- 
- [Service]
- Type=oneshot
--- 
-2.14.2
-
diff --git a/meta/recipes-multimedia/alsa/alsa-utils_1.1.5.bb 
b/meta/recipes-multimedia/alsa/alsa-utils_1.1.6.bb
similarity index 94%
rename from meta/recipes-multimedia/alsa/alsa-utils_1.1.5.bb
rename to meta/recipes-multimedia/alsa/alsa-utils_1.1.6.bb
index 33215bf159..b8ba0cb22a 100644
--- a/meta/recipes-multimedia/alsa/alsa-utils_1.1.5.bb
+++ b/meta/recipes-multimedia/alsa/alsa-utils_1.1.6.bb
@@ -4,7 +4,7 @@ BUGTRACKER = 
"http://alsa-project.org/main/index.php/Bug_Tracking;
 SECTION = "console/utils"
 LICENSE = "GPLv2+"
 LIC_FILES_CHKSUM = "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552 \
-
file://alsactl/utils.c;beginline=1;endline=20;md5=fe9526b055e246b5558809a5ae25c0b9"
+
file://alsactl/utils.c;beginline=1;endline=20;md5=2ce7f7b6739487fb72c689d46521f958"
 DEPENDS = "alsa-lib ncurses libsamplerate0"
 
 PACKAGECONFIG ??= "udev"
@@ -20,11 +20,10 @@ PACKAGECONFIG[udev] = "--with-udev-rules-dir=`pkg-config 
--variable=udevdir udev
 PACKAGECONFIG[manpages] = "--enable-xmlto, --disable-xmlto, xmlto-native 
docbook-xml-dtd4-native docbook-xsl-stylesheets-native"
 
 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 \
   "
 
-SRC_URI[md5sum] = "dfe6ea147a5e07a056919591c2f5dac3"
-SRC_URI[sha256sum] = 
"320bd285e91db6e7fd7db3c9ec6f55b02f35449ff273c7844780ac6a5a3de2e8"
+SRC_URI[md5sum] = "50ae75567459646b843bed78d916b002"
+SRC_URI[sha256sum] = 
"155caecc40b2220f686f34ba3655a53e3bdbc0586adb1056733949feaaf7d36e"
 
 # On build machines with python-docutils 

[OE-core] [PATCH 0/3] libvorbis and alsa recipe upgrades

2018-05-22 Thread Tanu Kaskinen
Tanu Kaskinen (3):
  libvorbis: 1.3.5 -> 1.3.6
  alsa-plugins: 1.1.5 -> 1.1.6
  alsa-utils: 1.1.5 -> 1.1.6

 ...plugins_1.1.5.bb => alsa-plugins_1.1.6.bb} |   8 +-
 ...s_1.1.5.bb => alsa-utils-scripts_1.1.6.bb} |   0
 ...t-systemd-unit-restore-the-volume-wh.patch |  56 --
 ...lsa-utils_1.1.5.bb => alsa-utils_1.1.6.bb} |   7 +-
 .../0001-configure-Check-for-clang.patch  |  20 ++--
 .../libvorbis/libvorbis/CVE-2017-14632.patch  |  62 ---
 .../libvorbis/libvorbis/CVE-2017-14633.patch  |  42 
 .../libvorbis/libvorbis/CVE-2018-5146.patch   | 100 --
 ...{libvorbis_1.3.5.bb => libvorbis_1.3.6.bb} |   9 +-
 9 files changed, 20 insertions(+), 284 deletions(-)
 rename meta/recipes-multimedia/alsa/{alsa-plugins_1.1.5.bb => 
alsa-plugins_1.1.6.bb} (94%)
 rename meta/recipes-multimedia/alsa/{alsa-utils-scripts_1.1.5.bb => 
alsa-utils-scripts_1.1.6.bb} (100%)
 delete mode 100644 
meta/recipes-multimedia/alsa/alsa-utils/0001-alsactl-don-t-let-systemd-unit-restore-the-volume-wh.patch
 rename meta/recipes-multimedia/alsa/{alsa-utils_1.1.5.bb => 
alsa-utils_1.1.6.bb} (94%)
 delete mode 100644 
meta/recipes-multimedia/libvorbis/libvorbis/CVE-2017-14632.patch
 delete mode 100644 
meta/recipes-multimedia/libvorbis/libvorbis/CVE-2017-14633.patch
 delete mode 100644 
meta/recipes-multimedia/libvorbis/libvorbis/CVE-2018-5146.patch
 rename meta/recipes-multimedia/libvorbis/{libvorbis_1.3.5.bb => 
libvorbis_1.3.6.bb} (64%)

-- 
2.17.0

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


[OE-core] [PATCH 1/3] libvorbis: 1.3.5 -> 1.3.6

2018-05-22 Thread Tanu Kaskinen
Rebased 0001-configure-Check-for-clang.patch.

Removed the backported CVE patches.

License-Update: copyright years refreshed

Signed-off-by: Tanu Kaskinen 
---
 .../0001-configure-Check-for-clang.patch  |  20 ++--
 .../libvorbis/libvorbis/CVE-2017-14632.patch  |  62 ---
 .../libvorbis/libvorbis/CVE-2017-14633.patch  |  42 
 .../libvorbis/libvorbis/CVE-2018-5146.patch   | 100 --
 ...{libvorbis_1.3.5.bb => libvorbis_1.3.6.bb} |   9 +-
 5 files changed, 13 insertions(+), 220 deletions(-)
 delete mode 100644 
meta/recipes-multimedia/libvorbis/libvorbis/CVE-2017-14632.patch
 delete mode 100644 
meta/recipes-multimedia/libvorbis/libvorbis/CVE-2017-14633.patch
 delete mode 100644 
meta/recipes-multimedia/libvorbis/libvorbis/CVE-2018-5146.patch
 rename meta/recipes-multimedia/libvorbis/{libvorbis_1.3.5.bb => 
libvorbis_1.3.6.bb} (64%)

diff --git 
a/meta/recipes-multimedia/libvorbis/libvorbis/0001-configure-Check-for-clang.patch
 
b/meta/recipes-multimedia/libvorbis/libvorbis/0001-configure-Check-for-clang.patch
index 7dad0cd8a5..b06029b98b 100644
--- 
a/meta/recipes-multimedia/libvorbis/libvorbis/0001-configure-Check-for-clang.patch
+++ 
b/meta/recipes-multimedia/libvorbis/libvorbis/0001-configure-Check-for-clang.patch
@@ -1,4 +1,4 @@
-From 44b4511784f9b51c514dff4ceb3cbeaf9c374d08 Mon Sep 17 00:00:00 2001
+From d619ccf6c11ab574466914c57994a82fb99401af Mon Sep 17 00:00:00 2001
 From: Khem Raj 
 Date: Wed, 22 Mar 2017 16:06:55 +
 Subject: [PATCH] configure: Check for clang
@@ -13,12 +13,12 @@ Upstream-Status: Pending
  1 file changed, 17 insertions(+), 2 deletions(-)
 
 diff --git a/configure.ac b/configure.ac
-index eddd02d..00ecba5 100644
+index 28b0a14..2d4e984 100644
 --- a/configure.ac
 +++ b/configure.ac
-@@ -93,6 +93,16 @@ AC_ARG_ENABLE(examples,
-   
- AM_CONDITIONAL(BUILD_EXAMPLES, [test "x$enable_examples" = xyes]) 
+@@ -98,6 +98,16 @@ AC_ARG_ENABLE(examples,
+ 
+ AM_CONDITIONAL(BUILD_EXAMPLES, [test "x$enable_examples" = xyes])
  
 +AC_MSG_CHECKING([whether C compiler is clang])
 +$CC -x c /dev/null -dM -E > conftest.txt 2>&1
@@ -33,9 +33,9 @@ index eddd02d..00ecba5 100644
  dnl --
  dnl Set build flags based on environment
  dnl --
-@@ -127,10 +137,15 @@ else
+@@ -132,10 +142,15 @@ else
AC_MSG_RESULT([$GCC_VERSION])
-   case $host in 
+   case $host in
*86-*-linux*)
 +  if test "$CC_CLANG" = "1"; then
 +  ieeefp=""
@@ -43,8 +43,8 @@ index eddd02d..00ecba5 100644
 +  ieefp="-mno-ieee-fp"
 +  fi
DEBUG="-g -Wall -Wextra -D_REENTRANT -D__NO_MATH_INLINES 
-fsigned-char"
--  CFLAGS="-O3 -ffast-math -mno-ieee-fp -D_REENTRANT -fsigned-char"
-+  CFLAGS="-O3 -ffast-math -D_REENTRANT -fsigned-char ${ieefp}"
+-  CFLAGS="-O3 -Wall -Wextra -ffast-math -mno-ieee-fp -D_REENTRANT 
-fsigned-char"
++  CFLAGS="-O3 -Wall -Wextra -ffast-math -D_REENTRANT 
-fsigned-char ${ieefp}"
  # PROFILE="-Wall -Wextra -pg -g -O3 -ffast-math 
-D_REENTRANT -fsigned-char -fno-inline -static"
 -  PROFILE="-Wall -Wextra -pg -g -O3 -ffast-math -mno-ieee-fp 
-D_REENTRANT -fsigned-char -fno-inline"
 +  PROFILE="-Wall -Wextra -pg -g -O3 -ffast-math ${ieefp} 
-D_REENTRANT -fsigned-char -fno-inline"
@@ -52,5 +52,5 @@ index eddd02d..00ecba5 100644
# glibc < 2.1.3 has a serious FP bug in the math inline header
# that will cripple Vorbis.  Look to see if the magic FP stack
 -- 
-1.8.3.1
+2.17.0
 
diff --git a/meta/recipes-multimedia/libvorbis/libvorbis/CVE-2017-14632.patch 
b/meta/recipes-multimedia/libvorbis/libvorbis/CVE-2017-14632.patch
deleted file mode 100644
index 4036b966fe..00
--- a/meta/recipes-multimedia/libvorbis/libvorbis/CVE-2017-14632.patch
+++ /dev/null
@@ -1,62 +0,0 @@
-From 39704ce16835e5c019bb03f6a94dc1f0677406c5 Mon Sep 17 00:00:00 2001
-From: =?UTF-8?q?Guido=20G=C3=BCnther?= 
-Date: Wed, 15 Nov 2017 18:22:59 +0100
-Subject: [PATCH] CVE-2017-14632: vorbis_analysis_header_out: Don't clear opb
- if not initialized
-
-If the number of channels is not within the allowed range
-we call oggback_writeclear altough it's not initialized yet.
-
-This fixes
-
-=23371== Invalid free() / delete / delete[] / realloc()
-==23371==at 0x4C2CE1B: free (vg_replace_malloc.c:530)
-==23371==by 0x829CA31: oggpack_writeclear (in 
/usr/lib/x86_64-linux-gnu/libogg.so.0.8.2)
-==23371==by 0x84B96EE: vorbis_analysis_headerout (info.c:652)
-==23371==by 0x9FBCBCC: ??? (in 
/usr/lib/x86_64-linux-gnu/sox/libsox_fmt_vorbis.so)
-==23371==by 0x4E524F1: ??? (in 
/usr/lib/x86_64-linux-gnu/libsox.so.2.0.1)
-==23371==by 0x4E52CCA: sox_open_write (in 
/usr/lib/x86_64-linux-gnu/libsox.so.2.0.1)
-

[OE-core] [PATCH 2/3] alsa-plugins: 1.1.5 -> 1.1.6

2018-05-22 Thread Tanu Kaskinen
License-Update: FSF address updated

Signed-off-by: Tanu Kaskinen 
---
 .../alsa/{alsa-plugins_1.1.5.bb => alsa-plugins_1.1.6.bb} | 8 
 1 file changed, 4 insertions(+), 4 deletions(-)
 rename meta/recipes-multimedia/alsa/{alsa-plugins_1.1.5.bb => 
alsa-plugins_1.1.6.bb} (94%)

diff --git a/meta/recipes-multimedia/alsa/alsa-plugins_1.1.5.bb 
b/meta/recipes-multimedia/alsa/alsa-plugins_1.1.6.bb
similarity index 94%
rename from meta/recipes-multimedia/alsa/alsa-plugins_1.1.5.bb
rename to meta/recipes-multimedia/alsa/alsa-plugins_1.1.6.bb
index 4bd0acc54f..17c1879f95 100644
--- a/meta/recipes-multimedia/alsa/alsa-plugins_1.1.5.bb
+++ b/meta/recipes-multimedia/alsa/alsa-plugins_1.1.6.bb
@@ -14,15 +14,15 @@ SECTION = "multimedia"
 # of LGPLv2.1 like the rest of the plugins.
 LICENSE = "LGPLv2.1 & GPLv2+"
 LIC_FILES_CHKSUM = "\
-file://COPYING;md5=7fbc338309ac38fefcd64b04bb903e34 \
-file://COPYING.GPL;md5=94d55d512a9ba36caa9b7df079bae19f \
+file://COPYING;md5=a916467b91076e631dd8edb7424769c7 \
+file://COPYING.GPL;md5=59530bdf33659b29e73d4adb9f9f6552 \
 
file://m4/attributes.m4;endline=33;md5=b25958da44c02231e3641f1bccef53eb \
 
file://rate/rate_samplerate.c;endline=35;md5=fd77bce85f4a338c0e8ab18430b69fae \
 "
 
 SRC_URI = "ftp://ftp.alsa-project.org/pub/plugins/${BP}.tar.bz2;
-SRC_URI[md5sum] = "f898b0e6e8acf3f6f82bc2a0f0c84a97"
-SRC_URI[sha256sum] = 
"797da5f8f53379fbea28817bc466de16affd2c07849e84f1af8d5e22f7bb7f1c"
+SRC_URI[md5sum] = "8387279e99feeb2ecffaac5f293223d7"
+SRC_URI[sha256sum] = 
"6f1d31ebe3b1fa1cc8dade60b7bed1cb2583ac998167002d350dc0a5e3e40c13"
 
 DEPENDS += "alsa-lib"
 
-- 
2.17.0

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


[OE-core] [PATCH] ethtool: update 4.15 -> 4.16

2018-05-22 Thread Changhyeok Bae
Signed-off-by: Changhyeok Bae 
---
 meta/recipes-extended/ethtool/ethtool/avoid_parallel_tests.patch   | 2 +-
 meta/recipes-extended/ethtool/{ethtool_4.15.bb => ethtool_4.16.bb} | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)
 rename meta/recipes-extended/ethtool/{ethtool_4.15.bb => ethtool_4.16.bb} (88%)

diff --git a/meta/recipes-extended/ethtool/ethtool/avoid_parallel_tests.patch 
b/meta/recipes-extended/ethtool/ethtool/avoid_parallel_tests.patch
index 59299e2..ccb4dd3 100644
--- a/meta/recipes-extended/ethtool/ethtool/avoid_parallel_tests.patch
+++ b/meta/recipes-extended/ethtool/ethtool/avoid_parallel_tests.patch
@@ -19,7 +19,7 @@ index cb61a02..c75b9a9 100644
 --- a/configure.ac
 +++ b/configure.ac
 @@ -2,7 +2,7 @@ dnl Process this file with autoconf to produce a configure 
script.
- AC_INIT(ethtool, 4.15, net...@vger.kernel.org)
+ AC_INIT(ethtool, 4.16, net...@vger.kernel.org)
  AC_PREREQ(2.52)
  AC_CONFIG_SRCDIR([ethtool.c])
 -AM_INIT_AUTOMAKE([gnu])
diff --git a/meta/recipes-extended/ethtool/ethtool_4.15.bb 
b/meta/recipes-extended/ethtool/ethtool_4.16.bb
similarity index 88%
rename from meta/recipes-extended/ethtool/ethtool_4.15.bb
rename to meta/recipes-extended/ethtool/ethtool_4.16.bb
index 8538a75..5ccb162 100644
--- a/meta/recipes-extended/ethtool/ethtool_4.15.bb
+++ b/meta/recipes-extended/ethtool/ethtool_4.16.bb
@@ -11,8 +11,8 @@ SRC_URI = 
"${KERNELORG_MIRROR}/software/network/ethtool/ethtool-${PV}.tar.gz \
file://avoid_parallel_tests.patch \
"
 
-SRC_URI[md5sum] = "a4b83b35a4bc7d3646d973b0baf36c78"
-SRC_URI[sha256sum] = 
"3996127fc22950b865287bd9d28c0aa3fcd44e914a5a4ef01de64872106dc178"
+SRC_URI[md5sum] = "2b50adf06fc1b033951edef5987c32f3"
+SRC_URI[sha256sum] = 
"25f8fc3a4b2cb2cbfe8741148d8e743127dd7c951df96d8d6dccb88741f8a27d"
 
 inherit autotools ptest
 RDEPENDS_${PN}-ptest += "make"
-- 
2.7.4

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


Re: [OE-core] [PATCH 0/2] bind update req. new pkg in core

2018-05-22 Thread Martin Hundebøll



On 2018-05-22 03:06, Khem Raj wrote:


On Mon, May 21, 2018 at 3:44 PM Armin Kuster > wrote:


With this update, the python-ply package is required.
I copied the one from meta-python to core. Once this hits master,
I will send a patch to remove the same recipe from
meta-python.


Can this be turned on/off using packageconfig ?
If so then it would be ideal


I did send a patch for (some of) this some time ago:
https://patchwork.openembedded.org/patch/148257/

// Martin

If not then please send the removal patches for meta-python regardless 
so it can be tested together




Armin Kuster (2):
   bind: update to 9.12.1
   python3-ply: add package needed by bind 9.12 update

  ...0001-build-use-pkg-config-to-find-libxml2.patch | 13 +++---
  ...-gen.c-extend-DIRNAMESIZE-from-256-to-512.patch | 13 +++---
  .../0001-lib-dns-gen.c-fix-too-long-error.patch    | 13 +++---
  .../bind/bind/bind-confgen-build-unix.o-once.patch | 48
--
  ...-searching-for-json-headers-searches-sysr.patch | 13 +++---
  .../bind/bind/dont-test-on-host.patch              | 17 
  .../use-python3-and-fix-install-lib-path.patch     | 36

  .../bind/{bind_9.10.6.bb  => bind_9.12.1.bb
}        | 21 --
  meta/recipes-devtools/python/python-ply.inc        | 18 
  meta/recipes-devtools/python/python-ply_3.11.bb
    |  2 +
  10 files changed, 49 insertions(+), 145 deletions(-)
  delete mode 100644
meta/recipes-connectivity/bind/bind/bind-confgen-build-unix.o-once.patch
  delete mode 100644
meta/recipes-connectivity/bind/bind/dont-test-on-host.patch
  delete mode 100644

meta/recipes-connectivity/bind/bind/use-python3-and-fix-install-lib-path.patch
  rename meta/recipes-connectivity/bind/{bind_9.10.6.bb
 => bind_9.12.1.bb } (86%)
  create mode 100644 meta/recipes-devtools/python/python-ply.inc
  create mode 100644 meta/recipes-devtools/python/python-ply_3.11.bb


-- 
2.7.4


-- 
___

Openembedded-core mailing list
Openembedded-core@lists.openembedded.org

http://lists.openembedded.org/mailman/listinfo/openembedded-core





--
MARTIN HUNDEBØLL, Prevas A/S
Software Developer

Hedeager 3, DK-8200 Aarhus N
Phone +45 87438070
Mobile +45 25562438
martin.hundeb...@prevas.dk
www.prevas.com
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/4] sstate: allow specifying indirect dependencies to exclude from sysroot

2018-05-22 Thread Richard Purdie
On Tue, 2018-05-22 at 13:48 +0100, André Draszik wrote:
> > Have you looked at the performance impact of this change? It looks
> > like
> > it will be compiling the regexp each time inside a tight loop which
> > gets called once per task dependency which will show up
> > significantly
> > on the performance chart.
> True, and no, I haven't. Is there a good way to compare the
> performance with
> and without this change other than using 'time'?

I'd try something like "bitbake  -P -n" which should generate a
profile output for the dry run of building the image. That would not
build anything but would include all the dependency calculations (and
unfortunately all the fork overheadin this case).

> > 
> > We already have a "*" syntax used in
> > SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS
> > and I'd rather use that syntax here to avoid the use of re if we
> > can
> > help it. Could you see if that is possible please? That would also
> > keep
> > the syntax compatible.
> OK.

I think it uses glob btw which is much lighter weight than re.

Cheers,

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


Re: [OE-core] [PATCH 1/4] sstate: allow specifying indirect dependencies to exclude from sysroot

2018-05-22 Thread André Draszik
On Tue, 2018-05-22 at 13:36 +0100, Richard Purdie wrote:
> On Tue, 2018-05-22 at 13:25 +0100, André Draszik wrote:
> > diff --git a/meta/classes/sstate.bbclass
> > b/meta/classes/sstate.bbclass
> > index 1a95f8f2b9..e5b86ad705 100644
> > --- a/meta/classes/sstate.bbclass
> > +++ b/meta/classes/sstate.bbclass
> > @@ -897,6 +897,7 @@ def setscene_depvalid(task, taskdependees,
> > notneeded, d, log=None):
> >  # task is included in taskdependees too
> >  # Return - False - We need this dependency
> >  #- True - We can skip this dependency
> > +import re
> >  
> >  def logit(msg, log):
> >  if log is not None:
> > @@ -957,6 +958,18 @@ def setscene_depvalid(task, taskdependees,
> > notneeded, d, log=None):
> >  # Nothing need depend on libc-initial/gcc-cross-initial
> >  if "-initial" in taskdependees[task][0]:
> >  continue
> > +# Allow excluding certain recursive dependencies. If a
> > recipe needs it should add a
> > +# specific dependency itself, rather than relying on one
> > of its dependees to pull
> > +# them in.
> > +# See also http://lists.openembedded.org/pipermail/opene
> > mbedded-core/2018-January/146324.html
> > +not_needed = False
> > +for excl in (d.getVar('SSTATE_EXCLUDEDEPS_SYSROOT') or
> > "").split():
> > +if re.match(excl.split('->', 1)[0],
> > taskdependees[dep][0]):
> > +if re.match(excl.split('->', 1)[1],
> > taskdependees[task][0]):
> > +not_needed = True
> > +break
> > +if not_needed:
> > +continue
> >  # For meta-extsdk-toolchain we want all sysroot
> > dependencies
> >  if taskdependees[dep][0] == 'meta-extsdk-toolchain':
> >  return False
> 
> Have you looked at the performance impact of this change? It looks like
> it will be compiling the regexp each time inside a tight loop which
> gets called once per task dependency which will show up significantly
> on the performance chart.

True, and no, I haven't. Is there a good way to compare the performance with
and without this change other than using 'time'?

> We already have a "*" syntax used in SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS
> and I'd rather use that syntax here to avoid the use of re if we can
> help it. Could you see if that is possible please? That would also keep
> the syntax compatible.

OK.

Cheers,
Andre'

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


Re: [OE-core] [PATCH 4/4] sstate: avoid indirect bison/flex-native dependencies (via SSTATE_EXCLUDEDEPS_SYSROOT)

2018-05-22 Thread André Draszik
On Tue, 2018-05-22 at 13:33 +0100, Richard Purdie wrote:
> On Tue, 2018-05-22 at 13:25 +0100, André Draszik wrote:
> > From: André Draszik 
> > 
> > Avoid adding flex-native or bison-native to the sysroot without a
> > specific
> > dependency in the recipe. This means indirect dependencies
> > (e.g. X -> Y -> binutils-cross -> flex-native) no longer meet the
> > dependency incidentally. This improves determinism and avoids build
> > failures when people switch to external toolchains.
> > 
> > Based on an idea by Richard Purdie:
> > http://lists.openembedded.org/pipermail/openembedded-core/2018-Ja
> > nuary/146324.html
> > 
> > Signed-off-by: André Draszik 
> > ---
> >  meta/conf/layer.conf | 6 ++
> >  1 file changed, 6 insertions(+)
> 
> The trouble was that the patch above of mine doesn't actually work and
> breaks things from what I remember, some issue showed up in testing on
> our autobuilder. I don't remember exactly what it breaks unfortunately
> :(.

Well, I did test this series with the images I build, and nothing broke,
except for one missing dependency in meta-oe, for which I've sent a patch
already.

I think your patch placed the skipping in the wrong place? Maybe it was
intentional, but I have it in a different place, and I don't return (True)
on a match, but continue iterating over the remaining items.

In any case, if there are other breakages caused by this, the autobuilder
would at least have a log message, so I can fix the patch hopefully... :-)

Cheers,
Andre'

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


Re: [OE-core] [PATCH 1/4] sstate: allow specifying indirect dependencies to exclude from sysroot

2018-05-22 Thread Richard Purdie
On Tue, 2018-05-22 at 13:25 +0100, André Draszik wrote:
> diff --git a/meta/classes/sstate.bbclass
> b/meta/classes/sstate.bbclass
> index 1a95f8f2b9..e5b86ad705 100644
> --- a/meta/classes/sstate.bbclass
> +++ b/meta/classes/sstate.bbclass
> @@ -897,6 +897,7 @@ def setscene_depvalid(task, taskdependees,
> notneeded, d, log=None):
>  # task is included in taskdependees too
>  # Return - False - We need this dependency
>  #- True - We can skip this dependency
> +import re
>  
>  def logit(msg, log):
>  if log is not None:
> @@ -957,6 +958,18 @@ def setscene_depvalid(task, taskdependees,
> notneeded, d, log=None):
>  # Nothing need depend on libc-initial/gcc-cross-initial
>  if "-initial" in taskdependees[task][0]:
>  continue
> +# Allow excluding certain recursive dependencies. If a
> recipe needs it should add a
> +# specific dependency itself, rather than relying on one
> of its dependees to pull
> +# them in.
> +# See also http://lists.openembedded.org/pipermail/opene
> mbedded-core/2018-January/146324.html
> +not_needed = False
> +for excl in (d.getVar('SSTATE_EXCLUDEDEPS_SYSROOT') or
> "").split():
> +if re.match(excl.split('->', 1)[0],
> taskdependees[dep][0]):
> +if re.match(excl.split('->', 1)[1],
> taskdependees[task][0]):
> +not_needed = True
> +break
> +if not_needed:
> +continue
>  # For meta-extsdk-toolchain we want all sysroot
> dependencies
>  if taskdependees[dep][0] == 'meta-extsdk-toolchain':
>  return False

Have you looked at the performance impact of this change? It looks like
it will be compiling the regexp each time inside a tight loop which
gets called once per task dependency which will show up significantly
on the performance chart.

We already have a "*" syntax used in SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS
and I'd rather use that syntax here to avoid the use of re if we can
help it. Could you see if that is possible please? That would also keep
the syntax compatible.

Cheers,

Richard

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


Re: [OE-core] [PATCH 4/4] sstate: avoid indirect bison/flex-native dependencies (via SSTATE_EXCLUDEDEPS_SYSROOT)

2018-05-22 Thread Richard Purdie
On Tue, 2018-05-22 at 13:25 +0100, André Draszik wrote:
> From: André Draszik 
> 
> Avoid adding flex-native or bison-native to the sysroot without a
> specific
> dependency in the recipe. This means indirect dependencies
> (e.g. X -> Y -> binutils-cross -> flex-native) no longer meet the
> dependency incidentally. This improves determinism and avoids build
> failures when people switch to external toolchains.
> 
> Based on an idea by Richard Purdie:
> http://lists.openembedded.org/pipermail/openembedded-core/2018-Ja
> nuary/146324.html
> 
> Signed-off-by: André Draszik 
> ---
>  meta/conf/layer.conf | 6 ++
>  1 file changed, 6 insertions(+)

The trouble was that the patch above of mine doesn't actually work and
breaks things from what I remember, some issue showed up in testing on
our autobuilder. I don't remember exactly what it breaks unfortunately
:(.

Cheers,

Richard

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


[OE-core] [PATCH 3/4] sstate: use SSTATE_EXCLUDEDEPS_SYSROOT for skipping base-passwd|shadow-sysroot recipes

2018-05-22 Thread André Draszik
From: André Draszik 

Use the newly introduced SSTATE_EXCLUDEDEPS_SYSROOT for specifying
the base-passwd|shadow-sysroot recipes to be excluded from a recipe sysroot.

Signed-off-by: André Draszik 
---
 meta/classes/sstate.bbclass | 3 ---
 meta/conf/layer.conf| 2 ++
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index 362729376a..531c8f3fdb 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -952,9 +952,6 @@ def setscene_depvalid(task, taskdependees, notneeded, d, 
log=None):
 
 # Consider sysroot depending on sysroot tasks
 if taskdependees[task][1] == 'do_populate_sysroot' and 
taskdependees[dep][1] == 'do_populate_sysroot':
-# base-passwd/shadow-sysroot don't need their dependencies
-if taskdependees[dep][0].endswith(("base-passwd", 
"shadow-sysroot")):
-continue
 # Allow excluding certain recursive dependencies. If a recipe 
needs it should add a
 # specific dependency itself, rather than relying on one of its 
dependees to pull
 # them in.
diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf
index a8bebe102c..0a8f8ed9eb 100644
--- a/meta/conf/layer.conf
+++ b/meta/conf/layer.conf
@@ -79,8 +79,10 @@ SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS += " \
 "
 
 # Nothing needs to depend on libc-initial/gcc-cross-initial
+# base-passwd/shadow-sysroot don't need their dependencies
 SSTATE_EXCLUDEDEPS_SYSROOT += "\
 .*->.*-initial.* \
+.*(base-passwd|shadow-sysroot)->.* \
 "
 
 # We need to keep bitbake tools in PATH
-- 
2.17.0

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


[OE-core] [PATCH 1/4] sstate: allow specifying indirect dependencies to exclude from sysroot

2018-05-22 Thread André Draszik
From: André Draszik 

Currently, a dependency on any -native recipe will pull in
all dependencies of that -native recipe in the recipe
sysroot. This behaviour might not always be wanted, e.g.
when that -native recipe depends on build-tools that are
not relevant for the current recipe.

This change adds a SSTATE_EXCLUDEDEPS_SYSROOT variable,
which will be evaluated for such recursive dependencies to
be excluded. The idea is similar to
   
http://lists.openembedded.org/pipermail/openembedded-core/2018-January/146324.html
except that the list is not hard-coded anymore.

SSTATE_EXCLUDEDEPS_SYSROOT is evaluated as two regular
expressions of recipe and dependency to ignore, e.g. in
the above flex-native / bison-native use-case, one would
specify

SSTATE_EXCLUDEDEPS_SYSROOT = ".*->(flex|bison)-native"

in layer.conf.

The existing special handling of "-initial" as well as
"base-passwd" and "shadow-sysroot" could also be
streamlined:

SSTATE_EXCLUDEDEPS_SYSROOT += "\
.*->.*-initial.* \
.*(base-passwd|shadow-sysroot)->.* \
"

Another anticipated user is meta-java, where certain newer
JDKs can only be bootstrapped (built) using older JDKs,
but it doesn't make much sense to copy all those older
JDKs and their own build tools (ant, etc.) into the
sysroot of recipes wanting to be built using the newer JDK
(only), e.g.:

SSTATE_EXCLUDEDEPS_SYSROOT += "\

openjdk-8-native->(ant-native|attr-native|coreutils-native|icedtea7-native|libxslt-native|make-native|openssl-native|zip-native|unzip-native)
 \
"

Signed-off-by: André Draszik 
---
 meta/classes/sstate.bbclass | 13 +
 1 file changed, 13 insertions(+)

diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index 1a95f8f2b9..e5b86ad705 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -897,6 +897,7 @@ def setscene_depvalid(task, taskdependees, notneeded, d, 
log=None):
 # task is included in taskdependees too
 # Return - False - We need this dependency
 #- True - We can skip this dependency
+import re
 
 def logit(msg, log):
 if log is not None:
@@ -957,6 +958,18 @@ def setscene_depvalid(task, taskdependees, notneeded, d, 
log=None):
 # Nothing need depend on libc-initial/gcc-cross-initial
 if "-initial" in taskdependees[task][0]:
 continue
+# Allow excluding certain recursive dependencies. If a recipe 
needs it should add a
+# specific dependency itself, rather than relying on one of its 
dependees to pull
+# them in.
+# See also 
http://lists.openembedded.org/pipermail/openembedded-core/2018-January/146324.html
+not_needed = False
+for excl in (d.getVar('SSTATE_EXCLUDEDEPS_SYSROOT') or "").split():
+if re.match(excl.split('->', 1)[0], taskdependees[dep][0]):
+if re.match(excl.split('->', 1)[1], 
taskdependees[task][0]):
+not_needed = True
+break
+if not_needed:
+continue
 # For meta-extsdk-toolchain we want all sysroot dependencies
 if taskdependees[dep][0] == 'meta-extsdk-toolchain':
 return False
-- 
2.17.0

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


[OE-core] [PATCH 2/4] sstate: use SSTATE_EXCLUDEDEPS_SYSROOT for skipping *-initial recipes

2018-05-22 Thread André Draszik
From: André Draszik 

Use the newly introduced SSTATE_EXCLUDEDEPS_SYSROOT for specifying
the *-initial recipes to be excluded from a recipe sysroot.

Signed-off-by: André Draszik 
---
 meta/classes/sstate.bbclass | 3 ---
 meta/conf/layer.conf| 5 +
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/classes/sstate.bbclass b/meta/classes/sstate.bbclass
index e5b86ad705..362729376a 100644
--- a/meta/classes/sstate.bbclass
+++ b/meta/classes/sstate.bbclass
@@ -955,9 +955,6 @@ def setscene_depvalid(task, taskdependees, notneeded, d, 
log=None):
 # base-passwd/shadow-sysroot don't need their dependencies
 if taskdependees[dep][0].endswith(("base-passwd", 
"shadow-sysroot")):
 continue
-# Nothing need depend on libc-initial/gcc-cross-initial
-if "-initial" in taskdependees[task][0]:
-continue
 # Allow excluding certain recursive dependencies. If a recipe 
needs it should add a
 # specific dependency itself, rather than relying on one of its 
dependees to pull
 # them in.
diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf
index 315fb936fa..a8bebe102c 100644
--- a/meta/conf/layer.conf
+++ b/meta/conf/layer.conf
@@ -78,5 +78,10 @@ SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS += " \
   weston-init->kbd \
 "
 
+# Nothing needs to depend on libc-initial/gcc-cross-initial
+SSTATE_EXCLUDEDEPS_SYSROOT += "\
+.*->.*-initial.* \
+"
+
 # We need to keep bitbake tools in PATH
 PATH := 
"${@os.path.dirname(bb.utils.which(d.getVar('PATH'),'bitbake'))}:${HOSTTOOLS_DIR}"
-- 
2.17.0

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


[OE-core] [PATCH 4/4] sstate: avoid indirect bison/flex-native dependencies (via SSTATE_EXCLUDEDEPS_SYSROOT)

2018-05-22 Thread André Draszik
From: André Draszik 

Avoid adding flex-native or bison-native to the sysroot without a specific
dependency in the recipe. This means indirect dependencies
(e.g. X -> Y -> binutils-cross -> flex-native) no longer meet the
dependency incidentally. This improves determinism and avoids build
failures when people switch to external toolchains.

Based on an idea by Richard Purdie:

http://lists.openembedded.org/pipermail/openembedded-core/2018-January/146324.html

Signed-off-by: André Draszik 
---
 meta/conf/layer.conf | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf
index 0a8f8ed9eb..106f9900dd 100644
--- a/meta/conf/layer.conf
+++ b/meta/conf/layer.conf
@@ -78,6 +78,12 @@ SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS += " \
   weston-init->kbd \
 "
 
+# Avoid adding flex-native or bison-native to the sysroot without a specific
+# dependency in the recipe. This means indirect dependencies
+# (e.g. X -> Y -> binutils-cross -> flex-native) no longer meet the
+# dependency incidentally. This improves determinism and avoids build
+# failures when people switch to external toolchains.
+SSTATE_EXCLUDEDEPS_SYSROOT += ".*->(flex|bison)-native"
 # Nothing needs to depend on libc-initial/gcc-cross-initial
 # base-passwd/shadow-sysroot don't need their dependencies
 SSTATE_EXCLUDEDEPS_SYSROOT += "\
-- 
2.17.0

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


Re: [OE-core] [PATCH 3/4] mtd-utils: upgrade 2.0.1 -> 2.0.2

2018-05-22 Thread Burton, Ross
On 22 May 2018 at 08:49, Andrea Adami  wrote:
> I am just wondering about this patch:we carry it since 2009  [1] and
> is still marked as pending.
> Why was it necessary to ignore part of the filesystem? Can you remember?

Make that at least 2004.  This commit in oe-classic touches it:

commit c70605ebdca81412a93b4d867de7931543193f26
Author: Chris Larson 
Date:   Thu Dec 23 21:24:57 2004 +

Update mtd to a current snapshot, change the mkfs.jffs2 -i cmdline option to
-I, and rename mtd and mtd-native to mtd-utils and mtd-utils-native,
respectively.

Prior history is lost to the git/bk migration unless anyone has a
BitKeeper clone to mine...

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


Re: [OE-core] [PATCH] e2fsprogs: 1.43.8 -> 1.44.2

2018-05-22 Thread Andrea Adami
On Tue, May 22, 2018 at 10:22 AM, Andrej Valek  wrote:
> License-Update: The checksum of LIC_FILES_CHKSUM has been changed due to
>  changed FSF address. The content of NOTICE has no change.
>
> Signed-off-by: Andrej Valek 
> ---
>  meta/recipes-devtools/e2fsprogs/e2fsprogs.inc  |  2 +-
>  ...ix-build-failure-in-swapfs.c-on-big-endia.patch | 40 
> --
>  ...fs-enable-the-metadata_csum-and-64bit-fea.patch | 36 ---
>  .../{e2fsprogs_1.43.8.bb => e2fsprogs_1.44.2.bb}   |  4 +--
>  4 files changed, 2 insertions(+), 80 deletions(-)
>  delete mode 100644 
> meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
>  delete mode 100644 
> meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
>  rename meta/recipes-devtools/e2fsprogs/{e2fsprogs_1.43.8.bb => 
> e2fsprogs_1.44.2.bb} (95%)
>
> diff --git a/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc 
> b/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
> index 81d90a134c..a54a11c354 100644
> --- a/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
> +++ b/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
> @@ -9,7 +9,7 @@ LICENSE_e2fsprogs-mke2fs = "GPLv2"
>  LICENSE_e2fsprogs-fsck = "GPLv2"
>  LICENSE_e2fsprogs-tune2fs = "GPLv2"
>  LICENSE_e2fsprogs-badblocks = "GPLv2"
> -LIC_FILES_CHKSUM = "file://NOTICE;md5=b48f21d765b875bd10400975d12c1ca2 \
> +LIC_FILES_CHKSUM = "file://NOTICE;md5=d50be0580c0b0a7fbc7a4830bbe6c12b \
>
> file://lib/ext2fs/ext2fs.h;beginline=1;endline=9;md5=596a8dedcb4e731c6b21c7a46fba6bef
>  \
>
> file://lib/e2p/e2p.h;beginline=1;endline=7;md5=8a74ade8f9d65095d70ef2d4bf48e36a
>  \
>
> file://lib/uuid/uuid.h.in;beginline=1;endline=32;md5=dbb8079e114a5f841934b99e59c8820a
>  \
> diff --git 
> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
>  
> b/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
> deleted file mode 100644
> index ff1284edc4..00
> --- 
> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
> +++ /dev/null
> @@ -1,40 +0,0 @@
> -From ae6603206176b771c9ed1f82569f68fc1257f014 Mon Sep 17 00:00:00 2001
> -From: Theodore Ts'o 
> -Date: Wed, 3 Jan 2018 01:32:02 -0500
> -Subject: [PATCH] libext2fs: fix build failure in swapfs.c on big-endian
> - systems
> -
> -Addresses-Debian-Bug: #886119
> -
> -Reported-by: James Clarke 
> -Signed-off-by: Theodore Ts'o 
> 
> -Upstream-Status: Backport
> -
> - lib/ext2fs/swapfs.c | 3 ++-
> - 1 file changed, 2 insertions(+), 1 deletion(-)
> -
> -diff --git a/lib/ext2fs/swapfs.c b/lib/ext2fs/swapfs.c
> -index 6e5cff97..b13dc422 100644
>  a/lib/ext2fs/swapfs.c
> -+++ b/lib/ext2fs/swapfs.c
> -@@ -19,6 +19,7 @@
> -
> - #include "ext2_fs.h"
> - #include "ext2fs.h"
> -+#include "ext2fsP.h"
> - #include 
> -
> - #ifdef WORDS_BIGENDIAN
> -@@ -358,7 +359,7 @@ void ext2fs_swap_inode_full(ext2_filsys fs, struct 
> ext2_inode_large *t,
> -   if (inode_includes(inode_size, i_projid))
> - t->i_projid = ext2fs_swab16(f->i_projid);
> -   /* catch new static fields added after i_projid */
> --  EXT2FS_BUILD_BUG_ON(sizeof(ext2_inode_large) != 160);
> -+  EXT2FS_BUILD_BUG_ON(sizeof(struct ext2_inode_large) != 160);
> -
> -   i = sizeof(struct ext2_inode) + extra_isize + sizeof(__u32);
> -   if (bufsize < (int) i)
> ---
> -2.16.1
> -
> diff --git 
> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
>  
> b/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
> deleted file mode 100644
> index 38e1bdb3d8..00
> --- 
> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
> +++ /dev/null
> @@ -1,36 +0,0 @@
> -From 0a392baf1874964651115d9f77b0daa6851d1daa Mon Sep 17 00:00:00 2001
> -From: Jonathan Liu 
> -Date: Tue, 1 Mar 2016 14:28:01 +1100
> -Subject: [PATCH] Revert "mke2fs: enable the metadata_csum and 64bit features
> - by default"
> -
> -This reverts commit cd27af3ecb83e8fd1e3eaa14994284a1818c7c15 as we
> -don't want to enable features by default that are not supported by
> -the latest stable e2fsprogs release.
> -
> -Upstream-Status: Inappropriate [configuration]
> -Signed-off-by: Jonathan Liu 
> -
> -Rebase to 1.43:
> -The upstream has disabled metadata_csum by default
> -this rebase just revert 64bit feature.
> -Signed-off-by: Hongxu Jia 
> 
> - misc/mke2fs.conf.in | 3 ++-
> - 1 file changed, 2 insertions(+), 1 deletion(-)
> -
> -Index: git/misc/mke2fs.conf.in
> 

[OE-core] [PATCH v2] e2fsprogs: 1.43.8 -> 1.44.2

2018-05-22 Thread Andrej Valek
License-Update: The checksum of LIC_FILES_CHKSUM has been changed due to
changed FSF address. The content of NOTICE has no change.

Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
 - metadata_csum has been enabled by default
0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
 - already merged in this version

This update needs to be queued with btrfs version >= 4.15.2|4.16.0.
 - already submitted previously

Signed-off-by: Andrej Valek 
---
 meta/recipes-devtools/e2fsprogs/e2fsprogs.inc  |  2 +-
 ...ix-build-failure-in-swapfs.c-on-big-endia.patch | 40 --
 ...fs-enable-the-metadata_csum-and-64bit-fea.patch | 36 ---
 .../{e2fsprogs_1.43.8.bb => e2fsprogs_1.44.2.bb}   |  4 +--
 4 files changed, 2 insertions(+), 80 deletions(-)
 delete mode 100644 
meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
 delete mode 100644 
meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
 rename meta/recipes-devtools/e2fsprogs/{e2fsprogs_1.43.8.bb => 
e2fsprogs_1.44.2.bb} (95%)

diff --git a/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc 
b/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
index 81d90a134c..a54a11c354 100644
--- a/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
+++ b/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
@@ -9,7 +9,7 @@ LICENSE_e2fsprogs-mke2fs = "GPLv2"
 LICENSE_e2fsprogs-fsck = "GPLv2"
 LICENSE_e2fsprogs-tune2fs = "GPLv2"
 LICENSE_e2fsprogs-badblocks = "GPLv2"
-LIC_FILES_CHKSUM = "file://NOTICE;md5=b48f21d765b875bd10400975d12c1ca2 \
+LIC_FILES_CHKSUM = "file://NOTICE;md5=d50be0580c0b0a7fbc7a4830bbe6c12b \
   
file://lib/ext2fs/ext2fs.h;beginline=1;endline=9;md5=596a8dedcb4e731c6b21c7a46fba6bef
 \
   
file://lib/e2p/e2p.h;beginline=1;endline=7;md5=8a74ade8f9d65095d70ef2d4bf48e36a 
\
   
file://lib/uuid/uuid.h.in;beginline=1;endline=32;md5=dbb8079e114a5f841934b99e59c8820a
 \
diff --git 
a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
 
b/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
deleted file mode 100644
index ff1284edc4..00
--- 
a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
+++ /dev/null
@@ -1,40 +0,0 @@
-From ae6603206176b771c9ed1f82569f68fc1257f014 Mon Sep 17 00:00:00 2001
-From: Theodore Ts'o 
-Date: Wed, 3 Jan 2018 01:32:02 -0500
-Subject: [PATCH] libext2fs: fix build failure in swapfs.c on big-endian
- systems
-
-Addresses-Debian-Bug: #886119
-
-Reported-by: James Clarke 
-Signed-off-by: Theodore Ts'o 

-Upstream-Status: Backport
-
- lib/ext2fs/swapfs.c | 3 ++-
- 1 file changed, 2 insertions(+), 1 deletion(-)
-
-diff --git a/lib/ext2fs/swapfs.c b/lib/ext2fs/swapfs.c
-index 6e5cff97..b13dc422 100644
 a/lib/ext2fs/swapfs.c
-+++ b/lib/ext2fs/swapfs.c
-@@ -19,6 +19,7 @@
- 
- #include "ext2_fs.h"
- #include "ext2fs.h"
-+#include "ext2fsP.h"
- #include 
- 
- #ifdef WORDS_BIGENDIAN
-@@ -358,7 +359,7 @@ void ext2fs_swap_inode_full(ext2_filsys fs, struct 
ext2_inode_large *t,
-   if (inode_includes(inode_size, i_projid))
- t->i_projid = ext2fs_swab16(f->i_projid);
-   /* catch new static fields added after i_projid */
--  EXT2FS_BUILD_BUG_ON(sizeof(ext2_inode_large) != 160);
-+  EXT2FS_BUILD_BUG_ON(sizeof(struct ext2_inode_large) != 160);
- 
-   i = sizeof(struct ext2_inode) + extra_isize + sizeof(__u32);
-   if (bufsize < (int) i)
--- 
-2.16.1
-
diff --git 
a/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
 
b/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
deleted file mode 100644
index 38e1bdb3d8..00
--- 
a/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-From 0a392baf1874964651115d9f77b0daa6851d1daa Mon Sep 17 00:00:00 2001
-From: Jonathan Liu 
-Date: Tue, 1 Mar 2016 14:28:01 +1100
-Subject: [PATCH] Revert "mke2fs: enable the metadata_csum and 64bit features
- by default"
-
-This reverts commit cd27af3ecb83e8fd1e3eaa14994284a1818c7c15 as we
-don't want to enable features by default that are not supported by
-the latest stable e2fsprogs release.
-
-Upstream-Status: Inappropriate [configuration]
-Signed-off-by: Jonathan Liu 
-
-Rebase to 1.43:
-The upstream has disabled metadata_csum by default
-this rebase just revert 64bit feature.
-Signed-off-by: Hongxu Jia 

- misc/mke2fs.conf.in | 3 ++-
- 1 file changed, 2 insertions(+), 1 deletion(-)
-
-Index: git/misc/mke2fs.conf.in

Re: [OE-core] [PATCH] e2fsprogs: 1.43.8 -> 1.44.2

2018-05-22 Thread Andrej Valek
Hi Ross,

This commit
https://git.kernel.org/pub/scm/fs/ext2/e2fsprogs.git/commit/misc/mke2fs.conf.in?id=d0b9e0a6aa7d6805338a43b4e372623352d8df09
has enabled metadata_csum by default. So, this patch is obsolete.

Anyway, I have to send v2 to update/patch btrfs due to changed interface
header (ext2_ext_attr.h).

Regards,
Andrej

On 05/22/18 12:21, Burton, Ross wrote:
> Can you justify the removal of
> Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch please?
> 
> Ross
> 
> On 22 May 2018 at 09:22, Andrej Valek  wrote:
>> License-Update: The checksum of LIC_FILES_CHKSUM has been changed due to
>>  changed FSF address. The content of NOTICE has no change.
>>
>> Signed-off-by: Andrej Valek 
>> ---
>>  meta/recipes-devtools/e2fsprogs/e2fsprogs.inc  |  2 +-
>>  ...ix-build-failure-in-swapfs.c-on-big-endia.patch | 40 
>> --
>>  ...fs-enable-the-metadata_csum-and-64bit-fea.patch | 36 ---
>>  .../{e2fsprogs_1.43.8.bb => e2fsprogs_1.44.2.bb}   |  4 +--
>>  4 files changed, 2 insertions(+), 80 deletions(-)
>>  delete mode 100644 
>> meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
>>  delete mode 100644 
>> meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
>>  rename meta/recipes-devtools/e2fsprogs/{e2fsprogs_1.43.8.bb => 
>> e2fsprogs_1.44.2.bb} (95%)
>>
>> diff --git a/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc 
>> b/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
>> index 81d90a134c..a54a11c354 100644
>> --- a/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
>> +++ b/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
>> @@ -9,7 +9,7 @@ LICENSE_e2fsprogs-mke2fs = "GPLv2"
>>  LICENSE_e2fsprogs-fsck = "GPLv2"
>>  LICENSE_e2fsprogs-tune2fs = "GPLv2"
>>  LICENSE_e2fsprogs-badblocks = "GPLv2"
>> -LIC_FILES_CHKSUM = "file://NOTICE;md5=b48f21d765b875bd10400975d12c1ca2 \
>> +LIC_FILES_CHKSUM = "file://NOTICE;md5=d50be0580c0b0a7fbc7a4830bbe6c12b \
>>
>> file://lib/ext2fs/ext2fs.h;beginline=1;endline=9;md5=596a8dedcb4e731c6b21c7a46fba6bef
>>  \
>>
>> file://lib/e2p/e2p.h;beginline=1;endline=7;md5=8a74ade8f9d65095d70ef2d4bf48e36a
>>  \
>>
>> file://lib/uuid/uuid.h.in;beginline=1;endline=32;md5=dbb8079e114a5f841934b99e59c8820a
>>  \
>> diff --git 
>> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
>>  
>> b/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
>> deleted file mode 100644
>> index ff1284edc4..00
>> --- 
>> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
>> +++ /dev/null
>> @@ -1,40 +0,0 @@
>> -From ae6603206176b771c9ed1f82569f68fc1257f014 Mon Sep 17 00:00:00 2001
>> -From: Theodore Ts'o 
>> -Date: Wed, 3 Jan 2018 01:32:02 -0500
>> -Subject: [PATCH] libext2fs: fix build failure in swapfs.c on big-endian
>> - systems
>> -
>> -Addresses-Debian-Bug: #886119
>> -
>> -Reported-by: James Clarke 
>> -Signed-off-by: Theodore Ts'o 
>> 
>> -Upstream-Status: Backport
>> -
>> - lib/ext2fs/swapfs.c | 3 ++-
>> - 1 file changed, 2 insertions(+), 1 deletion(-)
>> -
>> -diff --git a/lib/ext2fs/swapfs.c b/lib/ext2fs/swapfs.c
>> -index 6e5cff97..b13dc422 100644
>>  a/lib/ext2fs/swapfs.c
>> -+++ b/lib/ext2fs/swapfs.c
>> -@@ -19,6 +19,7 @@
>> -
>> - #include "ext2_fs.h"
>> - #include "ext2fs.h"
>> -+#include "ext2fsP.h"
>> - #include 
>> -
>> - #ifdef WORDS_BIGENDIAN
>> -@@ -358,7 +359,7 @@ void ext2fs_swap_inode_full(ext2_filsys fs, struct 
>> ext2_inode_large *t,
>> -   if (inode_includes(inode_size, i_projid))
>> - t->i_projid = ext2fs_swab16(f->i_projid);
>> -   /* catch new static fields added after i_projid */
>> --  EXT2FS_BUILD_BUG_ON(sizeof(ext2_inode_large) != 160);
>> -+  EXT2FS_BUILD_BUG_ON(sizeof(struct ext2_inode_large) != 160);
>> -
>> -   i = sizeof(struct ext2_inode) + extra_isize + sizeof(__u32);
>> -   if (bufsize < (int) i)
>> ---
>> -2.16.1
>> -
>> diff --git 
>> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
>>  
>> b/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
>> deleted file mode 100644
>> index 38e1bdb3d8..00
>> --- 
>> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
>> +++ /dev/null
>> @@ -1,36 +0,0 @@
>> -From 0a392baf1874964651115d9f77b0daa6851d1daa Mon Sep 17 00:00:00 2001
>> -From: Jonathan Liu 
>> -Date: Tue, 1 Mar 2016 14:28:01 +1100
>> -Subject: [PATCH] Revert "mke2fs: enable the metadata_csum and 64bit features
>> - by default"
>> -
>> -This reverts commit 

Re: [OE-core] [PATCH] arch-armv7a.inc: default to Thumb2 instruction set for armv7a and above

2018-05-22 Thread Khem Raj
On Tue, May 22, 2018 at 5:58 AM,   wrote:
> Andre McCurdy wrote:
>> Although there may still be specific cases which can benefit from the
>> ARM instruction set, the Thumb2 instruction set is generally a better
>> default for armv7a class CPUs. Distros such as Debian and Fedora have
>> been targeting Thumb2 by default for some time.
>>
>> Note that setting ARM_INSTRUCTION_SET has no effect unless
>> TUNE_FEATURES contains "thumb" (which is controlled by the "t" suffix
>> in DEFAULTTUNE, e.g. armv7vehf-neon -vs- armv7vethf-neon, etc) so out
>> of tree machine configs may need to update their DEFAULTTUNE to take
>> advantage of this change.
>
> I recently ran into some major problems due to thumb vs arm. It turns out 
> glibc doesn't support systems with thumb disabled anymore. See this bug 
> report for details:
>
> https://sourceware.org/bugzilla/show_bug.cgi?id=23031
>
> This might be useful input for this discussion also.
>

Thanks, this would be useful for us to check if kernel defconfig does
not disable CONFIG_ARM_THUMB
on armv6t2+ regardless of this patch. This patch however will require
this option enabled too.
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] e2fsprogs: 1.43.8 -> 1.44.2

2018-05-22 Thread Burton, Ross
Can you justify the removal of
Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch please?

Ross

On 22 May 2018 at 09:22, Andrej Valek  wrote:
> License-Update: The checksum of LIC_FILES_CHKSUM has been changed due to
>  changed FSF address. The content of NOTICE has no change.
>
> Signed-off-by: Andrej Valek 
> ---
>  meta/recipes-devtools/e2fsprogs/e2fsprogs.inc  |  2 +-
>  ...ix-build-failure-in-swapfs.c-on-big-endia.patch | 40 
> --
>  ...fs-enable-the-metadata_csum-and-64bit-fea.patch | 36 ---
>  .../{e2fsprogs_1.43.8.bb => e2fsprogs_1.44.2.bb}   |  4 +--
>  4 files changed, 2 insertions(+), 80 deletions(-)
>  delete mode 100644 
> meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
>  delete mode 100644 
> meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
>  rename meta/recipes-devtools/e2fsprogs/{e2fsprogs_1.43.8.bb => 
> e2fsprogs_1.44.2.bb} (95%)
>
> diff --git a/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc 
> b/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
> index 81d90a134c..a54a11c354 100644
> --- a/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
> +++ b/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
> @@ -9,7 +9,7 @@ LICENSE_e2fsprogs-mke2fs = "GPLv2"
>  LICENSE_e2fsprogs-fsck = "GPLv2"
>  LICENSE_e2fsprogs-tune2fs = "GPLv2"
>  LICENSE_e2fsprogs-badblocks = "GPLv2"
> -LIC_FILES_CHKSUM = "file://NOTICE;md5=b48f21d765b875bd10400975d12c1ca2 \
> +LIC_FILES_CHKSUM = "file://NOTICE;md5=d50be0580c0b0a7fbc7a4830bbe6c12b \
>
> file://lib/ext2fs/ext2fs.h;beginline=1;endline=9;md5=596a8dedcb4e731c6b21c7a46fba6bef
>  \
>
> file://lib/e2p/e2p.h;beginline=1;endline=7;md5=8a74ade8f9d65095d70ef2d4bf48e36a
>  \
>
> file://lib/uuid/uuid.h.in;beginline=1;endline=32;md5=dbb8079e114a5f841934b99e59c8820a
>  \
> diff --git 
> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
>  
> b/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
> deleted file mode 100644
> index ff1284edc4..00
> --- 
> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
> +++ /dev/null
> @@ -1,40 +0,0 @@
> -From ae6603206176b771c9ed1f82569f68fc1257f014 Mon Sep 17 00:00:00 2001
> -From: Theodore Ts'o 
> -Date: Wed, 3 Jan 2018 01:32:02 -0500
> -Subject: [PATCH] libext2fs: fix build failure in swapfs.c on big-endian
> - systems
> -
> -Addresses-Debian-Bug: #886119
> -
> -Reported-by: James Clarke 
> -Signed-off-by: Theodore Ts'o 
> 
> -Upstream-Status: Backport
> -
> - lib/ext2fs/swapfs.c | 3 ++-
> - 1 file changed, 2 insertions(+), 1 deletion(-)
> -
> -diff --git a/lib/ext2fs/swapfs.c b/lib/ext2fs/swapfs.c
> -index 6e5cff97..b13dc422 100644
>  a/lib/ext2fs/swapfs.c
> -+++ b/lib/ext2fs/swapfs.c
> -@@ -19,6 +19,7 @@
> -
> - #include "ext2_fs.h"
> - #include "ext2fs.h"
> -+#include "ext2fsP.h"
> - #include 
> -
> - #ifdef WORDS_BIGENDIAN
> -@@ -358,7 +359,7 @@ void ext2fs_swap_inode_full(ext2_filsys fs, struct 
> ext2_inode_large *t,
> -   if (inode_includes(inode_size, i_projid))
> - t->i_projid = ext2fs_swab16(f->i_projid);
> -   /* catch new static fields added after i_projid */
> --  EXT2FS_BUILD_BUG_ON(sizeof(ext2_inode_large) != 160);
> -+  EXT2FS_BUILD_BUG_ON(sizeof(struct ext2_inode_large) != 160);
> -
> -   i = sizeof(struct ext2_inode) + extra_isize + sizeof(__u32);
> -   if (bufsize < (int) i)
> ---
> -2.16.1
> -
> diff --git 
> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
>  
> b/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
> deleted file mode 100644
> index 38e1bdb3d8..00
> --- 
> a/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
> +++ /dev/null
> @@ -1,36 +0,0 @@
> -From 0a392baf1874964651115d9f77b0daa6851d1daa Mon Sep 17 00:00:00 2001
> -From: Jonathan Liu 
> -Date: Tue, 1 Mar 2016 14:28:01 +1100
> -Subject: [PATCH] Revert "mke2fs: enable the metadata_csum and 64bit features
> - by default"
> -
> -This reverts commit cd27af3ecb83e8fd1e3eaa14994284a1818c7c15 as we
> -don't want to enable features by default that are not supported by
> -the latest stable e2fsprogs release.
> -
> -Upstream-Status: Inappropriate [configuration]
> -Signed-off-by: Jonathan Liu 
> -
> -Rebase to 1.43:
> -The upstream has disabled metadata_csum by default
> -this rebase just revert 64bit feature.
> -Signed-off-by: Hongxu Jia 
> 
> - misc/mke2fs.conf.in | 3 ++-
> - 1 file changed, 2 

Re: [OE-core] [PATCH] arch-armv7a.inc: default to Thumb2 instruction set for armv7a and above

2018-05-22 Thread Jef.Driesen
Andre McCurdy wrote:
> Although there may still be specific cases which can benefit from the
> ARM instruction set, the Thumb2 instruction set is generally a better
> default for armv7a class CPUs. Distros such as Debian and Fedora have
> been targeting Thumb2 by default for some time.
>
> Note that setting ARM_INSTRUCTION_SET has no effect unless
> TUNE_FEATURES contains "thumb" (which is controlled by the "t" suffix
> in DEFAULTTUNE, e.g. armv7vehf-neon -vs- armv7vethf-neon, etc) so out
> of tree machine configs may need to update their DEFAULTTUNE to take
> advantage of this change.

I recently ran into some major problems due to thumb vs arm. It turns out glibc 
doesn't support systems with thumb disabled anymore. See this bug report for 
details:

https://sourceware.org/bugzilla/show_bug.cgi?id=23031

This might be useful input for this discussion also.

Jef
Disclaimer

This e-mail and its attachments is intended only for the person(s) or entity to 
which it is addressed. If you receive this e-mail by mistake, please delete 
this e-mail from your system and destroy all copies of it. It may contain 
confidential and/or privileged information. You should not copy it or use it 
for any purpose nor disclose its contents to any person unless allowed by a 
written document between the sender and the addressee.
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] e2fsprogs: 1.43.8 -> 1.44.2

2018-05-22 Thread Andrej Valek
License-Update: The checksum of LIC_FILES_CHKSUM has been changed due to
 changed FSF address. The content of NOTICE has no change.

Signed-off-by: Andrej Valek 
---
 meta/recipes-devtools/e2fsprogs/e2fsprogs.inc  |  2 +-
 ...ix-build-failure-in-swapfs.c-on-big-endia.patch | 40 --
 ...fs-enable-the-metadata_csum-and-64bit-fea.patch | 36 ---
 .../{e2fsprogs_1.43.8.bb => e2fsprogs_1.44.2.bb}   |  4 +--
 4 files changed, 2 insertions(+), 80 deletions(-)
 delete mode 100644 
meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
 delete mode 100644 
meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
 rename meta/recipes-devtools/e2fsprogs/{e2fsprogs_1.43.8.bb => 
e2fsprogs_1.44.2.bb} (95%)

diff --git a/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc 
b/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
index 81d90a134c..a54a11c354 100644
--- a/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
+++ b/meta/recipes-devtools/e2fsprogs/e2fsprogs.inc
@@ -9,7 +9,7 @@ LICENSE_e2fsprogs-mke2fs = "GPLv2"
 LICENSE_e2fsprogs-fsck = "GPLv2"
 LICENSE_e2fsprogs-tune2fs = "GPLv2"
 LICENSE_e2fsprogs-badblocks = "GPLv2"
-LIC_FILES_CHKSUM = "file://NOTICE;md5=b48f21d765b875bd10400975d12c1ca2 \
+LIC_FILES_CHKSUM = "file://NOTICE;md5=d50be0580c0b0a7fbc7a4830bbe6c12b \
   
file://lib/ext2fs/ext2fs.h;beginline=1;endline=9;md5=596a8dedcb4e731c6b21c7a46fba6bef
 \
   
file://lib/e2p/e2p.h;beginline=1;endline=7;md5=8a74ade8f9d65095d70ef2d4bf48e36a 
\
   
file://lib/uuid/uuid.h.in;beginline=1;endline=32;md5=dbb8079e114a5f841934b99e59c8820a
 \
diff --git 
a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
 
b/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
deleted file mode 100644
index ff1284edc4..00
--- 
a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-build-failure-in-swapfs.c-on-big-endia.patch
+++ /dev/null
@@ -1,40 +0,0 @@
-From ae6603206176b771c9ed1f82569f68fc1257f014 Mon Sep 17 00:00:00 2001
-From: Theodore Ts'o 
-Date: Wed, 3 Jan 2018 01:32:02 -0500
-Subject: [PATCH] libext2fs: fix build failure in swapfs.c on big-endian
- systems
-
-Addresses-Debian-Bug: #886119
-
-Reported-by: James Clarke 
-Signed-off-by: Theodore Ts'o 

-Upstream-Status: Backport
-
- lib/ext2fs/swapfs.c | 3 ++-
- 1 file changed, 2 insertions(+), 1 deletion(-)
-
-diff --git a/lib/ext2fs/swapfs.c b/lib/ext2fs/swapfs.c
-index 6e5cff97..b13dc422 100644
 a/lib/ext2fs/swapfs.c
-+++ b/lib/ext2fs/swapfs.c
-@@ -19,6 +19,7 @@
- 
- #include "ext2_fs.h"
- #include "ext2fs.h"
-+#include "ext2fsP.h"
- #include 
- 
- #ifdef WORDS_BIGENDIAN
-@@ -358,7 +359,7 @@ void ext2fs_swap_inode_full(ext2_filsys fs, struct 
ext2_inode_large *t,
-   if (inode_includes(inode_size, i_projid))
- t->i_projid = ext2fs_swab16(f->i_projid);
-   /* catch new static fields added after i_projid */
--  EXT2FS_BUILD_BUG_ON(sizeof(ext2_inode_large) != 160);
-+  EXT2FS_BUILD_BUG_ON(sizeof(struct ext2_inode_large) != 160);
- 
-   i = sizeof(struct ext2_inode) + extra_isize + sizeof(__u32);
-   if (bufsize < (int) i)
--- 
-2.16.1
-
diff --git 
a/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
 
b/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
deleted file mode 100644
index 38e1bdb3d8..00
--- 
a/meta/recipes-devtools/e2fsprogs/e2fsprogs/Revert-mke2fs-enable-the-metadata_csum-and-64bit-fea.patch
+++ /dev/null
@@ -1,36 +0,0 @@
-From 0a392baf1874964651115d9f77b0daa6851d1daa Mon Sep 17 00:00:00 2001
-From: Jonathan Liu 
-Date: Tue, 1 Mar 2016 14:28:01 +1100
-Subject: [PATCH] Revert "mke2fs: enable the metadata_csum and 64bit features
- by default"
-
-This reverts commit cd27af3ecb83e8fd1e3eaa14994284a1818c7c15 as we
-don't want to enable features by default that are not supported by
-the latest stable e2fsprogs release.
-
-Upstream-Status: Inappropriate [configuration]
-Signed-off-by: Jonathan Liu 
-
-Rebase to 1.43:
-The upstream has disabled metadata_csum by default
-this rebase just revert 64bit feature.
-Signed-off-by: Hongxu Jia 

- misc/mke2fs.conf.in | 3 ++-
- 1 file changed, 2 insertions(+), 1 deletion(-)
-
-Index: git/misc/mke2fs.conf.in
-===
 git.orig/misc/mke2fs.conf.in
-+++ git/misc/mke2fs.conf.in
-@@ -11,8 +11,9 @@
-   features = has_journal
-   }
-   ext4 = {
--  features = 
has_journal,extent,huge_file,flex_bg,uninit_bg,64bit,dir_nlink,extra_isize
-+  features = 

Re: [OE-core] [PATCH 3/4] mtd-utils: upgrade 2.0.1 -> 2.0.2

2018-05-22 Thread Andrea Adami
On Tue, May 22, 2018 at 1:17 AM, Denys Dmytriyenko  wrote:
> From: Denys Dmytriyenko 
>
> Signed-off-by: Denys Dmytriyenko 
> ---
>  meta/recipes-devtools/mtd/mtd-utils_git.bb | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-devtools/mtd/mtd-utils_git.bb 
> b/meta/recipes-devtools/mtd/mtd-utils_git.bb
> index a0e1cf7..97a3e0c 100644
> --- a/meta/recipes-devtools/mtd/mtd-utils_git.bb
> +++ b/meta/recipes-devtools/mtd/mtd-utils_git.bb
> @@ -10,9 +10,9 @@ inherit autotools pkgconfig update-alternatives
>  DEPENDS = "zlib lzo e2fsprogs util-linux"
>  RDEPENDS_mtd-utils-tests += "bash"
>
> -PV = "2.0.1+${SRCPV}"
> +PV = "2.0.2+${SRCPV}"
>
> -SRCREV = "9c6173559f95e939e66efb2ec3193d6f3618cf69"
> +SRCREV = "bc63d36e39f389c8c17f6a8e9db47f2acc884659"
>  SRC_URI = "git://git.infradead.org/mtd-utils.git \
> file://add-exclusion-to-mkfs-jffs2-git-2.patch \

Hi Denys,

I am just wondering about this patch:we carry it since 2009  [1] and
is still marked as pending.
Why was it necessary to ignore part of the filesystem? Can you remember?

Just curious...

Cheers
Andrea

[1] Denys Dmytriyenko 2009-03-17
http://cgit.openembedded.org/openembedded/log/recipes/mtd/mtd-utils/add-exclusion-to-mkfs-jffs2-git-2.patch


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