Opened 2 years ago

Closed 18 months ago

#21845 closed Cleanup/optimization (wontfix)

Deprecate .py suffix for django-admin.py

Reported by: apollo13 Owned by: nobody
Component: Core (Management commands) Version: master
Severity: Normal Keywords:
Cc: berker.peksag@… Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

We need to find a nice way to deprecate django-admin.py. Warnings probably won't work, we might wanna resort to "print(file=sys.stderr)". Docs also need updates :/

Change History (8)

comment:1 Changed 2 years ago by timo

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 2 years ago by carljm

Is there some missing context to this ticket? Is it just the .py suffix we're deprecating, or the script itself? If the latter, how are we expecting people to run startproject?

comment:3 Changed 2 years ago by mjtamlyn

I believe it's the .py suffix. In order to make the script executable on windows, we're now using a setuptools entry point. This is however impossible with a .py suffix. (I think)

comment:4 Changed 2 years ago by claudep

  • Summary changed from Deprecate django-admin.py to Deprecate .py suffix for django-admin.py

comment:5 Changed 2 years ago by apollo13

Fwiw, we don't have to deprecate it and can just install both variants, if we think the deprecation isn't worth it -- eg to many programs relying on it.

comment:6 Changed 2 years ago by berkerpeksag

  • Cc berker.peksag@… added

comment:7 Changed 2 years ago by timo

  • Severity changed from Release blocker to Normal

Both variants are installed as of #21827. I tend to think deprecation may not be worth it as it doesn't seem like we gain very much besides perhaps reduced confusion from not having two ways to do the same thing). I'll leave the ticket open, but I'm unmarking it as a release blocker.

comment:8 Changed 18 months ago by timgraham

  • Resolution set to wontfix
  • Status changed from new to closed

Doc updates were made in #23067. I'll close this for now until someone comes up with a good reason why we need to deprecate the .py variant.

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