-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 --ignore
flag to pip check
#11157
Comments
Thanks @wolfv. +1 to this idea. To save maintainers some following links, what came up in diff --git a/PKG-INFO b/PKG-INFO
index 40355c0..e0f6a39 100644
--- a/PKG-INFO
+++ b/PKG-INFO
@@ -14,7 +14,6 @@ Project-URL: Changelog, https://mesonpy.readthedocs.io/en/latest/changelog.html
Requires-Python: >=3.7
Requires-Dist: colorama; os_name == "nt"
Requires-Dist: meson>=0.60.0
-Requires-Dist: ninja
Requires-Dist: pep621>=0.3.0
Requires-Dist: tomli>=1.0.0
Requires-Dist: typing-extensions>=3.7.4; python_version < "3.8"
diff --git a/pyproject.toml b/pyproject.toml
index d61a210..b66565a 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -3,7 +3,6 @@ build-backend = 'mesonpy'
backend-path = ['.']
requires = [
'meson>=0.62.0',
- 'ninja',
'pep621>=0.3.0',
'tomli>=1.0.0',
'typing-extensions>=3.7.4; python_version<"3.8"',
@@ -28,7 +27,6 @@ classifiers = [
dependencies = [
'colorama; os_name == "nt"',
'meson>=0.60.0',
- 'ninja',
'pep621>=0.3.0', # not a hard dependency, only needed for projects that use PEP 621 metadata
'tomli>=1.0.0',
'typing-extensions>=3.7.4; python_version<"3.8"', It is not great that redistributors have to carry patches like that, and the ability to ignore checking for packages on the command line like @wolfv proposes would be a lot nicer.
|
This seems reasonable! A PR implementing this will be welcome. :) |
|
I don't know how pip does this usually but maybe a comma seperated list of packages? |
if you want env |
Bumping this 👍🏼 👁️ |
No need to bump, a pull requests is always welcomed. |
There is one in place, looking forward to it getting merged soon #11159 :) |
This issue can also occur if your application requires a module that is distributed under different package names. The Jupyter AI Conda recipe is experiencing this issue. We require the
Is there any workaround for this while we await #11159? For example, is there a way to modify |
What's the problem this feature will solve?
We currently use
pip check
quite a bit inconda-forge
to make sure that the dependency specifications that the dependencies that we have are correct.However, some packages
pip
sometimes specifies as dependencies can also be proper "system"-packages which do not register as pip/python packages. That makes it hard to runpip check
because of these "false negatives". Some examples arecmake
,ninja
, and likely some others.Describe the solution you'd like
If we could add a
--ignore
flag and a corresponding environment variable that could help us quite a bit.Alternative Solutions
Another solution would be to make
pip check
check if e.g. theninja
binary is on the$PATH
-- but that's brittle. Another solution would be to register system packages as pip installed packages but that's also not a proper solution.Additional context
https://github.com/FFY00/meson-python/issues/60
Code of Conduct
The text was updated successfully, but these errors were encountered: