Re: [OE-core] [PATCH 0/6] wic: fix for #11017

2017-02-23 Thread Patrick Ohly
On Tue, 2017-02-21 at 17:23 +0200, Ed Bartosh wrote:
> This patchset improves handling of wic native tool dependencies and
> fixes minor bugs in the wic code.

I haven't tested it, but the patches themselves look good to me. Thanks!

-- 
Best Regards, Patrick Ohly

The content of this message is my personal opinion only and although
I am an employee of Intel, the statements I make here in no way
represent Intel's position on the issue, nor am I authorized to speak
on behalf of Intel on this matter.



-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 0/6] wic: fix for #11017

2017-02-23 Thread Ed Bartosh
On Thu, Feb 23, 2017 at 12:18:04PM +0200, Ed Bartosh wrote:
> On Wed, Feb 22, 2017 at 10:53:35PM +0200, Ed Bartosh wrote:
> > On Wed, Feb 22, 2017 at 04:38:55PM +, Burton, Ross wrote:
> > > Hi Ed,
> > > 
> > > Something in your latest patchset is breaking selftest:
> > > 
> > > https://autobuilder.yoctoproject.org/main/builders/nightly-wic/builds/626/steps/CreateWicImages/logs/stdio
> > > 
> > > The broken commit is somewhere in ross/mut-wic.  Can you verify that I
> > > didn't leave something out, or otherwise chase this?
> > > 
> > I couldn't reproduce this so far. Will investigate further.
> > 
> > Can you merge this patchset please?
> > http://lists.openembedded.org/pipermail/openembedded-core/2017-February/132854.html
> > 
> > The failure is most probably caused by patchset from this thread.
> > 
> 
> Still not able to reproduce it. It works just fine in my environment.
> This time I used setup as close to AB as I could get: core-image-sato
> image, genericx86-64 machine, clean build.
> 
> [build-genericx86-64 (ed/wic/wip)]$ rm -rf tmp/
> 
> [build-genericx86-64 (ed/wic/wip)]$ bitbake core-image-sato
> WARNING: Host distribution "opensuse-13.2" has not been validated with
> this version of the build system; you may possibly experience unexpected
> failures. It is recommended that you use a tested distribution.
> Parsing recipes: 100%
> |###|
> Time: 0:00:13
> Parsing of 856 .bb files complete (0 cached, 856 parsed). 1322 targets,
> 53 skipped, 0 masked, 0 errors.
> NOTE: Resolving any missing task queue dependencies
> 
> Build Configuration:
> BB_VERSION= "1.33.1"
> BUILD_SYS = "x86_64-linux"
> NATIVELSBSTRING   = "opensuse-13.2"
> TARGET_SYS= "x86_64-poky-linux"
> MACHINE   = "genericx86-64"
> DISTRO= "poky"
> DISTRO_VERSION= "2.2+snapshot-20170222"
> TUNE_FEATURES = "m64 core2"
> TARGET_FPU= ""
> meta
> meta-poky
> meta-yocto-bsp=
> "ed/wic/wip:5d96bc6d1af89dda11f3266548fe1547152c86e1"
> 
> Initialising tasks: 100%
> ||
> Time: 0:00:15
> NOTE: Executing SetScene Tasks
> NOTE: Executing RunQueue Tasks
> NOTE: Tasks Summary: Attempted 6116 tasks of which 2807 didn't need to
> be rerun and all succeeded.
> 
> Summary: There was 1 WARNING message shown.
> 
> [build-genericx86-64 (ed/wic/wip)]$ wic create directdisk -e core-image-sato
> INFO: Creating image(s)...
> 
> INFO: The new image(s) can be found here:
>   ./directdisk-201702231207-sda.direct
> 
> The following build artifacts were used to create the image(s):
>   ROOTFS_DIR:
> /home/ed/git/yocto/poky/build-genericx86-64/tmp/work/genericx86_64-poky-linux/core-image-sato/1.0-r0/rootfs
>   BOOTIMG_DIR:
> /home/ed/git/yocto/poky/build-genericx86-64/tmp/work/core2-64-poky-linux/wic-tools/1.0-r0/recipe-sysroot/usr/share
>   KERNEL_DIR:
> /home/ed/git/yocto/poky/build-genericx86-64/tmp/deploy/images/genericx86-64
>   NATIVE_SYSROOT:
> /home/ed/git/yocto/poky/build-genericx86-64/tmp/work/genericx86_64-poky-linux/core-image-sato/1.0-r0/recipe-sysroot-native
> 
> INFO: The image(s) were created using OE kickstart file:
>   /home/ed/git/yocto/poky/scripts/lib/wic/canned-wks/directdisk.wks
> 
> 
> Could it be because of something in AB setup?
> 

Now I'm pretty sure it's because of AB setup.

I was not able to reproduce it as I used genericx86-64 machine mentioned
on the build page:
https://autobuilder.yoctoproject.org/main/builders/nightly-wic/builds/623

However, AB switched to qemux86-64 machine despite of what's specified.
This is very confusing.

Anyway, the reason of the failure is that AB builds set of tools for wic
in step 12: BuildImages Building syslinux syslinux-native parted-native
gptfdisk-native dosfstools-native mtools-native, but due to RSS those
tools are not accessible by wic. AB should build wic-tools instead.

Regards,
Ed

> > > On 21 February 2017 at 15:23, Ed Bartosh  
> > > wrote:
> > > 
> > > > Hi,
> > > >
> > > > This patchset improves handling of wic native tool dependencies and
> > > > fixes minor bugs in the wic code.
> > > >
> > > > The following changes since commit 9a0ddad5e178d16f302af2d05c3236
> > > > 8572849fd3:
> > > >
> > > >   wic: pluginbase: use global dictionary (2017-02-20 11:58:28 +0200)
> > > >
> > > > are available in the git repository at:
> > > >
> > > >   git://git.yoctoproject.org/poky-contrib ed/wic/wip
> > > >   http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/wic/wip
> > > >
> > > > Ed Bartosh (6):
> > > >   wic: use 2 sysroots to find native executable
> > > >   wic: use image recipe sysroot as default
> > > >   wic: exec_native_cmd: improve debug message
> > > >   wic: exec_native_cmd: fix undefined variable error
> > > >   

Re: [OE-core] [PATCH 0/6] wic: fix for #11017

2017-02-23 Thread Ed Bartosh
On Wed, Feb 22, 2017 at 10:53:35PM +0200, Ed Bartosh wrote:
> On Wed, Feb 22, 2017 at 04:38:55PM +, Burton, Ross wrote:
> > Hi Ed,
> > 
> > Something in your latest patchset is breaking selftest:
> > 
> > https://autobuilder.yoctoproject.org/main/builders/nightly-wic/builds/626/steps/CreateWicImages/logs/stdio
> > 
> > The broken commit is somewhere in ross/mut-wic.  Can you verify that I
> > didn't leave something out, or otherwise chase this?
> > 
> I couldn't reproduce this so far. Will investigate further.
> 
> Can you merge this patchset please?
> http://lists.openembedded.org/pipermail/openembedded-core/2017-February/132854.html
> 
> The failure is most probably caused by patchset from this thread.
> 

Still not able to reproduce it. It works just fine in my environment.
This time I used setup as close to AB as I could get: core-image-sato
image, genericx86-64 machine, clean build.

[build-genericx86-64 (ed/wic/wip)]$ rm -rf tmp/

[build-genericx86-64 (ed/wic/wip)]$ bitbake core-image-sato
WARNING: Host distribution "opensuse-13.2" has not been validated with
this version of the build system; you may possibly experience unexpected
failures. It is recommended that you use a tested distribution.
Parsing recipes: 100%
|###|
Time: 0:00:13
Parsing of 856 .bb files complete (0 cached, 856 parsed). 1322 targets,
53 skipped, 0 masked, 0 errors.
NOTE: Resolving any missing task queue dependencies

Build Configuration:
BB_VERSION= "1.33.1"
BUILD_SYS = "x86_64-linux"
NATIVELSBSTRING   = "opensuse-13.2"
TARGET_SYS= "x86_64-poky-linux"
MACHINE   = "genericx86-64"
DISTRO= "poky"
DISTRO_VERSION= "2.2+snapshot-20170222"
TUNE_FEATURES = "m64 core2"
TARGET_FPU= ""
meta
meta-poky
meta-yocto-bsp=
"ed/wic/wip:5d96bc6d1af89dda11f3266548fe1547152c86e1"

Initialising tasks: 100%
||
Time: 0:00:15
NOTE: Executing SetScene Tasks
NOTE: Executing RunQueue Tasks
NOTE: Tasks Summary: Attempted 6116 tasks of which 2807 didn't need to
be rerun and all succeeded.

Summary: There was 1 WARNING message shown.

[build-genericx86-64 (ed/wic/wip)]$ wic create directdisk -e core-image-sato
INFO: Creating image(s)...

INFO: The new image(s) can be found here:
  ./directdisk-201702231207-sda.direct

The following build artifacts were used to create the image(s):
  ROOTFS_DIR:
/home/ed/git/yocto/poky/build-genericx86-64/tmp/work/genericx86_64-poky-linux/core-image-sato/1.0-r0/rootfs
  BOOTIMG_DIR:
/home/ed/git/yocto/poky/build-genericx86-64/tmp/work/core2-64-poky-linux/wic-tools/1.0-r0/recipe-sysroot/usr/share
  KERNEL_DIR:
/home/ed/git/yocto/poky/build-genericx86-64/tmp/deploy/images/genericx86-64
  NATIVE_SYSROOT:
/home/ed/git/yocto/poky/build-genericx86-64/tmp/work/genericx86_64-poky-linux/core-image-sato/1.0-r0/recipe-sysroot-native

INFO: The image(s) were created using OE kickstart file:
  /home/ed/git/yocto/poky/scripts/lib/wic/canned-wks/directdisk.wks


Could it be because of something in AB setup?

Regards,
Ed

> > On 21 February 2017 at 15:23, Ed Bartosh  wrote:
> > 
> > > Hi,
> > >
> > > This patchset improves handling of wic native tool dependencies and
> > > fixes minor bugs in the wic code.
> > >
> > > The following changes since commit 9a0ddad5e178d16f302af2d05c3236
> > > 8572849fd3:
> > >
> > >   wic: pluginbase: use global dictionary (2017-02-20 11:58:28 +0200)
> > >
> > > are available in the git repository at:
> > >
> > >   git://git.yoctoproject.org/poky-contrib ed/wic/wip
> > >   http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/wic/wip
> > >
> > > Ed Bartosh (6):
> > >   wic: use 2 sysroots to find native executable
> > >   wic: use image recipe sysroot as default
> > >   wic: exec_native_cmd: improve debug message
> > >   wic: exec_native_cmd: fix undefined variable error
> > >   wic: add WKS_FILE_DEPENDS variable
> > >   wic: raise WicError instead of calling logger.error
> > >
> > >  meta-selftest/recipes-test/images/wic-image-minimal.bb |  2 +-
> > >  meta/classes/image_types_wic.bbclass   |  2 ++
> > >  scripts/lib/wic/plugins/source/bootimg-pcbios.py   |  4 ++--
> > >  scripts/lib/wic/utils/misc.py  | 13 +
> > >  scripts/wic|  2 +-
> > >  5 files changed, 15 insertions(+), 8 deletions(-)
> > >
> > > --
> > > Regards,
> > > Ed
> > >
> > > --
> > > ___
> > > Openembedded-core mailing list
> > > Openembedded-core@lists.openembedded.org
> > > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> > >
> 
> -- 
> --
> Regards,
> Ed
> -- 
> ___

Re: [OE-core] [PATCH 0/6] wic: fix for #11017

2017-02-22 Thread Ed Bartosh
On Wed, Feb 22, 2017 at 04:38:55PM +, Burton, Ross wrote:
> Hi Ed,
> 
> Something in your latest patchset is breaking selftest:
> 
> https://autobuilder.yoctoproject.org/main/builders/nightly-wic/builds/626/steps/CreateWicImages/logs/stdio
> 
> The broken commit is somewhere in ross/mut-wic.  Can you verify that I
> didn't leave something out, or otherwise chase this?
> 
I couldn't reproduce this so far. Will investigate further.

Can you merge this patchset please?
http://lists.openembedded.org/pipermail/openembedded-core/2017-February/132854.html

The failure is most probably caused by patchset from this thread.

Regards,
Ed

> On 21 February 2017 at 15:23, Ed Bartosh  wrote:
> 
> > Hi,
> >
> > This patchset improves handling of wic native tool dependencies and
> > fixes minor bugs in the wic code.
> >
> > The following changes since commit 9a0ddad5e178d16f302af2d05c3236
> > 8572849fd3:
> >
> >   wic: pluginbase: use global dictionary (2017-02-20 11:58:28 +0200)
> >
> > are available in the git repository at:
> >
> >   git://git.yoctoproject.org/poky-contrib ed/wic/wip
> >   http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/wic/wip
> >
> > Ed Bartosh (6):
> >   wic: use 2 sysroots to find native executable
> >   wic: use image recipe sysroot as default
> >   wic: exec_native_cmd: improve debug message
> >   wic: exec_native_cmd: fix undefined variable error
> >   wic: add WKS_FILE_DEPENDS variable
> >   wic: raise WicError instead of calling logger.error
> >
> >  meta-selftest/recipes-test/images/wic-image-minimal.bb |  2 +-
> >  meta/classes/image_types_wic.bbclass   |  2 ++
> >  scripts/lib/wic/plugins/source/bootimg-pcbios.py   |  4 ++--
> >  scripts/lib/wic/utils/misc.py  | 13 +
> >  scripts/wic|  2 +-
> >  5 files changed, 15 insertions(+), 8 deletions(-)
> >
> > --
> > Regards,
> > Ed
> >
> > --
> > ___
> > Openembedded-core mailing list
> > Openembedded-core@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-core
> >

-- 
--
Regards,
Ed
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 0/6] wic: fix for #11017

2017-02-22 Thread Burton, Ross
Hi Ed,

Something in your latest patchset is breaking selftest:

https://autobuilder.yoctoproject.org/main/builders/nightly-wic/builds/626/steps/CreateWicImages/logs/stdio

The broken commit is somewhere in ross/mut-wic.  Can you verify that I
didn't leave something out, or otherwise chase this?

Ross

On 21 February 2017 at 15:23, Ed Bartosh  wrote:

> Hi,
>
> This patchset improves handling of wic native tool dependencies and
> fixes minor bugs in the wic code.
>
> The following changes since commit 9a0ddad5e178d16f302af2d05c3236
> 8572849fd3:
>
>   wic: pluginbase: use global dictionary (2017-02-20 11:58:28 +0200)
>
> are available in the git repository at:
>
>   git://git.yoctoproject.org/poky-contrib ed/wic/wip
>   http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/wic/wip
>
> Ed Bartosh (6):
>   wic: use 2 sysroots to find native executable
>   wic: use image recipe sysroot as default
>   wic: exec_native_cmd: improve debug message
>   wic: exec_native_cmd: fix undefined variable error
>   wic: add WKS_FILE_DEPENDS variable
>   wic: raise WicError instead of calling logger.error
>
>  meta-selftest/recipes-test/images/wic-image-minimal.bb |  2 +-
>  meta/classes/image_types_wic.bbclass   |  2 ++
>  scripts/lib/wic/plugins/source/bootimg-pcbios.py   |  4 ++--
>  scripts/lib/wic/utils/misc.py  | 13 +
>  scripts/wic|  2 +-
>  5 files changed, 15 insertions(+), 8 deletions(-)
>
> --
> Regards,
> Ed
>
> --
> ___
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH 0/6] wic: fix for #11017

2017-02-21 Thread Ed Bartosh
Hi,

This patchset improves handling of wic native tool dependencies and
fixes minor bugs in the wic code.

The following changes since commit 9a0ddad5e178d16f302af2d05c32368572849fd3:

  wic: pluginbase: use global dictionary (2017-02-20 11:58:28 +0200)

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib ed/wic/wip
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=ed/wic/wip

Ed Bartosh (6):
  wic: use 2 sysroots to find native executable
  wic: use image recipe sysroot as default
  wic: exec_native_cmd: improve debug message
  wic: exec_native_cmd: fix undefined variable error
  wic: add WKS_FILE_DEPENDS variable
  wic: raise WicError instead of calling logger.error

 meta-selftest/recipes-test/images/wic-image-minimal.bb |  2 +-
 meta/classes/image_types_wic.bbclass   |  2 ++
 scripts/lib/wic/plugins/source/bootimg-pcbios.py   |  4 ++--
 scripts/lib/wic/utils/misc.py  | 13 +
 scripts/wic|  2 +-
 5 files changed, 15 insertions(+), 8 deletions(-)

--
Regards,
Ed

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core