Re: Texinfo master branch [Solaris 11 OmniOS]

[Bruno Haible]

Patrice Dumas wrote:
> > 4) The Solaris 11 OmniOS build fails due to a test failure:
> > 
> > FAIL: test_scripts/encoded_non_ascii_test_epub.sh
> > 
> > Detailed log:
> > -------------------------------------------------------------------------------
> > 
> > FAIL: test_scripts/encoded_non_ascii_test_epub.sh
> > =================================================
> > 
> > D: encoded/diffs/non_ascii_test_epub.diff (printed below)
> > Common subdirectories: 
> > ../../../tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package
> >  and encoded/out_parser/non_ascii_test_epub/osé_utf8_epub_package
> > Common subdirectories: 
> > ../../../tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB
> >  and encoded/out_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB
> > Common subdirectories: 
> > ../../../tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/META-INF
> >  and encoded/out_parser/non_ascii_test_epub/osé_utf8_epub_package/META-INF
> > Only in 
> > ../../../tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB:
> >  0
> > Common subdirectories: 
> > ../../../tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/images
> >  and 
> > encoded/out_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/images
> > Common subdirectories: 
> > ../../../tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/xhtml
> >  and encoded/out_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/xhtml
> > D: encoded/diffs/non_ascii_test_epub.diff (printed above)
> > testdir: encoded
> > driving_file: ../../../tp/tests/encoded/list-of-tests
> > made result dir: ../../../tp/tests/encoded/res_parser/
> > 
> > doing test non_ascii_test_epub, src_file 
> > ../../../tp/tests/encoded/osé_utf8.texi
> > format_option: 
> > texi2any.pl non_ascii_test_epub -> encoded/out_parser/non_ascii_test_epub
> >  /usr/bin/perl -w ../../../tp/tests/..//texi2any.pl  --force --conf-dir 
> > ../../../tp/tests/../t/init/ --conf-dir ../../../tp/tests/../init 
> > --conf-dir ../../../tp/tests/../ext -I ../../../tp/tests/encoded -I 
> > encoded/ -I ../../../tp/tests/ -I . -I built_input --error-limit=1000 -c 
> > TEST=1  --output encoded/out_parser/non_ascii_test_epub/ -D 
> > 'needcommandlineunicode Need command-line unicode' -c 
> > COMMAND_LINE_ENCODING=UTF-8 -c MESSAGE_ENCODING=UTF-8 -c 
> > OUTPUT_FILE_NAME_ENCODING=UTF-8 --init epub3.pm -c 
> > 'EPUB_CREATE_CONTAINER_FILE 0' ../../../tp/tests/encoded/osé_utf8.texi > 
> > encoded/out_parser/non_ascii_test_epub/osé_utf8.1 
> > 2>encoded/out_parser/non_ascii_test_epub/osé_utf8.2
> > 
> > all done, exiting with status 1
> > FAIL test_scripts/encoded_non_ascii_test_epub.sh (exit status: 1)
> > -------------------------------------------------------------------------------
> 
> This is very strange.  I do not see a file named 0 in the tarball in
> tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/
> 
> The texi2any call should not change anything in the res_parser/.....
> directories where the references for the test results are.  So my wild
> guess is that there is something in the tp/tests/run_parser_all.sh shell
> script driving the tests and comparisons with references that leads to
> this file with name "0" to be created in the
> tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/
> directory, probably something not portable.

It sounds like a bug in /bin/sh. And indeed, when I use bash to execute this
script, no problem any more. It's not the first /bin/sh bug on this system
that we encounter:
https://lists.gnu.org/archive/html/bug-gnulib/2016-01/msg00030.html

My workaround for this kind of bugs is to define the environment variable
  CONFIG_SHELL=/usr/bin/bash
in my environment.
But all invocations of the tp/tests/run_parser_all.sh script should then
run it through
  ${CONFIG_SHELL-/bin/sh} tp/tests/run_parser_all.sh
instead of
  tp/tests/run_parser_all.sh

Bruno