[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: brutefir ftbfs [sparc] Requires v9|v9a|v9b



Hello,

On Thu, 9 Dec 2004, Free Ekanayaka wrote:

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=269715

I've applied the following patch to the Makefile:
[patch skipped]
Before uploading the package I'd like to  test it, but I've not access
to any sparc machine.

Would somebody kindly try to build the package?

The solution you've proposed fails due to at least three different reasons:

* The changes to the CC_FLAGS which the patch implements are inside the
ifeq ($(UNAME),SunOS) ... endif block, so they would never be triggered on Debian.

* The attempt to determine the architecture is done by looking at the result of 'uname -p' command. On Debian uname does not support a '-p' option.

* Since you have forgot to create the 00list or 00list.sparc file in the debian/patches directory, the patch you describe does not even get applied during the build :-)

The corrected patch looks like this:
--snip--------------------------------------------------------------------
diff -urNad brutefir-1.0/Makefile /tmp/dpep.9PBbmd/brutefir-1.0/Makefile
--- brutefir-1.0/Makefile	2004-12-10 02:23:34.545793224 +0000
+++ /tmp/dpep.9PBbmd/brutefir-1.0/Makefile	2004-12-10 02:31:29.741552488 +0000
@@ -73,6 +73,9 @@
 ifeq ($(UNAME_M),i686)
 BRUTEFIR_OBJS	+= $(BRUTEFIR_IA32_OBJS)
 endif
+ifneq (,$(findstring sparc,$(UNAME_M)))
+CC_FLAGS += -Wa,-xarch=v8plus
+endif
 BRUTEFIR_LIBS	+= -ldl
 LDMULTIPLEDEFS	= -Xlinker --allow-multiple-definition
 # assume that we have alsa, osss and jack
--snip---------------------------------------------------------------------

You can find the corresponding dpatch scriplet at [0]. In order to get it applied, you need to create the debian/patches/00list.sparc file, which is also given there (by the way, if you want the amd64 patch to apply as well, creating debian/patch/00list.amd64 is a good idea). With these changes the package builds fine in a sid chroot, build log and the sparc deb are available at [0] as well. Unfortunately, I cannot test the package, since I do not have sound configured.

[0] http://www.wooyd.org/debian/brutefir/

Best regards,

Jurij Smakov                                        jurij@wooyd.org
Key: http://www.wooyd.org/pgpkey/                   KeyID: C99E03CC



Reply to: