[Emacs-diffs] /srv/bzr/emacs/emacs-24 r111318: * Makefile.in (install-man): Ignore gzip exit status.

[Glenn Morris]

------------------------------------------------------------
revno: 111318
committer: Glenn Morris <address@hidden>
branch nick: emacs-24
timestamp: Mon 2013-03-04 19:43:52 -0800
message:
  * Makefile.in (install-man): Ignore gzip exit status.
modified:
  ChangeLog
  Makefile.in

=== modified file 'ChangeLog'
--- a/ChangeLog 2013-02-14 20:05:10 +0000
+++ b/ChangeLog 2013-03-05 03:43:52 +0000
@@ -1,3 +1,7 @@
+2013-03-05  Glenn Morris  <address@hidden>
+
+       * Makefile.in (install-man): Ignore gzip exit status.
+
 2013-02-14  Paul Eggert  <address@hidden>
 
        Fix AIX port (Bug#13650).

=== modified file 'Makefile.in'
--- a/Makefile.in       2013-01-16 01:41:31 +0000
+++ b/Makefile.in       2013-03-05 03:43:52 +0000
@@ -621,6 +621,12 @@
              ${INSTALL_INFO} --info-dir=$(DESTDIR)${infodir} 
$(DESTDIR)${infodir}/$$elt); \
           done)
 
+## "gzip || true" is because some gzips exit with non-zero status
+## if compression would not reduce the file size.  Eg, the gzip in
+## OpenBSD 4.9 seems to do this (2013/03).  In Emacs, this can
+## only happen with the tiny ctags.1 manpage.  We don't really care if
+## ctags.1 is compressed or not.  "gzip -f" is another option here,
+## but not sure if portable.
 install-man:
        umask 022; ${MKDIR_P} $(DESTDIR)${man1dir}
        thisdir=`/bin/pwd`; \
@@ -631,7 +637,7 @@
           ${INSTALL_DATA} ${mansrcdir}/$${page} 
$(DESTDIR)${man1dir}/$${dest}); \
          ( [ -n "${GZIP_INFO}" ] && [ -n "${GZIP_PROG}" ] ) || continue ; \
          rm -f $(DESTDIR)${man1dir}/$${dest}.gz; \
-         ${GZIP_PROG} -9n $(DESTDIR)${man1dir}/$${dest}; \
+         ${GZIP_PROG} -9n $(DESTDIR)${man1dir}/$${dest} || true; \
        done
 
 ## Install those items from etc/ that need to end up elsewhere.