Hello, I would like to know if I could apply the attached patch to lldb.
Following the commit r179584 [1], we need, under GNU/Linux, to include uuid.h to obtain the uuid_t definition. Otherwise, the build fails with: /tmp/buildd/llvm-toolchain-3.3~svn179620/tools/lldb/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp:714:42: error: 'uuid_t' was not declared in this scope For the full log, see [2]. Since this adds a new dependency on lldb, I was to make sure people agree on this change (I can update the tools + documentation about this change). Thanks, Sylvestre [1] http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp?r1=178191&r2=179584&pathrev=179584 [2] http://llvm-jenkins.debian.net/job/llvm-toolchain-codecoverage-binaries/architecture=amd64,distribution=unstable/20/console
Index: source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp =================================================================== --- source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp (révision 179680) +++ source/Plugins/DynamicLoader/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp (copie de travail) @@ -30,6 +30,8 @@ #include "DynamicLoaderMacOSXDYLD.h" +#include <uuid/uuid.h> + //#define ENABLE_DEBUG_PRINTF // COMMENT THIS LINE OUT PRIOR TO CHECKIN #ifdef ENABLE_DEBUG_PRINTF #include <stdio.h>
_______________________________________________ lldb-dev mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
