libcdio-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Libcdio-devel] Rock Ridge deep directory support


From: Pete Batard
Subject: Re: [Libcdio-devel] Rock Ridge deep directory support
Date: Thu, 18 Jun 2020 09:22:43 +0100
User-agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.9.0

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

[Prev in Thread] Current Thread [Next in Thread]