Opened 9 months ago

Last modified 9 months ago

#68931 closed defect

py-sip builds contain incorrect code. — at Version 1

Reported by: SiggyF Owned by: michaelld (Michael Dickens)
Priority: Normal Milestone:
Component: ports Version:
Keywords: ventura sonoma Cc: reneeotten (Renee Otten)
Port: py-sip

Description (last modified by ryandesign (Ryan Carsten Schmidt))

There was a bug in the 3.8.0 version of py-sip. This bug was fixed two weeks ago in 3.8.1. This also solved #68828. Now the 3.8.1 is available. Somehow the old code ended up in the builds of the pre-build 3.8.1 version.

This can be checked in for example the file:

https://packages.macports.org/py311-sip/py311-sip-6.8.1_0.darwin_22.arm64.tbz2

There we see the old code:

opt/local/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/sipbuild/generator/outputs/code.py
8417:    if klass.docstring is None or klass.docstring.signature is not SocstringSignature.DISCARDED:

I don't understand how this old codes ends up in this file. I checked the distfile and that contains the correct code (without the typo *Soc*string).

Change History (1)

comment:1 Changed 9 months ago by ryandesign (Ryan Carsten Schmidt)

Cc: reneeotten added
Description: modified (diff)
Owner: set to michaelld
Status: newassigned

I assume you mean 6.8.0 and 6.8.1 rather than 3.8.0 and 3.8.1.

The problem is that the revision was not increased when #68898 was resolved. macOS Ventura and later contain a new implementation of patch that behaves differently from that in older macOS versions. In older macOS versions an error was issued when asked to apply a patch that was already applied, which was what prompted #68898. But the patch program in Ventura and later silently applied the patch in reverse.

Last edited 9 months ago by ryandesign (Ryan Carsten Schmidt) (previous) (diff)
Note: See TracTickets for help on using tickets.