Excellent. Many thanks. I didn't try this solution. I through there were built on purpose for some weird cross compatibility reasons.
I revert my commit in the Debian packages then :) Sylvestre On 17/04/2013 19:45, Malea, Daniel wrote: > Hi Sylvestre, > > I can confirm the patch fixes the build, but I'm not sure we want to > introduce a new dependency on libuuid-dev (or equivalent), especially for > a library that's unlikely to be used on Linux. IIRC the only reason we > build it was because I was too lazy to add the darwin-specific logic when > writing the original cmake files. > > I have a fix in place that conditionally builds the mac-specific > (problematic) libraries only on Darwin (and also speeds up the build > somewhat :) It's been committed in 179694. > > Now, on to the hangs... > > Thanks again for all your help with this issue, > Dan > > On 2013-04-17 5:57 AM, "Sylvestre Ledru" <[email protected]> wrote: > >> 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/Dynamic >> Loader/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/DynamicLoade >> r/MacOSX-DYLD/DynamicLoaderMacOSXDYLD.cpp?r1=178191&r2=179584&pathrev=1795 >> 84 >> [2] >> http://llvm-jenkins.debian.net/job/llvm-toolchain-codecoverage-binaries/ar >> chitecture=amd64,distribution=unstable/20/console >> > _______________________________________________ lldb-dev mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
