Re: [Debian-l10n-devel] https://i18n.debian.org/material/data/unstable.gz not updated since 2017-11-23 - too much packages to add to the ignore_list

[Nicolas François <nekral.lists@gmail.com>]

Hi Laura,

2018-04-17 16:39 GMT-07:00 Laura Arjona Reina <larjona@debian.org>:

> Unable to open /srv/mirrors/debian//pool/main/g/gcc-5/gcc-5_5.5.0.orig.tar.gz at /srv/i18n.debian.org//dl10n/git/dl10n-check line 463.
> read() on closed filehandle GEN3287 at /srv/i18n.debian.org//dl10n/git/lib/Debian/Pkg/Tar.pm line 176.
> Failed to read `/srv/mirrors/debian//pool/main/g/gcc-5/gcc-5_5.5.0.orig.tar.gz': Bad file descriptor at /srv/i18n.debian.org//dl10n/git/dl10n-check line 463.
> 'x' outside of string in unpack at /srv/i18n.debian.org//dl10n/git/lib/Debian/Pkg/Tar.pm line 368.

> I am sorry but I'm giving up adding packages as exception, we have a
> long list already and we're only in letter "g".
> 
> I've had a look at the code but frankly I have no idea about how to

> solve this problem :(

Can you try adding

    $text = undef;

after the line 178 in https://salsa.debian.org/l10n-team/dl10n/blob/master/lib/Debian/Pkg/Tar.pm

You will still have the error messages (Unable to open..., read() on closed filehandle..., Failed to read...) but I hope this will avoid the crash ('x' outside of string in unpack...)

(sorry, I could not test).

Note: The tools were designed at a time when CPU and disk were limited (there might have been more diversity in the packaging formats). This may not be the case now, and if somebody is interested in a (major) rewrite, it might be easier to just extract each source packages using dpkg-source (+ heuristic to apply patches) than to do this in-memory extract and patch mechanism.

Regards,

-- 

Nekral