[Pyrex] New Distutils implementation for Pyrex.

[Stefan Behnel]

Stefan Behnel wrote:
> Stefan Behnel wrote:
>> Billy G. Allie wrote:
>>>  Robert Bradshaw wrote:
>>>> On Aug 1, 2007, at 5:55 PM, Billy G. Allie wrote:
>>>>> pyrex-cplus: tells Pyrex to use the C++ compiler to compile the  
>>>>> generated file.
>>>>>     
>>>> Is this not handled by the "language" parameter in the  
>>>> distutils.core.Extension used in setup.py
>>>>   
>>> If Pyrex is not told to use the C++ compiler, it will generate a '.c'
>>> file instead of a '.cpp' file.
> 
> Then it should know about the "language" option rather than requiring its own one.

Here is a patch for that (I don't use C++, so I can't test it...)


> So, what I'd propose here is to just let Cython obey the include_dirs option
> of distutils and have it search its .pxd files there if they are not found in
> the same directory as the source file. I don't mind if Cython or my C compiler
> search a few directories more than necessary, as long as they find what I
> need. And having a single place where include directories are defined makes it
> easier to use and keeps us from defining new options.

And another patch that supports the Extension local include_dirs option.

Stefan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: language-option.patch
Type: text/x-diff
Size: 2996 bytes
Desc: not available
Url : http://lists.copyleft.no/pipermail/pyrex/attachments/20070806/d19493de/attachment-0002.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: include_dirs.patch
Type: text/x-diff
Size: 1717 bytes
Desc: not available
Url : http://lists.copyleft.no/pipermail/pyrex/attachments/20070806/d19493de/attachment-0003.bin