diff mbox series

dev-manual: wic.rst: Update native tools build command

Message ID 20230808084945.24457-1-dse@thaumatec.com
State New
Headers show
Series dev-manual: wic.rst: Update native tools build command | expand

Commit Message

Daniel Semkowicz Aug. 8, 2023, 8:49 a.m. UTC
Requirements list seems to be outdated. It is not possible to follow
instructions for Raw Mode as not all of the native tools are available.
All wic dependencies are gathered under wic-tools package. Some commands
in the instruction already use wic-tools native sysroot, but this
dependency is not specified in the requirements.
Update the command for building native tools to use wic-tools instead
of the seperate packages.

Signed-off-by: Daniel Semkowicz <dse@thaumatec.com>
---
 documentation/dev-manual/wic.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Michael Opdenacker Aug. 9, 2023, 5:36 p.m. UTC | #1
Hi Daniel

Many thanks for the patch!

On 08.08.23 at 10:49, Daniel Semkowicz wrote:
> Requirements list seems to be outdated. It is not possible to follow
> instructions for Raw Mode as not all of the native tools are available.
> All wic dependencies are gathered under wic-tools package. Some commands
> in the instruction already use wic-tools native sysroot, but this
> dependency is not specified in the requirements.
> Update the command for building native tools to use wic-tools instead
> of the seperate packages.
>
> Signed-off-by: Daniel Semkowicz <dse@thaumatec.com>
> ---
>   documentation/dev-manual/wic.rst | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/documentation/dev-manual/wic.rst b/documentation/dev-manual/wic.rst
> index 2a4408cdb..664f07a21 100644
> --- a/documentation/dev-manual/wic.rst
> +++ b/documentation/dev-manual/wic.rst
> @@ -92,7 +92,7 @@ system needs to meet the following requirements:
>   -  You must build several native tools, which are built to run on the
>      build system::
>   
> -      $ bitbake parted-native dosfstools-native mtools-native
> +      $ bitbake wic-tools
>   
>   -  Include "wic" as part of the
>      :term:`IMAGE_FSTYPES`


Reviewed-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
... and merged into master-next in yocto-docs.

Do you think the change also applies to Mickledore and Kirkstone?
Thanks again
Cheers
Michael.
Daniel Semkowicz Aug. 10, 2023, 7:01 a.m. UTC | #2
Hi Michael,


On Wed, Aug 9, 2023 at 7:36 PM Michael Opdenacker
<michael.opdenacker@bootlin.com> wrote:
>
> Hi Daniel
>
> Many thanks for the patch!
>
> On 08.08.23 at 10:49, Daniel Semkowicz wrote:
> > Requirements list seems to be outdated. It is not possible to follow
> > instructions for Raw Mode as not all of the native tools are available.
> > All wic dependencies are gathered under wic-tools package. Some commands
> > in the instruction already use wic-tools native sysroot, but this
> > dependency is not specified in the requirements.
> > Update the command for building native tools to use wic-tools instead
> > of the seperate packages.
> >
> > Signed-off-by: Daniel Semkowicz <dse@thaumatec.com>
> > ---
> >   documentation/dev-manual/wic.rst | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/documentation/dev-manual/wic.rst b/documentation/dev-manual/wic.rst
> > index 2a4408cdb..664f07a21 100644
> > --- a/documentation/dev-manual/wic.rst
> > +++ b/documentation/dev-manual/wic.rst
> > @@ -92,7 +92,7 @@ system needs to meet the following requirements:
> >   -  You must build several native tools, which are built to run on the
> >      build system::
> >
> > -      $ bitbake parted-native dosfstools-native mtools-native
> > +      $ bitbake wic-tools
> >
> >   -  Include "wic" as part of the
> >      :term:`IMAGE_FSTYPES`
>
>
> Reviewed-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
> ... and merged into master-next in yocto-docs.
>
> Do you think the change also applies to Mickledore and Kirkstone?

For sure, it also applies to Kirkstone.
I was not testing with Mickledore, but if this problem was already
present in Kirkstone, then I would be surprised if on Mickledore
it works correctly.

Best regards
Daniel

> Thanks again
> Cheers
> Michael.
> --
>
> Michael Opdenacker, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>
Michael Opdenacker Aug. 10, 2023, 7:35 a.m. UTC | #3
Hi Daniel,

On 10.08.23 at 09:01, Daniel Semkowicz wrote:
>
> For sure, it also applies to Kirkstone.
> I was not testing with Mickledore, but if this problem was already
> present in Kirkstone, then I would be surprised if on Mickledore
> it works correctly.


Right, thanks!
Michael.
diff mbox series

Patch

diff --git a/documentation/dev-manual/wic.rst b/documentation/dev-manual/wic.rst
index 2a4408cdb..664f07a21 100644
--- a/documentation/dev-manual/wic.rst
+++ b/documentation/dev-manual/wic.rst
@@ -92,7 +92,7 @@  system needs to meet the following requirements:
 -  You must build several native tools, which are built to run on the
    build system::
 
-      $ bitbake parted-native dosfstools-native mtools-native
+      $ bitbake wic-tools
 
 -  Include "wic" as part of the
    :term:`IMAGE_FSTYPES`