<br><br><div class="gmail_quote">On Sat, Apr 21, 2012 at 20:54, Eric Snow <span dir="ltr">&lt;<a href="mailto:ericsnowcurrently@gmail.com">ericsnowcurrently@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Sat, Apr 21, 2012 at 4:17 PM, Brett Cannon &lt;<a href="mailto:brett@python.org">brett@python.org</a>&gt; wrote:<br>
&gt; On Sat, Apr 21, 2012 at 12:10, Barry Warsaw &lt;<a href="mailto:barry@python.org">barry@python.org</a>&gt; wrote:<br>
</div><div class="im">&gt;&gt; Strictly speaking, I do think you need to deprecate the APIs.  I like<br>
&gt;&gt; Nick&#39;s<br>
&gt;&gt; suggestion to make them C wrappers which just call back into Python.<br>
&gt;<br>
&gt;<br>
&gt; That was my plan, but the amount of code it will take to wrap them is making<br>
&gt; me not care. =) For PyImport_GetMagicTag() I would need to expose a new<br>
&gt; attribute on sys or somewhere which specifies the VM name. For<br>
&gt; PyImport_GetMagicNumber() I have to do a bunch of bit twiddling to convert a<br>
&gt; bytes object into a long which I am just flat-out not in the mood to figure<br>
&gt; out how to do. And all of this will lead to the same amount of C code as<br>
&gt; there currently is for what is already implemented, so I just don&#39;t care<br>
&gt; anymore. =)<br>
<br>
</div>I thought I already (mostly) worked it all out in that patch on<br>
issue13959.  I felt really good about the approach for the magic tag<br>
and magic bytes.<br></blockquote><div><br></div><div>You didn&#39;t update Python/import.c in your patches so that the public C API continued to function. That&#39;s what is going to take a bunch of C code to continue to maintain, not the Python side of it.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Once find_module() and reload() are done in imp.py, I&#39;m hoping to<br>
follow up on a few things.  That includes the unresolved mailing list<br>
thread about sys.implementation (or whatever it was), which will help<br>
with the magic tag.  Anyway, I don&#39;t want to curtail the gutting of<br>
import.c quite yet (as he hears cries of &quot;bring out your dead!&quot;).<br></blockquote><div><br></div><div>Even w/ all of that gutted, a decent chunk of coding is holding on to dear life thanks to PyImport_ExecCodeModuleObject() (and those that call it). IOW the C API as it is currently exposed is going to end up being the limiting factor of how many lines get deleted in the very end.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-eric<br>
<br>
<br>
p.s.  I understand your sentiment here, considering that mothers are<br>
often exhausted by childbirth and the importlib bootstrap was a big<br>
baby.  You were in labor for, what, 6 years.  &lt;wink&gt;  [There&#39;s an<br>
analogy that could keep on giving. :) ]<br>
</blockquote></div><br><div>It&#39;s also about maintainability. It isn&#39;t worth upping complexity just to shift some stuff into Python code, especially when it is such simple stuff as the magic number and tag which places practically zero burden on other VMs to implement.</div>