1 |
commit: 28ada16af437c0f2955f222abd03730ae2e299d6 |
2 |
Author: Georgy Yakovlev <gyakovlev <AT> gentoo <DOT> org> |
3 |
AuthorDate: Mon Jun 28 18:12:02 2021 +0000 |
4 |
Commit: Georgy Yakovlev <gyakovlev <AT> gentoo <DOT> org> |
5 |
CommitDate: Mon Jun 28 18:46:57 2021 +0000 |
6 |
URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=28ada16a |
7 |
|
8 |
metadata/install-qa-check.d: add QA Notice string to all checks |
9 |
|
10 |
and re-wrap some lines to fit into 80 char limit |
11 |
|
12 |
Bug: https://bugs.gentoo.org/728046 |
13 |
Signed-off-by: Georgy Yakovlev <gyakovlev <AT> gentoo.org> |
14 |
|
15 |
metadata/install-qa-check.d/08gentoo-paths | 2 +- |
16 |
metadata/install-qa-check.d/60appdata-path | 4 ++-- |
17 |
metadata/install-qa-check.d/60distutils-use-setuptools | 6 ++++-- |
18 |
metadata/install-qa-check.d/60gtk-doc-paths | 6 +++--- |
19 |
metadata/install-qa-check.d/60python-pyc | 7 ++++--- |
20 |
metadata/install-qa-check.d/60python-tests | 2 +- |
21 |
6 files changed, 15 insertions(+), 12 deletions(-) |
22 |
|
23 |
diff --git a/metadata/install-qa-check.d/08gentoo-paths b/metadata/install-qa-check.d/08gentoo-paths |
24 |
index 5161aef9922..4d84925d853 100644 |
25 |
--- a/metadata/install-qa-check.d/08gentoo-paths |
26 |
+++ b/metadata/install-qa-check.d/08gentoo-paths |
27 |
@@ -66,7 +66,7 @@ gentoo_path_check() { |
28 |
# report |
29 |
# ------ |
30 |
if [[ -n ${bad_paths[@]} ]]; then |
31 |
- eqawarn "The ebuild is installing to one or more unexpected paths:" |
32 |
+ eqawarn "QA Notice: The ebuild is installing to one or more unexpected paths:" |
33 |
eqawarn |
34 |
eqatag -v non-gentoo-paths "${bad_paths[@]}" |
35 |
eqawarn |
36 |
|
37 |
diff --git a/metadata/install-qa-check.d/60appdata-path b/metadata/install-qa-check.d/60appdata-path |
38 |
index a2403083a49..06c5f91e8ce 100644 |
39 |
--- a/metadata/install-qa-check.d/60appdata-path |
40 |
+++ b/metadata/install-qa-check.d/60appdata-path |
41 |
@@ -13,8 +13,8 @@ appdata_path_check() { |
42 |
done < <(find "${ED%/}/usr/share/appdata" -name '*.xml' -print0 || die) |
43 |
|
44 |
if [[ ${found[@]} ]]; then |
45 |
- eqawarn "QA Notice: This package seems to install metainfo files into the following" |
46 |
- eqawarn "location(s):" |
47 |
+ eqawarn "QA Notice: This package seems to install metainfo files into the" |
48 |
+ eqawarn "following location(s):" |
49 |
eqawarn |
50 |
eqatag -v appdata.invalid-path "${found[@]#${D%/}}" |
51 |
eqawarn |
52 |
|
53 |
diff --git a/metadata/install-qa-check.d/60distutils-use-setuptools b/metadata/install-qa-check.d/60distutils-use-setuptools |
54 |
index effa65358aa..d08be885eb8 100644 |
55 |
--- a/metadata/install-qa-check.d/60distutils-use-setuptools |
56 |
+++ b/metadata/install-qa-check.d/60distutils-use-setuptools |
57 |
@@ -47,7 +47,9 @@ distutils_use_setuptools_check() { |
58 |
|
59 |
if [[ ${#expected[@]} -gt 1 ]] && has no "${expected[@]}"; then |
60 |
# 'no' and '[rb]depend' are mutually exclusive |
61 |
- eerror "The package seems to have used distutils and setuptools simultaneously." |
62 |
+ eerror "QA Notice: The package seems to have used distutils and setuptools" |
63 |
+ eerror "simultaneously" |
64 |
+ eerror "" |
65 |
eerror "This could mean the package has bad conditions:" |
66 |
eerror "https://dev.gentoo.org/~mgorny/python-guide/distutils.html#conditional-distutils-setuptools-use-in-packages" |
67 |
eerror "Please report a bug about this and CC python@" |
68 |
@@ -74,7 +76,7 @@ distutils_use_setuptools_check() { |
69 |
local def= |
70 |
[[ ${DISTUTILS_USE_SETUPTOOLS} == bdepend ]] && def=' (or unset)' |
71 |
|
72 |
- eqawarn "DISTUTILS_USE_SETUPTOOLS value is probably incorrect" |
73 |
+ eqawarn "QA Notice: DISTUTILS_USE_SETUPTOOLS value is probably incorrect" |
74 |
eqawarn " have: DISTUTILS_USE_SETUPTOOLS=${DISTUTILS_USE_SETUPTOOLS}${def}" |
75 |
if [[ ${expected[0]} == bdepend ]]; then |
76 |
eqawarn " expected: (unset)" |
77 |
|
78 |
diff --git a/metadata/install-qa-check.d/60gtk-doc-paths b/metadata/install-qa-check.d/60gtk-doc-paths |
79 |
index 866e871c221..8c12c260433 100644 |
80 |
--- a/metadata/install-qa-check.d/60gtk-doc-paths |
81 |
+++ b/metadata/install-qa-check.d/60gtk-doc-paths |
82 |
@@ -1,4 +1,4 @@ |
83 |
-# Copyright 1999-2015 Gentoo Foundation |
84 |
+# Copyright 1999-2021 Gentoo Authors |
85 |
# Distributed under the terms of the GNU General Public License v2 |
86 |
|
87 |
# QA check: ensure that gtk-doc files are installed in /usr/share/gtk-doc |
88 |
@@ -14,8 +14,8 @@ gtk_doc_path_check() { |
89 |
|
90 |
if [[ ${found[@]} ]]; then |
91 |
eqawarn |
92 |
- eqawarn "This package seems to install gtk-doc documentation into the following" |
93 |
- eqawarn "location(s):" |
94 |
+ eqawarn "QA Notice: This package seems to install gtk-doc documentation into the" |
95 |
+ eqawarn "following location(s):" |
96 |
eqawarn |
97 |
eqatag -v gtk-doc-paths.invalid-path "${found[@]#${D%/}}" |
98 |
eqawarn |
99 |
|
100 |
diff --git a/metadata/install-qa-check.d/60python-pyc b/metadata/install-qa-check.d/60python-pyc |
101 |
index ddc1bf531cd..e112cdce348 100644 |
102 |
--- a/metadata/install-qa-check.d/60python-pyc |
103 |
+++ b/metadata/install-qa-check.d/60python-pyc |
104 |
@@ -55,7 +55,8 @@ if [[ ${EAPI} == [6-7] ]]; then |
105 |
|
106 |
if [[ ${missing[@]} ]]; then |
107 |
eqawarn |
108 |
- eqawarn "This package installs one or more Python modules that are not byte-compiled." |
109 |
+ eqawarn "QA Notice: This package installs one or more Python modules that are" |
110 |
+ eqawarn "not byte-compiled." |
111 |
eqawarn "The following files are missing:" |
112 |
eqawarn |
113 |
eqatag -v python-pyc.missing "${missing[@]#${D}}" |
114 |
@@ -63,8 +64,8 @@ if [[ ${EAPI} == [6-7] ]]; then |
115 |
|
116 |
if [[ ${outdated[@]} ]]; then |
117 |
eqawarn |
118 |
- eqawarn "This package installs one or more compiled Python modules that have older" |
119 |
- eqawarn "timestamps than the corresponding source files:" |
120 |
+ eqawarn "QA Notice: This package installs one or more compiled Python modules that have" |
121 |
+ eqawarn "older timestamps than the corresponding source files:" |
122 |
eqawarn |
123 |
eqatag -v python-pyc.outdated "${outdated[@]#${D}}" |
124 |
fi |
125 |
|
126 |
diff --git a/metadata/install-qa-check.d/60python-tests b/metadata/install-qa-check.d/60python-tests |
127 |
index 713ed63d376..4c7892fbc93 100644 |
128 |
--- a/metadata/install-qa-check.d/60python-tests |
129 |
+++ b/metadata/install-qa-check.d/60python-tests |
130 |
@@ -17,7 +17,7 @@ python_test_check() { |
131 |
[[ -n ${any} ]] || return |
132 |
|
133 |
eqawarn |
134 |
- eqawarn 'QA Notice: this package seems to contain tests but they are not enabled.' |
135 |
+ eqawarn 'QA Notice: This package seems to contain tests but they are not enabled.' |
136 |
eqawarn 'Please either run tests (via distutils_enable_tests or declaring' |
137 |
eqawarn 'python_test yourself), or add RESTRICT="test" along with an explanatory' |
138 |
eqawarn 'comment if tests cannot be run.' |