From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 16892 invoked by alias); 25 Sep 2015 11:50:28 -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 16319 invoked by uid 89); 25 Sep 2015 11:50:28 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.2 X-HELO: usevmg20.ericsson.net Received: from usevmg20.ericsson.net (HELO usevmg20.ericsson.net) (198.24.6.45) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Fri, 25 Sep 2015 11:50:26 +0000 Received: from EUSAAHC004.ericsson.se (Unknown_Domain [147.117.188.84]) by usevmg20.ericsson.net (Symantec Mail Security) with SMTP id E5.02.32596.FB5D4065; Fri, 25 Sep 2015 07:04:00 +0200 (CEST) Received: from [142.133.110.144] (147.117.188.8) by smtp-am.internal.ericsson.com (147.117.188.86) with Microsoft SMTP Server id 14.3.248.2; Fri, 25 Sep 2015 07:50:24 -0400 Subject: Re: [PATCH] [RFC] gdb: add disable-docs option To: Romain Naour , Doug Evans , gdb-patches , Yao Qi , Thomas Petazzoni References: <1441532245-23124-1-git-send-email-romain.naour@openwide.fr> <20150924211221.GA10729@vapier.lan> <56050C9B.3050000@openwide.fr> From: Simon Marchi Message-ID: <560534FF.2020208@ericsson.com> Date: Fri, 25 Sep 2015 11:50:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <56050C9B.3050000@openwide.fr> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes X-SW-Source: 2015-09/txt/msg00572.txt.bz2 On 15-09-25 04:58 AM, Romain Naour wrote: > Hello, >=20 > Thanks for your reply! >=20 > Le 24/09/2015 23:46, Doug Evans a =C3=A9crit : >> On Thu, Sep 24, 2015 at 2:12 PM, Mike Frysinger wrot= e: >>> On 06 Sep 2015 11:37, Romain Naour wrote: >>>> If makeinfo is not found in the system then the missing >>>> script is used to warn the user. >>>> >>>> Before commit e30465112ed4c6320dd19107302057a5f7712cf2 the missing >>>> script returned 0 after printing the message. >>>> >>>> Now, missing return 127 (command not found) to the Makefile and >>>> the build fail. >>>> >>>> As suggested [1], add a new option to disable the documentation. >>> >>> aren't info pages shipped as part of the release ? so even if makeinfo= isn't >>> available, it doesn't matter as the pages aren't regenerated on the use= r's >>> system. maybe you're applying patches to the source that cause the doc= s to be >>> regenerated ? if that's the case, i think disabling the docs entirely = is the >>> wrong way to go. instead it should be skipping the regeneration step a= nd >>> installing the pages that already exist. alternatively, you can adjust= your >>> build to update the timestamps of the generated files so the build won'= t try >>> to regenerate them. >>> -mike >> >> Agreed. >> >> Sometimes releases have gone out with bad timestamps which need to be >> fixed, but yeah there should be no need to disable doc generation. >> >=20 > Actually, I really want to disable the documentation entirely since all > documentation installed by packages (like gdb) are automatically removed = from > the generated filesystem at the end of the build (see [1] and [2]). >=20 > Until gdb 7.8, ac_cv_prog_MAKEINFO=3Dmissing was used to workaround the b= uild > issue when makeinfo was missing. But it's not enough since gdb 7.9. >=20 > Best regards, > Romain Naour >=20 > [1] http://buildroot.uclibc.org/downloads/manual/manual.html#faq-no-doc-o= n-target >=20 > [2] http://git.buildroot.net/buildroot/tree/Makefile#n559 I don't know if this is acceptable, but I managed to effectively disable bu= ilding of the doc by setting MAKEINFO=3Dtrue (as in /bin/true) during the build. See http://git.eclipse.org/c/cdt/org.eclipse.cdt.git/tree/dsf-gdb/org.eclip= se.cdt.tests.dsf.gdb/scripts/download-build-gdb.sh