Hello Daniel, thanks for investigating time around the builderrors.
On Wed, May 15, 2013 at 03:19:19AM -0400, Daniel Kahn Gillmor wrote: > Package: icedove > Version: 17.0.5-1 > Severity: normal > > looking at, for example, the armel build log > > > https://buildd.debian.org/status/fetch.php?pkg=icedove&arch=armel&ver=17.0.5-1&stamp=1366372226 > > i see that icedove compilation fails during the creation of a zip file > that will eventually be placed in /usr/share/icedove/omni.ja > [...] > 0 dkg@alice:/tmp/cdtemp.c5tgG7$ diff -ruN icedove-17.0.5/mozilla/memory > iceweasel-17.0.5esr/memory > diff -ruN icedove-17.0.5/mozilla/memory/mozjemalloc/jemalloc.c > iceweasel-17.0.5esr/memory/mozjemalloc/jemalloc.c > --- icedove-17.0.5/mozilla/memory/mozjemalloc/jemalloc.c 2013-03-28 > 14:09:00.000000000 -0400 > +++ iceweasel-17.0.5esr/memory/mozjemalloc/jemalloc.c 2013-05-15 > 03:12:22.000000000 -0400 > @@ -1091,7 +1091,9 @@ > * controlling the malloc behavior are defined as compile-time constants > * for best performance and cannot be altered at runtime. > */ > +#if !defined(__ia64__) && !defined(__sparc__) > #define MALLOC_STATIC_SIZES 1 > +#endif > > #ifdef MALLOC_STATIC_SIZES > [...] > and iceweasel contains the following patch that icedove does not: > > 0 dkg@alice:/tmp/cdtemp.c5tgG7$ cat > iceweasel-17.0.5esr/debian/patches/porting/Don-t-hardcode-page-size-on-ia64-or-sparc.patch > From: Mike Hommey <[email protected]> > Date: Fri, 23 Mar 2012 09:57:04 +0100 > Subject: Don't hardcode page size on ia64 or sparc > > --- > memory/mozjemalloc/jemalloc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/memory/mozjemalloc/jemalloc.c b/memory/mozjemalloc/jemalloc.c > index 631e736..ba78597 100644 > --- a/memory/mozjemalloc/jemalloc.c > +++ b/memory/mozjemalloc/jemalloc.c > @@ -1091,7 +1091,9 @@ static unsigned ncpus; > * controlling the malloc behavior are defined as compile-time constants > * for best performance and cannot be altered at runtime. > */ > +#if !defined(__ia64__) && !defined(__sparc__) > #define MALLOC_STATIC_SIZES 1 > +#endif > > #ifdef MALLOC_STATIC_SIZES > > 0 dkg@alice:/tmp/cdtemp.c5tgG7$ > > that said, iceweasel 17.0.5esr-1 doesn't appear to build for these > architectures either, showing segfaults in the same places: > > https://buildd.debian.org/status/package.php?p=iceweasel&suite=sid > > Maybe a porter will have some insight? > > Regards, > > --dkg we ported the patch from Mike some days ago [1] and Christoph has prepared a new version of Icedove 17.0.5-2 localy. Yesterday the version 17.0.6 was released by Mozilla, so propably the skip the Debian Package for 17.0.5-2 and Christoph will push the new released 17.0.6 in a few days. With this new version we will take a deeper look at this builderrors on a Sparc machine for example. [1] http://anonscm.debian.org/gitweb/?p=pkg-mozilla/icedove.git;a=commitdiff;h=40d68d56f663f3159bda67231a70488f021442fa Regards Carsten -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
