[1/3] binutils: doc: remove stale things from Makefile

Message ID 20190917025731.300902-1-simon.marchi@polymtl.ca
State New
Headers show
Series
  • [1/3] binutils: doc: remove stale things from Makefile
Related show

Commit Message

Simon Marchi Sept. 17, 2019, 2:57 a.m.
The `install-data-local: install-info` part: the comment refers to when
the Makefile had the cygnus option.  It no longer uses that, so the info
file should get installed without that rule.  I verified that `make
install` still installed $(prefix)/share/info/binutils.info.

The `DISTCLEANFILES` part: the comment talks about Automake 1.9, we are
using 1.15.1, which doesn't appear to have this problem.  binutils.info
is built fine without this snippet.

Note: with my automake 1,15.1 (built from source), I get a spurious diff
in Makefile.in, where the

  runstatedir = @runstatedir@

line gets removed.  I am not sure why, but I haven't included it.

binutils/ChangeLog:

	* doc/Makefile.am (install-data-loca): Remove target.
	(DISTCLEANFILES): Remove variable.
	* doc/Makefile.in: Re-generate.
---
 binutils/doc/Makefile.am | 11 -----------
 binutils/doc/Makefile.in | 24 +++++++-----------------
 2 files changed, 7 insertions(+), 28 deletions(-)

-- 
2.23.0

Comments

Simon Marchi Sept. 17, 2019, 2:59 a.m. | #1
Oops, I sent this series to gdb-patches@ instead of binutils@, please ignore.  Sorry for the noise.

On 2019-09-16 10:57 p.m., Simon Marchi wrote:
> The `install-data-local: install-info` part: the comment refers to when

> the Makefile had the cygnus option.  It no longer uses that, so the info

> file should get installed without that rule.  I verified that `make

> install` still installed $(prefix)/share/info/binutils.info.

> 

> The `DISTCLEANFILES` part: the comment talks about Automake 1.9, we are

> using 1.15.1, which doesn't appear to have this problem.  binutils.info

> is built fine without this snippet.

> 

> Note: with my automake 1,15.1 (built from source), I get a spurious diff

> in Makefile.in, where the

> 

>   runstatedir = @runstatedir@

> 

> line gets removed.  I am not sure why, but I haven't included it.

> 

> binutils/ChangeLog:

> 

> 	* doc/Makefile.am (install-data-loca): Remove target.

> 	(DISTCLEANFILES): Remove variable.

> 	* doc/Makefile.in: Re-generate.

> ---

>  binutils/doc/Makefile.am | 11 -----------

>  binutils/doc/Makefile.in | 24 +++++++-----------------

>  2 files changed, 7 insertions(+), 28 deletions(-)

> 

> diff --git a/binutils/doc/Makefile.am b/binutils/doc/Makefile.am

> index 49958d242a4e..248e7cdcf348 100644

> --- a/binutils/doc/Makefile.am

> +++ b/binutils/doc/Makefile.am

> @@ -174,19 +174,8 @@ $(DEMANGLER_NAME).1: cxxfilt.man Makefile

>  	    -e 's/cxxfilt/$(DEMANGLER_NAME)/' < $$man \

>  		> $(DEMANGLER_NAME).1

>  

> -# We want install to imply install-info as per GNU standards, despite the

> -# cygnus option.

> -install-data-local: install-info

> -

>  MAINTAINERCLEANFILES = binutils.info

>  

> -# Automake 1.9 will only build info files in the objdir if they are

> -# mentioned in DISTCLEANFILES.  It doesn't have to be unconditional,

> -# though, so we use a bogus condition.

> -if GENINSRC_NEVER

> -DISTCLEANFILES = binutils.info

> -endif

> -

>  # Maintenance

>  

>  # We need it for the taz target in ../../Makefile.in.

> diff --git a/binutils/doc/Makefile.in b/binutils/doc/Makefile.in

> index c66caa832ecc..66b2d1dd7b70 100644

> --- a/binutils/doc/Makefile.in

> +++ b/binutils/doc/Makefile.in

> @@ -429,11 +429,6 @@ TEXI2DVI = texi2dvi -I "$(srcdir)" -I "$(top_srcdir)/../libiberty" \

>  

>  MOSTLYCLEANFILES = $(DEMANGLER_NAME).1

>  MAINTAINERCLEANFILES = binutils.info

> -

> -# Automake 1.9 will only build info files in the objdir if they are

> -# mentioned in DISTCLEANFILES.  It doesn't have to be unconditional,

> -# though, so we use a bogus condition.

> -@GENINSRC_NEVER_TRUE@DISTCLEANFILES = binutils.info

>  all: all-am

>  

>  .SUFFIXES:

> @@ -723,7 +718,6 @@ clean-generic:

>  distclean-generic:

>  	-test -z "$(CONFIG_CLEAN_FILES)" || rm -f $(CONFIG_CLEAN_FILES)

>  	-test . = "$(srcdir)" || test -z "$(CONFIG_CLEAN_VPATH_FILES)" || rm -f $(CONFIG_CLEAN_VPATH_FILES)

> -	-test -z "$(DISTCLEANFILES)" || rm -f $(DISTCLEANFILES)

>  

>  maintainer-clean-generic:

>  	@echo "This command is intended for maintainers to use"

> @@ -749,7 +743,7 @@ info: info-am

>  

>  info-am: $(INFO_DEPS) info-local

>  

> -install-data-am: install-data-local install-info-am install-man

> +install-data-am: install-info-am install-man

>  

>  install-dvi: install-dvi-am

>  

> @@ -897,12 +891,12 @@ uninstall-man: uninstall-man1

>  	clean-libtool cscopelist-am ctags-am dist-info distclean \

>  	distclean-generic distclean-libtool distdir dvi dvi-am html \

>  	html-am info info-am info-local install install-am \

> -	install-data install-data-am install-data-local install-dvi \

> -	install-dvi-am install-exec install-exec-am install-html \

> -	install-html-am install-info install-info-am install-man \

> -	install-man1 install-pdf install-pdf-am install-ps \

> -	install-ps-am install-strip installcheck installcheck-am \

> -	installdirs maintainer-clean maintainer-clean-aminfo \

> +	install-data install-data-am install-dvi install-dvi-am \

> +	install-exec install-exec-am install-html install-html-am \

> +	install-info install-info-am install-man install-man1 \

> +	install-pdf install-pdf-am install-ps install-ps-am \

> +	install-strip installcheck installcheck-am installdirs \

> +	maintainer-clean maintainer-clean-aminfo \

>  	maintainer-clean-generic mostlyclean mostlyclean-aminfo \

>  	mostlyclean-generic mostlyclean-libtool pdf pdf-am ps ps-am \

>  	tags-am uninstall uninstall-am uninstall-dvi-am \

> @@ -1028,10 +1022,6 @@ $(DEMANGLER_NAME).1: cxxfilt.man Makefile

>  	    -e 's/cxxfilt/$(DEMANGLER_NAME)/' < $$man \

>  		> $(DEMANGLER_NAME).1

>  

> -# We want install to imply install-info as per GNU standards, despite the

> -# cygnus option.

> -install-data-local: install-info

> -

>  # Maintenance

>  

>  # We need it for the taz target in ../../Makefile.in.

>

Patch

diff --git a/binutils/doc/Makefile.am b/binutils/doc/Makefile.am
index 49958d242a4e..248e7cdcf348 100644
--- a/binutils/doc/Makefile.am
+++ b/binutils/doc/Makefile.am
@@ -174,19 +174,8 @@  $(DEMANGLER_NAME).1: cxxfilt.man Makefile
 	    -e 's/cxxfilt/$(DEMANGLER_NAME)/' < $$man \
 		> $(DEMANGLER_NAME).1
 
-# We want install to imply install-info as per GNU standards, despite the
-# cygnus option.
-install-data-local: install-info
-
 MAINTAINERCLEANFILES = binutils.info
 
-# Automake 1.9 will only build info files in the objdir if they are
-# mentioned in DISTCLEANFILES.  It doesn't have to be unconditional,
-# though, so we use a bogus condition.
-if GENINSRC_NEVER
-DISTCLEANFILES = binutils.info
-endif
-
 # Maintenance
 
 # We need it for the taz target in ../../Makefile.in.
diff --git a/binutils/doc/Makefile.in b/binutils/doc/Makefile.in
index c66caa832ecc..66b2d1dd7b70 100644
--- a/binutils/doc/Makefile.in
+++ b/binutils/doc/Makefile.in
@@ -429,11 +429,6 @@  TEXI2DVI = texi2dvi -I "$(srcdir)" -I "$(top_srcdir)/../libiberty" \
 
 MOSTLYCLEANFILES = $(DEMANGLER_NAME).1
 MAINTAINERCLEANFILES = binutils.info
-
-# Automake 1.9 will only build info files in the objdir if they are
-# mentioned in DISTCLEANFILES.  It doesn't have to be unconditional,
-# though, so we use a bogus condition.
-@GENINSRC_NEVER_TRUE@DISTCLEANFILES = binutils.info
 all: all-am
 
 .SUFFIXES:
@@ -723,7 +718,6 @@  clean-generic:
 distclean-generic:
 	-test -z "$(CONFIG_CLEAN_FILES)" || rm -f $(CONFIG_CLEAN_FILES)
 	-test . = "$(srcdir)" || test -z "$(CONFIG_CLEAN_VPATH_FILES)" || rm -f $(CONFIG_CLEAN_VPATH_FILES)
-	-test -z "$(DISTCLEANFILES)" || rm -f $(DISTCLEANFILES)
 
 maintainer-clean-generic:
 	@echo "This command is intended for maintainers to use"
@@ -749,7 +743,7 @@  info: info-am
 
 info-am: $(INFO_DEPS) info-local
 
-install-data-am: install-data-local install-info-am install-man
+install-data-am: install-info-am install-man
 
 install-dvi: install-dvi-am
 
@@ -897,12 +891,12 @@  uninstall-man: uninstall-man1
 	clean-libtool cscopelist-am ctags-am dist-info distclean \
 	distclean-generic distclean-libtool distdir dvi dvi-am html \
 	html-am info info-am info-local install install-am \
-	install-data install-data-am install-data-local install-dvi \
-	install-dvi-am install-exec install-exec-am install-html \
-	install-html-am install-info install-info-am install-man \
-	install-man1 install-pdf install-pdf-am install-ps \
-	install-ps-am install-strip installcheck installcheck-am \
-	installdirs maintainer-clean maintainer-clean-aminfo \
+	install-data install-data-am install-dvi install-dvi-am \
+	install-exec install-exec-am install-html install-html-am \
+	install-info install-info-am install-man install-man1 \
+	install-pdf install-pdf-am install-ps install-ps-am \
+	install-strip installcheck installcheck-am installdirs \
+	maintainer-clean maintainer-clean-aminfo \
 	maintainer-clean-generic mostlyclean mostlyclean-aminfo \
 	mostlyclean-generic mostlyclean-libtool pdf pdf-am ps ps-am \
 	tags-am uninstall uninstall-am uninstall-dvi-am \
@@ -1028,10 +1022,6 @@  $(DEMANGLER_NAME).1: cxxfilt.man Makefile
 	    -e 's/cxxfilt/$(DEMANGLER_NAME)/' < $$man \
 		> $(DEMANGLER_NAME).1
 
-# We want install to imply install-info as per GNU standards, despite the
-# cygnus option.
-install-data-local: install-info
-
 # Maintenance
 
 # We need it for the taz target in ../../Makefile.in.