[PATCH 00/10] [telephony-team] Update Jami and related packages.

  • Done
  • quality assurance status badge
Details
3 participants
  • Andreas Enge
  • Liliana Marie Prikler
  • Maxim Cournoyer
Owner
unassigned
Submitted by
Maxim Cournoyer
Severity
normal
M
M
Maxim Cournoyer wrote on 18 Aug 2023 19:59
cover.1692381513.git.maxim.cournoyer@gmail.com
Maxim Cournoyer (10):
gnu: opendht: Update to 2.5.5.
gnu: mmtf-cpp: Update to 1.1.0.
gnu: avogadrolibs: Remove msgpack from native-inputs.
gnu: python-pymol: Remove msgpack from native-inputs.
gnu: msgpack: Deprecate with msgpack-c.
gnu: Add sdbus-cpp.
gnu: libnatpmp: Use gexps and simplify.
gnu: libnatpmp: Update home page.
gnu: libnatpmp: Install missing natpmp_declspec.h header.
gnu: jami: Update to 20230619.1.

gnu/local.mk | 2 +
gnu/packages/chemistry.scm | 12 ++--
gnu/packages/glib.scm | 43 +++++++++++
gnu/packages/jami.scm | 29 +++-----
gnu/packages/networking.scm | 41 ++++++-----
.../patches/jami-qml-tests-discovery.patch | 15 ++++
gnu/packages/patches/sdbus-c++-elogind.patch | 63 ++++++++++++++++
gnu/packages/serialization.scm | 71 +++++++++++++++----
gnu/packages/terminals.scm | 6 +-
gnu/packages/vim.scm | 6 +-
10 files changed, 223 insertions(+), 65 deletions(-)
create mode 100644 gnu/packages/patches/jami-qml-tests-discovery.patch
create mode 100644 gnu/packages/patches/sdbus-c++-elogind.patch


base-commit: 97b52b2d48b40ecbc711e30eb6f94af3fbc5bd6d
--
2.41.0
M
M
Maxim Cournoyer wrote on 18 Aug 2023 20:12
[PATCH telephony-team 01/10] gnu: opendht: Update to 2.5.5.
(address . 65371@debbugs.gnu.org)(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
d5d842caf9ce7268ed0d4cd505ef79d76d304864.1692382380.git.maxim.cournoyer@gmail.com
* gnu/packages/networking.scm (opendht): Update to 2.5.5.
[arguments]: Add a new relax-test-timeouts phase.
---

gnu/packages/networking.scm | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)

Toggle diff (41 lines)
diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 04461fec2d..55c938606f 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -3759,7 +3759,7 @@ (define-public restinio
(define-public opendht
(package
(name "opendht")
- (version "2.4.12")
+ (version "2.5.5")
(source (origin
(method git-fetch)
(uri (git-reference
@@ -3768,7 +3768,7 @@ (define-public opendht
(file-name (git-file-name name version))
(sha256
(base32
- "0yji5pziqxfvyfizk3fn9j59bqlfdwfa1a0y9jjfknb2mmlwwb9w"))))
+ "0i2x0h38h52aynm89n6cjxk0z7nngc1w1zf71zaqx5n54fxxawir"))))
(outputs '("out" "python" "tools" "debug"))
(build-system gnu-build-system)
(arguments
@@ -3796,6 +3796,14 @@ (define-public opendht
(substitute* "tests/Makefile.am"
(("\\bdhtrunnertester\\.(h|cpp)\\b")
""))))
+ (add-after 'unupack 'relax-test-timeouts
+ (lambda _
+ ;; At least the 'test_send_json' has been seen to fail
+ ;; non-deterministically, but it seems hard to reproducible that
+ ;; failure.
+ (substitute* "tests/httptester.cpp"
+ (("std::chrono::seconds\\(10)")
+ "std::chrono::seconds(30)"))))
(add-after 'unpack 'fix-python-installation-prefix
;; Specify the installation prefix for the compiled Python module
;; that would otherwise attempt to installs itself to Python's own

base-commit: 97b52b2d48b40ecbc711e30eb6f94af3fbc5bd6d
--
2.41.0
M
M
Maxim Cournoyer wrote on 18 Aug 2023 20:12
[PATCH telephony-team 02/10] gnu: mmtf-cpp: Update to 1.1.0.
(address . 65371@debbugs.gnu.org)(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
4b6e08285ccfd4585fb51a52921e0e65133be244.1692382380.git.maxim.cournoyer@gmail.com
* gnu/packages/chemistry.scm (mmtf-cpp): Update to 1.1.0.
[propagated-inputs]: Add msgpack-3.
---

gnu/packages/chemistry.scm | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)

Toggle diff (32 lines)
diff --git a/gnu/packages/chemistry.scm b/gnu/packages/chemistry.scm
index 05aaa827f8..25b31c7192 100644
--- a/gnu/packages/chemistry.scm
+++ b/gnu/packages/chemistry.scm
@@ -248,7 +248,7 @@ (define-public libmsym
(define-public mmtf-cpp
(package
(name "mmtf-cpp")
- (version "1.0.0")
+ (version "1.1.0")
(source
(origin
(method git-fetch)
@@ -258,11 +258,13 @@ (define-public mmtf-cpp
(file-name (git-file-name name version))
(sha256
(base32
- "17ylramda69plf5w0v5hxbl4ggkdi5s15z55cv0pljl12yvyva8l"))))
+ "0rs2f1ppgqz663c3m22p8wsq6z839bj59zy29chci46ypfhwv6ph"))))
(build-system cmake-build-system)
;; Tests require the soon-to-be-deprecated version 1 of the catch-framework.
- (arguments
- '(#:tests? #f))
+ (arguments '(#:tests? #f))
+ ;; There is no support for modern msgpack versions yet (see:
+ ;; https://github.com/rcsb/mmtf-cpp/issues/44).
+ (propagated-inputs (list msgpack-3)) ;included by mmtf/structure_data.hpp
(home-page "https://mmtf.rcsb.org/")
(synopsis "C++ API for the Macromolecular Transmission Format")
(description "This package is a library for the
--
2.41.0
M
M
Maxim Cournoyer wrote on 18 Aug 2023 20:12
[PATCH telephony-team 03/10] gnu: avogadrolibs: Remove msgpack from native-inputs.
(address . 65371@debbugs.gnu.org)(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
e1630b921fae0dd33e7cbc7bbb844ddfe90b9822.1692382380.git.maxim.cournoyer@gmail.com
It's now propagated by mmtf-cpp.

* gnu/packages/chemistry.scm (avogadrolibs)
[native-inputs]: Remove msgpack.
---

gnu/packages/chemistry.scm | 1 -
1 file changed, 1 deletion(-)

Toggle diff (14 lines)
diff --git a/gnu/packages/chemistry.scm b/gnu/packages/chemistry.scm
index 25b31c7192..4d5d3d0195 100644
--- a/gnu/packages/chemistry.scm
+++ b/gnu/packages/chemistry.scm
@@ -87,7 +87,6 @@ (define-public avogadrolibs
(native-inputs
(list eigen
mmtf-cpp
- msgpack
googletest
pkg-config
pybind11))
--
2.41.0
M
M
Maxim Cournoyer wrote on 18 Aug 2023 20:12
[PATCH telephony-team 04/10] gnu: python-pymol: Remove msgpack from native-inputs.
(address . 65371@debbugs.gnu.org)(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
93e9d4888847e06bd06c98cb94d972496b26238f.1692382380.git.maxim.cournoyer@gmail.com
It's now propagated by mmtf-cpp.

* gnu/packages/chemistry.scm (python-pymol)
[native-inputs]: Remove msgpack.
---

gnu/packages/chemistry.scm | 1 -
1 file changed, 1 deletion(-)

Toggle diff (14 lines)
diff --git a/gnu/packages/chemistry.scm b/gnu/packages/chemistry.scm
index 4d5d3d0195..df96a2f5c1 100644
--- a/gnu/packages/chemistry.scm
+++ b/gnu/packages/chemistry.scm
@@ -575,7 +575,6 @@ (define-public python-pymol
glew
libxml2
mmtf-cpp
- msgpack
python-pyqt
glm
netcdf))
--
2.41.0
M
M
Maxim Cournoyer wrote on 18 Aug 2023 20:12
[PATCH telephony-team 05/10] gnu: msgpack: Deprecate with msgpack-c.
(address . 65371@debbugs.gnu.org)(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
c03c2aa247e94c24385a2765cc5b6c6352d9696b.1692382380.git.maxim.cournoyer@gmail.com
* gnu/packages/serialization.scm (msgpack-c): New variable.
(msgpack-cxx): New variable.
(msgpack): Rename to...
(msgpack-3): ... this, and inherit from msgpack-c.
* gnu/packages/vim.scm (eovim) [inputs]: Replace msgpack with msgpack-3.
* gnu/packages/terminals.scm (tmate): Likewise.
* gnu/packages/networking.scm (opendht)
[propagated-inputs]: Replace msgpack with msgpack-cxx.
---

gnu/packages/networking.scm | 2 +-
gnu/packages/serialization.scm | 71 +++++++++++++++++++++++++++-------
gnu/packages/terminals.scm | 6 +--
gnu/packages/vim.scm | 6 +--
4 files changed, 62 insertions(+), 23 deletions(-)

Toggle diff (150 lines)
diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 55c938606f..a7cb4f876e 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -3847,7 +3847,7 @@ (define-public opendht
fmt
readline))
(propagated-inputs
- (list msgpack ;included in several installed headers
+ (list msgpack-cxx ;included in several installed headers
restinio ;included in opendht/http.h
;; The following are listed in the 'Requires.private' field of
;; opendht.pc:
diff --git a/gnu/packages/serialization.scm b/gnu/packages/serialization.scm
index 9c114aaf39..9f2aab250e 100644
--- a/gnu/packages/serialization.scm
+++ b/gnu/packages/serialization.scm
@@ -15,6 +15,7 @@
;;; Copyright © 2023 Alexey Abramov <levenson@mmer.org>
;;; Copyright © 2023 Sharlatan Hellseher <sharlatanus@gmail.com>
;;; Copyright © 2023 Vinicius Monego <monego@posteo.net>
+;;; Copyright © 2023 Maxim Cournoyer <maxim.cournoyer@gmail.com>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -222,8 +223,60 @@ (define-public cereal-1.3.0
(list doxygen gcc-10
(package-source cereal)))))
+(define-public msgpack-c
+ (package
+ (name "msgpack-c")
+ (version "6.0.0")
+ (source
+ (origin
+ (method url-fetch)
+ (uri
+ (string-append
+ "https://github.com/msgpack/msgpack-c/releases/download/"
+ "c-" version "/" name "-" version ".tar.gz"))
+ (sha256
+ (base32 "1fmf08babfsjq5qkgw034wk2nw6mayxp1qlkm7h55p2jqvigam1n"))
+ (snippet
+ '(let ((p (open-file "msgpack-c.pc.in" "a")))
+ (display "Requires: zlib\n" p)
+ (close-output-port p)))))
+ (build-system cmake-build-system)
+ (arguments (list #:configure-flags #~(list "-DMSGPACK_BUILD_TESTS=ON")))
+ (native-inputs (list googletest))
+ (propagated-inputs (list zlib)) ;zbuffer.h includes zlib.h
+ (home-page "https://www.msgpack.org")
+ (synopsis "Binary serialization library")
+ (description "Msgpack is a library for C that implements binary
+serialization.")
+ (license license:boost1.0)))
+
+(define-public msgpack-cxx
+ (package
+ (inherit msgpack-c)
+ (name "msgpack-cxx")
+ (version "6.1.0")
+ (source
+ (origin
+ (method url-fetch)
+ (uri
+ (string-append
+ "https://github.com/msgpack/msgpack-c/releases/download/"
+ "cpp-" version "/" name "-" version ".tar.gz"))
+ (sha256
+ (base32 "1rrrf3nskcv994z3pbq6a5z2021piz118rccmm1y7zlf7klygv93"))))
+ (build-system cmake-build-system)
+ (propagated-inputs (list boost zlib)) ;included in headers
+ (description "Msgpack is a library for C++ that implements binary
+serialization.")))
+
+;;; The msgpack package was split into msgpack-c and msgpack-cxx starting from
+;;; version 4.0.0.
(define-public msgpack
+ (deprecated-package "msgpack" msgpack-c))
+
+(define-public msgpack-3
(package
+ (inherit msgpack-c)
(name "msgpack")
(version "3.3.0")
(source
@@ -237,23 +290,13 @@ (define-public msgpack
'(let ((p (open-file "msgpack.pc.in" "a")))
(display
(string-append "Requires: " "zlib" "\n") p)
- (close-output-port p)
- #t))
+ (close-output-port p)))
(sha256
(base32 "0yzhq50ijvwrfkr97knhvn54lj3f4hr3zy39yq8wpf6xll94s4bf"))))
- (build-system cmake-build-system)
- (native-inputs
- (list googletest-1.8 pkg-config))
- (propagated-inputs
- (list zlib)) ;; Msgpack installs two headers (zbuffer.h,
- ;; zbuffer.hpp) which #include <zlib.h>. However, 'guix gc --references'
- ;; does not detect a store reference to zlib since these headers are not
- ;; compiled.
- (home-page "https://www.msgpack.org")
- (synopsis "Binary serialization library")
+ (native-inputs (list googletest-1.8))
(description "Msgpack is a library for C/C++ that implements binary
-serialization.")
- (license license:boost1.0)))
+serialization. This is the legacy version that predates the split into C and
+C++ specific packages.")))
(define-public libmpack
(package
diff --git a/gnu/packages/terminals.scm b/gnu/packages/terminals.scm
index 5fc72d7937..44641360ba 100644
--- a/gnu/packages/terminals.scm
+++ b/gnu/packages/terminals.scm
@@ -1172,10 +1172,8 @@ (define-public tmate
(base32
"0x5c31yq7ansmiy20a0qf59wagba9v3pq97mlkxrqxn4n1gcc6vi"))))
(build-system gnu-build-system)
- (inputs
- (list libevent libssh msgpack ncurses))
- (native-inputs
- (list autoconf automake pkg-config))
+ (inputs (list libevent libssh msgpack-3 ncurses))
+ (native-inputs (list autoconf automake pkg-config))
(home-page "https://tmate.io/")
(synopsis "Terminal sharing application")
(description "tmate is a terminal sharing application that allows you to
diff --git a/gnu/packages/vim.scm b/gnu/packages/vim.scm
index d58b022123..16f73e355f 100644
--- a/gnu/packages/vim.scm
+++ b/gnu/packages/vim.scm
@@ -814,10 +814,8 @@ (define-public eovim
(string-append start nvim))))))
(add-before 'build 'set-home
(lambda _ (setenv "HOME" "/tmp"))))))
- (native-inputs
- (list pkg-config))
- (inputs
- (list efl msgpack neovim))
+ (native-inputs (list pkg-config))
+ (inputs (list efl msgpack-3 neovim))
(home-page "https://github.com/jeanguyomarch/eovim/")
(synopsis "EFL GUI for Neovim")
(description "Graphical Neovim interface based on the @acronym{EFL, Enlightenment
--
2.41.0
M
M
Maxim Cournoyer wrote on 18 Aug 2023 20:12
[PATCH telephony-team 06/10] gnu: Add sdbus-cpp.
(address . 65371@debbugs.gnu.org)(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
7cd1d20fbbb1cf9c620748e22bdfef7feb83ff72.1692382380.git.maxim.cournoyer@gmail.com
* gnu/packages/glib.scm (sdbus-cpp): New variable.
* gnu/packages/patches/sdbus-c++-elogind.patch: New file.
* gnu/local.mk (dist_patch_DATA): Register it.
---

gnu/local.mk | 1 +
gnu/packages/glib.scm | 43 +++++++++++++
gnu/packages/patches/sdbus-c++-elogind.patch | 63 ++++++++++++++++++++
3 files changed, 107 insertions(+)
create mode 100644 gnu/packages/patches/sdbus-c++-elogind.patch

Toggle diff (137 lines)
diff --git a/gnu/local.mk b/gnu/local.mk
index abe8964f4d..de669c1177 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -1938,6 +1938,7 @@ dist_patch_DATA = \
%D%/packages/patches/scons-test-environment.patch \
%D%/packages/patches/screen-hurd-path-max.patch \
%D%/packages/patches/scsh-nonstring-search-path.patch \
+ %D%/packages/patches/sdbus-c++-elogind.patch \
%D%/packages/patches/sdl-libx11-1.6.patch \
%D%/packages/patches/seed-webkit.patch \
%D%/packages/patches/sendgmail-accept-ignored-gsuite-flag.patch \
diff --git a/gnu/packages/glib.scm b/gnu/packages/glib.scm
index a0b85b15d1..588133e93f 100644
--- a/gnu/packages/glib.scm
+++ b/gnu/packages/glib.scm
@@ -1290,6 +1290,49 @@ (define-public dbus-cxx
(home-page "https://dbus-cxx.github.io/")
(license license:gpl3)))
+(define-public sdbus-c++
+ ;; Use the latest commit, which includes unreleased fixes to the pkg-config
+ ;; file.
+ (let ((commit "3e84b254e9603935cb5fc180c4d2214d7024ccbb")
+ (revision "0"))
+ (package
+ (name "sdbus-c++")
+ (version (git-version "1.2.0" revision commit))
+ (source (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/Kistler-Group/sdbus-cpp")
+ (commit commit)))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32
+ "0xrcp49almi3kwzmwcwv8sayqjxx8m55s5grxarw2hl9jmghwlc3"))
+ (patches (search-patches "sdbus-c++-elogind.patch"))))
+ (build-system cmake-build-system)
+ (arguments
+ (list
+ ;; Avoid the integration test, which requires a system bus.
+ #:test-target "sdbus-c++-unit-tests"
+ #:configure-flags #~(list "-DBUILD_CODE_GEN=ON"
+ "-DBUILD_TESTS=ON"
+ ;; Do not install tests.
+ "-DTESTS_INSTALL_PATH=/tmp"
+ "-DCMAKE_VERBOSE_MAKEFILE=ON")
+ #:phases #~(modify-phases %standard-phases
+ (add-after 'unpack 'do-not-install-tests
+ (lambda _
+ (substitute* "tests/CMakeLists.txt"
+ (("/etc/dbus-1/system.d") "/tmp")))))))
+ (native-inputs (list googletest pkg-config))
+ (inputs (list expat))
+ (propagated-inputs (list elogind)) ;required by sdbus-c++.pc
+ (home-page "https://github.com/Kistler-Group/sdbus-cpp")
+ (synopsis "High-level C++ D-Bus library")
+ (description "@code{sdbus-c++} is a high-level C++ D-Bus library designed
+to provide easy-to-use yet powerful API in modern C++. It adds another layer
+of abstraction on top of @code{sd-bus}, the C D-Bus implementation by systemd.")
+ (license license:lgpl2.1+))))
+
(define-public appstream-glib
(package
(name "appstream-glib")
diff --git a/gnu/packages/patches/sdbus-c++-elogind.patch b/gnu/packages/patches/sdbus-c++-elogind.patch
new file mode 100644
index 0000000000..d01838edbc
--- /dev/null
+++ b/gnu/packages/patches/sdbus-c++-elogind.patch
@@ -0,0 +1,63 @@
+Submitted upstream: https://github.com/Kistler-Group/sdbus-cpp/pull/352
+
+From 8423c44b6c24ebd59db06ad33704265aa81a1c7a Mon Sep 17 00:00:00 2001
+From: Sven Eden <sven.eden@prydeworx.com>
+Date: Thu, 17 Aug 2023 23:16:19 -0400
+Subject: [PATCH] build: Add support for elogind.
+
+* CMakeLists.txt: Fallback to elogind when libsystemd could not be
+found. Set LIBSYSTEMD variable.
+* pkgconfig/sdbus-c++.pc.in (Description): Parameterize with above
+LIBSYSTEMD variable.
+
+Signed-off-by: Maxim Cournoyer <maxim.cournoyer@gmail.com>
+---
+ CMakeLists.txt | 11 +++++++++++
+ pkgconfig/sdbus-c++.pc.in | 2 +-
+ 2 files changed, 12 insertions(+), 1 deletion(-)
+
+diff --git a/CMakeLists.txt b/CMakeLists.txt
+index 94f71c7..40c15f8 100644
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -12,11 +12,22 @@ include(GNUInstallDirs) # Installation directories for `install` command and pkg
+ # PERFORMING CHECKS & PREPARING THE DEPENDENCIES
+ #-------------------------------
+
++set(LIBSYSTEMD "libsystemd")
++
+ option(BUILD_LIBSYSTEMD "Build libsystemd static library and incorporate it into libsdbus-c++" OFF)
+
+ if(NOT BUILD_LIBSYSTEMD)
+ find_package(PkgConfig REQUIRED)
+ pkg_check_modules(Systemd IMPORTED_TARGET GLOBAL libsystemd>=236)
++ if(NOT TARGET PkgConfig::Systemd)
++ message(WARNING "libsystemd not found, checking for libelogind instead")
++ pkg_check_modules(Systemd IMPORTED_TARGET GLOBAL libelogind>=236)
++ if(TARGET PkgConfig::Systemd)
++ set(LIBSYSTEMD "libelogind")
++ string(REPLACE "." ";" VERSION_LIST ${Systemd_VERSION})
++ list(GET VERSION_LIST 0 Systemd_VERSION)
++ endif()
++ endif()
+ if(NOT TARGET PkgConfig::Systemd)
+ message(FATAL_ERROR "libsystemd of version at least 236 is required, but was not found "
+ "(if you have systemd in your OS, you may want to install package containing pkgconfig "
+diff --git a/pkgconfig/sdbus-c++.pc.in b/pkgconfig/sdbus-c++.pc.in
+index 6ad010a..0703476 100644
+--- a/pkgconfig/sdbus-c++.pc.in
++++ b/pkgconfig/sdbus-c++.pc.in
+@@ -5,7 +5,7 @@ includedir=@CMAKE_INSTALL_FULL_INCLUDEDIR@
+
+ Name: @PROJECT_NAME@
+ Description: C++ library on top of sd-bus, a systemd D-Bus library
+-Requires@PKGCONFIG_REQS@: libsystemd
++Requires@PKGCONFIG_REQS@: @LIBSYSTEMD@
+ Version: @SDBUSCPP_VERSION@
+ Libs: -L${libdir} -l@PROJECT_NAME@
+ Cflags: -I${includedir}
+
+base-commit: 3e84b254e9603935cb5fc180c4d2214d7024ccbb
+--
+2.41.0
+
--
2.41.0
M
M
Maxim Cournoyer wrote on 18 Aug 2023 20:12
[PATCH telephony-team 07/10] gnu: libnatpmp: Use gexps and simplify.
(address . 65371@debbugs.gnu.org)(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
d7287793a2ece74bc5a42ee615c2d1fd5e88d1b5.1692382380.git.maxim.cournoyer@gmail.com
* gnu/packages/networking.scm (libnatpmp) [arguments]: Use gexps. Use gexp
variables and cc-for-target in make flags.
---

gnu/packages/networking.scm | 20 +++++++-------------
1 file changed, 7 insertions(+), 13 deletions(-)

Toggle diff (33 lines)
diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index a7cb4f876e..1b2e26c3f6 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -1435,19 +1435,13 @@ (define-public libnatpmp
"0w7wvf4yi8qv659dg9d3ndqvh3bqhgm21gd135spwhq6hhnfv106"))))
(build-system gnu-build-system)
(arguments
- `(#:phases
- (modify-phases %standard-phases
- (delete 'configure)
- (delete 'check)) ; no tests
- #:make-flags
- (let* ((target ,(%current-target-system))
- (gcc (if target
- (string-append target "-gcc")
- "gcc")))
- (list
- (string-append "CC=" gcc)
- (string-append "INSTALLPREFIX=" (assoc-ref %outputs "out"))
- (string-append "LDFLAGS=-Wl,-rpath=" %output "/lib")))))
+ (list #:phases #~(modify-phases %standard-phases
+ (delete 'configure)
+ (delete 'check)) ; no tests
+ #:make-flags
+ #~(list (string-append "CC=" #$(cc-for-target))
+ (string-append "INSTALLPREFIX=" #$output)
+ (string-append "LDFLAGS=-Wl,-rpath=" #$output "/lib"))))
(home-page "http://miniupnp.free.fr/libnatpmp.html")
(synopsis "C library implementing NAT-PMP")
(description
--
2.41.0
M
M
Maxim Cournoyer wrote on 18 Aug 2023 20:12
[PATCH telephony-team 08/10] gnu: libnatpmp: Update home page.
(address . 65371@debbugs.gnu.org)(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
77f26927ad3c38e745cb7d765ea1105c7ba7752e.1692382380.git.maxim.cournoyer@gmail.com
* gnu/packages/networking.scm (libnatpmp) [home-page]: Update.
---

gnu/packages/networking.scm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Toggle diff (15 lines)
diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index 1b2e26c3f6..b00306971c 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -1442,7 +1442,7 @@ (define-public libnatpmp
#~(list (string-append "CC=" #$(cc-for-target))
(string-append "INSTALLPREFIX=" #$output)
(string-append "LDFLAGS=-Wl,-rpath=" #$output "/lib"))))
- (home-page "http://miniupnp.free.fr/libnatpmp.html")
+ (home-page "https://miniupnp.tuxfamily.org/libnatpmp.html")
(synopsis "C library implementing NAT-PMP")
(description
"@code{libnatpmp} is a portable and asynchronous implementation of
--
2.41.0
M
M
Maxim Cournoyer wrote on 18 Aug 2023 20:13
[PATCH telephony-team 09/10] gnu: libnatpmp: Install missing natpmp_declspec.h header.
(address . 65371@debbugs.gnu.org)(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
d4de2df97dcb49eb9c441bbab0069e24f830013e.1692382380.git.maxim.cournoyer@gmail.com
* gnu/packages/networking.scm (libnatpmp) [arguments]: Specify the
"HEADERS=natpmp.h natpmp_declspec.h" make flag.
---

gnu/packages/networking.scm | 5 +++++
1 file changed, 5 insertions(+)

Toggle diff (18 lines)
diff --git a/gnu/packages/networking.scm b/gnu/packages/networking.scm
index b00306971c..faeaefba0c 100644
--- a/gnu/packages/networking.scm
+++ b/gnu/packages/networking.scm
@@ -1440,6 +1440,11 @@ (define-public libnatpmp
(delete 'check)) ; no tests
#:make-flags
#~(list (string-append "CC=" #$(cc-for-target))
+ ;; Explicitly specify the headers, otherwise only natpmp.h
+ ;; would be installed, referring to the missing
+ ;; natpmp_declspec.h (see:
+ ;; https://github.com/miniupnp/libnatpmp/issues/41).
+ (string-append "HEADERS=natpmp.h natpmp_declspec.h")
(string-append "INSTALLPREFIX=" #$output)
(string-append "LDFLAGS=-Wl,-rpath=" #$output "/lib"))))
(home-page "https://miniupnp.tuxfamily.org/libnatpmp.html")
--
2.41.0
M
M
Maxim Cournoyer wrote on 18 Aug 2023 20:13
[PATCH telephony-team 10/10] gnu: jami: Update to 20230619.1.
(address . 65371@debbugs.gnu.org)(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
c7dab5b98f03f3ca0eb8642ea2750cdda10edc95.1692382380.git.maxim.cournoyer@gmail.com
(%jami-version): Update to 20230619.1.
* gnu/packages/jami.scm (%jami-sources): Update hash. Apply
jami-qml-tests-discovery.patch patch.
(libjami) [inputs]: Replace dbus-c++ with sdbus-c++.
(jami) [arguments]: Delete obsolete change-directory/maybe phase.
Adjust check phase.
* gnu/packages/patches/jami-qml-tests-discovery.patch: New file.
* gnu/local.mk (dist_patch_DATA): Register it.

---

gnu/local.mk | 1 +
gnu/packages/jami.scm | 34 ++++++-------------
.../patches/jami-qml-tests-discovery.patch | 15 ++++++++
3 files changed, 27 insertions(+), 23 deletions(-)
create mode 100644 gnu/packages/patches/jami-qml-tests-discovery.patch

Toggle diff (118 lines)
diff --git a/gnu/local.mk b/gnu/local.mk
index de669c1177..ac6ccc2ab2 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -1412,6 +1412,7 @@ dist_patch_DATA = \
%D%/packages/patches/isync-openssl3-fix.patch \
%D%/packages/patches/itk-snap-alt-glibc-compat.patch \
%D%/packages/patches/jami-disable-integration-tests.patch \
+ %D%/packages/patches/jami-qml-tests-discovery.patch \
%D%/packages/patches/jami-libjami-headers-search.patch \
%D%/packages/patches/jamvm-1.5.1-aarch64-support.patch \
%D%/packages/patches/jamvm-1.5.1-armv7-support.patch \
diff --git a/gnu/packages/jami.scm b/gnu/packages/jami.scm
index db120f223e..07c25a0601 100644
--- a/gnu/packages/jami.scm
+++ b/gnu/packages/jami.scm
@@ -68,7 +68,7 @@ (define-module (gnu packages jami)
#:use-module (guix packages)
#:use-module (guix utils))
-(define %jami-version "20230323.0")
+(define %jami-version "20230619.1")
(define %jami-sources
;; Return an origin object of the tarball release sources archive of the
@@ -85,9 +85,10 @@ (define %jami-sources
'(delete-file-recursively "daemon/contrib/tarballs"))
(sha256
(base32
- "0vjsjr37cb87j9hqbmipyxn4877k1wn3l0vzca3l3ldgknglz7v2"))
+ "0qb8jvgsqak1hbhkksxj2cxkcy6mb46zl904lwhxfgr5992pl33z"))
(patches (search-patches "jami-disable-integration-tests.patch"
- "jami-libjami-headers-search.patch"))))
+ "jami-libjami-headers-search.patch"
+ "jami-qml-tests-discovery.patch"))))
;; Jami maintains a set of patches for some key dependencies (currently
;; pjproject and ffmpeg) of Jami that haven't yet been integrated upstream.
@@ -409,7 +410,6 @@ (define-public libjami
(inputs
(list alsa-lib
asio
- dbus-c++
eudev
ffmpeg-jami
guile-3.0
@@ -424,6 +424,7 @@ (define-public libjami
openssl
pjproject-jami
pulseaudio
+ sdbus-c++
speex
speexdsp
webrtc-audio-processing
@@ -467,11 +468,7 @@ (define-public jami
"-DENABLE_LIBWRAP=ON")
#:phases
#~(modify-phases %standard-phases
- (add-after 'unpack 'change-directory/maybe
- (lambda _
- ;; Allow building from the tarball or a git checkout.
- (false-if-exception (chdir "client-qt"))))
- (add-after 'change-directory/maybe 'fix-version-string
+ (add-after 'unpack 'fix-version-string
(lambda _
(substitute* "src/app/version.h"
(("VERSION_STRING")
@@ -488,22 +485,13 @@ (define-public jami
;; The tests require a writable HOME.
(setenv "HOME" "/tmp")
- (display "Running unittests...\n")
- (invoke "tests/unittests" "-mutejamid")
+ (display "Running unit tests...\n")
+ (invoke "tests/unit_tests")
- ;; XXX: There are currently multiple failures with the
- ;; functional tests (see:
- ;; https://git.jami.net/savoirfairelinux/jami-client-qt/-/issues/883),
- ;; so the code below is disabled for now.
- ;;
+ ;; XXX: The QML test suite fails, exiting with status code 1 (see:
+ ;; https://git.jami.net/savoirfairelinux/jami-client-qt/-/issues/883).
;; (display "Running functional tests...\n")
- ;; ;; This is to allow building from the source tarball or
- ;; ;; directly from the git repository.
- ;; (let ((tests-qml (if (file-exists? "../client-qt/tests")
- ;; "../client-qt/tests/qml"
- ;; "../tests/qml")))
- ;; (invoke "tests/qml_tests" "-mutejamid"
- ;; "-input" tests-qml))
+ ;; (invoke "tests/qml_tests")
))))))
(native-inputs
(list googletest
diff --git a/gnu/packages/patches/jami-qml-tests-discovery.patch b/gnu/packages/patches/jami-qml-tests-discovery.patch
new file mode 100644
index 0000000000..11fd69571c
--- /dev/null
+++ b/gnu/packages/patches/jami-qml-tests-discovery.patch
@@ -0,0 +1,15 @@
+Upstream status: https://review.jami.net/c/jami-client-qt/+/25640
+
+diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
+index d50908cf..587c9d15 100644
+--- a/tests/CMakeLists.txt
++++ b/tests/CMakeLists.txt
+@@ -73,6 +73,8 @@ endif()
+
+ string(TOUPPER ${CMAKE_BUILD_TYPE} BUILD_TYPE)
+
++set(QUICK_TEST_SOURCE_DIR "${CMAKE_SOURCE_DIR}tests/qml/src")
++
+ set(QML_TESTS_SOURCE_FILES
+ ${CMAKE_SOURCE_DIR}/tests/qml/main.cpp
+ ${TEST_QML_RESOURCES}
--
2.41.0
L
L
Liliana Marie Prikler wrote on 18 Aug 2023 20:53
Re: [bug#65371] [PATCH telephony-team 06/10] gnu: Add sdbus-cpp.
(name . Raghav Gururajan)(address . rg@raghavgururajan.name)
7bb97573bd0fbbce161a1300527664d927faa44c.camel@gmail.com
Am Freitag, dem 18.08.2023 um 14:12 -0400 schrieb Maxim Cournoyer:
Toggle quote (71 lines)
> * gnu/packages/glib.scm (sdbus-cpp): New variable.
> * gnu/packages/patches/sdbus-c++-elogind.patch: New file.
> * gnu/local.mk (dist_patch_DATA): Register it.
> ---
>
>  gnu/local.mk                                 |  1 +
>  gnu/packages/glib.scm                        | 43 +++++++++++++
>  gnu/packages/patches/sdbus-c++-elogind.patch | 63
> ++++++++++++++++++++
>  3 files changed, 107 insertions(+)
>  create mode 100644 gnu/packages/patches/sdbus-c++-elogind.patch
>
> diff --git a/gnu/local.mk b/gnu/local.mk
> index abe8964f4d..de669c1177 100644
> --- a/gnu/local.mk
> +++ b/gnu/local.mk
> @@ -1938,6 +1938,7 @@ dist_patch_DATA
> =                                         \
>    %D%/packages/patches/scons-test-environment.patch            \
>    %D%/packages/patches/screen-hurd-path-max.patch              \
>    %D%/packages/patches/scsh-nonstring-search-path.patch        \
> +  %D%/packages/patches/sdbus-c++-elogind.patch                 \
>    %D%/packages/patches/sdl-libx11-1.6.patch                    \
>    %D%/packages/patches/seed-webkit.patch                       \
>    %D%/packages/patches/sendgmail-accept-ignored-gsuite-
> flag.patch      \
> diff --git a/gnu/packages/glib.scm b/gnu/packages/glib.scm
> index a0b85b15d1..588133e93f 100644
> --- a/gnu/packages/glib.scm
> +++ b/gnu/packages/glib.scm
> @@ -1290,6 +1290,49 @@ (define-public dbus-cxx
>      (home-page "https://dbus-cxx.github.io/")
>      (license license:gpl3)))
>  
> +(define-public sdbus-c++
> +  ;; Use the latest commit, which includes unreleased fixes to the
> pkg-config
> +  ;; file.
> +  (let ((commit "3e84b254e9603935cb5fc180c4d2214d7024ccbb")
> +        (revision "0"))
> +    (package
> +      (name "sdbus-c++")
> +      (version (git-version "1.2.0" revision commit))
> +      (source (origin
> +                (method git-fetch)
> +                (uri (git-reference
> +                      (url
> "https://github.com/Kistler-Group/sdbus-cpp")
> +                      (commit commit)))
> +                (file-name (git-file-name name version))
> +                (sha256
> +                 (base32
> +                 
> "0xrcp49almi3kwzmwcwv8sayqjxx8m55s5grxarw2hl9jmghwlc3"))
> +                (patches (search-patches "sdbus-c++-
> elogind.patch"))))
> +      (build-system cmake-build-system)
> +      (arguments
> +       (list
> +        ;; Avoid the integration test, which requires a system bus.
> +        #:test-target "sdbus-c++-unit-tests"
> +        #:configure-flags #~(list "-DBUILD_CODE_GEN=ON"
> +                                  "-DBUILD_TESTS=ON"
> +                                  ;; Do not install tests.
> +                                  "-DTESTS_INSTALL_PATH=/tmp"
> +                                  "-DCMAKE_VERBOSE_MAKEFILE=ON")
> +        #:phases #~(modify-phases %standard-phases
> +                     (add-after 'unpack 'do-not-install-tests
> +                       (lambda _
> +                         (substitute* "tests/CMakeLists.txt"
> +                           (("/etc/dbus-1/system.d") "/tmp")))))))
Should we perhaps add those to a separate output? Are they useful on
their own?
Toggle quote (64 lines)
> +      (native-inputs (list googletest pkg-config))
> +      (inputs (list expat))
> +      (propagated-inputs (list elogind)) ;required by sdbus-c++.pc
> +      (home-page "https://github.com/Kistler-Group/sdbus-cpp")
> +      (synopsis "High-level C++ D-Bus library")
> +      (description "@code{sdbus-c++} is a high-level C++ D-Bus
> library designed
> +to provide easy-to-use yet powerful API in modern C++.  It adds
> another layer
> +of abstraction on top of @code{sd-bus}, the C D-Bus implementation
> by systemd.")
> +      (license license:lgpl2.1+))))
> +
>  (define-public appstream-glib
>    (package
>      (name "appstream-glib")
> diff --git a/gnu/packages/patches/sdbus-c++-elogind.patch
> b/gnu/packages/patches/sdbus-c++-elogind.patch
> new file mode 100644
> index 0000000000..d01838edbc
> --- /dev/null
> +++ b/gnu/packages/patches/sdbus-c++-elogind.patch
> @@ -0,0 +1,63 @@
> +Submitted upstream:
> https://github.com/Kistler-Group/sdbus-cpp/pull/352
> +
> +From 8423c44b6c24ebd59db06ad33704265aa81a1c7a Mon Sep 17 00:00:00
> 2001
> +From: Sven Eden <sven.eden@prydeworx.com>
> +Date: Thu, 17 Aug 2023 23:16:19 -0400
> +Subject: [PATCH] build: Add support for elogind.
> +
> +* CMakeLists.txt: Fallback to elogind when libsystemd could not be
> +found.  Set LIBSYSTEMD variable.
> +* pkgconfig/sdbus-c++.pc.in (Description): Parameterize with above
> +LIBSYSTEMD variable.
> +
> +Signed-off-by: Maxim Cournoyer <maxim.cournoyer@gmail.com>
> +---
> + CMakeLists.txt            | 11 +++++++++++
> + pkgconfig/sdbus-c++.pc.in |  2 +-
> + 2 files changed, 12 insertions(+), 1 deletion(-)
> +
> +diff --git a/CMakeLists.txt b/CMakeLists.txt
> +index 94f71c7..40c15f8 100644
> +--- a/CMakeLists.txt
> ++++ b/CMakeLists.txt
> +@@ -12,11 +12,22 @@ include(GNUInstallDirs) # Installation
> directories for `install` command and pkg
> + # PERFORMING CHECKS & PREPARING THE DEPENDENCIES
> + #-------------------------------
> +
> ++set(LIBSYSTEMD "libsystemd")
> ++
> + option(BUILD_LIBSYSTEMD "Build libsystemd static library and
> incorporate it into libsdbus-c++" OFF)
> +
> + if(NOT BUILD_LIBSYSTEMD)
> +     find_package(PkgConfig REQUIRED)
> +     pkg_check_modules(Systemd IMPORTED_TARGET GLOBAL
> libsystemd>=236)
> ++    if(NOT TARGET PkgConfig::Systemd)
> ++        message(WARNING "libsystemd not found, checking for
> libelogind instead")
Perhaps demote this to INFO?
Toggle quote (31 lines)
> ++        pkg_check_modules(Systemd IMPORTED_TARGET GLOBAL
> libelogind>=236)
> ++        if(TARGET PkgConfig::Systemd)
> ++            set(LIBSYSTEMD "libelogind")
> ++            string(REPLACE "." ";" VERSION_LIST ${Systemd_VERSION})
> ++            list(GET VERSION_LIST 0 Systemd_VERSION)
> ++        endif()
> ++    endif()
> +     if(NOT TARGET PkgConfig::Systemd)
> +         message(FATAL_ERROR "libsystemd of version at least 236 is
> required, but was not found "
> +                             "(if you have systemd in your OS, you
> may want to install package containing pkgconfig "
> +diff --git a/pkgconfig/sdbus-c++.pc.in b/pkgconfig/sdbus-c++.pc.in
> +index 6ad010a..0703476 100644
> +--- a/pkgconfig/sdbus-c++.pc.in
> ++++ b/pkgconfig/sdbus-c++.pc.in
> +@@ -5,7 +5,7 @@ includedir=@CMAKE_INSTALL_FULL_INCLUDEDIR@
> +
> + Name: @PROJECT_NAME@
> + Description: C++ library on top of sd-bus, a systemd D-Bus library
> +-Requires@PKGCONFIG_REQS@: libsystemd
> ++Requires@PKGCONFIG_REQS@: @LIBSYSTEMD@
> + Version: @SDBUSCPP_VERSION@
> + Libs: -L${libdir} -l@PROJECT_NAME@
> + Cflags: -I${includedir}
> +
> +base-commit: 3e84b254e9603935cb5fc180c4d2214d7024ccbb
> +--
> +2.41.0
> +
Otherwise LGTM.

Cheers
A
A
Andreas Enge wrote on 19 Aug 2023 11:53
Re: [bug#65371] [PATCH telephony-team 04/10] gnu: python-pymol: Remove msgpack from native-inputs.
(name . Maxim Cournoyer)(address . maxim.cournoyer@gmail.com)
ZOCRBF9iRkAH0B0w@jurong
Hello Maxim,

Am Fri, Aug 18, 2023 at 02:12:55PM -0400 schrieb Maxim Cournoyer:
Toggle quote (4 lines)
> It's now propagated by mmtf-cpp.
> * gnu/packages/chemistry.scm (python-pymol)
> [native-inputs]: Remove msgpack.

the patches 2 to 4, to which I was cc-ed, look good to me.

Andreas
M
M
Maxim Cournoyer wrote on 21 Aug 2023 16:57
Re: [bug#65371] [PATCH telephony-team 06/10] gnu: Add sdbus-cpp.
(name . Liliana Marie Prikler)(address . liliana.prikler@gmail.com)
87jztoe94v.fsf@gmail.com
Hi Liliana,
Liliana Marie Prikler <liliana.prikler@gmail.com> writes:
Toggle quote (74 lines)
> Am Freitag, dem 18.08.2023 um 14:12 -0400 schrieb Maxim Cournoyer:
>> * gnu/packages/glib.scm (sdbus-cpp): New variable.
>> * gnu/packages/patches/sdbus-c++-elogind.patch: New file.
>> * gnu/local.mk (dist_patch_DATA): Register it.
>> ---
>>
>>  gnu/local.mk                                 |  1 +
>>  gnu/packages/glib.scm                        | 43 +++++++++++++
>>  gnu/packages/patches/sdbus-c++-elogind.patch | 63
>> ++++++++++++++++++++
>>  3 files changed, 107 insertions(+)
>>  create mode 100644 gnu/packages/patches/sdbus-c++-elogind.patch
>>
>> diff --git a/gnu/local.mk b/gnu/local.mk
>> index abe8964f4d..de669c1177 100644
>> --- a/gnu/local.mk
>> +++ b/gnu/local.mk
>> @@ -1938,6 +1938,7 @@ dist_patch_DATA
>> =                                         \
>>    %D%/packages/patches/scons-test-environment.patch            \
>>    %D%/packages/patches/screen-hurd-path-max.patch              \
>>    %D%/packages/patches/scsh-nonstring-search-path.patch        \
>> +  %D%/packages/patches/sdbus-c++-elogind.patch                 \
>>    %D%/packages/patches/sdl-libx11-1.6.patch                    \
>>    %D%/packages/patches/seed-webkit.patch                       \
>>    %D%/packages/patches/sendgmail-accept-ignored-gsuite-
>> flag.patch      \
>> diff --git a/gnu/packages/glib.scm b/gnu/packages/glib.scm
>> index a0b85b15d1..588133e93f 100644
>> --- a/gnu/packages/glib.scm
>> +++ b/gnu/packages/glib.scm
>> @@ -1290,6 +1290,49 @@ (define-public dbus-cxx
>>      (home-page "https://dbus-cxx.github.io/")
>>      (license license:gpl3)))
>>  
>> +(define-public sdbus-c++
>> +  ;; Use the latest commit, which includes unreleased fixes to the
>> pkg-config
>> +  ;; file.
>> +  (let ((commit "3e84b254e9603935cb5fc180c4d2214d7024ccbb")
>> +        (revision "0"))
>> +    (package
>> +      (name "sdbus-c++")
>> +      (version (git-version "1.2.0" revision commit))
>> +      (source (origin
>> +                (method git-fetch)
>> +                (uri (git-reference
>> +                      (url
>> "https://github.com/Kistler-Group/sdbus-cpp")
>> +                      (commit commit)))
>> +                (file-name (git-file-name name version))
>> +                (sha256
>> +                 (base32
>> +                 
>> "0xrcp49almi3kwzmwcwv8sayqjxx8m55s5grxarw2hl9jmghwlc3"))
>> +                (patches (search-patches "sdbus-c++-
>> elogind.patch"))))
>> +      (build-system cmake-build-system)
>> +      (arguments
>> +       (list
>> +        ;; Avoid the integration test, which requires a system bus.
>> +        #:test-target "sdbus-c++-unit-tests"
>> +        #:configure-flags #~(list "-DBUILD_CODE_GEN=ON"
>> +                                  "-DBUILD_TESTS=ON"
>> +                                  ;; Do not install tests.
>> +                                  "-DTESTS_INSTALL_PATH=/tmp"
>> +                                  "-DCMAKE_VERBOSE_MAKEFILE=ON")
>> +        #:phases #~(modify-phases %standard-phases
>> +                     (add-after 'unpack 'do-not-install-tests
>> +                       (lambda _
>> +                         (substitute* "tests/CMakeLists.txt"
>> +                           (("/etc/dbus-1/system.d") "/tmp")))))))
> Should we perhaps add those to a separate output? Are they useful on
> their own?
My informal policy on tests installation has been "only maybe install
tests if they cannot be made to run at build time", which is not the
case here.
Toggle quote (65 lines)
>> +      (native-inputs (list googletest pkg-config))
>> +      (inputs (list expat))
>> +      (propagated-inputs (list elogind)) ;required by sdbus-c++.pc
>> +      (home-page "https://github.com/Kistler-Group/sdbus-cpp")
>> +      (synopsis "High-level C++ D-Bus library")
>> +      (description "@code{sdbus-c++} is a high-level C++ D-Bus
>> library designed
>> +to provide easy-to-use yet powerful API in modern C++.  It adds
>> another layer
>> +of abstraction on top of @code{sd-bus}, the C D-Bus implementation
>> by systemd.")
>> +      (license license:lgpl2.1+))))
>> +
>>  (define-public appstream-glib
>>    (package
>>      (name "appstream-glib")
>> diff --git a/gnu/packages/patches/sdbus-c++-elogind.patch
>> b/gnu/packages/patches/sdbus-c++-elogind.patch
>> new file mode 100644
>> index 0000000000..d01838edbc
>> --- /dev/null
>> +++ b/gnu/packages/patches/sdbus-c++-elogind.patch
>> @@ -0,0 +1,63 @@
>> +Submitted upstream:
>> https://github.com/Kistler-Group/sdbus-cpp/pull/352
>> +
>> +From 8423c44b6c24ebd59db06ad33704265aa81a1c7a Mon Sep 17 00:00:00
>> 2001
>> +From: Sven Eden <sven.eden@prydeworx.com>
>> +Date: Thu, 17 Aug 2023 23:16:19 -0400
>> +Subject: [PATCH] build: Add support for elogind.
>> +
>> +* CMakeLists.txt: Fallback to elogind when libsystemd could not be
>> +found.  Set LIBSYSTEMD variable.
>> +* pkgconfig/sdbus-c++.pc.in (Description): Parameterize with above
>> +LIBSYSTEMD variable.
>> +
>> +Signed-off-by: Maxim Cournoyer <maxim.cournoyer@gmail.com>
>> +---
>> + CMakeLists.txt            | 11 +++++++++++
>> + pkgconfig/sdbus-c++.pc.in |  2 +-
>> + 2 files changed, 12 insertions(+), 1 deletion(-)
>> +
>> +diff --git a/CMakeLists.txt b/CMakeLists.txt
>> +index 94f71c7..40c15f8 100644
>> +--- a/CMakeLists.txt
>> ++++ b/CMakeLists.txt
>> +@@ -12,11 +12,22 @@ include(GNUInstallDirs) # Installation
>> directories for `install` command and pkg
>> + # PERFORMING CHECKS & PREPARING THE DEPENDENCIES
>> + #-------------------------------
>> +
>> ++set(LIBSYSTEMD "libsystemd")
>> ++
>> + option(BUILD_LIBSYSTEMD "Build libsystemd static library and
>> incorporate it into libsdbus-c++" OFF)
>> +
>> + if(NOT BUILD_LIBSYSTEMD)
>> +     find_package(PkgConfig REQUIRED)
>> +     pkg_check_modules(Systemd IMPORTED_TARGET GLOBAL
>> libsystemd>=236)
>> ++    if(NOT TARGET PkgConfig::Systemd)
>> ++        message(WARNING "libsystemd not found, checking for
>> libelogind instead")
> Perhaps demote this to INFO?
I agree, but the patch is not entirely mine and more importantly has
already been merged :-).
Toggle quote (32 lines)
>> ++        pkg_check_modules(Systemd IMPORTED_TARGET GLOBAL
>> libelogind>=236)
>> ++        if(TARGET PkgConfig::Systemd)
>> ++            set(LIBSYSTEMD "libelogind")
>> ++            string(REPLACE "." ";" VERSION_LIST ${Systemd_VERSION})
>> ++            list(GET VERSION_LIST 0 Systemd_VERSION)
>> ++        endif()
>> ++    endif()
>> +     if(NOT TARGET PkgConfig::Systemd)
>> +         message(FATAL_ERROR "libsystemd of version at least 236 is
>> required, but was not found "
>> +                             "(if you have systemd in your OS, you
>> may want to install package containing pkgconfig "
>> +diff --git a/pkgconfig/sdbus-c++.pc.in b/pkgconfig/sdbus-c++.pc.in
>> +index 6ad010a..0703476 100644
>> +--- a/pkgconfig/sdbus-c++.pc.in
>> ++++ b/pkgconfig/sdbus-c++.pc.in
>> +@@ -5,7 +5,7 @@ includedir=@CMAKE_INSTALL_FULL_INCLUDEDIR@
>> +
>> + Name: @PROJECT_NAME@
>> + Description: C++ library on top of sd-bus, a systemd D-Bus library
>> +-Requires@PKGCONFIG_REQS@: libsystemd
>> ++Requires@PKGCONFIG_REQS@: @LIBSYSTEMD@
>> + Version: @SDBUSCPP_VERSION@
>> + Libs: -L${libdir} -l@PROJECT_NAME@
>> + Cflags: -I${includedir}
>> +
>> +base-commit: 3e84b254e9603935cb5fc180c4d2214d7024ccbb
>> +--
>> +2.41.0
>> +
> Otherwise LGTM.
Thanks for the review!
--
Thanks,
Maxim
M
M
Maxim Cournoyer wrote on 21 Aug 2023 17:03
Re: [bug#65371] [PATCH telephony-team 04/10] gnu: python-pymol: Remove msgpack from native-inputs.
(name . Andreas Enge)(address . andreas@enge.fr)
87fs4ce8ue.fsf@gmail.com
Hello,

Andreas Enge <andreas@enge.fr> writes:

Toggle quote (11 lines)
> Hello Maxim,
>
> Am Fri, Aug 18, 2023 at 02:12:55PM -0400 schrieb Maxim Cournoyer:
>> It's now propagated by mmtf-cpp.
>> * gnu/packages/chemistry.scm (python-pymol)
>> [native-inputs]: Remove msgpack.
>
> the patches 2 to 4, to which I was cc-ed, look good to me.
>
> Andreas

Thanks for reviewing these! Will push shortly and update the series
with the remaining ones.

--
Thanks,
Maxim
M
M
Maxim Cournoyer wrote on 10 Jan 04:37 +0100
Re: [bug#65371] [PATCH telephony-team 06/10] gnu: Add sdbus-cpp.
(name . Liliana Marie Prikler)(address . liliana.prikler@gmail.com)
87ply9uadl.fsf@gmail.com
Hi,

This refreshed series has now been merged.

--
Thanks,
Maxim
Closed
?