<div dir="ltr">RE pep8 changes, I recommend autopep8 which can fix an entire codebase to pep8 style (at the expense of a potentially very noisy commit).<div><br></div><div><a href="https://pypi.python.org/pypi/autopep8/">https://pypi.python.org/pypi/autopep8/</a><br></div><div><br></div><div><i>Note: google's yapf does something similar (but it's "alpha" stage). <a href="https://github.com/google/yapf">https://github.com/google/yapf</a></i></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 18 April 2015 at 13:05, Bernhard Janetzki <span dir="ltr"><<a href="mailto:bernhard@janetzki.eu" target="_blank">bernhard@janetzki.eu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello Holger and list,<br>
<span class=""><br>
> yes, I'd happily take such patches. (In general, whether I will really like<br>
> them we will see ;-) But I do consider pep8 to be very sensible so I assume to<br>
> like them :)<br>
><br>
> Just please keep them reviewable, that is, split them into as many seperate<br>
> ones as you deem feasable!<br>
><br>
> Also please base your work on the develop branch and not on the master branch!<br>
<br>
</span>I started to port to python 3: <a href="https://github.com/ierror/piuparts/commits/develop" target="_blank">https://github.com/ierror/piuparts/commits/develop</a><br>
<br>
# I successfully tested the following with python 2 and 3:<br>
* tested a deb file with piuparts.py<br>
* passed the tests. As far as i can see unittests.py is no longer needed? Can we remove it?<br>
<br>
# Env changes<br>
I created a <a href="http://tox.in" target="_blank">tox.in</a> file and adapted the “make check” command.<br>
<br>
# O_CLOEXEC - “stuck"<br>
There are some open calls with “e” mode in piuparts-slave.py. ATM i stuck on this because python 3 disallows the “e” usage. You can find some information about it here: <a href="https://bugs.python.org/issue12103" target="_blank">https://bugs.python.org/issue12103</a>. Input is very appreciated. :-)<br>
<br>
# Whats missing? (as far as i know)<br>
* piuparts-slave.py (O_CLOEXEC…)<br>
* piuparts-master-backend.py (O_CLOEXEC…)<br>
* testing testing testing… ;)<br>
<br>
Thanks for review, and have a nice day,<br>
Boerni<br>
<br>_______________________________________________<br>
py3porters-devel mailing list<br>
<a href="mailto:py3porters-devel@lists.alioth.debian.org">py3porters-devel@lists.alioth.debian.org</a><br>
<a href="https://lists.alioth.debian.org/mailman/listinfo/py3porters-devel" target="_blank">https://lists.alioth.debian.org/mailman/listinfo/py3porters-devel</a><br>
<br></blockquote></div><br></div>