<br><br><div class="gmail_quote">On Tue Dec 02 2014 at 10:13:01 AM <<a href="mailto:dawn@burble.org">dawn@burble.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Dec 02, 2014 at 05:45:21PM +0000, Abid, Hafiz wrote:<br><br>
> >Side comment, But regarding conditionally compiling the file, i prefer doing that at the CMake level (only include it in the set of inputs if the platform is correct) instead of wrapping the whole file in ifdefs.<br>
> Agree. But if problem is handled at the build level then autoconf based system should be updated too. Anyway, this issue can be dealt separately.<br>
<br>
For those of us that have no clue on how to work with XCode projects,<br>
tweaking just the cmake and gnu makefiles can break those builds.  Lldb-mi<br>
wasn't being built in XCode last I checked, but I expect it will be some say.<br></blockquote><div><br></div><div>I think people are generally OK with this.  If it's going to end up requiring a major effort to fix up on another platform, it's a good idea to coordinate the efforts with them ahead of time.  But for something trivial like this where they will probably just have to make a minor tweak to their project settings, people are ok with doing that.</div></div>