-
-
Notifications
You must be signed in to change notification settings - Fork 747
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
Add pyproject.toml #6355
Add pyproject.toml #6355
Conversation
This blocks borgbackup#6264 because black configuration is stored in pyproject.toml. See <https://black.readthedocs.io/en/stable/usage_and_configuration/the_basics.html#configuration-format> I copied this from <https://setuptools.pypa.io/en/latest/userguide/quickstart.html#basic-use>
When using pyproject.toml the parent of setup.py is not on sys.path by default. See <pypa/setuptools#3134>
setup_crypto.crypto_ext_kwargs needs pkgconfig to find OpenSSL
Codecov Report
@@ Coverage Diff @@
## master #6355 +/- ##
==========================================
+ Coverage 83.11% 83.19% +0.07%
==========================================
Files 38 38
Lines 10426 10426
Branches 2042 2042
==========================================
+ Hits 8666 8674 +8
+ Misses 1254 1246 -8
Partials 506 506
Continue to review full report at Codecov.
|
I am working on this. |
btw, for now please no bigger / fundamental changes to setup.py / setup.cfg. |
from the codeql workflow:
Guess adding these lines before pip3 should fix it, can you try?:
|
A bug in pip or possibly setuptools broke editable installs into the per-user directory for packages that have pyproject.toml See pypa/setuptools#3019
Another workaround would be to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks!
This blocks #6264 because black configuration is stored in pyproject.toml.
See https://black.readthedocs.io/en/stable/usage_and_configuration/the_basics.html#configuration-format
I copied this from https://setuptools.pypa.io/en/latest/userguide/quickstart.html#basic-use