Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 08 Mar 2006 15:40:04 +1300
From:      Matthew Luckie <mjl@luckie.org.nz>
To:        FreeBSD-gnats-submit@FreeBSD.org
Subject:   ports/94211: [patch] unbreak net-im/libjingle on !i386
Message-ID:  <E1FGoaq-000E0V-8h@latex.plunket.luckie.org.nz>
Resent-Message-ID: <200603080250.k282o2uJ084410@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help

>Number:         94211
>Category:       ports
>Synopsis:       [patch] unbreak net-im/libjingle on !i386
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Wed Mar 08 02:50:02 GMT 2006
>Closed-Date:
>Last-Modified:
>Originator:     Matthew Luckie
>Release:        FreeBSD 6.0-RELEASE-p4 alpha
>Organization:
University of Waikato
>Environment:
System: FreeBSD latex.plunket.luckie.org.nz 6.0-RELEASE-p4 FreeBSD 6.0-RELEASE-p4 #0: Thu Jan 26 22:14:57 NZDT 2006 root@latex.plunket.luckie.org.nz:/usr/src/sys/alpha/compile/latex alpha


	
>Description:
	I don't have the original compile problem, but as can be seen in the
	patch below, the implementation of SendPacket does not match the
	prototype.  It triggers on !i386 since size_t is 8 bytes, whereas
	an unsigned int is 4 bytes.

	I'll send this patch upstream.
>How-To-Repeat:
	compile on LP64 machine. 
>Fix:
--- patch-libjingle begins here ---
diff -uNr libjingle.orig/Makefile libjingle/Makefile
--- libjingle.orig/Makefile	Wed Mar  8 07:09:04 2006
+++ libjingle/Makefile	Wed Mar  8 11:57:27 2006
@@ -82,10 +82,6 @@
 
 .include <bsd.port.pre.mk>
 
-.if ${ARCH} != "i386"
-BROKEN=		Does not compile on !i386
-.endif
-
 LDFLAGS+=-L${LOCALBASE}/lib -L${OPENSSLLIB} ${PTHREAD_LIBS}
 PKG_CONFIG?=${LOCALBASE}/bin/pkg-config
 
diff -uNr libjingle.orig/files/patch-talk__session__phone_voicechannel.cc libjingle/files/patch-talk__session__phone_voicechannel.cc
--- libjingle.orig/files/patch-talk__session__phone_voicechannel.cc	Thu Jan  1 12:00:00 1970
+++ libjingle/files/patch-talk__session__phone_voicechannel.cc	Wed Mar  8 15:23:55 2006
@@ -0,0 +1,11 @@
+--- talk/session/phone/voicechannel.cc.orig	Wed Mar  8 14:48:48 2006
++++ talk/session/phone/voicechannel.cc	Wed Mar  8 14:50:08 2006
+@@ -159,7 +159,7 @@
+   channel_->OnPacketReceived(data, (int)len);
+ }
+ 
+-void VoiceChannel::SendPacket(const void *data, unsigned int len) {
++void VoiceChannel::SendPacket(const void *data, size_t len) {
+   // SendPacket gets called from MediaEngine; send to socket
+   // MediaEngine will call us on a random thread.  The Send operation on the socket is
+   // special in that it can handle this.
--- patch-libjingle ends here ---


>Release-Note:
>Audit-Trail:
>Unformatted:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?E1FGoaq-000E0V-8h>