Gentoo Archives: gentoo-commits

From: Sam James <sam@g.o>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] repo/gentoo:master commit in: metadata/install-qa-check.d/
Date: Wed, 03 Aug 2022 01:53:10
Message-Id: 1659491581.bfeb100e92991a4df927215b4f32d8908c081797.sam@gentoo
1 commit: bfeb100e92991a4df927215b4f32d8908c081797
2 Author: Sam James <sam <AT> gentoo <DOT> org>
3 AuthorDate: Wed Aug 3 01:49:49 2022 +0000
4 Commit: Sam James <sam <AT> gentoo <DOT> org>
5 CommitDate: Wed Aug 3 01:53:01 2022 +0000
6 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=bfeb100e
7
8 metadata/install-qa-check.d: skip 60python-pc if gpep517 too old
9
10 When building libseccomp, say:
11 ```
12 * Verifying compiled files for python3.9
13 usage: /usr/lib/python-exec/python3.9/gpep517 [-h] {get-backend,build-wheel,install-wheel} ...
14 /usr/lib/python-exec/python3.9/gpep517: error: argument command: invalid choice: 'verify-pyc' (choose from 'get-backend', 'build-wheel', 'install-wheel')
15 strip: i686-pc-linux-gnu-strip --strip-unneeded -N __gentoo_check_ldflags__ -R .comment -R .GCC.command.line -R .note.gnu.gold-version
16 ```
17
18 The gpep517 >= dep is in distutils-r1 but this check applies to other
19 ebuilds. So, use has_version -b.
20
21 Signed-off-by: Sam James <sam <AT> gentoo.org>
22
23 metadata/install-qa-check.d/60python-pyc | 5 +++++
24 1 file changed, 5 insertions(+)
25
26 diff --git a/metadata/install-qa-check.d/60python-pyc b/metadata/install-qa-check.d/60python-pyc
27 index fd9434994f2f..37b0acaf7e0a 100644
28 --- a/metadata/install-qa-check.d/60python-pyc
29 +++ b/metadata/install-qa-check.d/60python-pyc
30 @@ -16,6 +16,11 @@ python_pyc_check() {
31 local missing=()
32 local stray=()
33
34 + # Avoid running the check if sufficiently new gpep517 is not installed
35 + # yet. It's valid to schedule (for merge order) >=gpep517-8 after
36 + # packages which have this check run if they don't use distutils-r1.
37 + has_version -b ">=dev-python/gpep517-8" || return
38 +
39 for prog in "${progs[@]}"; do
40 local impl=${prog%/*}
41 impl=${impl##*/}