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

Move to pyspf package #20

Closed
wants to merge 5 commits into from

Conversation

sloev
Copy link

@sloev sloev commented Jan 14, 2020

Hi there

Your module is called spf exactly like this (very popular) package SanicPluginsFrameworks

I want to be able to use both packages so i made this PR for you to consider, tests are green.

The pr moves all the code into the ./pyspf/ folder and i have changed the README and test to use this as well.

I have also added the dependencies into the setup.py so its easier to install.
(i specified individual versions of pydns according to python 2 or 3)

I hope you will consider merging this so i can use both libraries at the same time in the future!

Thanks for a great project!
kind regards - Johannes Valbjørn aka Sloev
:-)

@kitterma
Copy link
Collaborator

This project has been using the spf module name since 2004. I think it would be ridiculous for us to rename because of a less than three year old project that obviously failed to evaluate the name space before using it. There are many applications that use pyspf as it is and this would cause a lot of breakage.

Are you willing to find and provide updates for every application that uses pyspf?

@sloev
Copy link
Author

sloev commented Jan 14, 2020

Well if you have been developing this package since 2004 why is it then in this semi-unfinished non-conventional state?

I think your tone in the reply is off the charts and you should reconsider using words like ridiculous etc.

I am not affiliated to the Sanic project, i am just a user that wants to use both packages at once which you fail to let happen.

I will instead fork this package and publish it to pypi under a different name so people (that obey to conventions like packages etc) can still use your package.

By the way, here is some numbers for usage (as you can see: you're not the most popular kid in class):

https://pypistats.org/packages/pyspf

Downloads last day: 512
Downloads last week: 2,399
Downloads last month: 8,653

https://pypistats.org/packages/sanic-plugins-framework

Downloads last day: 1,411
Downloads last week: 7,014
Downloads last month: 24,904

so long and kind regards
Johannes

@kitterma
Copy link
Collaborator

Did you ask the other package to rename? Typically in the case of a namespace conflict, it's the newer project that renames. The structure of the package reflects it's age.

I agree it should be moved to setuptools, provide package requirements, and support more modern packaging approaches. We've recently been discussing it in #19 . I'm not the primary maintainer for the project.

If you're going to fork and rename something, I don't see why it should be us.

@sloev
Copy link
Author

sloev commented Jan 14, 2020

i will ask the other guys and refer to this discussion 👍

hope you have a good one if i don't come back :-)

kind regards
johannes

edit: here is a link to the issue in the other project:
ashleysommer/sanic-plugin-toolkit#16

@sloev
Copy link
Author

sloev commented Jan 15, 2020

Please find a reply from the other project here:
ashleysommer/sanic-plugin-toolkit#16 (comment)
It sounds like they will rename in next major version.

Kind regards

@kitterma
Copy link
Collaborator

Thank you for working to resolve this and letting us know. FTR, I'm not the project owner, just one of the developers on the project. Based on that plan, I'm going to close this PR.

@kitterma kitterma closed this Jan 15, 2020
@sloev
Copy link
Author

sloev commented Mar 23, 2021

@kitterma hi, just wanted to share that sanic plugin framework has changed package name going forward so the issue is resolved :-)
ashleysommer/sanic-plugin-toolkit#16 (comment)

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

Successfully merging this pull request may close these issues.

2 participants