Cyril Brulebois <kibi@debian.org> (2013-09-30): > Yes; debian-i18n membership ping. > > Can't see the page on alioth due to: > Permission denied. This project's administrator will have to grant you permission to view this page. > > so can't request it myself. You should be able to add me from your end > though. Nevermind, that probably should go through rt.d.o (and it did). Anyway, running the script against "unstable", with the first patch attached for debugging purposes, shows the package before 3depict is 0ad-data, which leads to an exec of "xd" which allocates a lot of memory. I suspect memory issue handling is poor, and leads to the nasty side effect we saw: an unrelated package gets blamed. The second (quick and dirty) patch seems to work around that, and the script is still running for now. I guess fixing error handling and maybe getting some more memory should be enough to get that part to work reliably. I think I'll drop -devel@ from any further replies since the call for help was answered, and people know where to follow the rest, should they be interested. Mraw, KiBi.
From 94ea088fc1de135abe80e19eb11151ae95bba4c3 Mon Sep 17 00:00:00 2001
From: Cyril Brulebois <kibi@debian.org>
Date: Mon, 30 Sep 2013 22:17:39 +0000
Subject: [PATCH 1/2] dl10n-check: mention parsed $path,$pkg in parse_tarball
---
dl10n-check | 2 ++
1 file changed, 2 insertions(+)
diff --git a/dl10n-check b/dl10n-check
index 4bad7a1..af38535 100755
--- a/dl10n-check
+++ b/dl10n-check
@@ -414,6 +414,8 @@ sub parse_tarball {
$data->version($pkg, shift);
$data->maintainer($pkg, shift);
$data->upstream($pkg, "debian");
+
+ print STDERR "Going to parse $path / $pkg\n";
# Debian::Pkg::DebSrc->new() seem to have bad time when no / is in there.
# A broken basename somewhere? FIXME properly
--
1.7.10.4
From ceabda4c9a2910a5f1b8768b9dc34db86a810b36 Mon Sep 17 00:00:00 2001
From: Cyril Brulebois <kibi@debian.org>
Date: Mon, 30 Sep 2013 22:19:24 +0000
Subject: [PATCH 2/2] dl10n-check: exclude 0ad-data, due to possible memory
shortage and bad error handling
---
dl10n-check | 3 +++
1 file changed, 3 insertions(+)
diff --git a/dl10n-check b/dl10n-check
index af38535..ae26261 100755
--- a/dl10n-check
+++ b/dl10n-check
@@ -462,6 +462,9 @@ sub parse_tarball {
return -1 if &$match($file);
return 0;
};
+ # Avoid ENOMEM or similar?
+ next
+ if $path =~ /0ad-data/;
my $deb = Debian::Pkg::DebSrc->new($path,
parse_dft => $match,
patch_parse_dft => $match_patch,
--
1.7.10.4
Attachment:
signature.asc
Description: Digital signature