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

[GNU bug Tracking System]

Your message dated Sat, 16 Feb 2019 22:15:43 +0000
with message-id <address@hidden>
and subject line Re: [bug#34246] [PATCH 0/3] Improve the upower-service style.
has caused the debbugs.gnu.org bug report #34246,
regarding [PATCH 0/3] Improve the upower-service style.
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
34246: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=34246
GNU Bug Tracking System
Contact address@hidden with problems

--- Begin Message --- Subject: [PATCH 0/3] Improve the upower-service style. Date: Tue, 29 Jan 2019 15:28:48 +0000 User-agent: mu4e 1.0; emacs 26.1

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

signature.asc
Description: PGP signature

--- End Message ---

--- Begin Message --- Subject: Re: [bug#34246] [PATCH 0/3] Improve the upower-service style. Date: Sat, 16 Feb 2019 22:15:43 +0000 User-agent: mu4e 1.0; emacs 26.1

Ludovic Courtès <address@hidden> writes:

> Hello!
>
> Christopher Baines <address@hidden> 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 :)

signature.asc
Description: PGP signature

--- End Message ---