From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 13CF915800A for ; Sat, 5 Aug 2023 15:10:26 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id F28992BC01E; Sat, 5 Aug 2023 15:10:21 +0000 (UTC) Received: from smtp.gentoo.org (woodpecker.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id BA3882BC013 for ; Sat, 5 Aug 2023 15:10:21 +0000 (UTC) Message-ID: Subject: Re: [gentoo-dev] [PATCH] python-utils-r1.eclass: Check for occluded packages before testing From: =?UTF-8?Q?Micha=C5=82_G=C3=B3rny?= To: gentoo-dev@lists.gentoo.org Date: Sat, 05 Aug 2023 17:10:17 +0200 In-Reply-To: <20230805123941.21249-1-mgorny@gentoo.org> References: <20230805123941.21249-1-mgorny@gentoo.org> Organization: Gentoo Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.48.4 Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-dev@lists.gentoo.org Reply-to: gentoo-dev@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply MIME-Version: 1.0 X-Archives-Salt: a7a37f50-78b2-47a6-a0ff-6aa8ee3964bb X-Archives-Hash: 5ef6d9c350c0311068d725cc0537541c On Sat, 2023-08-05 at 14:39 +0200, Micha=C5=82 G=C3=B3rny wrote: > Add a check for installed packages being occluded by the working > directory when calling epytest and eunittest. This is primarily meant > to detect C extensions being missed. By the way, I've finally figured out which package was when I've noticed the problem for this first time. It is dev-python/ijson -- if you remove the 'rm -rf' from test phase, you'd notice that a bunch of tests are being skipped because of missing C extension -- but that's easy to miss. With this patch, you get a more verbose QA warning. That said, I have no clue if we won't see many false positives, and that's why it's covered by a condition so that only people from Python team would enable it until we know better. --=20 Best regards, Micha=C5=82 G=C3=B3rny