On 30 Aug 2015, at 18:36, Luke Diamand <l...@diamand.org> wrote: > On 30 August 2015 at 11:18, Lars Schneider <larsxschnei...@gmail.com> wrote: >> Thanks for your feedback! >> >> I like the “handle big files” plugin kind of idea. However, I wonder if it >> makes sense to put more and more stuff into git-p4.py (>3000 LOC already). >> What do you think about splitting git-p4 into multiple files? > > I was wondering about that. I think for now, the simplicity of keeping > everything in one file is worth the slight extra pain. I don't imagine > that the big-file-handler code would be very large. OK.
> >> >> Regarding Python 3: >> Would you drop Python 2 support or do you want to support Python 2/3 in >> parallel? I would prefer the former… > > For quite some time we would need to support both; we can't just have > a release of git that one day breaks git-p4 for people stuck on Python > 2. But it might not be that hard to support both (though converting > all those print statements could be quite tiresome). Agreed. However supporting both versions increases code complexity as well as testing effort. Would a compromise like the following work? We fork “git-p4.py” to “git-p4-python2.py” and just apply important bug fixes to that file. All new development happens on a Python 3 only git-p4.py. Cheers, Lars-- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
