All of lore.kernel.org
 help / color / mirror / Atom feed
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



  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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.