On Fri, Oct 25, 2024 at 10:14:45PM +0100, Gavin Smith wrote: > On Fri, Oct 25, 2024 at 10:37:51PM +0200, Bruno Haible wrote: > > On Solaris 11 OmniOS, there is 1 test failure: > > > > > > 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 > > Thanks for running the tests.
I found another clue: the extra "0" file is most likely a directory. I looked at the failing "action" at the GitHub page: https://github.com/gnu-texinfo/ci-check/actions/runs/10974176909/job/30472463303 With a Github account, I could search the log files. I searched for the string "EPUB/0" to see if there was any mention of this file: ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/ ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8.1 ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8.2 ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/ ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/mimetype ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/ ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/ose_utf8.opf ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/0/ ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/images/ ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/images/1-an_image.png ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/images/2-an_image.png ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/xhtml/ ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/xhtml/Chapteur.xhtml ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/xhtml/nav_toc.xhtml ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/EPUB/xhtml/osé_utf8.xhtml ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/META-INF/ ci-check/ci-check/texinfo-7.1.90-20240921/tp/tests/encoded/res_parser/non_ascii_test_epub/osé_utf8_epub_package/META-INF/container.xml (lines 16171-16187) Note the trailing / after the 0, indicating it is a directory. So the directory actually exists and is not some error in the way 'diff' is run. This file list appears to be printed in the log after the tests run, so this directory may not have been present before the tests run. The same test was present in the texinfo-7.1.0.90 pretest, and this actually passed on this platform on Github: https://github.com/gnu-texinfo/ci-check/actions/runs/10973874934/job/30471839045 However, I have found no changes in ext/epub3.pm or tests/run_parser_all.pm that could explain the creation of this directory, this deep under res_parser (which shouldn't be touched by the tests running), in this specific case. (No other 0 directories appear in the log.)
