Github messages for voidlinux
 help / color / mirror / Atom feed
From: loreb <loreb@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: Re: [WIP] New package: mnemosyne-2.10.1
Date: Fri, 08 Sep 2023 16:25:42 +0200	[thread overview]
Message-ID: <20230908142542.0lEFPO5hmdc2xt_EytdhPyTF8FEacdBOKqlO_RmnF_8@z> (raw)
In-Reply-To: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-40063@inbox.vuxu.org>

[-- Attachment #1: Type: text/plain, Size: 1567 bytes --]

New comment by loreb on void-packages repository

https://github.com/void-linux/void-packages/pull/40063#issuecomment-1711759194

Comment:
The following (intentionally dirty) diff did it for me:
```
diff --git a/srcpkgs/python3-testfixtures/template b/srcpkgs/python3-testfixtures/template
index 6c71aad1..121bad83 100644
--- a/srcpkgs/python3-testfixtures/template
+++ b/srcpkgs/python3-testfixtures/template
@@ -12,10 +12,11 @@ license="MIT"
 homepage="https://github.com/Simplistix/testfixtures"
 distfiles="${PYPI_SITE}/t/testfixtures/testfixtures-${version}.tar.gz"
 checksum=456ce4f393160f27cd6840a5370ecf4a70f1721737f5e67a2af79e6c817804a4
+export PYTHONDONTWRITEBYTECODE=1 # NOPE
 
 do_check() {
-	cd build/lib
-	pytest -v --ignore="testfixtures/tests/test_django" # Fails to import
+	cd testfixtures &&
+	pytest -v --ignore="tests/test_django" # Fails to import
 }
 
 post_install() {
```

RANT: based on stackoverflow, I tried the python bytecode thing (did nothing, I just left it as a reminder that it's useless),
then I thought I understood when I read https://docs.pytest.org/en/latest/explanation/goodpractices.html#test-discovery but nope, no way...
I still don't know why it went all the way up instead of staying in build/lib, I expected it to stop at lib (you know, the first directory without __init__.py), but if it works it works.

And if you need more circular fun... at first I thought maybe they fixed the bug upstream, I tried pytest 7.4.2 and boom! circular dependencies.

Sorry for the rant, I just needed it.


  parent reply	other threads:[~2023-09-08 14:25 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-20 21:44 [PR PATCH] [WIP] New package: mnemosyne-2.9 Eloitor
2022-10-20 21:46 ` [PR PATCH] [Updated] " Eloitor
2022-10-20 22:13 ` Eloitor
2022-10-20 22:24 ` Eloitor
2022-10-20 23:31 ` classabbyamp
2022-10-20 23:32 ` classabbyamp
2022-10-21 13:02 ` [PR PATCH] [Updated] " Eloitor
2023-03-30  1:56 ` github-actions
2023-03-30 10:25 ` Eloitor
2023-05-26 11:10 ` [PR PATCH] [Updated] " Eloitor
2023-05-26 12:35 ` [PR PATCH] [Updated] [WIP] New package: mnemosyne-2.10.1 Eloitor
2023-05-26 12:50 ` Eloitor
2023-05-26 12:53 ` Eloitor
2023-05-26 12:55 ` Eloitor
2023-05-26 12:58 ` Eloitor
2023-06-14 16:21 ` loreb
2023-06-15 15:27 ` loreb
2023-08-29 13:02 ` loreb
2023-09-02 17:33 ` [PR PATCH] [Updated] " Eloitor
2023-09-02 18:06 ` Eloitor
2023-09-02 18:32 ` Eloitor
2023-09-02 18:33 ` Eloitor
2023-09-02 18:37 ` [PR PATCH] [Updated] " Eloitor
2023-09-04 15:20 ` loreb
2023-09-04 22:27 ` [PR PATCH] [Updated] " Eloitor
2023-09-05 13:57 ` [PR REVIEW] " loreb
2023-09-05 13:59 ` loreb
2023-09-05 14:33 ` [PR PATCH] [Updated] " Eloitor
2023-09-05 14:41 ` Eloitor
2023-09-05 14:41 ` Eloitor
2023-09-05 15:25 ` loreb
2023-09-05 16:55 ` Eloitor
2023-09-06  5:35 ` Eloitor
2023-09-06  5:51 ` Eloitor
2023-09-06  9:02 ` [PR PATCH] [Updated] " Eloitor
2023-09-06  9:05 ` Eloitor
2023-09-06 13:10 ` loreb
2023-09-06 17:04 ` [PR PATCH] [Updated] " Eloitor
2023-09-06 17:28 ` Eloitor
2023-09-06 17:41 ` Eloitor
2023-09-06 17:47 ` Eloitor
2023-09-06 18:02 ` Eloitor
2023-09-06 18:07 ` Eloitor
2023-09-07 12:32 ` loreb
2023-09-07 15:26 ` [PR PATCH] [Updated] " Eloitor
2023-09-07 15:28 ` Eloitor
2023-09-07 15:45 ` Eloitor
2023-09-07 15:51 ` Eloitor
2023-09-07 16:04 ` [PR PATCH] [Updated] " Eloitor
2023-09-07 16:08 ` Eloitor
2023-09-07 16:12 ` Eloitor
2023-09-07 16:19 ` Eloitor
2023-09-08 14:25 ` loreb [this message]
2023-09-08 15:56 ` [PR PATCH] [Updated] " Eloitor
2023-09-08 15:56 ` Eloitor
2023-12-08  1:46 ` github-actions
2023-12-09 16:12 ` loreb
2024-02-14  7:47 ` [PR PATCH] [Updated] " Eloitor
2024-02-14  8:02 ` [PR PATCH] [Updated] New package: mnemosyne-2.11 Eloitor
2024-02-14  8:03 ` Eloitor
2024-05-15  1:48 ` github-actions

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230908142542.0lEFPO5hmdc2xt_EytdhPyTF8FEacdBOKqlO_RmnF_8@z \
    --to=loreb@users.noreply.github.com \
    --cc=ml@inbox.vuxu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).