[groff] 01/01: [doc] Improve call to generation of `groff.pdf'. Problem reported by Axel Kielhorn <address@hidden>.

[Werner LEMBERG]

wl pushed a commit to branch master
in repository groff.

commit 8c46708258433a331f77e1a34729d74f0b7f5859
Author: Werner Lemberg <address@hidden>
Date:   Sun Oct 12 11:11:01 2014 +0200

    [doc] Improve call to generation of `groff.pdf'.
    Problem reported by Axel Kielhorn <address@hidden>.
    
    Since `groff.texinfo' contains latin-1 characters, some sed
    implementations (like the one used on Mac OS 10.8) fail if called
    with the wrong locale.  Basically, this is a `texi2dvi' bug, but we
    can set `LANG' by ourselves to improve the situation.
    
    * doc/Makefile.in (.texinfo.dvi, .texinfo.pdf): Set `LANG'
    environment variable to empty string.
---
 ChangeLog       |   15 ++++++++++++++-
 doc/Makefile.in |    9 +++++----
 2 files changed, 19 insertions(+), 5 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index bd87d09..18dbe42 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,19 @@
 2014-10-12  Werner LEMBERG  <address@hidden>
 
-       Correctly handle `MAKEINFO' environment variable.
+       [doc] Improve call to generation of `groff.pdf'.
+       Problem reported by Axel Kielhorn <address@hidden>.
+
+       Since `groff.texinfo' contains latin-1 characters, some sed
+       implementations (like the one used on Mac OS 10.8) fail if called
+       with the wrong locale.  Basically, this is a `texi2dvi' bug, but we
+       can set `LANG' by ourselves to improve the situation.
+
+       * doc/Makefile.in (.texinfo.dvi, .texinfo.pdf): Set `LANG'
+       environment variable to empty string.
+
+2014-10-12  Werner LEMBERG  <address@hidden>
+
+       [doc] Correctly handle `MAKEINFO' environment variable.
        Bug introduced in commit from 2014-03-29.
 
        * m4/groff.m4 (GROFF_MAKEINFO): Don't set MAKEINFO to empty string
diff --git a/doc/Makefile.in b/doc/Makefile.in
index ba33c0e..cc55acb 100644
--- a/doc/Makefile.in
+++ b/doc/Makefile.in
@@ -42,8 +42,9 @@ GROFF_BIN_PATH=`echo $(groff_bin_dirs) | sed -e 's|  *|:|g'`
 # usually not needed (however, to build from the repo you NEED it!)
 address@hidden@
 
-# Users who want to print out the groff manual are expected to have
-# a working TeX installation.
+# Users who want to print out the groff manual are expected to have a
+# working TeX installation.  Note that texi2dvi properly honours the
+# `MAKEINFO' environment variable.
 TEXI2DVI=texi2dvi
 
 groff_bin_dirs=\
@@ -101,9 +102,9 @@ imagedir=img
 .texinfo.txt:
        LANG= $(MAKEINFO) --enable-encoding -I$(srcdir) --plaintext -o $@ $<
 .texinfo.dvi:
-       MAKEINFO=$(MAKEINFO) $(TEXI2DVI) -e $<
+       LANG= MAKEINFO=$(MAKEINFO) $(TEXI2DVI) -e $<
 .texinfo.pdf:
-       MAKEINFO=$(MAKEINFO) $(TEXI2DVI) -e --pdf $<
+       LANG= MAKEINFO=$(MAKEINFO) $(TEXI2DVI) -e --pdf $<
 .texinfo.html:
        LANG= $(MAKEINFO) --enable-encoding -I$(srcdir) --html --no-split $< \
          && $(SHELL) $(srcdir)/fixinfo.sh $@