Gentoo Archives: gentoo-commits

From: Richard Farina <zerochaos@g.o>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] proj/qa-scripts:master commit in: /, tinderbox/
Date: Thu, 28 Aug 2014 18:12:53
Message-Id: 1409246254.b90b4fb30b9f9b0b37c8804d770dcc454fcc6bd1.zerochaos@gentoo
1 commit: b90b4fb30b9f9b0b37c8804d770dcc454fcc6bd1
2 Author: Rick Farina (Zero_Chaos) <zerochaos <AT> gentoo <DOT> org>
3 AuthorDate: Thu Aug 28 17:17:34 2014 +0000
4 Commit: Richard Farina <zerochaos <AT> gentoo <DOT> org>
5 CommitDate: Thu Aug 28 17:17:34 2014 +0000
6 URL: http://sources.gentoo.org/gitweb/?p=proj/qa-scripts.git;a=commit;h=b90b4fb3
7
8 adjust output for use with portage
9
10 Adjust output so that things look better when running stand-alone,
11 by category, or from within portage.
12
13 ---
14 depcheck | 22 ++++++++++++++--------
15 tinderbox/bashrc | 7 +++++--
16 2 files changed, 19 insertions(+), 10 deletions(-)
17
18 diff --git a/depcheck b/depcheck
19 index a0b022b..87fdacb 100755
20 --- a/depcheck
21 +++ b/depcheck
22 @@ -69,8 +69,10 @@ check_atom() {
23 return
24 fi
25
26 - einfo Checking ${atom} for undeclared dependencies
27 - eindent
28 + if [ -z "${INSIDE_PORTAGE}" ]; then
29 + einfo Checking ${atom} for undeclared dependencies
30 + eindent
31 + fi
32
33 local obj
34 for obj in $objects
35 @@ -198,8 +200,10 @@ check_atom() {
36 ewarn "Suspect RDEPEND: $(bold ${suspect_rdepends})"
37 fi
38
39 - echo
40 - eoutdent
41 + if [ -n "${CAT}" ]; then
42 + eoutdent
43 + echo
44 + fi
45
46 return $errors
47
48 @@ -277,8 +281,10 @@ else
49 check_package $1
50 fi
51
52 -if [ $? -eq 0 ]; then
53 - einfo Checking complete, no errors found
54 -else
55 - eerror Checking complete, errors found
56 +if [ $DEBUG = TRUE ]; then
57 + if [ $? -eq 0 ]; then
58 + einfo Checking complete, no errors found
59 + else
60 + eerror Checking complete, errors found
61 + fi
62 fi
63
64 diff --git a/tinderbox/bashrc b/tinderbox/bashrc
65 index 560c3fd..8533e2b 100644
66 --- a/tinderbox/bashrc
67 +++ b/tinderbox/bashrc
68 @@ -115,9 +115,12 @@ tinderbox_postinst_checks() {
69 fi
70
71 if type -P "${DEPCHECK_BIN}" > /dev/null ; then
72 - SANDBOX_ON=0 "${DEPCHECK_BIN}" ${PF} > "${BUILD_DATA_DIR}"/depcheck
73 + SANDBOX_ON=0 INSIDE_PORTAGE=1 "${DEPCHECK_BIN}" ${PF} > "${BUILD_DATA_DIR}"/depcheck
74 if [[ "${TINDERBOX_VERBOSE}" == true ]] ; then
75 - cat "${BUILD_DATA_DIR}"/depcheck
76 + if [ -f "${BUILD_DATA_DIR}"/depcheck ] && [ -n "$(cat "${BUILD_DATA_DIR}"/depcheck)" ] ; then
77 + eqawarn "QA Notice:"
78 + eqawarn "$(cat "${BUILD_DATA_DIR}"/depcheck)"
79 + fi
80 fi
81 fi
82 }