From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:37742) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jKwiY-0000XD-6z for guix-patches@gnu.org; Sun, 05 Apr 2020 00:17:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1jKwiU-0005OK-Pu for guix-patches@gnu.org; Sun, 05 Apr 2020 00:17:05 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:33498) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1jKwiU-0005O2-MP for guix-patches@gnu.org; Sun, 05 Apr 2020 00:17:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1jKwiU-00014b-Gq for guix-patches@gnu.org; Sun, 05 Apr 2020 00:17:02 -0400 Subject: [bug#40264] Linphone: 11-add-mediastreamer-v2 Resent-Message-ID: From: maxim.cournoyer@gmail.com References: <981ea9c838575bacfe934a652ceeca62@disroot.org> Date: Sun, 05 Apr 2020 00:16:29 -0400 In-Reply-To: <981ea9c838575bacfe934a652ceeca62@disroot.org> (Raghav Gururajan's message of "Thu, 02 Apr 2020 00:20:04 +0000") Message-ID: <87k12u36ua.fsf@hurd.i-did-not-set--mail-host-address--so-tickle-me> MIME-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 8bit 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: Raghav Gururajan Cc: 40264@debbugs.gnu.org Hi Raghav! "Raghav Gururajan" writes: > From 3cf73361203f7833c61a24f69b9ab4bb4da358ce Mon Sep 17 00:00:00 2001 > From: Raghav Gururajan > Date: Wed, 1 Apr 2020 20:18:20 -0400 > Subject: [PATCH 9/9] gnu: Add mediastreamer > > * gnu/packages/linphone.scm (mediastreamer): New variable. > --- > gnu/packages/linphone.scm | 83 +++++++++++++++++++++++++++++++++++++++ > 1 file changed, 83 insertions(+) > > diff --git a/gnu/packages/linphone.scm b/gnu/packages/linphone.scm > index 3be35bdb74..ce3ffa763b 100644 > --- a/gnu/packages/linphone.scm > +++ b/gnu/packages/linphone.scm > @@ -19,12 +19,24 @@ > > (define-module (gnu packages linphone) > #:use-module (gnu packages) > + #:use-module (gnu packages admin) > + #:use-module (gnu packages audio) > #:use-module (gnu packages base) > #: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 linux) > + #:use-module (gnu packages pulseaudio) > + #:use-module (gnu packages python) > #:use-module (gnu packages sqlite) > + #:use-module (gnu packages telephony) > #:use-module (gnu packages tls) > + #:use-module (gnu packages video) > + #:use-module (gnu packages xiph) > #:use-module (gnu packages xml) > + #:use-module (gnu packages xorg) > #:use-module ((guix licenses) #:prefix license:) > #:use-module (guix packages) > #:use-module (guix download) > @@ -252,3 +264,74 @@ protocol, written in C. It is fully portable and can be executed on many > platforms including both ARM and x86.") > (home-page "https://gitlab.linphone.org/BC/public/bzrtp") > (license license:gpl2+))) > + > +(define-public mediastreamer > + (package > + (name "mediastreamer") The variable name and package name should be "mediastreamer2", as per upstream. > + (version "2.16.1") > + (source > + (origin > + (method url-fetch) > + (uri > + (string-append "https://www.linphone.org/releases/sources/" name > + "/" name "-" version ".tar.gz")) > + (sha256 > + (base32 "0whpqr69wz0pnzvragkpfblxhd0rds8k06c3mw5a0ag216a1yd9k")) > + (patches > + (list > + ;; For fixing definitions. > + (origin > + (method url-fetch) > + (uri > + (string-append "https://gitlab.linphone.org/BC/public/mediastreamer2/commit/" > + "de3a24b795d7a78e78eab6b974e7ec5abf2259ac.diff")) > + (file-name "mediastreamer-msfactory.patch") > + (sha256 > + (base32 "18lsn9fc9di8bzr12ck3h9k9h8xvvhjzgkimbj9p4vmxxwhjaxn2"))))))) You can just use the "ENABLE_STRICT=NO", which will disable -Werror and make the package more resilient in the face of future compiler upgrades in Guix. No more need for that patch :-). > + (build-system cmake-build-system) > + (arguments > + `(#:tests? #f ; No test target > + #:configure-flags > + (list > + "-DENABLE_SRTP=NO" ; Using ZRTP instead Usually we try to offer the most features possible. I don't see why we shouldn't add SRTP support since we should already have the required dependencies. Also, I see there's a ENABLE_UNIT_TESTS=YES option for unit tests, which we should enable and try to run. > + "-DENABLE_STATIC=NO" ; Not required > + "-DENABLE_GSM=NO" ; Need recompilation with -fPIC Not a merge breaker, but it'd make sense to have GSM for telephony focused software :-). > + "-DENABLE_BV16=NO"))) ; Not available > + (native-inputs > + `(("dot" ,graphviz) > + ("doxygen" ,doxygen) > + ("python" ,python))) > + (inputs > + `(("alsa" ,alsa-lib) > + ("bcg729" ,bcg729) > + ("bcmatroska2" ,bcmatroska2) > + ("bctoolbox" ,bctoolbox) > + ("ffmpeg" ,ffmpeg) > + ("glew" ,glew) > + ("glu" ,glu) > + ("glx" ,mesa-utils) > + ("gsm" ,gsm) > + ("opengl" ,mesa) > + ("opus" ,opus) > + ("ortp" ,ortp) > + ("pcap" ,libpcap) > + ("portaudio" ,portaudio) > + ("pulseaudio" ,pulseaudio) > + ("spandsp" ,spandsp) > + ("speex" ,speex) > + ("speexdsp" ,speexdsp) > + ("srtp" ,libsrtp) > + ("theora" ,libtheora) > + ("turbojpeg" ,libjpeg-turbo) > + ("v4l" ,v4l-utils) > + ("vpx" ,libvpx) > + ("x11" ,libx11) > + ("xv" ,libxv) > + ("zrtp", bzrtp))) > + (synopsis "Belledonne Communications Streaming Engine") > + (description "Mediastreamer is a powerful and > lightweight streaming There are some strange characters in there (around 'powerful'). Please edit them out. Maxim