<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 3, 2017 at 2:32 AM, Pierre Tardy <span dir="ltr"><<a href="mailto:tardyp@gmail.com" target="_blank">tardyp@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi Craig,<div>So it is not realistic to ask people to try python3 as well unless we have a clear PASS/FAIL status of whether there is a regression or not.<br></div><div><br></div></div></blockquote><div><br></div><div>I previously asked you specifically on IRC if you could change your workflow so that:</div><div><br></div><div>(1) create two virtualenvs, one for Python 2.7, and one for Python 3.x</div><div>(2) as you are working on a patch, you work in a single workspace, but you test your patch</div><div> in the two separate virtualenvs, maybe in separate windows</div><div><br></div><div>This is additional work, and probably a change for how you are used to working,</div><div>but it helps reduce the potential for Python 3 regressions.</div><div>This is the way I have been working, so as to fix things on Python 3, but not to introduce Python 2 regressions.</div><div><br></div><div>It is important for you to be on board for working in this way,</div><div>because you are doing the majority of code reviews, branch merges, and new code development for buildbot.</div><div><br></div><div>It's obvious that you are not changing your workflow as I previously asked.</div><div>That's your choice, and I cannot force you to do what you don't want to do, but that is still annoying for me to see.</div><div><br></div><div>In addition to breaking </div><div>For example, in addition to breaking test_util_httpclientservice.py (which I fixed on Python 3 on Jan. 28,</div><div>and then submitted a patch to fix what you broke)</div><div>you have introduced new code via <a href="https://github.com/buildbot/buildbot/pull/2722">https://github.com/buildbot/buildbot/pull/2722</a> in the past 24 hours</div><div>which doesn't work on Python 3, so this is introducing new failures.</div><div><br></div><div>Am I asking too much of you to work with two virtualenvs and not</div><div>break things on Python 3, and introduce new code which doesn't work on Python 3?</div><div><br></div>
<div>--</div><div>Craig</div></div></div></div>