Bug 8031 - devtool selftest produces recipes that cause warnings
Summary: devtool selftest produces recipes that cause warnings
Status: VERIFIED FIXED
Alias: None
Product: Runtime Testing
Classification: QA/Testing
Component: general (show other bugs)
Version: 1.9
Hardware: All Multiple
: Low normal
Target Milestone: 1.9
Assignee: Paul Eggleton
QA Contact: Bogdan Alexandru Voiculescu
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2015-07-20 14:25 UTC by Ross Burton
Modified: 2015-09-11 12:18 UTC (History)
1 user (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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