GNU bug report logs - #59948
[PATCH] Add iio-sensor-proxy

Previous Next

To reply to this bug, email your comments to 59948 AT debbugs.gnu.org.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Florian <florhizome <at> posteo.net>
To: guix-patches <at> gnu.org
Subject: [PATCH] Add iio-sensor-proxy
Date: Sat, 10 Dec 2022 21:26:29 +0000

Message #8 received at 59948 <at> debbugs.gnu.org (full text, mbox):

From: florhizome <at> posteo.net
To: 59948 <at> debbugs.gnu.org
Cc: florhizome <florhizome <at> posteo.net>
Subject: [PATCH 1/2] gnu: Add libgudev-next
Date: Sun, 11 Dec 2022 16:16:09 +0000

From: florhizome <florhizome <at> posteo.net>

* gnu/packages/gnome.scm (libgudev-next): New variable.
* gnu/packages/freedesktop.scm (iio-sensor-proxy): New variable.
---
 gnu/packages/gnome.scm | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index 047c077d7d..43d1f5acc7 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -7070,6 +7070,19 @@ (define-public libgudev
 part of udev-extras, then udev, then systemd.  It's now a project on its own.")
     (license license:lgpl2.1+)))
 
+(define-public libgudev-next
+  (package
+    (inherit libgudev)
+    (name "libgudev-next")
+    (version "237")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "mirror://gnome/sources/" name "/"
+                                  version "/" name "-" version ".tar.xz"))
+              (sha256
+               (base32
+                "1al6nr492nzbm8ql02xhzwci2kwb1advnkaky3j9636jf08v41hd"))))))
+
 (define-public gvfs
   (package
     (name "gvfs")
-- 
2.38.1

Message #11 received at 59948 <at> debbugs.gnu.org (full text, mbox):

From: florhizome <at> posteo.net
To: 59948 <at> debbugs.gnu.org
Cc: florhizome <florhizome <at> posteo.net>
Subject: [PATCH 2/2] gnu: Add iio-sensor-proxy
Date: Sun, 11 Dec 2022 16:16:10 +0000

From: florhizome <florhizome <at> posteo.net>

---
 gnu/packages/freedesktop.scm | 51 ++++++++++++++++++++++++++++++++++++
 1 file changed, 51 insertions(+)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index 28d0c0722f..d3d66d5209 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -31,6 +31,7 @@
 ;;; Copyright © 2022 Daniel Meißner <daniel.meissner-i4k <at> ruhr-uni-bochum.de>
 ;;; Copyright © 2022 muradm <mail <at> muradm.net>
 ;;; Copyright © 2022 Petr Hodina <phodina <at> protonmail.com>
+;;; Copyright © 2022 florhizome <florhizome <at> posteo.net>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -810,6 +811,56 @@ (define-public elogind
 of a the system to know what users are logged in, and where.")
     (license license:lgpl2.1+)))
 
+(define-public iio-sensor-proxy
+ (package
+    (name "iio-sensor-proxy")
+    (version "3.4")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://gitlab.freedesktop.org/hadess/iio-sensor-proxy")
+             (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32
+         "0zf9kjhngs8k8vpwfiyra60vp4zic6gzs7axw1cd736my9pyhniv"))))
+    (build-system meson-build-system)
+    (arguments
+     (list #:configure-flags
+           #~(list "-Dsystemdsystemunitdir=false"
+                   (string-append "-Dudevrulesdir="
+                                  #$output "/lib/udev"))
+           #:glib-or-gtk? #t
+           #:phases
+           #~(modify-phases %standard-phases
+               (add-before 'configure 'fake-pkexec
+                 (lambda _
+                   (setenv "PKEXEC_UID" "-1")))
+               (add-before 'configure 'correct-polkit-dir
+                 (lambda _
+                   (substitute* "meson.build"
+                     (("polkit_gobject_dep\\..*")
+                      (string-append "'" #$output "/share/polkit-1/actions'"))))))))
+    (native-inputs
+     (list dbus
+           (list glib "bin")
+           gobject-introspection
+           python
+           python-dbusmock
+           python-psutil
+           pkg-config
+           umockdev))
+    (inputs
+     (list glib libgudev-next polkit))
+    (home-page "https://gitlab.freedesktop.org/hadess/iio-sensor-proxy")
+    (synopsis "Proxies sensor devices to applications through D-Bus")
+    (description "With a GNOME 3.18 (or newer) based system, orientation changes
+ will automatically be applied when rotating the panel, ambient light will be
+ used to change the screen brightness, and GeoClue will be able to read the compass
+data to show the direction in Maps.")
+    (license license:gpl3)))
+
 (define-public basu
   (package
     (name "basu")
-- 
2.38.1

Message #14 received at 59948 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: <florhizome <at> posteo.net>, <59948 <at> debbugs.gnu.org>
Subject: Re: [bug#59948] [PATCH 1/2] gnu: Add libgudev-next
Date: Sun, 11 Dec 2022 16:38:19 +0000

[Message part 1 (text/plain, inline)]

Heya,

On Sun Dec 11, 2022 at 4:16 PM GMT,  wrote:
> * gnu/packages/gnome.scm (libgudev-next): New variable.
> * gnu/packages/freedesktop.scm (iio-sensor-proxy): New variable.

Remove the second line from the commit messaage ;)

Otherwise LGTM.

    -- (

[signature.asc (application/pgp-signature, inline)]

Message #17 received at 59948 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: <florhizome <at> posteo.net>, <59948 <at> debbugs.gnu.org>
Subject: Re: [bug#59948] [PATCH 2/2] gnu: Add iio-sensor-proxy
Date: Sun, 11 Dec 2022 16:43:18 +0000

[Message part 1 (text/plain, inline)]

The (currently empty) commit message should have the removed line from the
previous patch.

On Sun Dec 11, 2022 at 4:16 PM GMT,  wrote:
> --- a/gnu/packages/freedesktop.scm
> +++ b/gnu/packages/freedesktop.scm

> @@ -810,6 +811,56 @@ (define-public elogind

> +(define-public iio-sensor-proxy
> + (package
> +    (name "iio-sensor-proxy")
    ^^^
Too much indentation ;) Two spaces are enough.

> +           #:phases
> +           #~(modify-phases %standard-phases
> +               (add-before 'configure 'fake-pkexec
> +                 (lambda _
> +                   (setenv "PKEXEC_UID" "-1")))

Could you add a comment explaining why this is necessary?

> +    (native-inputs
> +     (list dbus
> +           (list glib "bin")
> +           gobject-introspection
> +           python
> +           python-dbusmock
> +           python-psutil
> +           pkg-config
> +           umockdev))

Please sort these alphabetically by putting PKG-CONFIG before PYTHON.

> +    (inputs
> +     (list glib libgudev-next polkit))

INPUTS should typically come before NATIVE-INPUTS, imo.

> +    (description "With a GNOME 3.18 (or newer) based system, orientation changes
> + will automatically be applied when rotating the panel, ambient light will be
> + used to change the screen brightness, and GeoClue will be able to read the compass
> +data to show the direction in Maps.")

This isn't a full sentence.  Please write a description explaining exactly
what it does.

    -- (

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.