Author: jdevlieghere Date: Thu Oct 10 10:27:06 2019 New Revision: 374385 URL: http://llvm.org/viewvc/llvm-project?rev=374385&view=rev Log: [test] Pass DSYMUTIL and SDKROOT as part of the Make invocation.
Pass the DSYMUTIL and SDKROOT variables on the command line instead of the environment. This makes it easier to reproduce the make invocation during development. Differential revision: https://reviews.llvm.org/D68812 Modified: lldb/trunk/packages/Python/lldbsuite/test/plugins/builder_base.py Modified: lldb/trunk/packages/Python/lldbsuite/test/plugins/builder_base.py URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/packages/Python/lldbsuite/test/plugins/builder_base.py?rev=374385&r1=374384&r2=374385&view=diff ============================================================================== --- lldb/trunk/packages/Python/lldbsuite/test/plugins/builder_base.py (original) +++ lldb/trunk/packages/Python/lldbsuite/test/plugins/builder_base.py Thu Oct 10 10:27:06 2019 @@ -104,6 +104,23 @@ def getCCSpec(compiler): else: return "" +def getDsymutilSpec(): + """ + Helper function to return the key-value string to specify the dsymutil + used for the make system. + """ + if "DSYMUTIL" in os.environ: + return "DSYMUTIL={}".format(os.environ["DSYMUTIL"]) + return ""; + +def getSDKRootSpec(): + """ + Helper function to return the key-value string to specify the SDK root + used for the make system. + """ + if "SDKROOT" in os.environ: + return "SDKROOT={}".format(os.environ["SDKROOT"]) + return ""; def getCmdLine(d): """ @@ -145,8 +162,13 @@ def buildDefault( testname=None): """Build the binaries the default way.""" commands = [] - commands.append(getMake(testdir, testname) + ["all", getArchSpec(architecture), - getCCSpec(compiler), getCmdLine(dictionary)]) + commands.append(getMake(testdir, testname) + + ["all", + getArchSpec(architecture), + getCCSpec(compiler), + getDsymutilSpec(), + getSDKRootSpec(), + getCmdLine(dictionary)]) runBuildCommands(commands, sender=sender) @@ -164,8 +186,12 @@ def buildDwarf( """Build the binaries with dwarf debug info.""" commands = [] commands.append(getMake(testdir, testname) + - ["MAKE_DSYM=NO", getArchSpec(architecture), - getCCSpec(compiler), getCmdLine(dictionary)]) + ["MAKE_DSYM=NO", + getArchSpec(architecture), + getCCSpec(compiler), + getDsymutilSpec(), + getSDKRootSpec(), + getCmdLine(dictionary)]) runBuildCommands(commands, sender=sender) # True signifies that we can handle building dwarf. @@ -182,9 +208,12 @@ def buildDwo( """Build the binaries with dwarf debug info.""" commands = [] commands.append(getMake(testdir, testname) + - ["MAKE_DSYM=NO", "MAKE_DWO=YES", + ["MAKE_DSYM=NO", + "MAKE_DWO=YES", getArchSpec(architecture), getCCSpec(compiler), + getDsymutilSpec(), + getSDKRootSpec(), getCmdLine(dictionary)]) runBuildCommands(commands, sender=sender) @@ -206,6 +235,8 @@ def buildGModules( "MAKE_GMODULES=YES", getArchSpec(architecture), getCCSpec(compiler), + getDsymutilSpec(), + getSDKRootSpec(), getCmdLine(dictionary)]) lldbtest.system(commands, sender=sender) _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
