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

work around poor interaction between whatever changed in zope.interface 4.3.3's usage of namespace packages and py2app #701

Merged
merged 3 commits into from
Dec 14, 2016

Conversation

glyph
Copy link
Contributor

@glyph glyph commented Dec 14, 2016

@glyph glyph changed the title Fix zi 433 work around poor interaction between whatever changed in zope.interface 4.3.3's usage of namespace packages and py2app Dec 14, 2016
@codecov-io
Copy link

codecov-io commented Dec 14, 2016

Current coverage is 99.60% (diff: 100%)

Merging #701 into master will not change coverage

@@             master       #701   diff @@
==========================================
  Files            79         79          
  Lines          5564       5564          
  Methods           0          0          
  Messages          0          0          
  Branches        785        785          
==========================================
  Hits           5542       5542          
  Misses           10         10          
  Partials         12         12          

Powered by Codecov. Last update 71d33ff...7b033a5

@@ -24,6 +24,10 @@ pip install .;
# install dependencies that are needed to build and run the mac application
pip install -r requirements/mac-app.txt;

# work around zope.interface's namespace module shenanigans
# see https://github.com/zopefoundation/zope.interface/issues/67
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@glyph Should we track a todo to remove this with a mimic issue linked to zopefoundation/zope.interface#67?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Opened #702 to track this work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants