On 6/5/08, Stefan Behnel <[EMAIL PROTECTED]> wrote:
> Doesn't
>
>   module_name = full_module_name or self.extract_module_name(source, options)
>
>  work for you?

It's not working, but I believe I definitelly catched the problem...

> "full_module_name" is something that is extracted from
>  distutils, so it's the best guess we can start with if it's provided.

But then what are those lines near the begining of  Context.compile(),
pasted below ?

        if full_module_name is None:
            full_module_name, _ = os.path.splitext(source)
            full_module_name = re.sub(r'[\\/]', '.', full_module_name)
            full_module_name = re.sub(r'[^\w.]', '_', full_module_name)

Iff I comment-out those lines, then all works with your latest patch.
And now I believe they should be removed. It seems that all the
previous hackery was in fact what was letting Cython manage modules
better than Pyrex (which required *.pyx files with full dotted names
for package/module management, righ?). In short, the lines above seems
to be obsolete...



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


-- 
Lisandro Dalcín
---------------
Centro Internacional de Métodos Computacionales en Ingeniería (CIMEC)
Instituto de Desarrollo Tecnológico para la Industria Química (INTEC)
Consejo Nacional de Investigaciones Científicas y Técnicas (CONICET)
PTLC - Güemes 3450, (3000) Santa Fe, Argentina
Tel/Fax: +54-(0)342-451.1594
_______________________________________________
Cython-dev mailing list
[email protected]
http://codespeak.net/mailman/listinfo/cython-dev

Reply via email to