Hi Rocky,

On 2020.06.17 20:08, Rocky Bernstein wrote:
Thanks for doing the work and for merging.

I just ran "make distcheck" (building outside of source directory from a
tarball) and I get a couple of failures:

...
FAIL: check_multiextent.sh
PASS: check_fuzzyiso.sh
PASS: check_opts.sh
FAIL: check_deep_directory.sh

I'm pretty sure this is because we don't have a .in to generate the .sh and set srcdir and top_builddir to an absolute path, as is done in other tests.

I considered doing it, but then I found that check_fuzzyiso.sh wasn't, and since I didn't see errors from using relative paths during my testing, I thought we would be fine.

I'll look into generating the .sh from a .sh.in when I get a chance.

 From check_deep_directory.sh.log:

-- ../../../test/check_deep_directory.sh: Deep directory listing test ok.
/usr/bin/diff: ../../../test/data/deep_directory1_file: No such file or
directory
../../../test/check_deep_directory.sh: Deep directory read test level 8
failed.
../../../test/check_deep_directory.sh: failed command:
../src/iso-info ../src/iso-read --ignore --image
../../../test/data/deep-directory.iso --extract 1/2/3/4/5/6/7/8/8.txt
FAIL check_deep_directory.sh (exit status: 3)

And from check_multiextent.sh.sh:

-- ../../../test/check_multiextent.sh: Multiextent listing test ok.
/usr/bin/diff: ../../../test/data/multi_extent_file: No such file or
directory
../../../test/check_multiextent.sh: Multiextent read test failed.
../../../test/check_multiextent.sh: failed command:
../src/iso-info ../src/iso-read --ignore --image
../../../test/data/multi_extent_8k.iso --extract multi_extent_file
FAIL check_multiextent.sh (exit status: 3)

I think those new files mentioned need to be added to the check_DATA variable
test/data/Makefile.am.

They are already there.

Regards,

/Pete

Reply via email to