Opened 5 years ago

Closed 5 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.

Attachments (1)

18029-1.diff (46.8 KB) - added by Claude Paroz 5 years ago.
modpython removal

Download all attachments as: .zip

Change History (8)

Changed 5 years ago by Claude Paroz

Attachment: 18029-1.diff added

modpython removal

comment:1 Changed 5 years ago by Claude Paroz

Has patch: set

comment:2 Changed 5 years ago by Aymeric Augustin

Triage Stage: AcceptedReady for checkin

Looks good to me.

comment:3 Changed 5 years ago by Aymeric Augustin

Patch needs improvement: set
Triage Stage: Ready for checkinAccepted

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 Changed 5 years ago by Aymeric Augustin

Patch needs improvement: unset
Triage Stage: AcceptedReady 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 Changed 5 years ago by Claude Paroz

Resolution: fixed
Status: newclosed

In [17835]:

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

comment:6 Changed 5 years ago by Aymeric Augustin

Resolution: fixed
Status: closedreopened

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

comment:7 Changed 5 years ago by Claude Paroz

Resolution: fixed
Status: reopenedclosed

In [17849]:

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

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