From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5053 invoked by alias); 23 Aug 2014 03:13:31 -0000 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 Received: (qmail 5027 invoked by uid 89); 23 Aug 2014 03:13:26 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.5 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 X-HELO: bastet.se.axis.com Received: from bastet.se.axis.com (HELO bastet.se.axis.com) (195.60.68.11) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sat, 23 Aug 2014 03:13:24 +0000 Received: from localhost (localhost [127.0.0.1]) by bastet.se.axis.com (Postfix) with ESMTP id DA38D18099 for ; Sat, 23 Aug 2014 05:13:20 +0200 (CEST) Received: from bastet.se.axis.com ([IPv6:::ffff:127.0.0.1]) by localhost (bastet.se.axis.com [::ffff:127.0.0.1]) (amavisd-new, port 10024) with LMTP id GOSKfk4-vA3Z for ; Sat, 23 Aug 2014 05:13:16 +0200 (CEST) Received: from boulder.se.axis.com (boulder.se.axis.com [10.0.2.104]) by bastet.se.axis.com (Postfix) with ESMTP id 508BF18078 for ; Sat, 23 Aug 2014 05:13:16 +0200 (CEST) Received: from boulder.se.axis.com (localhost [127.0.0.1]) by postfix.imss71 (Postfix) with ESMTP id 39292F1B for ; Sat, 23 Aug 2014 05:13:16 +0200 (CEST) Received: from thoth.se.axis.com (thoth.se.axis.com [10.0.2.173]) by boulder.se.axis.com (Postfix) with ESMTP id 2E05354E for ; Sat, 23 Aug 2014 05:13:16 +0200 (CEST) Received: from ignucius.se.axis.com (ignucius.se.axis.com [10.88.21.50]) by thoth.se.axis.com (Postfix) with ESMTP id 1FA6834005; Sat, 23 Aug 2014 05:13:16 +0200 (CEST) Received: from ignucius.se.axis.com (localhost [127.0.0.1]) by ignucius.se.axis.com (8.12.8p1/8.12.8/Debian-2woody1) with ESMTP id s7N3DFQD018789; Sat, 23 Aug 2014 05:13:15 +0200 Received: (from hp@localhost) by ignucius.se.axis.com (8.12.8p1/8.12.8/Debian-2woody1) id s7N3DFps018785; Sat, 23 Aug 2014 05:13:15 +0200 Date: Sat, 23 Aug 2014 03:13:00 -0000 Message-Id: <201408230313.s7N3DFps018785@ignucius.se.axis.com> From: Hans-Peter Nilsson To: gdb-patches@sourceware.org In-reply-to: <201408200428.s7K4SNpt031643@ignucius.se.axis.com> (message from Hans-Peter Nilsson on Wed, 20 Aug 2014 06:28:23 +0200) Subject: Re: [RFC 0/4] Fix recent sim/*/config.in fallout: frv, m32r, sh64, iq2000 MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=iso-8859-1 Content-Transfer-Encoding: 8BIT X-SW-Source: 2014-08/txt/msg00512.txt.bz2 > From: Hans-Peter Nilsson > Date: Wed, 20 Aug 2014 06:28:23 +0200 > Recently, a change requiring -ldl due to bfd plugin default changing > from off to on, spread throughout the generated configury files, for > simulators as well as gdb. In that process, sim/*/config.in for the > sims below was regenerated for the first time since 2012. This had > build-breaking fallouts for some simulator showing bloopers in changes > from around 2013-03-23, attempts enabling sim-hardware and > dv-sockser.o. Here's my take on fixing that: let's return to the > state before those attempts. This is a request-for-comments, I'll > commit this in a few days unless someone has a better idea, one that > lets these simulators build *without* --{enable,disable} options and > where committing this would still not be an incremental improvement. Now committed. I'll stop here, noting that possible further cleanups would be to fix the buggy SIM_AC_OPTION_HARDWARE and decouple the prerequisites for dv-sockser.o from the act of actually enabling it (e.g. the now combined effect of HAVE_DV_SOCKSER). Please also, if you consider "re-enabling" simulated hardware for these sims, first investigate whether all the wires and plumbing is in place for it to actually make a difference in function for the particular sim. Also, test-cases for the enabled simulated hardware would be helpful, even a simple "hello, world" to a simulated serial port. brgds, H-P