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

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

|--==> Jurij Smakov writes:

  JS> Hello,
  JS> On Thu, 9 Dec 2004, Free Ekanayaka wrote:

  >>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 oooops!

Thanks for your patch. I've modified the package accordingly:


I'll ask to my usual uploader to upload it, but if somebody is willing
to do it please just contact me.



  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 +0000
  JS> +++ /tmp/dpep.9PBbmd/brutefir-1.0/Makefile	2004-12-10 02:31:29.741552488 +0000
  JS> @@ -73,6 +73,9 @@
  JS>   ifeq ($(UNAME_M),i686)
  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                                        jurij@wooyd.org
  JS> Key: http://www.wooyd.org/pgpkey/                   KeyID: C99E03CC

Reply to: