[oe-commits] Paul Eggleton : classes/package: set SUMMARY in do_split_packages()

[git at git.openembedded.org]

Module: openembedded-core.git
Branch: master
Commit: 2b3080831042ffab26d70c3feee232fc31aa591a
URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=2b3080831042ffab26d70c3feee232fc31aa591a

Author: Paul Eggleton <paul.eggleton at linux.intel.com>
Date:   Sat Dec 28 19:58:39 2013 +0000

classes/package: set SUMMARY in do_split_packages()

do_split_packages() is commonly used to split out plugin/module packages
dynamically within recipes. If it doesn't set SUMMARY for each of these
packages then they get a generic SUMMARY from the recipe, which isn't
particularly useful; so add a parameter to set this and default it from
the current description parameter (it ought to have been the other way
around, but the description parameter is what we currently have in use
by all recipes that use this function.)

Fixes [YOCTO #5406].

Signed-off-by: Paul Eggleton <paul.eggleton at linux.intel.com>

---

 meta/classes/package.bbclass | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index eb4cf44..9b4f11a 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -72,7 +72,7 @@ def legitimize_package_name(s):
     # Remaining package name validity fixes
     return s.lower().replace('_', '-').replace('@', '+').replace(',', '+').replace('/', '-')
 
-def do_split_packages(d, root, file_regex, output_pattern, description, postinst=None, recursive=False, hook=None, extra_depends=None, aux_files_pattern=None, postrm=None, allow_dirs=False, prepend=False, match_path=False, aux_files_pattern_verbatim=None, allow_links=False):
+def do_split_packages(d, root, file_regex, output_pattern, description, postinst=None, recursive=False, hook=None, extra_depends=None, aux_files_pattern=None, postrm=None, allow_dirs=False, prepend=False, match_path=False, aux_files_pattern_verbatim=None, allow_links=False, summary=None):
     """
     Used in .bb files to split up dynamically generated subpackages of a
     given package, usually plugins or modules.
@@ -116,6 +116,8 @@ def do_split_packages(d, root, file_regex, output_pattern, description, postinst
                       package name. Can be a single string item or a list
                       of strings for multiple items. Must include %s.
     allow_links    -- True to allow symlinks to be matched - default False
+    summary        -- Summary to set for each package. Must include %s;
+                      defaults to description if not set.
 
     """
 
@@ -161,6 +163,9 @@ def do_split_packages(d, root, file_regex, output_pattern, description, postinst
     if extra_depends == None:
         extra_depends = d.getVar("PN", True)
 
+    if not summary:
+        summary = description
+
     for o in sorted(objs):
         import re, stat
         if match_path:
@@ -206,6 +211,7 @@ def do_split_packages(d, root, file_regex, output_pattern, description, postinst
             if extra_depends != '':
                 d.appendVar('RDEPENDS_' + pkg, ' ' + extra_depends)
             d.setVar('DESCRIPTION_' + pkg, description % on)
+            d.setVar('SUMMARY_' + pkg, summary % on)
             if postinst:
                 d.setVar('pkg_postinst_' + pkg, postinst)
             if postrm: