Opened 13 years ago

Closed 13 years ago

#18029 closed Cleanup/optimization (fixed)

Remove mod_python

Reported by: Aymeric Augustin Owned by: nobody
Component: Core (Other) Version: 1.4
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

This ticket is just a placeholder to attach and review patches.

Change History (8)

by Claude Paroz, 13 years ago

Attachment: 18029-1.diff​ added

modpython removal

comment:1 by Claude Paroz, 13 years ago

Has patch: set

comment:2 by Aymeric Augustin, 13 years ago

Triage Stage: Accepted → Ready for checkin

Looks good to me.

comment:3 by Aymeric Augustin, 13 years ago

Patch needs improvement: set
Triage Stage: Ready for checkin → Accepted

Actually #13820 suggested porting the hotshot profiler rather than removing it outright. I had never seen it or heard about it before, so I don't feel qualified to decide its fate.

comment:4 by Aymeric Augustin, 13 years ago

Patch needs improvement: unset
Triage Stage: Accepted → Ready for checkin

After reading the ​discussion leading to the deprecation of mod_python, it appears that even two years ago, nobody still used the hotshot profiler, or was willing to save it.

comment:5 by Claude Paroz, 13 years ago

Resolution: → fixed
Status: new → closed

In [17835]:

Fixed #18029 -- Removed mod_python as of deprecation process. Thanks Aymeric Augustin for the review.

comment:6 by Aymeric Augustin, 13 years ago

Resolution: fixed
Status: closed → reopened

You apparently forgot to remove django/contrib/auth/handlers/modpython.py, /django/core/handlers/modpython.py and docs/howto/deployment/modpython.txt.

comment:7 by Claude Paroz, 13 years ago

Resolution: → fixed
Status: reopened → closed

In [17849]:

Fixed #18029 -- Removed leftover mod_python files forgotten in r17835.

Note: See TracTickets for help on using tickets.
Back to Top