From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 70504 invoked by alias); 22 Apr 2015 07:21:23 -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 70490 invoked by uid 89); 22 Apr 2015 07:21:22 -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,RCVD_IN_DNSWL_NONE,SPF_SOFTFAIL autolearn=no version=3.3.2 X-HELO: mtaout28.012.net.il Received: from mtaout28.012.net.il (HELO mtaout28.012.net.il) (80.179.55.184) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 22 Apr 2015 07:21:20 +0000 Received: from conversion-daemon.mtaout28.012.net.il by mtaout28.012.net.il (HyperSendmail v2007.08) id <0NN7002005OWRW00@mtaout28.012.net.il> for gdb-patches@sourceware.org; Wed, 22 Apr 2015 10:20:03 +0300 (IDT) Received: from HOME-C4E4A596F7 ([87.69.4.28]) by mtaout28.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0NN700KFD5PF4P70@mtaout28.012.net.il>; Wed, 22 Apr 2015 10:20:03 +0300 (IDT) Date: Wed, 22 Apr 2015 07:21:00 -0000 From: Eli Zaretskii Subject: Re: [PATCH 4/4] compile: Add 'set compile-gcc' In-reply-to: <20150421213657.14147.60506.stgit@host1.jankratochvil.net> To: Jan Kratochvil Cc: gdb-patches@sourceware.org, pmuldoon@redhat.com Reply-to: Eli Zaretskii Message-id: <83ioco4oid.fsf@gnu.org> References: <20150421213635.14147.15653.stgit@host1.jankratochvil.net> <20150421213657.14147.60506.stgit@host1.jankratochvil.net> X-IsSubscribed: yes X-SW-Source: 2015-04/txt/msg00813.txt.bz2 > From: Jan Kratochvil > Cc: Phil Muldoon > Date: Tue, 21 Apr 2015 23:36:57 +0200 > > parameter of GCC's set_arguments. GCC decides on the meaning of the > triplet_regexp parameter according to the success of: > > + if (access (triplet_regexp, X_OK) == 0) Does triplet_regexp include the .exe extension on platforms that need it? Otherwise, 'access' will fail on those platforms. (Sorry, couldn't see where's the actual code, so couldn't verify that myself.) > + add_setshow_string_cmd ("compile-gcc", class_support, > + &compile_gcc, > + _("Set compile command GCC driver filename"), > + _("Show compile command GCC driver filename"), > + _("\ > +It should be absolute pathname to the gcc executable.\n\ > +If empty the default target triplet will be searched in $PATH."), > + NULL, show_compile_gcc, &setlist, &showlist); This doesn't offer file-name completion, does it? I think it should. > + By default the environment variable > +@code{PATH} (@code{PATH} from shell that executed @value{GDBN}, not the > +one set by @value{GDBN} command @code{set environment}). This sentence is incomplete, I think you need to combine it with the next one. (I suggest to say the simple thing first, and then add the caveat about which PATH is searched, instead of having the caveat in the middle, making it harder to comprehend.) > +Besides the compiler driver @value{GDBN} needs also shared library > +@file{libcc1.so}. It is searched in default shared library search path > +(overridable with usual environment variable @code{LD_LIBRARY_PATH}), > +unrelated to @code{PATH} or @code{set compile-gcc} settings. This is Unix logic, btw. On Windows, libcc1.dll will either be in the same directory where the GCC driver is, or on PATH, or in libexec. So maybe you should qualify the above by saying this is on Posix hosts; we could then amend it when this is supported on Windows (assuming it isn't for now).