2017-02-20 21:12:49 +00:00
|
|
|
# Copyright (C) 2017 Intel Corporation
|
|
|
|
# Released under the MIT license (see COPYING.MIT)
|
|
|
|
|
|
|
|
import os
|
|
|
|
import unittest
|
2017-04-05 13:36:05 +00:00
|
|
|
from compatlayer import get_signatures, LayerType, check_command, get_depgraph
|
2017-02-20 21:12:49 +00:00
|
|
|
from compatlayer.case import OECompatLayerTestCase
|
|
|
|
|
|
|
|
class CommonCompatLayer(OECompatLayerTestCase):
|
|
|
|
def test_readme(self):
|
|
|
|
readme_file = os.path.join(self.tc.layer['path'], 'README')
|
|
|
|
self.assertTrue(os.path.isfile(readme_file),
|
|
|
|
msg="Layer doesn't contains README file.")
|
|
|
|
|
|
|
|
data = ''
|
|
|
|
with open(readme_file, 'r') as f:
|
|
|
|
data = f.read()
|
|
|
|
self.assertTrue(data,
|
|
|
|
msg="Layer contains README file but is empty.")
|
|
|
|
|
|
|
|
def test_parse(self):
|
2017-04-05 13:36:04 +00:00
|
|
|
check_command('Layer %s failed to parse.' % self.tc.layer['name'],
|
|
|
|
'bitbake -p')
|
2017-02-20 21:12:49 +00:00
|
|
|
|
|
|
|
def test_show_environment(self):
|
2017-04-05 13:36:04 +00:00
|
|
|
check_command('Layer %s failed to show environment.' % self.tc.layer['name'],
|
|
|
|
'bitbake -e')
|
2017-02-20 21:12:49 +00:00
|
|
|
|
|
|
|
def test_signatures(self):
|
|
|
|
if self.tc.layer['type'] == LayerType.SOFTWARE:
|
|
|
|
raise unittest.SkipTest("Layer %s isn't BSP or DISTRO one." \
|
|
|
|
% self.tc.layer['name'])
|
|
|
|
|
2017-04-05 13:36:05 +00:00
|
|
|
# task -> (old signature, new signature)
|
2017-02-20 21:12:49 +00:00
|
|
|
sig_diff = {}
|
2017-04-11 18:38:40 +00:00
|
|
|
curr_sigs, _ = get_signatures(self.td['builddir'], failsafe=True)
|
2017-02-20 21:12:49 +00:00
|
|
|
for task in self.td['sigs']:
|
2017-04-05 13:36:05 +00:00
|
|
|
if task in curr_sigs and \
|
|
|
|
self.td['sigs'][task] != curr_sigs[task]:
|
|
|
|
sig_diff[task] = (self.td['sigs'][task], curr_sigs[task])
|
2017-02-20 21:12:49 +00:00
|
|
|
|
|
|
|
if sig_diff:
|
2017-04-05 13:36:05 +00:00
|
|
|
# Beware, depgraph uses task=<pn>.<taskname> whereas get_signatures()
|
|
|
|
# uses <pn>:<taskname>. Need to convert sometimes. The output follows
|
|
|
|
# the convention from get_signatures() because that seems closer to
|
|
|
|
# normal bitbake output.
|
|
|
|
def sig2graph(task):
|
|
|
|
pn, taskname = task.rsplit(':', 1)
|
|
|
|
return pn + '.' + taskname
|
|
|
|
def graph2sig(task):
|
|
|
|
pn, taskname = task.rsplit('.', 1)
|
|
|
|
return pn + ':' + taskname
|
|
|
|
depgraph = get_depgraph()
|
|
|
|
depends = depgraph['tdepends']
|
|
|
|
|
|
|
|
# If a task A has a changed signature, but none of its
|
|
|
|
# dependencies, then we need to report it because it is
|
|
|
|
# the one which introduces a change. Any task depending on
|
|
|
|
# A (directly or indirectly) will also have a changed
|
|
|
|
# signature, but we don't need to report it. It might have
|
|
|
|
# its own changes, which will become apparent once the
|
|
|
|
# issues that we do report are fixed and the test gets run
|
|
|
|
# again.
|
|
|
|
sig_diff_filtered = []
|
|
|
|
for task, (old_sig, new_sig) in sig_diff.items():
|
|
|
|
deps_tainted = False
|
|
|
|
for dep in depends.get(sig2graph(task), ()):
|
|
|
|
if graph2sig(dep) in sig_diff:
|
|
|
|
deps_tainted = True
|
|
|
|
break
|
|
|
|
if not deps_tainted:
|
|
|
|
sig_diff_filtered.append((task, old_sig, new_sig))
|
2017-02-20 21:12:49 +00:00
|
|
|
|
2017-04-05 13:36:05 +00:00
|
|
|
msg = []
|
|
|
|
msg.append('Layer %s changed %d signatures, initial differences (first hash without, second with layer):' %
|
|
|
|
(self.tc.layer['name'], len(sig_diff)))
|
|
|
|
for diff in sorted(sig_diff_filtered):
|
yocto-compat-layer: include bitbake-diffsigs output
After filtering out potential false positives, it becomes feasible to
include the output of bitbake-diffsigs for those tasks which
definitely have a change.
Depends on bitbake-diffsigs with the "--signature" parameter.
Enhanced output now is:
AssertionError: False is not true : Layer meta-xxxx changed 120 signatures, initial differences (first hash without, second with layer):
gstreamer1.0-plugins-base:do_fetch: 76973f19f2e30d282152bdd7e4efe5bb -> e6e7c6fa9f2bd59d7d8d107f7c6ca1ac
Task dependencies changed from:
['PV', 'SRCREV', 'SRC_URI', 'SRC_URI[md5sum]', 'SRC_URI[sha256sum]', 'base_do_fetch']
to:
['GST_IMX_PATCHES_TO_APPEND', 'PV', 'SRCREV', 'SRC_URI', 'SRC_URI[md5sum]', 'SRC_URI[sha256sum]', 'base_do_fetch']
basehash changed from d679d30bd1ea41c56e57419b57587f3c to 090a79b45f5fa26d10f9d34e2ed7a1e6
List of dependencies for variable SRC_URI changed from '{'PV', 'SRC_URI[md5sum]', 'SRC_URI[sha256sum]'}' to '{'GST_IMX_PATCHES_TO_APPEND', 'PV', 'SRC_URI[md5sum]', 'SRC_URI[sha256sum]'}'
changed items: {'GST_IMX_PATCHES_TO_APPEND'}
Dependency on variable GST_IMX_PATCHES_TO_APPEND was added
Variable SRC_URI value changed:
" http://gstreamer.freedesktop.org/src/gst-plugins-base/gst-plugins-base-${PV}.tar.xz file://get-caps-from-src-pad-when-query-caps.patch file://0003-ssaparse-enhance-SSA-text-lines-parsing.patch file://0004-subparse-set-need_segment-after-sink-pad-received-GS.patch file://encodebin-Need-more-buffers-in-output-queue-for-bett.patch file://make-gio_unix_2_0-dependency-configurable.patch file://0001-introspection.m4-prefix-pkgconfig-paths-with-PKG_CON.patch file://0001-Makefile.am-don-t-hardcode-libtool-name-when-running.patch file://0002-Makefile.am-prefix-calls-to-pkg-config-with-PKG_CONF.patch file://0003-riff-add-missing-include-directories-when-calling-in.patch file://0004-rtsp-drop-incorrect-reference-to-gstreamer-sdp-in-Ma.patch [--] {+${GST_IMX_PATCHES_TO_APPEND}+}"
pulseaudio:do_install: 6bb6fe23e11a6d5fef9c3a25e73e4f9c -> 3f54ea75673a792e307197cfa6ef2694
basehash changed from ac4efcfa783bd04a5a98a2c38719aedd to 37679d99623a37c8df955da3a01415a5
Variable do_install value changed:
@@ -1,3 +1,7 @@
autotools_do_install
install -d ${D}${sysconfdir}/default/volatiles
install -m 0644 ${WORKDIR}/volatiles.04_pulse ${D}${sysconfdir}/default/volatiles/volatiles.04_pulse
+ if [ -e "${WORKDIR}/daemon.conf" ] && [ -e "${WORKDIR}/default.pa" ]; then
+ install -m 0644 ${WORKDIR}/daemon.conf ${D}${sysconfdir}/pulse/daemon.conf
+ install -m 0644 ${WORKDIR}/default.pa ${D}${sysconfdir}/pulse/default.pa
+ fi
[YOCTO #11161]
(From OE-Core rev: 312edd42b6cc553de4d476c76e8e36a882e11cdd)
Signed-off-by: Patrick Ohly <patrick.ohly@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
2017-04-11 18:38:39 +00:00
|
|
|
recipe, taskname = diff[0].rsplit(':', 1)
|
|
|
|
cmd = 'bitbake-diffsigs --task %s %s --signature %s %s' % \
|
|
|
|
(recipe, taskname, diff[1], diff[2])
|
2017-04-05 13:36:05 +00:00
|
|
|
msg.append(' %s: %s -> %s' % diff)
|
yocto-compat-layer: include bitbake-diffsigs output
After filtering out potential false positives, it becomes feasible to
include the output of bitbake-diffsigs for those tasks which
definitely have a change.
Depends on bitbake-diffsigs with the "--signature" parameter.
Enhanced output now is:
AssertionError: False is not true : Layer meta-xxxx changed 120 signatures, initial differences (first hash without, second with layer):
gstreamer1.0-plugins-base:do_fetch: 76973f19f2e30d282152bdd7e4efe5bb -> e6e7c6fa9f2bd59d7d8d107f7c6ca1ac
Task dependencies changed from:
['PV', 'SRCREV', 'SRC_URI', 'SRC_URI[md5sum]', 'SRC_URI[sha256sum]', 'base_do_fetch']
to:
['GST_IMX_PATCHES_TO_APPEND', 'PV', 'SRCREV', 'SRC_URI', 'SRC_URI[md5sum]', 'SRC_URI[sha256sum]', 'base_do_fetch']
basehash changed from d679d30bd1ea41c56e57419b57587f3c to 090a79b45f5fa26d10f9d34e2ed7a1e6
List of dependencies for variable SRC_URI changed from '{'PV', 'SRC_URI[md5sum]', 'SRC_URI[sha256sum]'}' to '{'GST_IMX_PATCHES_TO_APPEND', 'PV', 'SRC_URI[md5sum]', 'SRC_URI[sha256sum]'}'
changed items: {'GST_IMX_PATCHES_TO_APPEND'}
Dependency on variable GST_IMX_PATCHES_TO_APPEND was added
Variable SRC_URI value changed:
" http://gstreamer.freedesktop.org/src/gst-plugins-base/gst-plugins-base-${PV}.tar.xz file://get-caps-from-src-pad-when-query-caps.patch file://0003-ssaparse-enhance-SSA-text-lines-parsing.patch file://0004-subparse-set-need_segment-after-sink-pad-received-GS.patch file://encodebin-Need-more-buffers-in-output-queue-for-bett.patch file://make-gio_unix_2_0-dependency-configurable.patch file://0001-introspection.m4-prefix-pkgconfig-paths-with-PKG_CON.patch file://0001-Makefile.am-don-t-hardcode-libtool-name-when-running.patch file://0002-Makefile.am-prefix-calls-to-pkg-config-with-PKG_CONF.patch file://0003-riff-add-missing-include-directories-when-calling-in.patch file://0004-rtsp-drop-incorrect-reference-to-gstreamer-sdp-in-Ma.patch [--] {+${GST_IMX_PATCHES_TO_APPEND}+}"
pulseaudio:do_install: 6bb6fe23e11a6d5fef9c3a25e73e4f9c -> 3f54ea75673a792e307197cfa6ef2694
basehash changed from ac4efcfa783bd04a5a98a2c38719aedd to 37679d99623a37c8df955da3a01415a5
Variable do_install value changed:
@@ -1,3 +1,7 @@
autotools_do_install
install -d ${D}${sysconfdir}/default/volatiles
install -m 0644 ${WORKDIR}/volatiles.04_pulse ${D}${sysconfdir}/default/volatiles/volatiles.04_pulse
+ if [ -e "${WORKDIR}/daemon.conf" ] && [ -e "${WORKDIR}/default.pa" ]; then
+ install -m 0644 ${WORKDIR}/daemon.conf ${D}${sysconfdir}/pulse/daemon.conf
+ install -m 0644 ${WORKDIR}/default.pa ${D}${sysconfdir}/pulse/default.pa
+ fi
[YOCTO #11161]
(From OE-Core rev: 312edd42b6cc553de4d476c76e8e36a882e11cdd)
Signed-off-by: Patrick Ohly <patrick.ohly@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
2017-04-11 18:38:39 +00:00
|
|
|
msg.append(' %s' % cmd)
|
|
|
|
try:
|
|
|
|
output = check_command('Determining signature difference failed.',
|
|
|
|
cmd).decode('utf-8')
|
|
|
|
except RuntimeError as error:
|
|
|
|
output = str(error)
|
|
|
|
if output:
|
|
|
|
msg.extend([' ' + line for line in output.splitlines()])
|
|
|
|
msg.append('')
|
|
|
|
self.fail('\n'.join(msg))
|