Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Please rename the python package: registration -> django_registration #32

Closed
absurd34 opened this issue Jan 13, 2016 · 2 comments
Closed

Comments

@absurd34
Copy link

Hi James, maintainers,

please consider doing this, keeping a compatibility package "registration" (including a deprecation warning) around as long as needed.

Having a django-specific python package called "registration" around is confusing, and takes away this name for other possible uses (namespace pollution).

Current practice for third party django packages seems to be the "django_PACKAGE/" scheme, so I am suggesting this (don't use "django/PACKAGE" as maybe suggested elsewhere, this would pollute django's project namespace).

Fwiw, this is a followup to

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=567739

Thank you for considering!

Stephan

@ubernostrum
Copy link
Owner

At this time I have no plans to rename the module, since as you point out I'd need to keep a backwards-compatible fallback which would itself just "pollute" the namespace.

If distros want to package it under a different Python module name that's their call.

@absurd34
Copy link
Author

Hi James,

thanks for the quick reply; however...

On Mi, 2016-01-13 at 10:53 -0800, James Bennett wrote:

At this time I have no plans to rename the module, since as you point out I'd need to keep a backwards-compatible fallback which would itself just "pollute" the namespace.

... I do actually strongly disagree here.

This would merely pave the way so you can eventually (maybe in some
version 3.0 or 4.0) get rid of the fallback and the "pollution"
entirely.

If distros want to package it under a different Python module name that's their call.

This would make thing worse by far ;). For one thing, it would make
django apps incompatible between distros.

I.e., upstream is king here, I will follow ;).

Maybe, for documentation purposes, could label this "wontfix" or
"wishlist" (if github supports such a thing)?

Thx!

Stephan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants