All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 0/3] portable services: another use case of BR
@ 2022-02-21  4:30 Francois Perrad
  2022-02-21  4:30 ` [Buildroot] [PATCH 1/3] package/skeleton-init-portable: new skeleton Francois Perrad
                   ` (4 more replies)
  0 siblings, 5 replies; 9+ messages in thread
From: Francois Perrad @ 2022-02-21  4:30 UTC (permalink / raw)
  To: buildroot

This series allows to build a rootfs suitable for portable services with systemd.

A new value of "Init system" is introduced : BR2_INIT_PORTABLE.
BR2_INIT_PORTABLE is like BR2_INIT_NONE, but with all FOO_INSTALL_INIT_SYSTEMD executed.

A new type of skeleton is required : BR2_PACKAGE_SKELETON_INIT_PORTABLE.

see https://systemd.io/PORTABLE_SERVICES/

Francois Perrad (3):
  package/skeleton-init-portable: new skeleton
  portable: new init system
  portable: tweak some default values

 DEVELOPERS                                    |  1 +
 fs/squashfs/Config.in                         |  1 +
 package/Config.in                             |  1 +
 package/ifupdown-scripts/Config.in            |  2 +-
 package/pkg-generic.mk                        |  2 ++
 package/skeleton-init-portable/Config.in      |  7 ++++++
 .../skeleton-init-portable.mk                 | 23 +++++++++++++++++++
 system/Config.in                              | 18 +++++++++++----
 8 files changed, 50 insertions(+), 5 deletions(-)
 create mode 100644 package/skeleton-init-portable/Config.in
 create mode 100644 package/skeleton-init-portable/skeleton-init-portable.mk

-- 
2.32.0

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

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH 1/3] package/skeleton-init-portable: new skeleton
  2022-02-21  4:30 [Buildroot] [PATCH 0/3] portable services: another use case of BR Francois Perrad
@ 2022-02-21  4:30 ` Francois Perrad
  2022-02-21  4:30 ` [Buildroot] [PATCH 2/3] portable: new init system Francois Perrad
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 9+ messages in thread
From: Francois Perrad @ 2022-02-21  4:30 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
 DEVELOPERS                                    |  1 +
 package/Config.in                             |  1 +
 package/skeleton-init-portable/Config.in      |  7 ++++++
 .../skeleton-init-portable.mk                 | 23 +++++++++++++++++++
 4 files changed, 32 insertions(+)
 create mode 100644 package/skeleton-init-portable/Config.in
 create mode 100644 package/skeleton-init-portable/skeleton-init-portable.mk

diff --git a/DEVELOPERS b/DEVELOPERS
index ac702d8d0..7cafd6c95 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -1010,6 +1010,7 @@ F:	package/pkg-perl.mk
 F:	package/pkg-luarocks.mk
 F:	package/quickjs/
 F:	package/rings/
+F:	package/skeleton-init-portable
 F:	package/tekui/
 F:	package/wpebackend-fdo/
 F:	package/wpewebkit/
diff --git a/package/Config.in b/package/Config.in
index 10209d84d..0b947f682 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -6,6 +6,7 @@ menu "Target packages"
 	source "package/skeleton-init-common/Config.in"
 	source "package/skeleton-init-none/Config.in"
 	source "package/skeleton-init-openrc/Config.in"
+	source "package/skeleton-init-portable/Config.in"
 	source "package/skeleton-init-systemd/Config.in"
 	source "package/skeleton-init-sysv/Config.in"
 
diff --git a/package/skeleton-init-portable/Config.in b/package/skeleton-init-portable/Config.in
new file mode 100644
index 000000000..ca434451c
--- /dev/null
+++ b/package/skeleton-init-portable/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_SKELETON_INIT_PORTABLE
+	bool
+	select BR2_PACKAGE_HAS_SKELETON
+	select BR2_PACKAGE_SKELETON_INIT_COMMON
+
+config BR2_PACKAGE_PROVIDES_SKELETON
+	default "skeleton-init-portable" if BR2_PACKAGE_SKELETON_INIT_PORTABLE
diff --git a/package/skeleton-init-portable/skeleton-init-portable.mk b/package/skeleton-init-portable/skeleton-init-portable.mk
new file mode 100644
index 000000000..8174a5d82
--- /dev/null
+++ b/package/skeleton-init-portable/skeleton-init-portable.mk
@@ -0,0 +1,23 @@
+################################################################################
+#
+# skeleton-init-portable
+#
+################################################################################
+
+# The skeleton can't depend on the toolchain, since all packages depends on the
+# skeleton and the toolchain is a target package, as is skeleton.
+# Hence, skeleton would depends on the toolchain and the toolchain would depend
+# on skeleton.
+SKELETON_INIT_PORTABLE_ADD_TOOLCHAIN_DEPENDENCY = NO
+SKELETON_INIT_PORTABLE_ADD_SKELETON_DEPENDENCY = NO
+
+SKELETON_INIT_PORTABLE_DEPENDENCIES = skeleton-init-common
+
+SKELETON_INIT_PORTABLE_PROVIDES = skeleton
+
+define SKELETON_INIT_PORTABLE_INSTALL_TARGET_CMDS
+	touch $(TARGET_DIR)/etc/machine-id
+	mkdir -p $(TARGET_DIR)/var/tmp
+endef
+
+$(eval $(generic-package))
-- 
2.32.0

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

^ permalink raw reply related	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH 2/3] portable: new init system
  2022-02-21  4:30 [Buildroot] [PATCH 0/3] portable services: another use case of BR Francois Perrad
  2022-02-21  4:30 ` [Buildroot] [PATCH 1/3] package/skeleton-init-portable: new skeleton Francois Perrad
@ 2022-02-21  4:30 ` Francois Perrad
  2022-02-21  4:30 ` [Buildroot] [PATCH 3/3] portable: tweak some default values Francois Perrad
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 9+ messages in thread
From: Francois Perrad @ 2022-02-21  4:30 UTC (permalink / raw)
  To: buildroot

see https://systemd.io/PORTABLE_SERVICES/

Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
 package/pkg-generic.mk | 2 ++
 system/Config.in       | 9 +++++++++
 2 files changed, 11 insertions(+)

diff --git a/package/pkg-generic.mk b/package/pkg-generic.mk
index b3a7e1d60..e62e7aada 100644
--- a/package/pkg-generic.mk
+++ b/package/pkg-generic.mk
@@ -383,6 +383,8 @@ $(BUILD_DIR)/%/.stamp_target_installed:
 	+$($(PKG)_INSTALL_TARGET_CMDS)
 	$(if $(BR2_INIT_SYSTEMD),\
 		$($(PKG)_INSTALL_INIT_SYSTEMD))
+	$(if $(BR2_INIT_PORTABLE),\
+		$($(PKG)_INSTALL_INIT_SYSTEMD))
 	$(if $(BR2_INIT_SYSV)$(BR2_INIT_BUSYBOX),\
 		$($(PKG)_INSTALL_INIT_SYSV))
 	$(if $(BR2_INIT_OPENRC), \
diff --git a/system/Config.in b/system/Config.in
index a4ffbfa18..e71e57915 100644
--- a/system/Config.in
+++ b/system/Config.in
@@ -141,6 +141,15 @@ comment "systemd needs a glibc toolchain w/ SSP, headers >= 3.10, host and targe
 		!BR2_TOOLCHAIN_GCC_AT_LEAST_5 || \
 		!BR2_HOST_GCC_AT_LEAST_5
 
+config BR2_INIT_PORTABLE
+	bool "portable"
+	select BR2_ROOTFS_MERGED_USR
+	select BR2_PACKAGE_SKELETON_INIT_PORTABLE if BR2_ROOTFS_SKELETON_DEFAULT
+	help
+	  portable services with systemd
+	  
+	  https://systemd.io/PORTABLE_SERVICES/
+
 config BR2_INIT_NONE
 	bool "None"
 	select BR2_PACKAGE_SKELETON_INIT_NONE if BR2_ROOTFS_SKELETON_DEFAULT
-- 
2.32.0

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

^ permalink raw reply related	[flat|nested] 9+ messages in thread
* [Buildroot] [PATCH 3/3] portable: tweak some default values
  2022-02-21  4:30 [Buildroot] [PATCH 0/3] portable services: another use case of BR Francois Perrad
  2022-02-21  4:30 ` [Buildroot] [PATCH 1/3] package/skeleton-init-portable: new skeleton Francois Perrad
  2022-02-21  4:30 ` [Buildroot] [PATCH 2/3] portable: new init system Francois Perrad
@ 2022-02-21  4:30 ` Francois Perrad
  2022-08-21 21:17 ` [Buildroot] [PATCH 0/3] portable services: another use case of BR Yann E. MORIN
  2022-08-23 16:46 ` Yann E. MORIN
  4 siblings, 0 replies; 9+ messages in thread
From: Francois Perrad @ 2022-02-21  4:30 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Francois Perrad <francois.perrad@gadz.org>
---
 fs/squashfs/Config.in              | 1 +
 package/ifupdown-scripts/Config.in | 2 +-
 system/Config.in                   | 9 +++++----
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/fs/squashfs/Config.in b/fs/squashfs/Config.in
index 70d4a20cf..ad27470aa 100644
--- a/fs/squashfs/Config.in
+++ b/fs/squashfs/Config.in
@@ -1,5 +1,6 @@
 config BR2_TARGET_ROOTFS_SQUASHFS
 	bool "squashfs root filesystem"
+	default y if BR2_INIT_PORTABLE
 	help
 	  Build a squashfs root filesystem
 
diff --git a/package/ifupdown-scripts/Config.in b/package/ifupdown-scripts/Config.in
index 166094c51..1607fa696 100644
--- a/package/ifupdown-scripts/Config.in
+++ b/package/ifupdown-scripts/Config.in
@@ -1,6 +1,6 @@
 config BR2_PACKAGE_IFUPDOWN_SCRIPTS
 	bool "ifupdown scripts"
-	default y if BR2_ROOTFS_SKELETON_DEFAULT
+	default y if BR2_ROOTFS_SKELETON_DEFAULT && !BR2_PACKAGE_SKELETON_INIT_PORTABLE
 	depends on !BR2_PACKAGE_SYSTEMD_NETWORKD && !BR2_PACKAGE_NETIFRC
 	help
 	  Set of scripts used by ifupdown (either the standalone one,
diff --git a/system/Config.in b/system/Config.in
index e71e57915..3e90e94cd 100644
--- a/system/Config.in
+++ b/system/Config.in
@@ -238,7 +238,7 @@ if BR2_ROOTFS_SKELETON_DEFAULT
 
 config BR2_TARGET_ENABLE_ROOT_LOGIN
 	bool "Enable root login with password"
-	default y
+	default y if !BR2_INIT_PORTABLE
 	select BR2_PACKAGE_HOST_MKPASSWD if BR2_TARGET_GENERIC_ROOT_PASSWD != ""
 	help
 	  Allow root to log in with a password.
@@ -280,7 +280,8 @@ config BR2_TARGET_GENERIC_ROOT_PASSWD
 
 choice
 	bool "/bin/sh"
-	default BR2_SYSTEM_BIN_SH_DASH if !BR2_PACKAGE_BUSYBOX
+	default BR2_SYSTEM_BIN_SH_DASH if !BR2_PACKAGE_BUSYBOX && !BR2_INIT_PORTABLE
+	default BR2_SYSTEM_BIN_SH_NONE if BR2_INIT_PORTABLE
 	help
 	  Select which shell will provide /bin/sh.
 
@@ -330,7 +331,7 @@ config BR2_SYSTEM_BIN_SH
 
 menuconfig BR2_TARGET_GENERIC_GETTY
 	bool "Run a getty (login prompt) after boot"
-	default y
+	default y if !BR2_INIT_PORTABLE
 
 if BR2_TARGET_GENERIC_GETTY
 config BR2_TARGET_GENERIC_GETTY_PORT
@@ -388,7 +389,7 @@ endif
 
 config BR2_TARGET_GENERIC_REMOUNT_ROOTFS_RW
 	bool "remount root filesystem read-write during boot"
-	default y
+	default y if !BR2_INIT_PORTABLE
 	help
 	  The root filesystem is typically mounted read-only at boot.
 	  By default, buildroot remounts it in read-write mode early
-- 
2.32.0

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

^ permalink raw reply related	[flat|nested] 9+ messages in thread
* Re: [Buildroot] [PATCH 0/3] portable services: another use case of BR
  2022-02-21  4:30 [Buildroot] [PATCH 0/3] portable services: another use case of BR Francois Perrad
                   ` (2 preceding siblings ...)
  2022-02-21  4:30 ` [Buildroot] [PATCH 3/3] portable: tweak some default values Francois Perrad
@ 2022-08-21 21:17 ` Yann E. MORIN
  2022-08-22 21:24   ` Yann E. MORIN
  2022-08-23 16:46 ` Yann E. MORIN
  4 siblings, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2022-08-21 21:17 UTC (permalink / raw)
  To: Francois Perrad; +Cc: buildroot

François, All,

On 2022-02-21 05:30 +0100, Francois Perrad spake thusly:
> This series allows to build a rootfs suitable for portable services with systemd.
> 
> A new value of "Init system" is introduced : BR2_INIT_PORTABLE.
> BR2_INIT_PORTABLE is like BR2_INIT_NONE, but with all FOO_INSTALL_INIT_SYSTEMD executed.
> 
> A new type of skeleton is required : BR2_PACKAGE_SKELETON_INIT_PORTABLE.
> 
> see https://systemd.io/PORTABLE_SERVICES/

I don't see anything that can't be already managed with existing
configuration options.

All that this new so-called init system does, is change some defaults to
match a specific use-case. We had similar stuff back in the day (e.g. an
option to define a router, one to define something else, and so on), and
we removed all of them.

Granted, portable services are slightly different, but still all that
this series eventually does is change some defaults; it does not even
enforce the settings so they are not even required.

Even the new skeleton case can already be handled by providing a custom
skeleton, and it is a trivial one, so even that is not needed.

If you want to build a portable service, just build a system with
systemd and the rest of the settings set as required for that use-case.

So, my position is to reject this new pseudo-init system.

Regards,
Yann E. MORIN.

> Francois Perrad (3):
>   package/skeleton-init-portable: new skeleton
>   portable: new init system
>   portable: tweak some default values
> 
>  DEVELOPERS                                    |  1 +
>  fs/squashfs/Config.in                         |  1 +
>  package/Config.in                             |  1 +
>  package/ifupdown-scripts/Config.in            |  2 +-
>  package/pkg-generic.mk                        |  2 ++
>  package/skeleton-init-portable/Config.in      |  7 ++++++
>  .../skeleton-init-portable.mk                 | 23 +++++++++++++++++++
>  system/Config.in                              | 18 +++++++++++----
>  8 files changed, 50 insertions(+), 5 deletions(-)
>  create mode 100644 package/skeleton-init-portable/Config.in
>  create mode 100644 package/skeleton-init-portable/skeleton-init-portable.mk
> 
> -- 
> 2.32.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  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] 9+ messages in thread
* Re: [Buildroot] [PATCH 0/3] portable services: another use case of BR
  2022-08-21 21:17 ` [Buildroot] [PATCH 0/3] portable services: another use case of BR Yann E. MORIN
@ 2022-08-22 21:24   ` Yann E. MORIN
  2022-08-23 16:14     ` Arnout Vandecappelle
  0 siblings, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2022-08-22 21:24 UTC (permalink / raw)
  To: Francois Perrad; +Cc: buildroot

François, all,

On 2022-08-21 23:17 +0200, Yann E. MORIN spake thusly:
> On 2022-02-21 05:30 +0100, Francois Perrad spake thusly:
> > This series allows to build a rootfs suitable for portable services with systemd.
> > 
> > A new value of "Init system" is introduced : BR2_INIT_PORTABLE.
> > BR2_INIT_PORTABLE is like BR2_INIT_NONE, but with all FOO_INSTALL_INIT_SYSTEMD executed.
> > 
> > A new type of skeleton is required : BR2_PACKAGE_SKELETON_INIT_PORTABLE.
> > 
> > see https://systemd.io/PORTABLE_SERVICES/
> 
> I don't see anything that can't be already managed with existing
> configuration options.

Ah, I missed one thing: this allows installing systemd units bundled in
Buildroot, without using systemd as a init system.

This is flawed, IMO, because a lot of packages now install their own
units as part of their install procedure, so without systemd enabled,
those packages wold not install their units.

So, in practice, to be really usefull, this would still need systemd to
be enabled, and we're back to the situation where this series only
brings a few defaults as policy.

So, I'm still not convinced about it.

Regards,
Yann E. MORIN.

> All that this new so-called init system does, is change some defaults to
> match a specific use-case. We had similar stuff back in the day (e.g. an
> option to define a router, one to define something else, and so on), and
> we removed all of them.
> 
> Granted, portable services are slightly different, but still all that
> this series eventually does is change some defaults; it does not even
> enforce the settings so they are not even required.
> 
> Even the new skeleton case can already be handled by providing a custom
> skeleton, and it is a trivial one, so even that is not needed.
> 
> If you want to build a portable service, just build a system with
> systemd and the rest of the settings set as required for that use-case.
> 
> So, my position is to reject this new pseudo-init system.
> 
> Regards,
> Yann E. MORIN.
> 
> > Francois Perrad (3):
> >   package/skeleton-init-portable: new skeleton
> >   portable: new init system
> >   portable: tweak some default values
> > 
> >  DEVELOPERS                                    |  1 +
> >  fs/squashfs/Config.in                         |  1 +
> >  package/Config.in                             |  1 +
> >  package/ifupdown-scripts/Config.in            |  2 +-
> >  package/pkg-generic.mk                        |  2 ++
> >  package/skeleton-init-portable/Config.in      |  7 ++++++
> >  .../skeleton-init-portable.mk                 | 23 +++++++++++++++++++
> >  system/Config.in                              | 18 +++++++++++----
> >  8 files changed, 50 insertions(+), 5 deletions(-)
> >  create mode 100644 package/skeleton-init-portable/Config.in
> >  create mode 100644 package/skeleton-init-portable/skeleton-init-portable.mk
> > 
> > -- 
> > 2.32.0
> > 
> > _______________________________________________
> > buildroot mailing list
> > buildroot@buildroot.org
> > https://lists.buildroot.org/mailman/listinfo/buildroot
> 
> -- 
> .-----------------.--------------------.------------------.--------------------.
> |  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

-- 
.-----------------.--------------------.------------------.--------------------.
|  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] 9+ messages in thread
* Re: [Buildroot] [PATCH 0/3] portable services: another use case of BR
  2022-08-22 21:24   ` Yann E. MORIN
@ 2022-08-23 16:14     ` Arnout Vandecappelle
  0 siblings, 0 replies; 9+ messages in thread
From: Arnout Vandecappelle @ 2022-08-23 16:14 UTC (permalink / raw)
  To: Yann E. MORIN, Francois Perrad; +Cc: buildroot



On 22/08/2022 23:24, Yann E. MORIN wrote:
> François, all,
> 
> On 2022-08-21 23:17 +0200, Yann E. MORIN spake thusly:
>> On 2022-02-21 05:30 +0100, Francois Perrad spake thusly:
>>> This series allows to build a rootfs suitable for portable services with systemd.
>>>
>>> A new value of "Init system" is introduced : BR2_INIT_PORTABLE.
>>> BR2_INIT_PORTABLE is like BR2_INIT_NONE, but with all FOO_INSTALL_INIT_SYSTEMD executed.
>>>
>>> A new type of skeleton is required : BR2_PACKAGE_SKELETON_INIT_PORTABLE.
>>>
>>> see https://systemd.io/PORTABLE_SERVICES/
>>
>> I don't see anything that can't be already managed with existing
>> configuration options.
> 
> Ah, I missed one thing: this allows installing systemd units bundled in
> Buildroot, without using systemd as a init system.

  In a portable service, /sbin/init is not used anyway, so the init system is 
irrelevant.

  The only real difference is that systemd and its dependencies are not 
installed to the target, which reduces size a little.

> This is flawed, IMO, because a lot of packages now install their own
> units as part of their install procedure, so without systemd enabled,
> those packages wold not install their units.
> 
> So, in practice, to be really usefull, this would still need systemd to
> be enabled, and we're back to the situation where this series only
> brings a few defaults as policy.

  Yeah, to be fully useful, we should probably replace all conditions on 
BR2_INIT_SYSTEMD (and many on BR2_PACKAGE_SYSTEMD) with something else... In 
addition, packages that actually require libsystemd (e.g. clamav) would need to 
select BR2_PACKAGE_SYSTEMD - and if you use any of those, the advantage is lost 
again.

  So, although the use case exists, I don't think the difference is important 
enough to bother, and this series addresses only a small part of the problem 
space. So I agree with Yann.

  Regards,
  Arnout


> 
> So, I'm still not convinced about it.
> 
> Regards,
> Yann E. MORIN.
> 
>> All that this new so-called init system does, is change some defaults to
>> match a specific use-case. We had similar stuff back in the day (e.g. an
>> option to define a router, one to define something else, and so on), and
>> we removed all of them.
>>
>> Granted, portable services are slightly different, but still all that
>> this series eventually does is change some defaults; it does not even
>> enforce the settings so they are not even required.
>>
>> Even the new skeleton case can already be handled by providing a custom
>> skeleton, and it is a trivial one, so even that is not needed.
>>
>> If you want to build a portable service, just build a system with
>> systemd and the rest of the settings set as required for that use-case.
>>
>> So, my position is to reject this new pseudo-init system.
>>
>> Regards,
>> Yann E. MORIN.
>>
>>> Francois Perrad (3):
>>>    package/skeleton-init-portable: new skeleton
>>>    portable: new init system
>>>    portable: tweak some default values
>>>
>>>   DEVELOPERS                                    |  1 +
>>>   fs/squashfs/Config.in                         |  1 +
>>>   package/Config.in                             |  1 +
>>>   package/ifupdown-scripts/Config.in            |  2 +-
>>>   package/pkg-generic.mk                        |  2 ++
>>>   package/skeleton-init-portable/Config.in      |  7 ++++++
>>>   .../skeleton-init-portable.mk                 | 23 +++++++++++++++++++
>>>   system/Config.in                              | 18 +++++++++++----
>>>   8 files changed, 50 insertions(+), 5 deletions(-)
>>>   create mode 100644 package/skeleton-init-portable/Config.in
>>>   create mode 100644 package/skeleton-init-portable/skeleton-init-portable.mk
>>>
>>> -- 
>>> 2.32.0
>>>
>>> _______________________________________________
>>> buildroot mailing list
>>> buildroot@buildroot.org
>>> https://lists.buildroot.org/mailman/listinfo/buildroot
>>
>> -- 
>> .-----------------.--------------------.------------------.--------------------.
>> |  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
> 
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 9+ messages in thread
* Re: [Buildroot] [PATCH 0/3] portable services: another use case of BR
  2022-02-21  4:30 [Buildroot] [PATCH 0/3] portable services: another use case of BR Francois Perrad
                   ` (3 preceding siblings ...)
  2022-08-21 21:17 ` [Buildroot] [PATCH 0/3] portable services: another use case of BR Yann E. MORIN
@ 2022-08-23 16:46 ` Yann E. MORIN
  2022-08-23 18:02   ` François Perrad
  4 siblings, 1 reply; 9+ messages in thread
From: Yann E. MORIN @ 2022-08-23 16:46 UTC (permalink / raw)
  To: Francois Perrad; +Cc: buildroot

François, All,

On 2022-02-21 05:30 +0100, Francois Perrad spake thusly:
> This series allows to build a rootfs suitable for portable services with systemd.
> 
> A new value of "Init system" is introduced : BR2_INIT_PORTABLE.
> BR2_INIT_PORTABLE is like BR2_INIT_NONE, but with all FOO_INSTALL_INIT_SYSTEMD executed.
> 
> A new type of skeleton is required : BR2_PACKAGE_SKELETON_INIT_PORTABLE.
> 
> see https://systemd.io/PORTABLE_SERVICES/

Given the feedback on this series, and after discussing with the other
maintainers, I've now marked this series as rejected. Sorry for that.

If you believe the use-case is really important, we can reconsider it in
the light of a newer and more complete series that addresses the issues
that were raised in this review.

Thank you!

Regards,
Yann E. MORIN.

> Francois Perrad (3):
>   package/skeleton-init-portable: new skeleton
>   portable: new init system
>   portable: tweak some default values
> 
>  DEVELOPERS                                    |  1 +
>  fs/squashfs/Config.in                         |  1 +
>  package/Config.in                             |  1 +
>  package/ifupdown-scripts/Config.in            |  2 +-
>  package/pkg-generic.mk                        |  2 ++
>  package/skeleton-init-portable/Config.in      |  7 ++++++
>  .../skeleton-init-portable.mk                 | 23 +++++++++++++++++++
>  system/Config.in                              | 18 +++++++++++----
>  8 files changed, 50 insertions(+), 5 deletions(-)
>  create mode 100644 package/skeleton-init-portable/Config.in
>  create mode 100644 package/skeleton-init-portable/skeleton-init-portable.mk
> 
> -- 
> 2.32.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  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] 9+ messages in thread
* Re: [Buildroot] [PATCH 0/3] portable services: another use case of BR
  2022-08-23 16:46 ` Yann E. MORIN
@ 2022-08-23 18:02   ` François Perrad
  0 siblings, 0 replies; 9+ messages in thread
From: François Perrad @ 2022-08-23 18:02 UTC (permalink / raw)
  To: buildroot


[-- Attachment #1.1: Type: text/plain, Size: 2771 bytes --]

Le mar. 23 août 2022 à 18:46, Yann E. MORIN <yann.morin.1998@free.fr> a
écrit :

> François, All,
>
> On 2022-02-21 05:30 +0100, Francois Perrad spake thusly:
> > This series allows to build a rootfs suitable for portable services with
> systemd.
> >
> > A new value of "Init system" is introduced : BR2_INIT_PORTABLE.
> > BR2_INIT_PORTABLE is like BR2_INIT_NONE, but with all
> FOO_INSTALL_INIT_SYSTEMD executed.
> >
> > A new type of skeleton is required : BR2_PACKAGE_SKELETON_INIT_PORTABLE.
> >
> > see https://systemd.io/PORTABLE_SERVICES/
>
> Given the feedback on this series, and after discussing with the other
> maintainers, I've now marked this series as rejected. Sorry for that.
>
>
Thanks for these feedbacks.

François


> If you believe the use-case is really important, we can reconsider it in
> the light of a newer and more complete series that addresses the issues
> that were raised in this review.
>
> Thank you!
>
> Regards,
> Yann E. MORIN.
>
> > Francois Perrad (3):
> >   package/skeleton-init-portable: new skeleton
> >   portable: new init system
> >   portable: tweak some default values
> >
> >  DEVELOPERS                                    |  1 +
> >  fs/squashfs/Config.in                         |  1 +
> >  package/Config.in                             |  1 +
> >  package/ifupdown-scripts/Config.in            |  2 +-
> >  package/pkg-generic.mk                        |  2 ++
> >  package/skeleton-init-portable/Config.in      |  7 ++++++
> >  .../skeleton-init-portable.mk                 | 23 +++++++++++++++++++
> >  system/Config.in                              | 18 +++++++++++----
> >  8 files changed, 50 insertions(+), 5 deletions(-)
> >  create mode 100644 package/skeleton-init-portable/Config.in
> >  create mode 100644 package/skeleton-init-portable/
> skeleton-init-portable.mk
> >
> > --
> > 2.32.0
> >
> > _______________________________________________
> > buildroot mailing list
> > buildroot@buildroot.org
> > https://lists.buildroot.org/mailman/listinfo/buildroot
>
> --
>
> .-----------------.--------------------.------------------.--------------------.
> |  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
>

[-- Attachment #1.2: Type: text/html, Size: 4372 bytes --]

[-- Attachment #2: Type: text/plain, Size: 150 bytes --]

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

^ permalink raw reply	[flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-08-23 18:03 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-21  4:30 [Buildroot] [PATCH 0/3] portable services: another use case of BR Francois Perrad
2022-02-21  4:30 ` [Buildroot] [PATCH 1/3] package/skeleton-init-portable: new skeleton Francois Perrad
2022-02-21  4:30 ` [Buildroot] [PATCH 2/3] portable: new init system Francois Perrad
2022-02-21  4:30 ` [Buildroot] [PATCH 3/3] portable: tweak some default values Francois Perrad
2022-08-21 21:17 ` [Buildroot] [PATCH 0/3] portable services: another use case of BR Yann E. MORIN
2022-08-22 21:24   ` Yann E. MORIN
2022-08-23 16:14     ` Arnout Vandecappelle
2022-08-23 16:46 ` Yann E. MORIN
2022-08-23 18:02   ` François Perrad

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.