From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6083 invoked by alias); 22 May 2013 19:39:25 -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 6071 invoked by uid 89); 22 May 2013 19:39:24 -0000 X-Spam-SWARE-Status: No, score=-8.2 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.1 Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Wed, 22 May 2013 19:39:24 +0000 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r4MJdKL9013232 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Wed, 22 May 2013 15:39:20 -0400 Received: from [127.0.0.1] (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx02.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id r4MJdIAQ012495; Wed, 22 May 2013 15:39:19 -0400 Message-ID: <519D1EE6.10404@redhat.com> Date: Wed, 22 May 2013 19:39:00 -0000 From: Pedro Alves User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130311 Thunderbird/17.0.4 MIME-Version: 1.0 To: Eli Zaretskii CC: mbilal@codesourcery.com, gdb-patches@sourceware.org, jan.kratochvil@redhat.com Subject: Re: [PATCH 5/7] PR gdb/15224 , Change the default set history filename to ~/.gdb_history. References: <51877A32.1030503@codesourcery.com> <51877A99.4060503@codesourcery.com> <51877B42.7@codesourcery.com> <51877BC6.6080007@codesourcery.com> <51877C76.2000509@codesourcery.com> <518A0BA8.9000703@codesourcery.com> <51936CAF.2090508@codesourcery.com> <51937A87.2030403@codesourcery.com> <83ehd85q79.fsf@gnu.org> <519CFF3D.4080001@redhat.com> <83bo82ucpv.fsf@gnu.org> <519D099F.7090101@redhat.com> <83a9nmu8l0.fsf@gnu.org> In-Reply-To: <83a9nmu8l0.fsf@gnu.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-SW-Source: 2013-05/txt/msg00857.txt.bz2 On 05/22/2013 08:18 PM, Eli Zaretskii wrote: >> Scripts do have a means get the previous behavior. Simply adding >> this to .gdbinit: > > But .gdbinit in the current directory is no longer read by default, so > I can't, not without restoring the old behavior, which does involve > using a command that will cause old GDB's to barf. > >> The reasoning for changing the default is that we (Pedro/Jan/Muhammad) >> believe enabling history by default is a better default that having it >> disabled by default, as currently. > > Another gratuitous incompatibility. Sorry, but simply calling changes gratuitous when I've made an effort to explain why I believe they're good doesn't help. :-/ The idea is that enabling the feature by default will expose it to more users, who will benefit from it, most (educated-guessing here, of course) not being aware GDB presently can already use history from previous sessions. > >> Couple the fact that ".gdb_history" >> is a dot/hidden file, with enabling history saving by default, and users >> could end up with their filesystem littered with random hidden .gdb_history >> files. I think GDB shouldn't do that by default. So in order to enable >> history saving by default, we believe we should default to ~/.gdb_history >> instead first. bash also defaults to saving history under $HOME, and I'd >> think most other interactive programs/shell do so too. So it feels like >> GDB is the odd one out here. > > Being the odd one out is not a reason good enough to change behavior > that was the default for a long time. The default has been to not save the history at all. When weighing the pros and cons, I believe the pros outweigh the cons. That's just my opinion, and I've just tried to clarify why I have it. -- Pedro Alves