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

Bug#949252: mat2: autopkgtest needs update for new version of libimage-exiftool-perl



Source: mat2
Version: 0.10.0-1
Severity: serious
Tags: sid bullseye
User: debian-ci@lists.debian.org
Usertags: needs-update
Control: affects -1 src:libimage-exiftool-perl

[X-Debbugs-CC: debian-ci@lists.debian.org,
libimage-exiftool-perl@packages.debian.org]

Dear maintainers,

With a recent upload of libimage-exiftool-perl the autopkgtest of mat2
fails in testing when that autopkgtest is run with the binary packages
of libimage-exiftool-perl from unstable. It passes when run with only
packages from testing. In tabular form:
                        pass            fail
libimage-exiftool-perl  from testing    11.84-1
mat2                    from testing    0.10.0-1
all others              from testing    from testing

I copied some of the output at the bottom of this report.

Currently this regression is blocking the migration of
libimage-exiftool-perl to testing [1]. Of course, libimage-exiftool-perl
shouldn't just break your autopkgtest (or even worse, your package), but
it seems to me that the change in libimage-exiftool-perl was intended
(confirmed by gregoa on IRC) and your package needs to update to the new
situation.

If this is a real problem in your package (and not only in your
autopkgtest), the right binary package(s) from libimage-exiftool-perl
should really add a versioned Breaks on the unfixed version of (one of
your) package(s). Note: the Breaks is nice even if the issue is only in
the autopkgtest as it helps the migration software to figure out the
right versions to combine in the tests.

More information about this bug and the reason for filing it can be found on
https://wiki.debian.org/ContinuousIntegration/RegressionEmailInformation

Paul

[1] https://qa.debian.org/excuses.php?package=libimage-exiftool-perl

https://ci.debian.net/data/autopkgtest/testing/amd64/m/mat2/4024569/log.gz

=================================== FAILURES
===================================
_______________________ TestCleaning.test_all_parametred
_______________________

self = <tests.test_libmat2.TestCleaning testMethod=test_all_parametred>

    def test_all_parametred(self):
        for case in self.data:
            if 'ffmpeg' in case:
                try:
                    video._get_ffmpeg_path()
                except RuntimeError:
                    raise unittest.SkipTest

            print('[+] Testing %s' % case['name'])
            target = './tests/data/clean.' + case['name']
            shutil.copy('./tests/data/dirty.' + case['name'], target)
            p1 = case['parser'](target)

            meta = p1.get_meta()
            for k, v in case['meta'].items():
                if isinstance(v, dict):
                    for _k, _v in v.items():
                        self.assertEqual(meta[k][_k], _v)
                else:
                    self.assertEqual(meta[k], v)

            p1.lightweight_cleaning = True
            self.assertTrue(p1.remove_all())

            p2 = case['parser'](p1.output_filename)
>           self.assertEqual(p2.get_meta(), case['expected_meta'])
E           AssertionError: {'MajorBrand': 'MP4  Base Media v1 [IS0
144[457 chars]': 0} != {'CompatibleBrands': ['isom', 'iso2', 'avc1[432
chars]': 0}
E             {'CompatibleBrands': ['isom', 'iso2', 'avc1', 'mp41'],
E              'CompressorID': 'avc1',
E              'GraphicsMode': 'srcCopy',
E              'HandlerDescription': 'SoundHandler',
E              'HandlerType': 'Metadata',
E              'HandlerVendorID': 'Apple',
E              'MajorBrand': 'MP4  Base Media v1 [IS0 14496-12:2003]',
E           -  'MediaDataOffset': 48,
E           -  'MediaDataSize': 379872,
E              'MediaHeaderVersion': 0,
E              'MinorVersion': '0.2.0',
E           +  'MovieDataOffset': 48,
E              'MovieHeaderVersion': 0,
E              'NextTrackID': 3,
E              'PreferredRate': 1,
E              'Rotation': 0,
E              'TimeScale': 1000,
E              'TrackHeaderVersion': 0,
E              'TrackID': 1,
E              'TrackLayer': 0}

Attachment: signature.asc
Description: OpenPGP digital signature


Reply to: