Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/systemd: do not enable networking/time components by default
@ 2023-05-03 14:52 James Knight
  2023-05-03 19:27 ` Yann E. MORIN
  0 siblings, 1 reply; 2+ messages in thread
From: James Knight @ 2023-05-03 14:52 UTC (permalink / raw)
  To: buildroot; +Cc: James Knight, Norbert Lange, Sen Hastings, Yann E . MORIN

Tweaking the configuration of systemd to not enable networking and time
related service capabilities. The use of the system services are not
required, especially if other packages provide a respective capability
to a target (e.g. a target configured for chronyd will most likely not
want to use systemd's timedate/timesync services).

Signed-off-by: James Knight <james.d.knight@live.com>
---
 package/systemd/Config.in | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/package/systemd/Config.in b/package/systemd/Config.in
index bea717a4560f573fafbde53efb846f8be0b663c0..be1c29c2ec3772923a1164f25ce2d3fad78fd195 100644
--- a/package/systemd/Config.in
+++ b/package/systemd/Config.in
@@ -256,7 +256,6 @@ comment "homed support needs a toolchain w/ threads, dynamic library, kernel hea
 
 config BR2_PACKAGE_SYSTEMD_HOSTNAMED
 	bool "enable hostname daemon"
-	default y
 	help
 	  systemd-hostnamed is a system service that may be used as a
 	  mechanism to change the system's hostname.
@@ -329,7 +328,6 @@ config BR2_PACKAGE_SYSTEMD_MACHINED
 
 config BR2_PACKAGE_SYSTEMD_MYHOSTNAME
 	bool "enable myhostname NSS plugin"
-	default y
 	help
 	  nss-myhostname is a plug-in module for the GNU Name Service
 	  Switch (NSS) functionality of the GNU C Library (glibc),
@@ -340,7 +338,6 @@ config BR2_PACKAGE_SYSTEMD_MYHOSTNAME
 
 config BR2_PACKAGE_SYSTEMD_NETWORKD
 	bool "enable network manager"
-	default y
 	help
 	  systemd-networkd is a system service that manages networks.
 	  It detects and configures network devices as they appear, as
@@ -433,7 +430,6 @@ config BR2_PACKAGE_SYSTEMD_REPART
 
 config BR2_PACKAGE_SYSTEMD_RESOLVED
 	bool "enable resolve daemon"
-	default y
 	help
 	  systemd-resolved is a system service that provides network
 	  name resolution to local applications. It implements a
@@ -507,7 +503,6 @@ config BR2_PACKAGE_SYSTEMD_SYSUSERS
 
 config BR2_PACKAGE_SYSTEMD_TIMEDATED
 	bool "enable timedate daemon"
-	default y
 	help
 	  systemd-timedated is a system service that may be used as a
 	  mechanism to change the system clock and timezone, as well
@@ -517,7 +512,6 @@ config BR2_PACKAGE_SYSTEMD_TIMEDATED
 
 config BR2_PACKAGE_SYSTEMD_TIMESYNCD
 	bool "enable timesync daemon"
-	default y
 	help
 	  systemd-timesyncd is a service that may be used to
 	  synchronize the local system clock with a Network Time
-- 
2.40.1.windows.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Buildroot] [PATCH] package/systemd: do not enable networking/time components by default
  2023-05-03 14:52 [Buildroot] [PATCH] package/systemd: do not enable networking/time components by default James Knight
@ 2023-05-03 19:27 ` Yann E. MORIN
  0 siblings, 0 replies; 2+ messages in thread
From: Yann E. MORIN @ 2023-05-03 19:27 UTC (permalink / raw)
  To: James Knight; +Cc: Norbert Lange, Sen Hastings, buildroot

James, All,

On 2023-05-03 10:52 -0400, James Knight spake thusly:
> Tweaking the configuration of systemd to not enable networking and time
> related service capabilities. The use of the system services are not
> required, especially if other packages provide a respective capability
> to a target (e.g. a target configured for chronyd will most likely not
> want to use systemd's timedate/timesync services).

Those are only default settings, they can be disabled; people using
alternate solutions can still easily disable them.

However, systemd is a critical package: we want it to work "correctly"
by default, and we anyway expect people that want to configure it to
know what they are doing.

So, yes, basically, that's why those options default to 'y': unless the
user takes explicit action to use an alternative, they provide a
sane-by-default system: network is functional, hostname is set and there
is a proper resolver, and (for machines lacking an RTC) time and date
are set on boot (with NTP is available, otherwise the systemd release
date used as a sane default).

Also, if one were to argue that those features should be disabled by
default, then all such optional features should be disabled. But then,
that means vcosole would have to default to 'n', and this would by
default generate a system where everything started on a virtual console
(e.g. tty1 on HDMI...) be killed; see fc0787e76ea2 (package/systemd: set
vconsole support option to default y).

Finally, removing the defaults means that existing defconfig files will
suddenly fail to work as previously. We want to avoid that unless there
is a very compelling reason that a breakage is acceptable, and there is
no such reason IMHO.

Regards,
Yann E. MORIN.

> Signed-off-by: James Knight <james.d.knight@live.com>
> ---
>  package/systemd/Config.in | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/package/systemd/Config.in b/package/systemd/Config.in
> index bea717a4560f573fafbde53efb846f8be0b663c0..be1c29c2ec3772923a1164f25ce2d3fad78fd195 100644
> --- a/package/systemd/Config.in
> +++ b/package/systemd/Config.in
> @@ -256,7 +256,6 @@ comment "homed support needs a toolchain w/ threads, dynamic library, kernel hea
>  
>  config BR2_PACKAGE_SYSTEMD_HOSTNAMED
>  	bool "enable hostname daemon"
> -	default y
>  	help
>  	  systemd-hostnamed is a system service that may be used as a
>  	  mechanism to change the system's hostname.
> @@ -329,7 +328,6 @@ config BR2_PACKAGE_SYSTEMD_MACHINED
>  
>  config BR2_PACKAGE_SYSTEMD_MYHOSTNAME
>  	bool "enable myhostname NSS plugin"
> -	default y
>  	help
>  	  nss-myhostname is a plug-in module for the GNU Name Service
>  	  Switch (NSS) functionality of the GNU C Library (glibc),
> @@ -340,7 +338,6 @@ config BR2_PACKAGE_SYSTEMD_MYHOSTNAME
>  
>  config BR2_PACKAGE_SYSTEMD_NETWORKD
>  	bool "enable network manager"
> -	default y
>  	help
>  	  systemd-networkd is a system service that manages networks.
>  	  It detects and configures network devices as they appear, as
> @@ -433,7 +430,6 @@ config BR2_PACKAGE_SYSTEMD_REPART
>  
>  config BR2_PACKAGE_SYSTEMD_RESOLVED
>  	bool "enable resolve daemon"
> -	default y
>  	help
>  	  systemd-resolved is a system service that provides network
>  	  name resolution to local applications. It implements a
> @@ -507,7 +503,6 @@ config BR2_PACKAGE_SYSTEMD_SYSUSERS
>  
>  config BR2_PACKAGE_SYSTEMD_TIMEDATED
>  	bool "enable timedate daemon"
> -	default y
>  	help
>  	  systemd-timedated is a system service that may be used as a
>  	  mechanism to change the system clock and timezone, as well
> @@ -517,7 +512,6 @@ config BR2_PACKAGE_SYSTEMD_TIMEDATED
>  
>  config BR2_PACKAGE_SYSTEMD_TIMESYNCD
>  	bool "enable timesync daemon"
> -	default y
>  	help
>  	  systemd-timesyncd is a service that may be used to
>  	  synchronize the local system clock with a Network Time
> -- 
> 2.40.1.windows.1
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-05-03 19:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-05-03 14:52 [Buildroot] [PATCH] package/systemd: do not enable networking/time components by default James Knight
2023-05-03 19:27 ` Yann E. MORIN

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox