GNU bug report logs - #32785
[PATCH] services: wpa-supplicant: Extend to support configuration parameters.

Previous Next

Full log

Message #16 received at 32785-done <at> debbugs.gnu.org (full text, mbox):

From: Marius Bakke <mbakke <at> fastmail.com>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 32785-done <at> debbugs.gnu.org
Subject: Re: [bug#32785] [PATCH] services: wpa-supplicant: Extend to support
 configuration parameters.
Date: Wed, 17 Oct 2018 20:37:35 +0200

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

ludo <at> gnu.org (Ludovic Courtès) writes:

> Hello,
>
> Marius Bakke <mbakke <at> fastmail.com> skribis:
>
>> This allows using WPA Supplicant "standalone" without an additional
>> network manager.  The default configuration is unchanged.
>>
>> * gnu/services/networking.scm (<wpa-supplicant-configuration>): New record type.
>> (wpa-supplicant-shepherd-service): Pass configuration records to the daemon.
>> (wpa-supplicant-service-type): Adjust accordingly.
>> * doc/guix.texi (Networking Services): Document the new service type.
>
> [...]
>
>> +@deftp {Data Type} wpa-supplicant-manager-configuration
>                                      ^
> Should be ‘wpa-supplicant-configuration’.

Good catch; fixed!

>
>> +    (service-type (name 'wpa-supplicant)
>> +                  (extensions
>> +                   (list (service-extension shepherd-root-service-type
>> +                                            wpa-supplicant-shepherd-service)
>> +                         (service-extension dbus-root-service-type config->package)
>> +                         (service-extension profile-service-type config->package)))
>> +                  (default-value (wpa-supplicant-configuration)))))
>
> While you’re at it you can add a ‘description’ field.  :-)

See previous comment.

> I haven’t tested it but it LGTM.

I have done some testing and the default configuration seems to work
as before.  Pushed as acce0a474c1493ab18912bc46285248e4ccb0314.

Thank you!

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