Skip to content

Commit 01ef1ff

Browse files
authored
bugfix: change pipenv install to pipenv sync to avoid updating deps at build time (#157)
1 parent 8c9a299 commit 01ef1ff

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

bin/setup_venv.sh

+4-2
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,8 @@ pip3 install pipenv
183183
# Install certain utility packages like `nodeenv` and `wheel` that aid
184184
# in the installation of other build tools and dependencies
185185
# required by the other python packages.
186-
PIPENV_VERBOSITY=-1 PIPENV_PIPFILE="${script_dir}/../pulumi/python/Pipfile" pipenv install --dev
186+
# `pipenv sync` uses only the information in the `Pipfile.lock` ensuring repeatable builds
187+
PIPENV_VERBOSITY=-1 PIPENV_PIPFILE="${script_dir}/../pulumi/python/Pipfile" pipenv sync --dev
187188

188189
# Install node.js into virtual environment so that it can be used by Python
189190
# modules that make call outs to it.
@@ -194,7 +195,8 @@ else
194195
fi
195196

196197
# Install general package requirements
197-
PIPENV_VERBOSITY=-1 PIPENV_PIPFILE="${script_dir}/../pulumi/python/Pipfile" pipenv install
198+
# `pipenv sync` uses only the information in the `Pipfile.lock` ensuring repeatable builds
199+
PIPENV_VERBOSITY=-1 PIPENV_PIPFILE="${script_dir}/../pulumi/python/Pipfile" pipenv sync
198200

199201
# Install local common utilities module
200202
pip3 install "${script_dir}/../pulumi/python/utility/kic-pulumi-utils"

0 commit comments

Comments
 (0)