Re: [OE-core] [PATCHv2 2/4] distrodata: Remove unnecessary include of package_regex.inc

2015-06-16 Thread Burton, Ross
On 4 June 2015 at 19:42, Aníbal Limón anibal.li...@linux.intel.com wrote:

 This causes a warning when follow documentation to use distrodata
 class that points to include,

 include conf/distro/include/distro_alias.inc
 include conf/distro/include/recipe_color.inc
 include conf/distro/include/maintainers.inc
 include conf/distro/include/upstream_tracking.inc
 include conf/distro/include/package_regex.inc
 INHERIT+= distrodata


Can we revert this? Instead make the class include the configuration files
that it needs, and update the documentation.  As Paul pointed out in a bug
earlier several of these files can be deleted now anyway, and having
distrodata working with just a simple INHERIT is much neater.

(by which I mean: I've reverted this in master-under-test, shout if you
have a good argument against this)

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCHv2 2/4] distrodata: Remove unnecessary include of package_regex.inc

2015-06-04 Thread Aníbal Limón
This causes a warning when follow documentation to use distrodata
class that points to include,

include conf/distro/include/distro_alias.inc
include conf/distro/include/recipe_color.inc
include conf/distro/include/maintainers.inc
include conf/distro/include/upstream_tracking.inc
include conf/distro/include/package_regex.inc
INHERIT+= distrodata

Signed-off-by: Aníbal Limón anibal.li...@linux.intel.com
---
 meta/classes/distrodata.bbclass | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meta/classes/distrodata.bbclass b/meta/classes/distrodata.bbclass
index 83aa381..e1fc6dd 100644
--- a/meta/classes/distrodata.bbclass
+++ b/meta/classes/distrodata.bbclass
@@ -1,4 +1,3 @@
-include conf/distro/include/package_regex.inc
 addhandler distro_eventhandler
 distro_eventhandler[eventmask] = bb.event.BuildStarted
 python distro_eventhandler() {
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core