[bug#67657] [PATCH] services: connman: Add 'connman-general-configuration'.

[Bruno Victal]

Hi muradm,

On 2023-12-06 10:19, muradm wrote:
> +   "Enable background scanning.  Default is true.  If wifi is disconnected, 
> the
> +background scanning will follow a simple backoff mechanism from 3s up to 5

'back off'

> +   "List of Fallback timeservers.  These timeservers are used for NTP sync
> +when there are no timeservers set by the user or by the service, and when
> +use-gateways-as-timeservers? is @code{#f}.  These can contain a mixed

@code{use-gateways-as-timeservers?}

> +  (default-auto-connect-technologies
> +   maybe-list
> +   "List of technologies that are marked autoconnectable by default.  The
> +default value for this entry when empty is ethernet,wifi,cellular.  Services

Perhaps @code{ethernet}, @code{wifi} and @code{cellular}? Or put this in
lisp list syntax. (*)

> +that are automatically connected must have been set up and saved to
> +storage beforehand.")
> +  (default-favourite-technologies
> +   maybe-list
> +   "List of technologies that are marked favorite by default.  The default
> +value for this entry when empty is ethernet.  Connects to services from

See (*).

> +this technology even if not setup and saved to storage.")
> +  (always-connected-technologies
> +   maybe-list
> +   "List of technoolgies which are always connected regardless of
> +preferred-technologies setting (auto-connect? @code{#t}).  The default value

@code{(auto-connect? #t)}.

> +  (network-interface-blacklist
> +   maybe-list
> +   "List of blacklisted network interfaces.  Found interfaces will be
> +compared to the list and will not be handled by ConnMan, if their first
> +characters match any of the list entries.  Default value is
> +vmnet,vboxnet,virbr,ifb,veth.")

See (*).

> +  (tethering-technologies
> +   maybe-list
> +   "List of technologies that are allowed to enable tethering.  The
> +default value is wifi,bluetooth,gadget.

See (*).

> +here are used for tethering.  If one wants to tether ethernet, then add
> +\"ethernet\" in the list.  NOTE that if ethernet tethering is enabled,

… @samp{\"ethernet\"} to the list.

> +in READY state.  Default value is @code{#t}.")
> +  (online-check-ipv4-url
> +   maybe-string
> +   "IPv4 URL used during the online status check.  Please refer to
> +the README for more detailed  information.  Default value is
> +http://ipv4.connman.net/online/status.html.";)

@url{http://ipv4.connman.net/online/status.html}. (**)

> +  (online-check-ipv6-url
> +   maybe-string
> +   "IPv6 URL used during the online status check.  Please refer to
> +the README for more detailed  information.  Default value is
> +http://ipv6.connman.net/online/status.html.";)

See (**).

> +  (online-check-initial-interval
> +   maybe-number
> +   "Range of intervals between two online check requests.  Please
> +refer to the README for more detailed information.  Default values is 1.")

[…]

> +  (online-check-max-interval
> +   maybe-number
> +   "Range of intervals between two online check requests.  Please
> +refer to the README for more detailed information.  Default values is 1.")

Default values is @samp{1}.

> +  (enable-online-to-ready-transition?
> +   maybe-boolean
> +   "WARNING: Experimental feature!!! In addition to enable-online-check

@code{enable-online-check}

> +service to DISCONNECT state.  If this setting is @code{#t}, the HTTP GET
> +request keeps beeing called to guarantee that end-to-end connectivity

'being'.

> +  (localtime
> +   maybe-string
> +   "Path to localtime file.  Defaults to /etc/localtime.")

@file{/etc/localtime}

> +  (regdom-follows-timezone?
> +   maybe-boolean
> +   "Enable regdomain to be changed along timezone changes. With
> +this option set to true each time the timezone changes the first
> +present ISO3166 country code is being read from
> +/usr/share/zoneinfo/zone1970.tab and set as regdom value.  Default
> +value is @code{#f}.")

@file{/usr/share/zoneinfo/zone1970.tab}.

Perhaps expand 'regdom' as 'regulatory domain'? Likewise for the field
name. (i.e. regulatory-domain-follows-timezone?)
You can perform the field name readjustment by using a custom serializer:

--8<---------------cut here---------------start------------->8---
(regulatory-domain-follows-timezone?
 maybe-boolean
 "Allow the regulatory domain to be changed along timezone changes. With
 this option set to true each time the timezone changes the first
 present ISO3166 country code is being read from
 @file{/usr/share/zoneinfo/zone1970.tab} and set as the regulatory domain value.
 Default value is @code{#f}."
 (serializer
  (lambda (_ value)
   (connman-general-configuration-serialize-boolean "regdom-follows-timezone" 
value))))
--8<---------------cut here---------------end--------------->8---

> +  (resolv-conf
> +   maybe-string
> +   "Path to resolv.conf file.  If the file does not exist, but
> +intermediate directories exist, it will be created.  If this option
> +is not set, it tries to write into /var/run/connman/resolv.conf if
> +it fails (/var/run/connman does not exist or is not writeable).  If
> +you do not want to update resolv.conf, you can set /dev/null.")

Wrap the paths with @file{…}.

> +
> +;; ,in (gnu services networking) (connman-general-configuration-generate-doc)
> +(define (connman-general-configuration-generate-doc)
> +  (configuration->documentation 'connman-general-configuration))

This isn't needed.

-- 
Furthermore, I consider that nonfree software must be eradicated.

Cheers,
Bruno.

OpenPGP_signature.asc
Description: OpenPGP digital signature