<p dir="ltr">Yes, please enable tests if they _should_ work on other platforms.<br>
</p>
<div class="gmail_quote">On Nov 24, 2014 11:27 PM, "Zachary Turner" <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote">On Mon Nov 24 2014 at 11:03:58 PM <<a href="mailto:dawn@burble.org" target="_blank">dawn@burble.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thank you for your review!<br>
<br>
On Mon, Nov 24, 2014 at 11:18:40AM +0000, Abid, Hafiz wrote:<br>
> Hi Dawn,<br>
> I had a quick look at the tests and I think those will be useful addition. I have<br>
> the following questions though.<br>
><br>
> 1. I am wondering why have you restricted tests to darwin only?<br>
<br>
because I only tested on OSX and didn't want to break any builds.<br></blockquote><div><br></div><div> This is an understandable mentality, but I think for the greater good we should have a policy of having all tests enabled for all platforms, unless it exercises a specific feature of a platform which is known to not exist or not work the same way on other platforms.</div><div><br></div><div>A debugger has the unfortunate property that it is OS-specific by nature, and generally someone working on a particular platform either doesn't have access to or doesn't know anything about other platforms.  As a result, it's easy to get into a situation where everyone only writes tests for their own platform, or writes a feature in a way that is specific to their platform. </div><div><br></div><div>But I think it's important to remain vigilant against this.  We have build bots for most platforms, the notable exception being Windows, which I'm actively working on fixing, although it's a lot of work for one person to do so it will take some more time.</div><div><br></div><div>In the mean time, you should not restrict tests unless you have a specific reason, and if it breaks something you should either </div><div><br></div><div>a) revert your change and attempt to fix the broken tests before resubmitting (sometimes you can do this by yourself, just by looking at the failure and guessing why this might be broken on other platforms)</div><div><br></div><div>b) run the test on other platforms yourself (wherever possible) and fix the errors</div><div><br></div><div>c) ask for help.</div><div><br></div><div><br></div></div>
<br>_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br></blockquote></div>