Re: [Groff] new automake system

[Bertrand Garrigues]

Hello Bernd,

Thanks for testing!

On Mon, Sep 22 2014 at 01:23:56 AM, "Bernd Warken" <address@hidden> wrote:
>> Von: "Bertrand Garrigues" <address@hidden>
>> Then you can check in the TESTS file for the other targets.  Please note
>> that compared to the current build system:
>>   - There are 2 new targets, 'make check' and 'make distcheck'
>>   - Parallel build is supported.
>  
> With both checks I got an error:
>  
> 1) make check
> make[3]: Entering directory '/opt/development/groff/git/automake/groff/build'
> FAIL: contrib/gdiffmk/tests/gdiffmk_tests.sh
> make[4]: Entering directory '/opt/development/groff/git/automake/groff/build'
> make  all-am
> make[5]: Entering directory '/opt/development/groff/git/automake/groff/build'
> make[5]: Leaving directory '/opt/development/groff/git/automake/groff/build'
> make[4]: Leaving directory '/opt/development/groff/git/automake/groff/build'

- Could you please send me the content of build/contrib/gdiffmk/tests ?

- What happens if you call manually gdiffmk_tests.sh ? For example like
  this:
  
  $ cd build
  $ rm contrib/gdiffmk/tests/*
  $ abs_top_builddir=/opt/development/groff/git/automake/groff/build \
    abs_top_srcdir=/opt/development/groff/git/automake/groff/ \
    ../contrib/gdiffmk/tests/gdiffmk_tests.sh

> 2) make distcheck
> /usr/bin/texi2dvi: etex exited with bad status, quitting.
> Makefile:9257: recipe for target 'doc/groff.dvi' failed
> make[1]: *** [doc/groff.dvi] Error 1
> make[1]: Leaving directory 
> '/opt/development/groff/git/automake/groff/build/groff-1.22.2/_build'
> Makefile:7989: recipe for target 'distcheck' failed
> make: *** [distcheck] Error 1

What version of texinfo are you using ? I reproduced this problem with
texinfo 5.2. The problem is that the macro @LE{} works correctly with
texinfo 4.13, but not texinfo 5.2. I have the same problem on master,
the difference is that groff.pdf is not built by default, you have
manually build it:

  $ cd build/doc
  $ make groff.pdf

Here is a workaround (just avoid to use the LE macro), could you please
tell me if it works better ?

Regards,

diff --git a/doc/groff.texi b/doc/groff.texi
index 55a22da..8494951 100644
--- a/doc/groff.texi
+++ b/doc/groff.texi
@@ -16118,7 +16118,10 @@ The address@hidden@var{n} must be an integer in the 
range @math{-32767}
 to 32767.
 
 @table @asis
address@hidden @math{0 @LE{} @var{n} @LE{} 1000}
+
address@hidden FIXME: the @LE macro fails with texinfo 5.2
address@hidden @item @math{0 @LE{} @var{n} @LE{} 1000}
address@hidden @math{0 <= @var{n} <= 1000}
 Set the color for filling solid drawing objects to a shade of gray,
 where 0 corresponds to solid white, 1000 (the default) to solid black,
 and values in between to intermediate shades of gray; this is obsoleted

--
Bertrand Garrigues