Bug 267882 - devel/py-reedsolo Compatibility with pyhton 3.10
Summary: devel/py-reedsolo Compatibility with pyhton 3.10
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Fukang Chen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-11-20 18:18 UTC by Teodor Sigaev
Modified: 2022-11-22 15:03 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (loader)


Attachments
patch (2.73 KB, patch)
2022-11-20 18:18 UTC, Teodor Sigaev
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Teodor Sigaev 2022-11-20 18:18:11 UTC
Created attachment 238196 [details]
patch

Hi!

I would like to suggest small patch for supporting Python 3.10 by devel/py-reedsolo required by comms/py-esptool which is needed to work with ESP8266 at least.

Python 3.10 provides an pair setter/getter to object's reference counter instead of direct access to counter. Suggested patch wraps that by macro.
Comment 1 commit-hook freebsd_committer freebsd_triage 2022-11-22 12:15:30 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=c3aa37c126e21308f0e21947d96ff8b0e85e0fa8

commit c3aa37c126e21308f0e21947d96ff8b0e85e0fa8
Author:     Fukang Chen <loader@FreeBSD.org>
AuthorDate: 2022-11-22 11:37:54 +0000
Commit:     Fukang Chen <loader@FreeBSD.org>
CommitDate: 2022-11-22 12:13:30 +0000

    devel/py-reedsolo: Allow build with Python 3.10+

    - Regenerate creedsolo.c with Cython, which fixes
      build error with Python 3.10+

    - Switched Nose to Pytest, for the nosetests issue [0]

    [0]: https://github.com/nose-devs/nose/issues/1124

    PR:             267882
    Reported by:    teodorsigaev@gmail.com
    MFH:            2022Q4 (build fix)

 devel/py-reedsolo/Makefile | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)
Comment 2 Teodor Sigaev 2022-11-22 15:03:19 UTC
Thank you very much! It just works