This is an archive of the discontinued LLVM Phabricator instance.

[buildbot, windows] Pass the python path for the new windows buildbot correctly
ClosedPublic

Authored by stella.stamenova on Oct 11 2018, 12:04 PM.

Reviewers

asmith
gkistanova

Commits

Summary

Right now the path is passed to CMake as a string without quotes causing CMake to fail to find python

LGTM

This revision is now accepted and ready to land.Oct 15 2018, 3:39 PM

This revision was automatically updated to reflect the committed changes.

Path

Size

zorg/

trunk/

buildbot/

osuosl/

master/

config/

3 lines

Show First 20 Lines • Show All 987 Lines • ▼ Show 20 Lines	return [
extra_cmake_args=["-DLLVM_ENABLE_ASSERTIONS=True",		extra_cmake_args=["-DLLVM_ENABLE_ASSERTIONS=True",
"-DLLVM_USE_LINKER=gold"])},		"-DLLVM_USE_LINKER=gold"])},
{'name': "lldb-x64-windows-ninja",		{'name': "lldb-x64-windows-ninja",
'slavenames': ["win-py3-buildbot"],		'slavenames': ["win-py3-buildbot"],
'builddir': "lldb-x64-windows-ninja",		'builddir': "lldb-x64-windows-ninja",
'category' : 'lldb',		'category' : 'lldb',
'factory': LLDBBuilder.getLLDBWindowsCMakeBuildFactory(		'factory': LLDBBuilder.getLLDBWindowsCMakeBuildFactory(
clean=True,		clean=True,
python_source_dir="C:\\Program Files (x86)\\Microsoft Visual Studio\\Shared\\Python36_64",		python_source_dir=r'"C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python36_64"',
vs='%VS150COMNTOOLS%',
target_arch='x64',		target_arch='x64',
test=True,		test=True,
extra_cmake_args=['-DLLVM_ENABLE_ASSERTIONS=OFF'])}		extra_cmake_args=['-DLLVM_ENABLE_ASSERTIONS=OFF'])}
]		]

# LLD builders.		# LLD builders.
def _get_lld_builders():		def _get_lld_builders():
return [		return [
▲ Show 20 Lines • Show All 718 Lines • Show Last 20 Lines