From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 58885 invoked by alias); 25 Sep 2015 13:53:05 -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 58873 invoked by uid 89); 25 Sep 2015 13:53:05 -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,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 X-HELO: smtp.gentoo.org Received: from smtp.gentoo.org (HELO smtp.gentoo.org) (140.211.166.183) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Fri, 25 Sep 2015 13:53:03 +0000 Received: from vapier.lan (localhost [127.0.0.1]) by smtp.gentoo.org (Postfix) with SMTP id 54323340861; Fri, 25 Sep 2015 13:53:01 +0000 (UTC) Date: Fri, 25 Sep 2015 13:53:00 -0000 From: Mike Frysinger To: Romain Naour Cc: Doug Evans , gdb-patches , Yao Qi , Thomas Petazzoni Subject: Re: [PATCH] [RFC] gdb: add disable-docs option Message-ID: <20150925135301.GA21570@vapier.lan> Mail-Followup-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> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="k+w/mQv8wyuph6w0" Content-Disposition: inline In-Reply-To: <56050C9B.3050000@openwide.fr> X-IsSubscribed: yes X-SW-Source: 2015-09/txt/msg00579.txt.bz2 --k+w/mQv8wyuph6w0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Content-length: 2133 On 25 Sep 2015 10:58, Romain Naour wrote: > Le 24/09/2015 23:46, Doug Evans a =C3=A9crit : > > On Thu, Sep 24, 2015 at 2:12 PM, Mike Frysinger wrote: > >> 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 makeinf= o isn't > >> available, it doesn't matter as the pages aren't regenerated on the us= er's > >> system. maybe you're applying patches to the source that cause the do= cs 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 = and > >> installing the pages that already exist. alternatively, you can adjus= t your > >> build to update the timestamps of the generated files so the build won= 't try > >> to regenerate them. > >=20 > > Agreed. > >=20 > > 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]). your configure flag only impacts gdb though -- there are other subdirs (like the binutils related ones) that also install docs. in Gentoo, when we want to do this, we simply delete the /usr/share/... dirs, or we set it to a place like --docdir=3D/nukeme and then rm that. either way, i'm not sure the additional flags in this case are really worth the maintenance overhead when installing+deleting after the fact is trivial. do you have data to show that the overhead you're saving is significant ? -mike --k+w/mQv8wyuph6w0 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature Content-length: 819 -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJWBVG9AAoJEEFjO5/oN/WBdvcQAI7xR7TVjKQiWdz1o/ooVnVD EqjvBds9PPmXuT8S+MwKDCiRwxOGXBNW/iH69Zp+k8uA4nogs1HbdVSTrUVrKltH 5UOxX/Mnzs/sNdLOUX7JwB2MfMOGatd+au5psAESOTY+hNA1RVdUcSJyl8KB9pyO lancrgjoqTdKVmdIyKegwM5hLlbikjDiVXoxnGxZrar/GwN8/U+ctHf6MFQk2NZ/ W8CSZYJ8hoy8hwpMe/fEDd8Na5vHocJv8ZnxA8yEygizbxR61liA1YRKeEw7vELR uKtqWdHcGpfpcQxWV+Y2GJh2BsrK3RCD5g/DBvTSaWM3zQp6aHI9CREecybRnEj2 W6WpLge2yN+BHzYctuLG9BjwEIeWZky9aC5YPE9g5V78Soiu/N92EH7OlFUpwp9N Ye/fhFlsjFWaW8Knpl2eiJM4p2//rcR2yqq4Zonfuk00GvkGl/G7JzfYlZ6BNJHR i89ndU0Hvn9Y9gZ/MogrhARxlbrtoPQ2EIGugBhytPsHnZ4zEX38dPWEgsz0rjIk R20t5qhTEOtOczdZAxWE5GZUVvffwInUGuU6Fr9goEB43WrjGdWABcVHw7VZd+Rd QkZ+a/PEzjb05fdILaARw91h5mwaLUvCwALzD7ShdGUqLmnMa8mNDUden2OSnFnF 1bX9VyuvTP8zLr2Ro/dX =vwEh -----END PGP SIGNATURE----- --k+w/mQv8wyuph6w0--