move ECM from kdesupport to frameworks
ClosedPublic

Authored by sitter on Jul 10 2017, 9:49 AM.

Details

Summary

as per thread on frameworks-devel.
Also drop i18n.json as per other frameworks. i18n settings are inherited
from a base rule for all frameworks.

Diff Detail

Repository
R247 Metadata for repositories
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sitter created this revision.Jul 10 2017, 9:49 AM

Also see D6604 for proposed introduction of a metainfo.yaml to be consistent with other frameworks

dfaure edited edge metadata.

+1 from me.

bcooksley edited edge metadata.Jul 11 2017, 7:52 AM

Please don't delete projects/kdesupport/extra-cmake-modules/i18n.json - this should be moved as well.
The i18n.json files go into the kde_projects.xml file which is used by scripty.

Aside from that, this patch looks fine. Please also ensure that the dependency data files in kde-build-metadata are also updated.

Please don't delete projects/kdesupport/extra-cmake-modules/i18n.json - this should be moved as well.
The i18n.json files go into the kde_projects.xml file which is used by scripty.

For frameworks the i18n is entirely derived from the defaults file

e.g. https://phabricator.kde.org/source/sysadmin-repo-metadata/browse/master/projects/frameworks/kdelibs4support/

So, this should be fine.

I did, however, forget to mention: the change effectively replaces trunk with trunk_kf5. Which is part of the motivation behind the move into frameworks, so ECM is fully aligned with the rest of frameworks WRT i18n.

bcooksley accepted this revision.Jul 11 2017, 11:16 AM

Assuming David is okay with ECM moving to Frameworks, this should be fine.
I have checked and it turns out other Frameworks don't have i18n.json files - I thought the tooling required one to be present, but if the others get away with not having one it should be fine.

This revision is now accepted and ready to land.Jul 11 2017, 11:16 AM

Yes, I already said +1 in this review request ;-)

This revision was automatically updated to reflect the committed changes.