Skip to content

Commit 07dbf0a

Browse files
Apply suggestions from code review
Co-authored-by: Eric Snow <[email protected]>
1 parent cf61177 commit 07dbf0a

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

pyperformance/_benchmark.py

+5-2
Original file line numberDiff line numberDiff line change
@@ -165,8 +165,11 @@ def extra_opts(self):
165165
return self._get_metadata_value('extra_opts', ())
166166

167167
@property
168-
def install_setup(self):
169-
return self._get_metadata_value('install_setup', False)
168+
def setup_py(self):
169+
if not self._get_metadata_value('install_setup', False):
170+
return None
171+
filename = os.path.join(os.path.dirname(self.metafile), 'setup.py'))
172+
return filename if os.path.exists(filename) else None
170173

171174
# Other metadata keys:
172175
# * base

pyperformance/venv.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,10 @@ def from_benchmarks(cls, benchmarks):
2424
for bench in benchmarks or ():
2525
filename = bench.requirements_lockfile
2626
self._add_from_file(filename)
27-
if bench.install_setup:
28-
self._add(os.path.dirname(bench.metafile))
27+
if bench.setup_py:
28+
# pip doesn't support installing a setup.py,
29+
# but it does support installing from the directory it is in.
30+
self._add(os.path.dirname(bench.setup_py))
2931
return self
3032

3133
def __init__(self):

0 commit comments

Comments
 (0)