unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#59948] [PATCH] Add iio-sensor-proxy
@ 2022-12-10 21:26 Florian
  2022-12-11 16:16 ` [bug#59948] [PATCH 1/2] gnu: Add libgudev-next florhizome
  0 siblings, 1 reply; 5+ messages in thread
From: Florian @ 2022-12-10 21:26 UTC (permalink / raw)
  To: 59948





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

* [bug#59948] [PATCH 1/2] gnu: Add libgudev-next
  2022-12-10 21:26 [bug#59948] [PATCH] Add iio-sensor-proxy Florian
@ 2022-12-11 16:16 ` florhizome
  2022-12-11 16:16   ` [bug#59948] [PATCH 2/2] gnu: Add iio-sensor-proxy florhizome
  2022-12-11 16:38   ` [bug#59948] [PATCH 1/2] gnu: Add libgudev-next ( via Guix-patches via
  0 siblings, 2 replies; 5+ messages in thread
From: florhizome @ 2022-12-11 16:16 UTC (permalink / raw)
  To: 59948; +Cc: florhizome

From: florhizome <florhizome@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





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

* [bug#59948] [PATCH 2/2] gnu: Add iio-sensor-proxy
  2022-12-11 16:16 ` [bug#59948] [PATCH 1/2] gnu: Add libgudev-next florhizome
@ 2022-12-11 16:16   ` florhizome
  2022-12-11 16:43     ` ( via Guix-patches via
  2022-12-11 16:38   ` [bug#59948] [PATCH 1/2] gnu: Add libgudev-next ( via Guix-patches via
  1 sibling, 1 reply; 5+ messages in thread
From: florhizome @ 2022-12-11 16:16 UTC (permalink / raw)
  To: 59948; +Cc: florhizome

From: florhizome <florhizome@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@ruhr-uni-bochum.de>
 ;;; Copyright © 2022 muradm <mail@muradm.net>
 ;;; Copyright © 2022 Petr Hodina <phodina@protonmail.com>
+;;; Copyright © 2022 florhizome <florhizome@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





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

* [bug#59948] [PATCH 1/2] gnu: Add libgudev-next
  2022-12-11 16:16 ` [bug#59948] [PATCH 1/2] gnu: Add libgudev-next florhizome
  2022-12-11 16:16   ` [bug#59948] [PATCH 2/2] gnu: Add iio-sensor-proxy florhizome
@ 2022-12-11 16:38   ` ( via Guix-patches via
  1 sibling, 0 replies; 5+ messages in thread
From: ( via Guix-patches via @ 2022-12-11 16:38 UTC (permalink / raw)
  To: florhizome, 59948

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

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.

    -- (

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

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

* [bug#59948] [PATCH 2/2] gnu: Add iio-sensor-proxy
  2022-12-11 16:16   ` [bug#59948] [PATCH 2/2] gnu: Add iio-sensor-proxy florhizome
@ 2022-12-11 16:43     ` ( via Guix-patches via
  0 siblings, 0 replies; 5+ messages in thread
From: ( via Guix-patches via @ 2022-12-11 16:43 UTC (permalink / raw)
  To: florhizome, 59948

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

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.

    -- (

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

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

end of thread, other threads:[~2022-12-11 16:44 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-12-10 21:26 [bug#59948] [PATCH] Add iio-sensor-proxy Florian
2022-12-11 16:16 ` [bug#59948] [PATCH 1/2] gnu: Add libgudev-next florhizome
2022-12-11 16:16   ` [bug#59948] [PATCH 2/2] gnu: Add iio-sensor-proxy florhizome
2022-12-11 16:43     ` ( via Guix-patches via
2022-12-11 16:38   ` [bug#59948] [PATCH 1/2] gnu: Add libgudev-next ( via Guix-patches via

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