Skip to content

Commit df39196

Browse files
author
Prasanna Santhanam
committed
marvin: build fails when no setuptools/distribute found
appropriately warn the user to install setuptools. Also - fixed the jenkins url for builds.a.o Signed-off-by: Prasanna Santhanam <[email protected]>
1 parent 6df3d83 commit df39196

1 file changed

Lines changed: 9 additions & 5 deletions

File tree

tools/marvin/setup.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,13 @@
1919
try:
2020
from setuptools import setup, find_packages
2121
except ImportError:
22-
from distribute_setup import use_setuptools
23-
use_setuptools()
24-
from setuptools import setup, find_packages
22+
try:
23+
from distribute_setup import use_setuptools
24+
use_setuptools()
25+
from setuptools import setup, find_packages
26+
except ImportError:
27+
raise RuntimeError("python setuptools is required to build Marvin")
28+
2529

2630
VERSION = '0.1.0'
2731

@@ -35,10 +39,10 @@ def read(fname):
3539
author="Edison Su",
3640
author_email="[email protected]",
3741
maintainer="Prasanna Santhanam",
38-
maintainer_email="[email protected]",
42+
maintainer_email="[email protected]",
3943
long_description="Marvin is the Apache CloudStack python client written around the unittest framework",
4044
platforms=("Any",),
41-
url="https://builds.apache.org/view/CloudStack/job/cloudstack-marvin/",
45+
url="https://builds.apache.org/job/cloudstack-marvin/",
4246
packages=["marvin", "marvin.cloudstackAPI", "marvin.integration",
4347
"marvin.integration.lib", "marvin.sandbox",
4448
"marvin.sandbox.advanced", "marvin.sandbox.basic"],

0 commit comments

Comments
 (0)