Opened 16 years ago

Closed 16 years ago

#6857 closed (wontfix)

django.dispatch.robustapply does not work on jython

Reported by: Leo Soto M. Owned by: nobody
Component: Core (Other) Version: dev
Severity: Keywords: jython pypy
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Currently, pydispatcher expect that every im_func object contains its corresponding func_code. That is not always the case, at least on Jython, raising AttributeError.

Attachments (3)

robustapply_jython_compatible.patch (1.8 KB ) - added by Leo Soto M. 16 years ago.
robustapply_pypy_compatible.patch (1.3 KB ) - added by anto.cuni@… 16 years ago.
test_robustapply.py (1.8 KB ) - added by Maciej Fijalkowski 16 years ago.
Test (py.test) for robustapply

Download all attachments as: .zip

Change History (8)

by Leo Soto M., 16 years ago

comment:1 by James Bennett, 16 years ago

Triage Stage: UnreviewedDesign decision needed

We need to evaluate whether we're going to keep robust_apply in the dispatcher refactoring; if not this will be moot, but if so we'll need to take it into account.

comment:2 by Keith Bussell, 16 years ago

This would be fixed by #6814. As part of the clean-up/speed-up, robustapply has been removed.

by anto.cuni@…, 16 years ago

comment:3 by anto.cuni@…, 16 years ago

I've attached a new patch, that makes robustapply working on cpython, jython and pypy at the same time.

by Maciej Fijalkowski, 16 years ago

Attachment: test_robustapply.py added

Test (py.test) for robustapply

comment:4 by Maciej Fijalkowski, 16 years ago

Keywords: pypy added

I added a test (py.test style, I can move it to unittest style if anyone wants) to prove that robustapply behaves exactly the same everywhere.

comment:5 by Gary Wilson, 16 years ago

Resolution: wontfix
Status: newclosed

#6814 is likely to make it into 1.0 and it gets rid of the slow robust apply stuff. Let's focus our efforts there.

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