Hello Daniel, On Wed, May 15, 2013 at 10:38:36AM +0200, Carsten Schoenert wrote: > 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
Christoph pushed yesterday the latest changes and the builder start running. Unfortunately now we get "just" a segfault on Sparc and IA64 without any hints. Mipsel fails on the known Page-Size error because the latest patch to fix this is just for IA64 and Sparc. I appended a git patch to include Mipsel into that fixes. If you have any ideas or hardware there you can debug the problem thean any help would be appreciated. Regards Carsten
>From cecd0347488ec1093fedc45fbcc44d92535eba90 Mon Sep 17 00:00:00 2001 From: Carsten Schoenert <c.schoen...@t-online.de> Date: Mon, 20 May 2013 09:59:48 +0200 Subject: [PATCH] Fix build error for Mipsel too The Build of Mipsel fails because the macro MALLOC_STATIC_SIZES is defined and jemalloc use wrong alignments for Mipsel. Signed-off-by: Carsten Schoenert <c.schoen...@t-online.de> --- ...ualenv-changing-the-path-to-virtualenv.py.patch | 1 + ...Don-t-hardcode-page-size-on-ia64-or-sparc.patch | 35 -------------------- ...ardcode-page-size-on-ia64-sparc-or-mipsel.patch | 35 ++++++++++++++++++++ debian/patches/series | 2 +- 4 files changed, 37 insertions(+), 36 deletions(-) delete mode 100644 debian/patches/porting/Don-t-hardcode-page-size-on-ia64-or-sparc.patch create mode 100644 debian/patches/porting/Don-t-hardcode-page-size-on-ia64-sparc-or-mipsel.patch diff --git a/debian/patches/debian-hacks/virtualenv-changing-the-path-to-virtualenv.py.patch b/debian/patches/debian-hacks/virtualenv-changing-the-path-to-virtualenv.py.patch index 28115ab..f8c7ff0 100644 --- a/debian/patches/debian-hacks/virtualenv-changing-the-path-to-virtualenv.py.patch +++ b/debian/patches/debian-hacks/virtualenv-changing-the-path-to-virtualenv.py.patch @@ -6,6 +6,7 @@ Since Debian changes the structure of python-2.7 in Jessie we have to fix the configure scripts to let them find the virtualenv.py. The patch based on Guidos work for the Iceowl package because the change for the virtualenv.py is needed there too. + --- mozilla/configure | 2 +- mozilla/configure.in | 2 +- diff --git a/debian/patches/porting/Don-t-hardcode-page-size-on-ia64-or-sparc.patch b/debian/patches/porting/Don-t-hardcode-page-size-on-ia64-or-sparc.patch deleted file mode 100644 index 3cd3249..0000000 --- a/debian/patches/porting/Don-t-hardcode-page-size-on-ia64-or-sparc.patch +++ /dev/null @@ -1,35 +0,0 @@ -From: Mike Hommey <m...@glandium.org> -Date: Fri, 23 Mar 2012 09:57:04 +0100 -Subject: Don't hardcode page size on ia64 or sparc - -This patch is from the Iceweasel git repository, some words for -explaining. -On IA64 and Sparc the behavior of the malloc() with defined -MALLOC_STATIC_SIZES produces a error while jmalloc is running: - - ... - adding: hyphenation/hyph_uk.dic (deflated 70%) - <jemalloc>Compile-time page size does not divide the runtime one. - -So let just MALLOC_STATIC_SIZES undefined for IA64 and Sparc. - -Signed-off-by: Carsten Schoenert <c.schoen...@t-online.de> - ---- - mozilla/memory/mozjemalloc/jemalloc.c | 2 ++ - 1 file changed, 2 insertions(+) - -diff --git a/mozilla/memory/mozjemalloc/jemalloc.c b/mozilla/memory/mozjemalloc/jemalloc.c -index 631e736..ba78597 100644 ---- a/mozilla/memory/mozjemalloc/jemalloc.c -+++ b/mozilla/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 - diff --git a/debian/patches/porting/Don-t-hardcode-page-size-on-ia64-sparc-or-mipsel.patch b/debian/patches/porting/Don-t-hardcode-page-size-on-ia64-sparc-or-mipsel.patch new file mode 100644 index 0000000..d1e9334 --- /dev/null +++ b/debian/patches/porting/Don-t-hardcode-page-size-on-ia64-sparc-or-mipsel.patch @@ -0,0 +1,35 @@ +From: Mike Hommey <m...@glandium.org> +Date: Fri, 23 Mar 2012 09:57:04 +0100 +Subject: Don't hardcode page size on ia64, sparc or mipsel + +This patch is from the Iceweasel git repository, some words for +explaining. +On IA64, Sparc and Mipsel the behavior of the malloc() with defined +MALLOC_STATIC_SIZES produces a error while jemalloc is running: + + ... + adding: hyphenation/hyph_uk.dic (deflated 70%) + <jemalloc>Compile-time page size does not divide the runtime one. + +So let just MALLOC_STATIC_SIZES undefined for IA64, Sparc and Mipsel. + +Signed-off-by: Carsten Schoenert <c.schoen...@t-online.de> + +--- + mozilla/memory/mozjemalloc/jemalloc.c | 2 ++ + 1 file changed, 2 insertions(+) + +diff --git a/mozilla/memory/mozjemalloc/jemalloc.c b/mozilla/memory/mozjemalloc/jemalloc.c +index 631e736..a07954b 100644 +--- a/mozilla/memory/mozjemalloc/jemalloc.c ++++ b/mozilla/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__) && !defined(__mipsel__) + #define MALLOC_STATIC_SIZES 1 ++#endif + + #ifdef MALLOC_STATIC_SIZES + diff --git a/debian/patches/series b/debian/patches/series index 3147756..b8ba59a 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -32,5 +32,5 @@ debian-hacks/Fix-build-failure-for-header.py-and-typelib.py.patch porting/Another-fix-to-build-ipc-code-on-GNU-hurd-and-kfreeb.patch porting/Bug-814693-Allow-webrtc-to-build-on-more-architectur.patch fixes/fix-function-nsMsgComposeAndSend-to-to-respect-Replo.patch -porting/Don-t-hardcode-page-size-on-ia64-or-sparc.patch +porting/Don-t-hardcode-page-size-on-ia64-sparc-or-mipsel.patch debian-hacks/virtualenv-changing-the-path-to-virtualenv.py.patch -- 1.7.10.4
