From de38cd758ec12cbf2addb872f5b3d634f0012e2d Mon Sep 17 00:00:00 2001
From: Qi Luo <qiluo-msft@users.noreply.github.com>
Date: Fri, 29 Oct 2021 21:14:41 -0700
Subject: [PATCH] [build] Use pip to install setup.py dependency instead of
 `python setup.py install` (#9111)

#### Why I did it
Backport https://github.com/Azure/sonic-buildimage/pull/8997 to 202012 branch.
---
 slave.mk                                      |  2 ++
 .../0003-use-redis-3.5.3.patch                | 25 -------------------
 src/redis-dump-load.patch/series              |  1 -
 src/sonic-yang-models/setup.py                |  3 +--
 4 files changed, 3 insertions(+), 28 deletions(-)
 delete mode 100644 src/redis-dump-load.patch/0003-use-redis-3.5.3.patch

diff --git a/slave.mk b/slave.mk
index bc6e8f9707..e8a1b494bd 100644
--- a/slave.mk
+++ b/slave.mk
@@ -620,6 +620,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
diff --git a/src/redis-dump-load.patch/0003-use-redis-3.5.3.patch b/src/redis-dump-load.patch/0003-use-redis-3.5.3.patch
deleted file mode 100644
index a58c2059e8..0000000000
--- a/src/redis-dump-load.patch/0003-use-redis-3.5.3.patch
+++ /dev/null
@@ -1,25 +0,0 @@
-Pin the redis package to version 3.5.3 (the last version that supports both
-Python 2 and 3).
-
-Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
-
-diff --git a/requirements.txt b/requirements.txt
-index 7800f0f..3fc0632 100644
---- a/requirements.txt
-+++ b/requirements.txt
-@@ -1 +1 @@
--redis
-+redis==3.5.3
-diff --git a/setup.py b/setup.py
-index 8ccf31f..6db9ec4 100644
---- a/setup.py
-+++ b/setup.py
-@@ -17,7 +17,7 @@ setup(name=package_name,
-     author_email='oleg@bsdpower.com',
-     url='http://github.com/p/redis-dump-load',
-     py_modules=['redisdl'],
--    install_requires=['redis'],
-+    install_requires=['redis==3.5.3'],
-     data_files=[
-         (doc_dir, data_files),
-     ],
diff --git a/src/redis-dump-load.patch/series b/src/redis-dump-load.patch/series
index 50beb71a67..c73c3b5caa 100644
--- a/src/redis-dump-load.patch/series
+++ b/src/redis-dump-load.patch/series
@@ -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
diff --git a/src/sonic-yang-models/setup.py b/src/sonic-yang-models/setup.py
index e651bb0d37..553b44bc95 100644
--- a/src/sonic-yang-models/setup.py
+++ b/src/sonic-yang-models/setup.py
@@ -54,8 +54,7 @@ setup(
                          './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,
 )