Opened 17 years ago

Closed 17 years ago

Last modified 13 years ago

#7268 closed (duplicate)

Have a consistent handling of Django scripts for upstream distributors

Reported by: Raphaël Hertzog <hertzog@…> Owned by: nobody
Component: Tools Version: dev
Severity: Keywords:
Cc: gav@… Triage Stage: Design decision needed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Hello, I'm one of the debian maintainer of python-django and have some suggestions.

It would be nice if all the scripts in django/bin/ could:

  • be named consistently "django-<something>" so that they can be installed in /usr/bin/ without polluting the namespace (the manual pages in docs/man need to be renamed too)
  • all have the standard shebang line "#!/usr/bin/env python" (django/bin/profiling/gather_profile_stats.py lacks it)
  • be made executable by default (add the svn:executable property or make sure that setup.py add the executable bits)

Currently Debian already installs django-admin in /usr/bin together with the associated manual page. If the other scripts were renamed, we could so the same and have django-make-messages on the $PATH instead of hidden deep in the standard python hierarchy. BTW, I attach a patch for extras/django_bash_completion that also accept "djando-admin" as name for the django-admin.py script.

Thanks for considering those issues that matters mainly for distributors.

Attachments (1)

02_bash_completion.diff (1.4 KB ) - added by Raphael Hertzog <hertzog@…> 17 years ago.
Make bash completion file recognize "django-admin" command

Download all attachments as: .zip

Change History (10)

by Raphael Hertzog <hertzog@…>, 17 years ago

Attachment: 02_bash_completion.diff added

Make bash completion file recognize "django-admin" command

comment:1 by anonymous, 17 years ago

Cc: gav@… added

comment:2 by Simon Greenhill, 17 years ago

Has patch: set
Summary: Have a consistent handling of Django scriptsHave a consistent handling of Django scripts for upstream distributors
Triage Stage: UnreviewedReady for checkin

comment:3 by Adrian Holovaty, 17 years ago

(In [7658]) Added shebang line to bin/profiling/gather_profile_stats.py. Refs #7268

comment:4 by Russell Keith-Magee, 17 years ago

(In [7690]) Refs #7268 -- Added django-admin as an alias in the django-bash-completion script. Thanks, Raphael Hertzog <hertzog@…>

comment:5 by Russell Keith-Magee, 17 years ago

(In [7691]) Refs #7268 -- Added svn:executable flag for the Bash completion script.

comment:6 by Russell Keith-Magee, 17 years ago

Has patch: unset
milestone: 1.0 alpha
Triage Stage: Ready for checkinDesign decision needed

There is one outstanding part to this ticket - renaming of scripts and man pages to have a common prefix. This would be a backwards incompatible change, so we need to make a decision pre-v1. Moving to Design Decision Required with a version tag of 1.0-alpha to remind us to discuss this at a sprint.

comment:7 by Jannis Leidel, 17 years ago

FYI, I intend to move most of the scripts in django/bin to django.core.management.commands as discussed in #5522 (which has patches and docs).

comment:8 by Russell Keith-Magee, 17 years ago

Resolution: duplicate
Status: newclosed

Ok; Closing remaining parts of ticket in favour of #5522.

comment:9 by Jacob, 13 years ago

milestone: 1.0 alpha

Milestone 1.0 alpha deleted

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