From: Maxim Cournoyer <maxim.cournoyer@gmail.com>
To: Raghav Gururajan <rg@raghavgururajan.name>
Cc: 47274@debbugs.gnu.org
Subject: [bug#47274] Linphone Packages
Date: Fri, 26 Mar 2021 22:13:01 -0400 [thread overview]
Message-ID: <874kgxs5si.fsf_-_@gmail.com> (raw)
In-Reply-To: <cf75f6a8-fd76-cb05-0a46-903de3e882f6@raghavgururajan.name> (Raghav Gururajan's message of "Mon, 22 Mar 2021 00:44:06 -0400")
Hi,
> From c9e23524c79159f78e4c4738cad9544cfe3ac1ce Mon Sep 17 00:00:00 2001
> From: Raghav Gururajan <rg@raghavgururajan.name>
> Date: Tue, 16 Mar 2021 15:51:11 -0400
> Subject: [PATCH 30/53] gnu: mediastreamer2: Update to 4.4.34.
>
> * gnu/packages/linphone.scm (mediastreamer2) [source]: Switch to git repository.
> [version]: Update to 4.4.34.
> [patches]: Remove field.
> [outputs]: Rename "tester" to "test".
> [arguments]<#:configure-flags>[-DENABLE_STRICT]: Remove flag.
> [-DENABLE_BV16]: Remove flag.
> [-DCMAKE_C_FLAGS]: Remove flag.
> [-DCMAKE_CXX_FLAGS]: Remove flag.
> <#:phases>['separate-outputs]: Modify.
> [native-inputs]: Replace python with python-wrapper.
> * gnu/packages/patches/mediastreamer2-srtp2.patch: Remove file.
> * gnu/local.mk (dist_patch_DATA): Remove it.
> ---
> gnu/local.mk | 1 -
> gnu/packages/linphone.scm | 62 ++++---
> .../patches/mediastreamer2-srtp2.patch | 155 ------------------
> 3 files changed, 28 insertions(+), 190 deletions(-)
> delete mode 100644 gnu/packages/patches/mediastreamer2-srtp2.patch
>
> diff --git a/gnu/local.mk b/gnu/local.mk
> index 3d4147a879..5a69518923 100644
> --- a/gnu/local.mk
> +++ b/gnu/local.mk
> @@ -1391,7 +1391,6 @@ dist_patch_DATA = \
> %D%/packages/patches/mcrypt-CVE-2012-4527.patch \
> %D%/packages/patches/libmemcached-build-with-gcc7.patch \
> %D%/packages/patches/libmhash-hmac-fix-uaf.patch \
> - %D%/packages/patches/mediastreamer2-srtp2.patch \
> %D%/packages/patches/mesa-skip-tests.patch \
> %D%/packages/patches/mescc-tools-boot.patch \
> %D%/packages/patches/meson-for-build-rpath.patch \
> diff --git a/gnu/packages/linphone.scm b/gnu/packages/linphone.scm
> index e573e71932..025fd90c46 100644
> --- a/gnu/packages/linphone.scm
> +++ b/gnu/packages/linphone.scm
> @@ -569,56 +569,50 @@ API. It also comprises a simple HTTP/HTTPS client implementation.")
> (define-public mediastreamer2
> (package
> (name "mediastreamer2")
> - (version "2.16.1")
> + (version "4.4.34")
> (source
> (origin
> - (method url-fetch)
> - (uri
> - (string-append "https://www.linphone.org/releases/sources/"
> - "mediastreamer/mediastreamer-" version ".tar.gz"))
> + (method git-fetch)
> + (uri (git-reference
> + (url "git://git.linphone.org/mediastreamer2")
> + (commit version)))
> + (file-name (git-file-name name version))
> (sha256
> - (base32 "0whpqr69wz0pnzvragkpfblxhd0rds8k06c3mw5a0ag216a1yd9k"))
> - (patches (search-patches "mediastreamer2-srtp2.patch"))))
> - (outputs '("out" "doc" "tester"))
> + (base32 "0989h3d0h7qrx4kjx8gg09j8c5hvvi3h8qi1iq1dqbppwbaxbz8c"))))
> + (outputs '("out" "doc" "test"))
The change of the output name from tester to test seems arbitrary :-).
I'd argue that "tester" makes more sense as this outputs contains a
tester binary or command rather than a test suite.
> (build-system cmake-build-system)
> (arguments
> `(#:tests? #f ; No test target
> #:configure-flags
> - (list "-DENABLE_STATIC=NO" ; Not required
> - "-DENABLE_STRICT=NO" ; Would otherwise treat warnings as err
> - "-DENABLE_BV16=NO" ; Not available
> - "-DCMAKE_C_FLAGS=-DMS2_GIT_VERSION=\\\"unknown\\\""
> - "-DCMAKE_CXX_FLAGS=-DMS2_GIT_VERSION=\\\"unknown\\\"")
> + (list
> + "-DENABLE_STATIC=NO")
> #:phases
> (modify-phases %standard-phases
> (add-after 'install 'separate-outputs
> (lambda* (#:key outputs #:allow-other-keys)
> (let* ((out (assoc-ref outputs "out"))
> + (test (assoc-ref outputs "test"))
> + (test-name (string-append ,name "_tester"))
> (doc (assoc-ref outputs "doc"))
> - (tester (assoc-ref outputs "tester"))
> - (tester-name (string-append ,name "_tester")))
> - ;; Copy the tester executable.
While it may seem obvious, I believe the comments helped the human brain
keep track of the process flow more cheaply; please don't remove them.
> - (mkdir-p (string-append tester "/bin"))
> - (rename-file (string-append out "/bin/" tester-name)
> - (string-append tester "/bin/" tester-name))
> - ;; Copy the tester data files.
> - (copy-recursively (string-append out "/share/" tester-name)
> - (string-append tester "/share/" tester-name))
> - (delete-file-recursively (string-append out "/share/"
> - tester-name))
> - ;; Copy the HTML documentation.
> - (copy-recursively (string-append out "/share/doc/"
> - ,name "-" ,version "/html")
> - (string-append doc "/share/doc/"
> - ,name "-" ,version "/html"))
> - (delete-file-recursively (string-append out "/share/doc/"
> - ,name "-" ,version
> - "/html"))
> - #t))))))
> + (doc-name (string-append ,name "-4.4.0")))
> + (for-each mkdir-p
> + `(,(string-append test "/bin")
> + ,(string-append test "/share")
> + ,(string-append doc "/share/doc")))
> + (rename-file
> + (string-append out "/bin/" test-name)
> + (string-append test "/bin/" test-name))
> + (rename-file
> + (string-append out "/share/" test-name)
> + (string-append test "/share/" test-name))
> + (rename-file
> + (string-append out "/share/doc/" doc-name)
> + (string-append doc "/share/doc/" doc-name)))
> + #t)))))
> (native-inputs
> `(("dot" ,graphviz)
> ("doxygen" ,doxygen)
> - ("python" ,python)))
> + ("python" ,python-wrapper)))
> (inputs
> `(("alsa" ,alsa-lib)
> ("bcg729" ,bcg729)
I've squashed this one with the home page update and license change, as
they are small and are related to the update.
I still there's still stuff depending on this to be reviewed before it
can be pushed, so I'm amassing a stack here... please be patient :-)
Thank you,
Maxim
next prev parent reply other threads:[~2021-03-27 2:14 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-20 4:26 [bug#47274] Linphone Packages Raghav Gururajan via Guix-patches via
2021-03-21 6:13 ` Raghav Gururajan via Guix-patches via
2021-03-22 1:48 ` Raghav Gururajan via Guix-patches via
2021-03-22 4:44 ` Raghav Gururajan via Guix-patches via
2021-03-22 4:59 ` Raghav Gururajan via Guix-patches via
2021-03-29 4:03 ` Maxim Cournoyer
2021-03-29 6:05 ` Raghav Gururajan via Guix-patches via
2021-03-29 18:38 ` Maxim Cournoyer
2021-03-29 18:47 ` Raghav Gururajan via Guix-patches via
2021-03-24 4:12 ` Maxim Cournoyer
2021-03-25 13:07 ` Maxim Cournoyer
2021-03-29 5:16 ` Raghav Gururajan via Guix-patches via
2021-03-27 2:13 ` Maxim Cournoyer [this message]
2021-03-29 5:52 ` Raghav Gururajan via Guix-patches via
2021-03-27 5:05 ` Maxim Cournoyer
2021-03-29 5:53 ` Raghav Gururajan via Guix-patches via
2021-03-27 5:11 ` Maxim Cournoyer
2021-03-28 4:05 ` Raghav Gururajan via Guix-patches via
2021-03-28 4:43 ` Maxim Cournoyer
2021-03-29 6:02 ` Raghav Gururajan via Guix-patches via
2021-03-29 5:53 ` Raghav Gururajan via Guix-patches via
2021-03-28 0:52 ` Maxim Cournoyer
2021-03-29 5:55 ` Raghav Gururajan via Guix-patches via
2021-03-29 5:44 ` bug#47274: " Maxim Cournoyer
2021-03-29 6:13 ` [bug#47274] " Raghav Gururajan via Guix-patches via
2021-03-29 18:39 ` Maxim Cournoyer
2021-03-25 1:55 ` Maxim Cournoyer
2021-03-29 5:13 ` Raghav Gururajan via Guix-patches via
2021-03-28 4:01 ` Maxim Cournoyer
2021-03-29 6:01 ` Raghav Gururajan via Guix-patches via
2021-03-29 5:19 ` Maxim Cournoyer
2021-03-29 6:12 ` Raghav Gururajan via Guix-patches via
2021-03-24 2:25 ` Maxim Cournoyer
2021-03-29 5:07 ` Raghav Gururajan via Guix-patches via
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=874kgxs5si.fsf_-_@gmail.com \
--to=maxim.cournoyer@gmail.com \
--cc=47274@debbugs.gnu.org \
--cc=rg@raghavgururajan.name \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).