[PATCH 0/1] warning fix of sysvinit-inittab.

[PATCH 0/1] warning fix of sysvinit-inittab.

[Lianhao Lu]

This fixed the warning in do_populate_lic task in sysvinit-inittab.

The following changes since commit efd80fd23cb96ccc203893017938c1163d20b898:
  Richard Purdie (1):
        tcl: Fix bad RPATH QA warning

are available in the git repository at:

  git://git.yoctoproject.org/poky-contrib llu/warningfix
  http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=llu/warningfix

Lianhao Lu (1):
  sysvinit-inittab: Fixed license warning.

 .../sysvinit/sysvinit-inittab_2.88dsf.bb           |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

[PATCH 1/1] sysvinit-inittab: Fixed license warning.

[Lianhao Lu]

WARNING: .../sysvinit-inittab-2.88dsf-r6/sysvinit-2.88dsf/COPYING could
not be copied for some reason. It may not exist. WARN for now.

Signed-off-by: Lianhao Lu <lianhao.lu@intel.com>
---
 .../sysvinit/sysvinit-inittab_2.88dsf.bb           |    6 +++++-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
index 3a716d7..85e34c8 100644
--- a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
+++ b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
@@ -11,7 +11,11 @@ S = "${WORKDIR}/sysvinit-${PV}"
 
 INHIBIT_DEFAULT_DEPS = "1"
 
-do_configure() {
+do_unpack_append() {
+	bb.build.exec_func('do_copy_lic', d)
+}
+
+do_copy_lic () {
 	cp ${WORKDIR}/COPYING ${S}/
 }
 
-- 
1.7.0.4

Re: [PATCH 1/1] sysvinit-inittab: Fixed license warning.

[Saul Wold]

On 03/18/2012 06:52 PM, Lianhao Lu wrote:
> WARNING: .../sysvinit-inittab-2.88dsf-r6/sysvinit-2.88dsf/COPYING could
> not be copied for some reason. It may not exist. WARN for now.
>
> Signed-off-by: Lianhao Lu<lianhao.lu@intel.com>
> ---
>   .../sysvinit/sysvinit-inittab_2.88dsf.bb           |    6 +++++-
>   1 files changed, 5 insertions(+), 1 deletions(-)
>
> diff --git a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
> index 3a716d7..85e34c8 100644
> --- a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
> +++ b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
> @@ -11,7 +11,11 @@ S = "${WORKDIR}/sysvinit-${PV}"
>
>   INHIBIT_DEFAULT_DEPS = "1"
>
> -do_configure() {
> +do_unpack_append() {
> +	bb.build.exec_func('do_copy_lic', d)
> +}
> +
I think it would be better to do the copy in a do_configure_prepend() Or 
can we set ${S} to ${WORKDIR} and not have to copy it at all?


> +do_copy_lic () {
>   	cp ${WORKDIR}/COPYING ${S}/
>   }
>

Re: [PATCH 1/1] sysvinit-inittab: Fixed license warning.

[Richard Purdie]

On Mon, 2012-03-19 at 09:07 -0700, Saul Wold wrote:
> On 03/18/2012 06:52 PM, Lianhao Lu wrote:
> > WARNING: .../sysvinit-inittab-2.88dsf-r6/sysvinit-2.88dsf/COPYING could
> > not be copied for some reason. It may not exist. WARN for now.
> >
> > Signed-off-by: Lianhao Lu<lianhao.lu@intel.com>
> > ---
> >   .../sysvinit/sysvinit-inittab_2.88dsf.bb           |    6 +++++-
> >   1 files changed, 5 insertions(+), 1 deletions(-)
> >
> > diff --git a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
> > index 3a716d7..85e34c8 100644
> > --- a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
> > +++ b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
> > @@ -11,7 +11,11 @@ S = "${WORKDIR}/sysvinit-${PV}"
> >
> >   INHIBIT_DEFAULT_DEPS = "1"
> >
> > -do_configure() {
> > +do_unpack_append() {
> > +	bb.build.exec_func('do_copy_lic', d)
> > +}
> > +
> I think it would be better to do the copy in a do_configure_prepend() Or 
> can we set ${S} to ${WORKDIR} and not have to copy it at all?
> 
> 
> > +do_copy_lic () {
> >   	cp ${WORKDIR}/COPYING ${S}/
> >   }
> >

How about changing the license URL to point at:

file://${COREBASE}/meta/files/common-licenses/GPL-2.0

?

Cheers,

Richard