[gentoo-commits] proj/qa-scripts:master commit in: /

[Michael Palimaka]

commit:     bf90e71df4e003dada5464a941a20f4c70a4c755
Author:     Michael Palimaka <kensington <AT> gentoo <DOT> org>
AuthorDate: Wed Sep 24 16:40:19 2014 +0000
Commit:     Michael Palimaka <kensington <AT> gentoo <DOT> org>
CommitDate: Wed Sep 24 16:43:21 2014 +0000
URL:        
http://sources.gentoo.org/gitweb/?p=proj/qa-scripts.git;a=commit;h=bf90e71d

depcheck: ensure that missing RDEPEND is also reported during a virtual check

It was failing due to a difference in return code handling:
        a() {
                local x=$(false)
                echo $?
        }

        b() {
                local x
                x=$(false)
                echo $?
        }
        a
        b

returning:
        0
        1

---
 depcheck | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/depcheck b/depcheck
index dae0cd9..93abeb1 100755
--- a/depcheck
+++ b/depcheck
@@ -58,7 +58,8 @@ virtualcheck() {
        for virtual in $(qdepends --nocolor --name-only --${1} --query 
${libowner_pn} | grep ^virtual/)
        do
                debug Checking if ${virtual} is in dependencies
-               local isvirtualdep=$(qdepends --${1} ${atom} | grep ${virtual})
+               local isvirtualdep
+               isvirtualdep=$(qdepends --${1} ${atom} | grep ${virtual})
 
                if [ $? -eq 0 ]; then
                        used_virtuals+=( ${virtual} )