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

Make devserver run on Python 3 #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Make devserver run on Python 3 #75

wants to merge 1 commit into from

Conversation

bkabrda
Copy link

@bkabrda bkabrda commented Mar 27, 2013

Hi, since Django can run on Python 3 now, I thought it would be nice if devserver could, too.
I didn't use six library for compatibility, because I think the modifications are easy enough to keep them as they are...

@scottwoodall
Copy link

+1 as I was also needing/wanting this today.

@lithammer
Copy link

Should probably just use self.stdout.write() and self.stderr.write() instead of print() as suggest in the docs.

@keimlink
Copy link

What the status of this issue? Does django-devserver support Python 3 now?

coagulant added a commit to coagulant/django-devserver that referenced this pull request Oct 26, 2014
@gone
Copy link

gone commented Mar 15, 2015

+1 for this one - I've been using coagulant's fork

@ALenfant
Copy link

+1, this project seems very interesting and I'd love to use it, but unfortunately I am on Python 3 !

@mikaelengstrom
Copy link

+1

@FlipperPA
Copy link

Our organization is running on Python 3 exclusively now as well. Shall we start a branch and collaborate if there's enough interest?

@mikaelengstrom
Copy link

@FlipperPA I´m interested and could help out.

@alexryabtsev alexryabtsev mentioned this pull request Aug 30, 2015
@rattrayalex
Copy link

Is there a fork anyone is using?

@egrubbs
Copy link
Collaborator

egrubbs commented May 4, 2016

I know someone that is using this branch for python 3.
https://github.com/alexryabtsev/django-devserver/commits/master

@Ismael-VC
Copy link

@egrubbs why is Python 3 support not wanted?

@ebertti
Copy link

ebertti commented Apr 25, 2017

@dcramer Are you there?

@acdha
Copy link
Contributor

acdha commented Apr 25, 2017

@ebertti Unless you're his boss I don't think you should ask questions in that tone. If you want to help develop a project, perhaps test the outstanding pull request(s) and report your results.

@FlipperPA
Copy link

I'm guessing the maintainer probably doesn't have much time to give to this project, as he's the founder and CEO of Sentry, which has exploded over the past few years (congrats to you, @dcramer!). If you look at his commit history, it is pretty clear that's where his attention is, and should be. I wonder if he'd be amenable to someone taking this project over as maintainer, especially since Django 2.0 (coming in ~December) will only support Python 3 going forward.

Also worth considering, most of the features of this package can be accomplished in Django or are covered nicely in Django Debug Toolbar, for example:

  • Setting up logging to file is possible.
  • As of Django 1.8 (IIRC the version correctly), runserver is multithreaded by default (as I found out when running a single-threaded DB driver, hehe). The --no-threading option was introduced.
  • Django Debug Toolbar has awesome SQL parsing, explain syntax, and more
  • Using IPython's embed() is amazing for interactive debugging with Django's runserver

Has this project become redundant? If not, can any of the features be pushed upstream to Django's runserver or Django Debug Toolbar?

@acdha
Copy link
Contributor

acdha commented Apr 26, 2017

@FlipperPA The main reason I still use devserver is for speed & AJAX: I often want is the query count/time info for API requests. I've been wondering whether it might be better to basically do a modern Django fork which would add only that information and forgo everything else since it's also much easier: hook the database, cache, etc. calls on startup and then turn control over to the stock WSGI app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.