From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32001 invoked by alias); 11 Sep 2012 21:53:48 -0000 Received: (qmail 31993 invoked by uid 22791); 11 Sep 2012 21:53:47 -0000 X-SWARE-Spam-Status: No, hits=-3.1 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,KHOP_THREADED,RCVD_IN_DNSWL_NONE,RCVD_IN_HOSTKARMA_YE,RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from elasmtp-kukur.atl.sa.earthlink.net (HELO elasmtp-kukur.atl.sa.earthlink.net) (209.86.89.65) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 11 Sep 2012 21:53:34 +0000 Received: from [68.96.200.16] (helo=macbook2.local) by elasmtp-kukur.atl.sa.earthlink.net with esmtpa (Exim 4.67) (envelope-from ) id 1TBYOk-0004TY-0M for gdb-patches@sourceware.org; Tue, 11 Sep 2012 17:53:34 -0400 Message-ID: <504FB2D3.9030108@earthlink.net> Date: Tue, 11 Sep 2012 21:53:00 -0000 From: Stan Shebs User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:15.0) Gecko/20120824 Thunderbird/15.0 MIME-Version: 1.0 To: gdb-patches@sourceware.org Subject: Re: [RFC] Merge mi-cli.exp and mi2-cli.exp References: <1346419770-5718-1-git-send-email-yao@codesourcery.com> <50469CDA.1030406@earthlink.net> <504F5592.6000102@redhat.com> In-Reply-To: <504F5592.6000102@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-ELNK-Trace: ae6f8838ff913eba0cc1426638a40ef67e972de0d01da9405dc9d8ab359eefaa3d250ffa134c2590350badd9bab72f9c350badd9bab72f9c350badd9bab72f9c X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-09/txt/msg00185.txt.bz2 On 9/11/12 8:15 AM, Pedro Alves wrote: > On 09/05/2012 01:29 AM, Stan Shebs wrote: >> On 9/3/12 2:08 AM, Vladimir Prus wrote: >>> On 31.08.2012 17:29, Yao Qi wrote: >>>> Unless I miss something, the intention of copying tests here is to test >>>> both '-i=mi' and '-i=mi2' respectively. However, this duplicates the code, >>>> and increase the effort to maintain, IMO. >>> Yep, that was the intent -- with the extra twist that tests for MI and MI2 are not necessary >>> identical. In other words, MI2 tests are tests for MI2 when MI2 was declared "done", and the >>> idea was that the output with "-i=mi2" would remain the same for years. -i=mi is our current >>> version of MI, which may evolve, and when MI3 is declared "done", the current tests will >>> be copied to mi3-* tests to keep backward compatibility in future. >>> >>> I am not quite sure how relevant this plan is these days. >> >> That plan has pretty much fallen by the wayside. We should probably declare the current MI behavior as the "done" form of MI3, and disallow any incompatible changes. If someone wants to get ambitious, they are free to specify and implement MI4. :-) > I agree the plan has fallen by the wayside, but because the introduction of MI3 > as a valid setting was premature and a mistake. MI2 is "done" in the sense that we > will never change MI2's grammar and input/output in a backward incompatible way. > We're free to add new output records, new commands, etc., but that isn't considered > backward incompatible. That's all still layered on the same protocol. > > [...] > But more than that, in reality, we stopped supporting MI1 almost 10 years ago: > > http://sourceware.org/ml/gdb-patches/2004-02/msg00352.html That's a good point, seemed like it was just yesterday. :-) A little poking around the net shows that it's been quite a while since anybody has used MI1. > So my opinion is that we revisit the policy a bit, and backtrack a the > mi-.*exp vs mi2-.*exp idea, get rid of the duplication, and call everything > "MI2", as it is in practice (must be, because that's how we run the tests). > When we really introduce an incompatible change that actually justifies MI3, > _then_ we should revisit the policy of whether to mass copying/rename tests, or > share them, depending on how big the difference between the versions would be, > and therefore depending on the practicality of the different options. > Sounds like a fine idea - and MI1 has already been deprecated for a long time, so we can just whack things now. I'd also like to scrub the comment references to what MI3 will do. While it makes sense for comments to mention ideas for future work, it's misleading to imply that we've committed our future selves to make any particular change on any particular schedule. Stan stan@codesourcery.com