Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib])
@ 2013-11-04  9:56 Hongxu Jia
  2013-11-04  9:56 ` [PATCH 1/5] insane.bbclass: support to skip unsafe references in binaries qa check Hongxu Jia
                   ` (5 more replies)
  0 siblings, 6 replies; 11+ messages in thread
From: Hongxu Jia @ 2013-11-04  9:56 UTC (permalink / raw)
  To: openembedded-core; +Cc: saul.wold

Change in V2: The fourth patch fixed the QA Issue on udev, it was missed on previous version.

//Hongxu

The following changes since commit f3541226b8b1187e79dec0f6f9f3c58cedf9ac9b:

  bitbake: hob: do not display the "Package list may be incomplete!" dialog (2013-11-01 17:59:31 +0000)

are available in the git repository at:

  git://git.pokylinux.org/poky-contrib hongxu/fix-qa-issue
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=hongxu/fix-qa-issue

Hongxu Jia (5):
  insane.bbclass: support to skip  unsafe references in binaries qa
    check.
  kmod: fix QA issue
  udev: fix  QA issue
  udev: fix QA issue
  nfs-utils: fix QA issue

 meta/classes/insane.bbclass                        |  4 ++++
 ...et-sbindir-can-be-overridden-at-config-ti.patch | 24 ++++++++++++++++++++++
 .../nfs-utils/nfs-utils_1.2.8.bb                   |  1 +
 meta/recipes-core/udev/udev.inc                    |  4 ++--
 meta/recipes-kernel/kmod/kmod_git.bb               |  1 +
 5 files changed, 32 insertions(+), 2 deletions(-)
 create mode 100644 meta/recipes-connectivity/nfs-utils/nfs-utils/osd_login-let-sbindir-can-be-overridden-at-config-ti.patch

-- 
1.8.1.2



^ permalink raw reply	[flat|nested] 11+ messages in thread

* [PATCH 1/5] insane.bbclass: support to skip unsafe references in binaries qa check.
  2013-11-04  9:56 [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Hongxu Jia
@ 2013-11-04  9:56 ` Hongxu Jia
  2013-11-04  9:56 ` [PATCH 2/5] kmod: fix QA issue Hongxu Jia
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 11+ messages in thread
From: Hongxu Jia @ 2013-11-04  9:56 UTC (permalink / raw)
  To: openembedded-core; +Cc: saul.wold

Add a variable to support skipping the unsafe references in binaries qa check.

[YOCTO #5142]

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 meta/classes/insane.bbclass | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index eb440c2..20bbab0 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -318,6 +318,10 @@ def package_qa_check_unsafe_references_in_binaries(path, name, d, elf, messages)
 	if elf:
 		import subprocess as sub
 		pn = d.getVar('PN', True)
+		# Skip for any packages with unsafe-ref-bin in INSANE_SKIP
+		if 'unsafe-ref-bin' in (d.getVar('INSANE_SKIP_' + pn, True) or "").split():
+			bb.note("Package %s skipping unsafe references in binaries QA test" % (pn))
+			return
 
 		exec_prefix = d.getVar('exec_prefix', True)
 		sysroot_path = d.getVar('STAGING_DIR_TARGET', True)
-- 
1.8.1.2



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 2/5] kmod: fix QA issue
  2013-11-04  9:56 [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Hongxu Jia
  2013-11-04  9:56 ` [PATCH 1/5] insane.bbclass: support to skip unsafe references in binaries qa check Hongxu Jia
@ 2013-11-04  9:56 ` Hongxu Jia
  2013-11-04  9:56 ` [PATCH 3/5] udev: " Hongxu Jia
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 11+ messages in thread
From: Hongxu Jia @ 2013-11-04  9:56 UTC (permalink / raw)
  To: openembedded-core; +Cc: saul.wold

The commit 7163ebd92a799b8f000b2b6f303b20de468b5f90
Revert "kmod: Use base_libdir for installing libkmod"

It caused a QA Issue WARNING:
...
WARNING: QA Issue: kmod: /bin/kmod, installed in the base_prefix, requires a shared library under exec_prefix (/usr): libkmod.so.2 => /usr/lib64/libkmod.so.2 (0x00000000dead1000)
...

But we did this reasonably, we should skip the qa check explicitly.

[YOCTO #5142]

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 meta/recipes-kernel/kmod/kmod_git.bb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-kernel/kmod/kmod_git.bb b/meta/recipes-kernel/kmod/kmod_git.bb
index f90b9aa..b370016 100644
--- a/meta/recipes-kernel/kmod/kmod_git.bb
+++ b/meta/recipes-kernel/kmod/kmod_git.bb
@@ -15,6 +15,7 @@ RCONFLICTS_libkmod2 += "module-init-tools-insmod-static module-init-tools-depmod
 bindir = "${base_bindir}"
 sbindir = "${base_sbindir}"
 # libdir = "${base_libdir}"
+INSANE_SKIP_${PN} = "unsafe-ref-bin"
 
 do_install_append () {
         install -dm755 ${D}${base_bindir}
-- 
1.8.1.2



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 3/5] udev: fix QA issue
  2013-11-04  9:56 [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Hongxu Jia
  2013-11-04  9:56 ` [PATCH 1/5] insane.bbclass: support to skip unsafe references in binaries qa check Hongxu Jia
  2013-11-04  9:56 ` [PATCH 2/5] kmod: fix QA issue Hongxu Jia
@ 2013-11-04  9:56 ` Hongxu Jia
  2013-11-04  9:56 ` [PATCH 4/5] " Hongxu Jia
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 11+ messages in thread
From: Hongxu Jia @ 2013-11-04  9:56 UTC (permalink / raw)
  To: openembedded-core; +Cc: saul.wold

...
WARNING: QA Issue: udev: /lib/libgudev-1.0.so.0.1.1, installed in the base_prefix, requires a shared library under exec_prefix (/usr): libgobject-2.0.so.0 => /usr/lib/libgobject-2.0.so.0 (0xdead3000)
WARNING: QA Issue: udev: /lib/libgudev-1.0.so.0.1.1, installed in the base_prefix, requires a shared library under exec_prefix (/usr): libffi.so.6 => /usr/lib/libffi.so.6 (0xdead4000)
WARNING: QA Issue: udev: /lib/libgudev-1.0.so.0.1.1, installed in the base_prefix, requires a shared library under exec_prefix (/usr): libglib-2.0.so.0 => /usr/lib/libglib-2.0.so.0 (0xdead5000)
...

Install libgudev-1.0.so.0.1.1 in ${libdir} rather than ${base_libdir}
fixed this issue.

[YOCTO #5142]

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 meta/recipes-core/udev/udev.inc | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
index 2ff8f00..2f883a0 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -32,14 +32,13 @@ inherit autotools pkgconfig update-rc.d
 
 libexecdir = "${base_libdir}"
 EXTRA_OECONF = "--disable-introspection \
-                --with-rootlibdir=${base_libdir} \
+                --with-rootlibdir=${libdir} \
                 --with-pci-ids-path=${datadir}/pci.ids \
                 ac_cv_file__usr_share_pci_ids=no \
                 ac_cv_file__usr_share_hwdata_pci_ids=no \
                 ac_cv_file__usr_share_misc_pci_ids=yes \
                 --sbindir=${base_sbindir} \
                 --libexecdir=${nonarch_base_libdir} \
-                --with-rootlibdir=${base_libdir} \
                 --with-rootprefix= \
                 --without-systemdsystemunitdir \
                "
-- 
1.8.1.2



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 4/5] udev: fix QA issue
  2013-11-04  9:56 [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Hongxu Jia
                   ` (2 preceding siblings ...)
  2013-11-04  9:56 ` [PATCH 3/5] udev: " Hongxu Jia
@ 2013-11-04  9:56 ` Hongxu Jia
  2013-11-04  9:56 ` [PATCH 5/5] nfs-utils: " Hongxu Jia
  2013-11-04 11:39 ` [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Richard Purdie
  5 siblings, 0 replies; 11+ messages in thread
From: Hongxu Jia @ 2013-11-04  9:56 UTC (permalink / raw)
  To: openembedded-core; +Cc: saul.wold

The commit 7163ebd92a799b8f000b2b6f303b20de468b5f90
Revert "kmod: Use base_libdir for installing libkmod"

It caused a QA Issue WARNING:
...
WARNING: QA Issue: udev: /lib/udev/udevd, installed in the base_prefix, requires a shared library under exec_prefix (/usr): libkmod.so.2 => /usr/lib/libkmod.so.2 (0xdead3000)
...

But we did this reasonably, we should skip the qa check explicitly.

[YOCTO #5142]

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 meta/recipes-core/udev/udev.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
index 2f883a0..92a11e0 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -30,6 +30,7 @@ SRC_URI = "${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.gz \
 
 inherit autotools pkgconfig update-rc.d
 
+INSANE_SKIP_${PN} = "unsafe-ref-bin"
 libexecdir = "${base_libdir}"
 EXTRA_OECONF = "--disable-introspection \
                 --with-rootlibdir=${libdir} \
-- 
1.8.1.2



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH 5/5] nfs-utils: fix QA issue
  2013-11-04  9:56 [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Hongxu Jia
                   ` (3 preceding siblings ...)
  2013-11-04  9:56 ` [PATCH 4/5] " Hongxu Jia
@ 2013-11-04  9:56 ` Hongxu Jia
  2013-11-04 11:39 ` [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Richard Purdie
  5 siblings, 0 replies; 11+ messages in thread
From: Hongxu Jia @ 2013-11-04  9:56 UTC (permalink / raw)
  To: openembedded-core; +Cc: saul.wold

...
WARNING: QA Issue: nfs-utils: Found a reference to /usr/ in /home/jiahongxu/yocto/build-20131025-gcc5/bitbake_build/tmp/work/x86_64-wrs-linux/nfs-utils/1.2.8-r1/packages-split/nfs-utils/sbin/osd_login
WARNING: QA Issue: Shell scripts in base_bindir and base_sbindir should not reference anything in exec_prefix
...

Install osd_login in ${sbindir} rather than ${base_sbindir} fixed this issue

[YOCTO #5142]

Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
---
 ...et-sbindir-can-be-overridden-at-config-ti.patch | 24 ++++++++++++++++++++++
 .../nfs-utils/nfs-utils_1.2.8.bb                   |  1 +
 2 files changed, 25 insertions(+)
 create mode 100644 meta/recipes-connectivity/nfs-utils/nfs-utils/osd_login-let-sbindir-can-be-overridden-at-config-ti.patch

diff --git a/meta/recipes-connectivity/nfs-utils/nfs-utils/osd_login-let-sbindir-can-be-overridden-at-config-ti.patch b/meta/recipes-connectivity/nfs-utils/nfs-utils/osd_login-let-sbindir-can-be-overridden-at-config-ti.patch
new file mode 100644
index 0000000..ffa5b2f
--- /dev/null
+++ b/meta/recipes-connectivity/nfs-utils/nfs-utils/osd_login-let-sbindir-can-be-overridden-at-config-ti.patch
@@ -0,0 +1,24 @@
+osd_login: let sbindir can be overridden at config time
+
+Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>
+Upstream-Status: Pending
+---
+ utils/osd_login/Makefile.am | 4 ----
+ 1 file changed, 4 deletions(-)
+
+diff --git a/utils/osd_login/Makefile.am b/utils/osd_login/Makefile.am
+--- a/utils/osd_login/Makefile.am
++++ b/utils/osd_login/Makefile.am
+@@ -1,9 +1,5 @@
+ ## Process this file with automake to produce Makefile.in
+ 
+-# These binaries go in /sbin (not /usr/sbin), and that cannot be
+-# overridden at config time.
+-sbindir = /sbin
+-
+ sbin_SCRIPTS = osd_login
+ 
+ MAINTAINERCLEANFILES = Makefile.in
+-- 
+1.8.1.2
+
diff --git a/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.8.bb b/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.8.bb
index e3e8136..84dd623 100644
--- a/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.8.bb
+++ b/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.8.bb
@@ -15,6 +15,7 @@ RRECOMMENDS_${PN} = "kernel-module-nfsd"
 SRC_URI = "${KERNELORG_MIRROR}/linux/utils/nfs-utils/${PV}/nfs-utils-${PV}.tar.bz2 \
            file://nfs-utils-1.0.6-uclibc.patch \
            file://nfs-utils-1.2.3-sm-notify-res_init.patch \
+           file://osd_login-let-sbindir-can-be-overridden-at-config-ti.patch \
            file://nfsserver \
            file://nfs-utils.conf \
            file://nfs-server.service \
-- 
1.8.1.2



^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib])
  2013-11-04  9:56 [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Hongxu Jia
                   ` (4 preceding siblings ...)
  2013-11-04  9:56 ` [PATCH 5/5] nfs-utils: " Hongxu Jia
@ 2013-11-04 11:39 ` Richard Purdie
  2013-11-04 11:48   ` Otavio Salvador
                     ` (2 more replies)
  5 siblings, 3 replies; 11+ messages in thread
From: Richard Purdie @ 2013-11-04 11:39 UTC (permalink / raw)
  To: Hongxu Jia; +Cc: saul.wold, openembedded-core

On Mon, 2013-11-04 at 17:56 +0800, Hongxu Jia wrote:
> Change in V2: The fourth patch fixed the QA Issue on udev, it was missed on previous version.
> 
> //Hongxu
> 
> The following changes since commit f3541226b8b1187e79dec0f6f9f3c58cedf9ac9b:
> 
>   bitbake: hob: do not display the "Package list may be incomplete!" dialog (2013-11-01 17:59:31 +0000)
> 
> are available in the git repository at:
> 
>   git://git.pokylinux.org/poky-contrib hongxu/fix-qa-issue
>   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=hongxu/fix-qa-issue
> 
> Hongxu Jia (5):
>   insane.bbclass: support to skip  unsafe references in binaries qa
>     check.
>   kmod: fix QA issue
>   udev: fix  QA issue
>   udev: fix QA issue
>   nfs-utils: fix QA issue

Last time these warnings came up I asked for a clear plan about what
needed to live in / and what needed to live in /usr and how we were
going to fix things once and for all (or forget this idea).

Where is the plan? Why is it ok to just ignore these particular
warnings?

Cheers,

Richard



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib])
  2013-11-04 11:39 ` [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Richard Purdie
@ 2013-11-04 11:48   ` Otavio Salvador
  2013-11-04 12:10     ` Richard Purdie
  2013-11-04 12:08   ` Hongxu Jia
  2013-11-09  5:26   ` ChenQi
  2 siblings, 1 reply; 11+ messages in thread
From: Otavio Salvador @ 2013-11-04 11:48 UTC (permalink / raw)
  To: Richard Purdie; +Cc: Patches and discussions about the oe-core layer, Saul Wold

On Mon, Nov 4, 2013 at 9:39 AM, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Mon, 2013-11-04 at 17:56 +0800, Hongxu Jia wrote:
>> Change in V2: The fourth patch fixed the QA Issue on udev, it was missed on previous version.
>>
>> //Hongxu
>>
>> The following changes since commit f3541226b8b1187e79dec0f6f9f3c58cedf9ac9b:
>>
>>   bitbake: hob: do not display the "Package list may be incomplete!" dialog (2013-11-01 17:59:31 +0000)
>>
>> are available in the git repository at:
>>
>>   git://git.pokylinux.org/poky-contrib hongxu/fix-qa-issue
>>   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=hongxu/fix-qa-issue
>>
>> Hongxu Jia (5):
>>   insane.bbclass: support to skip  unsafe references in binaries qa
>>     check.
>>   kmod: fix QA issue
>>   udev: fix  QA issue
>>   udev: fix QA issue
>>   nfs-utils: fix QA issue
>
> Last time these warnings came up I asked for a clear plan about what
> needed to live in / and what needed to live in /usr and how we were
> going to fix things once and for all (or forget this idea).
>
> Where is the plan? Why is it ok to just ignore these particular
> warnings?

I don't think supporting /bin and /sbin separated than /usr is worth the work.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib])
  2013-11-04 11:39 ` [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Richard Purdie
  2013-11-04 11:48   ` Otavio Salvador
@ 2013-11-04 12:08   ` Hongxu Jia
  2013-11-09  5:26   ` ChenQi
  2 siblings, 0 replies; 11+ messages in thread
From: Hongxu Jia @ 2013-11-04 12:08 UTC (permalink / raw)
  To: Richard Purdie; +Cc: saul.wold, openembedded-core

On 11/04/2013 07:39 PM, Richard Purdie wrote:
> On Mon, 2013-11-04 at 17:56 +0800, Hongxu Jia wrote:
>> Change in V2: The fourth patch fixed the QA Issue on udev, it was missed on previous version.
>>
>> //Hongxu
>>
>> The following changes since commit f3541226b8b1187e79dec0f6f9f3c58cedf9ac9b:
>>
>>    bitbake: hob: do not display the "Package list may be incomplete!" dialog (2013-11-01 17:59:31 +0000)
>>
>> are available in the git repository at:
>>
>>    git://git.pokylinux.org/poky-contrib hongxu/fix-qa-issue
>>    http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=hongxu/fix-qa-issue
>>
>> Hongxu Jia (5):
>>    insane.bbclass: support to skip  unsafe references in binaries qa
>>      check.
>>    kmod: fix QA issue
>>    udev: fix  QA issue
>>    udev: fix QA issue
>>    nfs-utils: fix QA issue
> Last time these warnings came up I asked for a clear plan about what
> needed to live in / and what needed to live in /usr and how we were
> going to fix things once and for all (or forget this idea).
>
> Where is the plan? Why is it ok to just ignore these particular
> warnings?

I am afraid there is no plan, let's drop this patch.

//Hongxu

> Cheers,
>
> Richard
>



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib])
  2013-11-04 11:48   ` Otavio Salvador
@ 2013-11-04 12:10     ` Richard Purdie
  0 siblings, 0 replies; 11+ messages in thread
From: Richard Purdie @ 2013-11-04 12:10 UTC (permalink / raw)
  To: Otavio Salvador
  Cc: Patches and discussions about the oe-core layer, Saul Wold

On Mon, 2013-11-04 at 09:48 -0200, Otavio Salvador wrote:
> On Mon, Nov 4, 2013 at 9:39 AM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
> > On Mon, 2013-11-04 at 17:56 +0800, Hongxu Jia wrote:
> >> Change in V2: The fourth patch fixed the QA Issue on udev, it was missed on previous version.
> >>
> >> //Hongxu
> >>
> >> The following changes since commit f3541226b8b1187e79dec0f6f9f3c58cedf9ac9b:
> >>
> >>   bitbake: hob: do not display the "Package list may be incomplete!" dialog (2013-11-01 17:59:31 +0000)
> >>
> >> are available in the git repository at:
> >>
> >>   git://git.pokylinux.org/poky-contrib hongxu/fix-qa-issue
> >>   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=hongxu/fix-qa-issue
> >>
> >> Hongxu Jia (5):
> >>   insane.bbclass: support to skip  unsafe references in binaries qa
> >>     check.
> >>   kmod: fix QA issue
> >>   udev: fix  QA issue
> >>   udev: fix QA issue
> >>   nfs-utils: fix QA issue
> >
> > Last time these warnings came up I asked for a clear plan about what
> > needed to live in / and what needed to live in /usr and how we were
> > going to fix things once and for all (or forget this idea).
> >
> > Where is the plan? Why is it ok to just ignore these particular
> > warnings?
> 
> I don't think supporting /bin and /sbin separated than /usr is worth the work.

I keep hearing that some people do need this. I'm not against it in
principle, equally I do want to see a plan and/or complete patchset
before we take more patches for it.

Cheers,

Richard



^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib])
  2013-11-04 11:39 ` [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Richard Purdie
  2013-11-04 11:48   ` Otavio Salvador
  2013-11-04 12:08   ` Hongxu Jia
@ 2013-11-09  5:26   ` ChenQi
  2 siblings, 0 replies; 11+ messages in thread
From: ChenQi @ 2013-11-09  5:26 UTC (permalink / raw)
  To: openembedded-core

On 11/04/2013 07:39 PM, Richard Purdie wrote:
> On Mon, 2013-11-04 at 17:56 +0800, Hongxu Jia wrote:
>> Change in V2: The fourth patch fixed the QA Issue on udev, it was missed on previous version.
>>
>> //Hongxu
>>
>> The following changes since commit f3541226b8b1187e79dec0f6f9f3c58cedf9ac9b:
>>
>>    bitbake: hob: do not display the "Package list may be incomplete!" dialog (2013-11-01 17:59:31 +0000)
>>
>> are available in the git repository at:
>>
>>    git://git.pokylinux.org/poky-contrib hongxu/fix-qa-issue
>>    http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=hongxu/fix-qa-issue
>>
>> Hongxu Jia (5):
>>    insane.bbclass: support to skip  unsafe references in binaries qa
>>      check.
>>    kmod: fix QA issue
>>    udev: fix  QA issue
>>    udev: fix QA issue
>>    nfs-utils: fix QA issue
> Last time these warnings came up I asked for a clear plan about what
> needed to live in / and what needed to live in /usr and how we were
> going to fix things once and for all (or forget this idea).
>
> Where is the plan? Why is it ok to just ignore these particular
> warnings?
>
> Cheers,
>
> Richard

Hi Richard,

I want to propose the following solution for this unsafe reference issue.

The solution is based on the following two principles.
1. With /usr on a seperate partition, system should boot up without any error.
2. Without /usr, system should be able to boot into single user mode without any 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.

The final goals of the solution:
1. Performing a world build doesn't report any warning about unsafe references.
2. System can boot into runlevel 5 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.


Patches of this solution have been sent to this mailing list.

Best Regards,
Chen Qi

> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
>



^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2013-11-09  5:26 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-04  9:56 [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Hongxu Jia
2013-11-04  9:56 ` [PATCH 1/5] insane.bbclass: support to skip unsafe references in binaries qa check Hongxu Jia
2013-11-04  9:56 ` [PATCH 2/5] kmod: fix QA issue Hongxu Jia
2013-11-04  9:56 ` [PATCH 3/5] udev: " Hongxu Jia
2013-11-04  9:56 ` [PATCH 4/5] " Hongxu Jia
2013-11-04  9:56 ` [PATCH 5/5] nfs-utils: " Hongxu Jia
2013-11-04 11:39 ` [PATCH V2 0/5] fix WARNING: QA Issue on kmod/udev/nfs-utils (unsafe references [/ references to /usr/lib]) Richard Purdie
2013-11-04 11:48   ` Otavio Salvador
2013-11-04 12:10     ` Richard Purdie
2013-11-04 12:08   ` Hongxu Jia
2013-11-09  5:26   ` ChenQi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox