From b21d192d8f46c8696559e48a78ad3bd88e0403d7 Mon Sep 17 00:00:00 2001
From: Davide Madrisan <davide.madrisan@gmail.com>
Date: Fri, 19 Jul 2013 18:39:18 +0200
Subject: [PATCH] test01_pkgquality: Check for wrong file attributes in bin and
 lib directories.

Do ignore files with name matching '*.so.owner'.
These are text files needed by OCaml, not libraries.

Signed-off-by: Davide Madrisan <davide.madrisan@gmail.com>
---
 ChangeLog                  | 7 ++++++-
 tests/test01_pkgquality.in | 5 +++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index f925f6b..e9bc7bc 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -6,7 +6,12 @@ Sat Jul 13 2013 Madrisan <davide.madrisan(a)gmail.com>
     Replace 'notify --error' by 'notify.error'.
 
   * tests/test01_pkgquality - Davide Madrisan:
-    Test fo bad BuildArch tags: modify regexp to catch OCaml library files.
+    Test for bad BuildArch tags: modify regexp to catch OCaml library files.
+
+  * tests/test01_pkgquality - Davide Madrisan:
+    Check for wrong file attributes in bin and lib directories:
+    Do ignore files with name matching '*.so.owner'. These are text files
+    used by OCaml.
 
 -------------------------------------------------------------------------------
 
diff --git a/tests/test01_pkgquality.in b/tests/test01_pkgquality.in
index 4f8d497..a61c534 100644
--- a/tests/test01_pkgquality.in
+++ b/tests/test01_pkgquality.in
@@ -218,8 +218,9 @@ $"checking for wrong file attributes in bin and lib directories""${NORM}..."
    for pck in ${rpmpkg_name[@]}; do
       pushd $tmpextractdir/$i >/dev/null
       for f in $( find . -type f \( \
-         \( -name '*.so.*' -not -perm 755 \) -or \
-         \( -name '*.so' -not -perm 755 \) -or \
+         \( -name '*.so.*' -and -not -name '*.so.owner' \
+               -and -not -perm 755 \) -or \
+         \( -name '*.so' -and -not -perm 755 \) -or \
          \( \( -path './bin/*' -or \
                -path './sbin/*' -or \
                -path './usr/bin/*' -or \