From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-74708-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 15435 invoked by alias); 30 Jul 2010 23:23:30 -0000
Received: (qmail 15426 invoked by uid 22791); 30 Jul 2010 23:23:29 -0000
X-SWARE-Spam-Status: No, hits=-2.0 required=5.0	tests=AWL,BAYES_00,T_RP_MATCHES_RCVD
X-Spam-Check-By: sourceware.org
Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (38.113.113.100)    by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 30 Jul 2010 23:23:26 +0000
Received: (qmail 8094 invoked from network); 30 Jul 2010 23:23:24 -0000
Received: from unknown (HELO orlando.localnet) (pedro@127.0.0.2)  by mail.codesourcery.com with ESMTPA; 30 Jul 2010 23:23:24 -0000
From: Pedro Alves <pedro@codesourcery.com>
To: Jan Kratochvil <jan.kratochvil@redhat.com>
Subject: Re: [patch 2/6] Merge target_signal definitions to one place
Date: Fri, 30 Jul 2010 23:23:00 -0000
User-Agent: KMail/1.13.2 (Linux/2.6.32-24-generic; KDE/4.4.2; x86_64; ; )
Cc: gdb-patches@sourceware.org
References: <20100726225113.GC3597@host1.dyn.jankratochvil.net> <201007291221.54035.pedro@codesourcery.com> <20100730230154.GA11464@host1.dyn.jankratochvil.net>
In-Reply-To: <20100730230154.GA11464@host1.dyn.jankratochvil.net>
MIME-Version: 1.0
Content-Type: Text/Plain;  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Message-Id: <201007310023.22210.pedro@codesourcery.com>
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: 2010-07/txt/msg00599.txt.bz2

On Saturday 31 July 2010 00:01:54, Jan Kratochvil wrote:
> On Thu, 29 Jul 2010 13:21:53 +0200, Pedro Alves wrote:
> 
> > Should we update gdbserver's Makefile dependencies to make
> > signals.c depend on signals.inc?
> 
> OK although it already had no depenency on include/gdb/signals.h so it is in
> fact unrelated to this patch splitting include/gdb/signals.h to .h and .def.
> Fixed in this patch.

Thanks.  In that case, you should also update the server.o rule, which
depends on server.c, which now depends on signals.def.

> I find it approved just except the gdbserver dependencies.

Yeah.  It's all okay.

-- 
Pedro Alves