Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Tom Zanussi <tom.zanussi@linux.intel.com>
To: "João Henrique Ferreira de Freitas" <joaohf@gmail.com>
Cc: Otavio Salvador <otavio@ossystems.com.br>,
	Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 0/5] wic: Add --rootfs option to --source param
Date: Mon, 31 Mar 2014 09:39:52 -0500	[thread overview]
Message-ID: <1396276792.11878.24.camel@empanada> (raw)
In-Reply-To: <5338CA5F.2080104@gmail.com>

On Sun, 2014-03-30 at 22:52 -0300, João Henrique Ferreira de Freitas
wrote:
> Hi Tom,
> 
> Let's back in this context. This is really important and I was waiting 
> to complete the previous stage.
> 
> Em 17-03-2014 13:11, Otavio Salvador escreveu:
> >>>
> >>> If my understand is right, I like the feature. My only concern is
> >>> people overusing it and adding contents which are not 'tracked' in the
> >>> build system in a product release which seems attractive when we first
> >>> think about it but cause some management, tracking and authenticity
> >>> check problems in long term.
> >>>
> >>>
> >>> I don't know how to better address this from wic perspective. Usually
> >>> we end doing multiple images as part of the build process for those
> >>> special cases and I don't know how wic could be 'told' about those
> >>> secondary rootfs existence.
> >>>
> 
> I have been working on it since Otavio's concerns about 'contents which 
> are not tracked'.
> 
> So extending my previous patches (80% done) to handle this situation is 
> quite easy. Like this:
> 
> bitbake directdisk-multi-image-e img1=core-image-minimal -e 
> img2=core-image-minimal -e img3=core-image-minimal
> 
> directdisk-multi-image.wks:
> 
> part /boot --source bootimg-pcbios --ondisk sda --fstype=msdos --label 
> boot --active --align 1024
> 
> part / --source rootfs --image-name=img1 --ondisk sda --fstype=ext3 
> --label primary --align 1024
> 
> part /standby --source rootfs --image-name=img2 --ondisk sda 
> --fstype=ext3 --label secondary --align 1024
> 
> part /root --source rootfs --image-name=img3 --ondisk sda --fstype=ext3 
> --label root_sec --align 1024
> 
> 
> If the user put '--image-name' and '--rootfs-dir' the '--image-name' 
> takes precedence.
> 
> As wic is a generic tool, the user could prefer to use images from OE or 
> any other rootfs-dir.
> 
> Tom, what do you think? Could I go ahead?
> 

So is the idea that you want to allow the user the convenience of being
able to use the equivalent of '-e imagename' for any partition, rather
than having to explicitly specify the full path?

'-e imagename' was always meant as just a convenience to the user, since
currently the easiest way to generate the artifacts is to first create
an oe image.  If you think about it, having to create an oe image or
images in order to create another (wic-generated) image doesn't really
make a lot of sense - it's basically just a temporary situation pending
better integration, etc.  So for that reason, I wouldn't want to see the
idea of an --image-name incorporated into the image-creation .wks files.

But I think you could accomplish the same thing by just allowing the
indirect string e.g. rootfs2 to resolve to either a full path, or as an
image name, which would be treated as an instance of '-e image-name' for
that partition.

For example, we have the unmodified parttion in the .wks file as usual:

  part /standby --source rootfs --rootfs-dir=rootfs2 --ondisk sda --fstype=ext3 --label secondary --align 1024

Which could be resolved as either a full path to the rootfs dir:

  wic create directdisk-multi-indirect-both --rootfs-dir rootfs2=/home/trz/yocto/master-cur/build/tmp/work/crownbay-poky-linux/core-image-minimal

Or extracted from the '-e' ROOTFS_DIR output from the 'core-image-minimal' image:

  wic create directdisk-multi-indirect-both --rootfs-dir rootfs2=core-image-minimal

Does that make sense for this?

Tom


> Thanks
> 




  reply	other threads:[~2014-03-31 14:40 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-15 21:17 [PATCH 0/5] wic: Add --rootfs option to --source param João Henrique Ferreira de Freitas
2014-03-15 21:17 ` [PATCH 1/5] wic: Add RootfsPlugin João Henrique Ferreira de Freitas
2014-03-15 21:17 ` [PATCH 2/5] wic: Hook up RootfsPlugin plugin João Henrique Ferreira de Freitas
2014-03-15 21:17 ` [PATCH 3/5] wic: Add rootfs_dir argument to do_prepare_partition() method João Henrique Ferreira de Freitas
2014-03-15 21:17 ` [PATCH 4/5] wic: Use partition label to be part of rootfs filename João Henrique Ferreira de Freitas
2014-03-15 21:17 ` [PATCH 5/5] wic: Add option --rootfs to --source João Henrique Ferreira de Freitas
2014-03-17 14:53 ` [PATCH 0/5] wic: Add --rootfs option to --source param Otavio Salvador
2014-03-17 15:47   ` João Henrique Freitas
2014-03-17 16:11     ` Otavio Salvador
2014-03-17 16:20       ` João Henrique Freitas
2014-03-31  1:52       ` João Henrique Ferreira de Freitas
2014-03-31 14:39         ` Tom Zanussi [this message]
2014-03-31 16:29           ` João Henrique Freitas
2014-03-21 15:54 ` Tom Zanussi
2014-03-23  2:25   ` João Henrique Ferreira de Freitas
2014-03-24 20:13     ` Tom Zanussi
2014-03-25  2:28       ` João Henrique Ferreira de Freitas
2014-03-26  2:42 ` [PATCH v2 0/7] " João Henrique Ferreira de Freitas
2014-03-26  2:42   ` [PATCH v2 1/7] wic: Add RootfsPlugin João Henrique Ferreira de Freitas
2014-03-26  2:42   ` [PATCH v2 2/7] wic: Hook up RootfsPlugin plugin João Henrique Ferreira de Freitas
2014-03-26  2:42   ` [PATCH v2 3/7] wic: Add rootfs_dir argument to do_prepare_partition() method João Henrique Ferreira de Freitas
2014-03-26  2:42   ` [PATCH v2 4/7] wic: Use partition label to be part of rootfs filename João Henrique Ferreira de Freitas
2014-03-26  2:42   ` [PATCH v2 5/7] wic: Add option --rootfs-dir to --source João Henrique Ferreira de Freitas
2014-03-26  2:42   ` [PATCH v2 6/7] wic: Report all ROOTFS_DIR artifacts João Henrique Ferreira de Freitas
2014-03-26  2:42   ` [PATCH v2 7/7] wic: Extend --rootfs-dir to connect rootfs-dirs João Henrique Ferreira de Freitas
2014-03-27 20:15   ` [PATCH v2 0/7] wic: Add --rootfs option to --source param Tom Zanussi
2014-03-27 22:12     ` João Henrique Ferreira de Freitas
2014-03-27 22:07 ` [PATCH v3 " João Henrique Ferreira de Freitas
2014-03-27 22:07   ` [PATCH v3 1/7] wic: Add RootfsPlugin João Henrique Ferreira de Freitas
2014-03-27 22:07   ` [PATCH v3 2/7] wic: Hook up RootfsPlugin plugin João Henrique Ferreira de Freitas
2014-03-27 22:07   ` [PATCH v3 3/7] wic: Add rootfs_dir argument to do_prepare_partition() method João Henrique Ferreira de Freitas
2014-03-27 22:07   ` [PATCH v3 4/7] wic: Use partition label to be part of rootfs filename João Henrique Ferreira de Freitas
2014-03-27 22:07   ` [PATCH v3 5/7] wic: Add option --rootfs-dir to --source João Henrique Ferreira de Freitas
2014-03-27 22:07   ` [PATCH v3 6/7] wic: Report all ROOTFS_DIR artifacts João Henrique Ferreira de Freitas
2014-03-27 22:07   ` [PATCH v3 7/7] wic: Extend --rootfs-dir to connect rootfs-dirs João Henrique Ferreira de Freitas
2014-03-28 21:38   ` [PATCH v3 0/7] wic: Add --rootfs option to --source param Tom Zanussi
2014-03-29  3:24     ` João Henrique Ferreira de Freitas
2014-03-29 19:04       ` Tom Zanussi
2014-03-29  3:12 ` [PATCH v4 " João Henrique Ferreira de Freitas
2014-03-29  3:12   ` [PATCH v4 1/7] wic: Add RootfsPlugin João Henrique Ferreira de Freitas
2014-03-29  3:12   ` [PATCH v4 2/7] wic: Hook up RootfsPlugin plugin João Henrique Ferreira de Freitas
2014-03-29  3:12   ` [PATCH v4 3/7] wic: Add rootfs_dir argument to do_prepare_partition() method João Henrique Ferreira de Freitas
2014-03-29  3:12   ` [PATCH v4 4/7] wic: Use partition label to be part of rootfs filename João Henrique Ferreira de Freitas
2014-03-29  3:12   ` [PATCH v4 5/7] wic: Add option --rootfs-dir to --source João Henrique Ferreira de Freitas
2014-03-29  3:12   ` [PATCH v4 6/7] wic: Report all ROOTFS_DIR artifacts João Henrique Ferreira de Freitas
2014-03-29  3:12   ` [PATCH v4 7/7] wic: Extend --rootfs-dir to connect rootfs-dirs João Henrique Ferreira de Freitas
2014-03-29 19:09   ` [PATCH v4 0/7] wic: Add --rootfs option to --source param Tom Zanussi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1396276792.11878.24.camel@empanada \
    --to=tom.zanussi@linux.intel.com \
    --cc=joaohf@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=otavio@ossystems.com.br \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox