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

Bug#732437: marked as done (mesa: Add IEEE arm64 support)

Your message dated Wed, 25 Dec 2013 20:01:46 +0100
with message-id <20131225190146.GO4822@betterave.cristau.org>
and subject line Re: Bug#732437: mesa: Add IEEE arm64 support
has caused the Debian Bug report #732437,
regarding mesa: Add IEEE arm64 support
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
732437: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=732437
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Source: mesa
Version: 9.2.2-1
Severity: normal
Tags: patch
User: debian-arm@lists.debian.org
Usertag: arm64

This package fails to build for arm64. 

It just needs __aarch64__ adding to a great long list of arches for
which USE_IEEE (and IEEE_ONE) is defined. This list seems to list all
sensible arches. Are there any arches for which USE_IEEE should not be
defined? If not a better fix would be to simply remove the specific arch
list so that each new arch didn't have to pointlessly add itself. Or it
could just list the small number of exceptions?  This is a question for
upstream on which I am not qualified to judge, but the current code in
src/mesa/main/compiler.h  seems a candidate for updating.

In the meantime the attached patch adds the new compiler arch define so
it builds.

-- System Information:
Debian Release: 7.3
  APT prefers stable
  APT policy: (990, 'stable')
Architecture: i386 (i686)

Kernel: Linux 3.2.0-kvm-i386-20110111 (SMP w/1 CPU core)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

diff -u mesa-9.2.2/debian/changelog mesa-9.2.2/debian/changelog
--- mesa-9.2.2/debian/changelog
+++ mesa-9.2.2/debian/changelog
@@ -1,3 +1,9 @@
+mesa (9.2.2-1arm64) unstable; urgency=low
+
+  * Add support for arm64/aarch64
+
+ -- Wookey <wookey@debian.org>  Tue, 17 Dec 2013 15:29:02 +0000
+
 mesa (9.2.2-1) unstable; urgency=low
 
   * New upstream release.
only in patch2:
unchanged:
--- mesa-9.2.2.orig/src/mesa/main/compiler.h
+++ mesa-9.2.2/src/mesa/main/compiler.h
@@ -313,7 +313,7 @@
     defined(ia64) || defined(__ia64__) || \
     defined(__hppa__) || defined(hpux) || \
     defined(__mips) || defined(_MIPS_ARCH) || \
-    defined(__arm__) || \
+    defined(__arm__) || defined(__aarch64__) || \
     defined(__sh__) || defined(__m32r__) || \
     (defined(__sun) && defined(_IEEE_754)) || \
     defined(__alpha__)

--- End Message ---
--- Begin Message --- 
Version: 10.0.0-1

On Tue, Dec 17, 2013 at 23:35:39 +0000, Wookey wrote:

> Source: mesa
> Version: 9.2.2-1
> Severity: normal
> Tags: patch
> User: debian-arm@lists.debian.org
> Usertag: arm64
> 
> This package fails to build for arm64. 
> 
AFAICT this was applied upstream in 10.0.

Cheers,
Julien

Attachment: signature.asc
Description: Digital signature


--- End Message ---
Reply to: