Bug#593700: The package has a generic C version of this function

Chris Butler chrisb at debian.org
Sat Jan 1 16:53:42 UTC 2011


On Mon, Nov 15, 2010 at 09:29:44PM -0500, Asheesh Laroia wrote:
> Take a look at line 35 of source/snd_qf/snd_mix.c. There is (what
> appears to be) a portable C version of the
> S_WriteLinearBlastStereo16 function.
> 
> Now to take a look at what guides the choices at compile time....

This seems to have been caused by the following change:

  * New upstream release. (Closes: #544787).
      + Don't apply fix_different_archs.diff, upstream source different.

Part of that patch includes the following change to
./warsow_0.42_src/source/game/q_shared.h:

-#if ( defined ( _M_IX86 ) || defined ( __i386__ ) || defined ( __ia64__ ) ) && !defined ( C_ONLY )
+#if (defined(_M_IX86) || defined(__i386__)) && !defined(C_ONLY)
 #define id386
 #else
 #ifdef id386


i.e. it stops it from defining "id386" when __ia6__ is defined. "id386" is
used to determine when to use the x86 assembler code.

The location of this file has changed, it is now at
./source/gameshared/q_arch.h, but the patch is easily applied to that file
instead. Refreshed patch is attached.


This bug doesn't affect the version of the package that's currently in
testing, since this new upstream release never migrated.

-- 
Chris Butler <chrisb at debian.org>
  GnuPG Key ID: 4096R/49E3ACD3
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix_different_archs.diff
Type: text/x-diff
Size: 1367 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-games-devel/attachments/20110101/4fc45b19/attachment.diff>


More information about the Pkg-games-devel mailing list