Re: [PATCH V8 5/6] volatile-conf-minimal: add recipe

[Saul Wold <sgw@linux.intel.com>]

On 01/21/2013 10:36 PM, Qi.Chen@windriver.com wrote:
> From: Chen Qi <Qi.Chen@windriver.com>
>
> This recipe provides a config file for core-image-minimal.
> This config file is used for volatile storage handling in case of a
> readonly rootfs.
>
> Also, let core-image-minimal depend on this recipe.
>
> [YOCTO #3406]
>
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> ---
>   meta/recipes-core/images/core-image-minimal.bb     |    3 +++
>   .../files/volatiles-readonly-minimal               |    8 ++++++++
>   .../volatile-conf-minimal_1.0.bb                   |   15 +++++++++++++++
>   3 files changed, 26 insertions(+)
>   create mode 100644 meta/recipes-core/readonly-volatile-conf/files/volatiles-readonly-minimal
>   create mode 100644 meta/recipes-core/readonly-volatile-conf/volatile-conf-minimal_1.0.bb
>
> diff --git a/meta/recipes-core/images/core-image-minimal.bb b/meta/recipes-core/images/core-image-minimal.bb
> index 0d2613e..f6baeae 100644
> --- a/meta/recipes-core/images/core-image-minimal.bb
> +++ b/meta/recipes-core/images/core-image-minimal.bb
> @@ -6,6 +6,9 @@ IMAGE_LINGUAS = " "
>
>   LICENSE = "MIT"
>
> +RDEPENDS += "volatile-conf-minimal"
This needs to be RDEPENDS_${PN}

> +IMAGE_INSTALL += "volatile-conf-minimal"
> +
I am still digging into this idea of having a image specific volatile 
package, RP is not wild about it either.  What's different now from 
before is the minimal version cut down from the orignal version?  Is 
there any reason we can't use a larger volatile conf?

Sau!

>   inherit core-image
>
>   IMAGE_ROOTFS_SIZE = "8192"
> diff --git a/meta/recipes-core/readonly-volatile-conf/files/volatiles-readonly-minimal b/meta/recipes-core/readonly-volatile-conf/files/volatiles-readonly-minimal
> new file mode 100644
> index 0000000..b7146a2
> --- /dev/null
> +++ b/meta/recipes-core/readonly-volatile-conf/files/volatiles-readonly-minimal
> @@ -0,0 +1,8 @@
> +# This configuration file lists filesystem objects specific to minimal image
> +# with read-only rootfs.
> +# This configuration file will only be applied if the image is minimal and the
> +# rootfs is read-only.
> +# For the detailed format information, refer to /etc/default/volatiles/00_core.
> +d root root 0755 /var/volatile/lib/ none
> +d root root 0755 /var/volatile/lib/urandom/ none
> +l root root 0755 /var/lib/urandom /var/volatile/lib/urandom
> diff --git a/meta/recipes-core/readonly-volatile-conf/volatile-conf-minimal_1.0.bb b/meta/recipes-core/readonly-volatile-conf/volatile-conf-minimal_1.0.bb
> new file mode 100644
> index 0000000..c5744e1
> --- /dev/null
> +++ b/meta/recipes-core/readonly-volatile-conf/volatile-conf-minimal_1.0.bb
> @@ -0,0 +1,15 @@
> +SUMMARY = "Provide config files for core-image-minimal"
> +DESCRIPTION = "This package contains a config file for core-image-minimal. \
> +It is used to handle volatile storage in case of a read-only rootfs."
> +
> +LICENSE = "MIT"
> +LIC_FILES_CHKSUM = "file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420"
> +
> +SRC_URI = "file://volatiles-readonly-minimal"
> +
> +PR = "r1"
> +
> +do_install () {
> +	install -d ${D}${sysconfdir}/default/volatiles
> +	install -m 0644 ${WORKDIR}/volatiles-readonly-minimal ${D}${sysconfdir}/default/volatiles/00_core_readonly
> +}
>