--- Begin Message ---
Subject: |
[PATCH 00/12] Update jami and add jami-qt |
Date: |
Wed, 31 Mar 2021 01:14:49 -0400 |
Hello!
I've finally picked up the packaging of Jami, which was becoming quite
outdated in Guix. I haven't had much time to test the result yet, but a
priori it seems to function as intended.
Let me know how it works for you!
Maxim
Maxim Cournoyer (12):
gnu: asio: Update to 1.18.1.
gnu: restbed: Update to 4.7.
gnu: restinio: Update to 0.6.13.
gnu: Add nettle-3.7.
gnu: opendht: Update to 2.2.0rc4.
gnu: pjproject: Update to 2.11.
gnu: pjproject-jami: Fix CVE-2020-15260 and CVE-2021-21375.
ffmpeg-jami: Patch with the patches from Jami 20210326.1.cfba013.
gnu: libring: Update to 20210326.1.cfba013.
gnu: libringclient: Update to 20210326.1.cfba013.
gnu: jami: Update to 20210326.1.cfba013 and rename to jami-gnome.
gnu: Add jami-qt.
gnu/local.mk | 1 -
gnu/packages/emulators.scm | 2 +-
gnu/packages/jami.scm | 287 ++++++++++++--------
gnu/packages/nettle.scm | 17 +-
gnu/packages/networking.scm | 131 +++++----
gnu/packages/patches/opendht-fix-jami.patch | 33 ---
gnu/packages/telephony.scm | 22 +-
7 files changed, 272 insertions(+), 221 deletions(-)
delete mode 100644 gnu/packages/patches/opendht-fix-jami.patch
--
2.31.1
--- End Message ---
--- Begin Message ---
Subject: |
Re: [bug#47513] [PATCH 00/12] Update jami and add jami-qt |
Date: |
Fri, 02 Apr 2021 07:52:22 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Hi Maxime!
Maxime Devos <maximedevos@telenet.be> writes:
> On Wed, 2021-03-31 at 01:14 -0400, Maxim Cournoyer wrote:
>> Hello!
>>
>> I've finally picked up the packaging of Jami, which was becoming quite
>> outdated in Guix. I haven't had much time to test the result yet, but a
>> priori it seems to function as intended.
>>
>> Let me know how it works for you!
>
> It seems to work for me, however I needed to make a tiny change:
>
> <start snip>
> diff --git a/gnu/packages/jami.scm b/gnu/packages/jami.scm
> index 67befe63a2..b73ad027fe 100644
> --- a/gnu/packages/jami.scm
> +++ b/gnu/packages/jami.scm
> @@ -554,7 +554,7 @@ decentralized calling using P2P-DHT.")
>
> ;;; Keep this until the Qt client matures enough to become the
> ;;; main 'jami' client.
> -(deprecated-package jami jami-gnome)
> +#;(deprecated-package jami jami-gnome)
>
> (define-public jami-qt
> (package
> <end snip>
Jack in the thread made me notice about that as well; deprecated-package
returns a package object, so it needs to be public-define'd the usual
way:
+
+;;; Keep this until the Qt client matures enough to become the
+;;; main 'jami' client.
+(define-public jami
+ (deprecated-package "jami" jami-gnome))
And yes the package name is a string. I fixed this in a v2 patch series
I sent along a problem propagating libringclient instead of libring for
the jami-qt client (see:
https://debbugs.gnu.org/cgi/bugreport.cgi?bug=47513#95). MUMI
(issues.guix.gnu.org) sadly doesn't seem to have picked it up.
[...]
> Also, I didn't have any opportunity to actually call someone.
[...]
OK! I've tested it on my side with actual calls, so I feel confident it
works fine. I've made a few last changes (using LIBWRAP when building
libringclient so that it talks directly with libring instead of going
through DBus, which allows to have it work even in a 'guix pack' and
cleaning the list of imports) and pushed to master as
e6b447cf756831e8339e6462647004d88132fa07.
Thanks for the review!
Maxim
--- End Message ---