fix: only install files in package directory

madpah / requirements-parser

A Pip requirements file parser.

https://requirements-parser.readthedocs.io

Apache License 2.0

125 stars 41 forks source link

fix: only install files in package directory #82

Closed gavin09 closed 11 months ago

gavin09 commented 1 year ago

Fix https://github.com/madpah/requirements-parser/issues/66. With this merge request, the package will not install files outside the package directory.

More, this merge request fixes the lintian error “ unknown-file-in-python-module-directory” in Debian. I want to help upload this fantastic package to the newest version (0.0.5) to Debian with this fix.

jkowalleck commented 1 year ago

your fix casues that wheels will not have the files added, but the files MUST be included for legal reasons ...

a very undesirable solution, for legal purposes.

build backend has not proper functionality for it, yet. a fix depends on https://github.com/python-poetry/poetry/issues/8441

gavin09 commented 11 months ago

Thanks for your feedback. To my understanding, with your merge, wheel and sdist can both provide legal information perfectly. I tend to wait for your merge, so I would like to close this MR.