Skip to contentSkip to conversation

poky-tiny: Update for DISTRO_FEATURES_INITMAN

PATCH Superseded Saul Wold 2 participants 4 messages Jan 25, 2013

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore

Saul Wold

author
#0 lore raw

PATCH 1/1 poky-tiny: Update for DISTRO_FEATURES_INITMAN

This patch accounts for the addition of DISTRO_FEATURES_INITMAN which sets the VIRTUAL_RUNTIME_init_manager, so that does not need to be set.

---
 meta-yocto/conf/distro/poky-tiny.conf |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
meta-yocto/conf/distro/poky-tiny.conf +2 -1
index 347280a..761844f 100644
@@ -84,6 +84,8 @@ DISTRO_FEATURES_NET = "ipv4"
 DISTRO_FEATURES_USB = "usbhost"
 #DISTRO_FEATURES_USBGADGET = "usbgadget"
 #DISTRO_FEATURES_WIFI = "wifi"
+# This is needed for DISTRO_FEATURES_BACKFILL
+DISTRO_FEATURES_INITMAN = "tiny-init"
 
 DISTRO_FEATURES = "${DISTRO_FEATURES_TINY} \
                    ${DISTRO_FEATURES_NET} \
@@ -96,7 +98,6 @@ DISTRO_FEATURES = "${DISTRO_FEATURES_TINY} \
 # Use tmpdevfs and the busybox runtime services
 VIRTUAL-RUNTIME_dev_manager = ""
 VIRTUAL-RUNTIME_login_manager = ""
-VIRTUAL-RUNTIME_init_manager = "tiny-init"
 VIRTUAL-RUNTIME_keymaps = ""
 
 # FIXME: Consider adding "modules" to MACHINE_FEATURES and using that in
Signature · 1 line
1.7.9.5
#1 lore raw
Quoted 4 lines · Saul Wold

This patch accounts for the addition of DISTRO_FEATURES_INITMAN which sets the VIRTUAL_RUNTIME_init_manager, so that does not need to be set.

I thought I understood it... until I read the new comment. What is this DISTRO_FEATURED_BACKFILL thing?

--
Darren

Quoted 26 lines
---
 meta-yocto/conf/distro/poky-tiny.conf |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
meta-yocto/conf/distro/poky-tiny.conf +2 -1
index 347280a..761844f 100644
@@ -84,6 +84,8 @@ DISTRO_FEATURES_NET = "ipv4"
 DISTRO_FEATURES_USB = "usbhost"
 #DISTRO_FEATURES_USBGADGET = "usbgadget"
 #DISTRO_FEATURES_WIFI = "wifi"
+# This is needed for DISTRO_FEATURES_BACKFILL
+DISTRO_FEATURES_INITMAN = "tiny-init"
 
 DISTRO_FEATURES = "${DISTRO_FEATURES_TINY} \
                    ${DISTRO_FEATURES_NET} \
@@ -96,7 +98,6 @@ DISTRO_FEATURES = "${DISTRO_FEATURES_TINY} \
 # Use tmpdevfs and the busybox runtime services
 VIRTUAL-RUNTIME_dev_manager = ""
 VIRTUAL-RUNTIME_login_manager = ""
-VIRTUAL-RUNTIME_init_manager = "tiny-init"
 VIRTUAL-RUNTIME_keymaps = ""
 
 # FIXME: Consider adding "modules" to MACHINE_FEATURES and using that in
Signature · 3 lines
Darren Hart
Intel Open Source Technology Center
Yocto Project - Technical Lead - Linux Kernel
author
#2 lore raw
Quoted 10 lines · Darren Hart
Quoted 4 lines · Saul Wold

This patch accounts for the addition of DISTRO_FEATURES_INITMAN which sets the VIRTUAL_RUNTIME_init_manager, so that does not need to be set.

I thought I understood it... until I read the new comment. What is this DISTRO_FEATURED_BACKFILL thing?

It ensure that items in the backfill show up in all DISTRO_FEATURES, think of it as a required core part of DISTRO_FEATURES.

See Git Rev: 72ce5c4cde612515451226a1b10f5b348a4a6a4c for a better explanation from Paul.

Re-reading this we should include pulseaudio in DISTRO_FEATURES_BACKFILL_CONSIDERED for poky-tiny as a separate patch

Sau!

Quoted 30 lines

--
Darren

Quoted 26 lines
---
  meta-yocto/conf/distro/poky-tiny.conf |    3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)
meta-yocto/conf/distro/poky-tiny.conf +2 -1
index 347280a..761844f 100644
@@ -84,6 +84,8 @@ DISTRO_FEATURES_NET = "ipv4"
  DISTRO_FEATURES_USB = "usbhost"
  #DISTRO_FEATURES_USBGADGET = "usbgadget"
  #DISTRO_FEATURES_WIFI = "wifi"
+# This is needed for DISTRO_FEATURES_BACKFILL
+DISTRO_FEATURES_INITMAN = "tiny-init"
 
  DISTRO_FEATURES = "${DISTRO_FEATURES_TINY} \
                     ${DISTRO_FEATURES_NET} \
@@ -96,7 +98,6 @@ DISTRO_FEATURES = "${DISTRO_FEATURES_TINY} \
  # Use tmpdevfs and the busybox runtime services
  VIRTUAL-RUNTIME_dev_manager = ""
  VIRTUAL-RUNTIME_login_manager = ""
-VIRTUAL-RUNTIME_init_manager = "tiny-init"
  VIRTUAL-RUNTIME_keymaps = ""
 
  # FIXME: Consider adding "modules" to MACHINE_FEATURES and using that in
#3 lore raw
Quoted 8 lines · Saul Wold
Quoted 7 lines · Darren Hart
Quoted 4 lines · Saul Wold

This patch accounts for the addition of DISTRO_FEATURES_INITMAN which sets the VIRTUAL_RUNTIME_init_manager, so that does not need to be set.

Quoted 9 lines
Quoted 3 lines

I thought I understood it... until I read the new comment. What is this DISTRO_FEATURED_BACKFILL thing?

It ensure that items in the backfill show up in all DISTRO_FEATURES, think of it as a required core part of DISTRO_FEATURES.

See Git Rev: 72ce5c4cde612515451226a1b10f5b348a4a6a4c for a better explanation from Paul.

OK, I see how it works, but I think that comment confuses more than it enlightens. I don't need to know anything about BACKFILL to understand the meaning of DISTRO_FEATURES_INITMAN. It clearly overrides the init manager used for images of this distro. I suggest dropping the Backfill comment.

As for the actual change, it looks good to me, and thanks for doing it. Does this remove the unwanted bash dependency?

--
Darren

Quoted 36 lines

Re-reading this we should include pulseaudio in DISTRO_FEATURES_BACKFILL_CONSIDERED for poky-tiny as a separate patch

Sau!

Quoted 30 lines

--
Darren

Quoted 26 lines
---
  meta-yocto/conf/distro/poky-tiny.conf |    3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)
meta-yocto/conf/distro/poky-tiny.conf +2 -1
index 347280a..761844f 100644
@@ -84,6 +84,8 @@ DISTRO_FEATURES_NET = "ipv4"
  DISTRO_FEATURES_USB = "usbhost"
  #DISTRO_FEATURES_USBGADGET = "usbgadget"
  #DISTRO_FEATURES_WIFI = "wifi"
+# This is needed for DISTRO_FEATURES_BACKFILL
+DISTRO_FEATURES_INITMAN = "tiny-init"
 
  DISTRO_FEATURES = "${DISTRO_FEATURES_TINY} \
                     ${DISTRO_FEATURES_NET} \
@@ -96,7 +98,6 @@ DISTRO_FEATURES = "${DISTRO_FEATURES_TINY} \
  # Use tmpdevfs and the busybox runtime services
  VIRTUAL-RUNTIME_dev_manager = ""
  VIRTUAL-RUNTIME_login_manager = ""
-VIRTUAL-RUNTIME_init_manager = "tiny-init"
  VIRTUAL-RUNTIME_keymaps = ""
 
  # FIXME: Consider adding "modules" to MACHINE_FEATURES and using that in
Signature · 3 lines
Darren Hart
Intel Open Source Technology Center
Yocto Project - Technical Lead - Linux Kernel