Skip to content

Commit 3d44a08

Browse files
committed
getrmpytools: install poetry before RLTest
1 parent ed8290a commit 3d44a08

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

bin/getrmpytools

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,8 @@ class RMPyToolsSetup(paella.Setup):
7272

7373
def macos(self):
7474
self.pip_install("psutil==5.8.0")
75+
# without this, RLTest installation may fail (pip install does not install dev dependencies)
76+
self.pip_install("poetry")
7577

7678
def alpine(self):
7779
self.install("linux-headers")
@@ -87,18 +89,20 @@ class RMPyToolsSetup(paella.Setup):
8789
self.rltest_version = 'master'
8890
self.ramp_version = 'master'
8991

92+
redis_py_cluster_installed = False
9093
if self.redispy_version is None:
9194
if self.modern:
9295
self.pip_install("--no-cache-dir redis~=4.2.2")
9396
else:
9497
self.pip_install("--no-cache-dir git+https://github.com/redisfab/[email protected]")
9598

9699
self.pip_install("--no-cache-dir git+https://github.com/redisfab/[email protected]")
100+
redis_py_cluster_installed = True
97101
else:
98102
self.pip_install("--no-cache-dir git+https://github.com/{ORG}/redis-py.git@{VER}".\
99103
format(ORG=self.redispy_org, VER=self.redispy_version))
100104

101-
if not self.master and not self.modern:
105+
if not self.master and not self.modern and not redis_py_cluster_installed:
102106
# redis-py-cluster should be installed from git due to redis-py dependency
103107
# self.pip_install("--no-cache-dir git+https://github.com/Grokzen/redis-py-cluster.git@master")
104108
# self.pip_install("--no-cache-dir redis-py-cluster")

0 commit comments

Comments
 (0)