package.bbclass: Breakout the strip from the populate_packages

Break out the file split/strip from the populate_packages.

Signed-off-by: Mark Hatle <mark.hatle@windriver.com>

1 files changed, 36 insertions, 22 deletions

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index fd29aaa031..fa1abf0dda 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -11,24 +11,27 @@
 #
 # b) package_do_split_locales - Split out the locale files, updates FILES and PACKAGES
 #
-# c) populate_packages - Split the files in PKGD into separate packages in PKGDEST/<pkgname>
+# c) split_and_strip_files - split the files into runtime and debug and strip them.
+#    Debug files include debug info split, and associated sources that end up in -dbg packages
+#
+# d) populate_packages - Split the files in PKGD into separate packages in PKGDEST/<pkgname>
 #    Also triggers the binary stripping code to put files in -dbg packages.
 #
-# d) package_do_filedeps - Collect perfile run-time dependency metadata
+# e) package_do_filedeps - Collect perfile run-time dependency metadata
 #    The data is stores in FILER{PROVIDES,DEPENDS}_file_pkg variables with
 #    a list of affected files in FILER{PROVIDES,DEPENDS}FLIST_pkg
 #
-# e) package_do_shlibs - Look at the shared libraries generated and autotmatically add any 
+# f) package_do_shlibs - Look at the shared libraries generated and autotmatically add any 
 #    depenedencies found. Also stores the package name so anyone else using this library 
 #    knows which package to depend on.
 #
-# f) package_do_pkgconfig - Keep track of which packages need and provide which .pc files
+# g) package_do_pkgconfig - Keep track of which packages need and provide which .pc files
 #
-# g) read_shlibdeps - Reads the stored shlibs information into the metadata
+# h) read_shlibdeps - Reads the stored shlibs information into the metadata
 #
-# h) package_depchains - Adds automatic dependencies to -dbg and -dev packages
+# i) package_depchains - Adds automatic dependencies to -dbg and -dev packages
 #
-# i) emit_pkgdata - saves the packaging data into PKGDATA_DIR for use in later 
+# j) emit_pkgdata - saves the packaging data into PKGDATA_DIR for use in later 
 #    packaging steps
 
 inherit packagedata
@@ -163,7 +166,7 @@ python () {
 }
 
 def runstrip(file, d):
-    # Function to strip a single file, called from populate_packages below
+    # Function to strip a single file, called from split_and_strip_files below
     # A working 'file' (one which works on the target architecture)
     # is necessary for this stuff to work, hence the addition to do_package[depends]
 
@@ -329,16 +332,11 @@ python perform_packagecopy () {
         os.system('tar -cf - -C %s -ps . | tar -xf - -C %s' % (dest, dvar))
 }
 
-python populate_packages () {
-        import glob, stat, errno, re
+python split_and_strip_files () {
+        import stat
 
-        workdir = bb.data.getVar('WORKDIR', d, True)
-        outdir = bb.data.getVar('DEPLOY_DIR', d, True)
         dvar = bb.data.getVar('PKGD', d, True)
-        packages = bb.data.getVar('PACKAGES', d, True)
-        pn = bb.data.getVar('PN', d, True)
 
-        bb.mkdirhier(outdir)
         os.chdir(dvar)
 
         def isexec(path):
@@ -348,6 +346,28 @@ python populate_packages () {
                         return 0
                 return (s[stat.ST_MODE] & stat.S_IEXEC)
 
+        # Figure out which packages we want to process
+        if (bb.data.getVar('INHIBIT_PACKAGE_STRIP', d, True) != '1'):
+                for root, dirs, files in os.walk(dvar):
+                        for f in files:
+                                file = os.path.join(root, f)
+                                if not os.path.islink(file) and not os.path.isdir(file) and isexec(file):
+                                        runstrip(file, d)
+
+}
+
+python populate_packages () {
+        import glob, stat, errno, re
+
+        workdir = bb.data.getVar('WORKDIR', d, True)
+        outdir = bb.data.getVar('DEPLOY_DIR', d, True)
+        dvar = bb.data.getVar('PKGD', d, True)
+        packages = bb.data.getVar('PACKAGES', d, True)
+        pn = bb.data.getVar('PN', d, True)
+
+        bb.mkdirhier(outdir)
+        os.chdir(dvar)
+
         # Sanity check PACKAGES for duplicates - should be moved to 
         # sanity.bbclass once we have the infrastucture
         package_list = []
@@ -360,13 +380,6 @@ python populate_packages () {
                 else:
                         package_list.append(pkg)
 
-        if (bb.data.getVar('INHIBIT_PACKAGE_STRIP', d, True) != '1'):
-                for root, dirs, files in os.walk(dvar):
-                        for f in files:
-                                file = os.path.join(root, f)
-                                if not os.path.islink(file) and not os.path.isdir(file) and isexec(file):
-                                        runstrip(file, d)
-
         pkgdest = bb.data.getVar('PKGDEST', d, True)
         os.system('rm -rf %s' % pkgdest)
 
@@ -1081,6 +1094,7 @@ PACKAGE_PREPROCESS_FUNCS ?= ""
 PACKAGEFUNCS ?= "perform_packagecopy \
                 ${PACKAGE_PREPROCESS_FUNCS} \
                 package_do_split_locales \
+                split_and_strip_files \
                 populate_packages \
                 package_do_filedeps \
                 package_do_shlibs \