Robert wrote:
> Dag Sverre Seljebotn wrote:
>> Robert (not Bradshaw), can you perhaps submit the patch with the reload
>> stuff taken out for now?
>
> what is the time schedule?
> a new one is there:
> http://trac.cython.org/cython_trac/attachment/ticket/312/
> http://trac.cython.org/cython_trac/attachment/ticket/312/pyxi3.patch

A release candidate is dropped any time I get time for it, certainly
within the next 24 hours. After then this would have to go in 0.11.3 (it
might have to anyway, not sure yet).

> did you with install(... reload_support=True) ? or without?

With.

> I just saw that on Linux you can (even without any
> Cython/pyximport) overwrite the .so,  then reload(themodule) and:
> Bus error or Segfault.
>
> On Windows you cannot overwrite/remove/rename a loaded DLL file -
> and I (implicitly) relied on that - even when
> reload_support=False. The build,linking actually fails automatically.

I will have to try your new patch, but don't think I'll get time before
the release.

> this is what implicitely happens/happened here with
> reload_support=True , loading from another temporary file - so far
> on Windows. Yet he iterated just upon the write blockage. Yet low
> level via imp.load_dynamic (no restriction on the shape of filename).
> ( The thing with extra python file is 95%+ not necessary I guess.
> As its a low level DLL/filesystem thing, not about the name in
> sys.modules or so)

I find threads on the internet with people struggling with the PEP which
pyximport uses and reload functionality; I'm fairly sure it is more
complicated than this.

I'd rather have a patch with only the configuration-file related changes,
otherwise this will definitely have to wait for the next release (if I was
less time-constrained I would have been kinder :-) ).

(It's also nicer if you can use "hg export" to create the patches.)

Dag Sverre

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

Reply via email to