[Lldb-commits] [PATCH] add tests for lldb-mi

[Colin Riley]

Thanks for all of this. It's great folk are using lldb-mi more!

Sorry for not replying to your original mail about the tests, Dawn. Have 
been OOO/afk. Abid brought up the same points I would have anyway :)

Cheers,
Colin

On 25/11/2014 10:44, Abid, Hafiz wrote:
>
> I have enabled the tests for all hosts apart from 
> 'test_lldbmi_interrupt' which was giving error to me on Linux. I will 
> see if I can figure out why it is failing there. The patch is committed.
>
> Regards,
>
> Abid
>
> *From:*Vince Harron [mailto:vharron at google.com]
> *Sent:* 25 November 2014 08:08
> *To:* Zachary Turner
> *Cc:* lldb-commits at cs.uiuc.edu; dawn at burble.org; Abid, Hafiz
> *Subject:* Re: [Lldb-commits] [PATCH] add tests for lldb-mi
>
> Yes, please enable tests if they _should_ work on other platforms.
>
> On Nov 24, 2014 11:27 PM, "Zachary Turner" <zturner at google.com 
> <mailto:zturner at google.com>> wrote:
>
> On Mon Nov 24 2014 at 11:03:58 PM <dawn at burble.org 
> <mailto:dawn at burble.org>> wrote:
>
> Thank you for your review!
>
> On Mon, Nov 24, 2014 at 11:18:40AM +0000, Abid, Hafiz wrote:
> > Hi Dawn,
> > I had a quick look at the tests and I think those will be useful 
> addition. I have
> > the following questions though.
> >
> > 1. I am wondering why have you restricted tests to darwin only?
>
> because I only tested on OSX and didn't want to break any builds.
>
>  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.
>
> 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.
>
> 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.
>
> In the mean time, you should not restrict tests unless you have a 
> specific reason, and if it breaks something you should either
>
> 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)
>
> b) run the test on other platforms yourself (wherever possible) and 
> fix the errors
>
> c) ask for help.
>
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu <mailto:lldb-commits at cs.uiuc.edu>
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
>
>
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

-- 
- Colin Riley
Games Technology Director

Codeplay Software Ltd
45 York Place, Edinburgh, EH1 3HP
Tel: 0131 466 0503
Fax: 0131 557 6600
Website: http://www.codeplay.com
Twitter: https://twitter.com/codeplaysoft

This email and any attachments may contain confidential and /or privileged information and is for use by the addressee only. If you are not the intended recipient, please notify Codeplay Software Ltd immediately and delete the message from your computer. You may not copy or forward it,or use or disclose its contents to any other person. Any views or other information in this message which do not relate to our business are not authorized by Codeplay software Ltd, nor does this message form part of any contract unless so stated.
As internet communications are capable of data corruption Codeplay Software Ltd does not accept any responsibility for any changes made to this message after it was sent. Please note that Codeplay Software Ltd does not accept any liability or responsibility for viruses and it is your responsibility to scan any attachments.
Company registered in England and Wales, number: 04567874
Registered office: 81 Linkfield Street, Redhill RH1 6BY



---
This email is free from viruses and malware because avast! Antivirus protection is active.
http://www.avast.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20141125/854a6255/attachment.html>