Bug 8031

Summary:	devtool selftest produces recipes that cause warnings
Product:	[QA/Testing] Runtime Testing	Reporter:	Ross Burton <ross.burton>
Component:	general	Assignee:	Paul Eggleton <bluelightning>
Status:	VERIFIED FIXED	QA Contact:	Bogdan Alexandru Voiculescu <bogdanx.a.voiculescu>
Severity:	normal
Priority:	Low	CC:	bogdanx.a.voiculescu
Version:	1.9
Target Milestone:	1.9
Hardware:	All
OS:	Multiple
Whiteboard:
OS type for building Yocto:	---	Type of Regression:	---
Verified:		Documentation change:	No (bug/feature does not impact docs)

Description Ross Burton 2015-07-20 14:25:35 UTC

2015-07-20 15:07:23 - test_devtool_add_library (oeqa.selftest.devtool.DevtoolTests) ...
2015-07-20 15:08:49,605 - selftest.base - WARNING - tearDown commands have failed: bitbake -c cleansstate libftdi

Digging around the cooker logs reveals that the LICENSE value in the recipe isn't valid:

WARNING: libftdi: LICENSE value "Unknown GPLv2" has an invalid format - license names must be separated by the following characters to indicate the license selection: &|()

(I'm not sure why selftest doesn't report this as a problem when it builds the recipe, only when tearing down the test case)

We should either fix devtool to create valid LICENSE statements, or alternatively if the recipe is intended to produce a warning because the intention cannot be determined by the tool the test case should use a library that doesn't produce warnings when it is working as expected.

Comment 1 Paul Eggleton 2015-07-20 16:08:45 UTC

I don't think this is fixable - devtool cannot know whether it's a & or a | situation. It's just filling the list in for you, it's up to you to figure out the details (because it cannot).

I take the point that perhaps we should use a different library in the test.

Comment 2 Paul Eggleton 2015-07-31 09:42:32 UTC

FYI, whilst the proximity of these two warnings suggests that they are related, I believe the issue is actually that libftdi no longer exists when the -c cleansstate gets run because "devtool reset" deletes it. I've sent a patch for that.

Comment 3 Paul Eggleton 2015-08-05 09:08:28 UTC

teardown warning removed in this commit now merged to master:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=93360e8a1c0657542bb0536ad0281389d89b2d5d

I'm now marking this as resolved.

Comment 4 Bogdan Alexandru Voiculescu 2015-09-11 12:18:41 UTC

verified on master d4585fa23d56a58c583ee33db067a23177a936b1
with meta-oe d36e2d1066f50036080f978583a58fe79ecfac54