[python-committers] Re: A urlparse regression in minor version

2020-02-16 Thread Barry Warsaw
On Feb 16, 2020, at 10:20, Ned Deily wrote: > Rather than continuing this change in 3.9 introducing yet another, even more > unexpected behavior, I think we should first try to address what appears to > me to be the (a?) root cause issue: urlparse's API is not suited for parsing > both strictl

[python-committers] Re: A urlparse regression in minor version

2020-02-16 Thread Senthil Kumaran
Thank you all. This is reverted in 3.8.2 and 3.7.7 - with NEWS explaining the behavior clearly. For the points that Ned brought up for 3.9, I will continue in the bug report https://bugs.python.org/issue27657 This discussion can be considered complete and closed. Thanks for the support. On Sun, F

[python-committers] Re: A urlparse regression in minor version

2020-02-16 Thread Łukasz Langa
OK, let's revert this for 3.8.2. I will make 3.8.2rc2 with this to highlight the revert and get some testing in. -- Best regards, Łukasz Langa > On 16 Feb 2020, at 19:31, Senthil Kumaran wrote: > >  > I have created the PRs for the revert in 3.8.2 and 3.7.7 > > https://github.com/python/cp

[python-committers] Re: A urlparse regression in minor version

2020-02-16 Thread Senthil Kumaran
I have created the PRs for the revert in 3.8.2 and 3.7.7 https://github.com/python/cpython/pull/18525 (3.8) - Acceptance definitely depends on the RM ( Łukasz' s call) and I will support it. https://github.com/python/cpython/pull/18526 (3.7) - Let this be reverted only if 3.8 gets reverted. @Ned

[python-committers] Re: A urlparse regression in minor version

2020-02-16 Thread Ned Deily
On Feb 16, 2020, at 07:21, Antoine Pitrou wrote: > FWIW, I agree with Senthil here. A slight behaviour change in 3.9 is > fine, especially in an area where the "right" semantics are not > immediately obvious. What we want to avoid is breaking behaviour > changes in bugfix releases. I agree tota

[python-committers] Re: A urlparse regression in minor version

2020-02-16 Thread Antoine Pitrou
FWIW, I agree with Senthil here. A slight behaviour change in 3.9 is fine, especially in an area where the "right" semantics are not immediately obvious. What we want to avoid is breaking behaviour changes in bugfix releases. Regards Antoine. Le 16/02/2020 à 13:13, Senthil Kumaran a écrit :

[python-committers] Re: A urlparse regression in minor version

2020-02-16 Thread Senthil Kumaran
On Sun, Feb 16, 2020 at 2:20 AM Ned Deily wrote: > > > For 3.9.0, I recommend we reconsider this change (temporarily reverting > it) and consider whether an API change to accommodate the various use cases > would be better > For 3.9. - I am ready to defend the patch even at the cost of the break

[python-committers] Re: A urlparse regression in minor version

2020-02-16 Thread Ned Deily
On Feb 15, 2020, at 09:00, Senthil Kumaran wrote: > As we have to a decision here, my vote is to revert the patch in 3.8.2 and > 3.7.7 > I have gone back-and-forth with this thinking, and it seems revert might > address some definite complaints we have got. > The problem is contained to single v