Skip to content

Commit

Permalink
MNT: Remove 'pip install' options in recipes
Browse files Browse the repository at this point in the history
* Remove --no-deps --no-build-isolation 'pip install' options in recipes as
  the build tool (e.g. conda-build, rattler-build) will enforce all required
  'pip install' options itself at build time.
  • Loading branch information
matthewfeickert committed Nov 18, 2024
1 parent 5b29307 commit d222e89
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions grayskull/strategy/pypi.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,9 +63,7 @@ class PypiStrategy(AbstractStrategy):
def fetch_data(recipe, config, sections=None):
update_recipe(recipe, config, sections or ALL_SECTIONS)
if not (recipe["build"] and recipe["build"]["script"]):
recipe["build"]["script"] = (
"<{ PYTHON }} -m pip install . -vv --no-deps --no-build-isolation"
)
recipe["build"]["script"] = "<{ PYTHON }} -m pip install . -vv"


def merge_pypi_sdist_metadata(
Expand Down
2 changes: 1 addition & 1 deletion tests/data/poetry/langchain-expected.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ build:
entry_points:
- langchain-server = langchain.server:main
noarch: python
script: {{ PYTHON }} -m pip install . -vv --no-deps --no-build-isolation
script: {{ PYTHON }} -m pip install . -vv
number: 0

requirements:
Expand Down

0 comments on commit d222e89

Please sign in to comment.