From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 87790 invoked by alias); 22 Sep 2017 12:31:29 -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 86968 invoked by uid 89); 22 Sep 2017 12:31:28 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-6.9 required=5.0 tests=BAYES_00,GIT_PATCH_1,RP_MATCHES_RCVD,SPF_HELO_PASS autolearn=ham version=3.3.2 spammy= X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 22 Sep 2017 12:31:26 +0000 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 8E129883C2; Fri, 22 Sep 2017 12:31:25 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 8E129883C2 Authentication-Results: ext-mx02.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx02.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=palves@redhat.com Received: from [127.0.0.1] (ovpn04.gateway.prod.ext.ams2.redhat.com [10.39.146.4]) by smtp.corp.redhat.com (Postfix) with ESMTP id 1233F63623; Fri, 22 Sep 2017 12:31:21 +0000 (UTC) Subject: Re: [PATCH v3 4/5] Implement "set cwd" command on GDB To: Eli Zaretskii , Sergio Durigan Junior References: <20170912042325.14927-1-sergiodj@redhat.com> <20170921225926.23132-1-sergiodj@redhat.com> <20170921225926.23132-5-sergiodj@redhat.com> <83poajcg9a.fsf@gnu.org> Cc: gdb-patches@sourceware.org From: Pedro Alves Message-ID: Date: Fri, 22 Sep 2017 12:31:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <83poajcg9a.fsf@gnu.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-SW-Source: 2017-09/txt/msg00673.txt.bz2 On 09/22/2017 09:02 AM, Eli Zaretskii wrote: >> From: Sergio Durigan Junior > >> +@kindex set cwd >> +@cindex change inferior's working directory >> +@item set cwd @r{[}@var{directory}@r{]} >> +Set the inferior's working directory to @var{directory}. If not >> +given, @var{directory} uses @file{'~'}. > > I think we should document here what does "~" mean on MS-Windows, > especially since, when HOME is not in the environment, Gnulib's glob > module doesn't behave according to MS platform recommendations (which > say not to create files directly below %HOMEDRIVE%%HOMEPATH%). > > More generally, I think we should say here that the argument is > glob-expanded, because this is user-visible behavior (right?). Also, > how will TAB-completion react to input of this command? will it expand > the input typed so far? Actually, should the command default to ~ at all? Shouldn't we make "set cwd" clear the setting to the default state, i.e., empty? Otherwise, how do you get back to the default state? >> +@kindex show cwd >> +@cindex show inferior's working directory >> +@item show cwd >> +Show the inferior's working directory. If no directory has been >> +specified by @code{set cwd}, then the default inferior's working >> +directory is the same as @value{GDBN}'s working directory. > > Does this show the original value typed by the user, or the expanded > value? E.g., if the user types "set cwd ~/foo", what will "show cwd" > display? If it shows the unexpanded form, does that mean the actual > cwd will change if, say, HOME changes? > > Should we store the cwd after tilde-expansion? I don't think so, because this is an inferior setting. I.e., e.g., when remote debugging, only the target (remote side) can do the expansion, and the user may set this before connecting to a target (remote, native, etc.), even. Thanks, Pedro Alves