Jonathan Liu [Wed, 22 May 2013 23:37:07 +0000 (09:37 +1000)]
populate-volatile.sh: remove repeated leading slashes in TNAME
This avoids triple slashes in the generated /etc/volatile.cache to
reduce disk usage and in the output when verbose mode is enabled.
As all the paths for volatiles start with a slash, we can change
TNAME=${ROOT_DIR}/${TNAME} to TNAME=${ROOT_DIR}${TNAME}. To avoid
a double slash when ROOT_DIR is /, we strip the extra slash from
ROOT_DIR.
Signed-off-by: Jonathan Liu <net147@gmail.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Jonathan Liu [Thu, 23 May 2013 04:02:14 +0000 (14:02 +1000)]
dbus: remove /var/run from package as it is created on startup
/var/run is a tmpfs that is created on startup.
For sysvinit, /var/run/dbus is created by populate-volatiles.sh.
For systemd, /var/run/dbus is created implicitly by dbus.socket when
creating a listen stream socket at /var/run/dbus/system_bus_socket.
Signed-off-by: Jonathan Liu <net147@gmail.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Saul Wold [Thu, 23 May 2013 17:13:09 +0000 (10:13 -0700)]
binutils: the cat should always succeed when building config.cache
There is a race condition between when the CONFIG_SITE list is generated and
then used here via the cat to create the config.cache and in *all* configure
scripts when reading from CONFIG_SITE.
The race in this case is that the sstate setscene task runs on a package that
contains a site config file (ncurses in this case) and then bitbake decides
that it needs to rebuild and cleans out the site config file, so it existed
for siteinfo_get_files() to find in SITECONFIG_SYSROOTCACHE and then was removed
for the rebuild. When bintuils tried to run the do_configure() task which creates
the binutil's version of config.cache it reads from CONFIG_SITE which now contains
the non-existant site config file. (confused yet ;-)?
Currently the configure script does a test -r to ensure the file is readable
before using it, therefore having the cat succeed regardless of the file being
available is consistent behaviour.
Phil Blundell [Tue, 21 May 2013 10:40:06 +0000 (11:40 +0100)]
cross: Clear do_populate_sysroot[stamp-extra-info] so that stamps are not MACHINE specific
Otherwise we will try to run the setscene task once per MACHINE which will lead to
file conflicts in the sysroot for packages like gcc-cross. The stamps
are already namespaced by TARGET_ARCH which should be sufficient.
Signed-off-by: Phil Blundell <philb-mXXj517/zsQ@public.gmane.org> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Martin Jansa [Tue, 21 May 2013 22:18:53 +0000 (00:18 +0200)]
kernel: Remove unnecessary cd and rm, fix MODULE_TARBALL_SYMLINK_NAME creation
* DEPLOYDIR is used as target for every other file deployed by do_deploy,
this was just unnecessary switching CWD to create relative symlink and I
have no idea why rm -f was used together with ln -sf.
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Jesse Zhang [Tue, 21 May 2013 02:43:04 +0000 (10:43 +0800)]
perl-tests: convert to ptest
On 05/17/2013 05:31 PM, Björn Stenberg wrote:
> Unless there are clear advantages with patching t/TEST that I have overlooked, I suggest using sed in run-ptest instead.
Ok.. Here is the new commit.
Removed the patch and added a call to sed in run-ptest. See attachment
for the test log.
From: Jesse Zhang <sen.zhang@windriver.com>
Date: Tue, 14 May 2013 02:53:30 -0400
Subject: [PATCH 1/1] perl-tests: convert to ptest
Replace PERL_TEST_DIR with PTEST_PATH, and rename "tests" with "ptest"
in various places. Also add a run-ptest script.
Kang Kai [Tue, 21 May 2013 09:11:58 +0000 (17:11 +0800)]
sudo: set CFLAGS for build mksigname and mksiglist
mksigname and mksiglist are compiled by BUILD_CC for build host. When
there are some options in CFLAGS that BUILD_CC doesn't support,
compilation fails.
Build for arm on a x86 host, if option "-mapcs-frame" is provided, error
occurs with:
| cc1: error: unrecognized command line option "-mapcs-frame"
Pass BUILD_CFLAGS to CFLAGS to fix that kind of failure.
Signed-off-by: Kang Kai <kai.kang@windriver.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Tomas Frydrych [Tue, 21 May 2013 15:08:42 +0000 (16:08 +0100)]
clutter: further improvements
* Removed unnecessary gtk-doc related depends/configure options (handled in
gtk-doc class),
* Moved more configure options into PACKAGECONFIG,
* Fixed missing comma in PACKAGECONFIG[glx] setup,
* Removed no longer required DOLT-related config fragment.
Signed-off-by: Tomas Frydrych <tomas@sleepfive.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Tomas Frydrych [Tue, 21 May 2013 15:08:41 +0000 (16:08 +0100)]
cogl: further improvements
* Removed unnecessary gtk-doc related depends/configure options (handled in
gtk-doc class)
* Moved more configure options into PACKAGECONFIG
* Fixed typo in RDEPENDS of PACKAGECONFIG[gl] setup
* Removed no longer needed DOLT-related configure fragment
Signed-off-by: Tomas Frydrych <tomas@sleepfive.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Mike Crowe [Mon, 20 May 2013 12:22:45 +0000 (13:22 +0100)]
kernel.bbclass: Add dependency on binutils
Compiling the Linux kernel requires binutils; kernel.bbclass uses
INHIBIT_DEFAULT_DEPS so it had better depend on binutils explicitly.
(The lack of this dependency isn't always a problem because binutils
is required to build gcc-cross but if gcc-cross is reconstructed from
the sstate cache then gcc-cross's dependency on binutils-cross is
ignored due to being in the safe dependency list in
setscene_depvalid.)
Signed-off-by: Mike Crowe <mac@mcrowe.com> Acked-by: Phil Blundell <philb@gnu.org> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Ross Burton [Thu, 16 May 2013 16:54:08 +0000 (17:54 +0100)]
librsvg: clean up native depends
inherit gnome was only used for the SRC_URI, so just inherit gnomebase and
remove the native-specific DEPENDS which was only required as the gnome class
pulls in an impossible hicolor-icon-theme-native dependency.
Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Ross Burton [Thu, 16 May 2013 16:52:00 +0000 (17:52 +0100)]
librsvg: add more PACKAGECONFIG options
Add options for the gdk-pixbuf loaders and the GTK+ 2 theme engine. The theme
engine is generally unused so don't enable that by default, but enable the
gdk-pixbuf loader and also enable the croco feature which is required for
parsing CSS embedded into SVG.
Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Ross Burton [Fri, 17 May 2013 08:54:39 +0000 (09:54 +0100)]
gdk-pixbuf: remove native loader cache generation, use the class
We assumed that the only relevant gdk-pixbuf loaders in the native environment
were the ones in gdk-pixbuf itself, as the icon cache is only for PNG files.
However, glib-compile-resources can transform SVG files to raw image data, and
done natively this requires the SVG loader to be registered. The current
implementation relies on this assumption by generating the cache based on the
staging directory during install, so if gdk-pixbuf-native is re-installed to the
sysroot after librsvg-native it will overwrite the loader cache.
So, remove the code in do_install that updated the cache, and rely on the new
logic in pixbufcache.bbclass that updates the cache when it's installed into the
sysroot itself.
Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Jonathan Liu [Sat, 18 May 2013 02:34:40 +0000 (12:34 +1000)]
util-linux: Update to 2.23
Remove license patch as it is integrated upstream.
Add backports of upstream loopdev regression fixes.
Updated uclibc-__progname-conflict.patch because it didn't apply.
Added bash-completion and partx sub-packages.
Signed-off-by: Jonathan Liu <net147@gmail.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Richard Purdie [Thu, 23 May 2013 10:40:40 +0000 (10:40 +0000)]
buildhistory: Sync with bitbake changes
Adjust to the recent changes in the git fetcher. This code should work
with current and previous versions of bitbake and can be cleaned up when
we move to new bitbake versions.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Chen Qi [Thu, 23 May 2013 04:43:36 +0000 (12:43 +0800)]
rpm-postinsts.bb: enable postinst logging
Enable postinst logging by checking the configuration in ${sysconfdir}
/default/postinst.
In this way, the postinst logging is enabled if 'debug-tweaks' is
in IMAGE_FEATURES, and at the same time, we avoid unnecessary rebuild
if IMAGE_FEATURES is changed.
[YOCTO #4262]
Signed-off-by: Chen Qi <Qi.Chen@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Chen Qi [Thu, 23 May 2013 04:43:35 +0000 (12:43 +0800)]
opkg: modify the run-postinst script to enable postinst logging
Enable postinst logging by checking the configuration in ${sysconfdir}
/default/postinst.
In this way, the postinst logging is enabled if 'debug-tweaks' is
in IMAGE_FEATURES, and at the same time, we avoid unnecessary rebuilt
if IMAGE_FEATURES is changed.
[YOCTO #4262]
Signed-off-by: Chen Qi <Qi.Chen@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Chen Qi [Thu, 23 May 2013 04:43:34 +0000 (12:43 +0800)]
dpkg: modify the run-postinst script to enable postinst logging
Enable postinst logging by checking the configuration in ${sysconfdir}
/default/postinst.
In this way, the postinst logging is enabled if 'debug-tweaks' is
in IMAGE_FEATURES, and at the same time, we avoid unnecessary rebuilt
if IMAGE_FEATURES is changed.
[YOCTO #4262]
Signed-off-by: Chen Qi <Qi.Chen@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Chen Qi [Thu, 23 May 2013 04:43:33 +0000 (12:43 +0800)]
image.bbclass: add postinst_enable_logging
Add a function postinst_enable_logging, so that when 'debug-tweaks'
is in IMAGE_FEATURES, we create ${sysconfdir}/default/postinst config
file, which is sourced by run-postinst scripts to determine whether
to log or not, and where to log.
[YOCTO #4262]
Signed-off-by: Chen Qi <Qi.Chen@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Richard Purdie [Tue, 21 May 2013 07:35:31 +0000 (08:35 +0100)]
update-alternatives: Drop deprecated code, update to show error message
This code has been deprecated for a while and confuses the class, lets drop it
and just give the user hard error messages instead of the current warnings/fixups.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Tomas Frydrych [Mon, 20 May 2013 15:05:24 +0000 (16:05 +0100)]
mx: Update to improved naming scheme
The package has been renamed to mx-1.0, keeping up with the upstream versioning
policy. The 1.14.7 tag does not build with clutter 1.14, so git revision 9b1db6b8060bd00b121a692f942404a24ae2960f from the 1.14 branch is used instead.
Signed-off-by: Tomas Frydrych <tomas@sleepfive.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Tomas Frydrych [Mon, 20 May 2013 15:05:21 +0000 (16:05 +0100)]
clutter: Replace with 1.14.4 version
The package has been renamed to clutter-1.0 instead of clutter-vmajor.vminor,
keeping up with the upstream versioning policy (all 1.x packages install
clutter-1.0 pc file and headers and are backward compatible), and to simplify
dependency management (worth noting that since the 1.x development files are
not parall installable, it is not possible to use two versions of clutter 1.x
at the same time anyway).
Package configuration is provided via PACKAGECONFIG options as follows:
Tomas Frydrych [Mon, 20 May 2013 15:05:20 +0000 (16:05 +0100)]
cogl: Replace with 1.14.0 version
The package has been renamed to cogl-1.0 instead of cogl-vmajor.vminor, keeping
up with the upstream versioning policy (e.g., all 1.x packages install cogl-1.0
pc file and headers and are backward compatible), and to simplify dependency
management (worth noting that since the 1.x development files are not parallel
installable, it is not possible to use two versions of cogl 1.x at the same
time anyway).
Package configuration is provided via PACKAGECONFIG options as follows:
GL flavour: 'gl' for big GL or 'gles2' for GLES2
(GLES1 is availabe in cogl, but not supporeted here at present.)
EGL platform: 'egl-null' -- PVR-style null platform
'egl-kms' -- kms platform provide by Mesa
'egl-x11' -- egl over xlib platform
(Additional EGL platforms, e.g., Wayland are supported by cogl,
but not supported here at present.)
GLX: 'glx' for the GLX extension support (implies 'gl')
Default configuration is 'glx'; typical configuration providing 'native' egl
on embedded HW would be 'gles2 egl-null'.
Signed-off-by: Tomas Frydrych <tomas@sleepfive.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Joe Slater [Thu, 16 May 2013 17:11:33 +0000 (17:11 +0000)]
zlib: put shared libraries in base_libdir
We put the shared libraries in base_libdir because at least
one library under base_libdir, pam_cracklib.so, needs them
and will cause a qa warning when it is built.
Signed-off-by: Joe Slater <jslater@windriver.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Andrei Dinu [Fri, 17 May 2013 10:37:25 +0000 (10:37 +0000)]
cdrtools-native : upgrade to 3.00
Upgrade from 2.01 -> 3.00
- Updated md5 of the license file because new information
was added by the owner.
- Removed glibc-conflict-rename.patch because it is not
required anymore.
- Updated no_usr_src.patch because it didn't apply.
Signed-off-by: Andrei Dinu <andrei.adrianx.dinu@intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Jonathan Liu [Thu, 16 May 2013 02:37:33 +0000 (02:37 +0000)]
opkg: remove unnecessary extra leading slash in run-postinsts
Signed-off-by: Jonathan Liu <net147@gmail.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Andreas Müller [Thu, 16 May 2013 23:28:42 +0000 (23:28 +0000)]
qt: fix warning -Wunused-local-typedefs introduced by gcc 4.8
in meta-oe maliit-framework failed with
| In file included from /home/andreas/tmp/oe-core-eglibc/sysroots/overo/usr/include/qt4/QtCore/QtCore:53:0,
| from ../utils/core-utils.cpp:18:
| /home/andreas/tmp/oe-core-eglibc/sysroots/overo/usr/include/qt4/QtCore/qtconcurrentfilter.h: In function 'QtConcurrent::ThreadEngineStarter<void> QtConcurrent::filterInternal(Sequence&, KeepFunctor, Redu
| /home/andreas/tmp/oe-core-eglibc/sysroots/overo/usr/include/qt4/QtCore/qtconcurrentfilter.h:108:47: error: typedef 'Iterator' locally defined but not used [-Werror=unused-local-typedefs]
| typedef typename Sequence::const_iterator Iterator;
| ^
| cc1plus: all warnings being treated as errors
Signed-off-by: Andreas Müller <schnitzeltony@googlemail.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Robert Yang [Tue, 7 May 2013 06:09:31 +0000 (06:09 +0000)]
image_types.bbclass: remove the old images
Remove the old image before the new one generated to save disk
space when RM_OLD_IMAGE is set to 1, this is an easy way to keep
the DEPLOY_DIR_IMAGE clean.
[YOCTO #4391]
Signed-off-by: Robert Yang <liezhi.yang@windriver.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Cristian Iorga [Thu, 16 May 2013 11:44:39 +0000 (14:44 +0300)]
python-pygobject-native: fix build dependency on python-native
The class-native override is undoing the dependency that distutils-base
added which we require. This patch adds in the missing dependency manually
to ensure the build functions correctly.
Fixes [YOCTO #4502].
Signed-off-by: Cristian Iorga <cristian.iorga@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Saul Wold [Thu, 16 May 2013 18:03:06 +0000 (21:03 +0300)]
Revert "initramfs-live-install*: fix the "install" boot option"
This was not the correct fix for this issues, it turns out that
base-files package was getting installed un-intentionally when
rpm-postinsts was split out. The base-files recipe lays down the
link that caused the cat failure.
Saul Wold [Thu, 16 May 2013 18:03:05 +0000 (21:03 +0300)]
rpm-postinst: remove un-needed RDEPENDS on base-files
This was transferred un-intentionally when we split the rpm code,
the base-files then layed down additional files (specificaly /etc/mtab)
in the initramfs image and caused problems for the installer code.
Removing the RDEPENDS will fix the image generation issue and thus
the installer, so we will need to revert a previous patch that attempted
to fix the earlier issue #4229.
[YOCTO #4504]
[YOCTO #4229]
Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Ross Burton [Thu, 16 May 2013 17:05:43 +0000 (18:05 +0100)]
libcroco: allow native builds
Use BBCLASSEXTEND to get native builds, and inherit gnomebase instead of gnome.
libcroco uses the class just for the SRC_URI helpers, and the full class was
pulling in many unrequired dependencies including hicolor-icon-theme-native,
which doesn't exist.
Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Ross Burton [Thu, 16 May 2013 17:05:30 +0000 (18:05 +0100)]
gtk+3: disable Xinerama support
GTK+ has enabled by default support for Xinerama, but it's not a build
dependency so this can be enabled or disabled depending if xinerama has been
built previously.
This can cause problems with sstate, and result in situations where
PKG_CHECK_MODULES() for GTK+ can appear successful but the CFLAGS are not set.
Signed-off-by: Ross Burton <ross.burton@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Bruce Ashfield [Thu, 16 May 2013 21:47:45 +0000 (14:47 -0700)]
linux-yocto/3.8: set qemumips64 SRCREVs
When qemumips64 was enabled, its machine specific SRCREV was missed.
Without it, qemumips64 builds from the wrong branch and we miss important
oprofile/ftrace fixes that are required for boot.
Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
tcf-agent: Use common two-digit sequence numbers for init script links
The common init script links use two-digit sequence numbers and
systemd interprets everything behind the two-digits as service name.
This leads to the following failure during boot:
Cannot add dependency job for unit 9tcf-agent.service
Signed-off-by: Stefan Herbrechtsmeier <stefan@herbrechtsmeier.net> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Stefan Stanacar [Tue, 14 May 2013 16:21:35 +0000 (19:21 +0300)]
connman: replace hardcoded path in init script and systemd service
The connman init script sources a setup file from /usr/lib/connman,
so we end up with no network in qemu multilib enabled images.
The init script it's installed by connman and because wired-setup
it's installed by another package (connman-conf) we can't use
libexecdir here and now (in the init script and systemd service file).
Once libexecdir changes from ${libdir}/${bpn} to something else like
/usr/libexec we could use that instead of ${libdir}/connman.
Changed in v2: - better commit message
[YOCTO #4493]
Signed-off-by: Stefan Stanacar <stefanx.stanacar@intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Laurentiu Palcu [Tue, 14 May 2013 11:23:29 +0000 (14:23 +0300)]
systemd: make udev-hwdb postinstall run on host
Run udevadm inside qemu in order to create the HW database index on
host.
The alternative would be to build a native version
of udevadm which would imply several things: split out systemd recipe,
create a common inc file and create a new recipe for udevadm native.
However, this latter solution might also add up to the build time
(the native recipe would need to run configure, make udevadm, install),
besides the time spent in the actual postinstall...
So, having the postinstall run through qemu is a good trade-off.
Mike Looijmans [Tue, 14 May 2013 06:28:17 +0000 (08:28 +0200)]
busybox-mdev: Startup at level S03 instead of S06
The udev hotplug deamon starts at 03, so mdev should start at the same
point.
This fixes the bug that when modutils (runs at 04) initializes devices
that the mdev hotplug will often not pick them up, resulting in missing
device nodes and similar problems.
Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Bruce Ashfield [Mon, 13 May 2013 05:42:23 +0000 (22:42 -0700)]
linux-yocto: add qemumips64 compatibility
The 3.8 and linux-yocto-dev kernel trees carry qemumips64 support. Adding
the board to the compatibility directly in the recipes is the first step
to adding mips64 support to the broader set of packages.
Khem Raj [Mon, 13 May 2013 05:42:22 +0000 (22:42 -0700)]
runqemu: Replace use of ifconfig with ip
ifconfig and its ilk (net-tools package) is deprecated in favour of iproute2 package
and is now removed by many distro's e.g. Archlinux. So we replace ifconfig with ip utility
Bruce Ashfield [Mon, 13 May 2013 04:06:14 +0000 (21:06 -0700)]
kern-tools: refresh and make dash compliant
The separately packaged merge_config.sh in the kern-tools package was
missing upstream fixes, and in particular a change that ensures it is
dash compatible.
By grabbing that upstream commit and rebasing the existing patches on
top of the new baseline, we are up to date and working on systems
where /bin/sh is dash.
[YOCTO #4473]
Signed-off-by: Bruce Ashfield <bruce.ashfield@windriver.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Jonathan Liu [Thu, 9 May 2013 03:24:03 +0000 (13:24 +1000)]
boot-directdisk: Scope HDDDIR and HDDIMG variables to avoid conflicts
These variables should not be shared with other image classes.
The bootimg class also has an HDDDIR variable that could be overwritten
if executing concurrently in the same image recipe.
Signed-off-by: Jonathan Liu <net147@gmail.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Jonathan Liu [Thu, 9 May 2013 03:24:02 +0000 (13:24 +1000)]
syslinux: Update to 4.06
Patches are now part of upstream or no longer needed.
Added new util-linux dependency for isohybrid.
Paths updated to reflect directory structure changes.
Add CFLAGS and LDFLAGS overrides.
[YOCTO #4438]
Signed-off-by: Jonathan Liu <net147@gmail.com> Signed-off-by: Saul Wold <sgw@linux.intel.com>
Martin Jansa [Mon, 13 May 2013 10:07:43 +0000 (10:07 +0000)]
bitbake.conf: define empty EXTRA_IMAGE_FEATURES before using them
* fixes build without EXTRA_IMAGE_FEATURES defined:
ERROR: '${EXTRA_IMAGE_FEATURES}' in IMAGE_FEATURES is not a valid
image feature. Valid features: dbg-pkgs ....
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>