Skip to content

GitPython on OSX 10.7 and Python 2.7 Bug #1

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

Closed
kennethreitz opened this issue Jul 4, 2011 · 7 comments
Closed

GitPython on OSX 10.7 and Python 2.7 Bug #1

kennethreitz opened this issue Jul 4, 2011 · 7 comments
Assignees

Comments

@kennethreitz
Copy link
Contributor

Not sure what's causing this, but help would be appreciated :)

Exception TypeError: "'NoneType' object is not callable" in <bound method WindowCursor.__del__ of <smmap.mman.WindowCursor object at 0x101547f18>> ignored

@Byron
Copy link
Member

Byron commented Jul 5, 2011

How could I reproduce this ? Its not happening when running the unittests.

@Autoplectic Autoplectic mentioned this issue Dec 5, 2011
@kennethreitz
Copy link
Contributor Author

I'd really love to see this fixed as soon as possible. I'll pay you money.

@Byron
Copy link
Member

Byron commented Jan 17, 2012

I must apologize for being so slow with this, but my life kept me on the edge and made it hard to get started on non-commercial projects again. However, money is not the issue here.

Would it help you if I just merged your fix in ? I wouldn't want to release it though as I intend to get rid of the getrefcount() calls alltogether.

Tomorrow I will move into another office, which keeps the commercial work, so I can, should and must continue on working on my opensource projects at home - plenty of issues piled up in the past months.

@Autoplectic
Copy link

could you merge my fix and state in the commit messages that it is only a temporary fix, until you can refactor and fix things the way you envision? it would be nice to have a version of smmap out on pypi which didn't raise exceptions, even if the fix is just a temporary hack. the proper solution -- your re-write of things -- could be in a following major release version.

@kennethreitz
Copy link
Contributor Author

@Autoplectic +100.

@ghost ghost assigned Byron Jan 18, 2012
@Byron
Copy link
Member

Byron commented Jan 18, 2012

This is now fixed using Kenneth's workaround.
A new version was uploaded to pipy: http://pypi.python.org/pypi/smmap/0.8.2 .

Doing this wasn't so bad after all, lets call it a start :).
Thanks for all your patience !

@Byron Byron closed this as completed Jan 18, 2012
@kennethreitz
Copy link
Contributor Author

You sir, are a gentleman and a scholar!

Enjoy your cake!

✨ 🍰 ✨

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

No branches or pull requests

3 participants