pwlib needs porting for aarch64

Bug #1156282 reported by Dirk Mueller on 2013-03-17
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro AArch64 cross-distro work
Medium
Unassigned

Bug Description

pwlib (dependency of openh323) is not recognizing the aarch64 architecture as a 64 bit platform and fails compilation then later on.

I've developed the following patch for 1.11

--- configure.ac
+++ configure.ac
@@ -141,6 +141,11 @@
                    LIB64=1 ;
                   ;;

+ aarch64) MACHTYPE=aarch64 ;
+ P_64BIT=1 ;
+ LIB64=1 ;
+ ;;
+
    alpha | alphaev56 | alphaev6 | alphaev67 | alphaev7) MACHTYPE=alpha ;
                   P_64BIT=1 ;
                   ;;
--- make/unix.mak
+++ make/unix.mak
@@ -277,6 +277,10 @@
 endif
 endif

+ifneq (,$(findstring aarch64, $(MACHTYPE)))
+MACHTYPE := aarch64
+endif
+
 ifneq (,$(findstring powerpc, $(MACHTYPE)))
 MACHTYPE := ppc
 endif
@@ -435,6 +439,10 @@
 LDLIBS += -lresolv
 endif

+ifeq ($(MACHTYPE),aarch64)
+STDCCFLAGS += -DP_64BIT
+endif
+
 ifeq ($(MACHTYPE),ppc64)
 STDCCFLAGS += -DP_64BIT
 endif

Riku Voipio (riku-voipio) wrote :

Hi,

Thanks for the patch. You might want to take notice that pwlib has been replaced by ptlib which seemingly does not need this patch (but would probably benefit from it). You might want to migrate to ptiib to get the latest code.

Changed in linaro-aarch64:
status: New → Confirmed
importance: Undecided → Medium
Riku Voipio (riku-voipio) wrote :

Patch against ptlib, to mark aarch64 as a 64bit Architecture.

Riku Voipio (riku-voipio) wrote :

The patch has also been posted upstream: https://sourceforge.net/p/opalvoip/bugs/429/

Riku Voipio (riku-voipio) wrote :

Upstream has accepted the patch to ptlib svn.

Riku Voipio (riku-voipio) wrote :

This has been fixed in ptlib 2.12 release. Note that ptlib 2.10 might still be broken despite compiling on aarch64.

Changed in linaro-aarch64:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers