* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 @ 2023-07-20 17:18 Distopico 2023-07-20 19:33 ` Saku Laesvuori via Guix-patches via ` (5 more replies) 0 siblings, 6 replies; 29+ messages in thread From: Distopico @ 2023-07-20 17:18 UTC (permalink / raw) To: 64748; +Cc: Distopico, rg, hako, saku Fixes telegram-desktop call not working with openSSL 3 see: https://github.com/telegramdesktop/tdesktop/issues/26108 * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to a45d8b8 revision commit --- gnu/packages/telegram.scm | 31 +++++++++++++++++++++++++------ 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm index aa01c2f692..aeab249198 100644 --- a/gnu/packages/telegram.scm +++ b/gnu/packages/telegram.scm @@ -2,6 +2,7 @@ ;;; Copyright © 2021 Raghav Gururajan <rg@raghavgururajan.name> ;;; Copyright © 2022 Hilton Chain <hako@ultrarare.space> ;;; Copyright © 2023 Saku Laesvuori <saku@laesvuori.fi> +;;; Copyright © 2023 Camilo Q.S. (Distopico) <distopico@riseup.net> ;;; ;;; This file is part of GNU Guix. ;;; @@ -96,6 +97,21 @@ (define libyuv-for-telegram-desktop (base32 "0mm56p8iapfild2xdw4w8zi35c3xm06fgagiali644gnxdmnym6c"))))) +(define libsrtp-for-telegram-desktop + (let ((commit "a566a9cfcd619e8327784aa7cff4a1276dc1e895") + (revision "1494")) + (origin + (method git-fetch) + (uri (git-reference + (url "https://github.com/cisco/libsrtp") + (commit commit))) + (file-name (git-file-name + "libsrtp-for-telegram-desktop" + (git-version "0" revision commit))) + (sha256 + (base32 + "1ichw2v9s2mggi5p2wbbmlg55q4r48dxi3ks7ykfcfkmh7pb1w1s"))))) + (define cmake-helpers-for-telegram-desktop (origin (method git-fetch) @@ -265,8 +281,8 @@ (define tgcalls-for-telegram-desktop "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z")))) (define-public webrtc-for-telegram-desktop - (let ((commit "5098730b9eb6173f0b52068fe2555b7c1015123a") - (revision "328")) + (let ((commit "a45d8b8f0a99bd0e5118dda1dc4a8b7b3ad5dcfd") + (revision "388")) (hidden-package (package (name "webrtc-for-telegram-desktop") @@ -282,14 +298,14 @@ (define-public webrtc-for-telegram-desktop (file-name (git-file-name name version)) (sha256 - (base32 "1lk54zlrff59rj5k9dylsgz4sdds4728psrk8m3v9qn5y8d6z8qy")) + (base32 "1qs3ikkd6l56brj40cv6wlhx5gj5avisj9mj8ypjfwcyw9hb2n5y")) (modules '((guix build utils) (ice-9 ftw) (srfi srfi-1))) (snippet #~(begin (let ((keep - '("libsrtp" "rnnoise" + '("rnnoise" "libsrtp_config" ;; Not available in Guix. "pffft"))) (with-directory-excursion "src/third_party" @@ -311,9 +327,12 @@ (define-public webrtc-for-telegram-desktop (add-after 'unpack 'unpack-additional-sources (lambda _ (let* ((third-party (string-append (getcwd) "/src/third_party")) - (libyuv-to (string-append third-party "/libyuv"))) + (libyuv-to (string-append third-party "/libyuv")) + (libsrtp-to (string-append third-party "/libsrtp"))) (copy-recursively #$libyuv-for-telegram-desktop - libyuv-to))))))) + libyuv-to) + (copy-recursively #$libsrtp-for-telegram-desktop + libsrtp-to))))))) (native-inputs (list pkg-config python-wrapper yasm)) (inputs (list abseil-cpp-cxxstd17 base-commit: 13cb9b302868b5a966a6ae177412c474084f4bf1 -- 2.41.0 ^ permalink raw reply related [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-20 17:18 [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 Distopico @ 2023-07-20 19:33 ` Saku Laesvuori via Guix-patches via 2023-07-20 20:21 ` Distopico 2023-07-20 23:26 ` [bug#64748] [PATCH v2] " Distopico ` (4 subsequent siblings) 5 siblings, 1 reply; 29+ messages in thread From: Saku Laesvuori via Guix-patches via @ 2023-07-20 19:33 UTC (permalink / raw) To: Distopico; +Cc: rg, 64748, hako [-- Attachment #1: Type: text/plain, Size: 1445 bytes --] > Fixes telegram-desktop call not working with openSSL 3 > > see: https://github.com/telegramdesktop/tdesktop/issues/26108 > > * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to a45d8b8 revision commit You should probably mention adding `libsrtp-for-telegram-desktop` in the ChangeLog. > @@ -96,6 +97,21 @@ (define libyuv-for-telegram-desktop > (base32 > "0mm56p8iapfild2xdw4w8zi35c3xm06fgagiali644gnxdmnym6c"))))) > > +(define libsrtp-for-telegram-desktop > + (let ((commit "a566a9cfcd619e8327784aa7cff4a1276dc1e895") > + (revision "1494")) > + (origin > + (method git-fetch) > + (uri (git-reference > + (url "https://github.com/cisco/libsrtp") > + (commit commit))) > + (file-name (git-file-name > + "libsrtp-for-telegram-desktop" > + (git-version "0" revision commit))) > + (sha256 > + (base32 > + "1ichw2v9s2mggi5p2wbbmlg55q4r48dxi3ks7ykfcfkmh7pb1w1s"))))) > + Have you tried using the libsrtp version packaged in Guix as an input to `webrtc-for-telegram-desktop` instead of bundling it? If that doesn't work, have you tried packaging libsrtp 2.5.0 and using it as an input? It would probably be relatively easy with record inheriting. I'm quite new to Guix (and couldn't push changes even if they were perfect) so someone else might have more comments on this. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-20 19:33 ` Saku Laesvuori via Guix-patches via @ 2023-07-20 20:21 ` Distopico 2023-07-20 21:06 ` Saku Laesvuori via Guix-patches via 0 siblings, 1 reply; 29+ messages in thread From: Distopico @ 2023-07-20 20:21 UTC (permalink / raw) To: Saku Laesvuori; +Cc: rg, 64748, hako [-- Attachment #1: Type: text/plain, Size: 860 bytes --] On 2023-07-20, Saku Laesvuori <saku@laesvuori.fi> wrote: > > You should probably mention adding `libsrtp-for-telegram-desktop` in the > ChangeLog. Looks like it's not longer necessary, Changelog is auto-generated, that said in the main Changelog > > Have you tried using the libsrtp version packaged in Guix as an input to > `webrtc-for-telegram-desktop` instead of bundling it? If that doesn't > work, have you tried packaging libsrtp 2.5.0 and using it as an input? > It would probably be relatively easy with record inheriting. > > I'm quite new to Guix (and couldn't push changes even if they were > perfect) so someone else might have more comments on this. > I tried but it require source code inside libsrtp folder, for example '/crypto/cipher/cipher.c', also libsrtp package is the compiled version and webrtc-.. require source code from libsrtp. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 515 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-20 20:21 ` Distopico @ 2023-07-20 21:06 ` Saku Laesvuori via Guix-patches via 2023-07-20 23:09 ` Distopico 0 siblings, 1 reply; 29+ messages in thread From: Saku Laesvuori via Guix-patches via @ 2023-07-20 21:06 UTC (permalink / raw) To: Distopico; +Cc: rg, 64748, hako [-- Attachment #1: Type: text/plain, Size: 1737 bytes --] > > You should probably mention adding `libsrtp-for-telegram-desktop` in the > > ChangeLog. > > Looks like it's not longer necessary, Changelog is auto-generated, that > said in the main Changelog I mean the commit message. Now your commit message's ChangeLog only mentions updating webrtc-for-telegram-desktop. See my last telegram commit 6192acf8b77 for an example of what I'm talking about. I would assume the ChangeLog file is autogenerated based on the commit messages. > > Have you tried using the libsrtp version packaged in Guix as an input to > > `webrtc-for-telegram-desktop` instead of bundling it? If that doesn't > > work, have you tried packaging libsrtp 2.5.0 and using it as an input? > > It would probably be relatively easy with record inheriting. > > > > I'm quite new to Guix (and couldn't push changes even if they were > > perfect) so someone else might have more comments on this. > > I tried but it require source code inside libsrtp folder, for example > '/crypto/cipher/cipher.c', also libsrtp package is the compiled version > and webrtc-.. require source code from libsrtp. You would probably have to patch the `CMakeLists.txt` so it doesn't try to build libsrtp from source, see what I did with crc32c in the mentioned commit for reference. It is quite possible that telegram doesn't actually need the source code and they are simply using a git submodule to simplify their own dependency versioning and development. You might also have to add a new libsrtp version because the one packaged in Guix is not configured to use openssl unlike telegram expects. I'd still try the packaged version first if compiling telegram a few more times isn't a big problem for you. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-20 21:06 ` Saku Laesvuori via Guix-patches via @ 2023-07-20 23:09 ` Distopico 2023-07-21 7:22 ` Saku Laesvuori via Guix-patches via 0 siblings, 1 reply; 29+ messages in thread From: Distopico @ 2023-07-20 23:09 UTC (permalink / raw) To: Saku Laesvuori; +Cc: rg, 64748, hako [-- Attachment #1: Type: text/plain, Size: 1745 bytes --] On 2023-07-21, Saku Laesvuori <saku@laesvuori.fi> wrote: > > I mean the commit message. Now your commit message's ChangeLog only > mentions updating webrtc-for-telegram-desktop. See my last telegram > commit 6192acf8b77 for an example of what I'm talking about. > > I would assume the ChangeLog file is autogenerated based on the commit > messages. > Make sense, I'll update the patch to add more description > > You would probably have to patch the `CMakeLists.txt` so it doesn't try > to build libsrtp from source, see what I did with crc32c in the > mentioned commit for reference. It is quite possible that telegram > doesn't actually need the source code and they are simply using a git > submodule to simplify their own dependency versioning and development. > > You might also have to add a new libsrtp version because the one > packaged in Guix is not configured to use openssl unlike telegram > expects. I'd still try the packaged version first if compiling telegram > a few more times isn't a big problem for you. > I tried but didn't work, this change is similar to **libyuv**, that have reference to the source code of that external library: example https://github.com/desktop-app/tg_owt/blob/83-sdk/src/api/video/i010_buffer.cc#L17 So the case of **libsrtp** is similar, check for example: - https://github.com/desktop-app/tg_owt/blob/master/src/pc/srtp_session_unittest.cc#L25 - https://github.com/desktop-app/tg_owt/blob/master/src/pc/external_hmac.cc#L18 - https://github.com/desktop-app/tg_owt/blob/master/src/pc/BUILD.gn#L174 So change that will require patch several files, not just `CMakeLists.txt`, patch all those file will be make this package hard to maintain. but you can try if you want and let me know if works. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 515 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-20 23:09 ` Distopico @ 2023-07-21 7:22 ` Saku Laesvuori via Guix-patches via 2023-07-22 6:36 ` Raghav Gururajan via Guix-patches via 0 siblings, 1 reply; 29+ messages in thread From: Saku Laesvuori via Guix-patches via @ 2023-07-21 7:22 UTC (permalink / raw) To: Distopico; +Cc: rg, 64748, hako [-- Attachment #1: Type: text/plain, Size: 1458 bytes --] > > You would probably have to patch the `CMakeLists.txt` so it doesn't try > > to build libsrtp from source, see what I did with crc32c in the > > mentioned commit for reference. It is quite possible that telegram > > doesn't actually need the source code and they are simply using a git > > submodule to simplify their own dependency versioning and development. > > > > You might also have to add a new libsrtp version because the one > > packaged in Guix is not configured to use openssl unlike telegram > > expects. I'd still try the packaged version first if compiling telegram > > a few more times isn't a big problem for you. > > I tried but didn't work, this change is similar to **libyuv**, that have > reference to the source code of that external library: example > https://github.com/desktop-app/tg_owt/blob/83-sdk/src/api/video/i010_buffer.cc#L17 > > So the case of **libsrtp** is similar, check for example: > - https://github.com/desktop-app/tg_owt/blob/master/src/pc/srtp_session_unittest.cc#L25 > - https://github.com/desktop-app/tg_owt/blob/master/src/pc/external_hmac.cc#L18 > - https://github.com/desktop-app/tg_owt/blob/master/src/pc/BUILD.gn#L174 > > So change that will require patch several files, not just > `CMakeLists.txt`, patch all those file will be make this package hard to > maintain. I see. I think libsrtp is small enough that unbundling it is not worth the effort of patching all those files. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-21 7:22 ` Saku Laesvuori via Guix-patches via @ 2023-07-22 6:36 ` Raghav Gururajan via Guix-patches via 2023-07-22 8:25 ` Saku Laesvuori via Guix-patches via 0 siblings, 1 reply; 29+ messages in thread From: Raghav Gururajan via Guix-patches via @ 2023-07-22 6:36 UTC (permalink / raw) To: Saku Laesvuori, Distopico; +Cc: hako, 64748 [-- Attachment #1.1: Type: text/plain, Size: 269 bytes --] If libsrtp in Guix uses something other than openssl by default, then we could create a package-definition "libsrtp-openssl" which inherits from "libsrtp" and build-flagged to use openssl. We can then use that for telegram desktop. Thoughts? Regards, RG. [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 236 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-22 6:36 ` Raghav Gururajan via Guix-patches via @ 2023-07-22 8:25 ` Saku Laesvuori via Guix-patches via 2023-07-24 17:07 ` Distopico 0 siblings, 1 reply; 29+ messages in thread From: Saku Laesvuori via Guix-patches via @ 2023-07-22 8:25 UTC (permalink / raw) To: Raghav Gururajan; +Cc: hako, 64748, Distopico [-- Attachment #1: Type: text/plain, Size: 644 bytes --] > If libsrtp in Guix uses something other than openssl by default, then we > could create a package-definition "libsrtp-openssl" which inherits from > "libsrtp" and build-flagged to use openssl. We can then use that for > telegram desktop. Yes, but we would still need to patch webrtc-for-telegram-desktop as the code currently refers to headers in the submodule directory. Now that I think of it, it could actually be relatively easy with a regex substitution (something like s!^#include "third_party/(srtp/[^"]*)"$!#include <\1>!) on every file. That's of course assuming that the code only refers to headers in the submodule. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-22 8:25 ` Saku Laesvuori via Guix-patches via @ 2023-07-24 17:07 ` Distopico 2023-07-24 20:17 ` Raghav Gururajan via Guix-patches via 0 siblings, 1 reply; 29+ messages in thread From: Distopico @ 2023-07-24 17:07 UTC (permalink / raw) To: Saku Laesvuori; +Cc: Raghav Gururajan, 64748, hako I just tested trying to replace all the libsrtp but not luck from side, they use for example: - third_party/libsrtp/crypto/include/crypto_types.h - third_party/libsrtp/include/srtp.h not exposed by "libsrtp" package, also they use a custom configuration for that library https://github.com/desktop-app/tg_owt/blob/master/src/third_party/libsrtp_config/config.h Also there is many other references to `libsrtp` .C source code https://github.com/desktop-app/tg_owt/blob/a45d8b8f0a99bd0e5118dda1dc4a8b7b3ad5dcfd/cmake/libsrtp.cmake#L4 I was checking nix implementation and they just use the submodules https://github.com/NixOS/nixpkgs/blob/nixos-23.05/pkgs/applications/networking/instant-messengers/telegram/telegram-desktop/tg_owt.nix#L19 and not "srtp" package. So I think the patch that i sent is a good solution, similar to the nix one, but if someone else want to try another option would be great. On 2023-07-22, Saku Laesvuori <saku@laesvuori.fi> wrote: > [[PGP Signed Part:Undecided]] >> If libsrtp in Guix uses something other than openssl by default, then we >> could create a package-definition "libsrtp-openssl" which inherits from >> "libsrtp" and build-flagged to use openssl. We can then use that for >> telegram desktop. > > Yes, but we would still need to patch webrtc-for-telegram-desktop as the > code currently refers to headers in the submodule directory. Now that I > think of it, it could actually be relatively easy with a regex substitution > (something like s!^#include "third_party/(srtp/[^"]*)"$!#include <\1>!) > on every file. That's of course assuming that the code only refers to > headers in the submodule. > > [[End of PGP Signed Part]] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-24 17:07 ` Distopico @ 2023-07-24 20:17 ` Raghav Gururajan via Guix-patches via 2023-07-26 7:24 ` Saku Laesvuori via Guix-patches via 0 siblings, 1 reply; 29+ messages in thread From: Raghav Gururajan via Guix-patches via @ 2023-07-24 20:17 UTC (permalink / raw) To: Distopico, Saku Laesvuori; +Cc: hako, 64748 [-- Attachment #1.1: Type: text/plain, Size: 780 bytes --] Hello, > I just tested trying to replace all the libsrtp but not luck from side, > they use for example: > - third_party/libsrtp/crypto/include/crypto_types.h > - third_party/libsrtp/include/srtp.h > >> Yes, but we would still need to patch webrtc-for-telegram-desktop as the >> code currently refers to headers in the submodule directory. Now that I >> think of it, it could actually be relatively easy with a regex substitution >> (something like s!^#include "third_party/(srtp/[^"]*)"$!#include <\1>!) >> on every file. That's of course assuming that the code only refers to >> headers in the submodule. I was wondering if we could add new phase after unpack that copies contents from libsrtp package to `[build-dir]/third_party/libsrtp`. Regards, RG. [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 236 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-24 20:17 ` Raghav Gururajan via Guix-patches via @ 2023-07-26 7:24 ` Saku Laesvuori via Guix-patches via 2023-07-26 18:54 ` Distopico 0 siblings, 1 reply; 29+ messages in thread From: Saku Laesvuori via Guix-patches via @ 2023-07-26 7:24 UTC (permalink / raw) To: Raghav Gururajan; +Cc: hako, 64748, Distopico [-- Attachment #1: Type: text/plain, Size: 345 bytes --] > I was wondering if we could add new phase after unpack that copies > contents from libsrtp package to `[build-dir]/third_party/libsrtp`. Simply replacing #$libsrtp-for-telegram-desktop with #$(package-source libsrtp) in the unpack-additional-sources phase seems to work (builds fine and telegram doesn't crash; I haven't tried calls). [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-26 7:24 ` Saku Laesvuori via Guix-patches via @ 2023-07-26 18:54 ` Distopico 0 siblings, 0 replies; 29+ messages in thread From: Distopico @ 2023-07-26 18:54 UTC (permalink / raw) To: Saku Laesvuori; +Cc: Raghav Gururajan, 64748, hako Tested with calls and works fine, changes in the last path On 2023-07-26, Saku Laesvuori <saku@laesvuori.fi> wrote: > [[PGP Signed Part:Undecided]] >> I was wondering if we could add new phase after unpack that copies >> contents from libsrtp package to `[build-dir]/third_party/libsrtp`. > > Simply replacing #$libsrtp-for-telegram-desktop with #$(package-source > libsrtp) in the unpack-additional-sources phase seems to work (builds fine and > telegram doesn't crash; I haven't tried calls). > > [[End of PGP Signed Part]] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v2] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-20 17:18 [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 Distopico 2023-07-20 19:33 ` Saku Laesvuori via Guix-patches via @ 2023-07-20 23:26 ` Distopico 2023-07-26 18:54 ` [bug#64748] [PATCH v3] " Distopico ` (3 subsequent siblings) 5 siblings, 0 replies; 29+ messages in thread From: Distopico @ 2023-07-20 23:26 UTC (permalink / raw) To: 64748; +Cc: Distopico, rg, hako, saku Fixes telegram-desktop call not working with openSSL 3 Add libsrtp-for-telegram-desktop because webrtc (tg_owt) now use `libsrtp` as submodule and in the code use code source directly from the submodule. see: https://github.com/telegramdesktop/tdesktop/issues/26108 * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to a45d8b8 revision commit * gnu/packages/telegram.scm (libsrtp-for-telegram-desktop): Add libsrtp a566a9 revision commit --- Add more context about the change in the commit message gnu/packages/telegram.scm | 31 +++++++++++++++++++++++++------ 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm index aa01c2f692..aeab249198 100644 --- a/gnu/packages/telegram.scm +++ b/gnu/packages/telegram.scm @@ -2,6 +2,7 @@ ;;; Copyright © 2021 Raghav Gururajan <rg@raghavgururajan.name> ;;; Copyright © 2022 Hilton Chain <hako@ultrarare.space> ;;; Copyright © 2023 Saku Laesvuori <saku@laesvuori.fi> +;;; Copyright © 2023 Camilo Q.S. (Distopico) <distopico@riseup.net> ;;; ;;; This file is part of GNU Guix. ;;; @@ -96,6 +97,21 @@ (define libyuv-for-telegram-desktop (base32 "0mm56p8iapfild2xdw4w8zi35c3xm06fgagiali644gnxdmnym6c"))))) +(define libsrtp-for-telegram-desktop + (let ((commit "a566a9cfcd619e8327784aa7cff4a1276dc1e895") + (revision "1494")) + (origin + (method git-fetch) + (uri (git-reference + (url "https://github.com/cisco/libsrtp") + (commit commit))) + (file-name (git-file-name + "libsrtp-for-telegram-desktop" + (git-version "0" revision commit))) + (sha256 + (base32 + "1ichw2v9s2mggi5p2wbbmlg55q4r48dxi3ks7ykfcfkmh7pb1w1s"))))) + (define cmake-helpers-for-telegram-desktop (origin (method git-fetch) @@ -265,8 +281,8 @@ (define tgcalls-for-telegram-desktop "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z")))) (define-public webrtc-for-telegram-desktop - (let ((commit "5098730b9eb6173f0b52068fe2555b7c1015123a") - (revision "328")) + (let ((commit "a45d8b8f0a99bd0e5118dda1dc4a8b7b3ad5dcfd") + (revision "388")) (hidden-package (package (name "webrtc-for-telegram-desktop") @@ -282,14 +298,14 @@ (define-public webrtc-for-telegram-desktop (file-name (git-file-name name version)) (sha256 - (base32 "1lk54zlrff59rj5k9dylsgz4sdds4728psrk8m3v9qn5y8d6z8qy")) + (base32 "1qs3ikkd6l56brj40cv6wlhx5gj5avisj9mj8ypjfwcyw9hb2n5y")) (modules '((guix build utils) (ice-9 ftw) (srfi srfi-1))) (snippet #~(begin (let ((keep - '("libsrtp" "rnnoise" + '("rnnoise" "libsrtp_config" ;; Not available in Guix. "pffft"))) (with-directory-excursion "src/third_party" @@ -311,9 +327,12 @@ (define-public webrtc-for-telegram-desktop (add-after 'unpack 'unpack-additional-sources (lambda _ (let* ((third-party (string-append (getcwd) "/src/third_party")) - (libyuv-to (string-append third-party "/libyuv"))) + (libyuv-to (string-append third-party "/libyuv")) + (libsrtp-to (string-append third-party "/libsrtp"))) (copy-recursively #$libyuv-for-telegram-desktop - libyuv-to))))))) + libyuv-to) + (copy-recursively #$libsrtp-for-telegram-desktop + libsrtp-to))))))) (native-inputs (list pkg-config python-wrapper yasm)) (inputs (list abseil-cpp-cxxstd17 base-commit: 13cb9b302868b5a966a6ae177412c474084f4bf1 -- 2.41.0 ^ permalink raw reply related [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v3] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-20 17:18 [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 Distopico 2023-07-20 19:33 ` Saku Laesvuori via Guix-patches via 2023-07-20 23:26 ` [bug#64748] [PATCH v2] " Distopico @ 2023-07-26 18:54 ` Distopico 2023-07-26 19:13 ` Saku Laesvuori via Guix-patches via 2023-07-26 19:21 ` [bug#64748] [PATCH v4] " Distopico ` (2 subsequent siblings) 5 siblings, 1 reply; 29+ messages in thread From: Distopico @ 2023-07-26 18:54 UTC (permalink / raw) To: 64748; +Cc: hako, Distopico, rg Fixes telegram-desktop call not working with openSSL 3 , Use sources from `libsrtp` instead of the submodule. gnu: libsrtp: Update to 2.5.0. see: https://github.com/telegramdesktop/tdesktop/issues/26108 * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to a45d8b8 revision commit * gnu/packages/telephony.scm (libsrtp): Update to 2.5.0. --- use sources from existing `libsrtp` package, update it to the last version gnu/packages/telegram.scm | 17 +++++++++++------ gnu/packages/telephony.scm | 4 ++-- 2 files changed, 13 insertions(+), 8 deletions(-) diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm index aa01c2f692..0b32b49d14 100644 --- a/gnu/packages/telegram.scm +++ b/gnu/packages/telegram.scm @@ -2,6 +2,7 @@ ;;; Copyright © 2021 Raghav Gururajan <rg@raghavgururajan.name> ;;; Copyright © 2022 Hilton Chain <hako@ultrarare.space> ;;; Copyright © 2023 Saku Laesvuori <saku@laesvuori.fi> +;;; Copyright © 2023 Camilo Q.S. (Distopico) <distopico@riseup.net> ;;; ;;; This file is part of GNU Guix. ;;; @@ -265,8 +266,8 @@ (define tgcalls-for-telegram-desktop "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z")))) (define-public webrtc-for-telegram-desktop - (let ((commit "5098730b9eb6173f0b52068fe2555b7c1015123a") - (revision "328")) + (let ((commit "a45d8b8f0a99bd0e5118dda1dc4a8b7b3ad5dcfd") + (revision "388")) (hidden-package (package (name "webrtc-for-telegram-desktop") @@ -282,14 +283,14 @@ (define-public webrtc-for-telegram-desktop (file-name (git-file-name name version)) (sha256 - (base32 "1lk54zlrff59rj5k9dylsgz4sdds4728psrk8m3v9qn5y8d6z8qy")) + (base32 "1qs3ikkd6l56brj40cv6wlhx5gj5avisj9mj8ypjfwcyw9hb2n5y")) (modules '((guix build utils) (ice-9 ftw) (srfi srfi-1))) (snippet #~(begin (let ((keep - '("libsrtp" "rnnoise" + '("rnnoise" "libsrtp_config" ;; Not available in Guix. "pffft"))) (with-directory-excursion "src/third_party" @@ -311,9 +312,13 @@ (define-public webrtc-for-telegram-desktop (add-after 'unpack 'unpack-additional-sources (lambda _ (let* ((third-party (string-append (getcwd) "/src/third_party")) - (libyuv-to (string-append third-party "/libyuv"))) + (libyuv-to (string-append third-party "/libyuv")) + (libsrtp-to (string-append third-party "/libsrtp"))) (copy-recursively #$libyuv-for-telegram-desktop - libyuv-to))))))) + libyuv-to) + (copy-recursively #$(package-source + libsrtp) + libsrtp-to))))))) (native-inputs (list pkg-config python-wrapper yasm)) (inputs (list abseil-cpp-cxxstd17 diff --git a/gnu/packages/telephony.scm b/gnu/packages/telephony.scm index aa65c4eaad..dc0adc7334 100644 --- a/gnu/packages/telephony.scm +++ b/gnu/packages/telephony.scm @@ -456,7 +456,7 @@ (define-public sipwitch (define-public libsrtp (package (name "libsrtp") - (version "2.4.2") + (version "2.5.0") (source (origin (method git-fetch) (uri (git-reference @@ -465,7 +465,7 @@ (define-public libsrtp (file-name (git-file-name name version)) (sha256 (base32 - "1gswpjm4jacfxmgglbf8hxi3yzsag4drk4q943p0wkmv21zj8l78")))) + "1ichw2v9s2mggi5p2wbbmlg55q4r48dxi3ks7ykfcfkmh7pb1w1s")))) (native-inputs (list psmisc ;some tests require 'killall' procps)) -- 2.41.0 ^ permalink raw reply related [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v3] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-26 18:54 ` [bug#64748] [PATCH v3] " Distopico @ 2023-07-26 19:13 ` Saku Laesvuori via Guix-patches via 2023-07-26 19:22 ` Distopico 0 siblings, 1 reply; 29+ messages in thread From: Saku Laesvuori via Guix-patches via @ 2023-07-26 19:13 UTC (permalink / raw) To: Distopico; +Cc: hako, 64748, rg [-- Attachment #1: Type: text/plain, Size: 730 bytes --] > diff --git a/gnu/packages/telephony.scm b/gnu/packages/telephony.scm > index aa65c4eaad..dc0adc7334 100644 > --- a/gnu/packages/telephony.scm > +++ b/gnu/packages/telephony.scm > @@ -456,7 +456,7 @@ (define-public sipwitch > (define-public libsrtp > (package > (name "libsrtp") > - (version "2.4.2") > + (version "2.5.0") This should be split to a separate commit (one package per commit). Also, according to `guix refresh --list-dependent libsrtp` libsrtp has 739 (>300) dependent packages and should thus not be updated on master to avoid a drop in substitute availability. Have you checked if the calls work with libsrtp 2.4.2 so that the telegram update could be applied to master without updating libsrtp? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v3] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-26 19:13 ` Saku Laesvuori via Guix-patches via @ 2023-07-26 19:22 ` Distopico 0 siblings, 0 replies; 29+ messages in thread From: Distopico @ 2023-07-26 19:22 UTC (permalink / raw) To: Saku Laesvuori; +Cc: hako, 64748, rg actually I tested first with 2.4.2 and it works fine from my side. On 2023-07-26, Saku Laesvuori <saku@laesvuori.fi> wrote: > [[PGP Signed Part:Undecided]] >> diff --git a/gnu/packages/telephony.scm b/gnu/packages/telephony.scm >> index aa65c4eaad..dc0adc7334 100644 >> --- a/gnu/packages/telephony.scm >> +++ b/gnu/packages/telephony.scm >> @@ -456,7 +456,7 @@ (define-public sipwitch >> (define-public libsrtp >> (package >> (name "libsrtp") >> - (version "2.4.2") >> + (version "2.5.0") > > This should be split to a separate commit (one package per commit). > > Also, according to `guix refresh --list-dependent libsrtp` libsrtp has > 739 (>300) dependent packages and should thus not be updated on master to avoid > a drop in substitute availability. Have you checked if the calls work > with libsrtp 2.4.2 so that the telegram update could be applied to > master without updating libsrtp? > > [[End of PGP Signed Part]] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v4] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-20 17:18 [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 Distopico ` (2 preceding siblings ...) 2023-07-26 18:54 ` [bug#64748] [PATCH v3] " Distopico @ 2023-07-26 19:21 ` Distopico 2023-07-26 20:09 ` Saku Laesvuori via Guix-patches via 2023-08-28 13:30 ` [bug#64748] [PATCH v6 0/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via 2023-08-28 14:56 ` [bug#64748] [PATCH v7 0/2] " Hilton Chain via Guix-patches via 5 siblings, 1 reply; 29+ messages in thread From: Distopico @ 2023-07-26 19:21 UTC (permalink / raw) To: 64748; +Cc: hako, Distopico, rg Fixes telegram-desktop call not working with openSSL 3 , Use sources from `libsrtp` instead of the submodule. see: https://github.com/telegramdesktop/tdesktop/issues/26108 * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to a45d8b8 revision commit --- gnu/packages/telegram.scm | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm index aa01c2f692..0b32b49d14 100644 --- a/gnu/packages/telegram.scm +++ b/gnu/packages/telegram.scm @@ -2,6 +2,7 @@ ;;; Copyright © 2021 Raghav Gururajan <rg@raghavgururajan.name> ;;; Copyright © 2022 Hilton Chain <hako@ultrarare.space> ;;; Copyright © 2023 Saku Laesvuori <saku@laesvuori.fi> +;;; Copyright © 2023 Camilo Q.S. (Distopico) <distopico@riseup.net> ;;; ;;; This file is part of GNU Guix. ;;; @@ -265,8 +266,8 @@ (define tgcalls-for-telegram-desktop "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z")))) (define-public webrtc-for-telegram-desktop - (let ((commit "5098730b9eb6173f0b52068fe2555b7c1015123a") - (revision "328")) + (let ((commit "a45d8b8f0a99bd0e5118dda1dc4a8b7b3ad5dcfd") + (revision "388")) (hidden-package (package (name "webrtc-for-telegram-desktop") @@ -282,14 +283,14 @@ (define-public webrtc-for-telegram-desktop (file-name (git-file-name name version)) (sha256 - (base32 "1lk54zlrff59rj5k9dylsgz4sdds4728psrk8m3v9qn5y8d6z8qy")) + (base32 "1qs3ikkd6l56brj40cv6wlhx5gj5avisj9mj8ypjfwcyw9hb2n5y")) (modules '((guix build utils) (ice-9 ftw) (srfi srfi-1))) (snippet #~(begin (let ((keep - '("libsrtp" "rnnoise" + '("rnnoise" "libsrtp_config" ;; Not available in Guix. "pffft"))) (with-directory-excursion "src/third_party" @@ -311,9 +312,13 @@ (define-public webrtc-for-telegram-desktop (add-after 'unpack 'unpack-additional-sources (lambda _ (let* ((third-party (string-append (getcwd) "/src/third_party")) - (libyuv-to (string-append third-party "/libyuv"))) + (libyuv-to (string-append third-party "/libyuv")) + (libsrtp-to (string-append third-party "/libsrtp"))) (copy-recursively #$libyuv-for-telegram-desktop - libyuv-to))))))) + libyuv-to) + (copy-recursively #$(package-source + libsrtp) + libsrtp-to))))))) (native-inputs (list pkg-config python-wrapper yasm)) (inputs (list abseil-cpp-cxxstd17 -- 2.41.0 ^ permalink raw reply related [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v4] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-26 19:21 ` [bug#64748] [PATCH v4] " Distopico @ 2023-07-26 20:09 ` Saku Laesvuori via Guix-patches via 2023-08-05 2:59 ` [bug#64748] [PATCH] " Distopico 0 siblings, 1 reply; 29+ messages in thread From: Saku Laesvuori via Guix-patches via @ 2023-07-26 20:09 UTC (permalink / raw) To: Distopico; +Cc: hako, 64748, rg [-- Attachment #1: Type: text/plain, Size: 517 bytes --] > Fixes telegram-desktop call not working with openSSL 3 , > Use sources from `libsrtp` instead of the submodule. > > see: https://github.com/telegramdesktop/tdesktop/issues/26108 > > * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to a45d8b8 revision commit > --- I wonder if this ChangeLog should mention something about adding libsrtp sources to the unpack-additional-sources phase. Otherwise it looks good to me and is just waiting for someone with more experience/commit access. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-07-26 20:09 ` Saku Laesvuori via Guix-patches via @ 2023-08-05 2:59 ` Distopico 2023-08-07 9:38 ` Hilton Chain via Guix-patches via 0 siblings, 1 reply; 29+ messages in thread From: Distopico @ 2023-08-05 2:59 UTC (permalink / raw) To: 64748; +Cc: hako, Distopico, rg * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to a45d8b8 revision commit Fixes `telegram-desktop` calls that are not working with OpenSSL 3. Now, `webrtc-for-telegram-desktop` uses sources from the Guix `libsrtp` package instead of the submodule in the `unpack-additional-sources` phase. see: https://github.com/telegramdesktop/tdesktop/issues/26108 --- * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to a45d8b8 revision commit Fixes `telegram-desktop` calls that are not working with OpenSSL 3. Now, `webrtc-for-telegram-desktop` uses sources from the Guix `libsrtp` package instead of the submodule in the `unpack-additional-sources` phase. gnu/packages/telegram.scm | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm index aa01c2f692..0b32b49d14 100644 --- a/gnu/packages/telegram.scm +++ b/gnu/packages/telegram.scm @@ -2,6 +2,7 @@ ;;; Copyright © 2021 Raghav Gururajan <rg@raghavgururajan.name> ;;; Copyright © 2022 Hilton Chain <hako@ultrarare.space> ;;; Copyright © 2023 Saku Laesvuori <saku@laesvuori.fi> +;;; Copyright © 2023 Camilo Q.S. (Distopico) <distopico@riseup.net> ;;; ;;; This file is part of GNU Guix. ;;; @@ -265,8 +266,8 @@ (define tgcalls-for-telegram-desktop "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z")))) (define-public webrtc-for-telegram-desktop - (let ((commit "5098730b9eb6173f0b52068fe2555b7c1015123a") - (revision "328")) + (let ((commit "a45d8b8f0a99bd0e5118dda1dc4a8b7b3ad5dcfd") + (revision "388")) (hidden-package (package (name "webrtc-for-telegram-desktop") @@ -282,14 +283,14 @@ (define-public webrtc-for-telegram-desktop (file-name (git-file-name name version)) (sha256 - (base32 "1lk54zlrff59rj5k9dylsgz4sdds4728psrk8m3v9qn5y8d6z8qy")) + (base32 "1qs3ikkd6l56brj40cv6wlhx5gj5avisj9mj8ypjfwcyw9hb2n5y")) (modules '((guix build utils) (ice-9 ftw) (srfi srfi-1))) (snippet #~(begin (let ((keep - '("libsrtp" "rnnoise" + '("rnnoise" "libsrtp_config" ;; Not available in Guix. "pffft"))) (with-directory-excursion "src/third_party" @@ -311,9 +312,13 @@ (define-public webrtc-for-telegram-desktop (add-after 'unpack 'unpack-additional-sources (lambda _ (let* ((third-party (string-append (getcwd) "/src/third_party")) - (libyuv-to (string-append third-party "/libyuv"))) + (libyuv-to (string-append third-party "/libyuv")) + (libsrtp-to (string-append third-party "/libsrtp"))) (copy-recursively #$libyuv-for-telegram-desktop - libyuv-to))))))) + libyuv-to) + (copy-recursively #$(package-source + libsrtp) + libsrtp-to))))))) (native-inputs (list pkg-config python-wrapper yasm)) (inputs (list abseil-cpp-cxxstd17 base-commit: cf9904bcc8dd03e73675475bb4d8746dc434e415 -- 2.41.0 ^ permalink raw reply related [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-08-05 2:59 ` [bug#64748] [PATCH] " Distopico @ 2023-08-07 9:38 ` Hilton Chain via Guix-patches via 2023-08-07 12:26 ` Hilton Chain via Guix-patches via 0 siblings, 1 reply; 29+ messages in thread From: Hilton Chain via Guix-patches via @ 2023-08-07 9:38 UTC (permalink / raw) To: Distopico; +Cc: Raghav Gururajan, Saku Laesvuori, 64748 Hi, Sorry for the late reply (in this thread). On Sat, 05 Aug 2023 10:59:50 +0800, Distopico wrote: > > * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to a45d8b8 revision commit > > Fixes `telegram-desktop` calls that are not working with OpenSSL 3. > Now, `webrtc-for-telegram-desktop` uses sources from the Guix `libsrtp` > package instead of the submodule in the `unpack-additional-sources` phase. > > see: https://github.com/telegramdesktop/tdesktop/issues/26108 > > --- > * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to a45d8b8 revision commit > > Fixes `telegram-desktop` calls that are not working with OpenSSL 3. > Now, `webrtc-for-telegram-desktop` uses sources from the Guix `libsrtp` > package instead of the submodule in the `unpack-additional-sources` phase. Have you checked <https://github.com/desktop-app/tg_owt/pull/123> (Unbundle libSRTP) yet? `git grep include.*libsrtp' shows that `srtp_priv.h' is used, and it's not present in libsrtp@2.4.2. But since you have tested, I think it's not an issue here. For commit message, I think there's a 80-character limit and the long sentences could be split into the [...] below as well. --8<---------------cut here---------------start------------->8--- * gnu/packages/telegram.scm (...): ... [snippet]: [...] [#:phases]<unpack-additional-sources>: [...] --8<---------------cut here---------------end--------------->8--- 0-388.a45d8b8 can be used to describe the version if you prefer. Patch LGTM, but if the aforementioned patch works, it's better to add it. Thanks ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 2023-08-07 9:38 ` Hilton Chain via Guix-patches via @ 2023-08-07 12:26 ` Hilton Chain via Guix-patches via 0 siblings, 0 replies; 29+ messages in thread From: Hilton Chain via Guix-patches via @ 2023-08-07 12:26 UTC (permalink / raw) To: Distopico; +Cc: Raghav Gururajan, Saku Laesvuori, 64748 On Mon, 07 Aug 2023 17:38:57 +0800, Hilton Chain wrote: > `git grep include.*libsrtp' shows that `srtp_priv.h' is used, and it's > not present in libsrtp@2.4.2. But since you have tested, I think it's > not an issue here. Found it in libsrtp source! So the reply's unrelated here. ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v6 0/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942. 2023-07-20 17:18 [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 Distopico ` (3 preceding siblings ...) 2023-07-26 19:21 ` [bug#64748] [PATCH v4] " Distopico @ 2023-08-28 13:30 ` Hilton Chain via Guix-patches via 2023-08-28 13:31 ` [bug#64748] [PATCH v6 1/2] gnu: libyuv-for-telegram-desktop: Update to 0-2439.77c2121 Hilton Chain via Guix-patches via 2023-08-28 13:31 ` [bug#64748] [PATCH v6 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via 2023-08-28 14:56 ` [bug#64748] [PATCH v7 0/2] " Hilton Chain via Guix-patches via 5 siblings, 2 replies; 29+ messages in thread From: Hilton Chain via Guix-patches via @ 2023-08-28 13:30 UTC (permalink / raw) To: 64748; +Cc: Hilton Chain, Distopico, Saku Laesvuori, Raghav Gururajan Finished my own request for unbundling libSRTP and updated the package to the latest commit. Don't have a Telegram account, so I can't test the functionality. Please let me know if it works. Thanks Distopico (1): gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942. Hilton Chain (1): gnu: libyuv-for-telegram-desktop: Update to 0-2439.77c2121. gnu/local.mk | 1 + ...or-telegram-desktop-unbundle-libsrtp.patch | 137 ++++++++++++++++++ gnu/packages/telegram.scm | 22 ++- 3 files changed, 152 insertions(+), 8 deletions(-) create mode 100644 gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch base-commit: 3b927302124de6eadaa8e0a7280002cd85c04d33 -- 2.41.0 ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v6 1/2] gnu: libyuv-for-telegram-desktop: Update to 0-2439.77c2121. 2023-08-28 13:30 ` [bug#64748] [PATCH v6 0/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via @ 2023-08-28 13:31 ` Hilton Chain via Guix-patches via 2023-08-28 13:31 ` [bug#64748] [PATCH v6 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via 1 sibling, 0 replies; 29+ messages in thread From: Hilton Chain via Guix-patches via @ 2023-08-28 13:31 UTC (permalink / raw) To: 64748; +Cc: Hilton Chain * gnu/packages/telegram.scm (libyuv-for-telegram-desktop): Update to 0-2439.77c2121. --- gnu/packages/telegram.scm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm index dffc6b8ff4..88daea7d7d 100644 --- a/gnu/packages/telegram.scm +++ b/gnu/packages/telegram.scm @@ -83,8 +83,8 @@ (define-module (gnu packages telegram) (define %telegram-version "4.8.1") (define libyuv-for-telegram-desktop - (let ((commit "00950840d1c9bcbb3eb6ebc5aac5793e71166c8b") - (revision "2212")) + (let ((commit "77c2121f7e6b8e694d6e908bbbe9be24214097da") + (revision "2439")) (origin (method git-fetch) (uri (git-reference @@ -95,7 +95,7 @@ (define libyuv-for-telegram-desktop (git-version "0" revision commit))) (sha256 (base32 - "0mm56p8iapfild2xdw4w8zi35c3xm06fgagiali644gnxdmnym6c"))))) + "1b4k8yskr9ffl5k8s9i0af1gn1pavsfixla26vh8bij69rdr7f9c"))))) (define cmake-helpers-for-telegram-desktop (origin -- 2.41.0 ^ permalink raw reply related [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v6 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942. 2023-08-28 13:30 ` [bug#64748] [PATCH v6 0/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via 2023-08-28 13:31 ` [bug#64748] [PATCH v6 1/2] gnu: libyuv-for-telegram-desktop: Update to 0-2439.77c2121 Hilton Chain via Guix-patches via @ 2023-08-28 13:31 ` Hilton Chain via Guix-patches via 1 sibling, 0 replies; 29+ messages in thread From: Hilton Chain via Guix-patches via @ 2023-08-28 13:31 UTC (permalink / raw) To: 64748; +Cc: Distopico From: Distopico <distopico@riseup.net> * gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch: New file. * gnu/local.mk (dist_patch_DATA): Register it. * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to 0-389.0532942, which fixes `telegram-desktop` calls that are not working with OpenSSL 3. [source]<patches>: Add the patch. <snippet>: Unbundle libsrtp. Keep libsrtp_config. [inputs]: Add libsrtp. --- gnu/local.mk | 1 + ...or-telegram-desktop-unbundle-libsrtp.patch | 137 ++++++++++++++++++ gnu/packages/telegram.scm | 16 +- 3 files changed, 149 insertions(+), 5 deletions(-) create mode 100644 gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch diff --git a/gnu/local.mk b/gnu/local.mk index 8f409b7ea8..684f9f3ec4 100644 --- a/gnu/local.mk +++ b/gnu/local.mk @@ -2094,6 +2094,7 @@ dist_patch_DATA = \ %D%/packages/patches/wdl-link-libs-and-fix-jnetlib.patch \ %D%/packages/patches/webkitgtk-adjust-bubblewrap-paths.patch \ %D%/packages/patches/webrtc-audio-processing-big-endian.patch \ + %D%/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch \ %D%/packages/patches/websocketpp-fix-for-cmake-3.15.patch \ %D%/packages/patches/wmctrl-64-fix.patch \ %D%/packages/patches/wmfire-update-for-new-gdk-versions.patch \ diff --git a/gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch b/gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch new file mode 100644 index 0000000000..486f0b1aa5 --- /dev/null +++ b/gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch @@ -0,0 +1,137 @@ +From 62672f3756ecf218252098211d78c13369ab6d28 Mon Sep 17 00:00:00 2001 +From: Nicholas Guriev <nicholas@guriev.su> +Date: Thu, 4 May 2023 16:21:09 +0300 +Subject: [PATCH] Unbundle libSRTP + +Avoid private symbols and link against system-wide libSRTP. The excluded code +in SrtpSession looks unreachable from the call integration in Telegram Desktop. +--- + CMakeLists.txt | 3 +++ + cmake/libsrtp.cmake | 13 +++++++++++++ + src/pc/external_hmac.cc | 1 - + src/pc/external_hmac.h | 9 ++++++--- + src/pc/srtp_session.cc | 16 ++++++++++++++-- + 5 files changed, 36 insertions(+), 6 deletions(-) + +diff --git a/CMakeLists.txt b/CMakeLists.txt +index af7d24c21..66bec8fdf 100644 +--- a/CMakeLists.txt ++++ b/CMakeLists.txt +@@ -2647,6 +2647,9 @@ if (TG_OWT_USE_PROTOBUF) + list(APPEND export_targets proto) + endif() + ++if (LIBSRTP_FOUND) ++ target_compile_definitions(tg_owt PRIVATE HAVE_LIBSRTP) ++endif() + if (NOT absl_FOUND) + list(APPEND export_targets libabsl) + endif() +diff --git a/cmake/libsrtp.cmake b/cmake/libsrtp.cmake +index 5124312d2..01f051606 100644 +--- a/cmake/libsrtp.cmake ++++ b/cmake/libsrtp.cmake +@@ -1,3 +1,16 @@ ++find_package(PkgConfig REQUIRED) ++pkg_check_modules(LIBSRTP libsrtp2) ++ ++if (LIBSRTP_FOUND) ++ add_library(libsrtp INTERFACE EXCLUDE_FROM_ALL) ++ add_library(tg_owt::libsrtp ALIAS libsrtp) ++ ++ target_include_directories(libsrtp INTERFACE ${LIBSRTP_INCLUDE_DIRS} ${LIBSRTP_CFLAGS_OTHER}) ++ target_link_libraries(libsrtp INTERFACE ${LIBSRTP_LINK_LIBRARIES} ${LIBSRTP_LDFLAGS_OTHER}) ++ ++ return() ++endif() ++ + add_library(libsrtp OBJECT EXCLUDE_FROM_ALL) + init_target(libsrtp) + add_library(tg_owt::libsrtp ALIAS libsrtp) +diff --git a/src/pc/external_hmac.cc b/src/pc/external_hmac.cc +index 27b5d0e5a..222f5d9ae 100644 +--- a/src/pc/external_hmac.cc ++++ b/src/pc/external_hmac.cc +@@ -15,7 +15,6 @@ + + #include "rtc_base/logging.h" + #include "rtc_base/zero_memory.h" +-#include "third_party/libsrtp/include/srtp.h" + + // Begin test case 0 */ + static const uint8_t kExternalHmacTestCase0Key[20] = { +diff --git a/src/pc/external_hmac.h b/src/pc/external_hmac.h +index c5071fc19..8fdc2f1a7 100644 +--- a/src/pc/external_hmac.h ++++ b/src/pc/external_hmac.h +@@ -30,9 +30,12 @@ + + #include <stdint.h> + +-#include "third_party/libsrtp/crypto/include/crypto_types.h" +-#include "third_party/libsrtp/include/srtp.h" +-#include "third_party/libsrtp/include/srtp_priv.h" ++#ifdef HAVE_LIBSRTP ++# include <srtp2/auth.h> ++# include <srtp2/srtp.h> ++#else ++# include "srtp_priv.h" ++#endif + + #define EXTERNAL_HMAC_SHA1 SRTP_HMAC_SHA1 + 1 + #define HMAC_KEY_LENGTH 20 +diff --git a/src/pc/srtp_session.cc b/src/pc/srtp_session.cc +index 7d1aaf2d6..7b5a789b0 100644 +--- a/src/pc/srtp_session.cc ++++ b/src/pc/srtp_session.cc +@@ -30,8 +30,12 @@ + #include "rtc_base/thread_annotations.h" + #include "rtc_base/time_utils.h" + #include "system_wrappers/include/metrics.h" +-#include "third_party/libsrtp/include/srtp.h" +-#include "third_party/libsrtp/include/srtp_priv.h" ++ ++#ifdef HAVE_LIBSRTP ++# include <srtp2/srtp.h> ++#else ++# include "srtp_priv.h" ++#endif + + namespace cricket { + +@@ -290,6 +294,9 @@ bool SrtpSession::UnprotectRtcp(void* p, int in_len, int* out_len) { + bool SrtpSession::GetRtpAuthParams(uint8_t** key, int* key_len, int* tag_len) { + RTC_DCHECK(thread_checker_.IsCurrent()); + RTC_DCHECK(IsExternalAuthActive()); ++#ifdef HAVE_LIBSRTP ++ return false; ++#else + if (!IsExternalAuthActive()) { + return false; + } +@@ -313,6 +320,7 @@ bool SrtpSession::GetRtpAuthParams(uint8_t** key, int* key_len, int* tag_len) { + *key_len = external_hmac->key_length; + *tag_len = rtp_auth_tag_len_; + return true; ++#endif + } + + int SrtpSession::GetSrtpOverhead() const { +@@ -336,6 +344,9 @@ bool SrtpSession::GetSendStreamPacketIndex(void* p, + int in_len, + int64_t* index) { + RTC_DCHECK(thread_checker_.IsCurrent()); ++#ifdef HAVE_LIBSRTP ++ return false; ++#else + srtp_hdr_t* hdr = reinterpret_cast<srtp_hdr_t*>(p); + srtp_stream_ctx_t* stream = srtp_get_stream(session_, hdr->ssrc); + if (!stream) { +@@ -346,6 +357,7 @@ bool SrtpSession::GetSendStreamPacketIndex(void* p, + *index = static_cast<int64_t>(rtc::NetworkToHost64( + srtp_rdbx_get_packet_index(&stream->rtp_rdbx) << 16)); + return true; ++#endif + } + + bool SrtpSession::DoSetKey(int type, diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm index 88daea7d7d..a3d5dc273a 100644 --- a/gnu/packages/telegram.scm +++ b/gnu/packages/telegram.scm @@ -1,8 +1,9 @@ ;;; GNU Guix --- Functional package management for GNU ;;; Copyright © 2021 Raghav Gururajan <rg@raghavgururajan.name> -;;; Copyright © 2022 Hilton Chain <hako@ultrarare.space> +;;; Copyright © 2022, 2023 Hilton Chain <hako@ultrarare.space> ;;; Copyright © 2023 Saku Laesvuori <saku@laesvuori.fi> ;;; Copyright © 2023 Lu Hui <luhux76@gmail.com> +;;; Copyright © 2023 Camilo Q.S. (Distopico) <distopico@riseup.net> ;;; ;;; This file is part of GNU Guix. ;;; @@ -266,8 +267,8 @@ (define tgcalls-for-telegram-desktop "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z")))) (define-public webrtc-for-telegram-desktop - (let ((commit "5098730b9eb6173f0b52068fe2555b7c1015123a") - (revision "328")) + (let ((commit "0532942ac6176a66ef184fb728a4cbb02958fc0b") + (revision "389")) (hidden-package (package (name "webrtc-for-telegram-desktop") @@ -283,14 +284,18 @@ (define-public webrtc-for-telegram-desktop (file-name (git-file-name name version)) (sha256 - (base32 "1lk54zlrff59rj5k9dylsgz4sdds4728psrk8m3v9qn5y8d6z8qy")) + (base32 "0fary99yl1ddk5zjpfy0pyb5brd268j41plcnvv9qjyf0wj9hf2k")) + (patches + (search-patches + ;; https://github.com/desktop-app/tg_owt/pull/123 + "webrtc-for-telegram-desktop-unbundle-libsrtp.patch")) (modules '((guix build utils) (ice-9 ftw) (srfi srfi-1))) (snippet #~(begin (let ((keep - '("libsrtp" "rnnoise" + '("libsrtp_config" "rnnoise" ;; Not available in Guix. "pffft"))) (with-directory-excursion "src/third_party" @@ -325,6 +330,7 @@ (define-public webrtc-for-telegram-desktop libdrm libglvnd libjpeg-turbo + libsrtp libvpx libxcomposite libxdamage -- 2.41.0 ^ permalink raw reply related [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v7 0/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942. 2023-07-20 17:18 [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 Distopico ` (4 preceding siblings ...) 2023-08-28 13:30 ` [bug#64748] [PATCH v6 0/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via @ 2023-08-28 14:56 ` Hilton Chain via Guix-patches via 2023-08-28 15:00 ` [bug#64748] [PATCH v7 1/2] gnu: libyuv-for-telegram-desktop: Update to 0-2439.77c2121 Hilton Chain via Guix-patches via 2023-08-28 15:00 ` [bug#64748] [PATCH v7 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via 5 siblings, 2 replies; 29+ messages in thread From: Hilton Chain via Guix-patches via @ 2023-08-28 14:56 UTC (permalink / raw) To: 64748; +Cc: Hilton Chain V6 -> V7: Don't keep the unnecessary libsrtp_config. Distopico (1): gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942. Hilton Chain (1): gnu: libyuv-for-telegram-desktop: Update to 0-2439.77c2121. gnu/local.mk | 1 + ...or-telegram-desktop-unbundle-libsrtp.patch | 137 ++++++++++++++++++ gnu/packages/telegram.scm | 22 ++- 3 files changed, 152 insertions(+), 8 deletions(-) create mode 100644 gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch base-commit: 4a49a89b09d07d96f4d6ef3c0ca93801370f7b20 -- 2.41.0 ^ permalink raw reply [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v7 1/2] gnu: libyuv-for-telegram-desktop: Update to 0-2439.77c2121. 2023-08-28 14:56 ` [bug#64748] [PATCH v7 0/2] " Hilton Chain via Guix-patches via @ 2023-08-28 15:00 ` Hilton Chain via Guix-patches via 2023-08-28 15:00 ` [bug#64748] [PATCH v7 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via 1 sibling, 0 replies; 29+ messages in thread From: Hilton Chain via Guix-patches via @ 2023-08-28 15:00 UTC (permalink / raw) To: 64748; +Cc: Hilton Chain * gnu/packages/telegram.scm (libyuv-for-telegram-desktop): Update to 0-2439.77c2121. --- (no change since v6) gnu/packages/telegram.scm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm index dffc6b8ff4..88daea7d7d 100644 --- a/gnu/packages/telegram.scm +++ b/gnu/packages/telegram.scm @@ -83,8 +83,8 @@ (define-module (gnu packages telegram) (define %telegram-version "4.8.1") (define libyuv-for-telegram-desktop - (let ((commit "00950840d1c9bcbb3eb6ebc5aac5793e71166c8b") - (revision "2212")) + (let ((commit "77c2121f7e6b8e694d6e908bbbe9be24214097da") + (revision "2439")) (origin (method git-fetch) (uri (git-reference @@ -95,7 +95,7 @@ (define libyuv-for-telegram-desktop (git-version "0" revision commit))) (sha256 (base32 - "0mm56p8iapfild2xdw4w8zi35c3xm06fgagiali644gnxdmnym6c"))))) + "1b4k8yskr9ffl5k8s9i0af1gn1pavsfixla26vh8bij69rdr7f9c"))))) (define cmake-helpers-for-telegram-desktop (origin -- 2.41.0 ^ permalink raw reply related [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v7 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942. 2023-08-28 14:56 ` [bug#64748] [PATCH v7 0/2] " Hilton Chain via Guix-patches via 2023-08-28 15:00 ` [bug#64748] [PATCH v7 1/2] gnu: libyuv-for-telegram-desktop: Update to 0-2439.77c2121 Hilton Chain via Guix-patches via @ 2023-08-28 15:00 ` Hilton Chain via Guix-patches via 2023-08-29 18:13 ` Distopico 1 sibling, 1 reply; 29+ messages in thread From: Hilton Chain via Guix-patches via @ 2023-08-28 15:00 UTC (permalink / raw) To: 64748; +Cc: Hilton Chain, Distopico From: Distopico <distopico@riseup.net> * gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch: New file. * gnu/local.mk (dist_patch_DATA): Register it. * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to 0-389.0532942, which fixes `telegram-desktop` calls that are not working with OpenSSL 3. [source]<patches>: Add the patch. <snippet>: Unbundle libsrtp. [inputs]: Add libsrtp. Signed-off-by: Hilton Chain <hako@ultrarare.space> --- gnu/local.mk | 1 + ...or-telegram-desktop-unbundle-libsrtp.patch | 137 ++++++++++++++++++ gnu/packages/telegram.scm | 16 +- 3 files changed, 149 insertions(+), 5 deletions(-) create mode 100644 gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch diff --git a/gnu/local.mk b/gnu/local.mk index 48187d7e64..5ab021eaed 100644 --- a/gnu/local.mk +++ b/gnu/local.mk @@ -2093,6 +2093,7 @@ dist_patch_DATA = \ %D%/packages/patches/wdl-link-libs-and-fix-jnetlib.patch \ %D%/packages/patches/webkitgtk-adjust-bubblewrap-paths.patch \ %D%/packages/patches/webrtc-audio-processing-big-endian.patch \ + %D%/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch \ %D%/packages/patches/websocketpp-fix-for-cmake-3.15.patch \ %D%/packages/patches/wmctrl-64-fix.patch \ %D%/packages/patches/wmfire-update-for-new-gdk-versions.patch \ diff --git a/gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch b/gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch new file mode 100644 index 0000000000..486f0b1aa5 --- /dev/null +++ b/gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch @@ -0,0 +1,137 @@ +From 62672f3756ecf218252098211d78c13369ab6d28 Mon Sep 17 00:00:00 2001 +From: Nicholas Guriev <nicholas@guriev.su> +Date: Thu, 4 May 2023 16:21:09 +0300 +Subject: [PATCH] Unbundle libSRTP + +Avoid private symbols and link against system-wide libSRTP. The excluded code +in SrtpSession looks unreachable from the call integration in Telegram Desktop. +--- + CMakeLists.txt | 3 +++ + cmake/libsrtp.cmake | 13 +++++++++++++ + src/pc/external_hmac.cc | 1 - + src/pc/external_hmac.h | 9 ++++++--- + src/pc/srtp_session.cc | 16 ++++++++++++++-- + 5 files changed, 36 insertions(+), 6 deletions(-) + +diff --git a/CMakeLists.txt b/CMakeLists.txt +index af7d24c21..66bec8fdf 100644 +--- a/CMakeLists.txt ++++ b/CMakeLists.txt +@@ -2647,6 +2647,9 @@ if (TG_OWT_USE_PROTOBUF) + list(APPEND export_targets proto) + endif() + ++if (LIBSRTP_FOUND) ++ target_compile_definitions(tg_owt PRIVATE HAVE_LIBSRTP) ++endif() + if (NOT absl_FOUND) + list(APPEND export_targets libabsl) + endif() +diff --git a/cmake/libsrtp.cmake b/cmake/libsrtp.cmake +index 5124312d2..01f051606 100644 +--- a/cmake/libsrtp.cmake ++++ b/cmake/libsrtp.cmake +@@ -1,3 +1,16 @@ ++find_package(PkgConfig REQUIRED) ++pkg_check_modules(LIBSRTP libsrtp2) ++ ++if (LIBSRTP_FOUND) ++ add_library(libsrtp INTERFACE EXCLUDE_FROM_ALL) ++ add_library(tg_owt::libsrtp ALIAS libsrtp) ++ ++ target_include_directories(libsrtp INTERFACE ${LIBSRTP_INCLUDE_DIRS} ${LIBSRTP_CFLAGS_OTHER}) ++ target_link_libraries(libsrtp INTERFACE ${LIBSRTP_LINK_LIBRARIES} ${LIBSRTP_LDFLAGS_OTHER}) ++ ++ return() ++endif() ++ + add_library(libsrtp OBJECT EXCLUDE_FROM_ALL) + init_target(libsrtp) + add_library(tg_owt::libsrtp ALIAS libsrtp) +diff --git a/src/pc/external_hmac.cc b/src/pc/external_hmac.cc +index 27b5d0e5a..222f5d9ae 100644 +--- a/src/pc/external_hmac.cc ++++ b/src/pc/external_hmac.cc +@@ -15,7 +15,6 @@ + + #include "rtc_base/logging.h" + #include "rtc_base/zero_memory.h" +-#include "third_party/libsrtp/include/srtp.h" + + // Begin test case 0 */ + static const uint8_t kExternalHmacTestCase0Key[20] = { +diff --git a/src/pc/external_hmac.h b/src/pc/external_hmac.h +index c5071fc19..8fdc2f1a7 100644 +--- a/src/pc/external_hmac.h ++++ b/src/pc/external_hmac.h +@@ -30,9 +30,12 @@ + + #include <stdint.h> + +-#include "third_party/libsrtp/crypto/include/crypto_types.h" +-#include "third_party/libsrtp/include/srtp.h" +-#include "third_party/libsrtp/include/srtp_priv.h" ++#ifdef HAVE_LIBSRTP ++# include <srtp2/auth.h> ++# include <srtp2/srtp.h> ++#else ++# include "srtp_priv.h" ++#endif + + #define EXTERNAL_HMAC_SHA1 SRTP_HMAC_SHA1 + 1 + #define HMAC_KEY_LENGTH 20 +diff --git a/src/pc/srtp_session.cc b/src/pc/srtp_session.cc +index 7d1aaf2d6..7b5a789b0 100644 +--- a/src/pc/srtp_session.cc ++++ b/src/pc/srtp_session.cc +@@ -30,8 +30,12 @@ + #include "rtc_base/thread_annotations.h" + #include "rtc_base/time_utils.h" + #include "system_wrappers/include/metrics.h" +-#include "third_party/libsrtp/include/srtp.h" +-#include "third_party/libsrtp/include/srtp_priv.h" ++ ++#ifdef HAVE_LIBSRTP ++# include <srtp2/srtp.h> ++#else ++# include "srtp_priv.h" ++#endif + + namespace cricket { + +@@ -290,6 +294,9 @@ bool SrtpSession::UnprotectRtcp(void* p, int in_len, int* out_len) { + bool SrtpSession::GetRtpAuthParams(uint8_t** key, int* key_len, int* tag_len) { + RTC_DCHECK(thread_checker_.IsCurrent()); + RTC_DCHECK(IsExternalAuthActive()); ++#ifdef HAVE_LIBSRTP ++ return false; ++#else + if (!IsExternalAuthActive()) { + return false; + } +@@ -313,6 +320,7 @@ bool SrtpSession::GetRtpAuthParams(uint8_t** key, int* key_len, int* tag_len) { + *key_len = external_hmac->key_length; + *tag_len = rtp_auth_tag_len_; + return true; ++#endif + } + + int SrtpSession::GetSrtpOverhead() const { +@@ -336,6 +344,9 @@ bool SrtpSession::GetSendStreamPacketIndex(void* p, + int in_len, + int64_t* index) { + RTC_DCHECK(thread_checker_.IsCurrent()); ++#ifdef HAVE_LIBSRTP ++ return false; ++#else + srtp_hdr_t* hdr = reinterpret_cast<srtp_hdr_t*>(p); + srtp_stream_ctx_t* stream = srtp_get_stream(session_, hdr->ssrc); + if (!stream) { +@@ -346,6 +357,7 @@ bool SrtpSession::GetSendStreamPacketIndex(void* p, + *index = static_cast<int64_t>(rtc::NetworkToHost64( + srtp_rdbx_get_packet_index(&stream->rtp_rdbx) << 16)); + return true; ++#endif + } + + bool SrtpSession::DoSetKey(int type, diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm index 88daea7d7d..4814c22b6c 100644 --- a/gnu/packages/telegram.scm +++ b/gnu/packages/telegram.scm @@ -1,8 +1,9 @@ ;;; GNU Guix --- Functional package management for GNU ;;; Copyright © 2021 Raghav Gururajan <rg@raghavgururajan.name> -;;; Copyright © 2022 Hilton Chain <hako@ultrarare.space> +;;; Copyright © 2022, 2023 Hilton Chain <hako@ultrarare.space> ;;; Copyright © 2023 Saku Laesvuori <saku@laesvuori.fi> ;;; Copyright © 2023 Lu Hui <luhux76@gmail.com> +;;; Copyright © 2023 Camilo Q.S. (Distopico) <distopico@riseup.net> ;;; ;;; This file is part of GNU Guix. ;;; @@ -266,8 +267,8 @@ (define tgcalls-for-telegram-desktop "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z")))) (define-public webrtc-for-telegram-desktop - (let ((commit "5098730b9eb6173f0b52068fe2555b7c1015123a") - (revision "328")) + (let ((commit "0532942ac6176a66ef184fb728a4cbb02958fc0b") + (revision "389")) (hidden-package (package (name "webrtc-for-telegram-desktop") @@ -283,14 +284,18 @@ (define-public webrtc-for-telegram-desktop (file-name (git-file-name name version)) (sha256 - (base32 "1lk54zlrff59rj5k9dylsgz4sdds4728psrk8m3v9qn5y8d6z8qy")) + (base32 "0fary99yl1ddk5zjpfy0pyb5brd268j41plcnvv9qjyf0wj9hf2k")) + (patches + (search-patches + ;; https://github.com/desktop-app/tg_owt/pull/123 + "webrtc-for-telegram-desktop-unbundle-libsrtp.patch")) (modules '((guix build utils) (ice-9 ftw) (srfi srfi-1))) (snippet #~(begin (let ((keep - '("libsrtp" "rnnoise" + '("rnnoise" ;; Not available in Guix. "pffft"))) (with-directory-excursion "src/third_party" @@ -325,6 +330,7 @@ (define-public webrtc-for-telegram-desktop libdrm libglvnd libjpeg-turbo + libsrtp libvpx libxcomposite libxdamage -- 2.41.0 ^ permalink raw reply related [flat|nested] 29+ messages in thread
* [bug#64748] [PATCH v7 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942. 2023-08-28 15:00 ` [bug#64748] [PATCH v7 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via @ 2023-08-29 18:13 ` Distopico 2023-09-01 9:01 ` bug#64748: " Hilton Chain via Guix-patches via 0 siblings, 1 reply; 29+ messages in thread From: Distopico @ 2023-08-29 18:13 UTC (permalink / raw) To: Hilton Chain; +Cc: 64748 [-- Attachment #1: Type: text/plain, Size: 9234 bytes --] Build and the call works fine from my side On 2023-08-28, Hilton Chain <hako@ultrarare.space> wrote: > From: Distopico <distopico@riseup.net> > > * gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch: New > file. > * gnu/local.mk (dist_patch_DATA): Register it. > * gnu/packages/telegram.scm (webrtc-for-telegram-desktop): Update to > 0-389.0532942, which fixes `telegram-desktop` calls that are not working with > OpenSSL 3. > [source]<patches>: Add the patch. > <snippet>: Unbundle libsrtp. > [inputs]: Add libsrtp. > > Signed-off-by: Hilton Chain <hako@ultrarare.space> > --- > gnu/local.mk | 1 + > ...or-telegram-desktop-unbundle-libsrtp.patch | 137 ++++++++++++++++++ > gnu/packages/telegram.scm | 16 +- > 3 files changed, 149 insertions(+), 5 deletions(-) > create mode 100644 gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch > > diff --git a/gnu/local.mk b/gnu/local.mk > index 48187d7e64..5ab021eaed 100644 > --- a/gnu/local.mk > +++ b/gnu/local.mk > @@ -2093,6 +2093,7 @@ dist_patch_DATA = \ > %D%/packages/patches/wdl-link-libs-and-fix-jnetlib.patch \ > %D%/packages/patches/webkitgtk-adjust-bubblewrap-paths.patch \ > %D%/packages/patches/webrtc-audio-processing-big-endian.patch \ > + %D%/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch \ > %D%/packages/patches/websocketpp-fix-for-cmake-3.15.patch \ > %D%/packages/patches/wmctrl-64-fix.patch \ > %D%/packages/patches/wmfire-update-for-new-gdk-versions.patch \ > diff --git a/gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch b/gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch > new file mode 100644 > index 0000000000..486f0b1aa5 > --- /dev/null > +++ b/gnu/packages/patches/webrtc-for-telegram-desktop-unbundle-libsrtp.patch > @@ -0,0 +1,137 @@ > +From 62672f3756ecf218252098211d78c13369ab6d28 Mon Sep 17 00:00:00 2001 > +From: Nicholas Guriev <nicholas@guriev.su> > +Date: Thu, 4 May 2023 16:21:09 +0300 > +Subject: [PATCH] Unbundle libSRTP > + > +Avoid private symbols and link against system-wide libSRTP. The excluded code > +in SrtpSession looks unreachable from the call integration in Telegram Desktop. > +--- > + CMakeLists.txt | 3 +++ > + cmake/libsrtp.cmake | 13 +++++++++++++ > + src/pc/external_hmac.cc | 1 - > + src/pc/external_hmac.h | 9 ++++++--- > + src/pc/srtp_session.cc | 16 ++++++++++++++-- > + 5 files changed, 36 insertions(+), 6 deletions(-) > + > +diff --git a/CMakeLists.txt b/CMakeLists.txt > +index af7d24c21..66bec8fdf 100644 > +--- a/CMakeLists.txt > ++++ b/CMakeLists.txt > +@@ -2647,6 +2647,9 @@ if (TG_OWT_USE_PROTOBUF) > + list(APPEND export_targets proto) > + endif() > + > ++if (LIBSRTP_FOUND) > ++ target_compile_definitions(tg_owt PRIVATE HAVE_LIBSRTP) > ++endif() > + if (NOT absl_FOUND) > + list(APPEND export_targets libabsl) > + endif() > +diff --git a/cmake/libsrtp.cmake b/cmake/libsrtp.cmake > +index 5124312d2..01f051606 100644 > +--- a/cmake/libsrtp.cmake > ++++ b/cmake/libsrtp.cmake > +@@ -1,3 +1,16 @@ > ++find_package(PkgConfig REQUIRED) > ++pkg_check_modules(LIBSRTP libsrtp2) > ++ > ++if (LIBSRTP_FOUND) > ++ add_library(libsrtp INTERFACE EXCLUDE_FROM_ALL) > ++ add_library(tg_owt::libsrtp ALIAS libsrtp) > ++ > ++ target_include_directories(libsrtp INTERFACE ${LIBSRTP_INCLUDE_DIRS} ${LIBSRTP_CFLAGS_OTHER}) > ++ target_link_libraries(libsrtp INTERFACE ${LIBSRTP_LINK_LIBRARIES} ${LIBSRTP_LDFLAGS_OTHER}) > ++ > ++ return() > ++endif() > ++ > + add_library(libsrtp OBJECT EXCLUDE_FROM_ALL) > + init_target(libsrtp) > + add_library(tg_owt::libsrtp ALIAS libsrtp) > +diff --git a/src/pc/external_hmac.cc b/src/pc/external_hmac.cc > +index 27b5d0e5a..222f5d9ae 100644 > +--- a/src/pc/external_hmac.cc > ++++ b/src/pc/external_hmac.cc > +@@ -15,7 +15,6 @@ > + > + #include "rtc_base/logging.h" > + #include "rtc_base/zero_memory.h" > +-#include "third_party/libsrtp/include/srtp.h" > + > + // Begin test case 0 */ > + static const uint8_t kExternalHmacTestCase0Key[20] = { > +diff --git a/src/pc/external_hmac.h b/src/pc/external_hmac.h > +index c5071fc19..8fdc2f1a7 100644 > +--- a/src/pc/external_hmac.h > ++++ b/src/pc/external_hmac.h > +@@ -30,9 +30,12 @@ > + > + #include <stdint.h> > + > +-#include "third_party/libsrtp/crypto/include/crypto_types.h" > +-#include "third_party/libsrtp/include/srtp.h" > +-#include "third_party/libsrtp/include/srtp_priv.h" > ++#ifdef HAVE_LIBSRTP > ++# include <srtp2/auth.h> > ++# include <srtp2/srtp.h> > ++#else > ++# include "srtp_priv.h" > ++#endif > + > + #define EXTERNAL_HMAC_SHA1 SRTP_HMAC_SHA1 + 1 > + #define HMAC_KEY_LENGTH 20 > +diff --git a/src/pc/srtp_session.cc b/src/pc/srtp_session.cc > +index 7d1aaf2d6..7b5a789b0 100644 > +--- a/src/pc/srtp_session.cc > ++++ b/src/pc/srtp_session.cc > +@@ -30,8 +30,12 @@ > + #include "rtc_base/thread_annotations.h" > + #include "rtc_base/time_utils.h" > + #include "system_wrappers/include/metrics.h" > +-#include "third_party/libsrtp/include/srtp.h" > +-#include "third_party/libsrtp/include/srtp_priv.h" > ++ > ++#ifdef HAVE_LIBSRTP > ++# include <srtp2/srtp.h> > ++#else > ++# include "srtp_priv.h" > ++#endif > + > + namespace cricket { > + > +@@ -290,6 +294,9 @@ bool SrtpSession::UnprotectRtcp(void* p, int in_len, int* out_len) { > + bool SrtpSession::GetRtpAuthParams(uint8_t** key, int* key_len, int* tag_len) { > + RTC_DCHECK(thread_checker_.IsCurrent()); > + RTC_DCHECK(IsExternalAuthActive()); > ++#ifdef HAVE_LIBSRTP > ++ return false; > ++#else > + if (!IsExternalAuthActive()) { > + return false; > + } > +@@ -313,6 +320,7 @@ bool SrtpSession::GetRtpAuthParams(uint8_t** key, int* key_len, int* tag_len) { > + *key_len = external_hmac->key_length; > + *tag_len = rtp_auth_tag_len_; > + return true; > ++#endif > + } > + > + int SrtpSession::GetSrtpOverhead() const { > +@@ -336,6 +344,9 @@ bool SrtpSession::GetSendStreamPacketIndex(void* p, > + int in_len, > + int64_t* index) { > + RTC_DCHECK(thread_checker_.IsCurrent()); > ++#ifdef HAVE_LIBSRTP > ++ return false; > ++#else > + srtp_hdr_t* hdr = reinterpret_cast<srtp_hdr_t*>(p); > + srtp_stream_ctx_t* stream = srtp_get_stream(session_, hdr->ssrc); > + if (!stream) { > +@@ -346,6 +357,7 @@ bool SrtpSession::GetSendStreamPacketIndex(void* p, > + *index = static_cast<int64_t>(rtc::NetworkToHost64( > + srtp_rdbx_get_packet_index(&stream->rtp_rdbx) << 16)); > + return true; > ++#endif > + } > + > + bool SrtpSession::DoSetKey(int type, > diff --git a/gnu/packages/telegram.scm b/gnu/packages/telegram.scm > index 88daea7d7d..4814c22b6c 100644 > --- a/gnu/packages/telegram.scm > +++ b/gnu/packages/telegram.scm > @@ -1,8 +1,9 @@ > ;;; GNU Guix --- Functional package management for GNU > ;;; Copyright © 2021 Raghav Gururajan <rg@raghavgururajan.name> > -;;; Copyright © 2022 Hilton Chain <hako@ultrarare.space> > +;;; Copyright © 2022, 2023 Hilton Chain <hako@ultrarare.space> > ;;; Copyright © 2023 Saku Laesvuori <saku@laesvuori.fi> > ;;; Copyright © 2023 Lu Hui <luhux76@gmail.com> > +;;; Copyright © 2023 Camilo Q.S. (Distopico) <distopico@riseup.net> > ;;; > ;;; This file is part of GNU Guix. > ;;; > @@ -266,8 +267,8 @@ (define tgcalls-for-telegram-desktop > "193m2gkvipijqbfd6a8mhg9nd63wlnshzgspk3pip57vk21l709z")))) > > (define-public webrtc-for-telegram-desktop > - (let ((commit "5098730b9eb6173f0b52068fe2555b7c1015123a") > - (revision "328")) > + (let ((commit "0532942ac6176a66ef184fb728a4cbb02958fc0b") > + (revision "389")) > (hidden-package > (package > (name "webrtc-for-telegram-desktop") > @@ -283,14 +284,18 @@ (define-public webrtc-for-telegram-desktop > (file-name > (git-file-name name version)) > (sha256 > - (base32 "1lk54zlrff59rj5k9dylsgz4sdds4728psrk8m3v9qn5y8d6z8qy")) > + (base32 "0fary99yl1ddk5zjpfy0pyb5brd268j41plcnvv9qjyf0wj9hf2k")) > + (patches > + (search-patches > + ;; https://github.com/desktop-app/tg_owt/pull/123 > + "webrtc-for-telegram-desktop-unbundle-libsrtp.patch")) > (modules '((guix build utils) > (ice-9 ftw) > (srfi srfi-1))) > (snippet > #~(begin > (let ((keep > - '("libsrtp" "rnnoise" > + '("rnnoise" > ;; Not available in Guix. > "pffft"))) > (with-directory-excursion "src/third_party" > @@ -325,6 +330,7 @@ (define-public webrtc-for-telegram-desktop > libdrm > libglvnd > libjpeg-turbo > + libsrtp > libvpx > libxcomposite > libxdamage [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 515 bytes --] ^ permalink raw reply [flat|nested] 29+ messages in thread
* bug#64748: [PATCH v7 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942. 2023-08-29 18:13 ` Distopico @ 2023-09-01 9:01 ` Hilton Chain via Guix-patches via 0 siblings, 0 replies; 29+ messages in thread From: Hilton Chain via Guix-patches via @ 2023-09-01 9:01 UTC (permalink / raw) To: Distopico; +Cc: Hilton Chain, 64748-done On Wed, 30 Aug 2023 02:13:52 +0800, Distopico wrote: > > [1 <text/plain; utf-8 (quoted-printable)>] > Build and the call works fine from my side > Thanks! Pushed as 33018f8c2506a1ed0b805d595f3309b90153ac23. ^ permalink raw reply [flat|nested] 29+ messages in thread
end of thread, other threads:[~2023-09-01 9:05 UTC | newest] Thread overview: 29+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-07-20 17:18 [bug#64748] [PATCH] gnu: webrtc-for-telegram-desktop: Update to a45d8b8 Distopico 2023-07-20 19:33 ` Saku Laesvuori via Guix-patches via 2023-07-20 20:21 ` Distopico 2023-07-20 21:06 ` Saku Laesvuori via Guix-patches via 2023-07-20 23:09 ` Distopico 2023-07-21 7:22 ` Saku Laesvuori via Guix-patches via 2023-07-22 6:36 ` Raghav Gururajan via Guix-patches via 2023-07-22 8:25 ` Saku Laesvuori via Guix-patches via 2023-07-24 17:07 ` Distopico 2023-07-24 20:17 ` Raghav Gururajan via Guix-patches via 2023-07-26 7:24 ` Saku Laesvuori via Guix-patches via 2023-07-26 18:54 ` Distopico 2023-07-20 23:26 ` [bug#64748] [PATCH v2] " Distopico 2023-07-26 18:54 ` [bug#64748] [PATCH v3] " Distopico 2023-07-26 19:13 ` Saku Laesvuori via Guix-patches via 2023-07-26 19:22 ` Distopico 2023-07-26 19:21 ` [bug#64748] [PATCH v4] " Distopico 2023-07-26 20:09 ` Saku Laesvuori via Guix-patches via 2023-08-05 2:59 ` [bug#64748] [PATCH] " Distopico 2023-08-07 9:38 ` Hilton Chain via Guix-patches via 2023-08-07 12:26 ` Hilton Chain via Guix-patches via 2023-08-28 13:30 ` [bug#64748] [PATCH v6 0/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via 2023-08-28 13:31 ` [bug#64748] [PATCH v6 1/2] gnu: libyuv-for-telegram-desktop: Update to 0-2439.77c2121 Hilton Chain via Guix-patches via 2023-08-28 13:31 ` [bug#64748] [PATCH v6 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via 2023-08-28 14:56 ` [bug#64748] [PATCH v7 0/2] " Hilton Chain via Guix-patches via 2023-08-28 15:00 ` [bug#64748] [PATCH v7 1/2] gnu: libyuv-for-telegram-desktop: Update to 0-2439.77c2121 Hilton Chain via Guix-patches via 2023-08-28 15:00 ` [bug#64748] [PATCH v7 2/2] gnu: webrtc-for-telegram-desktop: Update to 0-389.0532942 Hilton Chain via Guix-patches via 2023-08-29 18:13 ` Distopico 2023-09-01 9:01 ` bug#64748: " Hilton Chain via Guix-patches via
Code repositories for project(s) associated with this external index https://git.savannah.gnu.org/cgit/guix.git This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.