Re: [PATCH] package install failure on systems without /sbin/ldconfig

[Amy Fong <amy.fong@windriver.com>]

On Mon, Jul 09, 2012 at 09:23:46PM -0700, Saul Wold wrote:
> On 07/09/2012 11:50 AM, Amy Fong wrote:
> >Package install failures due to issues in post install rules.
> >
> >package_do_shlibs only looks for libraries in a directory "lib", this should be
> >modified to the variable baselib.
> >
> >ldconfig_postinst_fragment failure observed on systems without /sbin/ldconfig
> >         [ -x /sbin/ldconfig ] && /sbin/ldconfig
> >results in the post install rule returning a failure. Modify to
> >         if [ -x /sbin/ldconfig ]; then /sbin/ldconfig; fi
> >
> If this is addressing bug #2723, then please mark it as follows:
> 
> [YOCTO #2723] in the commit message.
> 
> Also the commit subject should include the file name for example:
> package.bbclass: fix for package install failure on systems without
> /sbin/ldconfig
> 
> Thanks
> 	Sau!
> 

Thanks Saul, updated as follows....

[YOCTO #2723] package.bbclass: fix for package install failure on systems without /sbin/ldconfig
    
package_do_shlibs only looks for libraries in a directory "lib", this should be
modified to the variable baselib.
    
ldconfig_postinst_fragment failure observed on systems without /sbin/ldconfig
    [ -x /sbin/ldconfig ] && /sbin/ldconfig
results in the post install rule returning a failure. Modify to
    if [ -x /sbin/ldconfig ]; then /sbin/ldconfig; fi
    
Signed-off-by: Amy Fong <amy.fong@windriver.com>
---
 package.bbclass |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index 56ab25c..28a263c 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -1151,7 +1151,7 @@ emit_pkgdata[dirs] = "${PKGDESTWORK}/runtime"
 
 ldconfig_postinst_fragment() {
 if [ x"$D" = "x" ]; then
-	[ -x /sbin/ldconfig ] && /sbin/ldconfig
+	if [ -x /sbin/ldconfig ]; then /sbin/ldconfig ; fi
 fi
 }
 
@@ -1261,7 +1261,7 @@ python package_do_shlibs() {
 		return
 		
 	lib_re = re.compile("^.*\.so")
-	libdir_re = re.compile(".*/lib$")
+	libdir_re = re.compile(".*/%s$" % d.getVar('baselib', True))
 
 	packages = d.getVar('PACKAGES', True)
 	targetos = d.getVar('TARGET_OS', True)