Patchwork [0/8] Fixes about unsafe-references QA warnings

login
register
mail settings
Submitter Qi.Chen@windriver.com
Date Nov. 9, 2013, 5:28 a.m.
Message ID <cover.1383974819.git.Qi.Chen@windriver.com>
Download mbox
Permalink /patch/61353/
State New
Headers show

Pull-request

git://git.pokylinux.org/poky-contrib ChenQi/unsafe-reference

Comments

Qi.Chen@windriver.com - Nov. 9, 2013, 5:28 a.m.
From: Chen Qi <Qi.Chen@windriver.com>

This solution is based on the following two principles.
1. With /usr on a seperate partition, system should still boot without any error.
2. Without /usr, system should be able to boot into single user mode with error.

When a QA warning about unsafe references is encountered, the above two principles,
together with FHS, are taken into consideration before making a decision.

For example, libkmod is moved to /lib because /lib/udevd requires it; libgudevd
is moved to /usr/lib because it's a GObject wrapper for libudev and it's obviously
not necessary for booting into single user mode.

This patchset also contains several fixes for the sysvinit system.

The overall effect of all these patches:
1. Performing a world build doesn't report any warning about unsafe references.
2. System can boot into runlevel 3 without any error if /usr is on another partition.
3. System can boot into single user mode without any error even if /usr is missing.

---------------------------------------------------------------------------

The following changes since commit 4fdc3d77d4a875b7236536bf78849a4d1f6a7449:

  kbd: Fix stdarg related errors on uclibc (2013-11-08 17:31:36 +0000)

are available in the git repository at:

  git://git.pokylinux.org/poky-contrib ChenQi/unsafe-reference
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/unsafe-reference

Chen Qi (8):
  udev: fix dependency and location of udevadm
  initscripts: add setup-commands.sh
  zlib: install into base_libdir
  kmod: install libkmod into base_libdir
  udev: fix unsafe reference by installing libgudev in libdir
  insane.bbclass: make the checking stricter for unsafe references in
    scripts
  iputils: fix program location and QA warning
  busybox: install ping6 into bindir by default

 meta/classes/insane.bbclass                        |    2 +-
 .../busybox-install-ping6-to-usr-bin.patch         |   25 +++++++++++++++++
 meta/recipes-core/busybox/busybox_1.21.1.bb        |    1 +
 .../initscripts/initscripts-1.0/setup-commands.sh  |   24 ++++++++++++++++
 meta/recipes-core/initscripts/initscripts_1.0.bb   |    3 ++
 meta/recipes-core/udev/udev.inc                    |   16 ++++++++---
 meta/recipes-core/zlib/zlib_1.2.8.bb               |    2 +-
 meta/recipes-extended/iputils/iputils_s20121221.bb |   29 ++++++++++++--------
 meta/recipes-kernel/kmod/kmod_git.bb               |    7 +++--
 9 files changed, 89 insertions(+), 20 deletions(-)
 create mode 100644 meta/recipes-core/busybox/busybox-1.21.1/busybox-install-ping6-to-usr-bin.patch
 create mode 100644 meta/recipes-core/initscripts/initscripts-1.0/setup-commands.sh
Ross Burton - Nov. 11, 2013, 11:12 a.m.
On 9 November 2013 05:28,  <Qi.Chen@windriver.com> wrote:
> This solution is based on the following two principles.
> 1. With /usr on a seperate partition, system should still boot without any error.
> 2. Without /usr, system should be able to boot into single user mode with error.

Presumably this is with sysvinit.  Have you tested systemd with these
principles?

Ross
Qi.Chen@windriver.com - Nov. 11, 2013, 11:23 a.m.
On 11/11/2013 07:12 PM, Burton, Ross wrote:
> On 9 November 2013 05:28,  <Qi.Chen@windriver.com> wrote:
>> This solution is based on the following two principles.
>> 1. With /usr on a seperate partition, system should still boot without any error.
>> 2. Without /usr, system should be able to boot into single user mode with error.
> Presumably this is with sysvinit.  Have you tested systemd with these
> principles?
>
> Ross
>
>

Not yet ...
I can do such testing in a day or two.
I'll then file a bug and make patches if needed.

Best Regards,
Chen Qi