Bug 9389 - oeqa: split configuration from the code
Summary: oeqa: split configuration from the code
Status: RESOLVED WONTFIX
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: core (show other bugs)
Version: 2.2
Hardware: x86 Multiple
: Medium enhancement
Target Milestone: 2.2 M3
Assignee: Jose Lamego
QA Contact:
URL:
Whiteboard:
Depends on: 9390
Blocks:
  Show dependency tree
 
Reported: 2016-04-05 03:18 UTC by Robert Yang
Modified: 2016-08-16 22:46 UTC (History)
4 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Robert Yang 2016-04-05 03:18:44 UTC 
For example, in meta/lib/oeqa/selftest/archiver.py:

        features = 'INHERIT += "archiver"\n'
        features += 'ARCHIVER_MODE[src] = "original"\n'
        features += 'COPYLEFT_PN_INCLUDE = "%s"\n' % include_recipe
        features += 'COPYLEFT_PN_EXCLUDE = "%s"\n' % exclude_recipe


We can put these in a conf file, the benefit is:

* The user can easily know what is happened
* We can easily add or modify the test case
Comment 1 Leonardo Sandoval Gonzalez 2016-04-28 15:58:04 UTC 
Robert, where do you suggest to place the conf file?
Comment 2 Robert Yang 2016-04-29 06:10:13 UTC 
(In reply to comment #1)
> Robert, where do you suggest to place the conf file?

You can use a proper location you like, for example, in oeqa:
meta/lib/oeqa/selftest/config_files/signing.conf
Comment 3 Jose Lamego 2016-06-27 17:31:49 UTC 
Updating target milestone. Patches send pending.
Comment 4 Jose Lamego 2016-08-16 22:46:02 UTC 
A solution was developed for the requested feature: [1], however, the change proved to be too intrusive and the resulting code hard-to-read enough to make it nonviable to implement.

[1] http://lists.openembedded.org/pipermail/openembedded-core/2016-August/124867.html