Skip to content

Commit 049d886

Browse files
authored
Merge pull request #201231 from Homebrew/prowler-venv-dsl
prowler: use `virtualenv_install_with_resources`
2 parents 81edb8e + 5ddf2b5 commit 049d886

File tree

1 file changed

+2
-8
lines changed

1 file changed

+2
-8
lines changed

Formula/p/prowler.rb

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ class Prowler < Formula
2424
depends_on "cryptography"
2525
depends_on "libyaml"
2626
depends_on "numpy"
27-
depends_on "[email protected]"
27+
depends_on "[email protected]" # https://github.com/prowler-cloud/prowler/blob/master/pyproject.toml#L68
2828

2929
on_linux do
3030
depends_on "patchelf" => :build
@@ -731,15 +731,9 @@ class Prowler < Formula
731731
end
732732

733733
def install
734-
ENV["PIP_USE_PEP517"] = "1"
735-
736734
# Multiple resources require `setuptools`, so it must be installed first
737-
skipped = %w[plotly setuptools]
738-
venv = virtualenv_create(libexec, "python3.12")
739-
venv.pip_install resource("setuptools")
740-
venv.pip_install resources.reject { |r| skipped.include? r.name }
735+
venv = virtualenv_install_with_resources without: "plotly", start_with: "setuptools"
741736
venv.pip_install resource("plotly"), build_isolation: false
742-
venv.pip_install_and_link buildpath
743737
end
744738

745739
test do

0 commit comments

Comments
 (0)