From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 11052 invoked by alias); 2 May 2011 17:06:50 -0000 Received: (qmail 11029 invoked by uid 22791); 2 May 2011 17:06:49 -0000 X-SWARE-Spam-Status: No, hits=-1.2 required=5.0 tests=AWL,BAYES_00,SPF_SOFTFAIL X-Spam-Check-By: sourceware.org Received: from mtaout20.012.net.il (HELO mtaout20.012.net.il) (80.179.55.166) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 02 May 2011 17:06:33 +0000 Received: from conversion-daemon.a-mtaout20.012.net.il by a-mtaout20.012.net.il (HyperSendmail v2007.08) id <0LKK00A00VBD3T00@a-mtaout20.012.net.il> for gdb-patches@sourceware.org; Mon, 02 May 2011 20:06:31 +0300 (IDT) Received: from HOME-C4E4A596F7 ([84.228.185.101]) by a-mtaout20.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0LKK009MUVITLL60@a-mtaout20.012.net.il>; Mon, 02 May 2011 20:06:31 +0300 (IDT) Date: Mon, 02 May 2011 17:06:00 -0000 From: Eli Zaretskii Subject: Re: [patch 4/3] readline-6.2: Substitute inc-hist.texinfo In-reply-to: <20110502162014.GA28920@host1.jankratochvil.net> To: Jan Kratochvil Cc: gdb-patches@sourceware.org Reply-to: Eli Zaretskii Message-id: <83vcxt3ucj.fsf@gnu.org> References: <20110502141121.GA19958@host1.jankratochvil.net> <20110502162014.GA28920@host1.jankratochvil.net> X-IsSubscribed: yes 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 X-SW-Source: 2011-05/txt/msg00027.txt.bz2 > Date: Mon, 2 May 2011 18:20:14 +0200 > From: Jan Kratochvil > Cc: gdb-patches@sourceware.org > > On Mon, 02 May 2011 16:34:51 +0200, Eli Zaretskii wrote: > > > --- a/readline/doc/hsuser.texi > > > +++ b/readline/doc/hsuser.texi > > > @@ -26,9 +26,10 @@ into another language, under the above conditions for modified versions. > > > @node Using History Interactively > > > @chapter Using History Interactively > > > > > > -@ifclear BashFeatures > > > -@defcodeindex bt > > > -@end ifclear > > > +@c GDB bundling modification: > > > +@c @ifclear BashFeatures > > > +@c @defcodeindex bt > > > +@c @end ifclear > > > > Why do we need to comment this out? > > BTW it was already commented out the same way in readline-5.1. Yes, I know. But since we are cleaning up our act, might as well clean this as well ;-) > > > If BashFeatures is not set, the > > @defcodeindex line will not be in effect anyway. Is something setting > > BashFeatures anywhere when we build our docs? > > You are right BashFeatures is not set but this is @ifclear. Sorry, I don't know what I was smoking when I wrote that. But we can't use -DBashFeatures in the makeinfo command line, because it is really too tied up with Bash. We need a separate flag. > > @ifset HsUser > > @pxref{Programming with GNU History, , , history, GNU History Library}. > > @end ifset > > @ifclear HsUser > > @pxref{Programming with GNU History}. > > @end ifclear > > > > Then projects that include just hsuser.texi will need to pass the > > "-DHsUser" switch to makeinfo, while Readline will not. > > > > I'm okay with making these changes for now, to allow people building > > GDB without failures. But if Chet accepts the above change, we should > > switch to it in GDB, so that our doc/Makefile.in is ready for the next > > upstream release of Readline. > > I guess defined BashFeatures was partially meant for some inclusion, OK. Yes, it was meant for inclusion in the Bash manual. The problem is it mentions Bash explicitly, so it's not good for us. Go ahead and commit your changes, and I will see how to fix that and submit the changes to Chet. Thanks.