Subject: bug#34246: closed (Re: [bug#34246] [PATCH 0/3] Improve the upower-service style.)

From: [email-masked] (GNU bug Tracking System)
Date: Sat, 16 Feb 2019 22:16:02 +0000

Your bug report

#34246: [PATCH 0/3] Improve the upower-service style.

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
34246: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=34246
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: [bug#34246] [PATCH 0/3] Improve the upower-service style.
From: Christopher Baines <[email-masked]>
To: Ludovic Courtès <[email-masked]>
Cc: [email-masked]
Date: Sat, 16 Feb 2019 22:15:43 +0000
List-Id: <debbugs-submit.debbugs.gnu.org>

Ludovic Courtès <[email-masked]> writes:

> Hello!
>
> Christopher Baines <[email-masked]> skribis:
>
>> These patches update the style of the upower-service. Improving and
>> documenting the <upower-configuration> record, and deprecating the
>> upower-service procedure.
>>
>> Directly using record types as default values for service types, along
>> with default values for the fields in the record type is generally more
>> flexible and configurable than using procedures for service
>> configuration. It means that the configuration for the service can be
>> changed programatically by generating new configuration based off of the
>> original configuration, rather than having to rerun the procedure that
>> created the configuration in the first place.
>>
>>
>> Christopher Baines (3):
>>   services: Improve the upower-configuration record.
>>   services: Improve the upower-service-type.
>>   services: desktop: Deprecate the upower-service procedure.
>
> All three patches LGTM.  Thanks for taking the time to do this!  :-)

Great, I've pushed these patches to master now :)

Subject: [PATCH 0/3] Improve the upower-service style.
From: Christopher Baines <[email-masked]>
To: [email-masked]
Date: Tue, 29 Jan 2019 15:28:48 +0000
List-Id: <debbugs-submit.debbugs.gnu.org>

These patches update the style of the upower-service. Improving and
documenting the <upower-configuration> record, and deprecating the
upower-service procedure.

Directly using record types as default values for service types, along
with default values for the fields in the record type is generally more
flexible and configurable than using procedures for service
configuration. It means that the configuration for the service can be
changed programatically by generating new configuration based off of the
original configuration, rather than having to rerun the procedure that
created the configuration in the first place.

Christopher Baines (3):
  services: Improve the upower-configuration record.
  services: Improve the upower-service-type.
  services: desktop: Deprecate the upower-service procedure.

 doc/guix.texi            | 79 ++++++++++++++++++++++++++--------
 gnu/services/desktop.scm | 92 +++++++++++++++++++++++++++-------------
 2 files changed, 123 insertions(+), 48 deletions(-)