All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH v3 1/1] package/systemd: add a menu entry to enable portable services
@ 2020-11-26 13:06 Francois Gervais
  0 siblings, 0 replies; 4+ messages in thread
From: Francois Gervais @ 2020-11-26 13:06 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Francois Gervais <fgervais@distech-controls.com>
---
Changes v2 -> v3:
  - Squash both patches in 1

 package/systemd/Config.in  | 17 +++++++++++++++++
 package/systemd/systemd.mk |  7 ++++++-
 2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/package/systemd/Config.in b/package/systemd/Config.in
index ec34478e3d..3966ba2fc7 100644
--- a/package/systemd/Config.in
+++ b/package/systemd/Config.in
@@ -343,6 +343,23 @@ comment "polkit support needs a toolchain with C++, wchar, NPTL, gcc >= 4.9"
 		!BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 || \
 		!BR2_USE_WCHAR
 
+config BR2_PACKAGE_SYSTEMD_PORTABLED
+	bool "enable portable services"
+	help
+	  Portable services are systemd services that can be dynamically
+	  attached and detached from the system.
+
+	  These services must come with their own root directory which
+	  they are bond to through an automatically generated drop-in.
+
+	  They also have restrictions applied by the host system in the
+	  form of profiles.
+
+	  This functionality is provided by the system service systemd-portabled
+	  along with the corresponding CLI portablectl.
+
+	  https://systemd.io/PORTABLE_SERVICES/
+
 config BR2_PACKAGE_SYSTEMD_QUOTACHECK
 	bool "enable quotacheck tools"
 	help
diff --git a/package/systemd/systemd.mk b/package/systemd/systemd.mk
index cb12f667d6..5aed87f452 100644
--- a/package/systemd/systemd.mk
+++ b/package/systemd/systemd.mk
@@ -33,7 +33,6 @@ SYSTEMD_CONF_OPTS += \
 	-Dman=false \
 	-Dmount-path=/usr/bin/mount \
 	-Dnss-systemd=true \
-	-Dportabled=false \
 	-Dquotacheck-path=/usr/sbin/quotacheck \
 	-Dquotaon-path=/usr/sbin/quotaon \
 	-Drootlibdir='/usr/lib' \
@@ -417,6 +416,12 @@ else
 SYSTEMD_CONF_OPTS += -Dpolkit=false
 endif
 
+ifeq ($(BR2_PACKAGE_SYSTEMD_PORTABLED),y)
+SYSTEMD_CONF_OPTS += -Dportabled=true
+else
+SYSTEMD_CONF_OPTS += -Dportabled=false
+endif
+
 ifeq ($(BR2_PACKAGE_SYSTEMD_NETWORKD),y)
 SYSTEMD_CONF_OPTS += -Dnetworkd=true
 SYSTEMD_NETWORKD_USER = systemd-network -1 systemd-network -1 * - - - systemd Network Management
-- 
2.17.1

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

* [Buildroot] [PATCH v3 1/1] package/systemd: add a menu entry to enable portable services
  2020-12-04 13:19 ` Gervais, Francois
@ 2020-12-15 21:02   ` Arnout Vandecappelle
  0 siblings, 0 replies; 4+ messages in thread
From: Arnout Vandecappelle @ 2020-12-15 21:02 UTC (permalink / raw)
  To: buildroot



On 04/12/2020 14:19, Gervais, Francois via buildroot wrote:
> Is there something more required on my part for this patch?
> 
> The delay might be normal but since my previous patches where
> merged in quite quickly I'd like to make sure I'm not missing something.

 It's not really normal, but it is usual :-) We have a huge patch backlog, as
you can see on http://patchwork.ozlabs.org/project/buildroot/list/ It can
sometimes take up to a year before even a simple patch gets merged, unfortunately.

 That said, I've applied your patch now.

> From: Gervais, Francois <FGervais@distech-controls.com>
> Sent: November 26, 2020 10:55 AM
> To: buildroot at buildroot.org <buildroot@buildroot.org>
> Cc: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>; Yann E . MORIN <yann.morin.1998@free.fr>; Norbert Lange <nolange79@gmail.com>; Gervais, Francois <FGervais@distech-controls.com>
> Subject: [PATCH v3 1/1] package/systemd: add a menu entry to enable portable services 
> ?
> Signed-off-by: Francois Gervais <fgervais@distech-controls.com>
> Reviewed-by: Norbert Lange <nolange79@gmail.com>
> ---
> ?package/systemd/Config.in? | 17 +++++++++++++++++
> ?package/systemd/systemd.mk |? 7 ++++++-
> ?2 files changed, 23 insertions(+), 1 deletion(-)
> 
> diff --git a/package/systemd/Config.in b/package/systemd/Config.in
> index ec34478e3d..3966ba2fc7 100644
> --- a/package/systemd/Config.in
> +++ b/package/systemd/Config.in
> @@ -343,6 +343,23 @@ comment "polkit support needs a toolchain with C++, wchar, NPTL, gcc >= 4.9"
> ???????????????? !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 || \
> ???????????????? !BR2_USE_WCHAR
> ?
> +config BR2_PACKAGE_SYSTEMD_PORTABLED
> +?????? bool "enable portable services"
> +?????? help
> +???????? Portable services are systemd services that can be dynamically
> +???????? attached and detached from the system.
> +
> +???????? These services must come with their own root directory which
> +???????? they are bond to through an automatically generated drop-in.
> +
> +???????? They also have restrictions applied by the host system in the
> +???????? form of profiles.
> +
> +???????? This functionality is provided by the system service systemd-portabled

 This line was too long, as reported by utils/check-package. I fixed that while
applying.

 Regards,
 Arnout

> +???????? along with the corresponding CLI portablectl.
> +
> +???????? https://systemd.io/PORTABLE_SERVICES/
> +
> ?config BR2_PACKAGE_SYSTEMD_QUOTACHECK
> ???????? bool "enable quotacheck tools"
> ???????? help
> diff --git a/package/systemd/systemd.mk b/package/systemd/systemd.mk
> index cb12f667d6..5aed87f452 100644
> --- a/package/systemd/systemd.mk
> +++ b/package/systemd/systemd.mk
> @@ -33,7 +33,6 @@ SYSTEMD_CONF_OPTS += \
> ???????? -Dman=false \
> ???????? -Dmount-path=/usr/bin/mount \
> ???????? -Dnss-systemd=true \
> -?????? -Dportabled=false \
> ???????? -Dquotacheck-path=/usr/sbin/quotacheck \
> ???????? -Dquotaon-path=/usr/sbin/quotaon \
> ???????? -Drootlibdir='/usr/lib' \
> @@ -417,6 +416,12 @@ else
> ?SYSTEMD_CONF_OPTS += -Dpolkit=false
> ?endif
> ?
> +ifeq ($(BR2_PACKAGE_SYSTEMD_PORTABLED),y)
> +SYSTEMD_CONF_OPTS += -Dportabled=true
> +else
> +SYSTEMD_CONF_OPTS += -Dportabled=false
> +endif
> +
> ?ifeq ($(BR2_PACKAGE_SYSTEMD_NETWORKD),y)
> ?SYSTEMD_CONF_OPTS += -Dnetworkd=true
> ?SYSTEMD_NETWORKD_USER = systemd-network -1 systemd-network -1 * - - - systemd Network Management
> 

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

* [Buildroot] [PATCH v3 1/1] package/systemd: add a menu entry to enable portable services
  2020-11-26 15:55 Francois Gervais
@ 2020-12-04 13:19 ` Gervais, Francois
  2020-12-15 21:02   ` Arnout Vandecappelle
  0 siblings, 1 reply; 4+ messages in thread
From: Gervais, Francois @ 2020-12-04 13:19 UTC (permalink / raw)
  To: buildroot

Is there something more required on my part for this patch?

The delay might be normal but since my previous patches where
merged in quite quickly I'd like to make sure I'm not missing something.


From: Gervais, Francois <FGervais@distech-controls.com>
Sent: November 26, 2020 10:55 AM
To: buildroot at buildroot.org <buildroot@buildroot.org>
Cc: Maxime Hadjinlian <maxime.hadjinlian@gmail.com>; Yann E . MORIN <yann.morin.1998@free.fr>; Norbert Lange <nolange79@gmail.com>; Gervais, Francois <FGervais@distech-controls.com>
Subject: [PATCH v3 1/1] package/systemd: add a menu entry to enable portable services 
?
Signed-off-by: Francois Gervais <fgervais@distech-controls.com>
Reviewed-by: Norbert Lange <nolange79@gmail.com>
---
?package/systemd/Config.in? | 17 +++++++++++++++++
?package/systemd/systemd.mk |? 7 ++++++-
?2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/package/systemd/Config.in b/package/systemd/Config.in
index ec34478e3d..3966ba2fc7 100644
--- a/package/systemd/Config.in
+++ b/package/systemd/Config.in
@@ -343,6 +343,23 @@ comment "polkit support needs a toolchain with C++, wchar, NPTL, gcc >= 4.9"
???????????????? !BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 || \
???????????????? !BR2_USE_WCHAR
?
+config BR2_PACKAGE_SYSTEMD_PORTABLED
+?????? bool "enable portable services"
+?????? help
+???????? Portable services are systemd services that can be dynamically
+???????? attached and detached from the system.
+
+???????? These services must come with their own root directory which
+???????? they are bond to through an automatically generated drop-in.
+
+???????? They also have restrictions applied by the host system in the
+???????? form of profiles.
+
+???????? This functionality is provided by the system service systemd-portabled
+???????? along with the corresponding CLI portablectl.
+
+???????? https://systemd.io/PORTABLE_SERVICES/
+
?config BR2_PACKAGE_SYSTEMD_QUOTACHECK
???????? bool "enable quotacheck tools"
???????? help
diff --git a/package/systemd/systemd.mk b/package/systemd/systemd.mk
index cb12f667d6..5aed87f452 100644
--- a/package/systemd/systemd.mk
+++ b/package/systemd/systemd.mk
@@ -33,7 +33,6 @@ SYSTEMD_CONF_OPTS += \
???????? -Dman=false \
???????? -Dmount-path=/usr/bin/mount \
???????? -Dnss-systemd=true \
-?????? -Dportabled=false \
???????? -Dquotacheck-path=/usr/sbin/quotacheck \
???????? -Dquotaon-path=/usr/sbin/quotaon \
???????? -Drootlibdir='/usr/lib' \
@@ -417,6 +416,12 @@ else
?SYSTEMD_CONF_OPTS += -Dpolkit=false
?endif
?
+ifeq ($(BR2_PACKAGE_SYSTEMD_PORTABLED),y)
+SYSTEMD_CONF_OPTS += -Dportabled=true
+else
+SYSTEMD_CONF_OPTS += -Dportabled=false
+endif
+
?ifeq ($(BR2_PACKAGE_SYSTEMD_NETWORKD),y)
?SYSTEMD_CONF_OPTS += -Dnetworkd=true
?SYSTEMD_NETWORKD_USER = systemd-network -1 systemd-network -1 * - - - systemd Network Management
-- 
2.17.1

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

* [Buildroot] [PATCH v3 1/1] package/systemd: add a menu entry to enable portable services
@ 2020-11-26 15:55 Francois Gervais
  2020-12-04 13:19 ` Gervais, Francois
  0 siblings, 1 reply; 4+ messages in thread
From: Francois Gervais @ 2020-11-26 15:55 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Francois Gervais <fgervais@distech-controls.com>
Reviewed-by: Norbert Lange <nolange79@gmail.com>
---
 package/systemd/Config.in  | 17 +++++++++++++++++
 package/systemd/systemd.mk |  7 ++++++-
 2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/package/systemd/Config.in b/package/systemd/Config.in
index ec34478e3d..3966ba2fc7 100644
--- a/package/systemd/Config.in
+++ b/package/systemd/Config.in
@@ -343,6 +343,23 @@ comment "polkit support needs a toolchain with C++, wchar, NPTL, gcc >= 4.9"
 		!BR2_TOOLCHAIN_GCC_AT_LEAST_4_9 || \
 		!BR2_USE_WCHAR
 
+config BR2_PACKAGE_SYSTEMD_PORTABLED
+	bool "enable portable services"
+	help
+	  Portable services are systemd services that can be dynamically
+	  attached and detached from the system.
+
+	  These services must come with their own root directory which
+	  they are bond to through an automatically generated drop-in.
+
+	  They also have restrictions applied by the host system in the
+	  form of profiles.
+
+	  This functionality is provided by the system service systemd-portabled
+	  along with the corresponding CLI portablectl.
+
+	  https://systemd.io/PORTABLE_SERVICES/
+
 config BR2_PACKAGE_SYSTEMD_QUOTACHECK
 	bool "enable quotacheck tools"
 	help
diff --git a/package/systemd/systemd.mk b/package/systemd/systemd.mk
index cb12f667d6..5aed87f452 100644
--- a/package/systemd/systemd.mk
+++ b/package/systemd/systemd.mk
@@ -33,7 +33,6 @@ SYSTEMD_CONF_OPTS += \
 	-Dman=false \
 	-Dmount-path=/usr/bin/mount \
 	-Dnss-systemd=true \
-	-Dportabled=false \
 	-Dquotacheck-path=/usr/sbin/quotacheck \
 	-Dquotaon-path=/usr/sbin/quotaon \
 	-Drootlibdir='/usr/lib' \
@@ -417,6 +416,12 @@ else
 SYSTEMD_CONF_OPTS += -Dpolkit=false
 endif
 
+ifeq ($(BR2_PACKAGE_SYSTEMD_PORTABLED),y)
+SYSTEMD_CONF_OPTS += -Dportabled=true
+else
+SYSTEMD_CONF_OPTS += -Dportabled=false
+endif
+
 ifeq ($(BR2_PACKAGE_SYSTEMD_NETWORKD),y)
 SYSTEMD_CONF_OPTS += -Dnetworkd=true
 SYSTEMD_NETWORKD_USER = systemd-network -1 systemd-network -1 * - - - systemd Network Management
-- 
2.17.1

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

end of thread, other threads:[~2020-12-15 21:02 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-26 13:06 [Buildroot] [PATCH v3 1/1] package/systemd: add a menu entry to enable portable services Francois Gervais
2020-11-26 15:55 Francois Gervais
2020-12-04 13:19 ` Gervais, Francois
2020-12-15 21:02   ` Arnout Vandecappelle

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.