You're making a valid point. I was only noting that this is not the first instance where've conditionally compiled mac platform files.
J > On Nov 4, 2015, at 6:00 PM, Jim Ingham <[email protected]> wrote: > > Why? > > Jim > >> On Nov 4, 2015, at 5:43 PM, Jason Molenda <[email protected]> wrote: >> >> We do the same thing in other MacOSX platform files where they depend on >> Apple specific libraries. >> >> >>> On Nov 4, 2015, at 5:39 PM, Jim Ingham via lldb-commits >>> <[email protected]> wrote: >>> >>> Why are we trying to build this at all on Linux? Seems odd to build an >>> empty file rather than just not build it. >>> >>> Jim >>> >>>> On Nov 4, 2015, at 5:23 PM, Chaoren Lin via lldb-commits >>>> <[email protected]> wrote: >>>> >>>> Author: chaoren >>>> Date: Wed Nov 4 19:23:19 2015 >>>> New Revision: 252124 >>>> >>>> URL: http://llvm.org/viewvc/llvm-project?rev=252124&view=rev >>>> Log: >>>> Fix build on Linux. >>>> >>>> Modified: >>>> lldb/trunk/source/Plugins/Platform/MacOSX/PlatformiOSSimulatorCoreSimulatorSupport.mm >>>> >>>> Modified: >>>> lldb/trunk/source/Plugins/Platform/MacOSX/PlatformiOSSimulatorCoreSimulatorSupport.mm >>>> URL: >>>> http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Platform/MacOSX/PlatformiOSSimulatorCoreSimulatorSupport.mm?rev=252124&r1=252123&r2=252124&view=diff >>>> ============================================================================== >>>> --- >>>> lldb/trunk/source/Plugins/Platform/MacOSX/PlatformiOSSimulatorCoreSimulatorSupport.mm >>>> (original) >>>> +++ >>>> lldb/trunk/source/Plugins/Platform/MacOSX/PlatformiOSSimulatorCoreSimulatorSupport.mm >>>> Wed Nov 4 19:23:19 2015 >>>> @@ -9,6 +9,8 @@ >>>> >>>> #include "PlatformiOSSimulatorCoreSimulatorSupport.h" >>>> >>>> +#if defined (__APPLE__) // This Plugin uses the Mac-specific >>>> source/Host/macosx/cfcpp utilities >>>> + >>>> // C Includes >>>> // C++ Includes >>>> // Other libraries and framework includes >>>> @@ -771,3 +773,5 @@ CoreSimulatorSupport::DeviceSet::GetFanc >>>> >>>> return dev; >>>> } >>>> + >>>> +#endif // __APPLE__ >>>> >>>> >>>> _______________________________________________ >>>> lldb-commits mailing list >>>> [email protected] >>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits >>> >>> _______________________________________________ >>> lldb-commits mailing list >>> [email protected] >>> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits >> > _______________________________________________ lldb-commits mailing list [email protected] http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
