Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Saul Wold <sgw@linux.intel.com>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Cc: Scott Garman <scott.a.garman@intel.com>
Subject: Re: [PATCH 0/7] New QA tests for unsafe references to exec_prefix [v2]
Date: Thu, 05 Jan 2012 16:31:51 -0800	[thread overview]
Message-ID: <4F0640F7.50903@linux.intel.com> (raw)
In-Reply-To: <cover.1325649220.git.scott.a.garman@intel.com>

On 01/03/2012 07:56 PM, Scott Garman wrote:
> Hello,
>
> This is a P2 feature for Yocto 1.2. It adds two new QA tests to check
> files in base_bindir, base_sbindir, and base_libdir for references to
> files under exec_prefix (e.g, dynamic linking for binaries and
> references to exec_prefix in scripts).
>
> The reason for these tests is that exec_prefix (typically /usr) can
> be kept on a separate partition that may be mounted late in the boot
> process, or may not be mounted at all during system recovery
> operations.
>
> It was my original goal to make the dynamic linking test a QA error
> upon failure, but it turned out the amount of recipe work needed was
> much larger than I expected. So for the time being they have been
> changed to generate QA warmings.
>
> The libraries I've moved to base_libdir in this pull request should
> take care of all binaries in /bin and /sbin for packages we include
> in core-image-minimal and -sato.
>
> I would like to ask recipe maintainers to look for these QA warnings
> and fix them ASAP, as I believe we'll be changing this to a QA error
> before the release of 1.2.
>
> This is a second pull request that is rebased against the latest
> master with merge conflicts resolved, and only sets BUILD_LDFLAGS
> in ncurses.inc for virtclass-native.
>
> Scott
>
> The following changes since commit de679a3036ebef1c7d7b8ee23f05590c95e498d9:
>
>    pseudo: ensure libs are included in package (2012-01-03 12:10:59 +0000)
>
> are available in the git repository at:
>    git://git.pokylinux.org/poky-contrib sgarman/per-recipe-sanitychk-final
>    http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=sgarman/per-recipe-sanitychk-final
>
> Scott Garman (7):
>    ncurses: move libraries to base_libdir
>    libcap: move library to base_libdir
>    util-linux: move libraries to base_libdir
>    e2fsprogs: move libraries to base_libdir
>    libusb1: move libraries to base_libdir
>    libusb-compat: move libraries to base_libdir
>    insane.bbclass: add QA tests for unsafe references to exec_prefix
>
>   meta/classes/insane.bbclass                        |  111 +++++++++++++++++++-
>   meta/recipes-core/ncurses/ncurses.inc              |   27 +++--
>   meta/recipes-core/util-linux/util-linux.inc        |   20 ++--
>   meta/recipes-core/util-linux/util-linux_2.20.1.bb  |    2 +-
>   .../e2fsprogs/e2fsprogs_1.41.14.bb                 |   29 +++--
>   meta/recipes-support/libcap/libcap.inc             |   15 +++-
>   meta/recipes-support/libcap/libcap_2.22.bb         |    2 +-
>   meta/recipes-support/libusb/libusb-compat_0.1.3.bb |   12 ++-
>   meta/recipes-support/libusb/libusb1_1.0.8.bb       |   16 +++-
>   9 files changed, 196 insertions(+), 38 deletions(-)
>

Merged into OE-Core

Thanks
	Sau!



  parent reply	other threads:[~2012-01-06  0:39 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-04  3:56 [PATCH 0/7] New QA tests for unsafe references to exec_prefix [v2] Scott Garman
2012-01-04  3:56 ` [PATCH 1/7] ncurses: move libraries to base_libdir Scott Garman
2012-01-04  3:56 ` [PATCH 2/7] libcap: move library " Scott Garman
2012-01-04  3:56 ` [PATCH 3/7] util-linux: move libraries " Scott Garman
2012-01-04  3:56 ` [PATCH 4/7] e2fsprogs: " Scott Garman
2012-01-04  3:56 ` [PATCH 5/7] libusb1: " Scott Garman
2012-01-04  3:56 ` [PATCH 6/7] libusb-compat: " Scott Garman
2012-01-04  3:56 ` [PATCH 7/7] insane.bbclass: add QA tests for unsafe references to exec_prefix Scott Garman
2012-01-06  0:31 ` Saul Wold [this message]
2012-01-06  0:52   ` [PATCH 0/7] New QA tests for unsafe references to exec_prefix [v2] Andreas Müller

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=4F0640F7.50903@linux.intel.com \
    --to=sgw@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=scott.a.garman@intel.com \
    /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