Your message dated Wed, 4 Jan 2012 23:07:10 +0100
with message-id <20120104220710.ga6...@jwilk.net>
and subject line Re: Bug#621778: stimfit: FTBFS on powerpc: Both BIG_ENDIAN and
LITTLE_ENDIAN defined
has caused the Debian Bug report #621778,
regarding stimfit: FTBFS on powerpc: Both BIG_ENDIAN and LITTLE_ENDIAN defined
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)
--
621778: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=621778
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: src:stimfit
Version: 0.10.13-1
Severity: serious
Tags: patch
The package FTBFS on powerpc [0]:
...
|checking for kernel... checking for cpu type... other
...
|config.status: executing libtool commands
|/usr/bin/make
|make[1]: Entering directory
`/build/buildd-stimfit_0.10.13-1-powerpc-sQOf40/stimfit-0.10.13'
|make[2]: Entering directory
`/build/buildd-stimfit_0.10.13-1-powerpc-sQOf40/stimfit-0.10.13'
|Making all in src
|make[3]: Entering directory
`/build/buildd-stimfit_0.10.13-1-powerpc-sQOf40/stimfit-0.10.13/src'
|Making all in app
|make[4]: Entering directory
`/build/buildd-stimfit_0.10.13-1-powerpc-sQOf40/stimfit-0.10.13/src/app'
| CXX channel.o
|In file included from /usr/include/endian.h:37,
| from /usr/include/wctype.h:61,
| from /usr/include/wx-2.8/wx/wxchar.h:521,
| from /usr/include/wx-2.8/wx/debug.h:22,
| from /usr/include/wx-2.8/wx/defs.h:521,
| from /usr/include/wx-2.8/wx/wx.h:15,
| from ./../core/channel.cpp:16:
|/usr/include/bits/endian.h:27:4: error: #error Both BIG_ENDIAN and
LITTLE_ENDIAN defined!
|make[4]: *** [libstimfit_la-channel.lo] Error 1
|make[4]: Leaving directory
`/build/buildd-stimfit_0.10.13-1-powerpc-sQOf40/stimfit-0.10.13/src/app'
After looking at configure.in I've found these:
| AC_MSG_CHECKING(for cpu type)
| case `uname -p` in
| powerpc)
| AC_MSG_RESULT(powerpc)
| CPPFLAGS="${CPPFLAGS} -D__BIG_ENDIAN__"
| ;;
| *)
| AC_MSG_RESULT(other)
| CPPFLAGS="${CPPFLAGS} -D__LITTLE_ENDIAN__"
| ;;
| esac
This is bogus for a few reason. "uname -p" returns usually unknown :) The
intention was probably to use the -m option but then it should match
against ppc. Even that is wrong. The application should never try to define it
itself. Take a look on this:
|bigeasy@baako:~$ gcc -o endian endian.c
|bigeasy@baako:~$ ./endian
|BIG ENDIAN
|bigeasy@baako:~$ gcc -o endian endian.c -D__LITTLE_ENDIAN__
|In file included from /usr/include/endian.h:37,
| from endian.c:2:
|/usr/include/bits/endian.h:27:4: error: #error Both BIG_ENDIAN and
LITTLE_ENDIAN defined!
|bigeasy@baako:~$ cat endian.c
|#include <stdio.h>
|#include <endian.h>
|
|int main(void)
|{
| printf("%s\n",
|#ifdef __BIG_ENDIAN__
| "BIG ENDIAN"
|#endif
|#ifdef __LITTLE_ENDIAN__
| "LITTLE ENDIAN"
|#endif
| );
| return 0;
|}
Please find attach a patch. It seems to build now but I have no clue if
it works. A testsuite could show that :)
[0]
https://buildd.debian.org/status/fetch.php?pkg=stimfit&arch=powerpc&ver=0.10.13-1&stamp=1301922826
Sebastian
>From a3a6e3ba059e5229e85aa3d6361bff2b0f886740 Mon Sep 17 00:00:00 2001
From: Sebastian Andrzej Siewior <sebast...@breakpoint.cc>
Date: Fri, 8 Apr 2011 18:36:56 +0000
Subject: [PATCH] remove endian defines
Signed-off-by: Sebastian Andrzej Siewior <sebast...@breakpoint.cc>
---
configure | 16 ----------------
configure.in | 12 ------------
2 files changed, 0 insertions(+), 28 deletions(-)
diff --git a/configure b/configure
index bb933d5..b842a59 100755
--- a/configure
+++ b/configure
@@ -15158,22 +15158,6 @@ if test "$with_biosig" = "yes" ; then
LIBBIOSIG_LDFLAGS="-lbiosig"
fi
-
-{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for cpu type" >&5
-$as_echo_n "checking for cpu type... " >&6; }
-case `uname -p` in
- powerpc)
- { $as_echo "$as_me:${as_lineno-$LINENO}: result: powerpc" >&5
-$as_echo "powerpc" >&6; }
- CPPFLAGS="${CPPFLAGS} -D__BIG_ENDIAN__"
- ;;
- *)
- { $as_echo "$as_me:${as_lineno-$LINENO}: result: other" >&5
-$as_echo "other" >&6; }
- CPPFLAGS="${CPPFLAGS} -D__LITTLE_ENDIAN__"
- ;;
-esac
-
# GUI stuff not required for standalone module
if test "$enable_module" != "yes"; then
# Checks for libraries.
diff --git a/configure.in b/configure.in
index 6eca9a3..c62bd06 100644
--- a/configure.in
+++ b/configure.in
@@ -145,18 +145,6 @@ if test "$with_biosig" = "yes" ; then
fi
AC_SUBST(LIBBIOSIG_LDFLAGS)
-AC_MSG_CHECKING(for cpu type)
-case `uname -p` in
- powerpc)
- AC_MSG_RESULT(powerpc)
- CPPFLAGS="${CPPFLAGS} -D__BIG_ENDIAN__"
- ;;
- *)
- AC_MSG_RESULT(other)
- CPPFLAGS="${CPPFLAGS} -D__LITTLE_ENDIAN__"
- ;;
-esac
-
# GUI stuff not required for standalone module
if test "$enable_module" != "yes"; then
# Checks for libraries.
--
1.7.4.1
--- End Message ---
--- Begin Message ---
Version: 0.10.18-1
* Yaroslav Halchenko <deb...@onerussian.com>, 2011-09-13, 08:40:
I think we can close it this way ;-)
Let's try again. :)
--
Jakub Wilk
--- End Message ---
