Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

some more work on py 3.3 port #736

Merged
merged 2 commits into from May 22, 2013
Merged

some more work on py 3.3 port #736

merged 2 commits into from May 22, 2013

Conversation

ThomasWaldmann
Copy link
Contributor

still lots of bytes vs unicode stuff to do

also some strange test fails on 2.7, please have a look

removed init_jinja_globals hack from app.py after consulting mitsuhiko
(didn't work on py 3.3 "as is")

removed with_statement future imports, not needed any more

needs more work on 2.7 as well as on 3.3
@ghost
Copy link

ghost commented May 22, 2013

Why did you remove from __future__ import with_statement?

@ThomasWaldmann
Copy link
Contributor Author

because the port target is 2.6/2.7/3.3 and all these have the with statement (without using the future import).

@ghost
Copy link

ghost commented May 22, 2013

There's no need to break backward compatibility with 2.5. http://flask.pocoo.org/docs/installation/, third paragraph.

@untitaker
Copy link
Contributor

@Gioi It's already decided not to support anything older than 2.6, also Werkzeug won't support 2.5 anymore.

@ThomasWaldmann
Copy link
Contributor Author

And there is a need / a reason to break it, see mitsuhiko's nice article there:

http://lucumr.pocoo.org/2013/5/21/porting-to-python-3-redux/

@ghost
Copy link

ghost commented May 22, 2013

Thank you, I simply didn't know that. So, let's add a notice to the docs ;-)

@untitaker
Copy link
Contributor

40fad2e

DasIch added a commit that referenced this pull request May 22, 2013
@DasIch DasIch merged commit ee013e0 into pallets:sprint-branch May 22, 2013
@ghost
Copy link

ghost commented May 28, 2013

So we're going to drop 2.5 compatibility without any notice on the website and current version docs.
Not so nice.

@untitaker
Copy link
Contributor

@Gioi I did link a commit with such changes to the docs.

@DasIch
Copy link
Contributor

DasIch commented May 28, 2013

@Gioi We already did that with itsdangerous and jinja2. If this approach would cause problems people would already complain but so far no one has. On the other side people do quite regularly complain about lack of 3.x support and the amount of complaints on that front are continually increasing.

The community has made their opinion clear and I don't see a reason to appeal to a group that may or may not exist.

@ghost
Copy link

ghost commented May 28, 2013

@untitaker I have noticed it, have you noticed that http://flask.pocoo.org/docs/installation/ (aka current version documentation) is not updated?
@DasIch I'm not against dropping 2.5 compatibility, but I am complaining about the way it has been decided and communicated.
A bit of kindness wouldn't hurt.

@untitaker
Copy link
Contributor

Hmm, yeah, i actually have to agree with @Gioi, it could've been announced in the current docs. I doubt many users will be surprised by the drop of 2.5 though, since the majority is following Armin's Twitter account, and he did mention it there.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants