Re: PATCH: PR ld/10536: Incorrect install-pdf/install-html

["H.J. Lu" <hjl.tools@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1384 bytes --]

On Tue, Aug 18, 2009 at 1:27 PM, Paolo Bonzini<bonzini@gnu.org> wrote:
> On 08/18/2009 09:52 PM, Ralf Wildenhues wrote:
>>
>> * H.J. Lu wrote on Tue, Aug 18, 2009 at 09:44:50PM CEST:
>>>
>>> On Tue, Aug 18, 2009 at 12:40 PM, Ralf Wildenhues wrote:
>>>>
>>>> * H.J. Lu wrote on Tue, Aug 18, 2009 at 09:36:23PM CEST:
>>>>
>>>> Please see this proposed patch set which will obsolete your patch:
>>>> <http://thread.gmane.org/gmane.comp.gnu.binutils/42947/focus=26464>
>>>> as part of the move to Autoconf 2.64, Automake 1.11
>>>
>>> Does your patch cover libiberty?
>>
>> Yes.  My plan is to update all that is contained in the GCC and
>> binutils plus gdb trees at once; and look at the other packages
>> that have their main home in src about what to do after that; see
>> <http://thread.gmane.org/gmane.comp.gnu.binutils/42947/focus=43065>
>> for which directories exactly are in that second list.
>
> I think he meant that libiberty does not use automake.  The libiberty patch
> looks fine, but is _not_ approved until libiberty has switched to Autoconf
> 2.64.  Please resubmit later.  The other parts will be obsolete.
>
> Paolo
>

Hi,

Here is the patch for libiberty.  OK for gcc and src?

Thanks.


-- 
H.J.
---
libiberty/

2009-08-18  H.J. Lu  <hongjiu.lu@intel.com>

	 PR ld/10536
	 * Makefile.in (install-html-recursive): Removed.

[-- Attachment #2: binutils-install-2.patch --]
[-- Type: application/octet-stream, Size: 1535 bytes --]

libiberty/

2009-08-18  H.J. Lu  <hongjiu.lu@intel.com>

	 PR ld/10536
	 * Makefile.in (install-html-recursive): Removed.

Index: libiberty/Makefile.in
===================================================================
--- libiberty/Makefile.in	(revision 6555)
+++ libiberty/Makefile.in	(working copy)
@@ -264,14 +264,14 @@ HTMLS = libiberty.html
 
 html: $(HTMLS)
 
-.PHONY: install-html install-html-am install-html-recursive
+.PHONY: install-html install-html-am
 
 NORMAL_INSTALL = :
 mkdir_p = mkdir -p --
  
 html__strip_dir = `echo $$p | sed -e 's|^.*/||'`;
 
-install-html: install-html-recursive  install-html-am
+install-html: install-html-am
 
 install-html-am: $(HTMLS)
 	@$(NORMAL_INSTALL)
@@ -290,31 +290,6 @@ install-html-am: $(HTMLS)
 	  fi; \
 	done
 
-install-html-recursive:
-	@failcom='exit 1'; \
-	for f in x $$MAKEFLAGS; do \
-	  case $$f in \
-	    *=* | --[!k]*);; \
-	    *k*) failcom='fail=yes';; \
-	  esac; \
-	done; \
-	dot_seen=no; \
-	target=`echo $@ | sed s/-recursive//`; \
-	list='$(SUBDIRS)'; for subdir in $$list; do \
-	  echo "Making $$target in $$subdir"; \
-	  if test "$$subdir" = "."; then \
-	    dot_seen=yes; \
-	    local_target="$$target-am"; \
-	  else \
-	    local_target="$$target"; \
-	  fi; \
-	  (cd $$subdir && $(MAKE) $(AM_MAKEFLAGS) $$local_target) \
-	  || eval $$failcom; \
-	done; \
-	if test "$$dot_seen" = "no"; then \
-	  $(MAKE) $(AM_MAKEFLAGS) "$$target-am" || exit 1; \
-	fi; test -z "$$fail"
-
 TEXISRC = \
 	$(srcdir)/libiberty.texi \
 	$(srcdir)/copying-lib.texi \