[OE-core] [PATCH] Revert "mdadm: fix gcc8 maybe-uninitialized/format-overflow warning"

2019-04-06 Thread Khem Raj
This reverts commit 630281663893cdcfa9c4323b717b415d87d5510f.

Signed-off-by: Khem Raj 
---
 ...-uninitialized-format-overflow-warni.patch | 60 ---
 meta/recipes-extended/mdadm/mdadm_4.1.bb  |  1 -
 2 files changed, 61 deletions(-)
 delete mode 100644 
meta/recipes-extended/mdadm/files/0001-mdadm-gcc8-maybe-uninitialized-format-overflow-warni.patch

diff --git 
a/meta/recipes-extended/mdadm/files/0001-mdadm-gcc8-maybe-uninitialized-format-overflow-warni.patch
 
b/meta/recipes-extended/mdadm/files/0001-mdadm-gcc8-maybe-uninitialized-format-overflow-warni.patch
deleted file mode 100644
index 237f83a87e..00
--- 
a/meta/recipes-extended/mdadm/files/0001-mdadm-gcc8-maybe-uninitialized-format-overflow-warni.patch
+++ /dev/null
@@ -1,60 +0,0 @@
-From bf457a83834932ba06de3528b8779a023e73fa7b Mon Sep 17 00:00:00 2001
-From: Changqing Li 
-Date: Tue, 12 Mar 2019 16:17:29 +0800
-Subject: [PATCH] mdadm: gcc8 maybe-uninitialized/format-overflow warning
-
-while compiled with -Werror=maybe-uninitialized/-Werror=format-overflow=,
-it failed
-
-[snip]
-| Incremental.c: In function 'Incremental_container':
-| Incremental.c:1593:3: error: 'mdfd' may be used uninitialized in this 
function [-Werror=maybe-uninitialized]
-| close(mdfd);
-| ^~~
-
-[snip]
-super-intel.c: In function 'apply_takeover_update':
-| super-intel.c:9615:15: error: '%d' directive writing between 1 and 11 bytes 
into a region of size 7 [-Werror=format-overflow=]
-| " MISSING_%d", du->index);
-| ^~
-
-Upstream-Status: Submitted [https://github.com/neilbrown/mdadm/pull/36]
-
-Signed-off-by: Changqing Li 

- Incremental.c | 2 +-
- super-intel.c | 4 ++--
- 2 files changed, 3 insertions(+), 3 deletions(-)
-
-diff --git a/Incremental.c b/Incremental.c
-index a4ff7d4..b667868 100644
 a/Incremental.c
-+++ b/Incremental.c
-@@ -1500,7 +1500,7 @@ static int Incremental_container(struct supertype *st, 
char *devname,
-   return 0;
-   }
-   for (ra = list ; ra ; ra = ra->next) {
--  int mdfd;
-+  int mdfd = 0;
-   char chosen_name[1024];
-   struct map_ent *mp;
-   struct mddev_ident *match = NULL;
-diff --git a/super-intel.c b/super-intel.c
-index 10d7218..c3741ea 100644
 a/super-intel.c
-+++ b/super-intel.c
-@@ -9612,9 +9612,9 @@ static int apply_takeover_update(struct 
imsm_update_takeover *u,
-   du->major = 0;
-   du->index = (i * 2) + 1;
-   sprintf((char *)du->disk.serial,
--  " MISSING_%d", du->index);
-+  " MISSING_%hu", du->index);
-   sprintf((char *)du->serial,
--  "MISSING_%d", du->index);
-+  "MISSING_%hu", du->index);
-   du->next = super->missing;
-   super->missing = du;
-   }
--- 
-2.7.4
-
diff --git a/meta/recipes-extended/mdadm/mdadm_4.1.bb 
b/meta/recipes-extended/mdadm/mdadm_4.1.bb
index 9862a38dc3..947706ff5f 100644
--- a/meta/recipes-extended/mdadm/mdadm_4.1.bb
+++ b/meta/recipes-extended/mdadm/mdadm_4.1.bb
@@ -19,7 +19,6 @@ SRC_URI = 
"${KERNELORG_MIRROR}/linux/utils/raid/mdadm/${BPN}-${PV}.tar.xz \
file://0001-fix-gcc-8-format-truncation-warning.patch \
   file://mdadm.init \
   file://mdmonitor.service \
-   
file://0001-mdadm-gcc8-maybe-uninitialized-format-overflow-warni.patch \
"
 SRC_URI[md5sum] = "51bf3651bd73a06c413a2f964f299598"
 SRC_URI[sha256sum] = 
"ab7688842908d3583a704d491956f31324c3a5fc9f6a04653cb75d19f1934f4a"
-- 
2.21.0

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


Re: [OE-core] [PATCH] gstreamer1.0-python: Remove --enable-introspection

2019-04-06 Thread Khem Raj
On Sat, Apr 6, 2019 at 11:27 AM Andreas Müller  wrote:
>
> On Sat, Apr 6, 2019 at 6:20 PM Khem Raj  wrote:
> >
> > This option is not available and recently it is reported as error
> >
> > do_configure: QA Issue: gstreamer1.0-python: configur
> > e was passed unrecognised options: --enable-introspection 
> > [unknown-configure-option]
> >
> Send out a patch already addressing [1] but honestly I parts of this
> parts more: remove option instead of ignoring.
>
> Cann you add:
>
> -inherit autotools pkgconfig distutils3-base upstream-version-is-even
> gobject-introspection
> +inherit autotools pkgconfig distutils3-base upstream-version-is-even
> gobject-introspection distro_features_check
> +# gobject-introspection is mandatory and cannot be configured
> +REQUIRED_DISTRO_FEATURES = "gobject-introspection-data"
>
> or shall I send V3?

seems better please do

>
> [1] 
> http://lists.openembedded.org/pipermail/openembedded-core/2019-April/280958.html
>
> Andreas
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] gstreamer1.0-python: Remove --enable-introspection

2019-04-06 Thread Andreas Müller
On Sat, Apr 6, 2019 at 8:27 PM Andreas Müller  wrote:
AArgh recent comment went out way too early.

This patch won't work if gobject-introspection-data is not in distro
features. Then there would be unknown configure option
--disable-introspection.

So what do others think: ignore unknown options (then my patch is OK) or remove?

Andreas
>
> On Sat, Apr 6, 2019 at 6:20 PM Khem Raj  wrote:
> >
> > This option is not available and recently it is reported as error
> >
> > do_configure: QA Issue: gstreamer1.0-python: configur
> > e was passed unrecognised options: --enable-introspection 
> > [unknown-configure-option]
> >
> Send out a patch already addressing [1] but honestly I parts of this
> parts more: remove option instead of ignoring.
>
> Cann you add:
>
> -inherit autotools pkgconfig distutils3-base upstream-version-is-even
> gobject-introspection
> +inherit autotools pkgconfig distutils3-base upstream-version-is-even
> gobject-introspection distro_features_check
> +# gobject-introspection is mandatory and cannot be configured
> +REQUIRED_DISTRO_FEATURES = "gobject-introspection-data"
>
> or shall I send V3?
>
> [1] 
> http://lists.openembedded.org/pipermail/openembedded-core/2019-April/280958.html
>
> Andreas
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] makedepend: update to 1.0.6

2019-04-06 Thread Oleksandr Kravchuk
Signed-off-by: Oleksandr Kravchuk 
---
 .../xorg-util/{makedepend_1.0.5.bb => makedepend_1.0.6.bb}| 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta/recipes-graphics/xorg-util/{makedepend_1.0.5.bb => 
makedepend_1.0.6.bb} (82%)

diff --git a/meta/recipes-graphics/xorg-util/makedepend_1.0.5.bb 
b/meta/recipes-graphics/xorg-util/makedepend_1.0.6.bb
similarity index 82%
rename from meta/recipes-graphics/xorg-util/makedepend_1.0.5.bb
rename to meta/recipes-graphics/xorg-util/makedepend_1.0.6.bb
index 727ab54965..2760edd173 100644
--- a/meta/recipes-graphics/xorg-util/makedepend_1.0.5.bb
+++ b/meta/recipes-graphics/xorg-util/makedepend_1.0.6.bb
@@ -17,5 +17,5 @@ BBCLASSEXTEND = "native"
 
 LIC_FILES_CHKSUM = "file://COPYING;md5=43a6eda34b48ee821b3b66f4f753ce4f"
 
-SRC_URI[md5sum] = "efb2d7c7e22840947863efaedc175747"
-SRC_URI[sha256sum] = 
"503903d41fb5badb73cb70d7b3740c8b30fe1cc68c504d3b6a85e6644c4e5004"
+SRC_URI[md5sum] = "8a583055c84914060c35b6c2f963fc07"
+SRC_URI[sha256sum] = 
"845f6708fc850bf53f5b1d0fb4352c4feab3949f140b26f71b22faba354c3365"
-- 
2.17.1

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


Re: [OE-core] [PATCH] gstreamer1.0-python: Remove --enable-introspection

2019-04-06 Thread Andreas Müller
On Sat, Apr 6, 2019 at 6:20 PM Khem Raj  wrote:
>
> This option is not available and recently it is reported as error
>
> do_configure: QA Issue: gstreamer1.0-python: configur
> e was passed unrecognised options: --enable-introspection 
> [unknown-configure-option]
>
Send out a patch already addressing [1] but honestly I parts of this
parts more: remove option instead of ignoring.

Cann you add:

-inherit autotools pkgconfig distutils3-base upstream-version-is-even
gobject-introspection
+inherit autotools pkgconfig distutils3-base upstream-version-is-even
gobject-introspection distro_features_check
+# gobject-introspection is mandatory and cannot be configured
+REQUIRED_DISTRO_FEATURES = "gobject-introspection-data"

or shall I send V3?

[1] 
http://lists.openembedded.org/pipermail/openembedded-core/2019-April/280958.html

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


Re: [OE-core] Worries kernels supported

2019-04-06 Thread Manjukumar Harthikote Matha



> -Original Message-
> From: openembedded-core-boun...@lists.openembedded.org
> [mailto:openembedded-core-boun...@lists.openembedded.org] On Behalf
> Of Khem Raj
> Sent: Friday, April 5, 2019 2:54 PM
> To: akuster808 
> Cc: yo...@yoctoproject.org; Patches and discussions about the oe-core layer
> 
> Subject: Re: [OE-core] Worries kernels supported
> 
> On Thu, Apr 4, 2019 at 2:29 PM akuster808  wrote:
> >
> > Hello,
> >
> > I noticed there are 3 kernels in Warrior. 4.18, 4.19 an 5.0. Do we
> > really need 4.18?
> > I see its the default version for poky-tiny. 4.18 is EOL but
> > maintained by Windriver.
> 
> I would prefer it to be 4.19 since thats at least LTS

I agree with Khem, keep 4.19 since it is LTS

> 
> >
> > 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


[OE-core] [PATCH] gstreamer1.0-python: Remove --enable-introspection

2019-04-06 Thread Khem Raj
This option is not available and recently it is reported as error

do_configure: QA Issue: gstreamer1.0-python: configur
e was passed unrecognised options: --enable-introspection 
[unknown-configure-option]

Signed-off-by: Khem Raj 
---
 meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.14.4.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.14.4.bb 
b/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.14.4.bb
index 8d45ca2668..3358352128 100644
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.14.4.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-python_1.14.4.bb
@@ -18,6 +18,7 @@ S = "${WORKDIR}/${PNREAL}-${PV}"
 
 inherit autotools pkgconfig distutils3-base upstream-version-is-even 
gobject-introspection
 
+EXTRA_OECONF_remove = "--enable-introspection"
 do_install_append() {
 # gstpythonplugin hardcodes the location of the libpython from the build
 # workspace and then fails at runtime. We can override it using
-- 
2.21.0

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


[OE-core] [PATCH v2] fontconfig: update to 2.13.1

2019-04-06 Thread Oleksandr Kravchuk
Removed patches were upstreammed.

License position in fccache.c was changed since code segment was moved
in the file.

Package requires uuid, therefore dependency on util-linux was added.

inherit gettext was added to fix:
...%s required but not in DEPENDS for file %s. Missing inherit gettext?

Signed-off-by: Oleksandr Kravchuk 
---
 c-Fix-define-for-HAVE_POSIX_FADVISE.patch | 33 ---
 ...cxml.c-avoid-double-free-of-filename.patch | 28 
 .../fontconfig/revert-static-pkgconfig.patch  | 12 +++
 ...tconfig_2.12.6.bb => fontconfig_2.13.1.bb} | 13 
 4 files changed, 12 insertions(+), 74 deletions(-)
 delete mode 100644 
meta/recipes-graphics/fontconfig/fontconfig/0001-src-fccache.c-Fix-define-for-HAVE_POSIX_FADVISE.patch
 delete mode 100644 
meta/recipes-graphics/fontconfig/fontconfig/0001-src-fcxml.c-avoid-double-free-of-filename.patch
 rename meta/recipes-graphics/fontconfig/{fontconfig_2.12.6.bb => 
fontconfig_2.13.1.bb} (82%)

diff --git 
a/meta/recipes-graphics/fontconfig/fontconfig/0001-src-fccache.c-Fix-define-for-HAVE_POSIX_FADVISE.patch
 
b/meta/recipes-graphics/fontconfig/fontconfig/0001-src-fccache.c-Fix-define-for-HAVE_POSIX_FADVISE.patch
deleted file mode 100644
index d9bce212f6..00
--- 
a/meta/recipes-graphics/fontconfig/fontconfig/0001-src-fccache.c-Fix-define-for-HAVE_POSIX_FADVISE.patch
+++ /dev/null
@@ -1,33 +0,0 @@
-From ab9522177a8396a51812fdbebb6387df451a8499 Mon Sep 17 00:00:00 2001
-From: Robert Yang 
-Date: Mon, 24 Dec 2018 11:03:58 +0800
-Subject: [PATCH] src/fccache.c: Fix define for HAVE_POSIX_FADVISE
-
-Otherwise, there would be build errors in the following 2 cases:
-* define HAVE_POSIX_FADVISE
-Or:
-* undef HAVE_POSIX_FADVISE
-
-Upstream-Status: Backport 
[https://gitlab.freedesktop.org/fontconfig/fontconfig/commit/586e35450e9ca7c1dc647ceb9d75ac8ed08c5c16]
-
-Signed-off-by: Robert Yang 

- fccache.c |2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/src/fccache.c b/src/fccache.c
-index 6f3c68a..85cc4b4 100644
 a/src/fccache.c
-+++ b/src/fccache.c
-@@ -700,7 +700,7 @@ FcDirCacheMapFd (FcConfig *config, int fd, struct stat 
*fd_stat, struct stat *di
- {
- #if defined(HAVE_MMAP) || defined(__CYGWIN__)
-   cache = mmap (0, fd_stat->st_size, PROT_READ, MAP_SHARED, fd, 0);
--#if (HAVE_POSIX_FADVISE) && defined(POSIX_FADV_WILLNEED)
-+#if defined(HAVE_POSIX_FADVISE) && defined(POSIX_FADV_WILLNEED)
-   posix_fadvise (fd, 0, fd_stat->st_size, POSIX_FADV_WILLNEED);
- #endif
-   if (cache == MAP_FAILED)
--- 
-2.7.4
-
diff --git 
a/meta/recipes-graphics/fontconfig/fontconfig/0001-src-fcxml.c-avoid-double-free-of-filename.patch
 
b/meta/recipes-graphics/fontconfig/fontconfig/0001-src-fcxml.c-avoid-double-free-of-filename.patch
deleted file mode 100644
index 3ca9fde967..00
--- 
a/meta/recipes-graphics/fontconfig/fontconfig/0001-src-fcxml.c-avoid-double-free-of-filename.patch
+++ /dev/null
@@ -1,28 +0,0 @@
-From a163c81ac430cdc292add200f3e6a0c048be4d7e Mon Sep 17 00:00:00 2001
-From: Alexander Kanavin 
-Date: Wed, 11 Oct 2017 17:40:09 +0300
-Subject: [PATCH] src/fcxml.c: avoid double free() of filename
-
-It's also freed after bail1, so no need to do it here.
-
-Upstream-Status: Submitted 
[https://bugs.freedesktop.org/show_bug.cgi?id=103221]
-Signed-off-by: Alexander Kanavin 

- src/fcxml.c | 1 -
- 1 file changed, 1 deletion(-)
-
-diff --git a/src/fcxml.c b/src/fcxml.c
-index 19ce96a..edb6054 100644
 a/src/fcxml.c
-+++ b/src/fcxml.c
-@@ -3398,7 +3398,6 @@ FcConfigParseAndLoad (FcConfig   *config,
- 
- fd = FcOpen ((char *) filename, O_RDONLY);
- if (fd == -1) {
--  FcStrFree (filename);
-   goto bail1;
- }
- 
--- 
-2.14.1
-
diff --git 
a/meta/recipes-graphics/fontconfig/fontconfig/revert-static-pkgconfig.patch 
b/meta/recipes-graphics/fontconfig/fontconfig/revert-static-pkgconfig.patch
index fca1e65946..30d6423037 100644
--- a/meta/recipes-graphics/fontconfig/fontconfig/revert-static-pkgconfig.patch
+++ b/meta/recipes-graphics/fontconfig/fontconfig/revert-static-pkgconfig.patch
@@ -8,13 +8,13 @@ gently refuses and rightly so.
 Upstream-Status: Inappropriate [OE-Specific]
 Signed-off-by: Khem Raj 
 
-Index: fontconfig-2.11.93/fontconfig.pc.in
-===
 fontconfig-2.11.93.orig/fontconfig.pc.in
-+++ fontconfig-2.11.93/fontconfig.pc.in
+diff --git a/fontconfig.pc.in b/fontconfig.pc.in
+index 61b35fb..5bc9bab 100644
+--- a/fontconfig.pc.in
 b/fontconfig.pc.in
 @@ -15,4 +15,4 @@ Requires: @PKGCONFIG_REQUIRES@
  Requires.private: @PKGCONFIG_REQUIRES_PRIVATELY@
  Libs: -L${libdir} -lfontconfig
- Libs.private: @EXPAT_LIBS@ @FREETYPE_LIBS@ @ICONV_LIBS@ @LIBXML2_LIBS@
--Cflags: -I${includedir} @EXPAT_CFLAGS@ @FREETYPE_CFLAGS@ @ICONV_CFLAGS@ 
@LIBXML2_CFLAGS@
+ Libs.private: @ICONV_LIBS@ @PKG_EXPAT_LIBS@
+-Cflags: -I${includedir} @ICONV_CFLAGS@ @PKG_EXPAT_CFLAGS@
 +Cflags: -I${includedir}
diff --git a