tag 518732 patch pending tag 510500 patch pending thanks Peter De Wachter <[email protected]> (03/05/2009): > Hi. The following patch fixes this bug. FNSTSW is "Store FPU Status > Word". This status word is a 16-bit value so the destination register > should be %ax, not %eax. Apparently old versions of binutils were a > bit lax.
Indeed, that's what is used in other places. Fixed package uploaded to DELAYED/1 with the attached source debdiff. Mraw, KiBi.
+++ allegro4.2-4.2.2/debian/patches/series @@ -10,0 +11 @@ +060_fix_fnstsw.diff --- allegro4.2-4.2.2/debian/changelog +++ allegro4.2-4.2.2/debian/changelog @@ -1,3 +1,17 @@ +allegro4.2 (2:4.2.2-2.1) unstable; urgency=medium + + * Non-maintainer upload. + + * debian/patches/06_fix_fnstsw.diff: + + Fix FTBFS with binutils >= 1.9. Quoting the patch submitter: + “FNSTSW is "Store FPU Status Word". This status word is a 16-bit + value so the destination register should be %ax, not %eax.” + Thanks to Peter De Wachter (Closes: #510500, #518732). + + * Set urgency to “medium”, testing is also affected. + + -- Cyril Brulebois <[email protected]> Fri, 17 Jul 2009 20:36:38 +0200 + allegro4.2 (2:4.2.2-2) unstable; urgency=low * debian/control: --- allegro4.2-4.2.2.orig/debian/patches/060_fix_fnstsw.diff +++ allegro4.2-4.2.2/debian/patches/060_fix_fnstsw.diff @@ -0,0 +1,11 @@ +--- a/src/i386/icpus.s ++++ b/src/i386/icpus.s +@@ -67,7 +67,7 @@ FUNC(_i_is_fpu) + + fninit + movl $0x5A5A, %eax +- fnstsw %eax ++ fnstsw %ax + cmpl $0, %eax + jne is_fpu_not_found +
signature.asc
Description: Digital signature
