From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6602 invoked by alias); 15 May 2002 17:38:33 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 6578 invoked from network); 15 May 2002 17:38:29 -0000 Received: from unknown (HELO odin.inter.net.il) (192.114.186.10) by sources.redhat.com with SMTP; 15 May 2002 17:38:29 -0000 Received: from Zaretsky ([80.230.2.40]) by odin.inter.net.il (Mirapoint Messaging Server MOS 3.1.0.54-GA) with ESMTP id AEM41572; Wed, 15 May 2002 20:37:59 +0300 (IDT) Date: Wed, 15 May 2002 10:38:00 -0000 From: "Eli Zaretskii" To: ac131313@cygnus.com Message-Id: <3099-Wed15May2002203639+0300-eliz@is.elta.co.il> CC: gdb-patches@sources.redhat.com In-reply-to: <3CE20711.90407@cygnus.com> (message from Andrew Cagney on Wed, 15 May 2002 02:58:25 -0400) Subject: Re: [rfa/rfc] Revised multi-arch process Reply-to: Eli Zaretskii References: <3CE20711.90407@cygnus.com> X-SW-Source: 2002-05/txt/msg00612.txt.bz2 > Date: Wed, 15 May 2002 02:58:25 -0400 > From: Andrew Cagney > > (It also adds it do the doco, Eli?) Thanks for the heads up, I'm hard-pressed for free time lately, so might miss docs patches sometimes. Please ping if I do. I have a few minor comments: > +@section Converting an existing Target Architecture to Multi-arch Suggest an index entry here. Something like @cindex converting targets to multi-arch > +This section describes the current accepted best practice for converting > +an an existing target architecture to the multi-arch framework. The ``an'' is doubled here. > +@item > +Enable multi-arch level one > + > +@item > +Delete one or more files These two @item's need to end with a period, since the other two do. > +There isn't a size limit on a patch, however, more smaller patches are > +strongly encouraged. I think ``more'' is redundant. > +Temporally set @code{GDB_MULTI_ARCH} to GDB_MULTI_ARCH_PARTIAL and then GDB_MULTI_ARCH_PARTIAL should be in @code. Otherwise, okay; thanks.