[PATCH] classes/insane: fix libdir check regexes

[PATCH] classes/insane: fix libdir check regexes

[Paul Eggleton]

* Ensure these do not pick up e.g. /lib/systemd/system/uuidd.socket
* Fix broken exec_re regex (lib*. -> lib.*)

Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
 meta/classes/insane.bbclass | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index 27c225f..af419f1 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -265,8 +265,8 @@ def package_qa_check_libdir(d):
             full_path = os.path.join(root,file)
             my_files.append(full_path[len(pkgd):])
 
-    lib_re = re.compile("^/lib.*\.so")
-    exec_re = re.compile("^%s.*/lib*.\.so" % exec_prefix)
+    lib_re = re.compile("^/lib.*\.so[0-9.]*$")
+    exec_re = re.compile("^%s.*/lib.*\.so[0-9.]*$" % exec_prefix)
 
     for file in my_files:
         if lib_re.match(file):
-- 
1.8.1.2

Re: [PATCH] classes/insane: fix libdir check regexes

[Burton, Ross]

On 3 July 2013 14:40, Paul Eggleton <paul.eggleton@linux.intel.com> wrote:
> +    lib_re = re.compile("^/lib.*\.so[0-9.]*$")
> +    exec_re = re.compile("^%s.*/lib.*\.so[0-9.]*$" % exec_prefix)

I'd been meaning to double-check this assumption since the discussion
yesterday, and I'm glad I did.  My Debian machine has these files in
/usr/lib that don't match that regex:

libatlas.so.3gf
libblas.so.3gf
libcblas.so.3gf
libf77blas.so.3gf
liblapack_atlas.so.3gf
liblapack.so.3gf
libmozjs.so.10d
libnspr4.so.0d
libnss3.so.1d
libnssutil3.so.1d
libplc4.so.0d
libplds4.so.0d
libsmime3.so.1d
libssl3.so.1d

How about ^lib.+\.so(\..+)?$ to match a library filename?

Ross

Re: [PATCH] classes/insane: fix libdir check regexes

[Paul Eggleton]

On Wednesday 03 July 2013 15:35:15 Burton, Ross wrote:
> On 3 July 2013 14:40, Paul Eggleton <paul.eggleton@linux.intel.com> wrote:
> > +    lib_re = re.compile("^/lib.*\.so[0-9.]*$")
> > +    exec_re = re.compile("^%s.*/lib.*\.so[0-9.]*$" % exec_prefix)
> 
> I'd been meaning to double-check this assumption since the discussion
> yesterday, and I'm glad I did.  My Debian machine has these files in
> /usr/lib that don't match that regex:
> 
> libatlas.so.3gf
> libblas.so.3gf
> libcblas.so.3gf
> libf77blas.so.3gf
> liblapack_atlas.so.3gf
> liblapack.so.3gf
> libmozjs.so.10d
> libnspr4.so.0d
> libnss3.so.1d
> libnssutil3.so.1d
> libplc4.so.0d
> libplds4.so.0d
> libsmime3.so.1d
> libssl3.so.1d
> 
> How about ^lib.+\.so(\..+)?$ to match a library filename?

Hmm, right. That looks reasonable. v2 sent.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre