Opened 6 months ago

Closed 6 months ago

Last modified 6 months ago

#28954 closed Cleanup/optimization (fixed)

Remove Jython code and documentation

Reported by: Andrey Martyanov Owned by: nobody
Component: Core (Other) Version: 2.0
Severity: Normal Keywords: jython obsolete
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

Change History (12)

comment:1 Changed 6 months ago by Andrey Martyanov

Type: UncategorizedBug

comment:2 Changed 6 months ago by Simon Charette

Triage Stage: UnreviewedAccepted
Type: BugCleanup/optimization

At this point we don't support Jython at all given it's Py2k only.

comment:3 Changed 6 months ago by Tim Graham

Rather than removing the document, I would add a note indicating that you must use Django 1.11 if you want to use Jython.

comment:4 Changed 6 months ago by Claude Paroz

I don't think it's necessary to keep those docs. If we refer to older version of Django to use Jython, users can then refer to that version of the docs to find that document.

comment:5 Changed 6 months ago by Claude Paroz

Has patch: set

comment:6 in reply to:  4 Changed 6 months ago by Andrey Martyanov

Replying to Claude Paroz:

I don't think it's necessary to keep those docs. If we refer to older version of Django to use Jython, users can then refer to that version of the docs to find that document.

Agreed.

comment:7 Changed 6 months ago by Tim Graham

There was previous discussion about this where I gave the opinion that there's little advantage to removing Jython compatibility and documentation because when Jython releases a Python 3 version, we'll have to revert that commit and issue a new release to restore compatibility. At least if we leave things the way they are, there's a chance current Django releases may work with Jython 3.

comment:8 Changed 6 months ago by Claude Paroz

There's no real traction currently for Jython 3, and that for several years now. Things can change, of course, but even if Jython 3 were to be released, this would probably need a new round of testing and adaptations.

comment:9 Changed 6 months ago by Tim Graham

Component: DocumentationCore (Other)
Summary: Django Jython howto is obsoleteRemove Jython code and documentation
Triage Stage: AcceptedReady for checkin

It's not the choice I would make, but I don't have any interest in Jython support and won't veto your decision. I think it's unfortunate to break any external links pointing to the dev version of docs/howto/jython.txt by removing that page.

comment:10 Changed 6 months ago by Claude Paroz

I restored docs/howto/jython.txt with a simple header text. Could you review the wording?

comment:11 Changed 6 months ago by Claude Paroz

Resolution: fixed
Status: newclosed

In 23b21db3:

Fixed #28954 -- Removed Jython docs and specific code

Thanks Andrey Martyanov for the reporti, and Tim Graham for the review.

Last edited 6 months ago by Tim Graham (previous) (diff)

comment:12 Changed 6 months ago by Claude Paroz

In 830636d:

[2.0.x] Fixed #28594 -- Removed Jython docs and specific code

Thanks Andrey Martyanov for the report, and Tim Graham for the review.
Backport of 23b21db31bfcf2a065d485617412cca43a311a4c from master.

Last edited 6 months ago by Tim Graham (previous) (diff)
Note: See TracTickets for help on using tickets.
Back to Top