Add Release versions

native_client_sdk/src/build_tools/build_sdk.py

Index: native_client_sdk/src/build_tools/build_sdk.py
diff --git a/native_client_sdk/src/build_tools/build_sdk.py b/native_client_sdk/src/build_tools/build_sdk.py
index 38a5e4b5c4fc4c5a56fa611c823f8b73db2c0e7c..fcc4fe5c196c27602d1ea8363fcb6f29dd98213c 100755
--- a/native_client_sdk/src/build_tools/build_sdk.py
+++ b/native_client_sdk/src/build_tools/build_sdk.py
@@ -738,7 +738,7 @@ def GetWindowsEnvironment():
 
 
 def BuildStepMakeAll(pepperdir, platform, directory, step_name,
-                     clean=False, deps=True):
+                     clean=False, deps=True, config='Debug'):

[Sam Clegg, 2013/01/31 20:09:27]

Maybe make this a required (non-default) arg if the only two
call sites are below?

[noelallen1, 2013/01/31 20:59:24]

Those are the only two call sites in this file.  Providing a default gives me
backwards compatibility for the other use cases.

On 2013/01/31 20:09:27, Sam Clegg wrote:

   buildbot_common.BuildStep(step_name)
   make_dir = os.path.join(pepperdir, directory)
   makefile = os.path.join(make_dir, 'Makefile')
@@ -752,7 +752,7 @@ def BuildStepMakeAll(pepperdir, platform, directory, step_name,
       env = os.environ
       make = 'make'
 
-    extra_args = []
+    extra_args = ['CONFIG='+config]
     if not deps:
       extra_args += ['IGNORE_DEPS=1']
 
@@ -765,8 +765,10 @@ def BuildStepMakeAll(pepperdir, platform, directory, step_name,
 
 
 def BuildStepBuildLibraries(pepperdir, platform, directory, clean=True):
-  BuildStepMakeAll(pepperdir, platform, directory, 'Build Libraries',
-      clean=clean)
+  BuildStepMakeAll(pepperdir, platform, directory, 'Build Libraries Debug',
+      clean=clean, config='Debug')
+  BuildStepMakeAll(pepperdir, platform, directory, 'Build Libraries Release',
+      clean=clean, config='Release')
 
 
 def BuildStepGenerateNotice(pepperdir):