pwlib needs porting for aarch64

Bug #1156282 reported by Dirk Mueller
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro AArch64 cross-distro work
Fix Released
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

Revision history for this message
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
Revision history for this message
Riku Voipio (riku-voipio) wrote :

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

Revision history for this message
Riku Voipio (riku-voipio) wrote :

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

Revision history for this message
Riku Voipio (riku-voipio) wrote :

Upstream has accepted the patch to ptlib svn.

Revision history for this message
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.