On Wed, Dec 22, 2010 at 12:48 AM, Robert Bradshaw
<[email protected]> wrote:
> On Tue, Dec 21, 2010 at 11:49 PM, Alexey Borzenkov <[email protected]> wrote:
>> On Wed, Dec 22, 2010 at 1:19 AM, Alexey Borzenkov <[email protected]> wrote:
>>> I've recently started using Cython in one of my projects because I
>>> needed pyximport and automatic building of extensions and found that
>>> adding .pyxdep files makes Cython raise NameError because _test_files
>>> is not defined.
>> [...]
>>> Also, there's another problem with pyximport that makes it unsafe to
>>> use it without build_in_temp=False. For example, if I have
>>> package1/module.pyx and package2/module.pyx then only one of those
>>> will build/import correctly,
>>
>> Btw, I just realized that Cython is finally using git and github, so
>> if anyone is interested here are my patches:
>>
>> https://github.com/snaury/cython/commit/b43f68280919e98767bd4d163cb1eaf88c4c857e
>> https://github.com/snaury/cython/commit/6f73b4280958f5d3f5fa77c3a4ed19b277784140
>
> Thanks. pyximport is really cool, but no one's been maintaining it,
> and it looks like we haven't been testing it either. (Code is either
> new, tested, or broken...) At first glance the patches look fine, I'll
> try to take a deeper look over the holidays (if no one beats me to
> it...)

Merged. Thanks!

- Robert
_______________________________________________
Cython-dev mailing list
[email protected]
http://codespeak.net/mailman/listinfo/cython-dev

Reply via email to