[OE-core] [poky][meta][sumo][PATCH] run-ptest: fix testcase test-names failure

[Aditya Tayade]

Start the dbus daemon first before executing test-names
using dbus-launch, this will set DBUS_SESSION_BUS_ADDRESS.

ERROR:Failed to open connection to system bus: Using X11 for
dbus-daemon autolaunch was disabled at compile time,
set your DBUS_SESSION_BUS_ADDRESS instead

Upstream-Status: Pending

Signed-off-by: Aditya Tayade 
---
 meta/recipes-core/dbus/dbus/run-ptest | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/dbus/dbus/run-ptest 
b/meta/recipes-core/dbus/dbus/run-ptest
index 8a8970e..444ef4a 100755
--- a/meta/recipes-core/dbus/dbus/run-ptest
+++ b/meta/recipes-core/dbus/dbus/run-ptest
@@ -13,6 +13,7 @@ output() {
 export DBUS_TEST_HOMEDIR=./test
 export XDG_RUNTIME_DIR=./test
 export LD_LIBRARY_PATH=/usr/lib/dbus-test/ptest/test/.libs
+export $(dbus-launch)

 files=`ls test/test-*`

--
2.7.4

This message contains information that may be privileged or confidential and is 
the property of the KPIT Technologies Ltd. It is intended only for the person 
to whom it is addressed. If you are not the intended recipient, you are not 
authorized to read, print, retain copy, disseminate, distribute, or use this 
message or any part thereof. If you receive this message in error, please 
notify the sender immediately and delete all copies of this message. KPIT 
Technologies Ltd. does not accept any liability for virus infected mails.
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] [oe][meta-oe][PATCH] tslib: upgrade to 1.17

[Hong Liu]

Upgrade tslib from 1.16 to 1.17

Signed-off-by: Hong Liu 
---
 .../recipes-graphics/tslib/{tslib_1.16.bb => tslib_1.17.bb}   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
 rename meta-oe/recipes-graphics/tslib/{tslib_1.16.bb => tslib_1.17.bb} (96%)

diff --git a/meta-oe/recipes-graphics/tslib/tslib_1.16.bb 
b/meta-oe/recipes-graphics/tslib/tslib_1.17.bb
similarity index 96%
rename from meta-oe/recipes-graphics/tslib/tslib_1.16.bb
rename to meta-oe/recipes-graphics/tslib/tslib_1.17.bb
index 8d23bd5eb..a095e69c6 100644
--- a/meta-oe/recipes-graphics/tslib/tslib_1.16.bb
+++ b/meta-oe/recipes-graphics/tslib/tslib_1.17.bb
@@ -17,8 +17,8 @@ SRC_URI = 
"https://github.com/kergoth/tslib/releases/download/${PV}/tslib-${PV}.
file://ts.conf \
file://tslib.sh \
 "
-SRC_URI[md5sum] = "254a022fc8aa89d164840fe8869c2441"
-SRC_URI[sha256sum] = 
"c9a54651337a701a66b074c603c313225579995a7910e519bbc7b1dcdab9a755"
+SRC_URI[md5sum] = "048c4c14aea95ec09d61faa340a2c6c3"
+SRC_URI[sha256sum] = 
"9fdd33e66976bf9cc799be33824ba66672f892826e9b22de12b14df15cb0fddc"
 
 UPSTREAM_CHECK_URI = "https://github.com/kergoth/tslib/releases;
 
-- 
2.20.1



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

[OE-core] [PATCH] musl: Update for few bugfixes

[Khem Raj]

We have seen/reported few dlopen related fixes which has been fixed
upstream, therefore upgrade

all set of changes

https://git.musl-libc.org/cgit/musl/log/?qt=range=6516282d2adfad2c7e66d854cde3357120c75dbd..43e7efb46555f13a556d92944ac05c19b8929b60

Signed-off-by: Khem Raj 
---
 .../musl/musl/out_of_bounds_read.patch| 20 ---
 meta/recipes-core/musl/musl_git.bb|  3 +--
 2 files changed, 1 insertion(+), 22 deletions(-)
 delete mode 100644 meta/recipes-core/musl/musl/out_of_bounds_read.patch

diff --git a/meta/recipes-core/musl/musl/out_of_bounds_read.patch 
b/meta/recipes-core/musl/musl/out_of_bounds_read.patch
deleted file mode 100644
index a681cc284e..00
--- a/meta/recipes-core/musl/musl/out_of_bounds_read.patch
+++ /dev/null
@@ -1,20 +0,0 @@
-Fix out of bounds read
-
-self->dtv hasn't been extended yet
-
-Upstream-Status: Pending
-Signed-off-by: Khem Raj 
-
-diff --git a/ldso/dynlink.c b/ldso/dynlink.c
-index e2c3259f..b23ea0df 100644
 a/ldso/dynlink.c
-+++ b/ldso/dynlink.c
-@@ -1374,7 +1376,7 @@ static void install_new_tls(void)
-   }
-   /* Install new dtls into the enlarged, uninstalled dtv copies. */
-   for (p=head; ; p=p->next) {
--  if (!p->tls_id || self->dtv[p->tls_id]) continue;
-+  if (p->tls_id <= old_cnt) continue;
-   unsigned char *mem = p->new_tls;
-   for (j=0; jhttp://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/4] Improve logger setup in devtool/recipetool/scriptutils

[Paul Eggleton]

On Tuesday, 5 March 2019 10:36:07 AM NZDT Christopher Larson wrote:
> From: Christopher Larson 
> 
> This improves logger setup in devtool and recipetool to send ERRORs to 
stderr,
> the way bitbake itself does, and also sends WARNINGs there, as those are
> essentially out of band as well. This makes it easier to script with devtool
> and recipetool, as any *real* subcommand output can be more easily captured
> without having to manually filter out the cruft.
> 
> The following changes since commit b11725db2d5549dc45d8ae36fbf94a5c8e342d69:
> 
>   xserver-xorg: Fix build errors with clang (2019-03-04 14:26:41 +)
> 
> are available in the Git repository at:
> 
>   g...@github.com:kergoth/openembedded-core scriptutils-log-improvements
> 
> for you to fetch changes up to c4f37568236d2e3d8c40d57399d0b275ad5b7c8b:
> 
>   oe.scriptutils: also send WARNING to stderr (2019-03-04 21:31:00 +)
> 
> 
> Christopher Larson (4):
>   oe.scriptutils: enable color in a more flexible way
>   oe.scriptutils: add logger_setup_filters
>   recipetool, devtool: set up the logging filters
>   oe.scriptutils: also send WARNING to stderr
> 

All of these look good to me.

Acked-by: Paul Eggleton 

Cheers
Paul


-- 

Paul Eggleton
Intel Open Source Technology Centre


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

[OE-core] [PATCH v2 1/1] send-error-report: Add --no-ssl to use http protocol

[Robert Yang]

The script use https protocol by default, but the error-report-web server's
https connection may not work (e.g., doesn't work with python 2.7.6), so add an
option --no-ssl to make it use http.

Signed-off-by: Robert Yang 
---
 scripts/send-error-report | 11 ---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/scripts/send-error-report b/scripts/send-error-report
index 3528cf9..0ed7cc9 100755
--- a/scripts/send-error-report
+++ b/scripts/send-error-report
@@ -62,7 +62,7 @@ def edit_content(json_file_path):
 
 def prepare_data(args):
 # attempt to get the max_log_size from the server's settings
-max_log_size = getPayloadLimit("https://"+args.server+"/ClientPost/JSON;)
+max_log_size = 
getPayloadLimit(args.protocol+args.server+"/ClientPost/JSON")
 
 if not os.path.isfile(args.error_file):
 log.error("No data file found.")
@@ -132,9 +132,9 @@ def send_data(data, args):
 headers={'Content-type': 'application/json', 'User-Agent': 
"send-error-report/"+version}
 
 if args.json:
-url = "https://"+args.server+"/ClientPost/JSON/;
+url = args.protocol+args.server+"/ClientPost/JSON/"
 else:
-url = "https://"+args.server+"/ClientPost/;
+url = args.protocol+args.server+"/ClientPost/"
 
 req = urllib.request.Request(url, data=data, headers=headers)
 try:
@@ -187,6 +187,11 @@ if __name__ == '__main__':
help="Return the result in json format, silences 
all other output",
action="store_true")
 
+arg_parse.add_argument("--no-ssl",
+   help="Use http instead of https protocol",
+   dest="protocol",
+   action="store_const", const="http://;, 
default="https://;)
+
 
 
 args = arg_parse.parse_args()
-- 
2.7.4

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

[OE-core] [PATCH v2 0/1] send-error-report: Add --no-ssl to use http protocol

[Robert Yang]

* V2
  Use store_const as Ross suggested

* V1
  Initial version

The following changes since commit e37a1ecc292b684daa49f2da2e19e0aa975f0959:

  squashfs-tools: add PACKAGECONFIGs for all the tunables (2019-03-04 17:37:23 
+)

are available in the git repository at:

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

Robert Yang (1):
  send-error-report: Add --no-ssl to use http protocol

 scripts/send-error-report | 11 ---
 1 file changed, 8 insertions(+), 3 deletions(-)

-- 
2.7.4

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

[OE-core] [PATCH] Add source package to ${PACKAGES}

[Joshua Watt]

Permanently adds the -src source package to ${PACKAGES} instead of
relying on creating it dynamically at packaging time. The source package
is now defined in bitbake.conf, just like -dev and -dbg packages.

For compatibility, the -src package is still added dynamically if it was
missing, since some recipes have a tendency to completely override
PACKAGES and do so without manually adding back the -src package.

This allows RDEPENDS on the -src packages, which wasn't previously
possible.

[YOCTO #13203]

Signed-off-by: Joshua Watt 
---
 meta/classes/package.bbclass | 28 +++-
 meta/conf/bitbake.conf   | 12 +++-
 2 files changed, 26 insertions(+), 14 deletions(-)

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index 749c7d9ea14..4c0a8595363 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -1128,7 +1128,7 @@ python populate_packages () {
 workdir = d.getVar('WORKDIR')
 outdir = d.getVar('DEPLOY_DIR')
 dvar = d.getVar('PKGD')
-packages = d.getVar('PACKAGES')
+packages = d.getVar('PACKAGES').split()
 pn = d.getVar('PN')
 
 bb.utils.mkdirhier(outdir)
@@ -1138,32 +1138,34 @@ python populate_packages () {
 
 split_source_package = (d.getVar('PACKAGE_DEBUG_SPLIT_STYLE') == 
'debug-with-srcpkg')
 
-# If debug-with-srcpkg mode is enabled then the src package is added
-# into the package list and the source directory as its main content
+# If debug-with-srcpkg mode is enabled then add the source package if it
+# doesn't exist and add the source file contents to the source package.
 if split_source_package:
 src_package_name = ('%s-src' % d.getVar('PN'))
-packages += (' ' + src_package_name)
+if not src_package_name in packages:
+packages.append(src_package_name)
 d.setVar('FILES_%s' % src_package_name, '/usr/src/debug')
 
 # Sanity check PACKAGES for duplicates
 # Sanity should be moved to sanity.bbclass once we have the infrastructure
 package_dict = {}
 
-for i, pkg in enumerate(packages.split()):
+for i, pkg in enumerate(packages):
 if pkg in package_dict:
 msg = "%s is listed in PACKAGES multiple times, this leads to 
packaging errors." % pkg
 package_qa_handle_error("packages-list", msg, d)
-# If debug-with-srcpkg mode is enabled then the src package will have
-# priority over dbg package when assigning the files.
-# This allows src package to include source files and remove them from 
dbg.
-elif split_source_package and pkg.endswith("-src"):
+# Ensure the source package gets the chance to pick up the source files
+# before the debug package by ordering it first in PACKAGES. Whether it
+# actually picks up any source files is controlled by
+# PACKAGE_DEBUG_SPLIT_STYLE.
+elif pkg.endswith("-src"):
 package_dict[pkg] = (10, i)
 elif autodebug and pkg.endswith("-dbg"):
 package_dict[pkg] = (30, i)
 else:
 package_dict[pkg] = (50, i)
-package_list = sorted(package_dict.keys(), key=package_dict.get)
-d.setVar('PACKAGES', ' '.join(package_list))
+packages = sorted(package_dict.keys(), key=package_dict.get)
+d.setVar('PACKAGES', ' '.join(packages))
 pkgdest = d.getVar('PKGDEST')
 
 seen = []
@@ -1181,7 +1183,7 @@ python populate_packages () {
 if "/.debug/" in path or path.endswith("/.debug"):
 debug.append(path)
 
-for pkg in package_list:
+for pkg in packages:
 root = os.path.join(pkgdest, pkg)
 bb.utils.mkdirhier(root)
 
@@ -1252,7 +1254,7 @@ python populate_packages () {
 
 # Handle LICENSE_EXCLUSION
 package_list = []
-for pkg in packages.split():
+for pkg in packages:
 if d.getVar('LICENSE_EXCLUSION-' + pkg):
 msg = "%s has an incompatible license. Excluding from packaging." 
% pkg
 package_qa_handle_error("incompatible-license", msg, d)
diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index f1540a0acf7..5a4c4bbde94 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -237,6 +237,10 @@ DESCRIPTION ?= "${SUMMARY}."
 #SUMMARY_${PN} ?= "${SUMMARY}"
 #DESCRIPTION_${PN} ?= "${DESCRIPTION}"
 
+SUMMARY_${PN}-src ?= "${SUMMARY} - Source files"
+DESCRIPTION_${PN}-src ?= "${DESCRIPTION}  \
+This package contains sources for debugging purposes."
+
 SUMMARY_${PN}-dbg ?= "${SUMMARY} - Debugging files"
 DESCRIPTION_${PN}-dbg ?= "${DESCRIPTION}  \
 This package contains ELF symbols and related sources for debugging purposes."
@@ -285,7 +289,7 @@ SOLIBSDEV_darwin = ".dylibbroken"
 PACKAGE_DEBUG_SPLIT_STYLE ?= "debug-with-srcpkg"
 
 PACKAGE_BEFORE_PN ?= ""
-PACKAGES = "${PN}-dbg ${PN}-staticdev ${PN}-dev ${PN}-doc ${PN}-locale 
${PACKAGE_BEFORE_PN} ${PN}"
+PACKAGES = "${PN}-src ${PN}-dbg ${PN}-staticdev ${PN}-dev ${PN}-doc 

Re: [OE-core] [thud][PATCH] openssl10: Fix mutliple include assumptions for des.h in opensslconf.h

[Denys Dmytriyenko]

Armin,

Not sure when would be the next batch of thud backports - as discussed 
earlier, I needed this fix to un-break the last set of backported updates 
from last Monday. Please let me know when this can be merged to thud.
Thanks!

-- 
Denys


On Mon, Mar 04, 2019 at 01:11:13PM -0500, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko 
> 
> The fix is heavily based on Khem's previous fix for bn.h/BN_LLONG breakage:
> https://git.openembedded.org/openembedded-core/commit/?id=f787b0bb9b0626ddbf2ac94cb206c76716a3773d
> 
> (From OE-Core rev: 914e1520bf9c45e14bce9993c9131a2c0702b9c9)
> 
> Signed-off-by: Denys Dmytriyenko 
> Cc: Khem Raj 
> Signed-off-by: Richard Purdie 
> Signed-off-by: Denys Dmytriyenko 
> ---
>  .../openssl10/0001-Fix-DES_LONG-breakage.patch | 35 
> ++
>  .../openssl/openssl10_1.0.2q.bb|  1 +
>  2 files changed, 36 insertions(+)
>  create mode 100644 
> meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch
> 
> diff --git 
> a/meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch
>  
> b/meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch
> new file mode 100644
> index 000..7243fb4
> --- /dev/null
> +++ 
> b/meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch
> @@ -0,0 +1,35 @@
> +From 1aec49cc45e7cf5ebc059a77081ac3ea2a5aff7b Mon Sep 17 00:00:00 2001
> +From: Denys Dmytriyenko 
> +Date: Fri, 1 Mar 2019 13:14:56 -0500
> +Subject: [PATCH] Fix DES_LONG breakage
> +
> +Mimic previous BN_LLONG fix by Khem Raj here. Re-use its description:
> +
> +opensslconf.h is defining DES_LONG only when included from des.h which
> +is not robust at all, especially when include guards are used and
> +multiple inclusions of a given header is not allowed. so lets take out
> +the nesting constraint and add OPENSSL_SYS_UEFI constraint instead
> +
> +Upstream-Status: Inappropriate [ fixed differently with OpenSSL 1.1+ ]
> +
> +Signed-off-by: Denys Dmytriyenko 
> +---
> + crypto/opensslconf.h.in | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/crypto/opensslconf.h.in b/crypto/opensslconf.h.in
> +index a10c10f..0147a4d 100644
> +--- a/crypto/opensslconf.h.in
>  b/crypto/opensslconf.h.in
> +@@ -48,7 +48,7 @@
> + #endif
> + #endif
> + 
> +-#if (defined(HEADER_NEW_DES_H) || defined(HEADER_DES_H)) && 
> !defined(DES_LONG)
> ++#if !defined(OPENSSL_SYS_UEFI) && !defined(DES_LONG)
> + /* If this is set to 'unsigned int' on a DEC Alpha, this gives about a
> +  * %20 speed up (longs are 8 bytes, int's are 4). */
> + #ifndef DES_LONG
> +-- 
> +2.7.4
> +
> diff --git a/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb 
> b/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb
> index 355dbdc..54af100 100644
> --- a/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb
> +++ b/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb
> @@ -41,6 +41,7 @@ SRC_URI = 
> "http://www.openssl.org/source/openssl-${PV}.tar.gz \
> file://0001-openssl-force-soft-link-to-avoid-rare-race.patch \
> file://0001-allow-manpages-to-be-disabled.patch \
> file://0001-Fix-BN_LLONG-breakage.patch \
> +   file://0001-Fix-DES_LONG-breakage.patch \
> "
>  
>  SRC_URI_append_class-target = " \
> -- 
> 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

Re: [OE-core] [OE-Core][PATCH v2 3/3] devtool: provide support for devtool menuconfig command.

[Paul Eggleton]

On Saturday, 26 January 2019 8:53:02 AM NZDT Sai Hari Chandana Kalluri wrote:
> +def menuconfig(args, config, basepath, workspace):
> +"""Entry point for the devtool 'menuconfig' subcommand"""
> +
> +rd = "" 
> +kconfigpath = ""
> +pn_src = ""
> +localfilesdir = ""
> +workspace_dir = ""
> +tinfoil = setup_tinfoil(basepath=basepath)
> +try:
> +  rd = parse_recipe(config, tinfoil, args.component, appends=True, 
> filter_workspace=False)
> +  if not rd:
> + return 1
> +
> +  pn =  rd.getVar('PN', True)
> +  if pn not in workspace:
> + raise DevtoolError("Run devtool modify before calling menuconfig 
> for %s" %pn)

Strictly speaking we have check_workspace_recipe() for this. It could be that 
we should extend the message printed by that to be a bit more friendly and 
suggest devtool add/modify/upgrade first, but I'd like to be consistent.

>...
> +  #add check to see if oe_local_files exists or not
> +  localfilesdir = os.path.join(pn_src,'oe-local-files') 
> +  if not os.path.exists(localfilesdir):
> +  bb.utils.mkdirhier(localfilesdir)
> +  #Add gitignore to ensure source tree is clean
> +  gitignorefile = os.path.join(localfilesdir,'.gitignore')
> +  with open(gitignorefile, 'w') as f:
> +  f.write('# Ignore local files, by default. Remove this 
> file if you want to commit the directory to Git\n')
> +  f.write('*\n')

We should already be handling this in devtool-source.bbclass, but I'm guessing 
the hardlinking you're adding bypasses that, so we likely need to handle that 
separately during devtool modify rather than here (preferably without 
duplicating code).

> +parser_menuconfig = subparsers.add_parser('menuconfig',help='allows 
> altering the system component configuration', description='launches the make 
> menuconfig command, allows user to make changes to configuration. creates a 
> config fragment', group='advanced') 
> +parser_menuconfig.add_argument('component', help='compenent to alter 
> config')

To be consistent with other subcommands and a bit more readable this should be 
something like:

parser_menuconfig = subparsers.add_parser('menuconfig', help='Alter 
build-time configuration for a recipe', description='Launches the "make 
menuconfig" command (for recipes where do_menuconfig is available), allowing 
you to make changes to the build-time configuration. Creates a config fragment 
corresponding to changes made.', group='advanced') 
parser_menuconfig.add_argument('recipename', help='Recipe to reconfigure')

Additionally, I would really like to see some oe-selftest tests for this along 
with the changes.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre


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

Re: [OE-core] [OE-Core][PATCH v2 1/3] devtool modify: Update devtool modify to copy source from work-shared if its already downloaded.

[Paul Eggleton]

Thanks for looking into this problem - this is a tricky one. My apologies for 
the delay in reviewing. Comments below.

On Saturday, 26 January 2019 8:53:00 AM NZDT Sai Hari Chandana Kalluri wrote:
> --- a/scripts/lib/devtool/standard.py
> +++ b/scripts/lib/devtool/standard.py
> @@ -717,6 +717,72 @@ def _check_preserve(config, recipename):
>  tf.write(line)
>  os.rename(newfile, origfile)
>  
> +# Function links a file from src location to dest location
> +def copy_file(c,dest):
> +import errno
> +destdir = os.path.dirname(dest)
> +if os.path.islink(c):
> +   linkto = os.readlink(c)
> +   if os.path.lexists(dest):
> +if not os.path.islink(dest):
> +raise OSError(errno.EEXIST, "Link %s already exists as a 
> file" % dest, dest)
> +if os.readlink(dest) == linkto:
> +return dest
> +raise OSError(errno.EEXIST, "Link %s already exists to a 
> different location? (%s vs %s)" % (dest, os.readlink(dest), linkto), dest)
> +   os.symlink(linkto, dest)
> +else:
> +   try:
> +   os.link(c, dest)
> +   except OSError as err:
> +   if err.errno == errno.EXDEV:
> +bb.utils.copyfile(c, dest)
> +   else:
> +raise
> +
> +# Function creates folders in a given target location
> +def copy_dirs(root,dirs,target):
> +for d in dirs:
> +destdir =  os.path.join(target,d)
> +if os.path.islink(os.path.join(root,d)):
> + linkto = os.readlink(os.path.join(root,d))
> + os.symlink(linkto,destdir) 
> +else:
> + bb.utils.mkdirhier(target+d)
> +
> +# Function to link src dir to dest dir
> +def copy_src_to_ws(srcdir,srctree):
> +target = srctree
> +if os.path.exists(target):
> +raise DevtoolError('source already in your workspace')
> +
> +bb.utils.mkdirhier(target)
> +for root,dirs,files in os.walk(srcdir):
> +#convert abspath to relpath for root
> +destdir = root.replace(srcdir,"")
> +target = srctree+destdir+"/"
> +copy_dirs(root,dirs,target)  
> +for f in files:
> +   copy_file(os.path.join(root,f),os.path.join(target,f))

We already have a function to do this - oe.path.copyhardlinktree(). I 
appreciate that does not symlink if it can't hard link as you're doing here, 
but given the side-effects if the kernel source goes away I think we'd prefer 
"hard link or copy" rather than "hard link or symlink".

> @@ -852,10 +990,12 @@ def modify(args, config, basepath, workspace):
>  f.write('\ndo_patch() {\n'
>  ':\n'
>  '}\n')
> -f.write('\ndo_configure_append() {\n'
> -'cp ${B}/.config ${S}/.config.baseline\n'
> -'ln -sfT ${B}/.config ${S}/.config.new\n'
> -'}\n')
> +
> +if rd.getVarFlag('do_menuconfig','task'):
> +f.write('\ndo_configure_append() {\n'
> +'cp ${B}/.config ${S}/.config.baseline\n'
> +'ln -sfT ${B}/.config ${S}/.config.new\n'
> +'}\n')
>  if initial_rev:
>  f.write('\n# initial_rev: %s\n' % initial_rev)
>  for commit in commits:

This change does not appear to be directly related to the rest of the commit, 
can it be split out and properly described?

Thanks,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre


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

[OE-core] [PATCH 2/2] btrfs-tools: pass V=1 to get verbose builds

[Ross Burton]

Quiet builds are nice for interactive builds, but hide information we'd like in
the logs.

Signed-off-by: Ross Burton 
---
 meta/recipes-devtools/btrfs-tools/btrfs-tools_4.20.1.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-devtools/btrfs-tools/btrfs-tools_4.20.1.bb 
b/meta/recipes-devtools/btrfs-tools/btrfs-tools_4.20.1.bb
index a6ca71d8689..3647232dc68 100644
--- a/meta/recipes-devtools/btrfs-tools/btrfs-tools_4.20.1.bb
+++ b/meta/recipes-devtools/btrfs-tools/btrfs-tools_4.20.1.bb
@@ -27,6 +27,7 @@ CLEANBROKEN = "1"
 PACKAGECONFIG[manpages] = "--enable-documentation, --disable-documentation, 
asciidoc-native xmlto-native"
 EXTRA_OECONF = " --disable-zstd"
 EXTRA_OECONF_append_libc-musl = " --disable-backtrace "
+EXTRA_OEMAKE = "V=1"
 
 do_configure_prepend() {
# Upstream doesn't ship this and autoreconf won't install it as 
automake isn't used.
-- 
2.11.0

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

[OE-core] [PATCH 1/2] btrfs-tools: remove redundant cross-compilation patch

[Ross Burton]

This patch was to use BUILD_CC instead of CC when building mktables, a binary
used to generate tables.c.

However due to cross-compilation problems such as this, upstream since 4.12.1
(f929b62 specifically) avoids the entire issue and ships tables.c in the
repository, updating it as required.

Signed-off-by: Ross Burton 
---
 ...-Makefile-build-mktables-using-native-gcc.patch | 27 --
 .../btrfs-tools/btrfs-tools_4.20.1.bb  |  1 -
 2 files changed, 28 deletions(-)
 delete mode 100644 
meta/recipes-devtools/btrfs-tools/btrfs-tools/0001-Makefile-build-mktables-using-native-gcc.patch

diff --git 
a/meta/recipes-devtools/btrfs-tools/btrfs-tools/0001-Makefile-build-mktables-using-native-gcc.patch
 
b/meta/recipes-devtools/btrfs-tools/btrfs-tools/0001-Makefile-build-mktables-using-native-gcc.patch
deleted file mode 100644
index 467de327515..000
--- 
a/meta/recipes-devtools/btrfs-tools/btrfs-tools/0001-Makefile-build-mktables-using-native-gcc.patch
+++ /dev/null
@@ -1,27 +0,0 @@
-From e58369f6d36bc51eb59d6afa34c1cae3ff0810ef Mon Sep 17 00:00:00 2001
-From: Alexander Kanavin 
-Date: Mon, 7 Aug 2017 14:10:38 +0300
-Subject: [PATCH] Makefile: build mktables using native gcc
-
-It's a throwaway helper binary used during build, and so it needs to
-be native.
-
-Upstream-Status: Inappropriate [oe specific]
-Signed-off-by: Alexander Kanavin 

- Makefile | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-Index: git/Makefile
-===
 git.orig/Makefile
-+++ git/Makefile
-@@ -343,7 +343,7 @@ version.h: version.sh version.h.in confi
- 
- mktables: kernel-lib/mktables.c
-   @echo "[CC] $@"
--  $(Q)$(CC) $(CFLAGS) $< -o $@
-+  $(Q)$(BUILD_CC) $(BUILD_CFLAGS) $< -o $@
- 
- # the target can be regenerated manually using mktables, but a local copy is
- # kept so the build process is simpler
diff --git a/meta/recipes-devtools/btrfs-tools/btrfs-tools_4.20.1.bb 
b/meta/recipes-devtools/btrfs-tools/btrfs-tools_4.20.1.bb
index 1b4f515cb62..a6ca71d8689 100644
--- a/meta/recipes-devtools/btrfs-tools/btrfs-tools_4.20.1.bb
+++ b/meta/recipes-devtools/btrfs-tools/btrfs-tools_4.20.1.bb
@@ -16,7 +16,6 @@ RDEPENDS_${PN} = "libgcc"
 
 SRCREV = "6c33832b8c7bf2d7b64d6aed023c0bda43a03311"
 SRC_URI = "git://git.kernel.org/pub/scm/linux/kernel/git/kdave/btrfs-progs.git 
\
-   file://0001-Makefile-build-mktables-using-native-gcc.patch \
file://0001-Add-LDFLAGS-when-building-libbtrfsutil.so.patch \

file://0001-Add-a-possibility-to-specify-where-python-modules-ar.patch \
"
-- 
2.11.0

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

[OE-core] [PATCH 2/4] oe.scriptutils: add logger_setup_filters

[Christopher Larson]

From: Christopher Larson 

This function sets up the default bb.msg formatter and log filters, which by
default sends ERROR messages to stderr rather than stdout.

Signed-off-by: Christopher Larson 
---
 scripts/lib/scriptutils.py | 15 +++
 1 file changed, 15 insertions(+)

diff --git a/scripts/lib/scriptutils.py b/scripts/lib/scriptutils.py
index 0633c7066e..c361d0741d 100644
--- a/scripts/lib/scriptutils.py
+++ b/scripts/lib/scriptutils.py
@@ -29,6 +29,7 @@ import tempfile
 import importlib
 from importlib import machinery
 
+
 def logger_create(name, stream=None):
 logger = logging.getLogger(name)
 loggerhandler = logging.StreamHandler(stream=stream)
@@ -37,6 +38,20 @@ def logger_create(name, stream=None):
 logger.setLevel(logging.INFO)
 return logger
 
+
+def logger_setup_filters(logger):
+import bb.msg
+
+console = logging.StreamHandler(sys.stdout)
+errconsole = logging.StreamHandler(sys.stderr)
+bb.msg.addDefaultlogFilter(console, bb.msg.BBLogFilterStdOut)
+bb.msg.addDefaultlogFilter(errconsole, bb.msg.BBLogFilterStdErr)
+format_str = "%(levelname)s: %(message)s"
+console.setFormatter(bb.msg.BBLogFormatter(format_str))
+errconsole.setFormatter(bb.msg.BBLogFormatter(format_str))
+logger.handlers = [console, errconsole]
+
+
 def logger_setup_color(logger, color='auto'):
 from bb.msg import BBLogFormatter
 
-- 
2.20.1

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

[OE-core] [PATCH 3/4] recipetool, devtool: set up the logging filters

[Christopher Larson]

From: Christopher Larson 

We want errors going to stderr, the way they do in bitbake itself.

Signed-off-by: Christopher Larson 
---
 scripts/devtool| 2 ++
 scripts/recipetool | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/scripts/devtool b/scripts/devtool
index 0e578c0de3..de1847eef3 100755
--- a/scripts/devtool
+++ b/scripts/devtool
@@ -282,11 +282,13 @@ def main():
 logger.debug('Using standard bitbake path %s' % bitbakepath)
 scriptpath.add_oe_lib_path()
 
+scriptutils.logger_setup_filters(logger)
 scriptutils.logger_setup_color(logger, global_args.color)
 
 if global_args.bbpath is None:
 try:
 tinfoil = setup_tinfoil(config_only=True, basepath=basepath)
+tinfoil.logger.handlers = logger.handlers
 try:
 global_args.bbpath = tinfoil.config_data.getVar('BBPATH')
 finally:
diff --git a/scripts/recipetool b/scripts/recipetool
index 3a3c9b7445..bac9c72107 100755
--- a/scripts/recipetool
+++ b/scripts/recipetool
@@ -75,9 +75,11 @@ def main():
 logger.debug('Found bitbake path: %s' % bitbakepath)
 scriptpath.add_oe_lib_path()
 
+scriptutils.logger_setup_filters(logger)
 scriptutils.logger_setup_color(logger, global_args.color)
 
 tinfoil = tinfoil_init(False)
+tinfoil.logger.handlers = logger.handlers
 try:
 for path in (tinfoil.config_data.getVar('BBPATH').split(':')
  + [scripts_path]):
-- 
2.20.1

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

[OE-core] [PATCH 1/4] oe.scriptutils: enable color in a more flexible way

[Christopher Larson]

From: Christopher Larson 

Rather than recreating handlers and forcing them, iterate over the handlers
and enable color on ones we can handle. This makes it easier to handle color
properly when we introduce the bb.msg default log filters.

Signed-off-by: Christopher Larson 
---
 scripts/lib/scriptutils.py | 13 +++--
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/scripts/lib/scriptutils.py b/scripts/lib/scriptutils.py
index 3c60c3a1e6..0633c7066e 100644
--- a/scripts/lib/scriptutils.py
+++ b/scripts/lib/scriptutils.py
@@ -39,12 +39,12 @@ def logger_create(name, stream=None):
 
 def logger_setup_color(logger, color='auto'):
 from bb.msg import BBLogFormatter
-console = logging.StreamHandler(sys.stdout)
-formatter = BBLogFormatter("%(levelname)s: %(message)s")
-console.setFormatter(formatter)
-logger.handlers = [console]
-if color == 'always' or (color=='auto' and console.stream.isatty()):
-formatter.enable_color()
+
+for handler in logger.handlers:
+if (isinstance(handler, logging.StreamHandler) and
+isinstance(handler.formatter, BBLogFormatter)):
+if color == 'always' or (color == 'auto' and 
handler.stream.isatty()):
+handler.formatter.enable_color()
 
 
 def load_plugins(logger, plugins, pluginpath):
@@ -69,6 +69,7 @@ def load_plugins(logger, plugins, pluginpath):
 plugin.plugin_init(plugins)
 plugins.append(plugin)
 
+
 def git_convert_standalone_clone(repodir):
 """If specified directory is a git repository, ensure it's a standalone 
clone"""
 import bb.process
-- 
2.20.1

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

[OE-core] [PATCH 4/4] oe.scriptutils: also send WARNING to stderr

[Christopher Larson]

From: Christopher Larson 

stderr isn't just for errors, but anything out of band. It's highly unlikely
that warning messages are anything but out of band, which makes scripting our
tools more troublesome, since the script calling it has to filter out the
warnings from stdout to get the real output.

Signed-off-by: Christopher Larson 
---
 scripts/lib/scriptutils.py | 6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/scripts/lib/scriptutils.py b/scripts/lib/scriptutils.py
index c361d0741d..d1a50207c5 100644
--- a/scripts/lib/scriptutils.py
+++ b/scripts/lib/scriptutils.py
@@ -43,9 +43,11 @@ def logger_setup_filters(logger):
 import bb.msg
 
 console = logging.StreamHandler(sys.stdout)
+bb.msg.addDefaultlogFilter(console)
 errconsole = logging.StreamHandler(sys.stderr)
-bb.msg.addDefaultlogFilter(console, bb.msg.BBLogFilterStdOut)
-bb.msg.addDefaultlogFilter(errconsole, bb.msg.BBLogFilterStdErr)
+bb.msg.addDefaultlogFilter(errconsole)
+console.addFilter(lambda r: r.levelno < logging.WARNING)
+errconsole.addFilter(lambda r: r.levelno >= logging.WARNING)
 format_str = "%(levelname)s: %(message)s"
 console.setFormatter(bb.msg.BBLogFormatter(format_str))
 errconsole.setFormatter(bb.msg.BBLogFormatter(format_str))
-- 
2.20.1

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

[OE-core] [PATCH 0/4] Improve logger setup in devtool/recipetool/scriptutils

[Christopher Larson]

From: Christopher Larson 

This improves logger setup in devtool and recipetool to send ERRORs to stderr,
the way bitbake itself does, and also sends WARNINGs there, as those are
essentially out of band as well. This makes it easier to script with devtool
and recipetool, as any *real* subcommand output can be more easily captured
without having to manually filter out the cruft.

The following changes since commit b11725db2d5549dc45d8ae36fbf94a5c8e342d69:

  xserver-xorg: Fix build errors with clang (2019-03-04 14:26:41 +)

are available in the Git repository at:

  g...@github.com:kergoth/openembedded-core scriptutils-log-improvements

for you to fetch changes up to c4f37568236d2e3d8c40d57399d0b275ad5b7c8b:

  oe.scriptutils: also send WARNING to stderr (2019-03-04 21:31:00 +)


Christopher Larson (4):
  oe.scriptutils: enable color in a more flexible way
  oe.scriptutils: add logger_setup_filters
  recipetool, devtool: set up the logging filters
  oe.scriptutils: also send WARNING to stderr

 scripts/devtool|  2 ++
 scripts/lib/scriptutils.py | 30 --
 scripts/recipetool |  2 ++
 3 files changed, 28 insertions(+), 6 deletions(-)

-- 
2.20.1

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

Re: [OE-core] [OE-Core][master][PATCH] standard.py: Provide an additional option for devtool reset

[Paul Eggleton]

On Friday, 8 February 2019 2:57:32 PM NZDT Sai Hari Chandana Kalluri wrote:
> The devtool reset command cleans the sysroot for a recipe in workspace.
> It also removes the append file but leaves the source code as in
> workspace. The source is not cleaned intentionally and the user has to
> manually remove it before calling devtool modify again.
> 
> Provide the user with an option to remove the source code from workspace
> by adding a flag to the devtool reset command. The --rm-source option for
> the devtool reset command will also clean the source code from the
> workspace along with the sysroot and the append file.
> Ex: devtool reset --rm-source zip or devtool reset -r zip

When I wrote this I made a conscious decision not to delete the source, in 
case the user has unsaved work in the source tree; at least then the user has 
to take an explicit step (an additional rm -rf) in order to delete it. You 
might argue that adding an option is the same as that explicit step, but I am 
still hesitant as you might for example pick a previously used command from 
your command history with -r in it and only realise after you've run it that 
you've just blown away a bunch of your work.

However, I have received the feedback from a number of people that they find 
it annoying that they have to delete the source themselves before being able 
to run devtool on the recipe again. I'd like to hear from the wider community 
on the following two questions:

1) Does the convenience of having this kind of option outweigh the potential 
danger of deleting unsaved work?

2) Are there issues that anyone is experiencing that force you to run devtool 
clean/finish and then re-run devtool modify (or add, or upgrade), making this 
kind of situation come up more frequently? Or is it mostly that the source 
directories end up being left around to stumble over some time later?

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre


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

Re: [OE-core] [PATCH] Support kmeta directory usage with devtool modify/finish

[Paul Eggleton]

On Saturday, 15 December 2018 1:39:22 PM NZDT Jaewon Lee wrote:
> When using Kmeta directories, devtool finish will add every single file
> in the directory to the bbappend. This is because in the current
> implementation, the get_recipe_local_files function treats the kmeta
> directory like a file. Modifying the function to loop through the
> provided directories and return all included files instead of just the
> top level directory. This will enable correct file to file comparison
> when determing which files are new/changed and need to be added to the
> bbappend.
> 
> Adding an extra check in devtool-source.bbclass to not copy the cfg file
> if its already included somewhere in the kmeta directory
> 
> Also during 'modify', when moving necessary files in the kmeta directory
> from the workdir to oe-local-files, the dangling parent directories are
> left behind.  This in itself is not an issue as the temporary devtool
> workspace is automatically deleted, but this causes an incorrect include
> directory to be added in kernel-yocto.bbclass.  Changing the order of
> the if statements to catch the correct conditional. This is safe to do
> as when not in the devtool context, there will be no oe-local-files
> directory.

Looks good to me. My apologies for the extreme delay in reviewing.

Acked-by: Paul Eggleton 


-- 

Paul Eggleton
Intel Open Source Technology Centre


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

[OE-core] [poky][meta][sumo][PATCH] lighttpd: added ptest

[Aditya Tayade]

Reused lighttpd internal tests and did minimal patching to fix paths.

Upstream-Status: Pending

Signed-off-by: Aditya Tayade 
---
 ...-use-standard-sbin-and-lib-path-for-ptest.patch | 37 
 meta/recipes-extended/lighttpd/lighttpd/run-ptest  |  4 ++
 meta/recipes-extended/lighttpd/lighttpd_1.4.48.bb  | 66 +-
 3 files changed, 106 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-extended/lighttpd/lighttpd/0002-use-standard-sbin-and-lib-path-for-ptest.patch
 create mode 100644 meta/recipes-extended/lighttpd/lighttpd/run-ptest

diff --git 
a/meta/recipes-extended/lighttpd/lighttpd/0002-use-standard-sbin-and-lib-path-for-ptest.patch
 
b/meta/recipes-extended/lighttpd/lighttpd/0002-use-standard-sbin-and-lib-path-for-ptest.patch
new file mode 100644
index 000..cd4b2b3
--- /dev/null
+++ 
b/meta/recipes-extended/lighttpd/lighttpd/0002-use-standard-sbin-and-lib-path-for-ptest.patch
@@ -0,0 +1,37 @@
+lighttpd-ptest: use standard installation path
+
+ptest should do testing against installed library or executable
+at standard installation path, hence using standard installation
+path for MODULES_PATH and LIGHTTPD_PATH variables.
+
+Upsteam-Status: Pending
+
+Signed-off-by: Aditya Tayade 
+
+diff --git a/tests/LightyTest.pm b/tests/LightyTest.pm
+index 74caea1..2de02d0 100644
+--- a/tests/LightyTest.pm
 b/tests/LightyTest.pm
+@@ -65,19 +65,10 @@ sub new {
+ $lpath = (defined $ENV{'srcdir'} ? $ENV{'srcdir'} : '.');
+ $self->{SRCDIR} = abs_path($lpath);
+
++$self->{SBINDIR} = '/usr/sbin';
++$self->{MODULES_PATH} = '/usr/lib';
+
+-if (mtime($self->{BASEDIR}.'/src/lighttpd') > 
mtime($self->{BASEDIR}.'/build/lighttpd')) {
+-$self->{BINDIR} = $self->{BASEDIR}.'/src';
+-if (mtime($self->{BASEDIR}.'/src/.libs')) {
+-$self->{MODULES_PATH} = $self->{BASEDIR}.'/src/.libs';
+-} else {
+-$self->{MODULES_PATH} = $self->{BASEDIR}.'/src';
+-}
+-} else {
+-$self->{BINDIR} = $self->{BASEDIR}.'/build';
+-$self->{MODULES_PATH} = $self->{BASEDIR}.'/build';
+-}
+-$self->{LIGHTTPD_PATH} = $self->{BINDIR}.'/lighttpd';
++$self->{LIGHTTPD_PATH} = $self->{SBINDIR}.'/lighttpd';
+ $self->{PORT} = 2048;
+
+ my ($name, $aliases, $addrtype, $net) = gethostbyaddr(inet_aton("127.0.0.1"), 
AF_INET);
diff --git a/meta/recipes-extended/lighttpd/lighttpd/run-ptest 
b/meta/recipes-extended/lighttpd/lighttpd/run-ptest
new file mode 100644
index 000..b6781c0
--- /dev/null
+++ b/meta/recipes-extended/lighttpd/lighttpd/run-ptest
@@ -0,0 +1,4 @@
+#!/bin/bash
+
+make -C tests -k check-TESTS
+make -C src -k runtest-TESTS
diff --git a/meta/recipes-extended/lighttpd/lighttpd_1.4.48.bb 
b/meta/recipes-extended/lighttpd/lighttpd_1.4.48.bb
index 3cc..6a9677e 100644
--- a/meta/recipes-extended/lighttpd/lighttpd_1.4.48.bb
+++ b/meta/recipes-extended/lighttpd/lighttpd_1.4.48.bb
@@ -9,6 +9,42 @@ SECTION = "net"
 RDEPENDS_${PN} = "lighttpd-module-dirlisting \
   lighttpd-module-indexfile \
   lighttpd-module-staticfile"
+
+RDEPENDS_${PN}-ptest += "bash \
+ ${PN} \
+ ${PN}-module-compress \
+ ${PN}-module-simple-vhost \
+ ${PN}-module-rewrite \
+ ${PN}-module-cgi \
+ ${PN}-module-evhost \
+ ${PN}-module-redirect \
+ ${PN}-module-fastcgi \
+ ${PN}-module-setenv \
+ ${PN}-module-extforward \
+ ${PN}-module-secdownload \
+ ${PN}-module-auth \
+ ${PN}-module-authn-file \
+ ${PN}-module-status \
+ ${PN}-module-expire \
+ ${PN}-module-userdir \
+ ${PN}-module-ssi \
+ ${PN}-module-proxy \
+ perl-module-tap-base \
+ perl-module-tap-parser-sourcehandler-executable \
+ perl-module-tap-formatter-file \
+ perl-module-tap-formatter-file-session \
+ perl-module-tap-formatter-base \
+ perl-module-tap-formatter-session \
+ perl-module-tap-formatter-console-session \
+ perl-module-tap-parser-scheduler \
+ perl-module-tap-parser \
+ perl-module-io-socket \
+ perl-module-test-more \
+ perl-module-test-harness \
+ perl-module-tap-object \
+ perl-module-tap-formatter-console \
+"
+
 RRECOMMENDS_${PN} = "lighttpd-module-access \
  lighttpd-module-accesslog"

@@ -18,6 +54,8 @@ SRC_URI = 
"http://download.lighttpd.net/lighttpd/releases-1.4.x/lighttpd-${PV}.t
 file://lighttpd \
 

Re: [OE-core] [meta-oe][sumo][PATCH] run-ptest: supplied user inputs to add_person_cpp

[Khem Raj]

this should go to oe-devel ml and with right subject prefix.

On Mon, Mar 4, 2019 at 10:39 AM Aditya Tayade  wrote:
>
> During protobuf-ptest execution, add_person_cpp waits for user
> inputs to write data into test.data file. Fixed this by supplying
> dummy data through standard input.
>
> Upstream-Status: Pending
>
> Signed-off-by: Aditya Tayade 
> ---
>  meta-oe/recipes-devtools/protobuf/protobuf/run-ptest | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta-oe/recipes-devtools/protobuf/protobuf/run-ptest 
> b/meta-oe/recipes-devtools/protobuf/protobuf/run-ptest
> index 7c3a8d1..dc6c83d 100644
> --- a/meta-oe/recipes-devtools/protobuf/protobuf/run-ptest
> +++ b/meta-oe/recipes-devtools/protobuf/protobuf/run-ptest
> @@ -8,7 +8,7 @@ for write_exe_full_path in ${DIR}/add_person_*; do
>  if [ -x "${write_exe_full_path}" ]; then
>  write_exe=`basename ${write_exe_full_path}`
>  echo "Generating new test file using ${write_exe}..."
> -${write_exe_full_path} "${TEST_FILE}"
> +printf "1234\nname\nn...@example.com\n" | ${write_exe_full_path} 
> "${TEST_FILE}"
>  RETVAL=$?
>  [ $RETVAL -eq 0 ] || exit $RETVAL
>
> --
> 2.7.4
>
> This message contains information that may be privileged or confidential and 
> is the property of the KPIT Technologies Ltd. It is intended only for the 
> person to whom it is addressed. If you are not the intended recipient, you 
> are not authorized to read, print, retain copy, disseminate, distribute, or 
> use this message or any part thereof. If you receive this message in error, 
> please notify the sender immediately and delete all copies of this message. 
> KPIT Technologies Ltd. does not accept any liability for virus infected mails.
> --
> ___
> 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-oe][sumo][PATCH] run-ptest: supplied user inputs to add_person_cpp

[Aditya Tayade]

During protobuf-ptest execution, add_person_cpp waits for user
inputs to write data into test.data file. Fixed this by supplying
dummy data through standard input.

Upstream-Status: Pending

Signed-off-by: Aditya Tayade 
---
 meta-oe/recipes-devtools/protobuf/protobuf/run-ptest | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-devtools/protobuf/protobuf/run-ptest 
b/meta-oe/recipes-devtools/protobuf/protobuf/run-ptest
index 7c3a8d1..dc6c83d 100644
--- a/meta-oe/recipes-devtools/protobuf/protobuf/run-ptest
+++ b/meta-oe/recipes-devtools/protobuf/protobuf/run-ptest
@@ -8,7 +8,7 @@ for write_exe_full_path in ${DIR}/add_person_*; do
 if [ -x "${write_exe_full_path}" ]; then
 write_exe=`basename ${write_exe_full_path}`
 echo "Generating new test file using ${write_exe}..."
-${write_exe_full_path} "${TEST_FILE}"
+printf "1234\nname\nn...@example.com\n" | ${write_exe_full_path} "${TEST_FILE}"
 RETVAL=$?
 [ $RETVAL -eq 0 ] || exit $RETVAL

--
2.7.4

This message contains information that may be privileged or confidential and is 
the property of the KPIT Technologies Ltd. It is intended only for the person 
to whom it is addressed. If you are not the intended recipient, you are not 
authorized to read, print, retain copy, disseminate, distribute, or use this 
message or any part thereof. If you receive this message in error, please 
notify the sender immediately and delete all copies of this message. KPIT 
Technologies Ltd. does not accept any liability for virus infected mails.
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [thud][PATCH] openssl10: Fix mutliple include assumptions for des.h in opensslconf.h

[Denys Dmytriyenko]

From: Denys Dmytriyenko 

The fix is heavily based on Khem's previous fix for bn.h/BN_LLONG breakage:
https://git.openembedded.org/openembedded-core/commit/?id=f787b0bb9b0626ddbf2ac94cb206c76716a3773d

(From OE-Core rev: 914e1520bf9c45e14bce9993c9131a2c0702b9c9)

Signed-off-by: Denys Dmytriyenko 
Cc: Khem Raj 
Signed-off-by: Richard Purdie 
Signed-off-by: Denys Dmytriyenko 
---
 .../openssl10/0001-Fix-DES_LONG-breakage.patch | 35 ++
 .../openssl/openssl10_1.0.2q.bb|  1 +
 2 files changed, 36 insertions(+)
 create mode 100644 
meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch

diff --git 
a/meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch 
b/meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch
new file mode 100644
index 000..7243fb4
--- /dev/null
+++ 
b/meta/recipes-connectivity/openssl/openssl10/0001-Fix-DES_LONG-breakage.patch
@@ -0,0 +1,35 @@
+From 1aec49cc45e7cf5ebc059a77081ac3ea2a5aff7b Mon Sep 17 00:00:00 2001
+From: Denys Dmytriyenko 
+Date: Fri, 1 Mar 2019 13:14:56 -0500
+Subject: [PATCH] Fix DES_LONG breakage
+
+Mimic previous BN_LLONG fix by Khem Raj here. Re-use its description:
+
+opensslconf.h is defining DES_LONG only when included from des.h which
+is not robust at all, especially when include guards are used and
+multiple inclusions of a given header is not allowed. so lets take out
+the nesting constraint and add OPENSSL_SYS_UEFI constraint instead
+
+Upstream-Status: Inappropriate [ fixed differently with OpenSSL 1.1+ ]
+
+Signed-off-by: Denys Dmytriyenko 
+---
+ crypto/opensslconf.h.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/crypto/opensslconf.h.in b/crypto/opensslconf.h.in
+index a10c10f..0147a4d 100644
+--- a/crypto/opensslconf.h.in
 b/crypto/opensslconf.h.in
+@@ -48,7 +48,7 @@
+ #endif
+ #endif
+ 
+-#if (defined(HEADER_NEW_DES_H) || defined(HEADER_DES_H)) && !defined(DES_LONG)
++#if !defined(OPENSSL_SYS_UEFI) && !defined(DES_LONG)
+ /* If this is set to 'unsigned int' on a DEC Alpha, this gives about a
+  * %20 speed up (longs are 8 bytes, int's are 4). */
+ #ifndef DES_LONG
+-- 
+2.7.4
+
diff --git a/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb 
b/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb
index 355dbdc..54af100 100644
--- a/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb
+++ b/meta/recipes-connectivity/openssl/openssl10_1.0.2q.bb
@@ -41,6 +41,7 @@ SRC_URI = "http://www.openssl.org/source/openssl-${PV}.tar.gz 
\
file://0001-openssl-force-soft-link-to-avoid-rare-race.patch \
file://0001-allow-manpages-to-be-disabled.patch \
file://0001-Fix-BN_LLONG-breakage.patch \
+   file://0001-Fix-DES_LONG-breakage.patch \
"
 
 SRC_URI_append_class-target = " \
-- 
2.7.4

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

Re: [OE-core] [PATCH 3/4] packagegroup-core-base-utils: Introduce new base-utils packagegroup

[Tom Rini]

On Mon, Mar 04, 2019 at 10:44:00AM -0600, Mark Hatle wrote:
> On 3/1/19 4:26 PM, Tom Rini wrote:
> > This packagegroup is intended to provide all of the functionality found
> > in other VIRTUAL-RUNTIME_base-utils providers through full packages
> > rather than an all-on-one package.  Document how to make use of this
> > in local.conf.sample.extended.  Introduce VIRTUAL-RUNTIME-vim and
> > default this to vim-tiny to allow for a differently features vim to be
> > used instead.
> > 
> > Signed-off-by: Tom Rini 
> > ---
> >  meta/conf/local.conf.sample.extended   |  8 
> >  .../packagegroups/packagegroup-core-base-utils.bb  | 49 
> > ++
> >  2 files changed, 57 insertions(+)
> >  create mode 100644 
> > meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
> > 
> > diff --git a/meta/conf/local.conf.sample.extended 
> > b/meta/conf/local.conf.sample.extended
> > index 8062e4c5721b..d00221893ad9 100644
> > --- a/meta/conf/local.conf.sample.extended
> > +++ b/meta/conf/local.conf.sample.extended
> > @@ -384,3 +384,11 @@
> >  #VIRTUAL-RUNTIME_login_manager = "shadow-base"
> >  #VIRTUAL-RUNTIME_init_manager = "systemd"
> >  #VIRTUAL-RUNTIME_initscripts = "systemd-compat-units"
> > +
> > +#
> > +# Use a full set of packages instead of busybox for base utils
> > +#
> > +#PREFERRED_PROVIDER_base-utils = "packagegroup-core-base-utils"
> > +#VIRTUAL-RUNTIME_base-utils = "packagegroup-core-base-utils"
> > +#VIRTUAL-RUNTIME_base-utils-hwclock = "util-linux-hwclock"
> > +#VIRTUAL-RUNTIME_base-utils-syslog = "syslog"
> > diff --git 
> > a/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb 
> > b/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
> > new file mode 100644
> > index ..061dade0bc03
> > --- /dev/null
> > +++ b/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
> > @@ -0,0 +1,49 @@
> > +#
> > +# Copyright (C) 2019 Konsulko Group
> > +#
> > +
> > +SUMMARY = "Full-featured set of base utils"
> > +DESCRIPTION = "Package group bringing in packages needed to provide much 
> > of the base utils type functionality found in busybox"
> > +
> > +inherit packagegroup
> > +
> > +VIRTUAL-RUNTIME_vim ?= "vim-tiny"
> > +
> > +RDEPENDS_${PN} = "\
> > +base-passwd \
> > +bash \
> > +bind-utils \
> > +bzip2 \
> > +coreutils \
> > +cpio \
> > +dhcp-client \
> > +diffutils \
> > +e2fsprogs \
> > +ed \
> > +file \
> > +findutils \
> > +gawk \
> > +grep \
> > +gzip \
> > +inetutils \
> > +iproute2 \
> > +kmod \
> > +less \
> > +lzip \
> > +ncurses-tools \
> > +net-tools \
> > +parted \
> > +patch \
> > +procps \
> > +psmisc \
> > +sed \
> > +shadow-base \
> > +tar \
> > +time \
> > +unzip \
> > +util-linux \
> > +${VIRTUAL-RUNTIME_vim} \
> > +wget \
> > +which \
> > +xz \
> > +"
> > 
> 
> Sorry missed this earlier.  On note, we have a similar recipe in our local 
> layer.
> 
> I figured you may want to look at it to determine if anything might be 
> missing.
> 
> https://github.com/WindRiver-OpenSourceLabs/wrlinux/blob/master-wr/recipes-base/packagegroups/packagegroup-busybox-replacement.bb
> 
> The parts I see missing (when comparing them) are:
> 
> bc

I had this initially but bc isn't enabled in busybox (anymore?).

> debianutils-run-parts
> dhcp-server (likely not needed)
> dpkg-start-stop
> ifupdown

These fall into the category of "Oh, I need to try without systemd".
I'll put those in and give it a spin.

> kbd

This is in, but via a dep rather than explicit, so I'll list it.

> In some configurations we also add "fbset, netcat, and usleep" as well... but 
> I
> think the reasonable approach is to match the default busybox configuration to
> whatever is in this list.

Ah fbset, going off on a quick tangent, I see 2.1 from 1999 is still the
latest and greatest.  Good times!  I'm not sure if we should move fbset
to oe-core however.  For netcat, we have netcat-openbsd in meta-oe which
is 'nc' and would match busybox saying CONFIG_NC=y and CONFIG_NETCAT=n.
For usleep, real sleep (and busybox sleep when configured) supports
decimals so I would be inclined to say sleep is good enough.

-- 
Tom


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

[OE-core] [PATCH 5/6] util-linux: remove spurious lzo-native dependency

[Ross Burton]

This was added with no explanation in 2010 (17f3c548c) and doesn't appear to be
needed.

Signed-off-by: Ross Burton 
---
 meta/recipes-core/util-linux/util-linux.inc | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-core/util-linux/util-linux.inc 
b/meta/recipes-core/util-linux/util-linux.inc
index cce684f2802..18c3af240eb 100644
--- a/meta/recipes-core/util-linux/util-linux.inc
+++ b/meta/recipes-core/util-linux/util-linux.inc
@@ -21,8 +21,6 @@ LIC_FILES_CHKSUM = 
"file://README.licensing;md5=1715f5ee3e01203ca1e1e0b9ee65918c
 #gtk-doc is not enabled as it requires xmlto which requires util-linux
 inherit autotools gettext manpages pkgconfig systemd update-alternatives 
python3-dir bash-completion ptest
 DEPENDS = "zlib ncurses virtual/crypt"
-DEPENDS_append_class-native = " lzo-native"
-DEPENDS_append_class-nativesdk = " lzo-native"
 
 MAJOR_VERSION = "${@'.'.join(d.getVar('PV').split('.')[0:2])}"
 SRC_URI = 
"${KERNELORG_MIRROR}/linux/utils/${BPN}/v${MAJOR_VERSION}/${BP}.tar.xz \
-- 
2.11.0

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

[OE-core] [PATCH 6/6] squashfs-tools: add PACKAGECONFIGs for all the tunables

[Ross Burton]

Upstream lets you control what compression algorithms are used, so add
PACKAGECONFIGs for them all.

Instead of bundling a copy of the SevenZIP SDK from 2009 (!), reuse XZ.  This
means we can drop Public Domain from the license list.

Also -e was removed from EXTRA_OEMAKE in 2016 (oe-core aeb653) so the fiddling
to work around that can be removed.

Signed-off-by: Ross Burton 
---
 .../squashfs-tools/squashfs-tools_git.bb   | 22 --
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb 
b/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
index 2f412634571..e871f1cbf54 100644
--- a/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
+++ b/meta/recipes-devtools/squashfs-tools/squashfs-tools_git.bb
@@ -2,16 +2,12 @@
 # and I don't think the kernel supports it any more.
 SUMMARY = "Tools for manipulating SquashFS filesystems"
 SECTION = "base"
-LICENSE = "GPL-2 & PD"
-LIC_FILES_CHKSUM = "file://../COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263 \
-
file://../../7zC.txt;beginline=12;endline=16;md5=2056cd6d919ebc3807602143c7449a7c
 \
-"
-DEPENDS = "attr zlib xz lzo lz4"
+LICENSE = "GPL-2"
+LIC_FILES_CHKSUM = "file://../COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
 
 PV = "4.3+gitr${SRCPV}"
 SRCREV = "9c1db6d13a51a2e009f0027ef336ce03624eac0d"
 SRC_URI = "git://github.com/plougher/squashfs-tools.git;protocol=https \
-   http://downloads.sourceforge.net/sevenzip/lzma465.tar.bz2;name=lzma 
\

file://0001-mksquashfs.c-get-inline-functions-work-with-C99.patch;striplevel=2 \
file://squashfs-tools-4.3-sysmacros.patch;striplevel=2 \
file://fix-compat.patch \
@@ -23,14 +19,20 @@ SRC_URI[lzma.sha256sum] = 
"c935fd04dd8e0e8c688a3078f3675d699679a90be81c12686837e
 
 S = "${WORKDIR}/git/squashfs-tools"
 
-# EXTRA_OEMAKE is typically: -e MAKEFLAGS=
-# the -e causes problems as CFLAGS is modified in the Makefile, so
-# we redefine EXTRA_OEMAKE here
-EXTRA_OEMAKE = "MAKEFLAGS= LZMA_SUPPORT=1 LZMA_DIR=../.. XZ_SUPPORT=1 
LZO_SUPPORT=1 LZ4_SUPPORT=1"
+EXTRA_OEMAKE = "${PACKAGECONFIG_CONFARGS}"
+
+PACKAGECONFIG ??= "gzip xz lzo lz4 lzma xattr"
+PACKAGECONFIG[gzip] = "GZIP_SUPPORT=1,GZIP_SUPPORT=0,zlib"
+PACKAGECONFIG[xz] = "XZ_SUPPORT=1,XZ_SUPPORT=0,xz"
+PACKAGECONFIG[lzo] = "LZO_SUPPORT=1,LZO_SUPPORT=0,lzo"
+PACKAGECONFIG[lz4] = "LZ4_SUPPORT=1,LZ4_SUPPORT=0,lz4"
+PACKAGECONFIG[lzma] = "LZMA_XZ_SUPPORT=1,LZMA_XZ_SUPPORT=0,xz"
+PACKAGECONFIG[xattr] = "XATTR_SUPPORT=1,XATTR_SUPPORT=0,attr"
 
 do_compile() {
oe_runmake mksquashfs unsquashfs
 }
+
 do_install () {
install -d ${D}${sbindir}
install -m 0755 mksquashfs ${D}${sbindir}/
-- 
2.11.0

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

[OE-core] [PATCH 4/6] mtd-utils: add lzo PACKAGECONFIG

[Ross Burton]

LZO support is optional so add a PACKAGECONFIG to control this.

Also clean up the xattr PACKAGECONFIG to explicitly enable/disable.

Signed-off-by: Ross Burton 
---
 meta/recipes-devtools/mtd/mtd-utils_git.bb | 7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-devtools/mtd/mtd-utils_git.bb 
b/meta/recipes-devtools/mtd/mtd-utils_git.bb
index f53c6c1122d..9ffac2ed3a8 100644
--- a/meta/recipes-devtools/mtd/mtd-utils_git.bb
+++ b/meta/recipes-devtools/mtd/mtd-utils_git.bb
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
 
 inherit autotools pkgconfig update-alternatives
 
-DEPENDS = "zlib lzo e2fsprogs util-linux"
+DEPENDS = "zlib e2fsprogs util-linux"
 RDEPENDS_mtd-utils-tests += "bash"
 
 PV = "2.0.2+${SRCPV}"
@@ -25,8 +25,9 @@ EXTRA_OECONF += "--enable-install-tests"
 # xattr support creates an additional compile-time dependency on acl because
 # the sys/acl.h header is needed. libacl is not needed and thus enabling xattr
 # regardless whether acl is enabled or disabled in the distro should be okay.
-PACKAGECONFIG ?= "${@bb.utils.filter('DISTRO_FEATURES', 'xattr', d)}"
-PACKAGECONFIG[xattr] = ",,acl,"
+PACKAGECONFIG ??= "${@bb.utils.filter('DISTRO_FEATURES', 'xattr', d)} lzo"
+PACKAGECONFIG[lzo] = "--with-lzo,--without-lzo,lzo"
+PACKAGECONFIG[xattr] = "--with-xattr,--without-xattr,acl"
 
 CPPFLAGS_append_riscv64  = " -pthread -D_REENTRANT"
 
-- 
2.11.0

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

[OE-core] [PATCH 3/6] packagegroup-core-tools-testapps: remove lrzsz

[Ross Burton]

We can assume that modern boards don't need X/Y/ZMODEM transfer tools for
testing purposes.  The few users that need this tool for testing can add it
themselves.

Signed-off-by: Ross Burton 
---
 meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb | 1 -
 1 file changed, 1 deletion(-)

diff --git 
a/meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb 
b/meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb
index 2b7e78911b3..a5fc1528593 100644
--- a/meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-core-tools-testapps.bb
@@ -38,7 +38,6 @@ X11TOOLS = "\
 
 RDEPENDS_${PN} = "\
 blktool \
-lrzsz \
 ${KEXECTOOLS} \
 alsa-utils-amixer \
 alsa-utils-aplay \
-- 
2.11.0

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

[OE-core] [PATCH 2/6] packagegroup-self-hosted: remove lrzsz

[Ross Burton]

A X/Y/ZMODEM transfer tool is not a required package for self-hosted builds.

Signed-off-by: Ross Burton 
---
 meta/recipes-core/packagegroups/packagegroup-self-hosted.bb | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb 
b/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
index 9a6721ce031..b296db5457e 100644
--- a/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
+++ b/meta/recipes-core/packagegroups/packagegroup-self-hosted.bb
@@ -128,7 +128,6 @@ RDEPENDS_packagegroup-self-hosted-extended = "\
 libaio \
 libusb1 \
 libxml2 \
-lrzsz \
 lsof \
 lzo \
 man \
-- 
2.11.0

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

[OE-core] [PATCH 1/6] pango: fix markup-parse test case

[Ross Burton]

The test case expects that printf(0x0) is (null) but for us this it is (NULL).

Use case-insensitive diff as the rest of the tests don't care about case.

Signed-off-by: Ross Burton 
---
 .../pango/pango/insensitive-diff.patch | 28 ++
 meta/recipes-graphics/pango/pango_1.42.4.bb|  3 ++-
 2 files changed, 30 insertions(+), 1 deletion(-)
 create mode 100644 meta/recipes-graphics/pango/pango/insensitive-diff.patch

diff --git a/meta/recipes-graphics/pango/pango/insensitive-diff.patch 
b/meta/recipes-graphics/pango/pango/insensitive-diff.patch
new file mode 100644
index 000..faaa96133f9
--- /dev/null
+++ b/meta/recipes-graphics/pango/pango/insensitive-diff.patch
@@ -0,0 +1,28 @@
+Do case-insensitive diffs as the test is sensitive as to whether 0x0 is printed
+as (null) or (NULL).
+
+Upstream-Status: Submitted 
[https://gitlab.gnome.org/GNOME/pango/merge_requests/44]
+Signed-off-by: Ross Burton 
+
+diff --git a/tests/markup-parse.c b/tests/markup-parse.c
+index 633f3e89..36d2c776 100644
+--- a/tests/markup-parse.c
 b/tests/markup-parse.c
+@@ -217,7 +217,7 @@ diff_with_file (const char  *file1,
+ GString *string,
+ GError **error)
+ {
+-  const char *command[] = { "diff", "-u", file1, NULL, NULL };
++  const char *command[] = { "diff", "-u", "-i", file1, NULL, NULL };
+   char *diff, *tmpfile;
+   int fd;
+ 
+@@ -237,7 +237,7 @@ diff_with_file (const char  *file1,
+   goto done;
+ }
+   close (fd);
+-  command[3] = tmpfile;
++  command[4] = tmpfile;
+ 
+   /* run diff command */
+   g_spawn_sync (NULL, (char **)command, NULL, G_SPAWN_SEARCH_PATH, NULL, 
NULL, , NULL, NULL, error);
diff --git a/meta/recipes-graphics/pango/pango_1.42.4.bb 
b/meta/recipes-graphics/pango/pango_1.42.4.bb
index e181bcb5005..d3357f82de9 100644
--- a/meta/recipes-graphics/pango/pango_1.42.4.bb
+++ b/meta/recipes-graphics/pango/pango_1.42.4.bb
@@ -16,7 +16,8 @@ GNOMEBASEBUILDCLASS = "meson"
 inherit gnomebase gtk-doc ptest-gnome upstream-version-is-even 
gobject-introspection
 
 SRC_URI += "file://run-ptest \
-"
+file://insensitive-diff.patch"
+
 SRC_URI[archive.md5sum] = "deb171a31a3ad76342d5195a1b5bbc7c"
 SRC_URI[archive.sha256sum] = 
"1d2b74cd63e8bd41961f2f8d952355aa0f9be6002b52c8aa7699d9f5da597c9d"
 
-- 
2.11.0

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

Re: [OE-core] [PATCH 3/4] packagegroup-core-base-utils: Introduce new base-utils packagegroup

[Mark Hatle]

On 3/1/19 4:26 PM, Tom Rini wrote:
> This packagegroup is intended to provide all of the functionality found
> in other VIRTUAL-RUNTIME_base-utils providers through full packages
> rather than an all-on-one package.  Document how to make use of this
> in local.conf.sample.extended.  Introduce VIRTUAL-RUNTIME-vim and
> default this to vim-tiny to allow for a differently features vim to be
> used instead.
> 
> Signed-off-by: Tom Rini 
> ---
>  meta/conf/local.conf.sample.extended   |  8 
>  .../packagegroups/packagegroup-core-base-utils.bb  | 49 
> ++
>  2 files changed, 57 insertions(+)
>  create mode 100644 
> meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
> 
> diff --git a/meta/conf/local.conf.sample.extended 
> b/meta/conf/local.conf.sample.extended
> index 8062e4c5721b..d00221893ad9 100644
> --- a/meta/conf/local.conf.sample.extended
> +++ b/meta/conf/local.conf.sample.extended
> @@ -384,3 +384,11 @@
>  #VIRTUAL-RUNTIME_login_manager = "shadow-base"
>  #VIRTUAL-RUNTIME_init_manager = "systemd"
>  #VIRTUAL-RUNTIME_initscripts = "systemd-compat-units"
> +
> +#
> +# Use a full set of packages instead of busybox for base utils
> +#
> +#PREFERRED_PROVIDER_base-utils = "packagegroup-core-base-utils"
> +#VIRTUAL-RUNTIME_base-utils = "packagegroup-core-base-utils"
> +#VIRTUAL-RUNTIME_base-utils-hwclock = "util-linux-hwclock"
> +#VIRTUAL-RUNTIME_base-utils-syslog = "syslog"
> diff --git 
> a/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb 
> b/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
> new file mode 100644
> index ..061dade0bc03
> --- /dev/null
> +++ b/meta/recipes-extended/packagegroups/packagegroup-core-base-utils.bb
> @@ -0,0 +1,49 @@
> +#
> +# Copyright (C) 2019 Konsulko Group
> +#
> +
> +SUMMARY = "Full-featured set of base utils"
> +DESCRIPTION = "Package group bringing in packages needed to provide much of 
> the base utils type functionality found in busybox"
> +
> +inherit packagegroup
> +
> +VIRTUAL-RUNTIME_vim ?= "vim-tiny"
> +
> +RDEPENDS_${PN} = "\
> +base-passwd \
> +bash \
> +bind-utils \
> +bzip2 \
> +coreutils \
> +cpio \
> +dhcp-client \
> +diffutils \
> +e2fsprogs \
> +ed \
> +file \
> +findutils \
> +gawk \
> +grep \
> +gzip \
> +inetutils \
> +iproute2 \
> +kmod \
> +less \
> +lzip \
> +ncurses-tools \
> +net-tools \
> +parted \
> +patch \
> +procps \
> +psmisc \
> +sed \
> +shadow-base \
> +tar \
> +time \
> +unzip \
> +util-linux \
> +${VIRTUAL-RUNTIME_vim} \
> +wget \
> +which \
> +xz \
> +"
> 

Sorry missed this earlier.  On note, we have a similar recipe in our local 
layer.

I figured you may want to look at it to determine if anything might be missing.

https://github.com/WindRiver-OpenSourceLabs/wrlinux/blob/master-wr/recipes-base/packagegroups/packagegroup-busybox-replacement.bb

The parts I see missing (when comparing them) are:

bc
debianutils-run-parts
dhcp-server (likely not needed)
dpkg-start-stop
ifupdown
kbd

In some configurations we also add "fbset, netcat, and usleep" as well... but I
think the reasonable approach is to match the default busybox configuration to
whatever is in this list.

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

Re: [OE-core] [PATCH 2/4] inetutils: Import version 1.9.4

[Tom Rini]

On Mon, Mar 04, 2019 at 09:47:57AM +, André Draszik wrote:
> On Fri, 2019-03-01 at 17:26 -0500, Tom Rini wrote:
> > In order to have more robust stand-alone network tools in oe-core, bring
> > in vim from meta-openembedded/meta-networking.  This imports the recipes
>  ^^^
> 
> vim for networking? :-)

Noted, fixed for next round.  Thanks!

-- 
Tom


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

Re: [OE-core] sstate hash equivalence breaks rm_work

[Tom Rini]

On Mon, Mar 04, 2019 at 04:21:48PM +, Richard Purdie wrote:
> On Mon, 2019-03-04 at 11:14 -0500, Tom Rini wrote:
> > On Mon, Mar 04, 2019 at 04:06:47PM +, Richard Purdie wrote:
> > > You mean like the BB_MIN_VERSION variable:
> > > 
> > > http://git.yoctoproject.org/cgit.cgi/poky/tree/meta/conf/sanity.conf#n6
> > > 
> > > ?
> > > 
> > > :)
> > > 
> > > The changes were intended to be backwards compatible hence it
> > > wasn't
> > > bumped. Clearly something isn't playing as intended...
> > 
> > Yes, that.  I think we haven't fully used that well historically
> > since I can only ever recall *kaboom* when using too old of a bitbake
> > rather than a nice sane error message.  Looking at
> > https://wiki.yoctoproject.org/wiki/Releases and then some quick git
> > history, we also must have had that mechanism available when I last
> > ran into something like that even for the fairly ancient jumping
> > around I've had to do a time or three.
> 
> That variable has been around for a long time. It does get incremented
> when we do nasty things which we know break things. If something sneaks
> in, its more problematic. We do try and avoid API breakage.
> 
> > In fact, can we make BB_MIN_VERSION match the table found there?  It
> > doesn't look like we have resources / time to test things on both
> > current and min versions, so we should I think have the min match
> > what we say should be used.
> 
> The problem is more "test what?" on the min version. In this case its
> rm_work which triggers it, the bugs tend to be unusual corner cases
> which our test matrix, vast as it is might not catch. Case in hand, we
> don't test rm_work on the autobuilder.
> 
> I'm torn on forcing people to update, I've had a lot of complaints
> about being being forced when they didn't need to in the past. Its also
> true that we're not as backwards compatible as we could be, i.e. using
> a modern bitbake on old metadata isn't always guaranteed either due to
> the same kinds of problems :(.
> 
> By this line of argument, we'd bump the minor bitbake version for every
> bug fix and force people to it, just to "protect" them from themselves
> :/

So, what's worse, people hit a bug that's fixed in a newer version, or
when people upgrade one set of branches they have to update another
repository too?

Honestly, I could see the "but why do I have to update bitbake too?"
complaint being more reasonable back in the early days after we moved to
proper layers.  Now that it's not just oe-core/poky but also
meta-openembedded and meta-$BSP and maybe 2-5 other layers you also need
to bump from branch to branch, adding bitbake to that mix seems a lot
less of a big deal.

I thought about this and re-drafted something a few times now.  I think
perhaps a happy compromise would be to move the BB_MIN_VERSION check
from a hard-failure to the same level of warning we do on
SANITY_TESTED_DISTROS, and make it track the suggested version for a
release.  This will both help to catch bugs that have been fixed already
but still let people that know what they want (or don't want) to do, do
it.

-- 
Tom


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

Re: [OE-core] [PATCH] libcomps: put PV in filename

[Burton, Ross]

On Mon, 4 Mar 2019 at 15:12, Richard Purdie
 wrote:
> I suspect we have this issue in a few cases and having PV next to the
> SRCREV may prompt people to keep things more in sync?

I'm sure we do, but I spotted this one.  The AUH will do the prompting
so I'm not sure what else needs to be kept in sync.

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

Re: [OE-core] sstate hash equivalence breaks rm_work

[Richard Purdie]

On Mon, 2019-03-04 at 11:14 -0500, Tom Rini wrote:
> On Mon, Mar 04, 2019 at 04:06:47PM +, Richard Purdie wrote:
> > You mean like the BB_MIN_VERSION variable:
> > 
> > http://git.yoctoproject.org/cgit.cgi/poky/tree/meta/conf/sanity.conf#n6
> > 
> > ?
> > 
> > :)
> > 
> > The changes were intended to be backwards compatible hence it
> > wasn't
> > bumped. Clearly something isn't playing as intended...
> 
> Yes, that.  I think we haven't fully used that well historically
> since I can only ever recall *kaboom* when using too old of a bitbake
> rather than a nice sane error message.  Looking at
> https://wiki.yoctoproject.org/wiki/Releases and then some quick git
> history, we also must have had that mechanism available when I last
> ran into something like that even for the fairly ancient jumping
> around I've had to do a time or three.

That variable has been around for a long time. It does get incremented
when we do nasty things which we know break things. If something sneaks
in, its more problematic. We do try and avoid API breakage.

> In fact, can we make BB_MIN_VERSION match the table found there?  It
> doesn't look like we have resources / time to test things on both
> current and min versions, so we should I think have the min match
> what we say should be used.

The problem is more "test what?" on the min version. In this case its
rm_work which triggers it, the bugs tend to be unusual corner cases
which our test matrix, vast as it is might not catch. Case in hand, we
don't test rm_work on the autobuilder.

I'm torn on forcing people to update, I've had a lot of complaints
about being being forced when they didn't need to in the past. Its also
true that we're not as backwards compatible as we could be, i.e. using
a modern bitbake on old metadata isn't always guaranteed either due to
the same kinds of problems :(.

By this line of argument, we'd bump the minor bitbake version for every
bug fix and force people to it, just to "protect" them from themselves
:/

Cheers,

Richard




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

Re: [OE-core] sstate hash equivalence breaks rm_work

[Joshua Watt]

On Mon, Mar 4, 2019, 10:06 AM Richard Purdie <
richard.pur...@linuxfoundation.org> wrote:

> On Mon, 2019-03-04 at 10:58 -0500, Tom Rini wrote:
> > Ah, so "poky" got me thinking this morning.  I was just on bitbake
> > v1.40
> > (since I usually use thud, and that's the version for that).  Moving
> > to
> > bitbake master/tip and I don't see the problem anymore.
> >
> > Is it possible to have some logic within OE to check the version of
> > bitbake and throw an error / warning?
>
> You mean like the BB_MIN_VERSION variable:
>
> http://git.yoctoproject.org/cgit.cgi/poky/tree/meta/conf/sanity.conf#n6
>
> ?
>
> :)
>
> The changes were intended to be backwards compatible hence it wasn't
> bumped. Clearly something isn't playing as intended...
>

Ah, older bitbake isn't going to set BB_UNIHASH, which I bet is what breaks
it. I bet if we add:

 BB_UNIHASH ?= "${BB_TASKHASH}"

somewhere (bitbake.conf?) it would fix it.


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

Re: [OE-core] sstate hash equivalence breaks rm_work

[Tom Rini]

On Mon, Mar 04, 2019 at 04:06:47PM +, Richard Purdie wrote:
> On Mon, 2019-03-04 at 10:58 -0500, Tom Rini wrote:
> > Ah, so "poky" got me thinking this morning.  I was just on bitbake
> > v1.40
> > (since I usually use thud, and that's the version for that).  Moving
> > to
> > bitbake master/tip and I don't see the problem anymore.
> > 
> > Is it possible to have some logic within OE to check the version of
> > bitbake and throw an error / warning?
> 
> You mean like the BB_MIN_VERSION variable:
> 
> http://git.yoctoproject.org/cgit.cgi/poky/tree/meta/conf/sanity.conf#n6
> 
> ?
> 
> :)
>
> The changes were intended to be backwards compatible hence it wasn't
> bumped. Clearly something isn't playing as intended...

Yes, that.  I think we haven't fully used that well historically since I
can only ever recall *kaboom* when using too old of a bitbake rather
than a nice sane error message.  Looking at
https://wiki.yoctoproject.org/wiki/Releases and then some quick git
history, we also must have had that mechanism available when I last ran
into something like that even for the fairly ancient jumping around I've
had to do a time or three.

In fact, can we make BB_MIN_VERSION match the table found there?  It
doesn't look like we have resources / time to test things on both
current and min versions, so we should I think have the min match what
we say should be used.

-- 
Tom


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

[OE-core] Reminder: Yocto Project Technical Team Meeting @ Monthly from 8am to 8:30am on the first Tuesday (PST)

[sjolley.yp.pm]

All,

 

Just a reminder we will hold the monthly Yocto Project Technical Meeting at
8am PST tomorrow. (3/5)  

 

Yocto Project Technical Team Meeting: We encourage people attending the
meeting to logon and announce themselves on the Yocto Project IRC chancel
during the meeting (optional):

Yocto IRC: http://webchat.freenode.net/?channels=#yocto

 

Wiki: https://www.yoctoproject.org/public-virtual-meetings/

 

WhenMonthly from 8am to 8:30am on the first Tuesday Pacific Time

Where   Zoom Meeting: https://zoom.us/j/990892712 

 

I will be tracking the minutes at:
https://docs.google.com/document/d/1Y5IIuE-z0Ykdl-DwuzUJh52flOZuhN_TSAfw2tdU
9pg/edit?ts=5c06b22d  Please request access if you want to assist in editing
them.  The world should have view access. 

 

Thanks,

 

Stephen K. Jolley

Yocto Project Program Manager

*Cell:(208) 244-4460

* Email:  sjolley.yp...@gmail.com
 

 

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

Re: [OE-core] sstate hash equivalence breaks rm_work

[Richard Purdie]

On Mon, 2019-03-04 at 10:58 -0500, Tom Rini wrote:
> Ah, so "poky" got me thinking this morning.  I was just on bitbake
> v1.40
> (since I usually use thud, and that's the version for that).  Moving
> to
> bitbake master/tip and I don't see the problem anymore.
> 
> Is it possible to have some logic within OE to check the version of
> bitbake and throw an error / warning?

You mean like the BB_MIN_VERSION variable:

http://git.yoctoproject.org/cgit.cgi/poky/tree/meta/conf/sanity.conf#n6

?

:)

The changes were intended to be backwards compatible hence it wasn't
bumped. Clearly something isn't playing as intended...

Cheers,

Richard

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

[OE-core] Yocto Project Unassigned Bugs - Help Needed

[sjolley.yp.pm]

All,

 

The triage team meets weekly and does its best to handle the bugs reported
into the Bugzilla. The number of people attending that meeting has fallen,
as have the number of people available to help fix bugs. One of the things
we hear users report is they don't know how to help. We (the triage team)
are therefore going to start reporting out the currently 296 unassigned
bugs.

 

We're hoping people may be able to spare some time now and again to help out
with these.

 

Bugs are split into two types, "true bugs" where things don't work as they
should and "enhancements" which are features we'd want to add to the system.

 

There are also roughly four different "priority" classes right now, "2.7",
"2.8", "2.99" and "Future", the more pressing/urgent issues being in "2.7"
and then "2.8".

 

Please review this link and if a bug is something you would be able to help
with either take ownership of the bug, or send me
(stephen.k.jol...@intel.com  ) an e-mail
with the bug number you would like and I will assign it to you (please make
sure you have a Bugzilla account).

 

The list is at:
https://wiki.yoctoproject.org/wiki/Bug_Triage#Unassigned_Bugs

 

Thanks,

 

Stephen K. Jolley

Yocto Project Program Manager

*Cell:(208) 244-4460

* Email:  sjolley.yp...@gmail.com
 

 

 

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

Re: [OE-core] sstate hash equivalence breaks rm_work

[Tom Rini]

On Sun, Mar 03, 2019 at 04:11:36PM -0600, Joshua Watt wrote:
> On Sun, Mar 3, 2019 at 3:25 PM Tom Rini  wrote:
> >
> > On Sun, Mar 03, 2019 at 03:21:12PM -0600, Joshua Watt wrote:
> > > On Sun, Mar 3, 2019, 2:47 PM Tom Rini  wrote:
> > >
> > > > Hey all,
> > > >
> > > > As part of the packagegroup-core-base-utils series I noticed that
> > > > with rm_work enabled like I usually have, every build was starting over
> > > > with rebuilding linux-libc-headers and going down from there.  I
> > > > finished bisecting this now and it comes down to:
> > >
> > > Just for reference, are you trying to use hash equivalence? It sounds like
> > > not, but I would like to make sure.
> >
> > I am not.
> >
> > > Also, it there an AB build that uses rm_work that would have seen this?
> >
> > OK, so I see I forgot a step that was clear to me, but, uh, not in the
> > email.  What breaks is for example:
> > bitbake core-image-minimal && bitbake core-image-minimal
> > That second build, rather than see "nothing to do" (or at least, almost
> > nothing to do) says "Oh, I need to build linux-libc-headers from
> > scratch" and possibly other stuff too, but that's the first thing I
> > notice.
> 
> Ok. I've started a build to test it. Would you mind opening a Bugzilla
> for it and assigning it to me?
> 
> For posterity, the poky commit SHA is adc37721a86c

Ah, so "poky" got me thinking this morning.  I was just on bitbake v1.40
(since I usually use thud, and that's the version for that).  Moving to
bitbake master/tip and I don't see the problem anymore.

Is it possible to have some logic within OE to check the version of
bitbake and throw an error / warning?

-- 
Tom


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

[OE-core] [PATCH 6/6] local.conf.sample: remove support for building against host libsdl

[Alexander Kanavin]

Signed-off-by: Alexander Kanavin 
---
 meta-poky/conf/local.conf.sample | 4 
 1 file changed, 4 deletions(-)

diff --git a/meta-poky/conf/local.conf.sample b/meta-poky/conf/local.conf.sample
index 2775e15d748..00795ae6a3f 100644
--- a/meta-poky/conf/local.conf.sample
+++ b/meta-poky/conf/local.conf.sample
@@ -239,12 +239,8 @@ BB_DISKMON_DIRS ??= "\
 #
 # By default qemu will build with a builtin VNC server where graphical output 
can be
 # seen. The two lines below enable the SDL and Gtk UI frontends as well.
-#
-# By default libsdl2-native will be built, if you want to use your host's 
libSDL instead
-# of the minimal libsdl built by libsdl2-native then uncomment the 
ASSUME_PROVIDED line below.
 PACKAGECONFIG_append_pn-qemu-system-native = " sdl gtk+"
 PACKAGECONFIG_append_pn-nativesdk-qemu = " sdl gtk+"
-#ASSUME_PROVIDED += "libsdl2-native"
 
 # CONF_VERSION is increased each time build/conf/ changes incompatibly and is 
used to
 # track the version of this file when it was generated. This can safely be 
ignored if
-- 
2.17.1

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

[OE-core] [PATCH 5/6] local.conf.sample: enable gtk+ frontend in addition to sdl

[Alexander Kanavin]

As SDL was found to be buggy for virgl-based GL passthrough,
gtk+ is enabled to allow that use case.

Signed-off-by: Alexander Kanavin 
---
 meta-poky/conf/local.conf.sample | 11 ++-
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/meta-poky/conf/local.conf.sample b/meta-poky/conf/local.conf.sample
index 9068e567dcd..2775e15d748 100644
--- a/meta-poky/conf/local.conf.sample
+++ b/meta-poky/conf/local.conf.sample
@@ -238,11 +238,12 @@ BB_DISKMON_DIRS ??= "\
 # Qemu configuration
 #
 # By default qemu will build with a builtin VNC server where graphical output 
can be
-# seen. The two lines below enable the SDL backend too. By default 
libsdl2-native will
-# be built, if you want to use your host's libSDL instead of the minimal 
libsdl built
-# by libsdl2-native then uncomment the ASSUME_PROVIDED line below.
-PACKAGECONFIG_append_pn-qemu-system-native = " sdl"
-PACKAGECONFIG_append_pn-nativesdk-qemu = " sdl"
+# seen. The two lines below enable the SDL and Gtk UI frontends as well.
+#
+# By default libsdl2-native will be built, if you want to use your host's 
libSDL instead
+# of the minimal libsdl built by libsdl2-native then uncomment the 
ASSUME_PROVIDED line below.
+PACKAGECONFIG_append_pn-qemu-system-native = " sdl gtk+"
+PACKAGECONFIG_append_pn-nativesdk-qemu = " sdl gtk+"
 #ASSUME_PROVIDED += "libsdl2-native"
 
 # CONF_VERSION is increased each time build/conf/ changes incompatibly and is 
used to
-- 
2.17.1

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

[OE-core] [PATCH 4/6] qemu: remove support for building against host sdl

[Alexander Kanavin]

This hasn't been the default for a long time (as some distros don't
support it), and with gtk+ being the new default shouldn't
be needed at all.

Signed-off-by: Alexander Kanavin 
---
 meta/recipes-devtools/qemu/qemu.inc | 8 
 1 file changed, 8 deletions(-)

diff --git a/meta/recipes-devtools/qemu/qemu.inc 
b/meta/recipes-devtools/qemu/qemu.inc
index 985289f5426..a9d28bf5a38 100644
--- a/meta/recipes-devtools/qemu/qemu.inc
+++ b/meta/recipes-devtools/qemu/qemu.inc
@@ -80,14 +80,6 @@ export LIBTOOL="${HOST_SYS}-libtool"
 
 B = "${WORKDIR}/build"
 
-do_configure_prepend_class-native() {
-   # Append build host pkg-config paths for native target since the host 
may provide sdl
-   BHOST_PKGCONFIG_PATH=$(PATH=/usr/bin:/bin pkg-config --variable pc_path 
pkg-config || echo "")
-   if [ ! -z "$BHOST_PKGCONFIG_PATH" ]; then
-   export PKG_CONFIG_PATH=$PKG_CONFIG_PATH:$BHOST_PKGCONFIG_PATH
-   fi
-}
-
 do_configure() {
 ${S}/configure ${EXTRA_OECONF}
 }
-- 
2.17.1

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

[OE-core] [PATCH 3/6] qemu: remove sdl option from PACKAGECONFIG default

[Alexander Kanavin]

This removes confusion over where qemu frontends are enabled for
native/nativesdk builds: currently they are also set in local.conf.sample
from poky distro, and with this change that becomes the only place.

Signed-off-by: Alexander Kanavin 
---
 meta/recipes-devtools/qemu/qemu_3.1.0.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/qemu/qemu_3.1.0.bb 
b/meta/recipes-devtools/qemu/qemu_3.1.0.bb
index 26a33b4d860..8457e2c2f87 100644
--- a/meta/recipes-devtools/qemu/qemu_3.1.0.bb
+++ b/meta/recipes-devtools/qemu/qemu_3.1.0.bb
@@ -17,6 +17,6 @@ PACKAGECONFIG ??= " \
 fdt sdl gtk+ kvm \
 ${@bb.utils.filter('DISTRO_FEATURES', 'alsa xen', d)} \
 "
-PACKAGECONFIG_class-nativesdk ??= "fdt sdl kvm virglrenderer glx"
+PACKAGECONFIG_class-nativesdk ??= "fdt kvm virglrenderer glx"
 
 
-- 
2.17.1

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

[OE-core] [PATCH 2/6] qemu: add a gtk+ frontend to target builds

[Alexander Kanavin]

sdl frontend remains enabled and available.

Signed-off-by: Alexander Kanavin 
---
 meta/recipes-devtools/qemu/qemu_3.1.0.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/qemu/qemu_3.1.0.bb 
b/meta/recipes-devtools/qemu/qemu_3.1.0.bb
index 3c404d408d0..26a33b4d860 100644
--- a/meta/recipes-devtools/qemu/qemu_3.1.0.bb
+++ b/meta/recipes-devtools/qemu/qemu_3.1.0.bb
@@ -14,7 +14,7 @@ do_install_append_class-nativesdk() {
 }
 
 PACKAGECONFIG ??= " \
-fdt sdl kvm \
+fdt sdl gtk+ kvm \
 ${@bb.utils.filter('DISTRO_FEATURES', 'alsa xen', d)} \
 "
 PACKAGECONFIG_class-nativesdk ??= "fdt sdl kvm virglrenderer glx"
-- 
2.17.1

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

[OE-core] [PATCH 1/6] qemu: enable virglrenderer and glx options for qemu-system-native and nativesdk-qemu

[Alexander Kanavin]

Note that to actually use accelerated GL passthrough, there are two options

1) a suitable frontend need to be also enabled - gtk+ seems to work well,
sdl was found to be buggy.

2) it is also possible to render off-screen with -display egl-headless option,
and see the output with a VNC viewer (for which, qemu needs to be started
with a VNC server):

$ runqemu kvm egl-headless publicvnc

Signed-off-by: Alexander Kanavin 
---
 meta/recipes-devtools/qemu/qemu-system-native_3.1.0.bb | 2 +-
 meta/recipes-devtools/qemu/qemu_3.1.0.bb   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-devtools/qemu/qemu-system-native_3.1.0.bb 
b/meta/recipes-devtools/qemu/qemu-system-native_3.1.0.bb
index 5bf528bec12..95369d7f6a2 100644
--- a/meta/recipes-devtools/qemu/qemu-system-native_3.1.0.bb
+++ b/meta/recipes-devtools/qemu/qemu-system-native_3.1.0.bb
@@ -9,7 +9,7 @@ DEPENDS = "glib-2.0-native zlib-native pixman-native 
qemu-native"
 
 EXTRA_OECONF_append = " --target-list=${@get_qemu_system_target_list(d)}"
 
-PACKAGECONFIG ??= "fdt alsa kvm"
+PACKAGECONFIG ??= "fdt alsa kvm virglrenderer glx"
 
 # Handle distros such as CentOS 5 32-bit that do not have kvm support
 PACKAGECONFIG_remove = "${@'kvm' if not 
os.path.exists('/usr/include/linux/kvm.h') else ''}"
diff --git a/meta/recipes-devtools/qemu/qemu_3.1.0.bb 
b/meta/recipes-devtools/qemu/qemu_3.1.0.bb
index 04d8bee99f2..3c404d408d0 100644
--- a/meta/recipes-devtools/qemu/qemu_3.1.0.bb
+++ b/meta/recipes-devtools/qemu/qemu_3.1.0.bb
@@ -17,6 +17,6 @@ PACKAGECONFIG ??= " \
 fdt sdl kvm \
 ${@bb.utils.filter('DISTRO_FEATURES', 'alsa xen', d)} \
 "
-PACKAGECONFIG_class-nativesdk ??= "fdt sdl kvm"
+PACKAGECONFIG_class-nativesdk ??= "fdt sdl kvm virglrenderer glx"
 
 
-- 
2.17.1

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

Re: [OE-core] [meta-oe][thud][PATCH] target-sdk-provides-dummy: add perl-module-overload

[akuster808]

On 3/3/19 8:18 PM, Oleksandr Kravchuk wrote:
> Signed-off-by: Oleksandr Kravchuk 

is this in master? What does it fix?

- armin
> ---
>  meta/recipes-core/meta/target-sdk-provides-dummy.bb | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/meta/recipes-core/meta/target-sdk-provides-dummy.bb 
> b/meta/recipes-core/meta/target-sdk-provides-dummy.bb
> index edf07c4a23..13741f0a64 100644
> --- a/meta/recipes-core/meta/target-sdk-provides-dummy.bb
> +++ b/meta/recipes-core/meta/target-sdk-provides-dummy.bb
> @@ -26,6 +26,7 @@ DUMMYPROVIDES = "\
>  perl-module-getopt-long \
>  perl-module-io-file \
>  perl-module-posix \
> +perl-module-overload \
>  perl-module-thread-queue \
>  perl-module-threads \
>  /bin/sh \

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

Re: [OE-core] [PATCH] libcomps: put PV in filename

[Richard Purdie]

On Mon, 2019-03-04 at 11:58 +, Ross Burton wrote:
> This isn't a git snapshot recipe but a release that is fetched over
> it.  For
> clarity, put the PV in the filename.
> 
> Signed-off-by: Ross Burton 
> ---
>  meta/recipes-devtools/libcomps/{libcomps_git.bb =>
> libcomps_0.10.0.bb} | 1 -
>  1 file changed, 1 deletion(-)
>  rename meta/recipes-devtools/libcomps/{libcomps_git.bb =>
> libcomps_0.10.0.bb} (98%)
> 
> diff --git a/meta/recipes-devtools/libcomps/libcomps_git.bb
> b/meta/recipes-devtools/libcomps/libcomps_0.10.0.bb
> similarity index 98%
> rename from meta/recipes-devtools/libcomps/libcomps_git.bb
> rename to meta/recipes-devtools/libcomps/libcomps_0.10.0.bb
> index ff6820851fe..1c4f3bde65e 100644
> --- a/meta/recipes-devtools/libcomps/libcomps_git.bb
> +++ b/meta/recipes-devtools/libcomps/libcomps_0.10.0.bb
> @@ -9,7 +9,6 @@ SRC_URI = "git://github.com/rpm-software-
> management/libcomps.git \
> file://0001-Add-crc32.c-to-sources-list.patch \
> "
>  
> -PV = "0.1.10"
>  SRCREV = "86a82fcd155c27092340d15a34f5c75c4da88243"

I suspect we have this issue in a few cases and having PV next to the
SRCREV may prompt people to keep things more in sync?

Cheers,

Richard


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

Re: [OE-core] [PATCH 1/5] target-sdk-provides-dummy: Extend to -dev and -src packages

[Matthias Schiffer]

On Fri, 2019-02-15 at 16:03 +, Richard Purdie wrote:
> This avoids errors when running populate_sdk under opkg:
> 
>  * Problem 1/1:
>  *   - package busybox-dev-1.30.1-r0.core2-64 requires busybox =
> 1.30.1-r0, but none of the providers can be installed
>  *
>  * Solution 1:
>  *   - allow deinstallation of target-sdk-provides-dummy-1.0-r0.sdk-
> provides-dummy-target
> 
>  * Solution 2:
>  *   - do not ask to install a package providing busybox-dev
> 
> Signed-off-by: Richard Purdie 

Hi,
is this patch going to be backported to thud?


Kind regards,
Matthias



> ---
>  meta/recipes-core/meta/target-sdk-provides-dummy.bb | 10 ++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/meta/recipes-core/meta/target-sdk-provides-dummy.bb
> b/meta/recipes-core/meta/target-sdk-provides-dummy.bb
> index 26716ace87b..60499f7b66a 100644
> --- a/meta/recipes-core/meta/target-sdk-provides-dummy.bb
> +++ b/meta/recipes-core/meta/target-sdk-provides-dummy.bb
> @@ -2,9 +2,17 @@ DUMMYARCH = "sdk-provides-dummy-target"
>  
>  DUMMYPROVIDES = "\
>  busybox \
> +busybox-dev \
> +busybox-src \
>  coreutils \
> +coreutils-dev \
> +coreutils-src \
>  bash \
> +bash-dev \
> +bash-src \
>  perl \
> +perl-dev \
> +perl-src \
>  perl-module-re \
>  perl-module-strict \
>  perl-module-vars \
> @@ -36,6 +44,8 @@ DUMMYPROVIDES = "\
>  /usr/bin/env \
>  /usr/bin/perl \
>  pkgconfig \
> +pkgconfig-dev \
> +pkgconfig-src \
>  "
>  
>  require dummy-sdk-package.inc
> -- 
> 2.20.1
> 

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

[OE-core] [PATCH] pango: fix markup-parse test case

[Ross Burton]

The test case expects that printf(0x0) is (null) but for us this it is (NULL).

Use case-insensitive diff as the rest of the tests don't care about case.

Signed-off-by: Ross Burton 
---
 .../pango/pango/insensitive-diff.patch | 28 ++
 meta/recipes-graphics/pango/pango_1.42.4.bb|  3 ++-
 2 files changed, 30 insertions(+), 1 deletion(-)
 create mode 100644 meta/recipes-graphics/pango/pango/insensitive-diff.patch

diff --git a/meta/recipes-graphics/pango/pango/insensitive-diff.patch 
b/meta/recipes-graphics/pango/pango/insensitive-diff.patch
new file mode 100644
index 000..faaa96133f9
--- /dev/null
+++ b/meta/recipes-graphics/pango/pango/insensitive-diff.patch
@@ -0,0 +1,28 @@
+Do case-insensitive diffs as the test is sensitive as to whether 0x0 is printed
+as (null) or (NULL).
+
+Upstream-Status: Submitted 
[https://gitlab.gnome.org/GNOME/pango/merge_requests/44]
+Signed-off-by: Ross Burton 
+
+diff --git a/tests/markup-parse.c b/tests/markup-parse.c
+index 633f3e89..36d2c776 100644
+--- a/tests/markup-parse.c
 b/tests/markup-parse.c
+@@ -217,7 +217,7 @@ diff_with_file (const char  *file1,
+ GString *string,
+ GError **error)
+ {
+-  const char *command[] = { "diff", "-u", file1, NULL, NULL };
++  const char *command[] = { "diff", "-u", "-i", file1, NULL, NULL };
+   char *diff, *tmpfile;
+   int fd;
+ 
+@@ -237,7 +237,7 @@ diff_with_file (const char  *file1,
+   goto done;
+ }
+   close (fd);
+-  command[3] = tmpfile;
++  command[4] = tmpfile;
+ 
+   /* run diff command */
+   g_spawn_sync (NULL, (char **)command, NULL, G_SPAWN_SEARCH_PATH, NULL, 
NULL, , NULL, NULL, error);
diff --git a/meta/recipes-graphics/pango/pango_1.42.4.bb 
b/meta/recipes-graphics/pango/pango_1.42.4.bb
index e181bcb5005..d3357f82de9 100644
--- a/meta/recipes-graphics/pango/pango_1.42.4.bb
+++ b/meta/recipes-graphics/pango/pango_1.42.4.bb
@@ -16,7 +16,8 @@ GNOMEBASEBUILDCLASS = "meson"
 inherit gnomebase gtk-doc ptest-gnome upstream-version-is-even 
gobject-introspection
 
 SRC_URI += "file://run-ptest \
-"
+file://insensitive-diff.patch"
+
 SRC_URI[archive.md5sum] = "deb171a31a3ad76342d5195a1b5bbc7c"
 SRC_URI[archive.sha256sum] = 
"1d2b74cd63e8bd41961f2f8d952355aa0f9be6002b52c8aa7699d9f5da597c9d"
 
-- 
2.11.0

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

Re: [OE-core] [PATCH v3] swig: fix build with musl

[Luca Boccassi]

On Mon, 2019-03-04 at 11:18 +, Burton, Ross wrote:
> On Mon, 4 Mar 2019 at 11:12,  wrote:
> > +mkdir -p ${WORKDIR}/build/Source/CParse
> 
> Use ${B} instead of ${WORKDIR}/build.
> 
> Ross

Thanks, please see v4

-- 
Kind regards,
Luca Boccassi


signature.asc
Description: This is a digitally signed message part
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH v4] swig: fix build with musl

[luca . boccassi]

From: Luca Boccassi 

Building with musl currently fails. The fix has been committed
upstream, so backport it.
Note that it requires running yacc again to regenerate the parser file,
so a bison-native dependency is needed. Also the upstream's configure
step creates the directory where the parser file is generated at
make-dist time, and for some reason running through bitbake does not
create it, so do it manually. All of this can be reverted once a new
swig version that includes the fix is released and rebased onto.

Signed-off-by: Luca Boccassi 
---
v2: fixed patch header
v3: added bison-native dependency and autoconf workaround
v4: switched from ${WORKDIR}/build to ${B}

 meta/recipes-devtools/swig/swig.inc   |   8 +-
 ...de-for-constant-expressions-containi.patch | 179 ++
 meta/recipes-devtools/swig/swig_3.0.12.bb |   1 +
 3 files changed, 187 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-devtools/swig/swig/0001-Fix-generated-code-for-constant-expressions-containi.patch

diff --git a/meta/recipes-devtools/swig/swig.inc 
b/meta/recipes-devtools/swig/swig.inc
index f7de40866a..00b9c5b605 100644
--- a/meta/recipes-devtools/swig/swig.inc
+++ b/meta/recipes-devtools/swig/swig.inc
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
"file://LICENSE;md5=e7807a6282784a7dde4c846626b08fc6 \
 
 SECTION = "devel"
 
-DEPENDS = "libpcre"
+DEPENDS = "libpcre bison-native"
 
 SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz"
 
@@ -42,6 +42,12 @@ EXTRA_AUTORECONF += "-I Tools/config"
 
 BBCLASSEXTEND = "native nativesdk"
 
+# necessary together with bison dependency until a new upstream version after
+# 3.0.12 includes 
0001-Fix-generated-code-for-constant-expressions-containi.patch
+do_configure_append() {
+mkdir -p ${B}/Source/CParse
+}
+
 do_install_append_class-nativesdk() {
 cd ${D}${bindir}
 ln -s swig swig2.0
diff --git 
a/meta/recipes-devtools/swig/swig/0001-Fix-generated-code-for-constant-expressions-containi.patch
 
b/meta/recipes-devtools/swig/swig/0001-Fix-generated-code-for-constant-expressions-containi.patch
new file mode 100644
index 00..f4ed782a82
--- /dev/null
+++ 
b/meta/recipes-devtools/swig/swig/0001-Fix-generated-code-for-constant-expressions-containi.patch
@@ -0,0 +1,179 @@
+From 90ba174fcea1618af57aa594199541d47a89b7f6 Mon Sep 17 00:00:00 2001
+From: William S Fulton 
+Date: Sun, 17 Sep 2017 19:02:55 +0100
+Subject: [PATCH 1/2] Fix generated code for constant expressions containing
+ wchar_t L literals.
+
+Such as:
+  # define __WCHAR_MAX(0x7fff + L'\0')
+
+Reported on swig-user mailing list.
+---
+ CHANGES.current | 5 +
+ Examples/test-suite/csharp/preproc_constants_c_runme.cs | 3 ++-
+ Examples/test-suite/csharp/preproc_constants_runme.cs   | 2 ++
+ Examples/test-suite/d/preproc_constants_c_runme.1.d | 2 ++
+ Examples/test-suite/d/preproc_constants_c_runme.2.d | 2 ++
+ Examples/test-suite/d/preproc_constants_runme.1.d   | 2 ++
+ Examples/test-suite/d/preproc_constants_runme.2.d   | 2 ++
+ Examples/test-suite/php/preproc_constants_c_runme.php   | 2 ++
+ Examples/test-suite/php/preproc_constants_runme.php | 2 ++
+ Examples/test-suite/php5/preproc_constants_c_runme.php  | 2 ++
+ Examples/test-suite/php5/preproc_constants_runme.php| 2 ++
+ Examples/test-suite/preproc_constants.i | 3 +++
+ Source/CParse/parser.y  | 2 +-
+ 13 files changed, 29 insertions(+), 2 deletions(-)
+
+Upstream-Status: Backport
+[https://github.com/swig/swig/commit/90ba174fcea1618af57aa594199541d47a89b7f6]
+Signed-off-by: Luca Boccassi 
+
+diff --git a/Examples/test-suite/csharp/preproc_constants_c_runme.cs 
b/Examples/test-suite/csharp/preproc_constants_c_runme.cs
+index 76c684d..1c28e49 100644
+--- a/Examples/test-suite/csharp/preproc_constants_c_runme.cs
 b/Examples/test-suite/csharp/preproc_constants_c_runme.cs
+@@ -61,7 +61,8 @@ public class runme {
+ assert( typeof(int) == preproc_constants_c.EXPR_LAND.GetType() );
+ assert( typeof(int) == preproc_constants_c.EXPR_LOR.GetType() );
+ assert( typeof(double) == preproc_constants_c.EXPR_CONDITIONAL.GetType() 
);
+-
++assert( typeof(int) == preproc_constants_c.EXPR_WCHAR_MAX.GetType() );
++assert( typeof(int) == preproc_constants_c.EXPR_WCHAR_MIN.GetType() );
+   }
+   static void assert(bool assertion) {
+ if (!assertion)
+diff --git a/Examples/test-suite/csharp/preproc_constants_runme.cs 
b/Examples/test-suite/csharp/preproc_constants_runme.cs
+index 9fae591..6b02e30 100644
+--- a/Examples/test-suite/csharp/preproc_constants_runme.cs
 b/Examples/test-suite/csharp/preproc_constants_runme.cs
+@@ -60,6 +60,8 @@ public class runme {
+ assert( typeof(bool) == preproc_constants.EXPR_LAND.GetType() );
+ assert( typeof(bool) == preproc_constants.EXPR_LOR.GetType() );
+ assert( typeof(double) == preproc_constants.EXPR_CONDITIONAL.GetType() );
++assert( 

Re: [OE-core] sstate hash equivalence breaks rm_work

[Tom Rini]

On Mon, Mar 04, 2019 at 10:17:21AM +, Burton, Ross wrote:
> On Sun, 3 Mar 2019 at 22:52, Tom Rini  wrote:
> > SSTATE_DIR = 
> > "${HOME}/work/OE/sstate-cache-${DISTRO_VERSION}${TARGET_VENDOR}"
> 
> FWIW there isn't much point in isolating sstate like that.  You'll
> just rebuild stuff that would have been available.

Martin already chimed in, but this is my "copy this to every project"
site.conf file.  I do modify that base path a little bit for per
project,  But easily removable state is good and important, especially
when chasing "recipe for proprietary thing behaves badly" bugs.

-- 
Tom


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

[OE-core] [PATCH] libcomps: put PV in filename

[Ross Burton]

This isn't a git snapshot recipe but a release that is fetched over it.  For
clarity, put the PV in the filename.

Signed-off-by: Ross Burton 
---
 meta/recipes-devtools/libcomps/{libcomps_git.bb => libcomps_0.10.0.bb} | 1 -
 1 file changed, 1 deletion(-)
 rename meta/recipes-devtools/libcomps/{libcomps_git.bb => libcomps_0.10.0.bb} 
(98%)

diff --git a/meta/recipes-devtools/libcomps/libcomps_git.bb 
b/meta/recipes-devtools/libcomps/libcomps_0.10.0.bb
similarity index 98%
rename from meta/recipes-devtools/libcomps/libcomps_git.bb
rename to meta/recipes-devtools/libcomps/libcomps_0.10.0.bb
index ff6820851fe..1c4f3bde65e 100644
--- a/meta/recipes-devtools/libcomps/libcomps_git.bb
+++ b/meta/recipes-devtools/libcomps/libcomps_0.10.0.bb
@@ -9,7 +9,6 @@ SRC_URI = 
"git://github.com/rpm-software-management/libcomps.git \
file://0001-Add-crc32.c-to-sources-list.patch \
"
 
-PV = "0.1.10"
 SRCREV = "86a82fcd155c27092340d15a34f5c75c4da88243"
 
 S = "${WORKDIR}/git"
-- 
2.11.0

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

Re: [OE-core] [PATCH 1/1] send-error-report: Add --no-ssl to use http protocol

[Yang, Liezhi]

Thanks, looks good, I will update it.

// Robert

Sent from mobile phone

> 在 2019年3月4日，18:57，Burton, Ross  写道：
> 
>> On Mon, 4 Mar 2019 at 10:30, Robert Yang  wrote:
>> +arg_parse.add_argument("--no-ssl",
>> +   help="Use http instead of https protocol",
>> +   action="store_true")
> 
> If you do this instead:
> 
> arg_parse.add_argument("--no-ssl", help="Use http instead of https protocol",
>  dest="protocol", action="store_const", const="http", default="https")
> 
> Then you can just use args.protocol instead of having to pass around
> the protocol name.
> 
> Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] openssl: Add cryptodev-linux PACKAGECONFIG

[Ovidiu Panait]

The old bsd cryptodev engine was removed in
https://github.com/openssl/openssl/pull/3699
and the new one added in:
https://github.com/openssl/openssl/pull/3744

It can be enabled by configuring with "enable-devcryptoeng".

Signed-off-by: Ovidiu Panait 
---
 meta/recipes-connectivity/openssl/openssl_1.1.1a.bb | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb 
b/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb
index 02fd66910e..4a626a4fcd 100644
--- a/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb
+++ b/meta/recipes-connectivity/openssl/openssl_1.1.1a.bb
@@ -28,6 +28,12 @@ SRC_URI[sha256sum] = 
"fc20130f8b7cbd2fb918b2f14e2f429e109c31ddd0fb38fc5d71d9ffed
 
 inherit lib_package multilib_header ptest
 
+PACKAGECONFIG ?= ""
+PACKAGECONFIG_class-native = ""
+PACKAGECONFIG_class-nativesdk = ""
+
+PACKAGECONFIG[cryptodev-linux] = 
"enable-devcryptoeng,disable-devcryptoeng,cryptodev-linux"
+
 B = "${WORKDIR}/build"
 do_configure[cleandirs] = "${B}"
 
-- 
2.20.1

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

Re: [OE-core] [PATCH v3] swig: fix build with musl

[Burton, Ross]

On Mon, 4 Mar 2019 at 11:12,  wrote:
> +mkdir -p ${WORKDIR}/build/Source/CParse

Use ${B} instead of ${WORKDIR}/build.

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

Re: [OE-core] [PATCH v2] swig: fix build with musl

[Luca Boccassi]

On Sat, 2019-03-02 at 23:03 +, Richard Purdie wrote:
> On Sat, 2019-03-02 at 18:12 +, Richard Purdie wrote:
> > On Fri, 2019-03-01 at 16:36 +, luca.bocca...@gmail.com wrote:
> > > From: Luca Boccassi 
> > > 
> > > Building with musl currently fails. The fix has been committed
> > > upstream, so backport it.
> > > Note that it requires the recent change to use autoconf's
> > > do_configure
> > > as it modifies a bison file which is used at make-dist time, so
> > > it
> > > triggers an autoconf rerun which was not working previously due
> > > to
> > > the
> > > custom do_configure step that was removed recently by:
> > > a9b1f9eca963 ("swig: Fix configure failure")
> > > 
> > > Signed-off-by: Luca Boccassi 
> > > ---
> > > v2: fixed upstream metadata format in patch header
> > 
> > Fails in testing:
> > 
> > https://autobuilder.yoctoproject.org/typhoon/#/builders/15/builds/585
> > 
> > May need a bison-native DEPENDS?

It does require bison, and I feel very silly as I have added it to the
internal build and forgot to include it in the patch! Sorry :-)

> I tried that, it fails with other problems :(
> 
> For example:
> https://autobuilder.yoctoproject.org/typhoon/#/builders/65/builds/353
> 
> Cheers,
> 
> Richard

I thought that would have been fixed by:

95c78d2166d3 ("swig: Fix configure failure")

Upstream's configure step creates that directory that is missing and
causes the build failure showed by the logs. I thought avoiding a
custom do_configure would fix that, but evidently it doesn't!

I've sent a v3 with the bison dependency and with a workaround I've
successfully used in our internal build, which creates the directory
manually in a do_configure_append block.

Thanks for the review!

-- 
Kind regards,
Luca Boccassi


signature.asc
Description: This is a digitally signed message part
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH v3] swig: fix build with musl

[luca . boccassi]

From: Luca Boccassi 

Building with musl currently fails. The fix has been committed
upstream, so backport it.
Note that it requires running yacc again to regenerate the parser file,
so a bison-native dependency is needed. Also the upstream's configure
step creates the directory where the parser file is generated at
make-dist time, and for some reason running through bitbake does not
create it, so do it manually. All of this can be reverted once a new
swig version that includes the fix is released and rebased onto.

Signed-off-by: Luca Boccassi 
---
 meta/recipes-devtools/swig/swig.inc   |   8 +-
 ...de-for-constant-expressions-containi.patch | 179 ++
 meta/recipes-devtools/swig/swig_3.0.12.bb |   1 +
 3 files changed, 187 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-devtools/swig/swig/0001-Fix-generated-code-for-constant-expressions-containi.patch

diff --git a/meta/recipes-devtools/swig/swig.inc 
b/meta/recipes-devtools/swig/swig.inc
index f7de40866a..2f33b07c19 100644
--- a/meta/recipes-devtools/swig/swig.inc
+++ b/meta/recipes-devtools/swig/swig.inc
@@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = 
"file://LICENSE;md5=e7807a6282784a7dde4c846626b08fc6 \
 
 SECTION = "devel"
 
-DEPENDS = "libpcre"
+DEPENDS = "libpcre bison-native"
 
 SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz"
 
@@ -42,6 +42,12 @@ EXTRA_AUTORECONF += "-I Tools/config"
 
 BBCLASSEXTEND = "native nativesdk"
 
+# necessary together with bison dependency until a new upstream version after
+# 3.0.12 includes 
0001-Fix-generated-code-for-constant-expressions-containi.patch
+do_configure_append() {
+mkdir -p ${WORKDIR}/build/Source/CParse
+}
+
 do_install_append_class-nativesdk() {
 cd ${D}${bindir}
 ln -s swig swig2.0
diff --git 
a/meta/recipes-devtools/swig/swig/0001-Fix-generated-code-for-constant-expressions-containi.patch
 
b/meta/recipes-devtools/swig/swig/0001-Fix-generated-code-for-constant-expressions-containi.patch
new file mode 100644
index 00..f4ed782a82
--- /dev/null
+++ 
b/meta/recipes-devtools/swig/swig/0001-Fix-generated-code-for-constant-expressions-containi.patch
@@ -0,0 +1,179 @@
+From 90ba174fcea1618af57aa594199541d47a89b7f6 Mon Sep 17 00:00:00 2001
+From: William S Fulton 
+Date: Sun, 17 Sep 2017 19:02:55 +0100
+Subject: [PATCH 1/2] Fix generated code for constant expressions containing
+ wchar_t L literals.
+
+Such as:
+  # define __WCHAR_MAX(0x7fff + L'\0')
+
+Reported on swig-user mailing list.
+---
+ CHANGES.current | 5 +
+ Examples/test-suite/csharp/preproc_constants_c_runme.cs | 3 ++-
+ Examples/test-suite/csharp/preproc_constants_runme.cs   | 2 ++
+ Examples/test-suite/d/preproc_constants_c_runme.1.d | 2 ++
+ Examples/test-suite/d/preproc_constants_c_runme.2.d | 2 ++
+ Examples/test-suite/d/preproc_constants_runme.1.d   | 2 ++
+ Examples/test-suite/d/preproc_constants_runme.2.d   | 2 ++
+ Examples/test-suite/php/preproc_constants_c_runme.php   | 2 ++
+ Examples/test-suite/php/preproc_constants_runme.php | 2 ++
+ Examples/test-suite/php5/preproc_constants_c_runme.php  | 2 ++
+ Examples/test-suite/php5/preproc_constants_runme.php| 2 ++
+ Examples/test-suite/preproc_constants.i | 3 +++
+ Source/CParse/parser.y  | 2 +-
+ 13 files changed, 29 insertions(+), 2 deletions(-)
+
+Upstream-Status: Backport
+[https://github.com/swig/swig/commit/90ba174fcea1618af57aa594199541d47a89b7f6]
+Signed-off-by: Luca Boccassi 
+
+diff --git a/Examples/test-suite/csharp/preproc_constants_c_runme.cs 
b/Examples/test-suite/csharp/preproc_constants_c_runme.cs
+index 76c684d..1c28e49 100644
+--- a/Examples/test-suite/csharp/preproc_constants_c_runme.cs
 b/Examples/test-suite/csharp/preproc_constants_c_runme.cs
+@@ -61,7 +61,8 @@ public class runme {
+ assert( typeof(int) == preproc_constants_c.EXPR_LAND.GetType() );
+ assert( typeof(int) == preproc_constants_c.EXPR_LOR.GetType() );
+ assert( typeof(double) == preproc_constants_c.EXPR_CONDITIONAL.GetType() 
);
+-
++assert( typeof(int) == preproc_constants_c.EXPR_WCHAR_MAX.GetType() );
++assert( typeof(int) == preproc_constants_c.EXPR_WCHAR_MIN.GetType() );
+   }
+   static void assert(bool assertion) {
+ if (!assertion)
+diff --git a/Examples/test-suite/csharp/preproc_constants_runme.cs 
b/Examples/test-suite/csharp/preproc_constants_runme.cs
+index 9fae591..6b02e30 100644
+--- a/Examples/test-suite/csharp/preproc_constants_runme.cs
 b/Examples/test-suite/csharp/preproc_constants_runme.cs
+@@ -60,6 +60,8 @@ public class runme {
+ assert( typeof(bool) == preproc_constants.EXPR_LAND.GetType() );
+ assert( typeof(bool) == preproc_constants.EXPR_LOR.GetType() );
+ assert( typeof(double) == preproc_constants.EXPR_CONDITIONAL.GetType() );
++assert( typeof(int) == preproc_constants.EXPR_WCHAR_MAX.GetType() );
++assert( typeof(int) == 

Re: [OE-core] sstate hash equivalence breaks rm_work

[Martin Jansa]

4TB of sstate is just for few weeks for me :).

On Mon, Mar 4, 2019 at 11:45 AM Burton, Ross  wrote:

> On Mon, 4 Mar 2019 at 10:27, Martin Jansa  wrote:
> > There is, you can prune outdated duplicate sstate archives while still
> separately building multiple DISTRO_VERSIONs.
>
> We don't all have 4TB disks? :)
>
> Ross
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/1] send-error-report: Add --no-ssl to use http protocol

[Burton, Ross]

On Mon, 4 Mar 2019 at 10:30, Robert Yang  wrote:
> +arg_parse.add_argument("--no-ssl",
> +   help="Use http instead of https protocol",
> +   action="store_true")

If you do this instead:

arg_parse.add_argument("--no-ssl", help="Use http instead of https protocol",
  dest="protocol", action="store_const", const="http", default="https")

Then you can just use args.protocol instead of having to pass around
the protocol name.

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

Re: [OE-core] GO runtime crashes

[Vincent Prince]

Khem, you can find some backtraces here:
https://github.com/prometheus/node_exporter/issues/1244, don't know if this
is enough?

Le dim. 3 mars 2019 à 16:43, Khem Raj  a écrit :

> On Sun, Mar 3, 2019 at 2:45 AM Vincent Prince
>  wrote:
> >
> > Khem,
> >
> > I'm not sure if we considere this issue as closed or if we need to dig
> root-cause here?
> >
> > I don't have any Go stuff on my build machine nor build docker.
> > Also, those random crashes came with Thud version, I didn't have any in
> Sumo, but there was a lot of changes between these two versions.
> >
> > Please tell me if I can help,
>
> I think it needs some root causing, but we know the symptoms. So
> debugging to crash and some backtraces might help
>
> > Vincent
> >
> >
> > Le jeu. 28 févr. 2019 à 19:24, Khem Raj  a écrit :
> >>
> >> On Thu, Feb 28, 2019 at 10:19 AM Damien Riegel 
> wrote:
> >> >
> >> > Hi,
> >> >
> >> > On Thu, 28 Feb 2019 at 09:04, Vincent Prince
> >> >  wrote:
> >> > >
> >> > > Hi Khem,
> >> > >
> >> > > GO_DYNLINK_x86-64 = "" solution works as well,
> >> > >
> >> > > Best regards,
> >> > > Vincent
> >> > >
> >> > > Le jeu. 28 févr. 2019 à 14:14, Khem Raj  a
> écrit :
> >> > >>
> >> > >> On Thu, Feb 28, 2019 at 1:35 AM Vincent Prince
> >> > >>  wrote:
> >> > >> >
> >> > >> > Hi Damien,
> >> > >> >
> >> > >> > With GO_LINKSHARED = "" it work fine.
> >> > >> >
> >> > >>
> >> > >> Please set
> >> > >> GO_DYNLINK_x86-64 = "1"
> >> > >> to
> >> > >> GO_DYNLINK_x86-64 = ""
> >> > >> in goarch.bbclass
> >> >
> >> > Well, making a change in the goarch.bbclass is way more intrusive than
> >> > changing the value in a single recipe, so it would be nice to (at
> least)
> >> > give some justification about this solution.
> >> >
> >>
> >> this will make static binaries instead, so I wonder if its doing some
> >> linking from host.
> >>
> >> > >>
> >> > >> > I'm very new to Go, does it mean that there is a mixing issue in
> go modules from Yocto and from node_exporter vendor folder, so it links
> wrongly at runtime?
> >> >
> >> > As to why the GO_LINKSHARED ="" works, I cannot tell. I just noticed
> >> > that our application was linked differently on my machine (where it
> >> > worked) and in Yocto (where it didn't work), so I tweaked link flags
> >> > until I got it to work.
> >> >
> >> > Cheers,
> >> > Damien
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] sstate hash equivalence breaks rm_work

[Burton, Ross]

On Mon, 4 Mar 2019 at 10:27, Martin Jansa  wrote:
> There is, you can prune outdated duplicate sstate archives while still 
> separately building multiple DISTRO_VERSIONs.

We don't all have 4TB disks? :)

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

[OE-core] [PATCH 0/1] send-error-report: Add --no-ssl to use http protocol

[Robert Yang]

The following changes since commit 9ad19eb12c24d1ddc967215af0ebafd2cd2cb665:

  mcextend: Add helper class useful for multiconfig (2019-03-03 15:38:08 +)

are available in the git repository at:

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

Robert Yang (1):
  send-error-report: Add --no-ssl to use http protocol

 scripts/send-error-report | 23 ---
 1 file changed, 16 insertions(+), 7 deletions(-)

-- 
2.7.4

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

Re: [OE-core] [PATCH] send-error-report: Use https instead of http protocol

[Robert Yang]

On 3/4/19 6:26 PM, Richard Purdie wrote:

On Mon, 2019-03-04 at 18:02 +0800, Robert Yang wrote:

Hi RP,

On 3/4/19 5:23 PM, Richard Purdie wrote:

On Mon, 2019-03-04 at 15:50 +0800, Robert Yang wrote:

Hi Khem,

We have an internal error web server, this patch breaks it:

You're accessing the development server over HTTPS, but it only
supports HTTP

So I tried to use runsslserver, but it doesn't work either since
the
host is
Ubuntu 14.04 (python 2.7.6):

$ python manage.py runsslserver

[snip]
   ssl_version=ssl.PROTOCOL_TLSv1_2,
AttributeError: 'module' object has no attribute
'PROTOCOL_TLSv1_2'

This is because python is old.

I think that make it default to https would make the server
harder to
set up,
so is it possible to:

* Make it default to http, and add an option such as --https (or
--
ssl) for https

Or:

* Add an option such as --no-ssl to make it use http.

I prefer the first one since https server is harder to setup than
http in django. I can work on it if no objections.


At this point the project does require python3 so python2 being old
shouldn't be a reason to be changing this.


The python3 + error-report-web doesn't work for me:

$ python3 manage.py runserver ip:8000

And when I tried to access http://128.224.156.132:8000, it raised
errors:

error-report-web/Post/views.py", line 16, in 
  from parser import Parser
ImportError: cannot import name 'Parser'

So I use python 2 for it. I thought that it didn't support python3,
or didn't
test well.


Fixing it to work with python3 would be the preferred option...


Sounds good, I will work on it.

// Robert



Cheers,

Richard



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

[OE-core] [PATCH 1/1] send-error-report: Add --no-ssl to use http protocol

[Robert Yang]

The script use https protocol by default, but the error-report-web server's
https connection may not work (e.g., doesn't work with python 2.7.6), so add an
option --no-ssl to make it use http.

Signed-off-by: Robert Yang 
---
 scripts/send-error-report | 23 ---
 1 file changed, 16 insertions(+), 7 deletions(-)

diff --git a/scripts/send-error-report b/scripts/send-error-report
index 3528cf9..f8039b8 100755
--- a/scripts/send-error-report
+++ b/scripts/send-error-report
@@ -60,9 +60,9 @@ def edit_content(json_file_path):
 return True
 return False
 
-def prepare_data(args):
+def prepare_data(args, protocol):
 # attempt to get the max_log_size from the server's settings
-max_log_size = getPayloadLimit("https://"+args.server+"/ClientPost/JSON;)
+max_log_size = getPayloadLimit(protocol+args.server+"/ClientPost/JSON")
 
 if not os.path.isfile(args.error_file):
 log.error("No data file found.")
@@ -128,13 +128,13 @@ def prepare_data(args):
 return data.encode('utf-8')
 
 
-def send_data(data, args):
+def send_data(data, args, protocol):
 headers={'Content-type': 'application/json', 'User-Agent': 
"send-error-report/"+version}
 
 if args.json:
-url = "https://"+args.server+"/ClientPost/JSON/;
+url = protocol+args.server+"/ClientPost/JSON/"
 else:
-url = "https://"+args.server+"/ClientPost/;
+url = protocol+args.server+"/ClientPost/"
 
 req = urllib.request.Request(url, data=data, headers=headers)
 try:
@@ -187,6 +187,10 @@ if __name__ == '__main__':
help="Return the result in json format, silences 
all other output",
action="store_true")
 
+arg_parse.add_argument("--no-ssl",
+   help="Use http instead of https protocol",
+   action="store_true")
+
 
 
 args = arg_parse.parse_args()
@@ -194,7 +198,12 @@ if __name__ == '__main__':
 if (args.json == False):
 print("Preparing to send errors to: "+args.server)
 
-data = prepare_data(args)
-send_data(data, args)
+if args.no_ssl:
+protocol = "http://;
+else:
+protocol = "https://;
+
+data = prepare_data(args, protocol)
+send_data(data, args, protocol)
 
 sys.exit(0)
-- 
2.7.4

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

Re: [OE-core] [PATCH] send-error-report: Use https instead of http protocol

[Richard Purdie]

On Mon, 2019-03-04 at 18:02 +0800, Robert Yang wrote:
> Hi RP,
> 
> On 3/4/19 5:23 PM, Richard Purdie wrote:
> > On Mon, 2019-03-04 at 15:50 +0800, Robert Yang wrote:
> > > Hi Khem,
> > > 
> > > We have an internal error web server, this patch breaks it:
> > > 
> > > You're accessing the development server over HTTPS, but it only
> > > supports HTTP
> > > 
> > > So I tried to use runsslserver, but it doesn't work either since
> > > the
> > > host is
> > > Ubuntu 14.04 (python 2.7.6):
> > > 
> > > $ python manage.py runsslserver
> > > 
> > > [snip]
> > >   ssl_version=ssl.PROTOCOL_TLSv1_2,
> > > AttributeError: 'module' object has no attribute
> > > 'PROTOCOL_TLSv1_2'
> > > 
> > > This is because python is old.
> > > 
> > > I think that make it default to https would make the server
> > > harder to
> > > set up,
> > > so is it possible to:
> > > 
> > > * Make it default to http, and add an option such as --https (or
> > > --
> > > ssl) for https
> > > 
> > > Or:
> > > 
> > > * Add an option such as --no-ssl to make it use http.
> > > 
> > > I prefer the first one since https server is harder to setup than
> > > http in django. I can work on it if no objections.
> > 
> > At this point the project does require python3 so python2 being old
> > shouldn't be a reason to be changing this.
> 
> The python3 + error-report-web doesn't work for me:
> 
> $ python3 manage.py runserver ip:8000
> 
> And when I tried to access http://128.224.156.132:8000, it raised
> errors:
> 
> error-report-web/Post/views.py", line 16, in 
>  from parser import Parser
> ImportError: cannot import name 'Parser'
> 
> So I use python 2 for it. I thought that it didn't support python3,
> or didn't
> test well.

Fixing it to work with python3 would be the preferred option...

Cheers,

Richard

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

Re: [OE-core] sstate hash equivalence breaks rm_work

[Martin Jansa]

There is, you can prune outdated duplicate sstate archives while still
separately building multiple DISTRO_VERSIONs.

On Mon, Mar 4, 2019 at 11:17 AM Burton, Ross  wrote:

> On Sun, 3 Mar 2019 at 22:52, Tom Rini  wrote:
> > SSTATE_DIR =
> "${HOME}/work/OE/sstate-cache-${DISTRO_VERSION}${TARGET_VENDOR}"
>
> FWIW there isn't much point in isolating sstate like that.  You'll
> just rebuild stuff that would have been available.
>
> Ross
> --
> ___
> 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] sstate hash equivalence breaks rm_work

[Burton, Ross]

On Sun, 3 Mar 2019 at 22:52, Tom Rini  wrote:
> SSTATE_DIR = "${HOME}/work/OE/sstate-cache-${DISTRO_VERSION}${TARGET_VENDOR}"

FWIW there isn't much point in isolating sstate like that.  You'll
just rebuild stuff that would have been available.

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

Re: [OE-core] [PATCH] send-error-report: Use https instead of http protocol

[Robert Yang]

Hi RP,

On 3/4/19 5:23 PM, Richard Purdie wrote:

On Mon, 2019-03-04 at 15:50 +0800, Robert Yang wrote:

Hi Khem,

We have an internal error web server, this patch breaks it:

You're accessing the development server over HTTPS, but it only
supports HTTP

So I tried to use runsslserver, but it doesn't work either since the
host is
Ubuntu 14.04 (python 2.7.6):

$ python manage.py runsslserver

[snip]
  ssl_version=ssl.PROTOCOL_TLSv1_2,
AttributeError: 'module' object has no attribute 'PROTOCOL_TLSv1_2'

This is because python is old.

I think that make it default to https would make the server harder to
set up,
so is it possible to:

* Make it default to http, and add an option such as --https (or --
ssl) for https

Or:

* Add an option such as --no-ssl to make it use http.

I prefer the first one since https server is harder to setup than
http in django. I can work on it if no objections.


At this point the project does require python3 so python2 being old
shouldn't be a reason to be changing this.


The python3 + error-report-web doesn't work for me:

$ python3 manage.py runserver ip:8000

And when I tried to access http://128.224.156.132:8000, it raised errors:

error-report-web/Post/views.py", line 16, in 
from parser import Parser
ImportError: cannot import name 'Parser'

So I use python 2 for it. I thought that it didn't support python3, or didn't
test well.



The project also needs to be seen to encourage secure practises so I
think https should be the default but adding an option to use http
should be ok.


Thanks, I will send a patch for it.

// Robert



Cheers,

Richard





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

Re: [OE-core] [PATCH 2/4] inetutils: Import version 1.9.4

[André Draszik]

On Fri, 2019-03-01 at 17:26 -0500, Tom Rini wrote:
> In order to have more robust stand-alone network tools in oe-core, bring
> in vim from meta-openembedded/meta-networking.  This imports the recipes
 ^^^

vim for networking? :-)

Cheers,
Andre'

> as of git commit:
> commit 408204073e6bdcd8ac586e05d5b75213417673f2
> Author: Martin Jansa 
> Date:   Thu Aug 16 20:39:15 2018 +
> 
> inetutils: fix build with glibc-2.28
> 
> Signed-off-by: Tom Rini 
> ---
>  .../0001-rcp-fix-to-work-with-large-files.patch|  31 
>  .../inetutils/inetutils/fix-disable-ipv6.patch |  83 +
>  ...tf-parse-pull-in-features.h-for-__GLIBC__.patch |  29 +++
>  .../inetutils/inetutils-1.8-0003-wchar.patch   |  14 ++
>  .../inetutils/inetutils-1.9-PATH_PROCNET_DEV.patch |  26 +++
>  ...ls-only-check-pam_appl.h-when-pam-enabled.patch |  40 
>  .../inetutils/inetutils/rexec.xinetd.inetutils |  20 ++
>  .../inetutils/inetutils/rlogin.xinetd.inetutils|  23 +++
>  .../inetutils/inetutils/rsh.xinetd.inetutils   |  21 +++
>  .../inetutils/inetutils/telnet.xinetd.inetutils|  13 ++
>  .../inetutils/inetutils/tftpd.xinetd.inetutils |  19 ++
>  .../inetutils/inetutils/version.patch  |  17 ++
>  .../inetutils/inetutils_1.9.4.bb   | 206
> +
>  13 files changed, 542 insertions(+)
>  create mode 100644 meta/recipes-connectivity/inetutils/inetutils/0001-
> rcp-fix-to-work-with-large-files.patch
>  create mode 100644 meta/recipes-connectivity/inetutils/inetutils/fix-
> disable-ipv6.patch
>  create mode 100644 meta/recipes-
> connectivity/inetutils/inetutils/inetutils-1.8-0001-printf-parse-pull-in-
> features.h-for-__GLIBC__.patch
>  create mode 100644 meta/recipes-
> connectivity/inetutils/inetutils/inetutils-1.8-0003-wchar.patch
>  create mode 100644 meta/recipes-
> connectivity/inetutils/inetutils/inetutils-1.9-PATH_PROCNET_DEV.patch
>  create mode 100644 meta/recipes-
> connectivity/inetutils/inetutils/inetutils-only-check-pam_appl.h-when-pam-
> enabled.patch
>  create mode 100644 meta/recipes-
> connectivity/inetutils/inetutils/rexec.xinetd.inetutils
>  create mode 100644 meta/recipes-
> connectivity/inetutils/inetutils/rlogin.xinetd.inetutils
>  create mode 100644 meta/recipes-
> connectivity/inetutils/inetutils/rsh.xinetd.inetutils
>  create mode 100644 meta/recipes-
> connectivity/inetutils/inetutils/telnet.xinetd.inetutils
>  create mode 100644 meta/recipes-
> connectivity/inetutils/inetutils/tftpd.xinetd.inetutils
>  create mode 100644 meta/recipes-
> connectivity/inetutils/inetutils/version.patch
>  create mode 100644 meta/recipes-connectivity/inetutils/inetutils_1.9.4.bb
> 
> diff --git a/meta/recipes-connectivity/inetutils/inetutils/0001-rcp-fix-
> to-work-with-large-files.patch b/meta/recipes-
> connectivity/inetutils/inetutils/0001-rcp-fix-to-work-with-large-
> files.patch
> new file mode 100644
> index ..d4764f586736
> --- /dev/null
> +++ b/meta/recipes-connectivity/inetutils/inetutils/0001-rcp-fix-to-work-
> with-large-files.patch
> @@ -0,0 +1,31 @@
> +Upstream-Status: Pending
> +
> +Subject: rcp: fix to work with large files
> +
> +When we copy file by rcp command, if the file > 2GB, it will fail.
> +The cause is that it used incorrect data type on file size in sink() of
> rcp.
> +
> +Signed-off-by: Chen Qi 
> +---
> + src/rcp.c | 4 ++--
> + 1 file changed, 2 insertions(+), 2 deletions(-)
> +
> +diff --git a/src/rcp.c b/src/rcp.c
> +index 21f55b6..bafa35f 100644
> +--- a/src/rcp.c
>  b/src/rcp.c
> +@@ -876,9 +876,9 @@ sink (int argc, char *argv[])
> +   enum
> +   { YES, NO, DISPLAYED } wrerr;
> +   BUF *bp;
> +-  off_t i, j;
> ++  off_t i, j, size;
> +   int amt, count, exists, first, mask, mode, ofd, omode;
> +-  int setimes, size, targisdir, wrerrno;
> ++  int setimes, targisdir, wrerrno;
> +   char ch, *cp, *np, *targ, *vect[1], buf[BUFSIZ];
> +   const char *why;
> + 
> +-- 
> +1.9.1
> +
> diff --git a/meta/recipes-connectivity/inetutils/inetutils/fix-disable-
> ipv6.patch b/meta/recipes-connectivity/inetutils/inetutils/fix-disable-
> ipv6.patch
> new file mode 100644
> index ..24c134fcac8d
> --- /dev/null
> +++ b/meta/recipes-connectivity/inetutils/inetutils/fix-disable-ipv6.patch
> @@ -0,0 +1,83 @@
> +Upstream: http://www.mail-archive.com/bug-inetutils@gnu.org/msg02103.html
> +
> +Upstream-Status: Pending
> +
> +Signed-off-by: Jackie Huang 
> +---
> + ping/ping_common.h | 20 
> + 1 file changed, 20 insertions(+)
> +
> +diff --git a/ping/ping_common.h b/ping/ping_common.h
> +index 1dfd1b5..3bfbd12 100644
> +--- a/ping/ping_common.h
>  b/ping/ping_common.h
> +@@ -17,10 +17,14 @@
> +   You should have received a copy of the GNU General Public License
> +   along with this program.  If not, see `http://www.gnu.org/licenses/'.
> */
> + 
> ++#include 
> ++
> + #include 
> + #include 
> + #include 
> ++#ifdef HAVE_IPV6
> + #include 
> ++#endif
> + #include 
> + #include 
> + 

Re: [OE-core] [PATCH] send-error-report: Use https instead of http protocol

[Richard Purdie]

On Mon, 2019-03-04 at 15:50 +0800, Robert Yang wrote:
> Hi Khem,
> 
> We have an internal error web server, this patch breaks it:
> 
> You're accessing the development server over HTTPS, but it only
> supports HTTP
> 
> So I tried to use runsslserver, but it doesn't work either since the
> host is
> Ubuntu 14.04 (python 2.7.6):
> 
> $ python manage.py runsslserver
> 
> [snip]
>  ssl_version=ssl.PROTOCOL_TLSv1_2,
> AttributeError: 'module' object has no attribute 'PROTOCOL_TLSv1_2'
> 
> This is because python is old.
> 
> I think that make it default to https would make the server harder to
> set up,
> so is it possible to:
> 
> * Make it default to http, and add an option such as --https (or --
> ssl) for https
> 
> Or:
> 
> * Add an option such as --no-ssl to make it use http.
> 
> I prefer the first one since https server is harder to setup than
> http in django. I can work on it if no objections.

At this point the project does require python3 so python2 being old
shouldn't be a reason to be changing this.

The project also needs to be seen to encourage secure practises so I
think https should be the default but adding an option to use http
should be ok.

Cheers,

Richard



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

[OE-core] [PATCH] masterimage: remove code duplication

[André Draszik]

From: André Draszik 

Two identical classes exist for SystemdbootTarget,
introduced in commit f9a61d3400ad
("gummiboot: Remove/change gummiboot references with systemd-boot")
poky commit 2dce2648e3d8

Remove the duplicated copy.

Signed-off-by: André Draszik 
---
 meta/lib/oeqa/controllers/masterimage.py | 40 
 1 file changed, 40 deletions(-)

diff --git a/meta/lib/oeqa/controllers/masterimage.py 
b/meta/lib/oeqa/controllers/masterimage.py
index a2912fc568..f175e119da 100644
--- a/meta/lib/oeqa/controllers/masterimage.py
+++ b/meta/lib/oeqa/controllers/masterimage.py
@@ -197,43 +197,3 @@ class SystemdbootTarget(MasterImageHardwareTarget):
 self.power_cycle(self.master)
 # there are better ways than a timeout but this should work for now
 time.sleep(120)
-
-
-class SystemdbootTarget(MasterImageHardwareTarget):
-
-def __init__(self, d):
-super(SystemdbootTarget, self).__init__(d)
-# this the value we need to set in the LoaderEntryOneShot EFI variable
-# so the system boots the 'test' bootloader label and not the default
-# The first four bytes are EFI bits, and the rest is an utf-16le string
-# (EFI vars values need to be utf-16)
-# $ echo -en "test\0" | iconv -f ascii -t utf-16le | hexdump -C
-#   74 00 65 00 73 00 74 00  00 00
|t.e.s.t...|
-self.efivarvalue = 
r'\x07\x00\x00\x00\x74\x00\x65\x00\x73\x00\x74\x00\x00\x00'
-self.deploy_cmds = [
-'mount -L boot /boot',
-'mkdir -p /mnt/testrootfs',
-'mount -L testrootfs /mnt/testrootfs',
-'modprobe efivarfs',
-'mount -t efivarfs efivarfs /sys/firmware/efi/efivars',
-'cp ~/test-kernel /boot',
-'rm -rf /mnt/testrootfs/*',
-'tar xvf ~/test-rootfs.%s -C /mnt/testrootfs' % 
self.image_fstype,
-'printf "%s" > 
/sys/firmware/efi/efivars/LoaderEntryOneShot-4a67b082-0a4c-41cf-b6c7-440b29bb8c4f'
 % self.efivarvalue
-]
-
-def _deploy(self):
-# make sure these aren't mounted
-self.master.run("umount /boot; umount /mnt/testrootfs; umount 
/sys/firmware/efi/efivars;")
-# from now on, every deploy cmd should return 0
-# else an exception will be thrown by sshcontrol
-self.master.ignore_status = False
-self.master.copy_to(self.rootfs, "~/test-rootfs." + self.image_fstype)
-self.master.copy_to(self.kernel, "~/test-kernel")
-for cmd in self.deploy_cmds:
-self.master.run(cmd)
-
-def _start(self, params=None):
-self.power_cycle(self.master)
-# there are better ways than a timeout but this should work for now
-time.sleep(120)
-- 
2.20.1

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