binutils: doc: move artifacts back to MAINTAINERCLEANFILES
authorSimon Marchi <simon.marchi@efficios.com>
Sat, 7 Mar 2020 03:06:34 +0000 (22:06 -0500)
committerSimon Marchi <simon.marchi@polymtl.ca>
Sat, 7 Mar 2020 03:06:34 +0000 (22:06 -0500)
In commit 2b44a6a237 (" binutils: doc: make `make clean` clean more
things"), I moved the doc build artifacts to MOSTLYCLEANFILES, which
made them get removed by "make clean".

Because generating binutils.info requires makeinfo, and we do not want
to require makeinfo when building from the tarball, binutils.info should
not get removed by "make clean" (otherwise, it won't be included in the
tarball).

And to be consistent with other projects (e.g. ld and gas), we also want
to ship the built man pages in the tarball.

This patch puts back all these in MAINTAINERCLEANFILES, so that they are
bundled in the tarball, and only cleaned if you use "make
maintainer-clean".

Tested by building a source release and confirming they are present.

binutils/ChangeLog:

PR 25491
* doc/Makefile.am: Rename MOSTLYCLEANFILES to MAINTAINERCLEANFILES.
* doc/Makefile.in: Re-generate.

binutils/ChangeLog
binutils/doc/Makefile.am
binutils/doc/Makefile.in

index 840fc4d5891ecb9c7493881b8aa011a653936b54..2db388daf4734c6d208905293afbdf70232cf34d 100644 (file)
@@ -1,3 +1,9 @@
+2020-03-06  Simon Marchi  <simon.marchi@efficios.com>
+
+       PR 25491
+       * doc/Makefile.am: Rename MOSTLYCLEANFILES to MAINTAINERCLEANFILES.
+       * doc/Makefile.in: Re-generate.
+
 2020-03-06  Nick Clifton  <nickc@redhat.com>
 
        * objcopy.c (check_new_section_flags): New function.  Reject the
index a1e07ec246e7006639ccdc619d8f9560ab6c99c7..af6370a23a8a7f694e38bf259a048cbff70f5262 100644 (file)
@@ -162,7 +162,7 @@ cxxfilt.man:        $(binutils_TEXI)
                mv -f $@.T$$$$ $@) || (rm -f $@.T$$$$ && exit 1)
        rm -f $(DEMANGLER_NAME).pod
 
-MOSTLYCLEANFILES     = $(man_MANS) binutils.info cxxfilt.man
+MAINTAINERCLEANFILES = $(man_MANS) binutils.info cxxfilt.man
 
 $(DEMANGLER_NAME).1: cxxfilt.man Makefile
        if test -f cxxfilt.man; then \
index b0e7b7b6c8969f6b5162fc101d4802e62c4bfa49..b00260bc25f86d30f27d91834ea516a1a961e171 100644 (file)
@@ -429,7 +429,7 @@ AM_MAKEINFOFLAGS = -I "$(srcdir)" -I "$(top_srcdir)/../libiberty" \
 TEXI2DVI = texi2dvi -I "$(srcdir)" -I "$(top_srcdir)/../libiberty" \
                    -I "$(top_srcdir)/../bfd/doc" -I ../../bfd/doc
 
-MOSTLYCLEANFILES = $(man_MANS) binutils.info cxxfilt.man
+MAINTAINERCLEANFILES = $(man_MANS) binutils.info cxxfilt.man
 all: all-am
 
 .SUFFIXES:
@@ -712,7 +712,6 @@ install-strip:
            "INSTALL_PROGRAM_ENV=STRIPPROG='$(STRIP)'" install; \
        fi
 mostlyclean-generic:
-       -test -z "$(MOSTLYCLEANFILES)" || rm -f $(MOSTLYCLEANFILES)
 
 clean-generic:
 
@@ -723,6 +722,7 @@ distclean-generic:
 maintainer-clean-generic:
        @echo "This command is intended for maintainers to use"
        @echo "it deletes files that may require special tools to rebuild."
+       -test -z "$(MAINTAINERCLEANFILES)" || rm -f $(MAINTAINERCLEANFILES)
 clean: clean-am
 
 clean-am: clean-aminfo clean-generic clean-libtool mostlyclean-am