From: Alexander Kanavin <alexander.kanavin@linux.intel.com>
To: Nathan Lynch <Nathan_Lynch@mentor.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCHv2 24/26] lttng-tools: do not install manpages in ptest package
Date: Mon, 31 Oct 2016 15:43:35 +0200 [thread overview]
Message-ID: <c3dd23a6-a778-2b02-0a4d-ef37624f5896@linux.intel.com> (raw)
In-Reply-To: <c39f2013-9bad-9e64-d2b8-721db94001a0@mentor.com>
On 10/28/2016 09:46 PM, Nathan Lynch wrote:
> This actually breaks running the lttng-tools test suite on the target:
>
> root@qemux86-64:/usr/lib/lttng-tools/ptest# cat run-ptest
> #!/bin/sh
> # Without --ignore-exit, the tap harness causes any FAILs within a
> # test plan to raise ERRORs; this is just noise.
> makeargs="LOG_DRIVER_FLAGS=--ignore-exit"
> make -t all >/dev/null 2>&1 && exec make -s $makeargs check 2>/dev/null
>
> root@qemux86-64:/usr/lib/lttng-tools/ptest# sh -x run-ptest
> + makeargs=LOG_DRIVER_FLAGS=--ignore-exit
> + make -t all
>
> root@qemux86-64:/usr/lib/lttng-tools/ptest# echo $?
> 2
>
> root@qemux86-64:/usr/lib/lttng-tools/ptest# make -s -t all
> make[2]: *** No rule to make target 'lttng-health-check.3', needed by
> 'all-am'. Stop.
> make[1]: *** [Makefile:537: all-recursive] Error 1
> make: *** [Makefile:570: all-recursive] Error 1
>
>
> I'd rather see "do not install shared libraries in ptest package" go in
> first as it fixes an existing problem without regressing lttng-tools-ptest.
>
> I'm looking at reworking do_install_ptest to do a selective copy, as you
> suggested earlier, so maybe just drop this patch?
Thanks for checking. This hasn't yet made it to master, so it could
simply be dropped from master-next. If this doesn't happen, I'll submit
a revertion patch.
Alex
next prev parent reply other threads:[~2016-10-31 13:45 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-28 15:12 [PATCHv2 00/26] Enable manpages wherever possible; drop SGML stack Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 01/26] manpages.bbclass: add a class Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 02/26] gummiboot: enable optional building of manpages Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 03/26] pm-utils: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 04/26] systemd-boot: add a comment about manpages being disabled Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 05/26] avahi: enable manpages Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 06/26] glib: enable optional building of manpages Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 07/26] systemd: enable building of manpages if api-documentation feature is in distro features Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 08/26] pixz: enable optional building of manpages Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 09/26] gtk+: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 10/26] json-glib: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 11/26] libsecret: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 12/26] libdrm: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 13/26] kmod: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 14/26] lttng: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 15/26] alsa-utils: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 16/26] pulseaudio: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 17/26] btrfs-tools: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 18/26] libuser: do not depend on SGML stack Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 19/26] systemd: " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 20/26] gtk+: remove dependency " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 21/26] gtk+3: remove SGML stack dependency Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 22/26] iputils: do not build or install manpages Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 23/26] Remove the SGML stack Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 24/26] lttng-tools: do not install manpages in ptest package Alexander Kanavin
2016-10-28 18:46 ` Nathan Lynch
2016-10-31 13:43 ` Alexander Kanavin [this message]
2016-10-28 15:12 ` [PATCHv2 25/26] lttng-tools: do not install shared libraries " Alexander Kanavin
2016-10-28 15:12 ` [PATCHv2 26/26] core-image-{lsb/rt/sato}: add documentation packages to the SDK images Alexander Kanavin
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=c3dd23a6-a778-2b02-0a4d-ef37624f5896@linux.intel.com \
--to=alexander.kanavin@linux.intel.com \
--cc=Nathan_Lynch@mentor.com \
--cc=openembedded-core@lists.openembedded.org \
/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