<br><br><div class="gmail_quote"><span style="font-size: large; font-weight: bold;">Forwarded conversation</span><br>Subject: <b class="gmail_sendername">Fixing zipfile.BadZipfile to zipfile.BadZipFile</b><br>------------------------<br>
<br><span class="undefined"><font color="#888">From: <b class="undefined">Boštjan Mejak</b> <span dir="ltr">&lt;<a href="mailto:bostjan.mejak@gmail.com">bostjan.mejak@gmail.com</a>&gt;</span><br>Date: Fri, Oct 15, 2010 at 11:02 PM<br>
To: <a href="mailto:python-dev@python.org">python-dev@python.org</a><br></font><br></span><br><span style="font-family:monospace;font-size:12px;line-height:17px"><pre style="white-space:pre-wrap;word-wrap:break-word">I am very glad you&#39;re reorganizing the Standard Library. Thumbs up! I hope everything will comply to PEP 8 after you&#39;re done.

Since you&#39;re reorganizing, I have my own contribution. I have attached a patch. The <a href="http://bugs.python.org/issue7351" target="_blank">issue7351</a> was not accepted at the time, so I hope you&#39;ll accept this fix now.

My point is that every class name in the module zipfile is like this:
- LargeZipFile
- ZipFile
- PyZipFile
.</pre><pre style="white-space:pre-wrap;word-wrap:break-word">.</pre><pre style="white-space:pre-wrap;word-wrap:break-word">.</pre><pre style="white-space:pre-wrap;word-wrap:break-word">So apply my patch to make the class name BadZipfile consistent to other class names in the zipfile module and let it be named BadZipFile. Thank you.</pre>

<pre style="white-space:pre-wrap;word-wrap:break-word"><br></pre><pre style="white-space:pre-wrap;word-wrap:break-word"><br></pre><pre style="white-space:pre-wrap;word-wrap:break-word">Best regards,</pre><pre style="white-space:pre-wrap;word-wrap:break-word">
Boštjan Mejak</pre></span>
<br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Éric Araujo</b> <span dir="ltr">&lt;<a href="mailto:merwok@netwok.org">merwok@netwok.org</a>&gt;</span><br>Date: Sun, Oct 17, 2010 at 11:00 PM<br>
To: <a href="mailto:python-dev@python.org">python-dev@python.org</a><br>Cc: Boštjan Mejak &lt;<a href="mailto:bostjan.mejak@gmail.com">bostjan.mejak@gmail.com</a>&gt;<br></font><br></span><br>Hello<br>
<br>
(A bit of context: The original message comes from bug #2775, “Implement<br>
PEP 3108”, a meta-bug tracking stdlib reorganization for py3k.)<br>
You may have missed the timeline: Most of the PEP 3108 changes have been<br>
done before the first 3.x release went out.  Now that we have 3.1 out as<br>
a stable and supported, we cannot reorganize and break compatibility<br>
anymore.<br>
<br>
(A note about PEP 8 compliance: Module names have been mostly fixed, but<br>
not all function/method names, for example in logging and unittest.  If<br>
I recall correctly, readability did not seem to make all the rewrites<br>
worth it.)<br>
&gt; a patch. The issue7351 &lt;<a href="http://bugs.python.org/issue7351" target="_blank">http://bugs.python.org/issue7351</a>&gt; was not<br>
I’ve just re-read the answers there and they are still valid.<br>
<br>
Ezio and me: “Your patch need to include an alias (BadZipfile =<br>
BadZipFile) to preserve compatibility with old pickles, as explains<br>
msg95477.”<br>
<br>
Antoine: “I don&#39;t think changing it for the sake of aesthetics is a good<br>
deal given that many existing programs will have to be converted to the<br>
new spelling.”<br>
<br>
Regards<br>
<br>
<br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Boštjan Mejak</b> <span dir="ltr">&lt;<a href="mailto:bostjan.mejak@gmail.com">bostjan.mejak@gmail.com</a>&gt;</span><br>Date: Mon, Oct 18, 2010 at 9:25 AM<br>
To: Éric Araujo &lt;<a href="mailto:merwok@netwok.org">merwok@netwok.org</a>&gt;<br></font><br></span><br>Shoot. Well, too bad. I thought Python is all about readability, but I think you developers don&#39;t take it very seriously. If I was a developer, I would certainly go to the trouble of all the rewrittes of package/module/class/method/function names that do not comply to PEP 8 and have them done by the time the first sub-version of Python 3 would be released.<div>
<div></div></div><br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Éric Araujo</b> <span dir="ltr">&lt;<a href="mailto:merwok@netwok.org">merwok@netwok.org</a>&gt;</span><br>Date: Mon, Oct 18, 2010 at 2:14 PM<br>
To: Boštjan Mejak &lt;<a href="mailto:bostjan.mejak@gmail.com">bostjan.mejak@gmail.com</a>&gt;<br></font><br></span><br>Hi<br>
<br>
Could you repost your message to the mailing list, so that other people<br>
can see it and eventually react?  Thanks.<br>
<br>
(P.S. please make sure not to top-post on the mailing list)<br>
<br>
Regards<br>
<br>
<br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Éric Araujo</b> <span dir="ltr">&lt;<a href="mailto:merwok@netwok.org">merwok@netwok.org</a>&gt;</span><br>Date: Mon, Oct 18, 2010 at 2:45 PM<br>
To: <a href="mailto:python-dev@python.org">python-dev@python.org</a><br></font><br></span><br>Hello<br>
<br>
[Sorry if this comes twice, connection errors here]<br>
&gt; a patch. The issue7351 &lt;<a href="http://bugs.python.org/issue7351" target="_blank">http://bugs.python.org/issue7351</a>&gt; was not<br>
_______________________________________________<br>
Python-Dev mailing list<br>
<a href="mailto:Python-Dev@python.org">Python-Dev@python.org</a><br>
<a href="http://mail.python.org/mailman/listinfo/python-dev" target="_blank">http://mail.python.org/mailman/listinfo/python-dev</a><br>
Unsubscribe: <a href="http://mail.python.org/mailman/options/python-dev/bostjan.mejak%40gmail.com" target="_blank">http://mail.python.org/mailman/options/python-dev/bostjan.mejak%40gmail.com</a><br>
<br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Nick Coghlan</b> <span dir="ltr">&lt;<a href="mailto:ncoghlan@gmail.com">ncoghlan@gmail.com</a>&gt;</span><br>Date: Mon, Oct 18, 2010 at 3:13 PM<br>
To: Éric Araujo &lt;<a href="mailto:merwok@netwok.org">merwok@netwok.org</a>&gt;<br>Cc: <a href="mailto:python-dev@python.org">python-dev@python.org</a><br></font><br></span><br>Correct. We went through this for one module that I recall (threading)<br>

and that was annoying enough that we mostly left things alone after<br>
that unless they were truly obnoxious.<br>
<br>
For threading we were able to clean a lot of things up in the process<br>
(such as adding properties where appropriate), but even so, we still<br>
made sure all the old names continued to work.<br>
<br>
Cheers,<br>
Nick.<br>
<font color="#888888"><br>
--<br>
Nick Coghlan   |   <a href="mailto:ncoghlan@gmail.com">ncoghlan@gmail.com</a>   |   Brisbane, Australia<br>
</font><div><div></div></div><br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Boštjan Mejak</b> <span dir="ltr">&lt;<a href="mailto:bostjan.mejak@gmail.com">bostjan.mejak@gmail.com</a>&gt;</span><br>
Date: Mon, Oct 18, 2010 at 4:36 PM<br>To: Nick Coghlan &lt;<a href="mailto:ncoghlan@gmail.com">ncoghlan@gmail.com</a>&gt;<br></font><br></span><br><div>Then please make an alias for this custom BadZipfile exception class.</div>

<div>Add  BadZipfile = BadZipFile  in the zipfile module.<br><br></div><div><div></div></div><br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Boštjan Mejak</b> <span dir="ltr">&lt;<a href="mailto:bostjan.mejak@gmail.com">bostjan.mejak@gmail.com</a>&gt;</span><br>
Date: Tue, Oct 19, 2010 at 10:34 PM<br>To: Nick Coghlan &lt;<a href="mailto:ncoghlan@gmail.com">ncoghlan@gmail.com</a>&gt;<br></font><br></span><br><div>Sorry, let me correct myself:</div><div> <span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse">Add  BadZipFile = BadZipfile in the zipfile module.</span></div>

<div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse"><br></span></div>Please add the above assignment statement in the right place to the zipfile module and create an alias. Thank you.<div>
<div></div></div><br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Boštjan Mejak</b> <span dir="ltr">&lt;<a href="mailto:bostjan.mejak@gmail.com">bostjan.mejak@gmail.com</a>&gt;</span><br>
Date: Wed, Oct 20, 2010 at 11:13 PM<br>To: Nick Coghlan &lt;<a href="mailto:ncoghlan@gmail.com">ncoghlan@gmail.com</a>&gt;<br></font><br></span><br>Ah, leave it as is. If in the future you will be able to rename BadZipfile to BadZipFile, please do.<div>
<div></div></div><br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Éric Araujo</b> <span dir="ltr">&lt;<a href="mailto:merwok@netwok.org">merwok@netwok.org</a>&gt;</span><br>Date: Wed, Oct 27, 2010 at 6:26 PM<br>
To: python-dev &lt;<a href="mailto:python-dev@python.org">python-dev@python.org</a>&gt;<br>Cc: Boštjan Mejak &lt;<a href="mailto:bostjan.mejak@gmail.com">bostjan.mejak@gmail.com</a>&gt;<br></font><br></span><br>[Reply received off-list quoted here]<br>

<br>
Le 18/10/2010 09:25, Boštjan Mejak a écrit :<br>
Readability has to be balanced with other important things:<br>
maintainability, discoverability, compatibility, rapidity and other<br>
things ending in -ity.  I’m personally glad that readability gets a huge<br>
place in the language definition itself (no braces, standard<br>
indentation), I follow 98 % of PEP 8 where I can, and have learned to<br>
accept that sometimes, a piece of code in the standard lib won’t get<br>
more readable.  Still beats some languages that I won’t name because<br>
dissing Java is too easy.  &lt;wink&gt;<br>
You are free to break compat in your code but it python-dev has a duty<br>
to its users.  Renames in threading have been deemed a good thing,<br>
renames in unittest not.  You can read the python-3000 archives to get<br>
an idea of the years of work and thousands of messages that went into<br>
py3k.  At some point, a release had to be done.  Now that there is a<br>
stable release in the 3.x line, compatibility rules apply.<br>
<br>
Practicality beats purity; now is better than never.<br>
<br>
Regards<br>
<br>
<br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Boštjan Mejak</b> <span dir="ltr">&lt;<a href="mailto:bostjan.mejak@gmail.com">bostjan.mejak@gmail.com</a>&gt;</span><br>Date: Wed, Oct 27, 2010 at 7:13 PM<br>
To: Éric Araujo &lt;<a href="mailto:merwok@netwok.org">merwok@netwok.org</a>&gt;<br></font><br></span><br>Since Python 3.2 accepts feature requests, take this fix as a feature request. Please forget about preserving the compatibility with old pickles.<div>
<div></div></div><br>----------<br><span class="undefined"><font color="#888">From: <b class="undefined">Éric Araujo</b> <span dir="ltr">&lt;<a href="mailto:merwok@netwok.org">merwok@netwok.org</a>&gt;</span><br>Date: Wed, Oct 27, 2010 at 7:16 PM<br>
To: Boštjan Mejak &lt;<a href="mailto:bostjan.mejak@gmail.com">bostjan.mejak@gmail.com</a>&gt;<br></font><br></span><br>Again, please send a copy to your messages to the mailing list (choose<br>
“reply to all” or “reply to list”), and don’t leave the whole message<br>
after your reply.  Thanks.<br>
<br>
<br></div><br>