From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-core@lists.openembedded.org
Subject: [RFC][PATCH] SHLIBSSEARCHDIRS
Date: Sun, 7 Jul 2013 01:15:54 +0200 [thread overview]
Message-ID: <1373152554-22840-1-git-send-email-Martin.Jansa@gmail.com> (raw)
In-Reply-To: <20130706230840.GE3288@jama>
---
meta/classes/package.bbclass | 17 ++++++++++++-----
1 file changed, 12 insertions(+), 5 deletions(-)
diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass
index 3f2deca..55035b5 100644
--- a/meta/classes/package.bbclass
+++ b/meta/classes/package.bbclass
@@ -1322,11 +1322,18 @@ python package_do_shlibs() {
libdir_re = re.compile(".*/%s$" % d.getVar('baselib', True))
shlibs_search_dirs = d.getVar('SHLIBSSEARCHDIRS', True)
- for dir in shlibs_search_dirs:
+ shlibs_search_dirs_re_txt = ""
+ for dir in shlibs_search_dirs.split(' '):
+ # strip leading and trailing slash, it's added in regexp
+ if dir.endswith("/"):
+ dir = dir[:-1]
+ if dir.startswith("/"):
+ dir = dir[1:]
if shlibs_search_dirs_re:
- shlibs_search_dirs_re += "\|"
- shlibs_search_dirs_re += "\(^.*\/%s\/.*$\)" % dir
- bb.debug(2, "will use following RE to search for provides sonames %s" % shlibs_search_dirs)
+ shlibs_search_dirs_re_txt += "\|"
+ shlibs_search_dirs_re_txt += "\(^.*/%s/.*$\)" % dir
+ shlibs_search_dirs_re = re.compile(shlibs_search_dirs_re_txt)
+ bb.debug(2, "will use following RE to search for provides sonames %s" % shlibs_search_dirs_re_txt)
packages = d.getVar('PACKAGES', True)
targetos = d.getVar('TARGET_OS', True)
@@ -1389,7 +1396,7 @@ python package_do_shlibs() {
if not private_libs or -1 == private_libs.find(this_soname):
sonames.append(this_soname)
else:
- bb.debug(2, "ignoring soname %s from %s, because path doesn't match %s" % (this_soname, file, shlibs_search_dirs))
+ bb.debug(2, "ignoring soname %s from %s, because path doesn't match %s" % (this_soname, file, shlibs_search_dirs_re_txt))
if libdir_re.match(os.path.dirname(file)):
needs_ldconfig = True
if snap_symlinks and (os.path.basename(file) != this_soname):
--
1.8.2.1
next prev parent reply other threads:[~2013-07-06 23:15 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-05 1:28 [PATCH 0/8] Missing deps discovered in per-recipe builds Martin Jansa
2013-07-05 1:28 ` [PATCH 1/8] (lib)telepathy*: add missing dependency on libxslt-native Martin Jansa
2013-07-05 1:28 ` [PATCH 2/8] gst-plugins-bad: inherit gsettings Martin Jansa
2013-07-05 1:28 ` [PATCH 3/8] gst-plugins-base: add missing dependency on glib-2.0-native Martin Jansa
2013-07-05 1:28 ` [PATCH 4/8] taglib: add missing dependency on zlib Martin Jansa
2013-07-05 1:28 ` [PATCH 5/8] quota: inherit pkgconfig Martin Jansa
2013-07-05 1:28 ` [PATCH 6/8] libpam: " Martin Jansa
2013-07-05 1:28 ` [PATCH 7/8] systemtap: " Martin Jansa
2013-07-05 1:29 ` [PATCH 8/8] eglibc-locale: add missing dependency on virtual/${TARGET_PREFIX}binutils Martin Jansa
2013-07-05 22:43 ` Martin Jansa
2013-07-06 8:39 ` [PATCHv2 1/2] " Martin Jansa
2013-07-06 8:39 ` [PATCHv2 2/2] subversion: explicitly disable sasl Martin Jansa
2013-07-08 14:06 ` Jonathan Liu
2013-07-08 15:01 ` Martin Jansa
2013-07-06 22:56 ` [PATCHv2 1/2] eglibc-locale: add missing dependency on virtual/${TARGET_PREFIX}binutils Chris Larson
2013-07-06 23:08 ` Martin Jansa
2013-07-06 23:15 ` Martin Jansa [this message]
2013-07-06 23:17 ` [RFC][PATCH] SHLIBSSEARCHDIRS Martin Jansa
2013-07-06 23:16 ` [RFC][PATCH] base.bbclass: add binutils to BASEDEPENDS Martin Jansa
2013-07-06 23:20 ` [PATCHv2 1/2] eglibc-locale: add missing dependency on virtual/${TARGET_PREFIX}binutils Chris Larson
2013-07-06 23:32 ` Martin Jansa
2013-07-08 3:30 ` Khem Raj
2013-07-05 11:17 ` [PATCH 0/8] Missing deps discovered in per-recipe builds Burton, Ross
2013-07-05 15:28 ` Paul Eggleton
2013-07-05 15:37 ` Burton, Ross
2013-07-06 12:29 ` Martin Jansa
2013-07-06 21:24 ` [RFC][PATCH] test-dependencies: add simple script to detect missing or autoenabled dependencies Martin Jansa
2013-07-06 21:39 ` Martin Jansa
2013-07-08 18:00 ` Randy MacLeod
2013-07-08 19:35 ` Martin Jansa
2013-07-10 12:38 ` [PATCHv2] " Martin Jansa
2013-07-10 12:41 ` Martin Jansa
2013-07-10 14:34 ` Burton, Ross
2013-07-10 14:57 ` Martin Jansa
2013-07-10 13:05 ` sstate-cache and making a package "host-dependent" Mike Looijmans
2013-07-10 17:39 ` Paul Eggleton
2013-07-10 17:45 ` Martin Jansa
2013-07-29 13:17 ` Mike Looijmans
2013-07-11 12:38 ` [PATCHv2] test-dependencies: add simple script to detect missing or autoenabled dependencies Martin Jansa
2013-07-11 13:28 ` Chris Larson
2013-07-11 13:49 ` Martin Jansa
2013-07-11 13:56 ` Chris Larson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1373152554-22840-1-git-send-email-Martin.Jansa@gmail.com \
--to=martin.jansa@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox