Bug 7895 - Tidy up recipe maintenance include files
Summary: Tidy up recipe maintenance include files
Status: VERIFIED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: core (show other bugs)
Version: 1.8
Hardware: All Multiple
: Low enhancement
Target Milestone: 1.9 M3
Assignee: Aníbal Limón
QA Contact: Bogdan Alexandru Voiculescu
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2015-06-16 09:02 UTC by Paul Eggleton
Modified: 2015-09-11 07:31 UTC (History)
7 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: Don't know


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Eggleton 2015-06-16 09:02:29 UTC
In meta-yocto/conf/distro/include we have a bunch of inc files that are intended to be used by distrodata.bbclass to work out who is maintaining each recipe and what the current state is. We also now have a database in the RRS which imports some of this and maintains other pieces itself.

We need to do a couple of things now:

1) Drop the files / definitions that no longer make sense now that we have the RRS. We can drop recipe_color.inc, and we should be able to drop the definitions other than RECIPE_NO_UPDATE_REASON in upstream_tracking.inc. Also, distro_alias.inc has a bunch of values that don't make a lot of sense out of context ("OSPDT" ?). Any items for recipes that we no longer have in OE-Core should be dropped as well.

2) Move the remaining files into OE-Core, so that they are in the same place as the recipes they refer to.
Comment 1 Alexander Kanavin 2015-06-30 13:30:07 UTC
I would go even further and split the contents of those files into respective recipes. Don't see why it needs to be separate from recipes and in one place. I can look into this after I'm done with checking the accuracy of upstream versions.
Comment 4 Paul Eggleton 2015-08-14 16:32:05 UTC
Is this really resolved though? Did you make the changes suggested under item 1 in the original report?
Comment 5 Aníbal Limón 2015-08-14 16:42:12 UTC
(In reply to comment #4)
> Is this really resolved though? Did you make the changes suggested under
> item 1 in the original report?

I dropped recipe_color.inc also un-updated entries in upstream_tracking.inc was removed.
Comment 6 Paul Eggleton 2015-08-14 16:56:53 UTC
OK, great - but the reason I ask because there are a number of entries still in distro_alias.inc for recipes that we removed a long time ago (e.g. aaina, libxtrap, msynctool, signgp, ...). I think we also need to do something about some of the values in distro_alias.inc as well as I mentioned above - the values other than Debian/Fedora/etc. just aren't going to make sense to most people.

Shall I create another bug for those things?
Comment 7 Alexander Kanavin 2015-08-17 12:31:00 UTC
Also maintainers.inc was not moved anywhere and remains in poky, for a reason that I didn't really understand ("the only missing file are maintainers.inc because it needs consensus between OE-Core and Yocto project to define a common set of maintainers").
Comment 8 Aníbal Limón 2015-08-17 14:46:00 UTC
(In reply to comment #7)
> Also maintainers.inc was not moved anywhere and remains in poky, for a
> reason that I didn't really understand ("the only missing file are
> maintainers.inc because it needs consensus between OE-Core and Yocto project
> to define a common set of maintainers").

I talked with RP about this move and the reason is Yocto project has it own maintainers that not necessarily are the same as in Openembedded project.
Comment 9 Aníbal Limón 2015-08-17 14:48:16 UTC
(In reply to comment #6)
> OK, great - but the reason I ask because there are a number of entries still
> in distro_alias.inc for recipes that we removed a long time ago (e.g. aaina,
> libxtrap, msynctool, signgp, ...). I think we also need to do something
> about some of the values in distro_alias.inc as well as I mentioned above -
> the values other than Debian/Fedora/etc. just aren't going to make sense to
> most people.
> 
> Shall I create another bug for those things?

Yes, please. I don't know how to proceed for clean up this file may be you are the person to do it.
Comment 10 Bogdan Alexandru Voiculescu 2015-09-11 07:31:12 UTC
Verified on master: d4585fa23d56a58c583ee33db067a23177a936b1

Now: ls ~/yocto/poky/meta-yocto/conf/distro/include
maintainers.inc  poky-floating-revisions.inc

before: 
ls ~/yocto/poky/meta-yocto/conf/distro/include/
distro_alias.inc  maintainers.inc  package_regex.inc  poky-floating-revisions.inc  recipe_color.inc  upstream_tracking.inc