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

pyproject include = ["LICENSE", "CONTRIBUTING.rst"...] #324

Open
cpina opened this issue Dec 24, 2023 · 1 comment
Open

pyproject include = ["LICENSE", "CONTRIBUTING.rst"...] #324

cpina opened this issue Dec 24, 2023 · 1 comment
Labels

Comments

@cpina
Copy link
Contributor

cpina commented Dec 24, 2023

This is a question.

In the pyproject.toml there is:

include = [
    "LICENSE",
    "CONTRIBUTING.rst",
    "CHANGELOG.rst",
    "scripts/ringcli.py"
]

I haven't done much Python packaging. But for other packages that I've done in Debian: this was never there, and because it's there I need to fix it in the Debian package (else the files go to an unexpected place).

If I install python-ring-doorbell with pip I see:

carles@pinux:~/p$ find . -iname "contributing.rst"
./venv/lib/python3.11/site-packages/CONTRIBUTING.rst
carles@pinux:~/p$ 

Which does not seem the correct place for CONTRIBUTING.rst (same for the other files).

I suggest to delete the include (I can do a quick PR if needed) to avoid installing those files in site-packages/ (this is using pip), besides making Debian package more standard. Of course, unless there is a good reason to have the include = there (which I might have missed, only seeing what's happening in other packages).

Also, I don't know the best way to include the files in the pip (I can have a look at other packages...) or if they should be in a subdirectory.

Thanks for considering it!

Copy link

There hasn't been any activity on this issue recently. This issue has been automatically marked as stale because of that. It will be closed if no further activity occurs.
Please make sure to update to the latest ring_doorbell version and check if that solves the issue.
Thank you for your contributions.

@github-actions github-actions bot added the stale Stale PRs and Issues label Apr 18, 2024
@sdb9696 sdb9696 added no-stale and removed stale Stale PRs and Issues labels Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants