[bug-gettext] recursive install-pdf fails in po directory (was: Re: bison-2.5 released [stable])

[Joel E. Denny]

I've added bug-gettext as this problem can be fixed in autopoint.

On Sun, 15 May 2011, Hans Aberg wrote:

> On 15 May 2011, at 03:29, Joel E. Denny wrote:
> 
> > Bison version 2.5 is now available.

> I think there was a problem with 'make install-pdf' - I had to go to the 
> directory doc/ to get it work. (Somehow, I got it installed, but I do 
> not remember exactly how.)

Thanks for reporting that.

This problem isn't unique to bison.  `make install-pdf' also fails when 
run from the root directory of the master branch of coreutils.

In both cases, the problem is that there is no install-pdf target in 
po/Makefile.in.in (or bison's runtime-po/Makefile.in.in), so the 
install-pdf recursion fails there.

Those po make files are generated by autopoint.  Bruno fixed the same 
problem for autopoint-generated intl make files a few years ago:

  http://lists.gnu.org/archive/html/bug-gnu-utils/2007-05/msg00025.html

Bruno, the following patch applies a similar fix to po make files and thus 
solves the problem for me in bison.  Can this be included in the next 
gettext release?

Thanks.

>From 46245f178c49e78a77502177f3c8a690efd31b50 Mon Sep 17 00:00:00 2001
From: Joel E. Denny <address@hidden>
Date: Sun, 29 May 2011 14:47:56 -0400
Subject: [PATCH] Fix recursion of install-* into po directories.

Bison's install-pdf bug reported by Hans Aberg at
<http://lists.gnu.org/archive/html/bug-bison/2011-05/msg00008.html>.
* gettext-runtime/po/Makefile.in.in
* gettext-tools/po/Makefile.in.in
(install-info, install-dvi, install-ps, install-pdf)
(install-html): New targets.
---
 ChangeLog                         |   10 ++++++++++
 gettext-runtime/po/Makefile.in.in |    2 ++
 gettext-tools/po/Makefile.in.in   |    2 ++
 3 files changed, 14 insertions(+), 0 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 376bb84..0f0c376 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2011-05-29  Joel E. Denny  <address@hidden>  (tiny change)
+
+       Fix recursion of install-* into po directories.
+       Bison's install-pdf bug reported by Hans Aberg at
+       <http://lists.gnu.org/archive/html/bug-bison/2011-05/msg00008.html>.
+       * gettext-runtime/po/Makefile.in.in
+       * gettext-tools/po/Makefile.in.in
+       (install-info, install-dvi, install-ps, install-pdf)
+       (install-html): New targets.
+
 2010-09-24  Bruno Haible  <address@hidden>
 
        * m4/libtool.m4: Update from libtool-2.4, with modifications.
diff --git a/gettext-runtime/po/Makefile.in.in 
b/gettext-runtime/po/Makefile.in.in
index e666eaa..5a1fab4 100644
--- a/gettext-runtime/po/Makefile.in.in
+++ b/gettext-runtime/po/Makefile.in.in
@@ -273,6 +273,8 @@ install-data-yes: all
 
 install-strip: install
 
+install-info install-dvi install-ps install-pdf install-html:
+
 installdirs: installdirs-exec installdirs-data
 installdirs-exec:
 installdirs-data: address@hidden@
diff --git a/gettext-tools/po/Makefile.in.in b/gettext-tools/po/Makefile.in.in
index e666eaa..5a1fab4 100644
--- a/gettext-tools/po/Makefile.in.in
+++ b/gettext-tools/po/Makefile.in.in
@@ -273,6 +273,8 @@ install-data-yes: all
 
 install-strip: install
 
+install-info install-dvi install-ps install-pdf install-html:
+
 installdirs: installdirs-exec installdirs-data
 installdirs-exec:
 installdirs-data: address@hidden@
-- 
1.7.0.4