[oe] [meta-oe][PATCH 00/14] new recipe: open-vm-tools recipe

Martin Jansa martin.jansa at gmail.com
Sun Jun 18 20:28:48 UTC 2017


And there is also this:

=== Incorrect PACKAGE_ARCH or sstate signatures (3) ===

Complete log:
http://logs.nslu2-linux.org/buildlogs/oe/world/rocko/log.signatures.20170618_014055.log/
    * ERROR: 3 issues were found in these recipes: open-vm-tools

    * ERROR: 1 errors found in signatures.qemux86.do_configure.sigdata.log
    * ERROR: 1 errors found in
signatures.qemux86.do_populate_sysroot.sigdata.log
    * ERROR: 1 errors found in
signatures.qemux86.do_package_write_ipk.sigdata.log

    * ERROR: open-vm-tools different signature for task
do_configure.sigdata between qemux86copy and qemux86
    * ERROR: 1 errors found in
/home/jenkins/oe/world/shr-core/tmp-glibc/sstate-diff/1497786055/signatures.qemux86.do_configure.sigdata.log
    * ERROR: open-vm-tools different signature for task
do_populate_sysroot.sigdata between qemux86copy and qemux86
    * ERROR: 1 errors found in
/home/jenkins/oe/world/shr-core/tmp-glibc/sstate-diff/1497786055/signatures.qemux86.do_populate_sysroot.sigdata.log
    * ERROR: open-vm-tools different signature for task
do_package_write_ipk.sigdata between qemux86copy and qemux86
    * ERROR: 1 errors found in
/home/jenkins/oe/world/shr-core/tmp-glibc/sstate-diff/1497786055/signatures.qemux86.do_package_write_ipk.sigdata.log
    * ERROR: 3 issues were found in these recipes: open-vm-tools

So it probably depends on some MACHINE specific variable without being
marked as MACHINE_ARCH.

Looking at:
https://github.com/XevoInc/meta-openembedded/blob/402d1b3917195beb2ab7cac5dc94aaed8992b5a1/meta-oe/recipes-support/open-vm-tools/open-vm-tools_10.1.5.bb

It's depending on kernel, so it needs to be MACHINE_ARCH, also I've noticed
few more issues:

DEPENDS is there twice:
https://github.com/XevoInc/meta-openembedded/blob/402d1b3917195beb2ab7cac5dc94aaed8992b5a1/meta-oe/recipes-support/open-vm-tools/open-vm-tools_10.1.5.bb#L24
https://github.com/XevoInc/meta-openembedded/blob/402d1b3917195beb2ab7cac5dc94aaed8992b5a1/meta-oe/recipes-support/open-vm-tools/open-vm-tools_10.1.5.bb#L39

What is supposed to set KERNEL_VERSION variable in:
https://github.com/XevoInc/meta-openembedded/blob/402d1b3917195beb2ab7cac5dc94aaed8992b5a1/meta-oe/recipes-support/open-vm-tools/open-vm-tools_10.1.5.bb#L58
?

Don't use tags in SRCREV, use git hash instead (otherwise bitbake fetcher
will reach upstream git repo every single time the recipe is parsed and
will cause failed builds when github isn't accessible).
https://github.com/XevoInc/meta-openembedded/blob/402d1b3917195beb2ab7cac5dc94aaed8992b5a1/meta-oe/recipes-support/open-vm-tools/open-vm-tools_10.1.5.bb#L35

There are many hardcoded paths like /usr/lib/ or /usr/sbin/, please use
proper variables like libdir, sbindir.

Once you fix these issues please resend all changes to the ML, it's much
better to review inline than this.

Regards,


On Sun, Jun 18, 2017 at 9:58 PM, Martin Jansa <martin.jansa at gmail.com>
wrote:

> Fails to configure on qemuarm.
>
> If arm isn't expected to be supported, can you please update
> COMPATIBLE_HOST restriction?
>
> http://errors.yoctoproject.org/Errors/Details/145599/
>
> On Thu, Jun 15, 2017 at 7:50 PM, Martin Jansa <martin.jansa at gmail.com>
> wrote:
>
>> OK, I'll try to remember to fix it before merge.
>>
>> I'm sorry about it, tweaking other people commits always strikes back :).
>>
>> On Thu, Jun 15, 2017 at 7:40 PM, Martin Kelly <mkelly at xevo.com> wrote:
>>
>>> On 06/15/2017 09:54 AM, Martin Jansa wrote:
>>>
>>>> It didn't make it to patchwork, because you've sent only the cover
>>>> letter. I've added it to master-next now.
>>>>
>>>> Next time please send also individual patches. So that it gets more
>>>> visibility for other people to comment and also that I can use patchwork
>>>> to pull it in.
>>>>
>>>> Regards,
>>>>
>>>>
>>> Small note: Looks like the initial commit (24b2ac) has a small typo that
>>> should ideally be fixed before it's merged into master:
>>>
>>> "open-vm-tools: add recipe from meat-foss-openswitch"
>>>
>>> "meat" --> "meta"
>>>
>>> On Thu, Jun 15, 2017 at 6:36 PM, Martin Kelly <mkelly at xevo.com
>>>> <mailto:mkelly at xevo.com>> wrote:
>>>>
>>>>     On 06/09/2017 03:22 PM, Martin Kelly wrote:
>>>>
>>>>         Hi all,
>>>>
>>>>         This patch series adds a new recipe for Open VM Tools to
>>>>         meta-oe. This
>>>>         recipe is invaluable for using Openembedded with VMWare, as
>>>>         without the
>>>>         tools, performance is very slow.
>>>>
>>>>         More information about Open VM Tools can be found here:
>>>>
>>>>         https://github.com/vmware/open-vm-tools
>>>>         <https://github.com/vmware/open-vm-tools>
>>>>
>>>>         Please note that, as mentioned in the first commit, this recipe
>>>> is
>>>>         modified from the recipe originally found in the Open-Switch
>>>>         repository.
>>>>         The first patch is directly from that repository, and the
>>>> following
>>>>         patches are modifications to it.
>>>>
>>>>         Original recipe:
>>>>         https://github.com/open-switch/ops-build
>>>>         <https://github.com/open-switch/ops-build>
>>>>         yocto/openswitch/meta-foss-openswitch/recipes-extended/open-
>>>> vm-tools/open-vm-tools_10.0.5.bb
>>>>         <http://open-vm-tools_10.0.5.bb>
>>>>
>>>>         Commit 9008de2d8e100f3f868c66765742bca9fa98f3f9
>>>>
>>>>         Although the original recipe packaging was Apache-licensed, I
>>>>         received
>>>>         permission from the author (Diego Dompe) to relicense it under
>>>>         the MIT
>>>>         license for inclusion in meta-openembedded:
>>>>
>>>>         https://lists.openswitch.net/pipermail/ops-dev/2017-April/01
>>>> 4715.html
>>>>         <https://lists.openswitch.net/pipermail/ops-dev/2017-April/0
>>>> 14715.html>
>>>>         https://lists.openswitch.net/pipermail/ops-dev/2017-April/01
>>>> 4716.html
>>>>
>>>>         <https://lists.openswitch.net/pipermail/ops-dev/2017-April/0
>>>> 14716.html>
>>>>
>>>>         --
>>>>         The following changes since commit
>>>>         41cf832cc9abd6f2293a6d612463a34a53a9a52a:
>>>>
>>>>           Revert "rrdtool: add gettext-native dependency" (2017-06-05
>>>>         11:13:09
>>>>         +0200)
>>>>
>>>>         are available in the git repository at:
>>>>
>>>>           https://github.com/XevoInc/meta-openembedded
>>>>         <https://github.com/XevoInc/meta-openembedded>
>>>>
>>>>         for you to fetch changes up to
>>>>         402d1b3917195beb2ab7cac5dc94aaed8992b5a1:
>>>>
>>>>           open-vm-tools: style changes (2017-06-07 22:31:34 +0000)
>>>>
>>>>         ------------------------------------------------------------
>>>> ----
>>>>         Martin Kelly (14):
>>>>               add open-vm-tools recipe
>>>>               open-vm-tools: add missing pkgconfig dependency
>>>>               open-vm-tools: add missing glib-2.0-native dependency
>>>>               open-vm-tools: clarify and expand licensing
>>>>               open-vm-tools: base_contains --> bb.utils.contains
>>>>               open-vm-tools: update version to 10.1.5
>>>>               open-vm-tools: don't use mspack
>>>>               open-vm-tools: don't use dnet-config
>>>>               open-vm-tools: add #include to fix build break
>>>>               open-vm-tools: add missing X11 dependencies
>>>>               open-vm-tools: remove kernel module logic
>>>>               open-vm-tools: line up configure options
>>>>               open-vm-tools: support FUSE
>>>>               open-vm-tools: style changes
>>>>
>>>>          .../recipes-support/open-vm-tools/files/tools.conf |  2 +
>>>>          .../open-vm-tools/files/vmtoolsd.service           | 11 +++
>>>>          .../open-vm-tools/0001-Fix-kernel-detection.patch  | 35
>>>> +++++++++
>>>>          .../0002-configure.ac-don-t-use-dnet-config.patch  | 41
>>>> ++++++++++
>>>>          .../0003-add-include-sys-sysmacros.h.patch         | 33
>>>> ++++++++
>>>>          .../open-vm-tools/open-vm-tools_10.1.5.bb
>>>>         <http://open-vm-tools_10.1.5.bb>          | 90
>>>>         ++++++++++++++++++++++
>>>>          6 files changed, 212 insertions(+)
>>>>          create mode 100644
>>>>         meta-oe/recipes-support/open-vm-tools/files/tools.conf
>>>>          create mode 100644
>>>>         meta-oe/recipes-support/open-vm-tools/files/vmtoolsd.service
>>>>          create mode 100644
>>>>         meta-oe/recipes-support/open-vm-tools/open-vm-tools/0001-Fix
>>>> -kernel-detection.patch
>>>>
>>>>          create mode 100644
>>>>         meta-oe/recipes-support/open-vm-tools/open-vm-tools/0002-con
>>>> figure.ac-don-t-use-dnet-config.patch
>>>>
>>>>          create mode 100644
>>>>         meta-oe/recipes-support/open-vm-tools/open-vm-tools/0003-add
>>>> -include-sys-sysmacros.h.patch
>>>>
>>>>          create mode 100644
>>>>         meta-oe/recipes-support/open-vm-tools/open-vm-tools_10.1.5.bb
>>>>         <http://open-vm-tools_10.1.5.bb>
>>>>         --
>>>>         2.1.4
>>>>
>>>>
>>>>     (ping)
>>>>
>>>>
>>>>
>>
>



More information about the Openembedded-devel mailing list