Package: guix-patches;
Reported by: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Date: Fri, 18 Aug 2023 18:00:01 UTC
Severity: normal
Tags: patch
Done: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 65371 in the body.
You can then email your comments to 65371 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
View this report as an mbox folder, status mbox, maintainer mbox
maxim.cournoyer <at> gmail.com, guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:00:02 GMT) Full text and rfc822 format available.Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
:maxim.cournoyer <at> gmail.com, guix-patches <at> gnu.org
.
(Fri, 18 Aug 2023 18:00:02 GMT) Full text and rfc822 format available.Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: guix-patches <at> gnu.org, maxim.cournoyer <at> gmail.com Subject: [PATCH 00/10] [telephony-team] Update Jami and related packages. Date: Fri, 18 Aug 2023 13:59:15 -0400
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
guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:18:01 GMT) Full text and rfc822 format available.Message #8 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: 65371 <at> debbugs.gnu.org Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: [PATCH telephony-team 01/10] gnu: opendht: Update to 2.5.5. Date: Fri, 18 Aug 2023 14:12:52 -0400
* 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(-) 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
andreas <at> enge.fr, efraim <at> flashner.co.il, bavier <at> posteo.net, guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:18:02 GMT) Full text and rfc822 format available.Message #11 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: 65371 <at> debbugs.gnu.org Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: [PATCH telephony-team 02/10] gnu: mmtf-cpp: Update to 1.1.0. Date: Fri, 18 Aug 2023 14:12:53 -0400
* 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(-) 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
andreas <at> enge.fr, efraim <at> flashner.co.il, bavier <at> posteo.net, guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:18:02 GMT) Full text and rfc822 format available.Message #14 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: 65371 <at> debbugs.gnu.org Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: [PATCH telephony-team 03/10] gnu: avogadrolibs: Remove msgpack from native-inputs. Date: Fri, 18 Aug 2023 14:12:54 -0400
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(-) 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
andreas <at> enge.fr, efraim <at> flashner.co.il, bavier <at> posteo.net, guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:18:03 GMT) Full text and rfc822 format available.Message #17 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: 65371 <at> debbugs.gnu.org Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: [PATCH telephony-team 04/10] gnu: python-pymol: Remove msgpack from native-inputs. Date: Fri, 18 Aug 2023 14:12:55 -0400
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(-) 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
guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:18:03 GMT) Full text and rfc822 format available.Message #20 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: 65371 <at> debbugs.gnu.org Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: [PATCH telephony-team 05/10] gnu: msgpack: Deprecate with msgpack-c. Date: Fri, 18 Aug 2023 14:12:56 -0400
* 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(-) 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 <at> mmer.org> ;;; Copyright © 2023 Sharlatan Hellseher <sharlatanus <at> gmail.com> ;;; Copyright © 2023 Vinicius Monego <monego <at> posteo.net> +;;; Copyright © 2023 Maxim Cournoyer <maxim.cournoyer <at> 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
liliana.prikler <at> gmail.com, maxim.cournoyer <at> gmail.com, rg <at> raghavgururajan.name, guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:18:04 GMT) Full text and rfc822 format available.Message #23 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: 65371 <at> debbugs.gnu.org Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: [PATCH telephony-team 06/10] gnu: Add sdbus-cpp. Date: Fri, 18 Aug 2023 14:12:57 -0400
* 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"))))))) + (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 <at> 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 <at> 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 <at> PKGCONFIG_REQS@: libsystemd ++Requires <at> PKGCONFIG_REQS@: @LIBSYSTEMD@ + Version: @SDBUSCPP_VERSION@ + Libs: -L${libdir} -l <at> PROJECT_NAME@ + Cflags: -I${includedir} + +base-commit: 3e84b254e9603935cb5fc180c4d2214d7024ccbb +-- +2.41.0 + -- 2.41.0
guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:18:04 GMT) Full text and rfc822 format available.Message #26 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: 65371 <at> debbugs.gnu.org Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: [PATCH telephony-team 07/10] gnu: libnatpmp: Use gexps and simplify. Date: Fri, 18 Aug 2023 14:12:58 -0400
* 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(-) 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
guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:18:04 GMT) Full text and rfc822 format available.Message #29 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: 65371 <at> debbugs.gnu.org Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: [PATCH telephony-team 08/10] gnu: libnatpmp: Update home page. Date: Fri, 18 Aug 2023 14:12:59 -0400
* gnu/packages/networking.scm (libnatpmp) [home-page]: Update. --- gnu/packages/networking.scm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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
guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:18:05 GMT) Full text and rfc822 format available.Message #32 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: 65371 <at> debbugs.gnu.org Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: [PATCH telephony-team 09/10] gnu: libnatpmp: Install missing natpmp_declspec.h header. Date: Fri, 18 Aug 2023 14:13:00 -0400
* 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(+) 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
maxim.cournoyer <at> gmail.com, guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:18:06 GMT) Full text and rfc822 format available.Message #35 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: 65371 <at> debbugs.gnu.org Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Subject: [PATCH telephony-team 10/10] gnu: jami: Update to 20230619.1. Date: Fri, 18 Aug 2023 14:13:01 -0400
(%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 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
guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Fri, 18 Aug 2023 18:54:01 GMT) Full text and rfc822 format available.Message #38 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Liliana Marie Prikler <liliana.prikler <at> gmail.com> To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>, 65371 <at> debbugs.gnu.org Cc: Raghav Gururajan <rg <at> raghavgururajan.name> Subject: Re: [bug#65371] [PATCH telephony-team 06/10] gnu: Add sdbus-cpp. Date: Fri, 18 Aug 2023 20:53:43 +0200
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? > + (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 <at> 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 <at> 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? > ++ 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 <at> PKGCONFIG_REQS@: libsystemd > ++Requires <at> PKGCONFIG_REQS@: @LIBSYSTEMD@ > + Version: @SDBUSCPP_VERSION@ > + Libs: -L${libdir} -l <at> PROJECT_NAME@ > + Cflags: -I${includedir} > + > +base-commit: 3e84b254e9603935cb5fc180c4d2214d7024ccbb > +-- > +2.41.0 > + Otherwise LGTM. Cheers
guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Sat, 19 Aug 2023 09:54:02 GMT) Full text and rfc822 format available.Message #41 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Andreas Enge <andreas <at> enge.fr> To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> Cc: 65371 <at> debbugs.gnu.org, Efraim Flashner <efraim <at> flashner.co.il>, Eric Bavier <bavier <at> posteo.net> Subject: Re: [bug#65371] [PATCH telephony-team 04/10] gnu: python-pymol: Remove msgpack from native-inputs. Date: Sat, 19 Aug 2023 11:53:08 +0200
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
guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Mon, 21 Aug 2023 14:58:01 GMT) Full text and rfc822 format available.Message #44 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: Liliana Marie Prikler <liliana.prikler <at> gmail.com> Cc: 65371 <at> debbugs.gnu.org, Raghav Gururajan <rg <at> raghavgururajan.name> Subject: Re: [bug#65371] [PATCH telephony-team 06/10] gnu: Add sdbus-cpp. Date: Mon, 21 Aug 2023 10:57:04 -0400
Hi Liliana, Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes: > 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. >> + (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 <at> 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 <at> 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 :-). >> ++ 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 <at> PKGCONFIG_REQS@: libsystemd >> ++Requires <at> PKGCONFIG_REQS@: @LIBSYSTEMD@ >> + Version: @SDBUSCPP_VERSION@ >> + Libs: -L${libdir} -l <at> PROJECT_NAME@ >> + Cflags: -I${includedir} >> + >> +base-commit: 3e84b254e9603935cb5fc180c4d2214d7024ccbb >> +-- >> +2.41.0 >> + > Otherwise LGTM. Thanks for the review! -- Thanks, Maxim
guix-patches <at> gnu.org
:bug#65371
; Package guix-patches
.
(Mon, 21 Aug 2023 15:04:02 GMT) Full text and rfc822 format available.Message #47 received at 65371 <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: Andreas Enge <andreas <at> enge.fr> Cc: 65371 <at> debbugs.gnu.org, Efraim Flashner <efraim <at> flashner.co.il>, Eric Bavier <bavier <at> posteo.net> Subject: Re: [bug#65371] [PATCH telephony-team 04/10] gnu: python-pymol: Remove msgpack from native-inputs. Date: Mon, 21 Aug 2023 11:03:21 -0400
Hello, Andreas Enge <andreas <at> enge.fr> writes: > 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
Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
:Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
:Message #52 received at 65371-done <at> debbugs.gnu.org (full text, mbox):
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com> To: Liliana Marie Prikler <liliana.prikler <at> gmail.com> Cc: Raghav Gururajan <rg <at> raghavgururajan.name>, 65371-done <at> debbugs.gnu.org Subject: Re: [bug#65371] [PATCH telephony-team 06/10] gnu: Add sdbus-cpp. Date: Tue, 09 Jan 2024 22:37:26 -0500
Hi, This refreshed series has now been merged. -- Thanks, Maxim
Debbugs Internal Request <help-debbugs <at> gnu.org>
to internal_control <at> debbugs.gnu.org
.
(Wed, 07 Feb 2024 12:24:15 GMT) Full text and rfc822 format available.
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.