Fix SBDebugger.GetOutputFileHandle() on OS X.
authorJim Ingham <jingham@apple.com>
Fri, 11 Mar 2016 01:57:45 +0000 (01:57 +0000)
committerJim Ingham <jingham@apple.com>
Fri, 11 Mar 2016 01:57:45 +0000 (01:57 +0000)
The swig typemaps had some magic for output File *'s on OS X that made:

SBDebugger.GetOutputFileHandle()

actually work.  That was protected by a "#ifdef __MACOSX__", but the corresponding define
got lost going from the Darwin shell scripts to the python scripts for running
swig, so the code was elided.  I need to pass the define to SWIG, but only when
targetting Darwin.

So I added a target-platform argument to prepare_bindings, and if that
is Darwin, I pass -D__APPLE__ to swig, and that activates this code again, and
GetOutputFileHandle works again.  Note, I only pass that argument for the Xcode
build.  I'm sure it is possible to do that for cmake, but my cmake-foo is weak.

I should have been able to write a test for this by creating a debugger, setting the
output file handle to something file, writing to it, getting the output file handle
and reading it.  But SetOutputFileHandle doesn't seem to work from Python, so I'd
have to write a pexpect test to test this, which I'd rather not do.

llvm-svn: 263183

lldb/lldb.xcodeproj/project.pbxproj
lldb/scripts/Python/prepare_binding_Python.py
lldb/scripts/Python/python-typemaps.swig
lldb/scripts/prepare_bindings.py

index eae5d44..5b8bc66 100644 (file)
                        );
                        runOnlyForDeploymentPostprocessing = 0;
                        shellPath = /bin/bash;
-                       shellScript = "/usr/bin/python $SRCROOT/scripts/prepare_bindings.py --find-swig --framework --src-root $SRCROOT --target-dir $TARGET_BUILD_DIR --config-build-dir $CONFIGURATION_BUILD_DIR";
+                       shellScript = "/usr/bin/python $SRCROOT/scripts/prepare_bindings.py --find-swig --framework --src-root $SRCROOT --target-dir $TARGET_BUILD_DIR --config-build-dir $CONFIGURATION_BUILD_DIR --target-platform Darwin";
                };
                4959511A1A1ACE9500F6F8FC /* Install Clang compiler headers */ = {
                        isa = PBXShellScriptBuildPhase;
index 1996841..28fc3e5 100644 (file)
@@ -16,7 +16,7 @@ import re
 import shutil
 import subprocess
 import sys
-
+import platform
 
 class SwigSettings(object):
     """Provides a single object to represent swig files and settings."""
@@ -205,6 +205,8 @@ def do_swig_rebuild(options, dependency_file, config_build_dir, settings):
         "-I\"%s\"" % os.path.normcase("./."),
         "-D__STDC_LIMIT_MACROS",
         "-D__STDC_CONSTANT_MACROS"]
+    if options.target_platform == "Darwin":
+        command.append("-D__APPLE__")
     if options.generate_dependency_file:
         command.append("-MMD -MF \"%s\"" % temp_dep_file_path)
     command.extend([
index 82e33e1..1b2afc2 100644 (file)
 
 %typemap(out) FILE * {
    char mode[4] = {0};
-#ifdef __MACOSX__
+#ifdef __APPLE__
    int i = 0;
    short flags = $1->_flags;
 
index 3165f23..6d70f5b 100755 (executable)
@@ -152,6 +152,11 @@ def process_args(args):
             "Specifies the build dir where the language binding "
             "should be placed"))
 
+    parser.add_argument(
+        "--target-platform",
+        help=(
+            "Specifies the platform we are building for."
+            "Should be the same as what platform.system() returns."))
     # Process args.
     options = parser.parse_args(args)