Re: Bug#269715: brutefir ftbfs [sparc] Requires v9|v9a|v9b
|--==> Jurij Smakov writes: JS> Hello, JS> 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: JS> [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? JS> The solution you've proposed fails due to at least three different JS> reasons: JS> * The changes to the CC_FLAGS which the patch implements are inside the JS> ifeq ($(UNAME),SunOS) ... endif block, so they would never be triggered on JS> Debian. JS> * The attempt to determine the architecture is done by looking at the JS> result of 'uname -p' command. On Debian uname does not support a '-p' JS> option. JS> * Since you have forgot to create the 00list or 00list.sparc file in the JS> debian/patches directory, the patch you describe does not even get applied JS> during the build :-) Big ps! Thanks for your patch. I've modified the package accordingly: http://people.miu-ft.org/~free/packages I'll ask to my usual uploader to upload it, but if somebody is willing to do it please just contact me. Cheers, Free JS> The corrected patch looks like this: JS> --snip JS> diff -urNad brutefir-1.0/Makefile /tmp/dpep.9PBbmd/brutefir-1.0/Makefile JS> --- brutefir-1.0/Makefile 2004-12-10 02:23:34.545793224 + JS> +++ /tmp/dpep.9PBbmd/brutefir-1.0/Makefile2004-12-10 02:31:29.741552488 + JS> @@ -73,6 +73,9 @@ JS> ifeq ($(UNAME_M),i686) JS> BRUTEFIR_OBJS += $(BRUTEFIR_IA32_OBJS) JS> endif JS> +ifneq (,$(findstring sparc,$(UNAME_M))) JS> +CC_FLAGS += -Wa,-xarch=v8plus JS> +endif JS> BRUTEFIR_LIBS += -ldl JS> LDMULTIPLEDEFS = -Xlinker --allow-multiple-definition JS> # assume that we have alsa, osss and jack JS> --snip- JS> You can find the corresponding dpatch scriplet at [0]. In order to get JS> it applied, you need to create the debian/patches/00list.sparc file, which JS> is also given there (by the way, if you want the amd64 patch to apply as JS> well, creating debian/patch/00list.amd64 is a good idea). With these JS> changes the package builds fine in a sid chroot, build log and the JS> sparc deb are available at [0] as well. Unfortunately, I cannot test JS> the package, since I do not have sound configured. JS> [0] http://www.wooyd.org/debian/brutefir/ JS> Best regards, JS> Jurij Smakov[EMAIL PROTECTED] JS> Key: http://www.wooyd.org/pgpkey/ KeyID: C99E03CC
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 + +++ /tmp/dpep.9PBbmd/brutefir-1.0/Makefile 2004-12-10 02:31:29.741552488 + @@ -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[EMAIL PROTECTED] Key: http://www.wooyd.org/pgpkey/ KeyID: C99E03CC
brutefir ftbfs [sparc] Requires v9|v9a|v9b
Hi all, sorry for the late reply. With regards to: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=269715 I've applied the following patch to the Makefile: diff -urNad brutefir-1.0/Makefile /tmp/dpep.rSAnEB/brutefir-1.0/Makefile --- brutefir-1.0/Makefile 2004-10-30 13:30:15.0 +0200 +++ /tmp/dpep.rSAnEB/brutefir-1.0/Makefile 2004-12-09 10:48:47.0 +0100 @@ -86,11 +86,13 @@ CC_WARN= -Wall -Wlong-long -Wpointer-arith -Wshadow \ -Wcast-qual -Wwrite-strings -Wmissing-prototypes -Wmissing-declarations \ -Wnested-externs +BRUTEFIR_LIBS += -lrt -ldl +endif + +# Sparc ifeq ($(UNAME_P),sparc) CC_FLAGS += -Wa,-xarch=v8plus endif -BRUTEFIR_LIBS += -lrt -ldl -endif # FreeBSD ifeq ($(UNAME),FreeBSD) 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? You can find the source at: http://people.miu-ft.org/~free/packages/ Thanks, Free
