From Dan Kenigsberg <dan...@redhat.com>:

Dan Kenigsberg has uploaded a new change for review.

Change subject: rpmlint: change to blacklist
......................................................................

rpmlint: change to blacklist

Currently, our rpms fail in 5 tests:

    explicit-lib-dependency
    no-binary
    zero-length
    non-readable
    non-standard-dir-perm

We should explicitly list the failing tests, to make sure that no new
failure is added.

Reviewed-on: 
https://gerrit.ovirt.org/#/q/I1a6e21b61b9b31a07d7cd3af2ed9ab8506c46c5a
Change-Id: I1a6e21b61b9b31a07d7cd3af2ed9ab8506c46c5a
Signed-off-by: Dan Kenigsberg <dan...@redhat.com>
---
M automation/check-patch.sh
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/44/69444/1

diff --git a/automation/check-patch.sh b/automation/check-patch.sh
index 939161b..732037d 100755
--- a/automation/check-patch.sh
+++ b/automation/check-patch.sh
@@ -30,5 +30,5 @@
     yum -y install "$EXPORT_DIR/"!(*.src).rpm
     export LC_ALL=C  # no idea why this is suddenly needed
     rpmlint "$EXPORT_DIR/"*.src.rpm
-    ! rpmlint "$EXPORT_DIR/"!(*.src).rpm | grep 
'\(dir-or-file-in-var-run\|wrong-script-interpreter\|non-executable-script\)'
+    ! rpmlint "$EXPORT_DIR/"!(*.src).rpm | grep ': E: ' | grep -v 
explicit-lib-dependency | grep -v no-binary | grep -v zero-length | grep -v 
non-readable | grep -v non-standard-dir-perm
 fi


-- 
To view, visit https://gerrit.ovirt.org/69444
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1a6e21b61b9b31a07d7cd3af2ed9ab8506c46c5a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to