Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for issue 8743 not available in python MacOS 3.5.1 #74325

Closed
HoracioHoyos mannequin opened this issue Apr 22, 2017 · 5 comments
Closed

Fix for issue 8743 not available in python MacOS 3.5.1 #74325

HoracioHoyos mannequin opened this issue Apr 22, 2017 · 5 comments
Labels
OS-mac stdlib Python modules in the Lib dir

Comments

@HoracioHoyos
Copy link
Mannequin

HoracioHoyos mannequin commented Apr 22, 2017

BPO 30139
Nosy @ronaldoussoren, @ned-deily, @bitdancer, @serhiy-storchaka

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2017-04-22.19:31:34.337>
created_at = <Date 2017-04-22.18:54:20.271>
labels = ['OS-mac', 'library']
title = 'Fix for issue 8743 not available in python MacOS 3.5.1'
updated_at = <Date 2017-04-22.19:50:56.163>
user = 'https://bugs.python.org/HoracioHoyos'

bugs.python.org fields:

activity = <Date 2017-04-22.19:50:56.163>
actor = 'ned.deily'
assignee = 'none'
closed = True
closed_date = <Date 2017-04-22.19:31:34.337>
closer = 'serhiy.storchaka'
components = ['Library (Lib)', 'macOS']
creation = <Date 2017-04-22.18:54:20.271>
creator = 'Horacio Hoyos'
dependencies = []
files = []
hgrepos = []
issue_num = 30139
keywords = []
message_count = 5.0
messages = ['292134', '292135', '292136', '292137', '292138']
nosy_count = 5.0
nosy_names = ['ronaldoussoren', 'ned.deily', 'r.david.murray', 'serhiy.storchaka', 'Horacio Hoyos']
pr_nums = []
priority = 'normal'
resolution = 'works for me'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue30139'
versions = ['Python 3.5']

@HoracioHoyos
Copy link
Mannequin Author

HoracioHoyos mannequin commented Apr 22, 2017

Hi all,

I was having issues while testing a custom Set implementation using the _collections_abc base MutableSet and found that my issue was apparently resolved with bpo-8743. My test is simple:

ms = MySetImpl()
ms & 'testword'

which should fail with TypeError, given that in the 8743 fix the __and__ incorporated a test for isinstance(other, Set).

Looking at the _collections_abc.py in my installation (/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/_collections_abc.py) I can not see the changes in the patches submitted for bpo-8743.

@HoracioHoyos HoracioHoyos mannequin added stdlib Python modules in the Lib dir OS-mac labels Apr 22, 2017
@bitdancer
Copy link
Member

Then you have a broken copy of 3.5.1. There is nothing OS-specific about that file.

@ned-deily
Copy link
Member

> Then you have a broken copy of 3.5.1.

Yes. You don't say from where you installed Python 3.5.1 but I just took a quick look at _collections_abc.py in both the versions installed by the python.org 3.5.1 macOS installer and by the current 3.5.3 version (which you should update to, or, better, 3.6.1) and the bpo-8743 changes appear to be there. If you still believe there is a problem with Python here, please reopen and provide a complete test case.

@serhiy-storchaka
Copy link
Member

But there is no neither an entry bpo-8743, nor neighbour entries bpo-14710 and bpo-13355 in Misc/NEWS. Seems Misc/NEWS is broken.

@ned-deily
Copy link
Member

But there is no neither an entry bpo-8743, nor neighbour entries bpo-14710 and bpo-13355 in Misc/NEWS. Seems Misc/NEWS is broken.

Hmm. They seem to have been properly preserved in the current 3.6 and master branch versions of Misc/HISTORY. Perhaps the 3.5 Misc/NEWS was corrupted in a merge along the way? In any case, this is not relevant to the OP's problem. Once the auto-generation of Misc/NEWS is implemented, we're going to have to go back and redo Misc/NEWS and Misc/HISTORY in active branches. I don't think there is a need for a separate issue to track that but, if you want to open one, Serhiy, feel free to do so.

@ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OS-mac stdlib Python modules in the Lib dir
Projects
None yet
Development

No branches or pull requests

3 participants