Re: [PATCH, WIP, v2] new variable: redshift

[Ludovic Courtès]

Florian Paul Schmidt <address@hidden> skribis:

> I wanted to try the --rounds=3 option, but since it was already built I
> was at a loss about how to make it run a few rounds..

‘--check’ can be used in that case.  I did --rounds=3 though and it
passes.  :-)

> From f35546e7679e39f5f9cd790c1a9cb27e7b7825b1 Mon Sep 17 00:00:00 2001
> From: Florian Paul Schmidt <address@hidden>
> Date: Thu, 10 Dec 2015 08:57:57 +0100
> Subject: [PATCH] gnu: Add redshift
>
> * gnu/packages/xdisorg.scm (redshift): New variable.

[...]

> +       ("geoclue" ,geoclue)))

This doesn’t work; instead one has to pass GLib to get Geoclue2 support,
which is what I did.

However, I found out that Geoclue2 seems to be broken on GuixSD:

--8<---------------cut here---------------start------------->8---
Dec 14 00:28:39 localhost dbus[435]: [system] Activating service 
name='org.freedesktop.GeoClue2' (using servicehelper)
Dec 14 00:28:39 localhost dbus[435]: [system] Successfully activated service 
'org.freedesktop.GeoClue2'
Dec 14 00:28:39 localhost dbus[435]: [system] Rejected send message, 1 matched 
rules; type="method_call", sender=":1.74" (uid=993 pid=24166 
comm="/gnu/store/sidpj3m8aarbvhflzvhgmp43gkmcx55z-geoclu") 
interface="org.freedesktop.DBus.Properties" member="GetAll" error 
name="(unset)" requested_reply="0" destination="org.freedesktop.DBus" (bus)
--8<---------------cut here---------------end--------------->8---

Ideas?

In the meantime, one can work around it by using:

  redshift -l LATITUDE:LONGITUDE

> +    (synopsis
> +     "Adjusts your screen's color temperature
> +according to your surroundings")

I’ve made this one line and slightly shorter…

> +    (description
> +     "Redshift adjusts the color temperature according to the
> +position of the sun.  A different color temperature is set during night and
> +daytime.  During twilight and early morning, the color temperature 
> transitions
> +smoothly from night to daytime temperature to allow your eyes to slowly
> +adapt.  At night the color temperature should be set to match the lamps in 
> your
> +room.")

… and reflowed this paragraph.

I also added a copyright line for you, and pushed.

Thanks!

Ludo’.