Skip to content

Commit 71b3806

Browse files
committed
Fix LLDB windows build
LLDB WoA buildbot is failing due to pid_t redefinition after recent changes in lldb/source/Plugins/ScriptInterpreter/Python/ScriptedProcessPythonInterface.cpp. Process.h includes PosixApi.h which defines pid_t. Python.h on windows also typedefs pid_t. To make sure that we include Python.h before PosixApi this patch renforces the workaround previously set up to guard this issue. https://lab.llvm.org/buildbot/#/builders/219 Reviewed By: mib Differential Revision: https://reviews.llvm.org/D145446
1 parent c75dbed commit 71b3806

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

lldb/source/Plugins/ScriptInterpreter/Python/ScriptedProcessPythonInterface.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,16 +7,17 @@
77
//===----------------------------------------------------------------------===//
88

99
#include "lldb/Host/Config.h"
10+
#if LLDB_ENABLE_PYTHON
11+
// LLDB Python header must be included first
12+
#include "lldb-python.h"
13+
#endif
1014
#include "lldb/Target/Process.h"
1115
#include "lldb/Utility/Log.h"
1216
#include "lldb/Utility/Status.h"
1317
#include "lldb/lldb-enumerations.h"
1418

1519
#if LLDB_ENABLE_PYTHON
1620

17-
// LLDB Python header must be included first
18-
#include "lldb-python.h"
19-
2021
#include "SWIGPythonBridge.h"
2122
#include "ScriptInterpreterPythonImpl.h"
2223
#include "ScriptedProcessPythonInterface.h"

0 commit comments

Comments
 (0)