Re: [Groff] release ready

[Axel Kielhorn]

Am 12.10.2014 um 13:10 schrieb Werner LEMBERG <address@hidden>:

> 
>> I had a problem with make install (again illegal byte sequence).
>> As a result I had an incomplete installation.
>> This was fixed with LANG=C.
> 
> Uh, oh, this is not enough information.  Please show us the *exact*
> error message.

During make:

sed -e "s;@VERSION@;1.22.3;" webpage.ms | GROFF_COMMAND_PREFIX= 
GROFF_BIN_PATH="`echo /Users/axel/src/groff/src/roff/groff 
/Users/axel/src/groff/src/roff/troff /Users/axel/src/groff/src/preproc/preconv 
/Users/axel/src/groff/src/preproc/pic /Users/axel/src/groff/src/preproc/eqn 
/Users/axel/src/groff/src/preproc/tbl /Users/axel/src/groff/src/preproc/grn 
/Users/axel/src/groff/src/preproc/refer 
/Users/axel/src/groff/src/preproc/soelim /Users/axel/src/groff/src/preproc/html 
/Users/axel/src/groff/src/devices/grops 
/Users/axel/src/groff/src/devices/gropdf 
/Users/axel/src/groff/src/devices/grohtml | sed -e 's|  *|:|g'`" 
/Users/axel/src/groff/src/roff/groff/groff -M/Users/axel/src/groff/tmac 
-M/Users/axel/src/groff/tmac -F/Users/axel/src/groff/font 
-F/Users/axel/src/groff/font -Upet -ww -Tps -ms -mwww >webpage.ps
sed: RE error: illegal byte sequence

sed -e "s;@VERSION@;1.22.3;" webpage.ms | GROFF_COMMAND_PREFIX= 
GROFF_BIN_PATH="`echo /Users/axel/src/groff/src/roff/groff 
/Users/axel/src/groff/src/roff/troff /Users/axel/src/groff/src/preproc/preconv 
/Users/axel/src/groff/src/preproc/pic /Users/axel/src/groff/src/preproc/eqn 
/Users/axel/src/groff/src/preproc/tbl /Users/axel/src/groff/src/preproc/grn 
/Users/axel/src/groff/src/preproc/refer 
/Users/axel/src/groff/src/preproc/soelim /Users/axel/src/groff/src/preproc/html 
/Users/axel/src/groff/src/devices/grops 
/Users/axel/src/groff/src/devices/gropdf 
/Users/axel/src/groff/src/devices/grohtml | sed -e 's|  *|:|g'`" 
/Users/axel/src/groff/src/roff/groff/groff -M/Users/axel/src/groff/tmac 
-M/Users/axel/src/groff/tmac -F/Users/axel/src/groff/font 
-F/Users/axel/src/groff/font -Upet -ww -P-j`basename webpage.ms | sed -e 
's|.ms$||'` \
                 -P-nrpb -P-I`basename webpage.ms | sed -e 's|.ms$||'` \
                 -P-Dimg -Thtml -ms >webpage.html
sed: RE error: illegal byte sequence

and during make install:

/usr/bin/install -c -m 644 ./m.tmac \
          /usr/local/share/groff/1.22.3/tmac/m.tmac
sed: RE error: illegal byte sequence
make[2]: *** [install_m] Error 1
make[1]: *** [contrib/mm] Error 2
make: *** [install] Error 2

As a result many files are missing in 
/usr/local/share/groff/1.22.3/tmac
(Only 4 files are installed.)


> 
>> download in devpdf really needs tabs, using space as a separator
>> does not work.  (And having a trailing space after the file name is
>> not a good idea.)
> 
> You mean `font/devpdf/download', right?  It behaves exactly as
> documented!  From the gropdf manpage:
> 
>  ... fields must be separated by tabs …

While the grops man page says:

      fields may be separated by tabs or spaces;

This is slightly inconsistent.
(Of course you can argue that these are different programs 
and expecting them to use the same syntax is naïve. I wouldn't disagree.)

> Deri can certainly give more details why spaces are not allowed,
> contrary to grops.

Axel