From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:34480) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jKmjB-00071y-GR for guix-patches@gnu.org; Sat, 04 Apr 2020 13:37:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1jKmjA-00066m-9F for guix-patches@gnu.org; Sat, 04 Apr 2020 13:37:05 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:33176) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1jKmjA-00066E-5a for guix-patches@gnu.org; Sat, 04 Apr 2020 13:37:04 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1jKmj8-0005Oz-3d for guix-patches@gnu.org; Sat, 04 Apr 2020 13:37:02 -0400 Subject: [bug#40264] Linphone: 12-add-belle-sip-v2 Resent-Message-ID: From: Maxim Cournoyer References: <20200402132447.1db0ed9b@scratchpost.org> <03bed573467e87d60306402b6772d4b6@disroot.org> <326baae6c830af093895314ee72572a9@disroot.org> <20200403133435.57d546b5@scratchpost.org> Date: Sat, 04 Apr 2020 13:36:32 -0400 In-Reply-To: <20200403133435.57d546b5@scratchpost.org> (Danny Milosavljevic's message of "Fri, 3 Apr 2020 13:34:35 +0200") Message-ID: <87a73r6tlr.fsf@gmail.com> MIME-Version: 1.0 Content-Type: text/plain List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Danny Milosavljevic Cc: Raghav Gururajan , 40264@debbugs.gnu.org Hello Danny and Raghav, Danny Milosavljevic writes: > Hi Raghav, > > On Thu, 02 Apr 2020 17:19:29 +0000 > "Raghav Gururajan" wrote: > >> I suspect it could be due to having two different definitions for bctoolbox and ortp. Could you try belle-sip after applying the patch http://debbugs.gnu.org/cgi/bugreport.cgi?bug=40326 > > I tried and it didn't help. > > However, I found and fixed the problem: > > From 1f847861de4ab73011d6243fb652f098e16106cb Mon Sep 17 00:00:00 2001 > From: Raghav Gururajan > Date: Wed, 1 Apr 2020 20:42:33 -0400 > Subject: [FIXME 2/2] gnu: Add belle-sip. > Tags: patch > > * gnu/packages/linphone.scm (belle-sip): New variable. > > Signed-off-by: Danny Milosavljevic > --- > gnu/packages/linphone.scm | 42 +++++++++++++++++++++++++++++++++++++++ > 1 file changed, 42 insertions(+) > > diff --git a/gnu/packages/linphone.scm b/gnu/packages/linphone.scm > index 18a11776e8..9cbbd5e7d6 100644 > --- a/gnu/packages/linphone.scm > +++ b/gnu/packages/linphone.scm > @@ -22,11 +22,13 @@ > #:use-module (gnu packages admin) > #:use-module (gnu packages audio) > #:use-module (gnu packages base) > + #:use-module (gnu packages compression) > #:use-module (gnu packages documentation) > #:use-module (gnu packages gl) > #:use-module (gnu packages gnome) > #:use-module (gnu packages graphviz) > #:use-module (gnu packages image) > + #:use-module (gnu packages java) > #:use-module (gnu packages linux) > #:use-module (gnu packages pulseaudio) > #:use-module (gnu packages python) > @@ -335,3 +337,43 @@ is responsible for receiving and sending all multimedia streams in Linphone, > including media capture, encoding and decoding, and rendering.") > (home-page "https://gitlab.linphone.org/BC/public/mediastreamer2") > (license license:gpl2+))) > + > +(define-public belle-sip > + (package > + (name "belle-sip") > + (version "1.6.3") > + (source > + (origin > + (method url-fetch) > + (uri > + (string-append "https://www.linphone.org/releases/sources/" name > + "/" name "-" version ".tar.gz")) > + (sha256 > + (base32 "0s55kggmgxap54dkw5856bgk4xg7yvbzialpxnjm0zhpic3hff1z")))) > + (build-system cmake-build-system) > + (arguments > + `(#:tests? #f ; Requires network access > + #:configure-flags > + (list > + "-DENABLE_STATIC=NO") > + #:phases > + (modify-phases %standard-phases > + (add-after 'unpack 'patch > + (lambda _ > + (substitute* "src/CMakeLists.txt" > + ;; ANTLR would use multithreaded DFA generation > + ;; otherwise--which would not be reproducible. > + (("-Xmultithreaded ") "")) > + #t))))) Nice find! Although note that this doesn't exist in the latest code, and that the latest release is nearly 3 years old (2017-07-21). Perhaps we should just try to package the latest master? Their git history suggests many bug fixes, including a memory leak. Maxim