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.

Sign up for GitHub

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

Jump to bottom

<registerAutoPackageIO> broken with zope.interface 4.7.0 #97

Closed
jamadden opened this issue Nov 11, 2019 · 2 comments · Fixed by #99
Closed

<registerAutoPackageIO> broken with zope.interface 4.7.0 #97

jamadden opened this issue Nov 11, 2019 · 2 comments · Fixed by #99
Labels
bug

Comments

@jamadden
Copy link
Member

Now that tagged values are inherited, we find more classes than we should, leading to duplicate registrations. (zopefoundation/zope.interface#144 (comment))
@jamadden jamadden added the bug label Nov 11, 2019
@jamadden
Copy link
Member Author

No tests in this package break when using 4.7.0 so we need to write one to duplicate the situation we observed.

@jamadden
Copy link
Member Author

It takes two or more implementations of a derived interface to trigger the problem. A class alias does the trick.

jamadden added a commit that referenced this issue Nov 12, 2019
@jamadden
De-duplicate inherited tagged values for registerAutoPackageIO.
79e82e5 
Fixes #97
@jamadden jamadden mentioned this issue Nov 12, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

De-duplicate inherited tagged values for registerAutoPackageIO. OpenNTI/nti.externalization
1 participant
@jamadden