unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#73981] [PATCH 0/4] gnu: update modem-manager and deps.
@ 2024-10-24  7:36 Zheng Junjie
  2024-10-24  7:40 ` [bug#73981] [PATCH 1/4] gnu: libmbim: Update to 1.30.0 Zheng Junjie
                   ` (4 more replies)
  0 siblings, 5 replies; 15+ messages in thread
From: Zheng Junjie @ 2024-10-24  7:36 UTC (permalink / raw)
  To: 73981

Zheng Junjie (4):
  gnu: libmbim: Update to 1.30.0.
  gnu: Add libqrtr-glib.
  gnu: libqmi: Update to 1.34.0.
  gnu: modem-manager: Update to 1.22.0.

 gnu/packages/freedesktop.scm | 104 +++++++++++++++++++++++++----------
 1 file changed, 74 insertions(+), 30 deletions(-)


base-commit: 0b3f419022d40d29b2e17e3abdeb0e9718976680
-- 
2.46.0





^ permalink raw reply	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH 1/4] gnu: libmbim: Update to 1.30.0.
  2024-10-24  7:36 [bug#73981] [PATCH 0/4] gnu: update modem-manager and deps Zheng Junjie
@ 2024-10-24  7:40 ` Zheng Junjie
  2024-11-11 14:22   ` Maxim Cournoyer
  2024-10-24  7:40 ` [bug#73981] [PATCH 2/4] gnu: Add libqrtr-glib Zheng Junjie
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 15+ messages in thread
From: Zheng Junjie @ 2024-10-24  7:40 UTC (permalink / raw)
  To: 73981

* gnu/packages/freedesktop.scm (libmbim): Update to 1.30.0.
[source]: Switch to git-fetch.
[build-system]: Switch to meson-build-system.
[native-inputs]: Add gobject-introspection, bash-completion, help2man. Replace
python-wrapper with python-minimal.

Change-Id: I607bff667a057e58bebb2f706d7021a4d1c8d763
---
 gnu/packages/freedesktop.scm | 21 +++++++++++++--------
 1 file changed, 13 insertions(+), 8 deletions(-)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index 0ab0bb51044..a7c740f3d05 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -1900,19 +1900,24 @@ (define-public accountsservice
 (define-public libmbim
   (package
     (name "libmbim")
-    (version "1.26.4")
+    (version "1.30.0")
     (source (origin
-              (method url-fetch)
-              (uri (string-append
-                    "https://www.freedesktop.org/software/libmbim/"
-                    "libmbim-" version ".tar.xz"))
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://gitlab.freedesktop.org/mobile-broadband/libmbim")
+                    (commit version)))
+              (file-name (git-file-name name version))
               (sha256
                (base32
-                "1ncaarl4lgc7i52rwz50yq701wk2rr478cjybxbifsjqqk2cx27n"))))
-    (build-system gnu-build-system)
+                "00kbjvpka51zrfjigzd3rk6r4x8hkg1xfj7d9zl9lccysnxyjx5h"))))
+    (build-system meson-build-system)
     (native-inputs
      (list `(,glib "bin") ; for glib-mkenums
-           pkg-config python-wrapper))
+           pkg-config
+           python-minimal
+           gobject-introspection
+           bash-completion
+           help2man))
     (propagated-inputs
      (list glib)) ; required by mbim-glib.pc
     (inputs
-- 
2.46.0





^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH 2/4] gnu: Add libqrtr-glib.
  2024-10-24  7:36 [bug#73981] [PATCH 0/4] gnu: update modem-manager and deps Zheng Junjie
  2024-10-24  7:40 ` [bug#73981] [PATCH 1/4] gnu: libmbim: Update to 1.30.0 Zheng Junjie
@ 2024-10-24  7:40 ` Zheng Junjie
  2024-11-11 14:28   ` Maxim Cournoyer
  2024-10-24  7:40 ` [bug#73981] [PATCH 3/4] gnu: libqmi: Update to 1.34.0 Zheng Junjie
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 15+ messages in thread
From: Zheng Junjie @ 2024-10-24  7:40 UTC (permalink / raw)
  To: 73981

* gnu/packages/freedesktop.scm (libqrtr-glib): New variable.

Change-Id: I84e26d0e8f69c8e5bc1c65925792aea296f6a97d
---
 gnu/packages/freedesktop.scm | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index a7c740f3d05..321ab2a1d1e 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -1932,6 +1932,37 @@ (define-public libmbim
      ;; The mbimcli tool is released under the GPLv2+ license.
      (list license:lgpl2.0+ license:gpl2+))))
 
+(define-public libqrtr-glib
+  (package
+    (name "libqrtr-glib")
+    (version "1.2.2")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://gitlab.freedesktop.org/mobile-broadband/libqrtr-glib")
+                    (commit version)))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "0bfn5kfscli0rrxvmzdl6ix5ikh0kxia7ad83rmh1hksfcwynwlh"))))
+    (build-system meson-build-system)
+    (inputs
+     (list libgudev libmbim))
+    (native-inputs
+     (list `(,glib "bin")
+           gtk-doc
+           pkg-config
+           gobject-introspection
+           bash-completion))
+    (propagated-inputs
+     (list glib))
+    (synopsis "Qualcomm IPC Router protocol helper library")
+    (description
+     "libqrtr-glib is a glib-based library to use and manage the QRTR (Qualcomm
+IPC Router) bus.")
+    (home-page "https://gitlab.freedesktop.org/mobile-broadband/libqrtr-glib")
+    (license license:lgpl2.1+)))
+
 (define-public libqmi
   (package
     (name "libqmi")
-- 
2.46.0





^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH 3/4] gnu: libqmi: Update to 1.34.0.
  2024-10-24  7:36 [bug#73981] [PATCH 0/4] gnu: update modem-manager and deps Zheng Junjie
  2024-10-24  7:40 ` [bug#73981] [PATCH 1/4] gnu: libmbim: Update to 1.30.0 Zheng Junjie
  2024-10-24  7:40 ` [bug#73981] [PATCH 2/4] gnu: Add libqrtr-glib Zheng Junjie
@ 2024-10-24  7:40 ` Zheng Junjie
  2024-11-12  1:06   ` Maxim Cournoyer
  2024-10-24  7:40 ` [bug#73981] [PATCH 4/4] gnu: modem-manager: Update to 1.22.0 Zheng Junjie
  2024-11-12  3:46 ` [bug#73981] [PATCH v2 0/4] update modem-manager and deps Zheng Junjie
  4 siblings, 1 reply; 15+ messages in thread
From: Zheng Junjie @ 2024-10-24  7:40 UTC (permalink / raw)
  To: 73981

* gnu/packages/freedesktop.scm (libqmi): Update to 1.34.0.
[inputs]: Add libmbim and libqrtr-glib.
[native-inputs]: Add gobject-introspection, bash-completion and
help2man. Replace python-wrapper with python-minimal.
[build-system]: Switch to meson-build-system.
[source]: Swich to git-fetch.

Change-Id: I8a023cc07e5484d8abd5df4615ad5f4115729c1a
---
 gnu/packages/freedesktop.scm | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index 321ab2a1d1e..ccba510303d 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -1966,21 +1966,25 @@ (define-public libqrtr-glib
 (define-public libqmi
   (package
     (name "libqmi")
-    (version "1.30.8")
+    (version "1.34.0")
     (source (origin
-              (method url-fetch)
-              (uri (string-append
-                    "https://www.freedesktop.org/software/libqmi/"
-                    "libqmi-" version ".tar.xz"))
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://gitlab.freedesktop.org/mobile-broadband/libqmi")
+                    (commit version)))
+              (file-name (git-file-name name version))
               (sha256
                (base32
-                "140rmjw436rh6rqmnfw6yaflpffd27ilwcv4s9jvvl1skv784946"))))
-    (build-system gnu-build-system)
+                "1m5y2sf14qd2i9mvbb68wxqlfwvpiprgz8zmcx6wb2cnjgsszmwp"))))
+    (build-system meson-build-system)
     (inputs
-     (list libgudev))
+     (list libgudev libmbim libqrtr-glib))
     (native-inputs
      (list `(,glib "bin") ; for glib-mkenums
-           pkg-config python-wrapper))
+           pkg-config python-minimal
+           gobject-introspection
+           bash-completion
+           help2man))
     (propagated-inputs
      (list glib)) ; required by qmi-glib.pc
     (synopsis "Library to communicate with QMI-powered modems")
-- 
2.46.0





^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH 4/4] gnu: modem-manager: Update to 1.22.0.
  2024-10-24  7:36 [bug#73981] [PATCH 0/4] gnu: update modem-manager and deps Zheng Junjie
                   ` (2 preceding siblings ...)
  2024-10-24  7:40 ` [bug#73981] [PATCH 3/4] gnu: libqmi: Update to 1.34.0 Zheng Junjie
@ 2024-10-24  7:40 ` Zheng Junjie
  2024-11-12  1:13   ` Maxim Cournoyer
  2024-11-12  3:46 ` [bug#73981] [PATCH v2 0/4] update modem-manager and deps Zheng Junjie
  4 siblings, 1 reply; 15+ messages in thread
From: Zheng Junjie @ 2024-10-24  7:40 UTC (permalink / raw)
  To: 73981

* gnu/packages/freedesktop.scm (modem-manager): Update to 1.22.0.
[source]: Switch to git-fetch.
[build-system]: Switch to meson-build-system.
[arguments]<#:configure-flags>: Adjust udevdir. Set -Dsystemdsystemunitdir=no.
[inputs]: Add libqrtr-glib and elogind.
[native-inputs]: Remove python; add python-minimal, libxslt, and
bash-completion.

Change-Id: Ife0560eb48f3cceaf22ccc6bb54010074028cddf
---
 gnu/packages/freedesktop.scm | 30 +++++++++++++++++-------------
 1 file changed, 17 insertions(+), 13 deletions(-)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index ccba510303d..6eec91a907f 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -2000,34 +2000,38 @@ (define-public libqmi
 (define-public modem-manager
   (package
     (name "modem-manager")
-    (version "1.18.12")
+    (version "1.22.0")
     (source (origin
-              (method url-fetch)
-              (uri (string-append
-                    "https://www.freedesktop.org/software/ModemManager/"
-                    "ModemManager-" version ".tar.xz"))
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://gitlab.freedesktop.org/mobile-broadband/ModemManager")
+                    (commit version)))
+              (file-name (git-file-name name version))
               (sha256
                (base32
-                "0c74n5jl1qvq2qlbwzfkgxny8smjcgkid1nhdnl6qnlmbn9f8r5l"))))
-    (build-system gnu-build-system)
+                "0fj4ibjfsxal3xfk3hrj4l9vg7zbj42k9lj7151illl2n3d5ngzw"))))
+    (build-system meson-build-system)
     (arguments
      (list
       #:configure-flags
-      #~(list (string-append "--with-udev-base-dir=" #$output "/lib/udev"))))
+      #~(list (string-append "-Dudevdir=" #$output "/lib/udev")
+              "-Dsystemdsystemunitdir=no")))
     (native-inputs
-     (list dbus
+     (list bash-completion
+           dbus
            gettext-minimal
            gobject-introspection
            `(,glib "bin")               ;for glib-mkenums
+           libxslt                      ; for xsltproc
            pkg-config
-           python
-           python-dbus
-           python-pygobject
+           python-minimal
+           python-dbus                  ; for test
+           python-pygobject             ; for test
            vala))
     (propagated-inputs
      (list glib))                       ;required by mm-glib.pc
     (inputs
-     (list libgudev libmbim libqmi polkit))
+     (list elogind libgudev libmbim libqmi libqrtr-glib polkit))
     (synopsis "Mobile broadband modems manager")
     (home-page "https://www.freedesktop.org/wiki/Software/ModemManager/")
     (description
-- 
2.46.0





^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH 1/4] gnu: libmbim: Update to 1.30.0.
  2024-10-24  7:40 ` [bug#73981] [PATCH 1/4] gnu: libmbim: Update to 1.30.0 Zheng Junjie
@ 2024-11-11 14:22   ` Maxim Cournoyer
  0 siblings, 0 replies; 15+ messages in thread
From: Maxim Cournoyer @ 2024-11-11 14:22 UTC (permalink / raw)
  To: Zheng Junjie; +Cc: 73981

Hi Zheng,

Zheng Junjie <zhengjunjie@iscas.ac.cn> writes:

Apologies, it seems I've duplicated your work in my just submitted
#74306!  Some comments below.

> * gnu/packages/freedesktop.scm (libmbim): Update to 1.30.0.
> [source]: Switch to git-fetch.
> [build-system]: Switch to meson-build-system.
> [native-inputs]: Add gobject-introspection, bash-completion, help2man. Replace
> python-wrapper with python-minimal.
>
> Change-Id: I607bff667a057e58bebb2f706d7021a4d1c8d763
> ---
>  gnu/packages/freedesktop.scm | 21 +++++++++++++--------
>  1 file changed, 13 insertions(+), 8 deletions(-)
>
> diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
> index 0ab0bb51044..a7c740f3d05 100644
> --- a/gnu/packages/freedesktop.scm
> +++ b/gnu/packages/freedesktop.scm
> @@ -1900,19 +1900,24 @@ (define-public accountsservice
>  (define-public libmbim
>    (package
>      (name "libmbim")
> -    (version "1.26.4")
> +    (version "1.30.0")
>      (source (origin
> -              (method url-fetch)
> -              (uri (string-append
> -                    "https://www.freedesktop.org/software/libmbim/"
> -                    "libmbim-" version ".tar.xz"))
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://gitlab.freedesktop.org/mobile-broadband/libmbim")
> +                    (commit version)))
> +              (file-name (git-file-name name version))
>                (sha256
>                 (base32
> -                "1ncaarl4lgc7i52rwz50yq701wk2rr478cjybxbifsjqqk2cx27n"))))
> -    (build-system gnu-build-system)
> +                "00kbjvpka51zrfjigzd3rk6r4x8hkg1xfj7d9zl9lccysnxyjx5h"))))
> +    (build-system meson-build-system)
>      (native-inputs
>       (list `(,glib "bin") ; for glib-mkenums
> -           pkg-config python-wrapper))
> +           pkg-config
> +           python-minimal
> +           gobject-introspection
> +           bash-completion
> +           help2man))

Here it would be nice to keep the inputs sorted alphabetically.
Shouldn't bash-completion be an 'input' instead of a 'native-input' ?

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH 2/4] gnu: Add libqrtr-glib.
  2024-10-24  7:40 ` [bug#73981] [PATCH 2/4] gnu: Add libqrtr-glib Zheng Junjie
@ 2024-11-11 14:28   ` Maxim Cournoyer
  0 siblings, 0 replies; 15+ messages in thread
From: Maxim Cournoyer @ 2024-11-11 14:28 UTC (permalink / raw)
  To: Zheng Junjie; +Cc: 73981

Zheng Junjie <zhengjunjie@iscas.ac.cn> writes:

> * gnu/packages/freedesktop.scm (libqrtr-glib): New variable.
>
> Change-Id: I84e26d0e8f69c8e5bc1c65925792aea296f6a97d
> ---
>  gnu/packages/freedesktop.scm | 31 +++++++++++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
>
> diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
> index a7c740f3d05..321ab2a1d1e 100644
> --- a/gnu/packages/freedesktop.scm
> +++ b/gnu/packages/freedesktop.scm
> @@ -1932,6 +1932,37 @@ (define-public libmbim
>       ;; The mbimcli tool is released under the GPLv2+ license.
>       (list license:lgpl2.0+ license:gpl2+))))
>  
> +(define-public libqrtr-glib
> +  (package
> +    (name "libqrtr-glib")
> +    (version "1.2.2")
> +    (source (origin
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://gitlab.freedesktop.org/mobile-broadband/libqrtr-glib")

Too wide of a line; you should break (source (origin ... as
 (source
  (origin ...))

Note that 'guix style' has issues in this regard, which I've reported to
our tracker.

> +                    (commit version)))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32
> +                "0bfn5kfscli0rrxvmzdl6ix5ikh0kxia7ad83rmh1hksfcwynwlh"))))
> +    (build-system meson-build-system)
> +    (inputs
> +     (list libgudev libmbim))
> +    (native-inputs
> +     (list `(,glib "bin")
> +           gtk-doc
> +           pkg-config
> +           gobject-introspection
> +           bash-completion))

My package only had (native-inputs (list gobject-introspection gtk-doc pkg-config))

> +    (propagated-inputs
> +     (list glib))

When propagating inputs, we typically leave a comment as to why, as it
shouldn't be made without a good reason (it hurts composability in a
profile).  Typically something like a

 ;required by some-package.pc  

inline comment when pkg-config files list them in Requires or
Requires.private (the later is consulted when retrieving Cflags with
pkg-config... even when using dynamic linking).

In my package I had listed it as an 'input'; I hadn't checked for .pc
files, but didn't have problems building modem-manager with it.

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH 3/4] gnu: libqmi: Update to 1.34.0.
  2024-10-24  7:40 ` [bug#73981] [PATCH 3/4] gnu: libqmi: Update to 1.34.0 Zheng Junjie
@ 2024-11-12  1:06   ` Maxim Cournoyer
  0 siblings, 0 replies; 15+ messages in thread
From: Maxim Cournoyer @ 2024-11-12  1:06 UTC (permalink / raw)
  To: Zheng Junjie; +Cc: 73981

Zheng Junjie <zhengjunjie@iscas.ac.cn> writes:

> * gnu/packages/freedesktop.scm (libqmi): Update to 1.34.0.
> [inputs]: Add libmbim and libqrtr-glib.

These should go to the propagated-inputs as they are required by
qmi-glib.pc, with an explanatory comment like

--8<---------------cut here---------------start------------->8---
;; These are required by qmi-glib.pc.
--8<---------------cut here---------------end--------------->8---

> [native-inputs]: Add gobject-introspection, bash-completion and
> help2man. Replace python-wrapper with python-minimal.

Not sure about bash-completion.  I'll research if it should be a native
input or a regular one.

Otherwise LGTM.

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH 4/4] gnu: modem-manager: Update to 1.22.0.
  2024-10-24  7:40 ` [bug#73981] [PATCH 4/4] gnu: modem-manager: Update to 1.22.0 Zheng Junjie
@ 2024-11-12  1:13   ` Maxim Cournoyer
  0 siblings, 0 replies; 15+ messages in thread
From: Maxim Cournoyer @ 2024-11-12  1:13 UTC (permalink / raw)
  To: Zheng Junjie; +Cc: 73981

Hi,

Zheng Junjie <zhengjunjie@iscas.ac.cn> writes:

> * gnu/packages/freedesktop.scm (modem-manager): Update to 1.22.0.
> [source]: Switch to git-fetch.
> [build-system]: Switch to meson-build-system.
> [arguments]<#:configure-flags>: Adjust udevdir. Set -Dsystemdsystemunitdir=no.
> [inputs]: Add libqrtr-glib and elogind.
> [native-inputs]: Remove python; add python-minimal, libxslt, and
> bash-completion.
>
> Change-Id: Ife0560eb48f3cceaf22ccc6bb54010074028cddf
> ---
>  gnu/packages/freedesktop.scm | 30 +++++++++++++++++-------------
>  1 file changed, 17 insertions(+), 13 deletions(-)
>
> diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
> index ccba510303d..6eec91a907f 100644
> --- a/gnu/packages/freedesktop.scm
> +++ b/gnu/packages/freedesktop.scm
> @@ -2000,34 +2000,38 @@ (define-public libqmi
>  (define-public modem-manager
>    (package
>      (name "modem-manager")
> -    (version "1.18.12")
> +    (version "1.22.0")
>      (source (origin
> -              (method url-fetch)
> -              (uri (string-append
> -                    "https://www.freedesktop.org/software/ModemManager/"
> -                    "ModemManager-" version ".tar.xz"))
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://gitlab.freedesktop.org/mobile-broadband/ModemManager")

Long line; please indent origin under source to help cope with that.

> +                    (commit version)))
> +              (file-name (git-file-name name version))
>                (sha256
>                 (base32
> -                "0c74n5jl1qvq2qlbwzfkgxny8smjcgkid1nhdnl6qnlmbn9f8r5l"))))
> -    (build-system gnu-build-system)
> +                "0fj4ibjfsxal3xfk3hrj4l9vg7zbj42k9lj7151illl2n3d5ngzw"))))
> +    (build-system meson-build-system)
>      (arguments
>       (list
>        #:configure-flags
> -      #~(list (string-append "--with-udev-base-dir=" #$output "/lib/udev"))))
> +      #~(list (string-append "-Dudevdir=" #$output "/lib/udev")

Is it /lib/udev or /lib/udev/rules.d ?  I had used the later.  It seems
you are correct, after checking the file hierarchy of my package:

--8<---------------cut here---------------start------------->8---
gnu/store/s9jad478d7ks5cybygiibgyrgywi7dhn-modem-manager-1.22.0/lib/udev/rules.d
/gnu/store/s9jad478d7ks5cybygiibgyrgywi7dhn-modem-manager-1.22.0/lib/udev/rules.d/rules.d
/gnu/store/s9jad478d7ks5cybygiibgyrgywi7dhn-modem-manager-1.22.0/lib/udev/rules.d/rules.d/77-mm-broadmobi-port-types.rules
--8<---------------cut here---------------end--------------->8---

> +              "-Dsystemdsystemunitdir=no")))
>      (native-inputs
> -     (list dbus
> +     (list bash-completion
> +           dbus
>             gettext-minimal
>             gobject-introspection
>             `(,glib "bin")               ;for glib-mkenums
> +           libxslt                      ; for xsltproc

nitpick: No space between ';' and the text for inline comments.

>             pkg-config
> -           python
> -           python-dbus
> -           python-pygobject
> +           python-minimal
> +           python-dbus                  ; for test
> +           python-pygobject             ; for test

Please keep inputs sorted, and ditto about the space in inline comment.

>             vala))

vala bindingsn are no longer generated by default, so you'll want to add
-Dvapi=true to the configure flags.

>      (propagated-inputs
>       (list glib))                       ;required by mm-glib.pc
>      (inputs
> -     (list libgudev libmbim libqmi polkit))
> +     (list elogind libgudev libmbim libqmi libqrtr-glib polkit))

I didn't list libqrtr-glib here since it should be propagated by libqmi.

Otherwise, LGTM.  Feel free to update your series with my suggestions
and I'll rebase my branch after merging it.  Sorry about the duplicated
work; at least it made reviewing it trivial :-).

-- 
Thanks,
Maxim




^ permalink raw reply	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH v2 0/4] update modem-manager and deps
  2024-10-24  7:36 [bug#73981] [PATCH 0/4] gnu: update modem-manager and deps Zheng Junjie
                   ` (3 preceding siblings ...)
  2024-10-24  7:40 ` [bug#73981] [PATCH 4/4] gnu: modem-manager: Update to 1.22.0 Zheng Junjie
@ 2024-11-12  3:46 ` Zheng Junjie
  2024-11-12  3:46   ` [bug#73981] [PATCH v2 1/4] gnu: libmbim: Update to 1.30.0 Zheng Junjie
                     ` (4 more replies)
  4 siblings, 5 replies; 15+ messages in thread
From: Zheng Junjie @ 2024-11-12  3:46 UTC (permalink / raw)
  To: 73981

Zheng Junjie (4):
  gnu: libmbim: Update to 1.30.0.
  gnu: Add libqrtr-glib.
  gnu: libqmi: Update to 1.34.0.
  gnu: modem-manager: Update to 1.22.0.

 gnu/packages/freedesktop.scm | 148 +++++++++++++++++++++++++----------
 1 file changed, 105 insertions(+), 43 deletions(-)


base-commit: b790db7589858fc77989b4d1f369c52bca6d6e7c
prerequisite-patch-id: f66cd2a9b573a4b884f66abe5e92465d04e1ebd6
prerequisite-patch-id: 148cdb72f12b8fae6e0b3cc2f9f691186c959593
-- 
2.46.0





^ permalink raw reply	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH v2 1/4] gnu: libmbim: Update to 1.30.0.
  2024-11-12  3:46 ` [bug#73981] [PATCH v2 0/4] update modem-manager and deps Zheng Junjie
@ 2024-11-12  3:46   ` Zheng Junjie
  2024-11-12  3:46   ` [bug#73981] [PATCH v2 2/4] gnu: Add libqrtr-glib Zheng Junjie
                     ` (3 subsequent siblings)
  4 siblings, 0 replies; 15+ messages in thread
From: Zheng Junjie @ 2024-11-12  3:46 UTC (permalink / raw)
  To: 73981

* gnu/packages/freedesktop.scm (libmbim): Update to 1.30.0.
[source]: Switch to git-fetch.
[build-system]: Switch to meson-build-system.
[native-inputs]: Add gobject-introspection, help2man. Replace
python-wrapper with python-minimal.
[inputs]: Add bash-completion.

Change-Id: I607bff667a057e58bebb2f706d7021a4d1c8d763
---
 gnu/packages/freedesktop.scm | 35 +++++++++++++++++++++--------------
 1 file changed, 21 insertions(+), 14 deletions(-)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index bd34615e682..7441737a4bd 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -1901,23 +1901,30 @@ (define-public accountsservice
 (define-public libmbim
   (package
     (name "libmbim")
-    (version "1.26.4")
-    (source (origin
-              (method url-fetch)
-              (uri (string-append
-                    "https://www.freedesktop.org/software/libmbim/"
-                    "libmbim-" version ".tar.xz"))
-              (sha256
-               (base32
-                "1ncaarl4lgc7i52rwz50yq701wk2rr478cjybxbifsjqqk2cx27n"))))
-    (build-system gnu-build-system)
+    (version "1.30.0")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://gitlab.freedesktop.org/mobile-broadband/libmbim")
+             (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32
+         "00kbjvpka51zrfjigzd3rk6r4x8hkg1xfj7d9zl9lccysnxyjx5h"))))
+    (build-system meson-build-system)
     (native-inputs
-     (list `(,glib "bin") ; for glib-mkenums
-           pkg-config python-wrapper))
+     (list `(,glib "bin")               ;for glib-mkenums
+           gobject-introspection
+           help2man
+           pkg-config
+           python-minimal))
     (propagated-inputs
-     (list glib)) ; required by mbim-glib.pc
+     (list glib))                       ;required by mbim-glib.pc
     (inputs
-     (list libgudev))
+     (list
+      bash-completion
+      libgudev))
     (synopsis "Library to communicate with MBIM-powered modems")
     (home-page "https://www.freedesktop.org/wiki/Software/libmbim/")
     (description
-- 
2.46.0





^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH v2 2/4] gnu: Add libqrtr-glib.
  2024-11-12  3:46 ` [bug#73981] [PATCH v2 0/4] update modem-manager and deps Zheng Junjie
  2024-11-12  3:46   ` [bug#73981] [PATCH v2 1/4] gnu: libmbim: Update to 1.30.0 Zheng Junjie
@ 2024-11-12  3:46   ` Zheng Junjie
  2024-11-12  3:46   ` [bug#73981] [PATCH v2 3/4] gnu: libqmi: Update to 1.34.0 Zheng Junjie
                     ` (2 subsequent siblings)
  4 siblings, 0 replies; 15+ messages in thread
From: Zheng Junjie @ 2024-11-12  3:46 UTC (permalink / raw)
  To: 73981

* gnu/packages/freedesktop.scm (libqrtr-glib): New variable.

Change-Id: I84e26d0e8f69c8e5bc1c65925792aea296f6a97d
---
 gnu/packages/freedesktop.scm | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index 7441737a4bd..1a192f0a806 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -1935,6 +1935,39 @@ (define-public libmbim
      ;; The mbimcli tool is released under the GPLv2+ license.
      (list license:lgpl2.0+ license:gpl2+))))
 
+(define-public libqrtr-glib
+  (package
+    (name "libqrtr-glib")
+    (version "1.2.2")
+    (source
+     (origin
+       (method git-fetch)
+       (uri
+        (git-reference
+         (url "https://gitlab.freedesktop.org/mobile-broadband/libqrtr-glib")
+         (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32
+         "0bfn5kfscli0rrxvmzdl6ix5ikh0kxia7ad83rmh1hksfcwynwlh"))))
+    (build-system meson-build-system)
+    (inputs
+     (list libgudev libmbim))
+    (native-inputs
+     (list bash-completion
+           `(,glib "bin")
+           gtk-doc
+           gobject-introspection
+           pkg-config))
+    (propagated-inputs
+     (list glib))                       ;required by mm-glib.pc
+    (synopsis "Qualcomm IPC Router protocol helper library")
+    (description
+     "libqrtr-glib is a glib-based library to use and manage the QRTR (Qualcomm
+IPC Router) bus.")
+    (home-page "https://gitlab.freedesktop.org/mobile-broadband/libqrtr-glib")
+    (license license:lgpl2.1+)))
+
 (define-public libqmi
   (package
     (name "libqmi")
-- 
2.46.0





^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH v2 3/4] gnu: libqmi: Update to 1.34.0.
  2024-11-12  3:46 ` [bug#73981] [PATCH v2 0/4] update modem-manager and deps Zheng Junjie
  2024-11-12  3:46   ` [bug#73981] [PATCH v2 1/4] gnu: libmbim: Update to 1.30.0 Zheng Junjie
  2024-11-12  3:46   ` [bug#73981] [PATCH v2 2/4] gnu: Add libqrtr-glib Zheng Junjie
@ 2024-11-12  3:46   ` Zheng Junjie
  2024-11-12  3:46   ` [bug#73981] [PATCH v2 4/4] gnu: modem-manager: Update to 1.22.0 Zheng Junjie
  2024-11-15 10:54   ` bug#73981: [PATCH v2 0/4] update modem-manager and deps Z572
  4 siblings, 0 replies; 15+ messages in thread
From: Zheng Junjie @ 2024-11-12  3:46 UTC (permalink / raw)
  To: 73981

* gnu/packages/freedesktop.scm (libqmi): Update to 1.34.0.
[inputs]: Add bash-completion.
[propagated-inputs]: Add libmbim and libqrtr-glib.
[native-inputs]: Add gobject-introspection and help2man. Replace
python-wrapper with python-minimal.
[build-system]: Switch to meson-build-system.
[source]: Swich to git-fetch.

Change-Id: I8a023cc07e5484d8abd5df4615ad5f4115729c1a
---
 gnu/packages/freedesktop.scm | 38 +++++++++++++++++++++++-------------
 1 file changed, 24 insertions(+), 14 deletions(-)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index 1a192f0a806..a3dc92108ec 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -1971,23 +1971,33 @@ (define-public libqrtr-glib
 (define-public libqmi
   (package
     (name "libqmi")
-    (version "1.30.8")
-    (source (origin
-              (method url-fetch)
-              (uri (string-append
-                    "https://www.freedesktop.org/software/libqmi/"
-                    "libqmi-" version ".tar.xz"))
-              (sha256
-               (base32
-                "140rmjw436rh6rqmnfw6yaflpffd27ilwcv4s9jvvl1skv784946"))))
-    (build-system gnu-build-system)
+    (version "1.34.0")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://gitlab.freedesktop.org/mobile-broadband/libqmi")
+             (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32
+         "1m5y2sf14qd2i9mvbb68wxqlfwvpiprgz8zmcx6wb2cnjgsszmwp"))))
+    (build-system meson-build-system)
     (inputs
-     (list libgudev))
+     (list
+      bash-completion
+      libgudev))
     (native-inputs
-     (list `(,glib "bin") ; for glib-mkenums
-           pkg-config python-wrapper))
+     (list `(,glib "bin")               ;for glib-mkenums
+           gobject-introspection
+           help2man
+           pkg-config
+           python-minimal))
+    ;; These are required by qmi-glib.pc.
     (propagated-inputs
-     (list glib)) ; required by qmi-glib.pc
+     (list glib
+           libmbim
+           libqrtr-glib))
     (synopsis "Library to communicate with QMI-powered modems")
     (home-page "https://www.freedesktop.org/wiki/Software/libqmi/")
     (description
-- 
2.46.0





^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [bug#73981] [PATCH v2 4/4] gnu: modem-manager: Update to 1.22.0.
  2024-11-12  3:46 ` [bug#73981] [PATCH v2 0/4] update modem-manager and deps Zheng Junjie
                     ` (2 preceding siblings ...)
  2024-11-12  3:46   ` [bug#73981] [PATCH v2 3/4] gnu: libqmi: Update to 1.34.0 Zheng Junjie
@ 2024-11-12  3:46   ` Zheng Junjie
  2024-11-15 10:54   ` bug#73981: [PATCH v2 0/4] update modem-manager and deps Z572
  4 siblings, 0 replies; 15+ messages in thread
From: Zheng Junjie @ 2024-11-12  3:46 UTC (permalink / raw)
  To: 73981

* gnu/packages/freedesktop.scm (modem-manager): Update to 1.22.0.
[source]: Switch to git-fetch.
[build-system]: Switch to meson-build-system.
[arguments]<#:configure-flags>: Adjust udevdir. Set -Dsystemdsystemunitdir=no,
Set -Dvapi=true.
[inputs]: Add bash-completion, libqrtr-glib and elogind.
[native-inputs]: Remove python; add python-minimal and libxslt.

Change-Id: Ife0560eb48f3cceaf22ccc6bb54010074028cddf
---
 gnu/packages/freedesktop.scm | 42 +++++++++++++++++++++++-------------
 1 file changed, 27 insertions(+), 15 deletions(-)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index a3dc92108ec..af35625c18f 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -2011,34 +2011,46 @@ (define-public libqmi
 (define-public modem-manager
   (package
     (name "modem-manager")
-    (version "1.18.12")
-    (source (origin
-              (method url-fetch)
-              (uri (string-append
-                    "https://www.freedesktop.org/software/ModemManager/"
-                    "ModemManager-" version ".tar.xz"))
-              (sha256
-               (base32
-                "0c74n5jl1qvq2qlbwzfkgxny8smjcgkid1nhdnl6qnlmbn9f8r5l"))))
-    (build-system gnu-build-system)
+    (version "1.22.0")
+    (source
+     (origin
+       (method git-fetch)
+       (uri
+        (git-reference
+         (url "https://gitlab.freedesktop.org/mobile-broadband/ModemManager")
+         (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32
+         "0fj4ibjfsxal3xfk3hrj4l9vg7zbj42k9lj7151illl2n3d5ngzw"))))
+    (build-system meson-build-system)
     (arguments
      (list
       #:configure-flags
-      #~(list (string-append "--with-udev-base-dir=" #$output "/lib/udev"))))
+      #~(list (string-append "-Dudevdir=" #$output "/lib/udev")
+              "-Dsystemdsystemunitdir=no"
+              "-Dvapi=true")))
     (native-inputs
      (list dbus
            gettext-minimal
            gobject-introspection
            `(,glib "bin")               ;for glib-mkenums
+           libxslt                      ;for xsltproc
            pkg-config
-           python
-           python-dbus
-           python-pygobject
+           python-minimal
+           python-dbus                  ;for test
+           python-pygobject             ;for test
            vala))
     (propagated-inputs
      (list glib))                       ;required by mm-glib.pc
     (inputs
-     (list libgudev libmbim libqmi polkit))
+     (list bash-completion
+           elogind
+           libgudev
+           libmbim
+           libqmi
+           libqrtr-glib
+           polkit))
     (synopsis "Mobile broadband modems manager")
     (home-page "https://www.freedesktop.org/wiki/Software/ModemManager/")
     (description
-- 
2.46.0





^ permalink raw reply related	[flat|nested] 15+ messages in thread

* bug#73981: [PATCH v2 0/4] update modem-manager and deps
  2024-11-12  3:46 ` [bug#73981] [PATCH v2 0/4] update modem-manager and deps Zheng Junjie
                     ` (3 preceding siblings ...)
  2024-11-12  3:46   ` [bug#73981] [PATCH v2 4/4] gnu: modem-manager: Update to 1.22.0 Zheng Junjie
@ 2024-11-15 10:54   ` Z572
  4 siblings, 0 replies; 15+ messages in thread
From: Z572 @ 2024-11-15 10:54 UTC (permalink / raw)
  To: 73981-done

[-- Attachment #1: Type: text/plain, Size: 547 bytes --]

Zheng Junjie <zhengjunjie@iscas.ac.cn> writes:

> Zheng Junjie (4):
>   gnu: libmbim: Update to 1.30.0.
>   gnu: Add libqrtr-glib.
>   gnu: libqmi: Update to 1.34.0.
>   gnu: modem-manager: Update to 1.22.0.
>
>  gnu/packages/freedesktop.scm | 148 +++++++++++++++++++++++++----------
>  1 file changed, 105 insertions(+), 43 deletions(-)
>
>
> base-commit: b790db7589858fc77989b4d1f369c52bca6d6e7c
> prerequisite-patch-id: f66cd2a9b573a4b884f66abe5e92465d04e1ebd6
> prerequisite-patch-id: 148cdb72f12b8fae6e0b3cc2f9f691186c959593
pushed, closing.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2024-11-15 10:57 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-24  7:36 [bug#73981] [PATCH 0/4] gnu: update modem-manager and deps Zheng Junjie
2024-10-24  7:40 ` [bug#73981] [PATCH 1/4] gnu: libmbim: Update to 1.30.0 Zheng Junjie
2024-11-11 14:22   ` Maxim Cournoyer
2024-10-24  7:40 ` [bug#73981] [PATCH 2/4] gnu: Add libqrtr-glib Zheng Junjie
2024-11-11 14:28   ` Maxim Cournoyer
2024-10-24  7:40 ` [bug#73981] [PATCH 3/4] gnu: libqmi: Update to 1.34.0 Zheng Junjie
2024-11-12  1:06   ` Maxim Cournoyer
2024-10-24  7:40 ` [bug#73981] [PATCH 4/4] gnu: modem-manager: Update to 1.22.0 Zheng Junjie
2024-11-12  1:13   ` Maxim Cournoyer
2024-11-12  3:46 ` [bug#73981] [PATCH v2 0/4] update modem-manager and deps Zheng Junjie
2024-11-12  3:46   ` [bug#73981] [PATCH v2 1/4] gnu: libmbim: Update to 1.30.0 Zheng Junjie
2024-11-12  3:46   ` [bug#73981] [PATCH v2 2/4] gnu: Add libqrtr-glib Zheng Junjie
2024-11-12  3:46   ` [bug#73981] [PATCH v2 3/4] gnu: libqmi: Update to 1.34.0 Zheng Junjie
2024-11-12  3:46   ` [bug#73981] [PATCH v2 4/4] gnu: modem-manager: Update to 1.22.0 Zheng Junjie
2024-11-15 10:54   ` bug#73981: [PATCH v2 0/4] update modem-manager and deps Z572

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).