From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-92236-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 3236 invoked by alias); 18 Jun 2012 09:08:31 -0000
Received: (qmail 3219 invoked by uid 22791); 18 Jun 2012 09:08:29 -0000
X-SWARE-Spam-Status: No, hits=-4.7 required=5.0	tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,KHOP_RCVD_TRUST,KHOP_THREADED,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_YE,TW_YM
X-Spam-Check-By: sourceware.org
Received: from mail-wi0-f177.google.com (HELO mail-wi0-f177.google.com) (209.85.212.177)    by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 18 Jun 2012 09:08:16 +0000
Received: by wibhm14 with SMTP id hm14so1190354wib.12        for <gdb-patches@sourceware.org>; Mon, 18 Jun 2012 02:08:15 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.180.101.170 with SMTP id fh10mr25732562wib.0.1340010495018; Mon, 18 Jun 2012 02:08:15 -0700 (PDT)
Received: by 10.216.235.140 with HTTP; Mon, 18 Jun 2012 02:08:14 -0700 (PDT)
In-Reply-To: <CAK1A=4y3_XDNkz1gCmHRWqBcN5KxPfcr0JzT9vaJQRJu7JVsvg@mail.gmail.com>
References: <CAK1A=4xcbM8HO86G2KM+SARLuG2MOaHzghjyumFr_j_dhzFsDQ@mail.gmail.com>	<4FD0AF8C.2040908@codesourcery.com>	<CAK1A=4yHpAXYsBjFGJdEsOq_w6JSwCSjq3hGX=G-rAYcSbZ5OQ@mail.gmail.com>	<CACTLOFpgNkvxsYuc4Q60GR0KXSHSi09Siv3qrpPBSz=Datwacw@mail.gmail.com>	<CAK1A=4y3_XDNkz1gCmHRWqBcN5KxPfcr0JzT9vaJQRJu7JVsvg@mail.gmail.com>
Date: Mon, 18 Jun 2012 09:08:00 -0000
Message-ID: <CAK1A=4yfXQkBrd5O7jo1sVtACXiBWiUqCFO8VWgtWWjV5kGvJw@mail.gmail.com>
Subject: Re: [PATCH] Makefile.in includes linux-record.c to be common for all arch. (arm-reversible>phase-3)
From: oza Pawandeep <oza.pawandeep@gmail.com>
To: Matt Rice <ratmice@gmail.com>
Cc: Yao Qi <yao@codesourcery.com>, 	"gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Content-Type: text/plain; charset=ISO-8859-1
X-IsSubscribed: yes
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2012-06/txt/msg00569.txt.bz2

please ind the latest patch which includes compilation of linux-record for ARM.

diff -urN orig/configure.tgt new/configure.tgt
--- orig/configure.tgt	2012-06-18 12:36:47.274501400 +0530
+++ new/configure.tgt	2012-06-18 12:31:47.335501400 +0530
@@ -76,7 +76,7 @@
 arm*-*-linux*)
 	# Target: ARM based machine running GNU/Linux
 	gdb_target_obs="arm-tdep.o arm-linux-tdep.o glibc-tdep.o \
-			solib-svr4.o symfile-mem.o linux-tdep.o"
+			solib-svr4.o symfile-mem.o linux-tdep.o linux-record.o"
 	build_gdbserver=yes
 	;;
 arm*-*-netbsd* | arm*-*-knetbsd*-gnu)

ok to check in ?

Regards,
Oza.

On Fri, Jun 8, 2012 at 6:13 PM, oza Pawandeep <oza.pawandeep@gmail.com> wrote:
> ok. It is clear now.
> shall incorporate linux-record where Yao suggested.
>
> Regards,
> Oza.
>
> On Fri, Jun 8, 2012 at 11:39 AM, Matt Rice <ratmice@gmail.com> wrote:
>> On Thu, Jun 7, 2012 at 10:20 AM, oza Pawandeep <oza.pawandeep@gmail.com> wrote:
>>> ok. sure will add it there as it is not currently linking.
>>> but on the contrary 'record.o' is also under the same section
>>> COMMON_OBS, which is also not required where no record support.
>>> hence it is confusing.
>>
>> I think that because record.o implements gdb commands, we want
>> commands to be available universally
>> e.g. we want an error that support for a command is not available in a
>> build of gdb, rather than 'Undefined command.'
>>
>> where linux-record appears to be plumbing, thus not necessary.