From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9753 invoked by alias); 7 Nov 2007 23:44:23 -0000 Received: (qmail 9744 invoked by uid 22791); 7 Nov 2007 23:44:22 -0000 X-Spam-Check-By: sourceware.org Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.31) with ESMTP; Wed, 07 Nov 2007 23:44:10 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id 8F2EA2AC399 for ; Wed, 7 Nov 2007 18:44:08 -0500 (EST) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id oRUSaD-HuDZM for ; Wed, 7 Nov 2007 18:44:08 -0500 (EST) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 551D22AC398 for ; Wed, 7 Nov 2007 18:44:08 -0500 (EST) Received: by joel.gnat.com (Postfix, from userid 1000) id E18C5E7ACA; Wed, 7 Nov 2007 18:44:05 -0500 (EST) Date: Wed, 07 Nov 2007 23:44:00 -0000 From: Joel Brobecker To: gdb-patches@sourceware.org Subject: Re: [rfc] Stop unlikely "run"'s earlier Message-ID: <20071107234405.GH3911@adacore.com> References: <20071107211833.GA28200@caradoc.them.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20071107211833.GA28200@caradoc.them.org> User-Agent: Mutt/1.4.2.2i 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: 2007-11/txt/msg00149.txt.bz2 > I don't think this behavior is useful. "run" should select a native > target when the current target is an executable or core file, but > not when it's connected to some other target. This patch changes > the behavior to: > > (gdb) run > The "remote" target can not run programs. Try "help target" or "continue". > > Any comments? Shall I commit this? I agree this is a more useful behavior. > 2007-11-07 Daniel Jacobowitz > > * infcmd.c (kill_if_already_running): Make static. Use > target_require_runnable. > * target.c (target_require_runnable): New. > * target.h (target_require_runnable): Declare. Looks good to me. -- Joel