From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15173 invoked by alias); 23 Nov 2002 01:42:43 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 15166 invoked from network); 23 Nov 2002 01:42:42 -0000 Received: from unknown (HELO mx1.redhat.com) (66.187.233.31) by sources.redhat.com with SMTP; 23 Nov 2002 01:42:42 -0000 Received: from int-mx1.corp.redhat.com (int-mx1.corp.redhat.com [172.16.52.254]) by mx1.redhat.com (8.11.6/8.11.6) with ESMTP id gAN1IaP29866 for ; Fri, 22 Nov 2002 20:18:36 -0500 Received: from pobox.corp.redhat.com (pobox.corp.redhat.com [172.16.52.156]) by int-mx1.corp.redhat.com (8.11.6/8.11.6) with ESMTP id gAN1ggD23174; Fri, 22 Nov 2002 20:42:42 -0500 Received: from localhost.redhat.com (romulus-int.sfbay.redhat.com [172.16.27.46]) by pobox.corp.redhat.com (8.11.6/8.11.6) with ESMTP id gAN1gew11175; Fri, 22 Nov 2002 20:42:41 -0500 Received: by localhost.redhat.com (Postfix, from userid 469) id 749A5FF79; Fri, 22 Nov 2002 20:38:25 -0500 (EST) From: Elena Zannoni MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-ID: <15838.56337.22520.798218@localhost.redhat.com> Date: Fri, 22 Nov 2002 17:42:00 -0000 To: gdb-patches@sources.redhat.com Cc: Andrew Cagney , Jim Blandy Subject: Re: [RFA] doc/Makefile.in install In-Reply-To: <15838.50063.626470.203356@localhost.redhat.com> References: <15838.36534.883345.847460@localhost.redhat.com> <3DDEACAE.4010607@redhat.com> <15838.50063.626470.203356@localhost.redhat.com> X-SW-Source: 2002-11/txt/msg00572.txt.bz2 Elena Zannoni writes: > Andrew Cagney writes: > > > > > I think, at some point in the distant past, it was controversial > > > whether packages should install their info files by default. I think > > > that was back when disks were much smaller than they are now. > > > Nowadays most packages install their info by default, so there's no > > > reason for GDB not to do so as well. > > > > So that's the reason! I knew there was one but couldn't remember what > > it was. > > > > Elena, also note: > > http://sources.redhat.com/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gdb&pr=723 > > http://sources.redhat.com/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gdb&pr=245 > > http://www.gnu.org/prep/standards_55.html#SEC55 > > > > Andrew > > > > Argh. There is way more than what meets the eye. Ok, I'll look into > this a bit more. > Hmmm... how about this? It seems to do the right thing for me. Can somebody give it a try on their system? thanks Elena 2002-11-22 Elena Zannoni * Makefile.in (install-info): Run the install-info command as part of the post install steps only. (uninstall-info): New target. (uninstall): New target. Index: Makefile.in =================================================================== RCS file: /cvs/uberbaum/gdb/doc/Makefile.in,v retrieving revision 1.24 diff -u -p -r1.24 Makefile.in --- Makefile.in 22 Nov 2002 21:02:20 -0000 1.24 +++ Makefile.in 23 Nov 2002 01:39:31 -0000 @@ -156,6 +156,7 @@ install-info: $(INFO_DEPS) else : ; fi; \ done; \ done + $(POST_INSTALL) @if $(SHELL) -c 'install-info --version | sed 1q | fgrep -s -v -i debian' >/dev/null 2>&1; then \ list='$(INFO_DEPS)'; \ for file in $$list; do \ @@ -164,6 +165,22 @@ install-info: $(INFO_DEPS) done; \ else : ; fi +uninstall-info: + $(PRE_UNINSTALL) + @if $(SHELL) -c 'install-info --version | sed 1q | fgrep -s -v -i debian' >/dev/null 2>&1; then \ + ii=yes; \ + else ii=; fi; \ + list='$(INFO_DEPS)'; \ + for file in $$list; do \ + test -z "$$ii" \ + || install-info --info-dir=$(DESTDIR)$(infodir) --remove $$file; \ + done + $(NORMAL_UNINSTALL) + list='$(INFO_DEPS)'; \ + for file in $$list; do \ + (cd $(DESTDIR)$(infodir) && rm -f $$file $$file-[0-9] $$file-[0-9][0-9]); \ + done + install-html: html for i in *.html ; do \ $(INSTALL_DATA) $$i $(htmldir)/$$i ; \ @@ -433,3 +450,5 @@ maintainer-clean realclean: distclean rm -f GDBvn.texi *.info* *.dvi *.ps *.html *.pdf install: install-info + +uninstall: uninstall-info