On Oct 6, 2008, at 4:33 PM, Robert Kern wrote:

> Robert Bradshaw wrote:
>> On Oct 6, 2008, at 3:54 PM, Robert Kern wrote:
>>>
>>> Oh, and don't forget to add cython.py to py_modules=, otherwise  
>>> I  don't think it
>>> will get installed correctly.
>> For those who want to ship sources without the cython dependancy,  
>> I  they would just copy Shadow.py and name in cython.py in their  
>> own  directory.
>
> Ah. I see the intended workflow, now.
>
>> Maybe there's a cleaner way to do this without renaming  either  
>> "cython" or "Cython." Or should we put it there for testing   
>> purposes?
>
> Personally, I'd just rename the thing to "cythonize" and avoid the  
> whole mess. I don't see much profit in building multiple  
> workarounds just to keep a name.

It's a question of making the (internal) code a bit uglier to make  
the user interface nicer.

>>> While I'm looking at the setup.py script, why are the modules in   
>>> pyximport being
>>> listed separately in py_modules= instead of being listed as a   
>>> package in packages=?
>> I'm not sure, other than that I wasn't able to get the "obvious"  
>> way  to work right away. I'm not distutils guru though, so if  
>> there's a  better way I'd welcome a patch.
>
> The attached should do.

Thanks. Have you tried it with pyximport on a fresh Python install?  
(I think this is what I did first and had issues, but I would be  
happy to be proven wrong.)

>
> -- 
> Robert Kern
>
> "I have come to believe that the whole world is an enigma, a  
> harmless enigma
>  that is made terrible by our own mad attempt to interpret it as  
> though it had
>  an underlying truth."
>   -- Umberto Eco
> diff -r f4b39b81687b setup.py
> --- a/setup.py  Wed Sep 03 18:24:45 2008 +0200
> +++ b/setup.py  Mon Oct 06 18:32:29 2008 -0500
> @@ -97,10 +97,8 @@
>
>      'Cython.Tests',
>      'Cython.Compiler.Tests',
> +    'pyximport',
>      ],
> -
> -  # pyximport
> -  py_modules = ["pyximport/pyximport", "pyximport/pyxbuild"],
>
>    **setup_args
>    )_______________________________________________
> Cython-dev mailing list
> [email protected]
> http://codespeak.net/mailman/listinfo/cython-dev

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

Reply via email to