Linphone

DoneSubmitted by Raghav Gururajan.
Details
3 participants
  • Danny Milosavljevic
  • Maxim Cournoyer
  • Raghav Gururajan
Owner
unassigned
Severity
normal
Blocked by
R
R
Raghav Gururajan wrote on 28 Mar 00:10 +0100
(address . guix-patches@gnu.org)
a3c3755c86e0d25ca3c4672b8865a9ed@disroot.org
Hello Guix!
I am creating this thread for Linphone packaging. I will be sending patches one by one with-in this thread. :-)
Regards,RG.
R
R
Raghav Gururajan wrote on 28 Mar 00:13 +0100
Linphone: 1-create-linphone-module
(address . 40264@debbugs.gnu.org)
721cd71eb8dc556ec10c48ce9d796457@disroot.org
R
R
Raghav Gururajan wrote on 28 Mar 00:14 +0100
Linphone: 2-update-linphone-module
(address . 40264@debbugs.gnu.org)
a75e74ed34f370d23de422c6bbdb6846@disroot.org
R
R
Raghav Gururajan wrote on 28 Mar 00:15 +0100
Linphone: 3-add-bcunit
(address . 40264@debbugs.gnu.org)
bf996a356191a54b46474d2ea8b1809f@disroot.org
R
R
Raghav Gururajan wrote on 28 Mar 00:21 +0100
Linphone: 4-add-bctoolbox
(address . 40264@debbugs.gnu.org)
af32b6c8b8982b9e256aeeecf1be8a75@disroot.org
R
R
Raghav Gururajan wrote on 28 Mar 19:40 +0100
Linphone: 5-add-belr
(address . 40264@debbugs.gnu.org)
8b1a36bcb780efabe63bfca08f773b14@disroot.org
R
R
Raghav Gururajan wrote on 28 Mar 20:40 +0100
Linphone: 6-add-belcard
(address . 40264@debbugs.gnu.org)
4ff8a2b29b9afc2877bffcba893a5137@disroot.org
R
R
Raghav Gururajan wrote on 28 Mar 22:22 +0100
Linphone: 7-add-bcmatroska2
(address . 40264@debbugs.gnu.org)
2f575bb6b0ae24d40ee27eb2c5dd7033@disroot.org
R
R
Raghav Gururajan wrote on 28 Mar 22:35 +0100
Linphone: 8-add-bcg729
(address . 40264@debbugs.gnu.org)
33ee757c05f31d4f633a9b3a70a64192@disroot.org
R
R
Raghav Gururajan wrote on 28 Mar 23:14 +0100
Linphone: 9-add-ortp
(address . 40264@debbugs.gnu.org)
97e3551fadbd0c3c7588497bd4732b3d@disroot.org
R
R
Raghav Gururajan wrote on 28 Mar 23:36 +0100
Linphone: 10-add-bzrtp
(address . 40264@debbugs.gnu.org)
2f53cae507e1ceda6e5f3795809627b9@disroot.org
R
R
Raghav Gururajan wrote on 29 Mar 10:58 +0200
Linphone: 11-add-mediastreamer
(address . 40264@debbugs.gnu.org)
8aa479d264eab2983ac0fe26c936f4a3@disroot.org
R
R
Raghav Gururajan wrote on 30 Mar 04:45 +0200
Linphone: 12-add-belle-sip
(address . 40264@debbugs.gnu.org)
7d170198ecb034b9bdb9b4f486c074d3@disroot.org
R
R
Raghav Gururajan wrote on 30 Mar 11:47 +0200
Linphone: 13-add-linphone
(address . 40264@debbugs.gnu.org)
84c667446cd5676fef41c3d9724646cb@disroot.org
R
R
Raghav Gururajan wrote on 30 Mar 18:50 +0200
Linphone: 14-add-linphoneqt
(address . 40264@debbugs.gnu.org)
a3f758231011fa20d14eea20b60fa955@disroot.org
D
D
Danny Milosavljevic wrote on 30 Mar 22:45 +0200
Re: [bug#40264] Linphone: 12-add-belle-sip
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200330224554.35e01f4c@scratchpost.org
Missing definition of libantlr3c?
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6CWoIACgkQ5xo1VCwwuqUhqQf/WZ38+Zt+gEUXjUE1waJnzHkmww3EmgECJaeK+jiY/YAk/Jhw9CraJ/L4DZuRa38j+Xz5KwKpWAgxfOuTBKLyB1efJRIJ8pJPUUkv23/dQvKQx6kR/QRPr4pCpgBY0kM1nKw6q+RTHK+by7XI0m9ktMBsM4Trx/GRhdrPoFglI4LYf+LhXhkuhymZnAvkzrUkdR4jz+a4IlvPu/Y+litpDv6MGQDnHbgEb1e5lTkT82z6icjT4VAKKxo0eeFofjVjx8JRrSCyFxVeRIDUG2IKAq2EyKaKK0xfYP/wnJ845DxvseedUFpS1JzpFROFMjo9pVzNP6a0KT/CZpK61i3xtg===cdNT-----END PGP SIGNATURE-----

M
M
Maxim Cournoyer wrote on 31 Mar 03:55 +0200
control message for bug #40264
(address . control@debbugs.gnu.org)
87ftdpz3rw.fsf@gmail.com
block 40264 by 40307quit
R
R
Raghav Gururajan wrote on 31 Mar 15:44 +0200
Linphone: 15-add-msamr
(address . 40264@debbugs.gnu.org)
09096101a69d6e948361590fca62bfe2@disroot.org
R
R
Raghav Gururajan wrote on 31 Mar 16:12 +0200
Linphone: 16-add-openh264
(address . 40264@debbugs.gnu.org)
c1718792f04007d4f1c740955e8165e6@disroot.org
R
R
Raghav Gururajan wrote on 31 Mar 16:31 +0200
Linphone: 17-add-mssilk
(address . 40264@debbugs.gnu.org)
cd9ebabb3a478c4b1313cfb18f1c9789@disroot.org
R
R
Raghav Gururajan wrote on 31 Mar 16:50 +0200
Linphone: 18-add-mswebrtc
(address . 40264@debbugs.gnu.org)
a6188ef65f1be1d86999757054f824ad@disroot.org
R
R
Raghav Gururajan wrote on 31 Mar 17:50 +0200
control message for bug #40264
(address . control@debbugs.gnu.org)
eedf882e4495e7e0f8820925ff63bbac@disroot.org
block 40264 by 40285
R
R
Raghav Gururajan wrote on 31 Mar 17:52 +0200
(address . control@debbugs.gnu.org)
e6b45e27d6ce70ee78a2f27203e1b466@disroot.org
block 40264 by 40291
R
R
Raghav Gururajan wrote on 31 Mar 17:53 +0200
(address . control@debbugs.gnu.org)
f5f33b58291b19ffd0d40569761d306a@disroot.org
block 40264 by 40348
R
R
Raghav Gururajan wrote on 31 Mar 17:54 +0200
(address . control@debbugs.gnu.org)
d985c87c4ecf788e446654a3279f64be@disroot.org
block 40264 by 40349
R
R
Raghav Gururajan wrote on 31 Mar 18:32 +0200
Re: [bug#40264] Linphone: 12-add-belle-sip
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
ac905cdd96d0de33c2aac606f1c4c614@disroot.org
Toggle quote (2 lines)> Missing definition of libantlr3c?
M
M
Maxim Cournoyer wrote on 1 Apr 04:38 +0200
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)
87h7y4vsjy.fsf@gmail.com
"Raghav Gururajan" <raghavgururajan@disroot.org> writes:
Toggle quote (6 lines)>> Missing definition of libantlr3c?>> http://debbugs.gnu.org/cgi/bugreport.cgi?bug=40291>> :-)
This has now been reviewed & merged.
M
M
Maxim Cournoyer wrote on 1 Apr 04:46 +0200
Re: [bug#40264] Linphone: 3-add-bcunit
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
87d08rx6rx.fsf@gmail.com
Hello Raghav,
"Raghav Gururajan" <raghavgururajan@disroot.org> writes:
Toggle quote (4 lines)> From fe32f106ca646c357c958e1b5ebd375b942f4f3c Mon Sep 17 00:00:00 2001> From: Raghav Gururajan <raghavgururajan@disroot.org>> Date: Fri, 27 Mar 2020 17:19:45 -0400> Subject: [PATCH 3/3] gnu: packages: linphone: Add bcunit
^ You can drop the 'packages: linphone:' part and add a dot.
Toggle quote (52 lines)>> * gnu/packages/linphone.scm (bcunit): New variable.> ---> gnu/packages/linphone.scm | 38 ++++++++++++++++++++++++++++++++++++++> 1 file changed, 38 insertions(+)>> diff --git a/gnu/packages/linphone.scm b/gnu/packages/linphone.scm> index 35c9ec955f..974fc62792 100644> --- a/gnu/packages/linphone.scm> +++ b/gnu/packages/linphone.scm> @@ -20,6 +20,7 @@> (define-module (gnu packages linphone)> #:use-module (gnu packages)> #:use-module (gnu packages base)> + #:use-module (gnu packages ncurses)> #:use-module ((guix licenses) #:prefix license:)> #:use-module (guix packages)> #:use-module (guix download)> @@ -28,3 +29,40 @@> #:use-module (guix build-system gnu)> #:use-module (guix build-system python)> #:use-module (guix build-system qt))> +> +(define-public bcunit> + (package> + (name "bcunit")> + (version "3.0.2")> + (source> + (origin> + (method url-fetch)> + (uri> + (string-append "https://www.linphone.org/releases/sources/" name> + "/" name "-" version ".tar.gz"))> + (sha256> + (base32 "0ylchj8w98ic2fkqpxc6yk4s6s0h0ql2zsz5n49jd7126m4h8dqk"))))> + (build-system cmake-build-system)> + (arguments> + '(#:tests? #f ; No test target> + #:configure-flags> + (list> + "-DENABLE_SHARED=YES"> + "-DENABLE_STATIC=YES"> + "-DENABLE_AUTOMATED=ON"> + "-DENABLE_BASIC=ON"> + "-DENABLE_CONSOLE=ON"> + "-DENABLE_CURSES=ON"> + "-DENABLE_DOC=ON"> + "-DENABLE_EXAMPLES=ON"> + "-DENABLE_TEST=OFF" ; Undefined reference to CU_trace_handler> + "-DENABLE_MEMTRACE=ON"> + "-DENABLE_DEPRECATED=OFF"))) ; Not required
Please only activate the switches which need to be activated (let's notrepeat the default values to keep the noise level/verbosity as low aspossible). You can usually see the defaults by inspecting theCMakeLists.txt file at the root of the project.
Toggle quote (5 lines)> + (inputs> + `(("curses" ,ncurses)))> + (synopsis "Belledonne Communications Unit Testing Framework")> + (description "BCUnit is a fork of the defunct project CUnit,> +with several fixes and patches applied. It is an unit testing
^ a
Toggle quote (4 lines)> +framework for writing, administering, and running unit tests in C.")> + (home-page "https://gitlab.linphone.org/BC/public/bcunit")> + (license license:lgpl2.0+)))
The first 3 commits of this series should be squashed together.
Also, please run the indent-code.el script for proper indentation.
Thank you!
Maxim
R
R
Raghav Gururajan wrote on 1 Apr 22:21 +0200
Linphone: 3-add-bcunit-v2
(address . 40264@debbugs.gnu.org)
e26853d1760b8f570fada2cb05de81be@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 00:13 +0200
Linphone: 4-add-bctoolbox-v2
(address . 40264@debbugs.gnu.org)
99eed829aeef1bc127cba30175a617c9@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 00:20 +0200
Linphone: 5-add-belr-v2
(address . 40264@debbugs.gnu.org)
c7e469595fde22e1a7b51d131cd6e25b@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 00:32 +0200
Linphone: 6-add-belcard-v2
(address . 40264@debbugs.gnu.org)
2963b434cfecf0f8deacc6736e3edfd3@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 00:40 +0200
Linphone: 7-add-bcmatroska2-v2
(address . 40264@debbugs.gnu.org)
559e5570d95d5ea6910cf75eb85cd514@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 00:46 +0200
Linphone: 8-add-bcg729-v2
(address . 40264@debbugs.gnu.org)
9fe8422fb271692967cbdd808ff096ab@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 00:56 +0200
Linphone: 9-add-ortp-v2
(address . 40264@debbugs.gnu.org)
168c40918ce771c964c0058bcd42542e@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 01:04 +0200
Linphone: 10-add-bzrtp-v2
(address . 40264@debbugs.gnu.org)
3c5c5f2875b1a7bef5660385242e425c@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 02:20 +0200
Linphone: 11-add-mediastreamer-v2
(address . 40264@debbugs.gnu.org)
981ea9c838575bacfe934a652ceeca62@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 02:44 +0200
Linphone: 12-add-belle-sip-v2
(address . 40264@debbugs.gnu.org)
03bed573467e87d60306402b6772d4b6@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 03:20 +0200
Linphone: 13-add-linphone-v2
(address . 40264@debbugs.gnu.org)
378e7af9f19adf07ee149f4cfa26ed7d@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 03:47 +0200
Linphone: 14-add-linphoneqt-v2
(address . 40264@debbugs.gnu.org)
4e3b1a6a2e403cd20d9d71b568217982@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 04:06 +0200
Linohone: 15-add-msamr-v2
(address . 40264@debbugs.gnu.org)
f37165e14eb7d87332a299c7565adff8@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 04:55 +0200
Linphone: 16-add-msopenh264-v2
(address . 40264@debbugs.gnu.org)
1849ec6eb37e7fef65a191f75e196bb9@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 05:00 +0200
Linphone: 17-add-mssilk-v2
(address . 40264@debbugs.gnu.org)
7e6b55f5cb406358625cf76126d32aab@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 05:06 +0200
Linphone: 18-add-mswebrtc-v2
(address . 40264@debbugs.gnu.org)
ff7cc3d1b9ee2c186bc6c9ae49cc7b64@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 10:56 +0200
Linphone: 14-add-linphoneqt-v3
(address . 40264@debbugs.gnu.org)
7c3a4ab21f75d9b0325cf7563c099353@disroot.org
D
D
Danny Milosavljevic wrote on 2 Apr 13:24 +0200
Re: [bug#40264] Linphone: 12-add-belle-sip-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200402132447.1db0ed9b@scratchpost.org
Hi Raghav,
belle-sip is not reproducible.
$ guix build --rounds=2 belle-sip[...]phase `compress-documentation' succeeded after 0.0 secondsoutput ‘/gnu/store/rykd8wp9ibjf3ad4nlg15vmyzgq9izsp-belle-sip-1.6.3’ of ‘/gnu/store/lqdmaxvlhxk25jr7whk5bvi0y1zbzp5a-belle-sip-1.6.3.drv’ differs from ‘/gnu/store/rykd8wp9ibjf3ad4nlg15vmyzgq9izsp-belle-sip-1.6.3-check’ from previous roundbuild of /gnu/store/lqdmaxvlhxk25jr7whk5bvi0y1zbzp5a-belle-sip-1.6.3.drv failed
$ diffoscope --exclude-directory-metadata /gnu/store/rykd8wp9ibjf3ad4nlg15vmyzgq9izsp-belle-sip-1.6.3 /gnu/store/rykd8wp9ibjf3ad4nlg15vmyzgq9izsp-belle-sip-1.6.3-check[...]
I'm not saying that this is a blocker for merging, but basically libbellesip.so (and libbellesip.so.0) doesn't always come out the same. Spooky.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6Fy38ACgkQ5xo1VCwwuqWjtwf/ele2hx1ABIrIuF/B3a5UJP2NtkGHhEO8D0p2qPultxfQXU/4XjTHMEeuHWYNgidzl+1PASgbjmnG9M29xdB0JTqRJ72zmBQezmcr9e6b4ERNhfVqT+Qej2S/xA3gLe+i4RRRZor1f5aPIBh32q4NDQLbiUqAoofiR8Cp3F/rtPoouIvJ8RoUPKjmz1Du4FuuotrnRvpNOTneSeZSo4Mn6Lw9wBfS55PHJA8Dl81jXLZt/KSvJSsuRAWiUEX79tHFhRCte2gzEzyq8/QEhi724J4YSXzjfDxhPkwWw43h5YuLTxCTpcoyFm8mpUBoYspAnU6HPd9fJsyJUYZw6OJgCA===eASO-----END PGP SIGNATURE-----

R
R
Raghav Gururajan wrote on 2 Apr 18:27 +0200
Re: Linphone: 14-add-linphoneqt-v3
(address . 40264@debbugs.gnu.org)
25dd38cc62537f7883f925dd8be73d8f@disroot.org
Hello Maxim and Danny!
At this point for linphoneqt a.k.a linphone-desktop, I am facing following issues.

When I build *without* `-DENABLE_DBUS=YES`and run the program, I get:
QSocketNotifier: Socket notifiers cannot be enabled or disabled from another threadQMutex: destroying locked mutex

When I build *with* `-DENABLE_DBUS=YES` and run the program, I get:
Segmentation Fault (Core Dumped)

I think the following patch is relevant, but when I use it, doesn't get successfully patched during the build.
https://gitlab.linphone.org/BC/public/linphone-desktop/commit/9cf08623e3092fa19366e5c07fbe06898a59f039.diff

Any ideas on how to fix this situation?
Thank you!
Regards,RG.
R
R
Raghav Gururajan wrote on 2 Apr 18:33 +0200
Re: [bug#40264] Linphone: 12-add-belle-sip-v2
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
e64341a66f09ee99c320b5faa4ff880e@disroot.org
Hi Danny!
Toggle quote (2 lines)> belle-sip is not reproducible.
Nice catch!
Toggle quote (3 lines)> I'm not saying that this is a blocker for merging, but basically libbellesip.so (and> libbellesip.so.0) doesn't always come out the same. Spooky.
Yeah, that is really is spooky. I even disabled the tests as it required network access. Is it possible that there are multiple values set for any environment variable, and it uses different values randomly during each build?
Regards,RG.
R
R
Raghav Gururajan wrote on 2 Apr 19:19 +0200
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
326baae6c830af093895314ee72572a9@disroot.org
Hi Danny!
Toggle quote (11 lines)>> belle-sip is not reproducible.> > Nice catch!> >> I'm not saying that this is a blocker for merging, but basically libbellesip.so (and>> libbellesip.so.0) doesn't always come out the same. Spooky.> > Yeah, that is really is spooky. I even disabled the tests as it required network access. Is it> possible that there are multiple values set for any environment variable, and it uses different> values randomly during each build?
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
Regards,RG.
R
R
Raghav Gururajan wrote on 2 Apr 22:53 +0200
Linphone: 14-add-linphoneqt-v4
(address . 40264@debbugs.gnu.org)
09e59dc1836ec7716bdd7dd110be2ab4@disroot.org
R
R
Raghav Gururajan wrote on 2 Apr 22:56 +0200
Re: Linphone: 14-add-linphoneqt-v3
(address . 40264@debbugs.gnu.org)
695292237245b0df3d7ad83cc350d0bc@disroot.org
Hello Maxim and Danny!
Toggle quote (19 lines)> At this point for linphoneqt a.k.a linphone-desktop, I am facing following issues.> > When I build *without* `-DENABLE_DBUS=YES`and run the program, I get:> > QSocketNotifier: Socket notifiers cannot be enabled or disabled from another thread> QMutex: destroying locked mutex> > When I build *with* `-DENABLE_DBUS=YES` and run the program, I get:> > Segmentation Fault (Core Dumped)> > I think the following patch is relevant, but when I use it, doesn't get successfully patched during> the build.> > https://gitlab.linphone.org/BC/public/linphone-desktop/commit/9cf08623e3092fa19366e5c07fbe06898a59f0> 9.diff> > Any ideas on how to fix this situation?
So I fixed "QSocketNotifier" error in version 4 patch (14-add-linphoneqt-v4). I still don't know how to fix "QMutex" error.
Regards,RG.
R
R
Raghav Gururajan wrote on 3 Apr 01:19 +0200
Linphone: 13-add-linphone-v3
(address . 40264@debbugs.gnu.org)
c67ed16f20f97e2110e1684e945e774c@disroot.org
R
R
Raghav Gururajan wrote on 3 Apr 02:24 +0200
Linphone: 14-add-linphoneqt-v5
(address . 40264@debbugs.gnu.org)
f48322cd660cbc535611107d80df3b26@disroot.org
D
D
Danny Milosavljevic wrote on 3 Apr 12:51 +0200
Re: [bug#40264] Linphone: 3-add-bcunit-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200403125133.3e3ef924@scratchpost.org
Hi Raghav,
thanks for the patch!
Pushed to guix master as commit d5a8e391ac2546241d0ff60c9a1fd2fb8ff57536.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6HFTUACgkQ5xo1VCwwuqWgggf/air2XVwp1Jv4BZGpruzjEBD1qlXaw6Q0YSphuCBP6DSRYPmVgLlMr6ddAFXW7roE/30OPXLsmzEXTp6Sk4RigGhcJZqcl9VAYhcv0EmPze0tiXhAxhLJgq2hD+VhcjhJZqoGzmpC8jKzXV16TlVZbmdx9XKm4FrX82fZAEzygAZFEbKqdn/sprYLrM5qPLbmxMESIFrQb+G07hTGX04jzlc1z0u1Yo02FkJkUbLLqJhhlvfGm0hoOPqN5bTxvJa9koGquXbxnrUwA+H8yT8J+7KgdrdHZyUUmNzKx5XKDiA/Peoy/ibeDdmiHhNP7q3if3VwrDA+gAzQ2qdNpsyT2g===PGFG-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 3 Apr 12:51 +0200
Re: [bug#40264] Linphone: 6-add-belcard-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200403125149.1941a928@scratchpost.org
Hi Raghav,
thanks for the patch!
Pushed to guix master as commit c8f69ec27b4d492f63edc71f0a83ec413ea85b96.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6HFUUACgkQ5xo1VCwwuqXjaAf+ObA5kl0JILyOEeVOO8bMoaSI3rJEDEHagY3DiUfVNQIZ4n6Z6sqpNoo2l6vJ1IsoQerqBX5OgdrkzoeMCCalrgkoFsoexjXOtCHGL9yJXukf6qqdzXgV3J7Eym+SL+AXDb02HP7XQzlyZuB/n3pnn6IHKcI24oK1ZK5/94QARGOYyCduDvUA++gbpB3PgR+ZM92t7V4Xig9j2k63O3qHQuZD2TcZ6QMHXSSccTjtzvFmHLLU32jw1HTcaA72Ge3Dq7ytKH0OIS7c361GT6cKl4MxTbgt0Ms1qD3rbF2F20+AyM01SLzTiYWl7CSdxemNsg7pPi30mkw31SntfIvgsA===pNMD-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 3 Apr 12:52 +0200
Re: [bug#40264] Linphone: 4-add-bctoolbox-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200403125202.68d5d62a@scratchpost.org
Hi Raghav,
thanks for the patch!
Pushed to guix master as commit bfaabfebac0fc6c3fba2ed4358556b393ef535ce.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6HFVIACgkQ5xo1VCwwuqXoPwgAhTqeoa+MhgoYQghghEfUjEy0JYfKTbsoJ0B1+fyWbrS3mCmd2tJCOxhKPDWG1QPrvUbh5Z64axpfgwrdSWfgQ1XUQP5OpgqLFY3I4L+neB91j7MpSl7FEEfmJNSdG0GRljccAgx/OUkHJGSHOMqCoPPyj7r7HuxLciGbfy8WLHgiGye1EMVHH38XjkjdRfhAEYUWGRaq/UrDKxeRz4HifL7+SRdGUKWFqpIL2llrM+FtV6x1vHGVM7zJLbo+yXTC8vdhndvw2FWBSF+zehIjAJelF37oAI61Bzxb1smlcb7YL1hiSWPkTwTA7SaAcoMugAhXHPuclE4TqD2zVaoNyw===UvIP-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 3 Apr 12:52 +0200
Re: [bug#40264] Linphone: 5-add-belr-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200403125217.37841377@scratchpost.org
Hi Raghav,
thanks for the patch!
Pushed to guix master as commit 63c73bcad0ccd44363611162010b24e2b60629f4.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6HFWEACgkQ5xo1VCwwuqVcxggApOSJ26mnkJyJYKM1gcoPjoIgTTrZiHxH0ZjiSAXLyuZaishQvGcnZg7Whs8gfb8Rvaua9R4VtzqJaEsSQqoDMHG9cRFEFtzz2QimbS7PJH65YRY4Jmg8Cv+8nKBVKjCQDLh3/ENYEpKF7e7lWH34CE9mgJOqg6ETwROZCYGeY1Xa5Qo+hZaZPjYwNzU4KOYcCMxAl+6KDeLOm4V4cHSUFhSQ6GLr8EY2mcu1im3+GW4Vk/KvQfMwbIMWtkIM6DzH7wbY548RKh4gmSieLZ6q5e8bOsZROu/BjhAZ3ZtMVNJ8mwKMISwWmv5cViHxp3NOt0ymOdM8CIHV3KtpF+figw===xoba-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 3 Apr 12:52 +0200
Re: [bug#40264] Linphone: 7-add-bcmatroska2-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200403125230.034ac114@scratchpost.org
Hi Raghav,
thanks for the patch!
Pushed to guix master as commit 28f6f1e0da2481ce854a893d72d6bdfc5ce97952.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6HFW4ACgkQ5xo1VCwwuqXz/Qf/QAp5+tfiibwiBtlfMH++cnNqrDGAxxK8YHGMj5u/1vxEAz8kB+2ZJsCgc2uFxk9ye+DLf5N235hfGi9dIwGfOrPtCHDGmUdli0rwzMb8eS3iJ160UDXqnEX0RbgLkRHdn4JhlPBkcunHuJvCJ2UFXBTC/7RqzUid3VIbJNYHb1CrqrrVrVfNJqrCsj5pcsYSX6EZGAOKU8SG8SATP43tHfKktYaVc4L/rzPrOrBezSd2kHs3wL1lPqVNWOJUohqyiazOrh2Vza9HbOeN1bFx61h8bS8ltz6Qg2y/GGy88lZ8HV271iC50g0Z6OinC+OxIQvhpvw4CgyKVF3Epi7QoQ===Q5OJ-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 3 Apr 13:01 +0200
Re: [bug#40264] Linphone: 8-add-bcg729-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200403130146.671feef9@scratchpost.org
Hi Raghav,
thanks for the patch!
Pushed to guix master as commit 02d5d72cfc525e1ec53a02f3a201872f261c7c37.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6HF5oACgkQ5xo1VCwwuqUGmgf/Q00X/LIcZHXoaI8TPgkKEk4UUYGr0Oe/wb0EfgE0i5Yd9xg55VqqzjKnWXe598hEZO9sCKaOzcKLR6X8EmPZG6QIqnsMDeYkMCsTGVmWdx0SvyJooGqGie7cQAZ+CWClkwLqtPnUC0RghqTJBSITiiL0g68WoeZWeySAKufN7diJSmvDVU+PNrmtdOEPiSRsPARSGC/rVKLFJPVMLFh8u83fOlwbrBKy/4pgFHSU/xc4Ey3DFliX705zmvsUqT9+oXBbDL78PYKPmkbTPiASVHrRVHQaXQGUWymVSEDSe2mUrRtQUkA1xGytRPrp4dEJpkJAW+yHOkmWLjbKqwuaAw===GAgZ-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 3 Apr 13:02 +0200
Re: [bug#40264] Linphone: 9-add-ortp-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200403130246.2f818ff7@scratchpost.org
Hi Raghav,
thanks for the patch!
Pushed to guix master as commit 13f401d3665a246f5e7d3a9076fdedfe744dd3fd.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6HF9YACgkQ5xo1VCwwuqUMWwf/VybTw9j4Y4pskzqdK5KJOmXfFl2mReqQqM5EowWrDwOkj0PdoN/G5Dp8w+TBpNyjiAkVqWVUF8enWMMzDxI+dy0dKAgBGfpJXfJkST5SUJTDkKR11XuLGeOIIVv2CSCN3JNDPD/jmSTJuaNnBPzvwJvz1L16mQcm6qcr6hK7zV8bNFYjJi5hjN2G8CM1gLnpx9tAnub11quSw/2GC5DCoy6v2Lh2bJh2YrEMXNV/nawqTQDotylBG/N4qgdrdV2RopVbaJ+xj+hXgqJdEDrlVhkNtMmgf4ksCs/fBWMW7Dd0mwuC0wllDTx8b5OaarviaB4wwrjJD3ZpOyyCpG6cyw===z5k7-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 3 Apr 13:09 +0200
Re: [bug#40264] Linphone: 10-add-bzrtp-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200403130931.494c6a5a@scratchpost.org
Hi Raghav,
thanks for the patch!
Pushed to guix master as commit f3697439cd226f769cdc5ce962f08ad3a1a8bfd6.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6HGWsACgkQ5xo1VCwwuqWG2Qf7BIrIrBRY2TDXr3PlXwmXEp37Ien/1IWMezbIc2FHSuPz+Sm6BsUJwR5oK7T8OnULlDW3FV0VvHzJI7H1aXRYFp1vmtumOLtHLrQQdu2wFFvh06SCCipUs59EUvCzHoGyku88nDBGTEy55eLwfmqjEKDGwzQa046LN9oJkGCTv3mHyUazmSQrEYc+d3vr0MLLdKh6U1+JSW56IWAf2l/yoM7AoSiBRpZcmD70tuBVxbm0puZupN1eBcQ1nGaLUQfZhmSBSB5HohdVoVahjSa5baJQXci+24KtUdYNZ81Tpk2ZOMcdeM4ypg21HHNRvmteRcbkLCjYEy/zikCSfV2G3A===x7i5-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 3 Apr 13:34 +0200
Re: [bug#40264] Linphone: 12-add-belle-sip-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200403133435.57d546b5@scratchpost.org
Hi Raghav,
On Thu, 02 Apr 2020 17:19:29 +0000"Raghav Gururajan" <raghavgururajan@disroot.org> wrote:
Toggle quote (2 lines)> 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 2001From: Raghav Gururajan <raghavgururajan@disroot.org>Date: Wed, 1 Apr 2020 20:42:33 -0400Subject: [FIXME 2/2] gnu: Add belle-sip.Tags: patch
* gnu/packages/linphone.scm (belle-sip): New variable.
Signed-off-by: Danny Milosavljevic <dannym@scratchpost.org>--- gnu/packages/linphone.scm | 42 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 42 insertions(+)
Toggle diff (62 lines)diff --git a/gnu/packages/linphone.scm b/gnu/packages/linphone.scmindex 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)))))+ (inputs+ `(("antlr3" ,antlr3-3.3)+ ("antlr3c" ,libantlr3c)+ ("bctoolbox" ,bctoolbox)+ ("java" ,icedtea)+ ("zlib" ,zlib)))+ (synopsis "Belledonne Communications SIP Library")+ (description "Belle-sip is a modern library implementing SIP transport,+transaction and dialog layers. It is written in C, with an object-oriented+API. It also comprises a simple HTTP/HTTPS client implementation.")+ (home-page "https://gitlab.linphone.org/BC/public/belle-sip")+ (license license:gpl2+)))
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6HH0sACgkQ5xo1VCwwuqXpeAf/fOvoBrSHAqTgLSAaww/E0Nl/QwiyzjIfGnr8YzAfdW5h026VKs/Flls5C4d32cESkCwOSDWHfk83C/0+Q9US8y2sTzS3aSZnd+Ocv2eKj/700wqaH1nJ86QbRG+ozrvigBRcA8IIXISCtH+fEqw+hFYQzhnmrgKgadU/0xMWRi0qH0KKqyRhNbRSO9+/PMfJKA/b3A3hTUGs8mYsDcZUffccqwdv576POEd+Tk0gwYGHH+JxFIenPgK8UNFvZahSIXoPqgZKFVHWXO7tXhpqCqgp/eWLQ687PJovJ80jbMPXpGaPInIOlCvvYRLtpn0yb2NqRnAJc6BWW1yCh9esgg===PfBy-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 3 Apr 13:38 +0200
Re: [bug#40264] Linphone: 11-add-mediastreamer-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200403133827.52d529cb@scratchpost.org
Hello everyone,
On Thu, 02 Apr 2020 00:20:04 +0000"Raghav Gururajan" <raghavgururajan@disroot.org> wrote:
Toggle quote (4 lines)> + (list
> + ;; For fixing definitions.
Yes, but why? What would happen if we didn't do it?
Toggle quote (7 lines)> + (origin> + (method url-fetch)> + (uri> + (string-append "https://gitlab.linphone.org/BC/public/mediastreamer2/commit/"> + "de3a24b795d7a78e78eab6b974e7ec5abf2259ac.diff"))> + (file-name "mediastreamer-msfactory.patch")
I'm not sure what the outcome of the latest discussion on carrying patches was.Should we carry patches in Guix now or fetch them from external source?
Toggle quote (2 lines)> + "-DENABLE_GSM=NO" ; Need recompilation with -fPIC
Uhhhhhhhhh if that's a library, it should have been compiled with "-fPIC", no?
Which library would need recompilation with "-fPIC"?
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6HIDMACgkQ5xo1VCwwuqWi+wf/WKVkGAm45SRnFevAkc1As6dz/Fs/QrwDL60LXsRonXWC0GgxHXI0zIw0HeJikqOxFq/AskbTFnV1cZwhGPPbYIVywTH8aJiXAYsrsp4bVXs26u8cOiOdzjaAE2UKmxuXaH5Fx+QozHOElNZRk0rlZcDBPE40cA2SJyLeYLYRG3OULCIZ1JfXZqKEFF3BP0pgDOiKz8dXDRoc7Pa7HFl4jTbmoO+rabjWYlEe4H6fX5gX/TPbmhKA+PIzVCzk9oKpl7rPE00LU2vYCQ3vv9DKzHkIaexbbwReU/FT92Oe4yfHtT9WHELP0ZcYZ0wVFLl6i4LgkMLfoOQMg7huTLQLIg===OdCq-----END PGP SIGNATURE-----

R
R
Raghav Gururajan wrote on 3 Apr 16:30 +0200
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
bd99706f2abe3a1b4217235ee954385e@disroot.org
Hi Danny!
Toggle quote (6 lines)>> + (list>> >> + ;; For fixing definitions.> > Yes, but why? What would happen if we didn't do it?
Otherwise, we get `-Werror` flag related compilation errors.
Toggle quote (10 lines)>> + (origin>> + (method url-fetch)>> + (uri>> + (string-append "https://gitlab.linphone.org/BC/public/mediastreamer2/commit">> + "de3a24b795d7a78e78eab6b974e7ec5abf2259ac.diff"))>> + (file-name "mediastreamer-msfactory.patch")> > I'm not sure what the outcome of the latest discussion on carrying patches was.> Should we carry patches in Guix now or fetch them from external source?
I think it's good as long as we use hash to verify. :-)
Toggle quote (6 lines)>> + "-DENABLE_GSM=NO" ; Need recompilation with -fPIC> > Uhhhhhhhhh if that's a library, it should have been compiled with "-fPIC", no?> > Which library would need recompilation with "-fPIC"?
I am not sure about this one. Since it is a feature, I was planning on to deal this later and disable gsm for now. I hope it is okay.
Regards,RG.
R
R
Raghav Gururajan wrote on 4 Apr 01:47 +0200
Linphone: 14-add-linphoneqt-v6
(address . 40264@debbugs.gnu.org)
c1a8272fa04d13d0bf8d4754f24cc33e@disroot.org
R
R
Raghav Gururajan wrote on 4 Apr 01:50 +0200
Re: Linphone: 14-add-linphoneqt-v3
(address . 40264@debbugs.gnu.org)
d312c2264580a6a5254246ebe212ef2b@disroot.org
Hello Maxim and Danny!
Toggle quote (22 lines)>> At this point for linphoneqt a.k.a linphone-desktop, I am facing following issues.>> >> When I build *without* `-DENABLE_DBUS=YES`and run the program, I get:>> >> QSocketNotifier: Socket notifiers cannot be enabled or disabled from another thread>> QMutex: destroying locked mutex>> >> When I build *with* `-DENABLE_DBUS=YES` and run the program, I get:>> >> Segmentation Fault (Core Dumped)>> >> I think the following patch is relevant, but when I use it, doesn't get successfully patched during>> the build.>> >> https://gitlab.linphone.org/BC/public/linphone-desktop/commit/9cf08623e3092fa19366e5c07fbe06898a59f0>> 9.diff>> >> Any ideas on how to fix this situation?> > So I fixed "QSocketNotifier" error in version 4 patch (14-add-linphoneqt-v4). I still don't know> how to fix "QMutex" error.
I have now fixed the "Qmutex error" in version 6 patch (14-add-linphoneqt-v6).
Regards,RG.
M
M
Maxim Cournoyer wrote on 4 Apr 19:36 +0200
Re: [bug#40264] Linphone: 12-add-belle-sip-v2
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)
87a73r6tlr.fsf@gmail.com
Hello Danny and Raghav,
Danny Milosavljevic <dannym@scratchpost.org> writes:
Toggle quote (75 lines)> Hi Raghav,>> On Thu, 02 Apr 2020 17:19:29 +0000> "Raghav Gururajan" <raghavgururajan@disroot.org> 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 <raghavgururajan@disroot.org>> 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 <dannym@scratchpost.org>> ---> 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). Perhapswe should just try to package the latest master? Their git historysuggests many bug fixes, including a memory leak.
Maxim
R
R
Raghav Gururajan wrote on 4 Apr 20:18 +0200
(address . 40264@debbugs.gnu.org)
8920d9146da7bf7f603d67c63450e2f6@disroot.org
Toggle quote (5 lines)> 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.
In their mail list, one of the dev mentioned that there is going to be a release this year. Also, for consistency, I think it is better that we stick to release tarballs and apply critical patches from master. :-)
Regards,RG.
M
M
maxim.cournoyer wrote on 5 Apr 06:08 +0200
Re: [bug#40264] Linphone: 11-add-mediastreamer-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)
87mu7q3787.fsf@hurd.i-did-not-set--mail-host-address--so-tickle-me
Hello Raghav,
"Raghav Gururajan" <raghavgururajan@disroot.org> writes:
Toggle quote (10 lines)> Hi Danny!>>>> + (list>>>>>> + ;; For fixing definitions.>>>> Yes, but why? What would happen if we didn't do it?>> Otherwise, we get `-Werror` flag related compilation errors.
Could you please clarify that in a comment above the patch?
Toggle quote (12 lines)>>> + (origin>>> + (method url-fetch)>>> + (uri>>> + (string-append "https://gitlab.linphone.org/BC/public/mediastreamer2/commit">>> + "de3a24b795d7a78e78eab6b974e7ec5abf2259ac.diff"))>>> + (file-name "mediastreamer-msfactory.patch")>>>> I'm not sure what the outcome of the latest discussion on carrying patches was.>> Should we carry patches in Guix now or fetch them from external source?>> I think it's good as long as we use hash to verify. :-)
The problem is that metadata or format of auto-generated patches maychange on a whim. In past discussions it was decided best to not usethem (that is, it's best to embed a copy of it in Guix sources) [0].
[0] https://lists.gnu.org/archive/html/guix-devel/2018-11/msg00554.html
I'll post more comments in your original message.
Maxim
M
M
maxim.cournoyer wrote on 5 Apr 06:16 +0200
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
87k12u36ua.fsf@hurd.i-did-not-set--mail-host-address--so-tickle-me
Hi Raghav!
"Raghav Gururajan" <raghavgururajan@disroot.org> writes:
Toggle quote (48 lines)> From 3cf73361203f7833c61a24f69b9ab4bb4da358ce Mon Sep 17 00:00:00 2001> From: Raghav Gururajan <raghavgururajan@disroot.org>> 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 perupstream.
Toggle quote (21 lines)> + (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 andmake the package more resilient in the face of future compiler upgradesin Guix. No more need for that patch :-).
Toggle quote (7 lines)> + (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 weshouldn't add SRTP support since we should already have the requireddependencies.
Also, I see there's a ENABLE_UNIT_TESTS=YES option for unit tests, whichwe should enable and try to run.
Toggle quote (3 lines)> + "-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 telephonyfocused software :-).
Toggle quote (36 lines)> + "-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'). Pleaseedit them out.
Maxim
M
M
maxim.cournoyer wrote on 5 Apr 06:18 +0200
Re: [bug#40264] Linphone: 12-add-belle-sip-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)
87h7xy36rk.fsf@hurd.i-did-not-set--mail-host-address--so-tickle-me
"Raghav Gururajan" <raghavgururajan@disroot.org> writes:
Toggle quote (10 lines)>> 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.>> In their mail list, one of the dev mentioned that there is going to be a release this year. Also, for consistency, I think it is better that we stick to release tarballs and apply critical patches from master. :-)>> Regards,> RG.
Sounds reasonable! Thanks for the information.
Maxim
R
R
Raghav Gururajan wrote on 5 Apr 09:09 +0200
(address . control@debbugs.gnu.org)
a4e6bfc5c3d3db8eb8801ba7ffd7d6e3@disroot.org
block 40264 by 40445
R
R
Raghav Gururajan wrote on 5 Apr 09:28 +0200
(address . 40264@debbugs.gnu.org)
8d37c9308eddb3c687689ed614c86386@disroot.org
R
R
Raghav Gururajan wrote on 5 Apr 09:29 +0200
Re: [bug#40264] Linphone: 11-add-mediastreamer-v2
(address . maxim.cournoyer@gmail.com)(address . 40264@debbugs.gnu.org)
a6350dd92f1f6ed04a2e41d2be5f0dd0@disroot.org
Hi Maxim!
Toggle quote (135 lines)>> From 3cf73361203f7833c61a24f69b9ab4bb4da358ce Mon Sep 17 00:00:00 2001>> From: Raghav Gururajan <raghavgururajan@disroot.org>>> 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 apowerfuland>> lightweight streaming> > There are some strange characters in there (around 'powerful'). Please> edit them out.
I fixed everything in 11-add-mediastreamer-v3. :-)
Regards,RG.
M
M
maxim.cournoyer wrote on 5 Apr 17:02 +0200
Re: [bug#40264] Linphone: 11-add-mediastreamer-v3
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
87eet22cx4.fsf@hurd.i-did-not-set--mail-host-address--so-tickle-me
"Raghav Gururajan" <raghavgururajan@disroot.org> writes:
Toggle quote (14 lines)> From c9f9ecc04aaff5c48490e3fc88b0b2b4ef79eafb Mon Sep 17 00:00:00 2001> From: Raghav Gururajan <raghavgururajan@disroot.org>> Date: Sun, 5 Apr 2020 03:25:10 -0400> Subject: [PATCH 3/3] gnu: Add mediastreamer2>> * gnu/packages/linphone.scm (mediastreamer2): New variable.> ---> gnu/packages/linphone.scm | 78 +++++++++> .../patches/mediastreamer-msfactory.patch | 31 ++++> .../patches/mediastreamer-srtp2.patch | 155 ++++++++++++++++++> 3 files changed, 264 insertions(+)> create mode 100644 gnu/packages/patches/mediastreamer-msfactory.patch> create mode 100644 gnu/packages/patches/mediastreamer-srtp2.patch
Is the mediastreamer-srtp2 patch really required? I could build thepackage successfully (with libsrtp) without it. Please rename thepatches mediastream2 to reflect the updated package name (the '2'suffix).
Also, patches must be registered in gnu/local.mk (in the dist_patch_DATAvariable), otherwise they won't be included in the sources of Guix.
Toggle quote (62 lines)> diff --git a/gnu/packages/linphone.scm b/gnu/packages/linphone.scm> index 0e32500b07..a3ec664a40 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,69 @@ 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 mediastreamer2> + (package> + (name "mediastreamer")> + (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 updating definitions.> + ;; Fixes "-Werror" flag related errors and MS2_GIT_VERSION error.> + (search-patch "mediastreamer-msfactory.patch")> + ;; For srtp2 compatibility.> + ;; Fixes locating srtp2 files.> + (search-patch "mediastreamer-srtp2.patch")))))> + (build-system cmake-build-system)> + (arguments> + `(#:tests? #f ; No test target> + #:configure-flags> + (list> + "-DENABLE_STATIC=NO" ; Not required> + "-DENABLE_BV16=NO"))) ; Not available
Could you investigate the ENABLE_UNIT_TESTS=yes option?
Toggle quote (34 lines)> + (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
I still see strange characters here: ^ ^
Otherwise, LGTM.
Maxim
D
D
Danny Milosavljevic wrote on 5 Apr 20:29 +0200
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200405202940.0408231a@scratchpost.org
Does
ENABLE_STRICT=NO
instead of mediastreamer-msfactory.patch work or not?
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KI5QACgkQ5xo1VCwwuqVjowf/eKd9GXnYTlYIMnIq/zVbbjc0o2LbEL+ozmRTVxal2KFk8eFg+j1c6YMraShEH3viH0kmpBPSmfA4fCJDrPzcrQwS3ihYndSfLphk+MvKpNDXsQW7OccSaVmISDsd4c0RodYsisjrlGR21bU1Mr1iKfH9fnR9t2cluFdoaTkz3a6pM6Du+OUFbVTNWJsXx/M9S/tWE9FLJobtcywXn52PwwUFsUE4G7F/o7coc5ca+sFFz/XPd+Sd8oCEbI1s2/wWXAWHiUvMvEBYsBdEmDG9OSozPvABR3LIVOu4cTKfx4pQbhuykldwZVCDRL5DeLdkr0g0d/wumZKSFgm4vTOC1Q===OYqw-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 5 Apr 21:06 +0200
Re: [bug#40264] Linphone: 12-add-belle-sip-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200405210614.6d41acb1@scratchpost.org
Pushed this patch to guix master as commit e7859da11371e9bd0fffc5235b9e980cccf4b239.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KLCYACgkQ5xo1VCwwuqWvWAgAjaqXLF3BbtCnwsbrF8C1gSxlMWi6yFu7bqvBsDhn9rNFXoBhmlM2gNUHFvzmupLRD2oeqcP0f+CTwnLfenwWVEojtzp3o1GSZDjh83xZeDzDeGh03VSPMDdReyp+t9jeZNqw1e5Dw5nRW5KzVy2Vve+Y+2mhU6FvpxTMmN115lmRPEHRHAKqT5n2nRUqT6jMHyh/w9ETEXSSZYYTEdipk6MKYqNooiE+LWEBQR6l0eS3F1fotFJs7HaiDuX3Pj8H81k4yRL3Tvz/vldARezTXo+nufFw5kfgwaLx0nxE6wwuLVBo5SqcGxSY4+7D4UmzgXFjr1/zwtyt+vIgCKQDlg===REIW-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 5 Apr 21:41 +0200
Re: [bug#40264] Linphone: 11-add-mediastreamer-v3
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200405214134.3558655a@scratchpost.org
Hi,
On Sun, 5 Apr 2020 20:29:40 +0200Danny Milosavljevic <dannym@scratchpost.org> wrote:
Toggle quote (6 lines)> Does > > ENABLE_STRICT=NO> > instead of mediastreamer-msfactory.patch work or not?
Yes.
So added mediastreamer2 without any patches to guix master ascommit b6b3a6a6c4f00815da576912907c7bdbf02ca11a.
If the patches are required after all (for example the srtp2 one--is itcorrect that this one updates the dependency from srtp1 to srtp2?), wecan still add them.
I've also renamed the package to mediastreamer2 like upstream does(I don't like the name so much--but upstream chose it, so be it).
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KNG4ACgkQ5xo1VCwwuqV8BAf/Z+L/B7kgTC4tNZPxS7ejJ7bDnQ7/X950PMaPL0pQiKtGhi9GCvJMzkcs5vlfOCR3ejzpQQ0CH+HQMR7CNPGMG4mOhcqszt5d+F6KMUyncRCFVZpUxQKHygoCXKlGiODkYdgvleOd4Ov/MDEkKB6HUSSJvEj8Z2qPcrEZi7utBe5LH1Ah0rPlkIuCm34PpB+KZ3oYuSCKDWA/KyPjZJdoJl0r6DVosW9DLNsJC2ahpF1da5CmuZj04SxyKzxIQ3D65zXfGhiksA7SBBH22hiHjHoLb3o20iv0teNJtITODPtS1+hp1QogvHbEYh9w+UMe7Kb5ii/fFvqW15GuqRgUHA===t3yO-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 5 Apr 21:49 +0200
Re: [bug#40264] Linphone: 13-add-linphone-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200405214954.250f20a3@scratchpost.org
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KNmIACgkQ5xo1VCwwuqXERAf/QxwOiRzDpapmmD1N7O0uoofTzdXkLGZnMiUJFErrafZpy2gcnY1R4qhClf9IedrvNftF4G3FYSzKyjxZlj3tZ7aZrGPfAl9SJFug9wjnR2mo581AlA0CvtrN2c2ZKuqJWnrLbewUYLrkuqQL8MObgae42C82CzdlJjTI3lX9TeMPF7ELb+2uXqLSCec1JJg/05bDZ1YUmbAuRk/nGVQKErDDWd7XsURSveoLKQIrUWicY+xapGv8GDDuDmW4VQmXPj81/zE9DdEbUOdTcBWZes3Nb26Ejmm8UUwU+/eXYl92Yo1INxK2X28zCCbVg0uqqObCZ9qQhuwxKeJ94zMWag===gxJz-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 5 Apr 21:57 +0200
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200405215701.1b13def1@scratchpost.org
Renamed package to liblinphone and pushed this patch to guix master ascommit 9732910dd336e9bca56991a0630619bc1578d2af.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KOA0ACgkQ5xo1VCwwuqXGhQf+OwAWiIR6kLrZimWPpE0TIozuTOgjpt8YOidXx/U2l9e/7L2pBvzDDAjIqOqyKM4PxSvumpgDrOM6xu+pILre/vx5gl7C/Zt5cvM2P9Jb5agFWnqDpU1IfOl0YpuTppuTasJr3x532eUPEe92gQPfgzC4sz42Wp5Vfv5R7qpjddi5M96NK+Uywm1WimMh7OdDzFLSiMC/sBj/3xL7JZwo+HG+XarsmEfCSk5H+FM0xpvPRQ7ZwemHND+a0pJ7fDXc27YJrBXjC+1Rn2AaJc5pKQVstrxitJQvstocOmun6z0xGGO2rDOMEX1qXop2musA2jUxQEWBoYvBNcprM7pR/w===oMdN-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 5 Apr 22:05 +0200
Re: [bug#40264] Linphone: 13-add-linphone-v3
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200405220543.688c94cd@scratchpost.org
Pushed to guix master as commit 02b32cdda68495344d8e5521db5aeaf5add03797.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KOhcACgkQ5xo1VCwwuqUKdAf+OWn0I9lH49QiSJzTeB4vKY8gPVV099DkwRbQodqlHNm6GKp7Gi7mK9dNSuAVGzwHFsGHjxXuVNDD+RwgpF1KZQaxyQ9MPHthHdhvql5H0PS0ugowlnxdcUF0X7DeOPyQikElT1m7HYzx07XVeqGLAjro9LH/f1YH3/QSwI3jqe4jf2IUL4vo9tvAwVnyUlpK7AdrRT1cK41uPlbvJWXbwMR5UF5hNhKkL9OzMzH7YGzq6mVpuQXVS0lKHkl8bCRrfU67ZBWdW55vLuA4JItgWrtwcLUUPq8rO3U1diW2EMLLZ11gY6zN61uI34sQPNW0elCYgA9oFg9L2+YNYEG0mQ===Hjry-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 5 Apr 22:19 +0200
Re: [bug#40264] Linphone: 14-add-linphoneqt-v6
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200405221932.50481199@scratchpost.org
This package is not reproducible. Probably something simple amiss.
diffoscope --exclude-directory-metadata /gnu/store/p9186fvq27dawzmij3q2ibllhdd07c6a-linphoneqt-4.1.1 /gnu/store/p9186fvq27dawzmij3q2ibllhdd07c6a-linphoneqt-4.1.1-check--- /gnu/store/p9186fvq27dawzmij3q2ibllhdd07c6a-linphoneqt-4.1.1+++ /gnu/store/p9186fvq27dawzmij3q2ibllhdd07c6a-linphoneqt-4.1.1-check├── bin│ ├── linphone│ │ ├── /gnu/store/mx2bgrpxkbdjsmhlxp9a30hbzcilk4cn-binutils-2.32/bin/readelf --wide --decompress --hex-dump=.rodata {}│ │ │ @@ -190213,17 +190213,17 @@│ │ │ 0x00801020 02000a00 12000309 3b093c09 3d000109 ........;.<.=...│ │ │ 0x00801030 3e000100 02010c05 56000200 0c000300 >.......V.......│ │ │ 0x00801040 f3023706 03000100 03004c00 4d045700 ..7.......L.M.W.│ │ │ 0x00801050 00000000 00000000 00000000 00000000 ................│ │ │ 0x00801060 00000000 00020000 00010000 00010000 ................│ │ │ 0x00801070 00000000 00000000 00000002 00000002 ................│ │ │ 0x00801080 00000002 00000000 00000000 00000018 ................│ │ │ - 0x00801090 00010000 00010000 00000000 01714bf9 .............qK.│ │ │ - 0x008010a0 65980000 00220001 00000001 0000280f e...."........(.│ │ │ - 0x008010b0 00000171 4bf96598 00000000 00000000 ...qK.e.........│ │ │ + 0x00801090 00010000 00010000 00000000 01714bf7 .............qK.│ │ │ + 0x008010a0 f3330000 00220001 00000001 0000280f .3..."........(.│ │ │ + 0x008010b0 00000171 4bf7f333 00000000 00000000 ...qK..3........│ │ │ 0x008010c0 000904eb 98c3006c 0061006e 00670075 .......l.a.n.g.u│ │ │ 0x008010d0 00610067 00650073 00020000 06be0065 .a.g.e.s.......e│ │ │ 0x008010e0 006e0002 000006d2 00660072 00000000 .n.......f.r....│ │ │ 0x008010f0 00000000 00000000 00000000 00000000 ................│ │ │ 0x00801100 0000280b 00008688 789ce57d 0b741cc5 ..(.....x..}.t..│ │ │ 0x00801110 9568c992 a5d16834 92654996 8531c236 .h....h4.eI..1.6│ │ │ 0x00801120 c626c618 639b9f31 8cf58905 922ceb63 .&..c..1.....,.c│ ├── linphone-tester│ │ ├── /gnu/store/mx2bgrpxkbdjsmhlxp9a30hbzcilk4cn-binutils-2.32/bin/readelf --wide --decompress --hex-dump=.rodata {}│ │ │ @@ -190443,17 +190443,17 @@│ │ │ 0x0080be80 02000a00 12000309 3b093c09 3d000109 ........;.<.=...│ │ │ 0x0080be90 3e000100 02010c05 56000200 0c000300 >.......V.......│ │ │ 0x0080bea0 f3023706 03000100 03004c00 4d045700 ..7.......L.M.W.│ │ │ 0x0080beb0 00000000 00000000 00000000 00000000 ................│ │ │ 0x0080bec0 00000000 00020000 00010000 00010000 ................│ │ │ 0x0080bed0 00000000 00000000 00000002 00000002 ................│ │ │ 0x0080bee0 00000002 00000000 00000000 00000018 ................│ │ │ - 0x0080bef0 00010000 00010000 00000000 01714bf9 .............qK.│ │ │ - 0x0080bf00 65980000 00220001 00000001 0000280f e...."........(.│ │ │ - 0x0080bf10 00000171 4bf96598 00000000 00000000 ...qK.e.........│ │ │ + 0x0080bef0 00010000 00010000 00000000 01714bf7 .............qK.│ │ │ + 0x0080bf00 f3330000 00220001 00000001 0000280f .3..."........(.│ │ │ + 0x0080bf10 00000171 4bf7f333 00000000 00000000 ...qK..3........│ │ │ 0x0080bf20 000904eb 98c3006c 0061006e 00670075 .......l.a.n.g.u│ │ │ 0x0080bf30 00610067 00650073 00020000 06be0065 .a.g.e.s.......e│ │ │ 0x0080bf40 006e0002 000006d2 00660072 00000000 .n.......f.r....│ │ │ 0x0080bf50 00000000 00000000 00000000 00000000 ................│ │ │ 0x0080bf60 0000280b 00008688 789ce57d 0b741cc5 ..(.....x..}.t..│ │ │ 0x0080bf70 9568c992 a5d16834 92654996 8531c236 .h....h4.eI..1.6│ │ │ 0x0080bf80 c626c618 639b9f31 8cf58905 922ceb63 .&..c..1.....,.c
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KPVQACgkQ5xo1VCwwuqUUVAgAhj4XYmllL09zmrrpbI4sdlJn3ojIGJpksfKLXlzMJ7Yo93ankRQ0fgTFZ1sWtnGiACsT8DGnKyWv3rtPzGqrS8+2gSKSNWkk4ltvzmwCZ6NpweO3DoL7eBCZno8mq7AQQHOh/bN2ngK9BTQ48ohr0Tx7ZI5cbcW3gnI5V8dHkxpReMECZoxM8axBD/HBTy19sv/KfKBVAgA7/2EJtQuJVFUs/ZEJa5lLbtEhUXuBHmUOuquOMDDZ77IpQMswo9mTKD5d6346jXibw1OQsD6RYIdOWt9hTXLxqvqiFNqur6sA9Hj4hsSeYOv3U1jaNI37sXWAowKTKI9G42BusYpVZg===cV8T-----END PGP SIGNATURE-----

R
R
Raghav Gururajan wrote on 6 Apr 00:52 +0200
Re: [bug#40264] Linphone: 11-add-mediastreamer-v3
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
e29fef7b821255b1dee9fe0c6bce81d3@disroot.org
Hi Danny!
Toggle quote (3 lines)> So added mediastreamer2 without any patches to guix master as> commit b6b3a6a6c4f00815da576912907c7bdbf02ca11a.
Thanks for the push.
Toggle quote (4 lines)> If the patches are required after all (for example the srtp2 one--is it> correct that this one updates the dependency from srtp1 to srtp2?), we> can still add them.
Oh no, that patch is required. MS won't build with srtp enabled, without it. You can see "MediaStreamer will be built without SRTP support". That patch fixes search and locate of srtp2 version of include and lib files.
Toggle quote (3 lines)> I've also renamed the package to mediastreamer2 like upstream does> (I don't like the name so much--but upstream chose it, so be it).
Shall we please keep the name "mediastreamer" for package and "mediastreamer2" for variable? The release tarballs have not been renamed. We'll change the package name when it is changed in releases by upstream. This will consistent with other packages in linphone.scm.
Regards,RG.
R
R
Raghav Gururajan wrote on 6 Apr 00:54 +0200
Re: [bug#40264] Linphone: 13-add-linphone-v2
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
6704fe2d157417c1e07675a07a22c4ff@disroot.org
Hi Danny!
Toggle quote (3 lines)> How come that https://github.com/BelledonneCommunications/liblinphone/releaseshas 4.3.1 and we> have 3.12.0 ?
Releases tag in gitlab.linphone.org are for internal purposes. Official releases are done at linphone.org/releases/sources.
R
R
Raghav Gururajan wrote on 6 Apr 00:55 +0200
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
5bdb705d8b39d3710623dcda2a346a3d@disroot.org
Hi Danny!
Toggle quote (3 lines)> Renamed package to liblinphone and pushed this patch to guix master as> commit 9732910dd336e9bca56991a0630619bc1578d2af.
Thanks for the push :-)
Shall we please keep the name "linphone" for package and "liblinphone" for variable? The release tarballs have not been renamed (linphone.org/releases/sources). We'll change the package name when it is changed in releases by upstream. This will consistent with other packages in linphone.scm.
Regards,RG.
D
D
Danny Milosavljevic wrote on 6 Apr 01:05 +0200
Re: [bug#40264] Linphone: 16-add-msopenh264-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200406010523.23676868@scratchpost.org
Thanks for the patch!
Pushed to guix master as commit 376fc95e29b24609f3cb7fca024e93a37c0adbfe.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KZDMACgkQ5xo1VCwwuqUDZAgAmjSd8T9x3Nrcu2Yvbo83J71KI9f2hK/56lYxULg8rCE6dLXzq8SDlA7YnlpvehhfwYxFbxEltdSLjYdaqwZb3N2cWDXYoSz20GYeQsLdtJ8257XWKabtZF0NsNwrE60yrBu6nOMhgj4VOfbjxsTEWvuwxBa2pITIW0dG2V9Fd1xOJUdtHWbIi5c+1lSQLB4fvKsboSSfHgAtC06tg5LdRiMh4BNIPvEburKbNgnTOFeNQThCDB0CC0VmR0xi65QPqXhhneVYKSXHEZOx0M+7spKhAedm/HPos+Fsg/4jY2ha70nnqrYXMxTLtpLckM3Khxf52tlXd2T5tRqbM/xqNg===XntI-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 6 Apr 01:06 +0200
Re: [bug#40264] Linphone: 17-add-mssilk-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200406010600.7d000d3c@scratchpost.org
Thanks for the patch!
Pushed to guix master as commit f689d3d7d9ba3fa25b4839bdd1e4b98905f4869c.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KZFgACgkQ5xo1VCwwuqXcuQf/dYZEYEF5V/mJax3etoTuJXYv1v+UO65Bw9txFIXgk4swyWUNIvuYVQTy17G1K0/ogAldis6jyKMxKmQFqBXD6YY/KXs0MKUmABg1PDSwTOGKYjsybcSNnfXZKyrH3eZBtQSExyFZJefPM2hhJk++LgGYq5tArOybC9tKQejckrTUhm3EbbxK5qQA0JURNMQkzRHiK6mNL9vnqXPkl00mR2Hc6HmAs+P5OB6iHJaFcQxVykxyAnXA3vGbMHllZ8vXYlqiihjlf9wF0jPJSjuqfI8IrDTq4yZp37FlUlki8irh+fWSo2bJDSb+yDKfB6WJnPQccv/85W+M3UEZHq8rKw===dGFU-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 6 Apr 01:06 +0200
Re: [bug#40264] Linphone: 18-add-mswebrtc-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200406010636.6bf39642@scratchpost.org
Thanks for the patch!
Pushed to guix master as commit fe8f1c0b194de2fa7aa5265a4a0424e82bd11af1.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KZHwACgkQ5xo1VCwwuqUbTwgAjT6Syt99npOT6yBT088yX6e+qkgTv+si2cG7f0pVpbaN8FUp8bCBz3Up2b8vjwMDId/3j7q2uAyxf8ymAp5OvWzfFsLi6Itd90pC0b08KXHHhdy4lElkk8/0LkGLX3JvYkJ6REkwcCZ4IuwgsNgWqrwE2FD6/PSDPDs4WM635JjOVFIYw4PhSkSbrQPD2bXNNZVvru5r7gwGC+KsHJXx92iXZx+mjxdlSXVIuF+v6ZZW5rL+XZkzrEub8WWTlI2+xGsuJlHeq8wMKGl6AfYxDPjOm4tLPcJkmdQWGGvYg4epiyPqAEInlUbK86Awl8L+jKjMZn9t6sekzpuiZskPig===SPFP-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 6 Apr 02:03 +0200
Re: [bug#40264] Linohone: 15-add-msamr-v2
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200406020337.2ae7bf2a@scratchpost.org
Hi Raghav,
thanks for the patch!
Pushed to guix master as commit b89201695582351fa5fa183c7a1aa26162ca271e.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KcdkACgkQ5xo1VCwwuqUgPgf+Mc66SL/CyZsQgfVC6lHU9XuHeUPBIVQgUo/nIEkQXuG/sQrXO10iTS7/FIhtXL0RBN2pmGgwZLIktZ4A9jDNNbmVevE+gZusxw2mW6JBUt3SlMeXSz8DH/vbqqMUaJ/c5iKh4qOfzfqioCLhOIk//jAU6pNyFinc6wWB0tnMZkgmTBwsmnKvHZJCb+fqAQfi+Ws9QNbsJ8T6hhb+A9bSjce+hDzAiArsWyLf4Tz2eC+5iWoQAsHjyoxffITcJDr7Z5elk1CW6Ej/alvrnoy5fM2hr9o9ATfHFPfxxukFBl9fkNsX5mH6QqJs0RACIOTtCYY9GdWvzAwhw3SNHmLymg===V9SE-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 6 Apr 02:14 +0200
Re: [bug#40264] Linphone: 11-add-mediastreamer-v3
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200406021448.3835ff95@scratchpost.org
Hi Raghav,
On Sun, 05 Apr 2020 22:52:25 +0000"Raghav Gururajan" <raghavgururajan@disroot.org> wrote:
Toggle quote (2 lines)> Oh no, that patch is required. MS won't build with srtp enabled, without it.
Okay--added in commit 2c56aeaea94d3914acf79f9bbabeef1b3ec8aa07.
Toggle quote (2 lines)> Shall we please keep the name "mediastreamer" for package and "mediastreamer2" for variable? The release tarballs have not been renamed. We'll change the package name when it is changed in releases by upstream. This will consistent with other packages in linphone.scm.
I'll see what the others say--but I'm not opposed to use the other name.
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6KdHgACgkQ5xo1VCwwuqUo3Qf+JXGR7Mxwh9oemD9fFL0g7hLRw6CEdvHtc6+rJjaTow/wS0DSpNOAbHLqo3yT51w7AGC587h58T/HIl0/+0HsbEHq8KKVD42TkWt7c9eNAt/2cKyDvgN69JAY/L7dKzgrF8pdFumLSV8E0DujBHAN4ldfNaB32Wz3iqB18zrZbOV5XZVcuG02yMhINxBQ0cCUB9iKvKILw6oS6yIb8z3QNvzUxyMwsTKFVSSZkmYCXKbXAWFPrzYWkgCy0qpPi2HlwzFZbu0tShisfKZIyHUeMviGzECXhKpVpndYoJ8MOVlqKMzxWKmJ3Eu2CpzdjJ4R1W+5i7Nv7scAuPfVc7TkuA===WVpW-----END PGP SIGNATURE-----

R
R
Raghav Gururajan wrote on 6 Apr 02:28 +0200
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
3c47cbc6b4f15bc10b3c6e4c9e1b992a@disroot.org
Hi Danny!
Toggle quote (4 lines)>> Oh no, that patch is required. MS won't build with srtp enabled, without it.> > Okay--added in commit 2c56aeaea94d3914acf79f9bbabeef1b3ec8aa07.
Thanks so much :-)
Toggle quote (6 lines)>> Shall we please keep the name "mediastreamer" for package and "mediastreamer2" for variable? The>> release tarballs have not been renamed. We'll change the package name when it is changed in>> releases by upstream. This will consistent with other packages in linphone.scm.> > I'll see what the others say--but I'm not opposed to use the other name.
Cool!
Regards,RG.
M
M
maxim.cournoyer wrote on 6 Apr 04:40 +0200
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)
871rp12v6t.fsf@hurd.i-did-not-set--mail-host-address--so-tickle-me
Hello,
"Raghav Gururajan" <raghavgururajan@disroot.org> writes:
Toggle quote (14 lines)> Hi Danny!>>>> Oh no, that patch is required. MS won't build with srtp enabled, without it.>>>> Okay--added in commit 2c56aeaea94d3914acf79f9bbabeef1b3ec8aa07.>> Thanks so much :-)>>>> Shall we please keep the name "mediastreamer" for package and "mediastreamer2" for variable? The>>> release tarballs have not been renamed. We'll change the package name when it is changed in>>> releases by upstream. This will consistent with other packages in linphone.scm.>>>> I'll see what the others say--but I'm not opposed to use the other name.
mediastreamer2 is better. Otherwise we get a confused mix ofmediastreamer-2.16.1 / mediastreamer2-2.16.1 under share/, for example.The name really is mediastreamer2 in the code/doc, everywhere except infor the tarball file name.
I'll push some refinements I made today in a moment, mostly adding thetester binary and data to a 'tester' output (it comes with about 20 MiBof data files).
Great work!
Maxim
R
R
Raghav Gururajan wrote on 6 Apr 05:01 +0200
Re: [bug#40264] Linphone: 14-add-linphoneqt-v6
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
1955a54edbd008dd21a13b7986e87d6d@disroot.org
Hi Danny!
Toggle quote (2 lines)> This package is not reproducible. Probably something simple amiss.
Nice catch! Let me try to see what causes it.
Btw, is there everything else good with the package?
Regards,RG.
R
R
Raghav Gururajan wrote on 6 Apr 06:04 +0200
Linphone: 14-add-linphoneqt-v7
(address . 40264@debbugs.gnu.org)
48c4a04dff5589ada795e823ed045dee@disroot.org
R
R
Raghav Gururajan wrote on 6 Apr 07:56 +0200
Re: [bug#40264] Linphone: 14-add-linphoneqt-v6
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
8ebdc73d8b615ca2efd4cbc37844c075@disroot.org
Hi Danny!
Toggle quote (6 lines)>> This package is not reproducible. Probably something simple amiss.> > Nice catch! Let me try to see what causes it.> > Btw, is there everything else good with the package?
Hmm, I think it is something to with computing QML files.
R
R
Raghav Gururajan wrote on 8 Apr 09:01 +0200
Replace source URI with mirror.
(address . 40264@debbugs.gnu.org)
9dc9f24033787a66c6dbaffdfde0e791@disroot.org
R
R
Raghav Gururajan wrote on 8 Apr 09:29 +0200
Mirror and Formatting
(address . 40264@debbugs.gnu.org)
5707875209a8eadf81873cc902f88aa5@disroot.org
D
D
Danny Milosavljevic wrote on 10 Apr 23:47 +0200
Re: [bug#40264] Linphone: 14-add-linphoneqt-v7
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200410234755.71263869@scratchpost.org
Hi Raghav,
if we add gcc 5 here, do we need to do something similar tohttps://lists.gnu.org/archive/html/guix-commits/2020-04/msg00774.htmltoo ornot?
Why is only gcc 5 working and not gcc 7?
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6Q6YsACgkQ5xo1VCwwuqU3/wgAhwls71o1GCom5Wi2OBeeZoD8P9CO57eF7WSlEAtn7TaUe0uh53aOfW3dG07PaWprdJGCAIpUABRsJyQqg+R/Goxx5H3XrtIIXztIkeQxnGLYH3+qwmDIiB+LQLw+nMnBp0dIo3NlRlXfmLXHDRt9Te8JjiWjVC7aHkuaqn0A+kkKwAkB8/v5WBol7LQ+lrm/JQwTUEu3ZwpOno/4owX2O5ztU9YyMAnbMpcvuka6iLPb2fPvVm1OMBRiWHMgtwAijdvvrlhAyCQtYyew2OjUh+x1Si0v/gSapikcESDNYhEs/8K3Oa8uubxlElUu+Qn5NPxpMloI8ORTzL7dI7hq6Q===jKiW-----END PGP SIGNATURE-----

D
D
Danny Milosavljevic wrote on 11 Apr 01:50 +0200
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
20200411015009.4a13d52f@scratchpost.org
Hi Raghav,
pushed to guix core-updates (with changes below) because, there, it workswithout weird special gcc versions.
Commit is c0cce2055757c18619ef1826c54ed3d5b9ec7eb0.
Moved all propagated-inputs to regular inputs--except qttools which I moved tonative-inputs.
Added qtgraphicaleffects to inputs.
Switched to qt build system so the programs actually start up successfully.
There is still the same reproducibility problems in bin/linphone andbin/linphone-tester:
--- /gnu/store/7g8kdcwzvybx7a8w87pnbypy7x0rf9wg-linphoneqt-4.1.1+++ /gnu/store/7g8kdcwzvybx7a8w87pnbypy7x0rf9wg-linphoneqt-4.1.1-check├── bin│ ├── linphone│ │ ├── /gnu/store/mx2bgrpxkbdjsmhlxp9a30hbzcilk4cn-binutils-2.32/bin/readelf --wide --decompress --hex-dump=.rodata {}│ │ │ @@ -190223,17 +190223,17 @@│ │ │ 0x008080c0 02000a00 12000309 3b093c09 3d000109 ........;.<.=...│ │ │ 0x008080d0 3e000100 02010c05 56000200 0c000300 >.......V.......│ │ │ 0x008080e0 f3023706 03000100 03004c00 4d045700 ..7.......L.M.W.│ │ │ 0x008080f0 00000000 00000000 00000000 00000000 ................│ │ │ 0x00808100 00000000 00020000 00010000 00010000 ................│ │ │ 0x00808110 00000000 00000000 00000002 00000002 ................│ │ │ 0x00808120 00000002 00000000 00000000 00000018 ................│ │ │ - 0x00808130 00010000 00010000 00000000 0171666d .............qfm│ │ │ - 0x00808140 dfb80000 00220001 00000001 0000280f ....."........(.│ │ │ - 0x00808150 00000171 666ddfb8 00000000 00000000 ...qfm..........│ │ │ + 0x00808130 00010000 00010000 00000000 0171666c .............qfl│ │ │ + 0x00808140 71bb0000 00220001 00000001 0000280f q...."........(.│ │ │ + 0x00808150 00000171 666c71bb 00000000 00000000 ...qflq.........│ │ │ 0x00808160 000904eb 98c3006c 0061006e 00670075 .......l.a.n.g.u│ │ │ 0x00808170 00610067 00650073 00020000 06be0065 .a.g.e.s.......e│ │ │ 0x00808180 006e0002 000006d2 00660072 00000000 .n.......f.r....│ │ │ 0x00808190 00000000 00000000 00000000 00000000 ................│ │ │ 0x008081a0 0000280b 00008688 789ce57d 0b741cc5 ..(.....x..}.t..│ │ │ 0x008081b0 9568c992 a5d16834 92654996 8531c236 .h....h4.eI..1.6│ │ │ 0x008081c0 c626c618 639b9f31 8cf58905 922ceb63 .&..c..1.....,.c│ ├── linphone-tester│ │ ├── /gnu/store/mx2bgrpxkbdjsmhlxp9a30hbzcilk4cn-binutils-2.32/bin/readelf --wide --decompress --hex-dump=.rodata {}│ │ │ @@ -190455,17 +190455,17 @@│ │ │ 0x00811f40 02000a00 12000309 3b093c09 3d000109 ........;.<.=...│ │ │ 0x00811f50 3e000100 02010c05 56000200 0c000300 >.......V.......│ │ │ 0x00811f60 f3023706 03000100 03004c00 4d045700 ..7.......L.M.W.│ │ │ 0x00811f70 00000000 00000000 00000000 00000000 ................│ │ │ 0x00811f80 00000000 00020000 00010000 00010000 ................│ │ │ 0x00811f90 00000000 00000000 00000002 00000002 ................│ │ │ 0x00811fa0 00000002 00000000 00000000 00000018 ................│ │ │ - 0x00811fb0 00010000 00010000 00000000 0171666d .............qfm│ │ │ - 0x00811fc0 dfb80000 00220001 00000001 0000280f ....."........(.│ │ │ - 0x00811fd0 00000171 666ddfb8 00000000 00000000 ...qfm..........│ │ │ + 0x00811fb0 00010000 00010000 00000000 0171666c .............qfl│ │ │ + 0x00811fc0 71bb0000 00220001 00000001 0000280f q...."........(.│ │ │ + 0x00811fd0 00000171 666c71bb 00000000 00000000 ...qflq.........│ │ │ + 0x00811fb0 00010000 00010000 00000000 0171666c .............qfl│ │ │ + 0x00811fc0 71bb0000 00220001 00000001 0000280f q...."........(.│ │ │ + 0x00811fd0 00000171 666c71bb 00000000 00000000 ...qflq.........│ │ │ 0x00811fe0 000904eb 98c3006c 0061006e 00670075 .......l.a.n.g.u│ │ │ 0x00811ff0 00610067 00650073 00020000 06be0065 .a.g.e.s.......e│ │ │ 0x00812000 006e0002 000006d2 00660072 00000000 .n.......f.r....│ │ │ 0x00812010 00000000 00000000 00000000 00000000 ................│ │ │ 0x00812020 0000280b 00008688 789ce57d 0b741cc5 ..(.....x..}.t..│ │ │ 0x00812030 9568c992 a5d16834 92654996 8531c236 .h....h4.eI..1.6│ │ │ 0x00812040 c626c618 639b9f31 8cf58905 922ceb63 .&..c..1.....,.c
-----BEGIN PGP SIGNATURE-----
iQEzBAEBCAAdFiEEds7GsXJ0tGXALbPZ5xo1VCwwuqUFAl6RBjEACgkQ5xo1VCwwuqWuBAf/cTCGpgEckTjOw56d0UBgaeGw0QDQq/gChJnjhQkWgsBLIsqj98sZIND/0ocO91GZdUd9h36kFcS05Xdy1VJaULb6QdZXxA5Q3LmsWPNqC2X9297qRExhUIPQmX+uTxU6VtMKcK/lqfu9aRYK9N0gl3Y+S/hw2v0Vz5C3oQLdpLA4jIJhXILzmfvj5R9O4u1vmmDnU5s3EI5DEvmguzaj6HZR4iAeiCn9T+AXcgqQ2nOXQW1+Wdl/lbd2RpxL41FWMX2lIyHXEYA4jbfgvBmdt0skuFt3aino2Oo0rQ4EWeMc0MCC1pZslvCkmlgdhZpZUh8BxUpTvoJI+JrCsjjLnQ===IlUq-----END PGP SIGNATURE-----

R
R
Raghav Gururajan wrote on 12 Apr 05:55 +0200
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)(address . 40264@debbugs.gnu.org)
b6632592eeb260f9d4dbe8759eb06430@disroot.org
Hi Danny!
Toggle quote (6 lines)> if we add gcc 5 here, do we need to do something similar to> https://lists.gnu.org/archive/html/guix-commits/2020-04/msg00774.html too or> not?> > Why is only gcc 5 working and not gcc 7?
The requirement for gcc 5 is related to Qt. It has something to do with C++ standard. The Qt package in qt.scm uses gcc 5 instead of gcc 7.
The reason for gcc 7 working for linphoneqt on core-updates, could also be due to updates in qt.
Regards,RG.
M
M
Maxim Cournoyer wrote on 15 Apr 04:07 +0200
Re: [bug#40264] Replace source URI with mirror.
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264@debbugs.gnu.org)
87v9m1h587.fsf@gmail.com
Hello Raghav!
"Raghav Gururajan" <raghavgururajan@disroot.org> writes:
Toggle quote (7 lines)> From a2a4099994e21087cca9149a70d18186bfba7411 Mon Sep 17 00:00:00 2001> From: Raghav Gururajan <raghavgururajan@disroot.org>> Date: Wed, 8 Apr 2020 02:53:06 -0400> Subject: [PATCH 2/2] gnu: linphone: Replace source URI with mirror.>> * gnu/packages/linphone.scm: Replace source URI with mirror.
To have a mirror makes sense when many locations provide the samecontent. Here there's only https://www.linphone.org/so I don't thinkadding a mirror for it is necessary.
Thank you,
Maxim
M
M
Maxim Cournoyer wrote on 15 Apr 04:50 +0200
Re: [bug#40264] Linphone: 14-add-linphoneqt-v7
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)
87lfmxh38g.fsf@gmail.com
Hello Danny,
Danny Milosavljevic <dannym@scratchpost.org> writes:
Toggle quote (70 lines)> Hi Raghav,>> pushed to guix core-updates (with changes below) because, there, it works> without weird special gcc versions.>> Commit is c0cce2055757c18619ef1826c54ed3d5b9ec7eb0.>> Moved all propagated-inputs to regular inputs--except qttools which I moved to> native-inputs.>> Added qtgraphicaleffects to inputs.>> Switched to qt build system so the programs actually start up successfully.>> There is still the same reproducibility problems in bin/linphone and> bin/linphone-tester:>> --- /gnu/store/7g8kdcwzvybx7a8w87pnbypy7x0rf9wg-linphoneqt-4.1.1> +++ /gnu/store/7g8kdcwzvybx7a8w87pnbypy7x0rf9wg-linphoneqt-4.1.1-check> ├── bin> │ ├── linphone> │ │ ├── /gnu/store/mx2bgrpxkbdjsmhlxp9a30hbzcilk4cn-binutils-2.32/bin/readelf --wide --decompress --hex-dump=.rodata {}> │ │ │ @@ -190223,17 +190223,17 @@> │ │ │ 0x008080c0 02000a00 12000309 3b093c09 3d000109 ........;.<.=...> │ │ │ 0x008080d0 3e000100 02010c05 56000200 0c000300 >.......V.......> │ │ │ 0x008080e0 f3023706 03000100 03004c00 4d045700 ..7.......L.M.W.> │ │ │ 0x008080f0 00000000 00000000 00000000 00000000 ................> │ │ │ 0x00808100 00000000 00020000 00010000 00010000 ................> │ │ │ 0x00808110 00000000 00000000 00000002 00000002 ................> │ │ │ 0x00808120 00000002 00000000 00000000 00000018 ................> │ │ │ - 0x00808130 00010000 00010000 00000000 0171666d .............qfm> │ │ │ - 0x00808140 dfb80000 00220001 00000001 0000280f ....."........(.> │ │ │ - 0x00808150 00000171 666ddfb8 00000000 00000000 ...qfm..........> │ │ │ + 0x00808130 00010000 00010000 00000000 0171666c .............qfl> │ │ │ + 0x00808140 71bb0000 00220001 00000001 0000280f q...."........(.> │ │ │ + 0x00808150 00000171 666c71bb 00000000 00000000 ...qflq.........> │ │ │ 0x00808160 000904eb 98c3006c 0061006e 00670075 .......l.a.n.g.u> │ │ │ 0x00808170 00610067 00650073 00020000 06be0065 .a.g.e.s.......e> │ │ │ 0x00808180 006e0002 000006d2 00660072 00000000 .n.......f.r....> │ │ │ 0x00808190 00000000 00000000 00000000 00000000 ................> │ │ │ 0x008081a0 0000280b 00008688 789ce57d 0b741cc5 ..(.....x..}.t..> │ │ │ 0x008081b0 9568c992 a5d16834 92654996 8531c236 .h....h4.eI..1.6> │ │ │ 0x008081c0 c626c618 639b9f31 8cf58905 922ceb63 .&..c..1.....,.c> │ ├── linphone-tester> │ │ ├── /gnu/store/mx2bgrpxkbdjsmhlxp9a30hbzcilk4cn-binutils-2.32/bin/readelf --wide --decompress --hex-dump=.rodata {}> │ │ │ @@ -190455,17 +190455,17 @@> │ │ │ 0x00811f40 02000a00 12000309 3b093c09 3d000109 ........;.<.=...> │ │ │ 0x00811f50 3e000100 02010c05 56000200 0c000300 >.......V.......> │ │ │ 0x00811f60 f3023706 03000100 03004c00 4d045700 ..7.......L.M.W.> │ │ │ 0x00811f70 00000000 00000000 00000000 00000000 ................> │ │ │ 0x00811f80 00000000 00020000 00010000 00010000 ................> │ │ │ 0x00811f90 00000000 00000000 00000002 00000002 ................> │ │ │ 0x00811fa0 00000002 00000000 00000000 00000018 ................> │ │ │ - 0x00811fb0 00010000 00010000 00000000 0171666d .............qfm> │ │ │ - 0x00811fc0 dfb80000 00220001 00000001 0000280f ....."........(.> │ │ │ - 0x00811fd0 00000171 666ddfb8 00000000 00000000 ...qfm..........> │ │ │ + 0x00811fb0 00010000 00010000 00000000 0171666c .............qfl> │ │ │ + 0x00811fc0 71bb0000 00220001 00000001 0000280f q...."........(.> │ │ │ + 0x00811fd0 00000171 666c71bb 00000000 00000000 ...qflq.........> │ │ │ + 0x00811fb0 00010000 00010000 00000000 0171666c .............qfl> │ │ │ + 0x00811fc0 71bb0000 00220001 00000001 0000280f q...."........(.> │ │ │ + 0x00811fd0 00000171 666c71bb 00000000 00000000 ...qflq.........> │ │ │ 0x00811fe0 000904eb 98c3006c 0061006e 00670075 .......l.a.n.g.u> │ │ │ 0x00811ff0 00610067 00650073 00020000 06be0065 .a.g.e.s.......e> │ │ │ 0x00812000 006e0002 000006d2 00660072 00000000 .n.......f.r....> │ │ │ 0x00812010 00000000 00000000 00000000 00000000 ................> │ │ │ 0x00812020 0000280b 00008688 789ce57d 0b741cc5 ..(.....x..}.t..> │ │ │ 0x00812030 9568c992 a5d16834 92654996 8531c236 .h....h4.eI..1.6> │ │ │ 0x00812040 c626c618 639b9f31 8cf58905 922ceb63 .&..c..1.....,.c
In my case I'm seeing slightly different outputs:
diffoscope --exclude-command stat /gnu/store/i3pqi987rdm3x023ya4zd2k3jxhaiqqy-linphoneqt-4.1.1{,-check}--- /gnu/store/i3pqi987rdm3x023ya4zd2k3jxhaiqqy-linphoneqt-4.1.1+++ /gnu/store/i3pqi987rdm3x023ya4zd2k3jxhaiqqy-linphoneqt-4.1.1-check├── bin│ ├── .linphone-real│ │ ├── /gnu/store/mx2bgrpxkbdjsmhlxp9a30hbzcilk4cn-binutils-2.32/bin/readelf --wide --decompress --hex-dump=.rodata {}│ │ │ @@ -190223,17 +190223,17 @@│ │ │ 0x008080c0 02000a00 12000309 3b093c09 3d000109 ........;.<.=...│ │ │ 0x008080d0 3e000100 02010c05 56000200 0c000300 >.......V.......│ │ │ 0x008080e0 f3023706 03000100 03004c00 4d045700 ..7.......L.M.W.│ │ │ 0x008080f0 00000000 00000000 00000000 00000000 ................│ │ │ 0x00808100 00000000 00020000 00010000 00010000 ................│ │ │ 0x00808110 00000000 00000000 00000002 00000002 ................│ │ │ 0x00808120 00000002 00000000 00000000 00000018 ................│ │ │ - 0x00808130 00010000 00010000 00000000 01717ba8 .............q{.│ │ │ - 0x00808140 1fbe0000 00220001 00000001 0000280f ....."........(.│ │ │ - 0x00808150 00000171 7ba81fba 00000000 00000000 ...q{...........│ │ │ + 0x00808130 00010000 00010000 00000000 01717ba6 .............q{.│ │ │ + 0x00808140 59760000 00220001 00000001 0000280f Yv..."........(.│ │ │ + 0x00808150 00000171 7ba65972 00000000 00000000 ...q{.Yr........│ │ │ 0x00808160 000904eb 98c3006c 0061006e 00670075 .......l.a.n.g.u│ │ │ 0x00808170 00610067 00650073 00020000 06be0065 .a.g.e.s.......e│ │ │ 0x00808180 006e0002 000006d2 00660072 00000000 .n.......f.r....│ │ │ 0x00808190 00000000 00000000 00000000 00000000 ................│ │ │ 0x008081a0 0000280b 00008688 789ce57d 0b741cc5 ..(.....x..}.t..│ │ │ 0x008081b0 9568c992 a5d16834 92654996 8531c236 .h....h4.eI..1.6│ │ │ 0x008081c0 c626c618 639b9f31 8cf58905 922ceb63 .&..c..1.....,.c│ ├── .linphone-tester-real│ │ ├── /gnu/store/mx2bgrpxkbdjsmhlxp9a30hbzcilk4cn-binutils-2.32/bin/readelf --wide --decompress --hex-dump=.rodata {}│ │ │ @@ -190455,17 +190455,17 @@│ │ │ 0x00811f40 02000a00 12000309 3b093c09 3d000109 ........;.<.=...│ │ │ 0x00811f50 3e000100 02010c05 56000200 0c000300 >.......V.......│ │ │ 0x00811f60 f3023706 03000100 03004c00 4d045700 ..7.......L.M.W.│ │ │ 0x00811f70 00000000 00000000 00000000 00000000 ................│ │ │ 0x00811f80 00000000 00020000 00010000 00010000 ................│ │ │ 0x00811f90 00000000 00000000 00000002 00000002 ................│ │ │ 0x00811fa0 00000002 00000000 00000000 00000018 ................│ │ │ - 0x00811fb0 00010000 00010000 00000000 01717ba8 .............q{.│ │ │ - 0x00811fc0 1fbe0000 00220001 00000001 0000280f ....."........(.│ │ │ - 0x00811fd0 00000171 7ba81fba 00000000 00000000 ...q{...........│ │ │ + 0x00811fb0 00010000 00010000 00000000 01717ba6 .............q{.│ │ │ + 0x00811fc0 59760000 00220001 00000001 0000280f Yv..."........(.│ │ │ + 0x00811fd0 00000171 7ba65972 00000000 00000000 ...q{.Yr........│ │ │ 0x00811fe0 000904eb 98c3006c 0061006e 00670075 .......l.a.n.g.u│ │ │ 0x00811ff0 00610067 00650073 00020000 06be0065 .a.g.e.s.......e│ │ │ 0x00812000 006e0002 000006d2 00660072 00000000 .n.......f.r....│ │ │ 0x00812010 00000000 00000000 00000000 00000000 ................│ │ │ 0x00812020 0000280b 00008688 789ce57d 0b741cc5 ..(.....x..}.t..│ │ │ 0x00812030 9568c992 a5d16834 92654996 8531c236 .h....h4.eI..1.6│ │ │ 0x00812040 c626c618 639b9f31 8cf58905 922ceb63 .&..c..1.....,.c
Adding more data points so that hopefully we can find out the cause ofthe non-reproducibility issue.
Maxim
M
M
Maxim Cournoyer wrote on 15 Apr 06:36 +0200
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)
87ftd5gybq.fsf@gmail.com
Hello again,
I've isolated the culprit: it's the translation auto-generated filesqrc_i18n.cpp. I've run a couple builds locally, then diffed the buildfiles, and found this suspicious:
diff -I build2 -I build3 -r build2/qrc_i18n.cpp build3/qrc_i18n.cpp1352c1352< 0x0,0x0,0x1,0x71,0x7b,0xea,0x68,0x8a,---
Toggle quote (1 lines)> 0x0,0x0,0x1,0x71,0x7b,0xec,0x98,0x61,
1355c1355< 0x0,0x0,0x1,0x71,0x7b,0xea,0x68,0x86,---
Toggle quote (2 lines)> 0x0,0x0,0x1,0x71,0x7b,0xec,0x98,0x61,
I then compared the object files for this source, e.g.: diffoscope--exclude-command statbuild{2,3}/CMakeFiles/app-library.dir/qrc_i18n.cpp.o
Hex dump of section '.rodata':│ 0x00000000 00000000 00020000 00010000 00010000 ................│ 0x00000010 00000000 00000000 00000002 00000002 ................│ 0x00000020 00000002 00000000 00000000 00000018 ................│ - 0x00000030 00010000 00010000 00000000 01717bea .............q{.│ - 0x00000040 688a0000 00220001 00000001 0000280f h...."........(.│ - 0x00000050 00000171 7bea6886 00000000 00000000 ...q{.h.........│ + 0x00000030 00010000 00010000 00000000 01717bec .............q{.│ + 0x00000040 98610000 00220001 00000001 0000280f .a..."........(.│ + 0x00000050 00000171 7bec9861 00000000 00000000 ...q{..a........│ 0x00000060 000904eb 98c3006c 0061006e 00670075 .......l.a.n.g.u│ 0x00000070 00610067 00650073 00020000 06be0065 .a.g.e.s.......e│ 0x00000080 006e0002 000006d2 00660072 00000000 .n.......f.r....│ 0x00000090 00000000 00000000 00000000 00000000 ................│ 0x000000a0 0000280b 00008688 789ce57d 0b741cc5 ..(.....x..}.t..│ 0x000000b0 9568c992 a5d16834 92654996 8531c236 .h....h4.eI..1.6│ 0x000000c0 c626c618 639b9f31 8cf58905 922ceb63 .&..c..1.....,.c
This matches the .rodata diff found in the main binary :-).
Maxim
M
M
Maxim Cournoyer wrote on 15 Apr 08:13 +0200
(name . Danny Milosavljevic)(address . dannym@scratchpost.org)
87blntgtti.fsf@gmail.com
Hi!
Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:
Toggle quote (42 lines)> Hello again,>> I've isolated the culprit: it's the translation auto-generated files> qrc_i18n.cpp. I've run a couple builds locally, then diffed the build> files, and found this suspicious:>> diff -I build2 -I build3 -r build2/qrc_i18n.cpp build3/qrc_i18n.cpp> 1352c1352> < 0x0,0x0,0x1,0x71,0x7b,0xea,0x68,0x8a,> --->> 0x0,0x0,0x1,0x71,0x7b,0xec,0x98,0x61,> 1355c1355> < 0x0,0x0,0x1,0x71,0x7b,0xea,0x68,0x86,> --->> 0x0,0x0,0x1,0x71,0x7b,0xec,0x98,0x61,>> I then compared the object files for this source, e.g.: diffoscope> --exclude-command stat> build{2,3}/CMakeFiles/app-library.dir/qrc_i18n.cpp.o>> Hex dump of section '.rodata':> │ 0x00000000 00000000 00020000 00010000 00010000 ................> │ 0x00000010 00000000 00000000 00000002 00000002 ................> │ 0x00000020 00000002 00000000 00000000 00000018 ................> │ - 0x00000030 00010000 00010000 00000000 01717bea .............q{.> │ - 0x00000040 688a0000 00220001 00000001 0000280f h...."........(.> │ - 0x00000050 00000171 7bea6886 00000000 00000000 ...q{.h.........> │ + 0x00000030 00010000 00010000 00000000 01717bec .............q{.> │ + 0x00000040 98610000 00220001 00000001 0000280f .a..."........(.> │ + 0x00000050 00000171 7bec9861 00000000 00000000 ...q{..a........> │ 0x00000060 000904eb 98c3006c 0061006e 00670075 .......l.a.n.g.u> │ 0x00000070 00610067 00650073 00020000 06be0065 .a.g.e.s.......e> │ 0x00000080 006e0002 000006d2 00660072 00000000 .n.......f.r....> │ 0x00000090 00000000 00000000 00000000 00000000 ................> │ 0x000000a0 0000280b 00008688 789ce57d 0b741cc5 ..(.....x..}.t..> │ 0x000000b0 9568c992 a5d16834 92654996 8531c236 .h....h4.eI..1.6> │ 0x000000c0 c626c618 639b9f31 8cf58905 922ceb63 .&..c..1.....,.c>> This matches the .rodata diff found in the main binary :-).>> Maxim
Fixed in commit a4f1a7e2db, on core-updates :-)
Maxim
M
M
maxim.cournoyer wrote on 16 Apr 06:49 +0200
Re: [bug#40264] Mirror and Formatting
(name . Raghav Gururajan)(address . raghavgururajan@disroot.org)(address . 40264-done@debbugs.gnu.org)
87lfmwkpbv.fsf@hurd.i-did-not-set--mail-host-address--so-tickle-me
Hello Raghav,
"Raghav Gururajan" <raghavgururajan@disroot.org> writes:
Toggle quote (207 lines)> From 5b3dc89e93749fb4ff1e2d246db20e61a54e6232 Mon Sep 17 00:00:00 2001> From: Raghav Gururajan <raghavgururajan@disroot.org>> Date: Wed, 8 Apr 2020 03:28:07 -0400> Subject: [PATCH 3/3] gnu: linphone: Fix formatting.>> * gnu/packages/linphone.scm: Fix formatting.> ---> gnu/packages/linphone.scm | 84 ++++++++++++++++++++-------------------> 1 file changed, 44 insertions(+), 40 deletions(-)>> diff --git a/gnu/packages/linphone.scm b/gnu/packages/linphone.scm> index 39109ff877..0e3281de4b 100644> --- a/gnu/packages/linphone.scm> +++ b/gnu/packages/linphone.scm> @@ -70,9 +70,9 @@> (list> "-DENABLE_STATIC=NO"))) ; Not required> (synopsis "Belledonne Communications Unit Testing Framework")> - (description "BCUnit is a fork of the defunct project CUnit,> -with several fixes and patches applied. It is an unit testing> -framework for writing, administering, and running unit tests in C.")> + (description "BCUnit is a fork of the defunct project CUnit, with several> +fixes and patches applied. It is an unit testing framework for writing,> +administering, and running unit tests in C.")> (home-page "https://gitlab.linphone.org/BC/public/bcunit")> (license license:lgpl2.0+)))> > @@ -124,9 +124,10 @@ Communications softwares like belle-sip, mediastreamer2 and linphone.")> (inputs> `(("bctoolbox" ,bctoolbox)))> (synopsis "Belledonne Communications Language Recognition Library")> - (description "Belr is Belledonne Communications' language recognition library,> -written in C++11. It parses text inputs formatted according to a language> -defined by an ABNF grammar, such as the protocols standardized at IETF.")> + (description "Belr is Belledonne Communications' language recognition> +library, written in C++11. It parses text inputs formatted according to a> +language defined by an ABNF grammar, such as the protocols standardized at> +IETF.")> (home-page "https://gitlab.linphone.org/BC/public/belr")> (license license:gpl3+)))> > @@ -152,7 +153,8 @@ defined by an ABNF grammar, such as the protocols standardized at IETF.")> `(("bctoolbox" ,bctoolbox)> ("belr" ,belr)))> (synopsis "Belledonne Communications VCard Library")> - (description "Belcard is a C++ library to manipulate VCard standard format.")> + (description "Belcard is a C++ library to manipulate VCard standard> +format.")> (home-page "https://gitlab.linphone.org/BC/public/belcard")> (license license:gpl3+)))> > @@ -175,16 +177,16 @@ defined by an ABNF grammar, such as the protocols standardized at IETF.")> (list> "-DENABLE_STATIC=NO"))) ; Not required> (synopsis "Belledonne Communications Media Container")> - (description "BcMatroska is a free and open standard multi-media> -container format. It can hold an unlimited number of video, audio,> -picture, or subtitle tracks in one file. ")> + (description "BcMatroska is a free and open standard multi-media container> +format. It can hold an unlimited number of video, audio, picture, or subtitle> +tracks in one file. ")> (home-page "https://gitlab.linphone.org/BC/public/bcmatroska2")> (license> (list> - ;; That license applies for Core C and LibEBML2.> + ;; For Core C and LibEBML2.> ;; https://www.matroska.org/node/47> license:bsd-4> - ;; That license applies for LibMatroska2.> + ;; For LibMatroska2.> ;; https://www.matroska.org/node/47> license:lgpl2.1+))))> > @@ -261,15 +263,15 @@ implements the RFC 3550 standard.")> `(#:tests? #f ; No test target> #:configure-flags> (list> - "-DENABLE_STATIC=NO")))> + "-DENABLE_STATIC=NO"))) ; Not required> (inputs> `(("bctoolbox" ,bctoolbox)> ("sqlite3" ,sqlite)> ("xml2" ,libxml2)))> (synopsis "Belledonne Communications ZRTP Library")> - (description "BZRTP is an implementation of ZRTP keys exchange> -protocol, written in C. It is fully portable and can be executed on many> -platforms including both ARM and x86.")> + (description "BZRTP is an implementation of ZRTP keys exchange 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+)))> > @@ -290,15 +292,15 @@ platforms including both ARM and x86.")> `(#:tests? #f ; Requires network access> #:configure-flags> (list> - "-DENABLE_STATIC=NO")> + "-DENABLE_STATIC=NO") ; Not required> #: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 ") ""))> + ;; ANTLR would use multithreaded DFA generation otherwise,> + ;; which would not be reproducible.> + (("-Xmultithreaded ") ""))> #t)))))> (inputs> `(("antlr3" ,antlr3-3.3)> @@ -325,7 +327,8 @@ API. It also comprises a simple HTTP/HTTPS client implementation.")> "/" name "-" version ".tar.gz"))> (sha256> (base32 "0whpqr69wz0pnzvragkpfblxhd0rds8k06c3mw5a0ag216a1yd9k"))> - (patches (search-patches "mediastreamer2-srtp2.patch"))))> + (patches> + (search-patches "mediastreamer2-srtp2.patch"))))> (outputs '("out" "doc" "tester"))> (build-system cmake-build-system)> (arguments> @@ -393,13 +396,12 @@ API. It also comprises a simple HTTP/HTTPS client implementation.")> ("vpx" ,libvpx)> ("x11" ,libx11)> ("xv" ,libxv)> - ("zrtp", bzrtp)))> + ("zrtp" ,bzrtp)))> (synopsis "Belledonne Communications Streaming Engine")> - (description "Mediastreamer2 is a powerful and lightweight> -streaming engine for telephony applications. This media processing> -and streaming toolkit is responsible for receiving and sending all> -multimedia streams in Linphone, including media capture, encoding and> -decoding, and rendering.")> + (description "Mediastreamer2 is a powerful and lightweight streaming engine> +for telephony applications. This media processing and streaming toolkit 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+)))> > @@ -461,11 +463,11 @@ decoding, and rendering.")> (propagated-inputs> `(("murrine" ,murrine))) ; Required for GTK UI> (synopsis "Belledonne Communications Softphone Library")> - (description "Liblinphone is a high-level SIP library integrating> -all calling and instant messaging features into an unified> -easy-to-use API. It is the cross-platform VoIP library on which the> -Linphone application is based on, and that anyone can use to add audio> -and video calls or instant messaging capabilities to an application.")> + (description "Liblinphone is a high-level SIP library integrating all> +calling and instant messaging features into an unified easy-to-use API. It is> +the cross-platform VoIP library on which the Linphone application is based on,> +and that anyone can use to add audio and video calls or instant messaging> +capabilities to an application.")> (home-page "https://gitlab.linphone.org/BC/public/liblinphone")> (license license:gpl2+)))> > @@ -487,11 +489,13 @@ and video calls or instant messaging capabilities to an application.")> (origin> (method url-fetch)> (uri> - (string-append "https://gitlab.linphone.org/BC/public/msopenh264/commit/"> - "493d147d28c9a0f788ba4e50b47a1ce7b18bf326.diff"))> + (string-append "https://gitlab.linphone.org/BC/public/msopenh264/"> + "commit/493d147d28c9a0f788ba4e50b47a1ce7b18bf326"> + ".diff"))> (file-name "msopenh264-openh264.patch")> (sha256> - (base32 "0mmd7nz5n9ian4rcwn200nldmy5j0dpdrna7r32rqnaw82bx3kdb")))))))> + (base32> + "0mmd7nz5n9ian4rcwn200nldmy5j0dpdrna7r32rqnaw82bx3kdb")))))))> (build-system cmake-build-system)> (arguments> `(#:tests? #f ; No test target> @@ -530,8 +534,8 @@ mediastreamer2 based on the openh264 library.")> `(("mediastreamer2" ,mediastreamer2)> ("ortp" ,ortp)))> (synopsis "Media Streamer SILK Codec")> - (description "MSSILK is a plugin of MediaStreamer, adding support for> -AMR codec. It is based on the Skype's SILK implementation.")> + (description "MSSILK is a plugin of MediaStreamer, adding support for AMR> +codec. It is based on the Skype's SILK implementation.")> (home-page "https://gitlab.linphone.org/BC/public/mssilk")> (license license:gpl2+)))> > @@ -552,7 +556,7 @@ AMR codec. It is based on the Skype's SILK implementation.")> `(#:tests? #f ; No test target> #:configure-flags> (list> - "-DENABLE_STATIC=NO")))> + "-DENABLE_STATIC=NO"))) ; Not required> (inputs> `(("bctoolbox" ,bctoolbox)> ("mediastreamer2" ,mediastreamer2)> @@ -588,7 +592,7 @@ WebRTC codec. It includes features from WebRTC, such as, iSAC and AECM.")> ("ortp" ,ortp)> ("voamrwbenc" ,vo-amrwbenc)))> (synopsis "Media Streamer AMR Codec")> - (description "MSAMR is a plugin of MediaStreamer, adding support for> -AMR codec. It is based on the opencore-amr implementation.")> + (description "MSAMR is a plugin of MediaStreamer, adding support for AMR> +codec. It is based on the opencore-amr implementation.")> (home-page "https://gitlab.linphone.org/BC/public/msamr")> (license license:gpl3+)))
Please not that we usually submit changes *per-package*, that is, onecommit per package. It's a convention we use. I'm making an exceptionhere because the changes are cosmetic only, but keep this in mind foryour next changes.
I've pushed a slightly modified version of the above changes in commit1988ef21f82430d4b380dd4887160a902d8796fd (on the core-updates branch).
Thank you for your contributions and your patience in the lengthy reviewprocess.
Maxim
Closed
?