Skip to content

Commit

Permalink
[build] Use pip to install setup.py dependency instead of `python set…
Browse files Browse the repository at this point in the history
…up.py install` (sonic-net#9111)

#### Why I did it
Backport sonic-net#8997 to 202012 branch.
  • Loading branch information
qiluo-msft authored and guxianghong committed May 21, 2022
1 parent 8a0ac85 commit 798910a
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 28 deletions.
2 changes: 2 additions & 0 deletions slave.mk
Original file line number Diff line number Diff line change
Expand Up @@ -606,6 +606,8 @@ $(addprefix $(PYTHON_WHEELS_PATH)/, $(SONIC_PYTHON_WHEELS)) : $(PYTHON_WHEELS_PA
pushd $($*_SRC_PATH) $(LOG_SIMPLE)
# apply series of patches if exist
if [ -f ../$(notdir $($*_SRC_PATH)).patch/series ]; then QUILT_PATCHES=../$(notdir $($*_SRC_PATH)).patch quilt push -a; fi
# Use pip instead of later setup.py to install dependencies into user home, but uninstall self
pip$($*_PYTHON_VERSION) install . && pip$($*_PYTHON_VERSION) uninstall --yes `python setup.py --name`
if [ ! "$($*_TEST)" = "n" ]; then python$($*_PYTHON_VERSION) setup.py test $(LOG); fi
python$($*_PYTHON_VERSION) setup.py bdist_wheel $(LOG)
# clean up
Expand Down
25 changes: 0 additions & 25 deletions src/redis-dump-load.patch/0003-use-redis-3.5.3.patch

This file was deleted.

1 change: 0 additions & 1 deletion src/redis-dump-load.patch/series
Original file line number Diff line number Diff line change
@@ -1,3 +1,2 @@
0001-Use-pipelines-when-dumping-52.patch
0002-Fix-setup.py-for-test-and-bdist_wheel.patch
0003-use-redis-3.5.3.patch
3 changes: 1 addition & 2 deletions src/sonic-yang-models/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,7 @@
'./yang-models/sonic-portchannel.yang',
'./yang-models/sonic-types.yang',
'./yang-models/sonic-versions.yang',
'./yang-models/sonic-vlan.yang',
'./yang-models/sonic_yang_tree']),
'./yang-models/sonic-vlan.yang']),
],
zip_safe=False,
)

0 comments on commit 798910a

Please sign in to comment.