public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
* [PATCH] sed: Skip 'build-deps' QA check for sed-ptest package
@ 2014-12-26 16:10 Otavio Salvador
  2014-12-31 19:16 ` Otavio Salvador
  0 siblings, 1 reply; 6+ messages in thread
From: Otavio Salvador @ 2014-12-26 16:10 UTC (permalink / raw)
  To: OpenEmbedded Core Mailing List; +Cc: Otavio Salvador

The sed-ptest package has runtime dependencies which are used only
when running the tests. So the 'build-dep' QA check can be skipped for
this specific package. Fixes the following QA error:

,----
| WARNING: QA Issue: sed-ptest rdepends on locale-base-ru-ru, but it
|     isn't a build dependency? [build-deps]
`----

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 meta/recipes-extended/sed/sed_4.2.2.bb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-extended/sed/sed_4.2.2.bb b/meta/recipes-extended/sed/sed_4.2.2.bb
index ea39dae..0139b18 100644
--- a/meta/recipes-extended/sed/sed_4.2.2.bb
+++ b/meta/recipes-extended/sed/sed_4.2.2.bb
@@ -14,7 +14,6 @@ SRC_URI[md5sum] = "4111de4faa3b9848a0686b2f260c5056"
 SRC_URI[sha256sum] = "fea0a94d4b605894f3e2d5572e3f96e4413bcad3a085aae7367c2cf07908b2ff"
 
 inherit autotools texinfo update-alternatives gettext ptest
-RDEPENDS_${PN}-ptest += "make locale-base-ru-ru ${PN}"
 
 EXTRA_OECONF = "--disable-acl \
                 ${@bb.utils.contains('PTEST_ENABLED', '1', '--enable-regex-tests', '', d)}"
@@ -40,4 +39,7 @@ do_install_ptest() {
 	oe_runmake -C ${TESTDIR} install-ptest BUILDDIR=${B} DESTDIR=${D}${PTEST_PATH} TESTDIR=${TESTDIR}
 }
 
+RDEPENDS_${PN}-ptest += "make locale-base-ru-ru ${PN}"
+INSANE_SKIP_${PN}-ptest += "build-deps"
+
 BBCLASSEXTEND = "native"
-- 
2.1.3



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

* Re: [PATCH] sed: Skip 'build-deps' QA check for sed-ptest package
  2014-12-26 16:10 [PATCH] sed: Skip 'build-deps' QA check for sed-ptest package Otavio Salvador
@ 2014-12-31 19:16 ` Otavio Salvador
  2015-01-04  2:03   ` ChenQi
  0 siblings, 1 reply; 6+ messages in thread
From: Otavio Salvador @ 2014-12-31 19:16 UTC (permalink / raw)
  To: OpenEmbedded Core Mailing List; +Cc: Otavio Salvador

On Fri, Dec 26, 2014 at 2:10 PM, Otavio Salvador
<otavio@ossystems.com.br> wrote:
> The sed-ptest package has runtime dependencies which are used only
> when running the tests. So the 'build-dep' QA check can be skipped for
> this specific package. Fixes the following QA error:
>
> ,----
> | WARNING: QA Issue: sed-ptest rdepends on locale-base-ru-ru, but it
> |     isn't a build dependency? [build-deps]
> `----
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>

Ping?

-- 
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] 6+ messages in thread

* Re: [PATCH] sed: Skip 'build-deps' QA check for sed-ptest package
  2014-12-31 19:16 ` Otavio Salvador
@ 2015-01-04  2:03   ` ChenQi
  2015-01-04  4:31     ` Otavio Salvador
  0 siblings, 1 reply; 6+ messages in thread
From: ChenQi @ 2015-01-04  2:03 UTC (permalink / raw)
  To: openembedded-core

On 01/01/2015 03:16 AM, Otavio Salvador wrote:
> On Fri, Dec 26, 2014 at 2:10 PM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>> The sed-ptest package has runtime dependencies which are used only
>> when running the tests. So the 'build-dep' QA check can be skipped for
>> this specific package. Fixes the following QA error:
>>
>> ,----
>> | WARNING: QA Issue: sed-ptest rdepends on locale-base-ru-ru, but it
>> |     isn't a build dependency? [build-deps]
>> `----
>>
>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> Ping?
>

Last week, I did a world build and I didn't get this QA warning.
Are there any specific build steps or configurations to reproduce the bug?

Regards,
Chen Qi


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

* Re: [PATCH] sed: Skip 'build-deps' QA check for sed-ptest package
  2015-01-04  2:03   ` ChenQi
@ 2015-01-04  4:31     ` Otavio Salvador
  2015-01-04  5:39       ` ChenQi
  0 siblings, 1 reply; 6+ messages in thread
From: Otavio Salvador @ 2015-01-04  4:31 UTC (permalink / raw)
  To: ChenQi; +Cc: Patches and discussions about the oe-core layer

On Sun, Jan 4, 2015 at 12:03 AM, ChenQi <Qi.Chen@windriver.com> wrote:
> On 01/01/2015 03:16 AM, Otavio Salvador wrote:
>>
>> On Fri, Dec 26, 2014 at 2:10 PM, Otavio Salvador
>> <otavio@ossystems.com.br> wrote:
>>>
>>> The sed-ptest package has runtime dependencies which are used only
>>> when running the tests. So the 'build-dep' QA check can be skipped for
>>> this specific package. Fixes the following QA error:
>>>
>>> ,----
>>> | WARNING: QA Issue: sed-ptest rdepends on locale-base-ru-ru, but it
>>> |     isn't a build dependency? [build-deps]
>>> `----
>>>
>>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>>
>> Ping?
>>
>
> Last week, I did a world build and I didn't get this QA warning.
> Are there any specific build steps or configurations to reproduce the bug?

ptest?

-- 
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] 6+ messages in thread

* Re: [PATCH] sed: Skip 'build-deps' QA check for sed-ptest package
  2015-01-04  4:31     ` Otavio Salvador
@ 2015-01-04  5:39       ` ChenQi
  2015-01-04 18:52         ` Otavio Salvador
  0 siblings, 1 reply; 6+ messages in thread
From: ChenQi @ 2015-01-04  5:39 UTC (permalink / raw)
  To: Otavio Salvador; +Cc: Patches and discussions about the oe-core layer

On 01/04/2015 12:31 PM, Otavio Salvador wrote:
> On Sun, Jan 4, 2015 at 12:03 AM, ChenQi <Qi.Chen@windriver.com> wrote:
>> On 01/01/2015 03:16 AM, Otavio Salvador wrote:
>>> On Fri, Dec 26, 2014 at 2:10 PM, Otavio Salvador
>>> <otavio@ossystems.com.br> wrote:
>>>> The sed-ptest package has runtime dependencies which are used only
>>>> when running the tests. So the 'build-dep' QA check can be skipped for
>>>> this specific package. Fixes the following QA error:
>>>>
>>>> ,----
>>>> | WARNING: QA Issue: sed-ptest rdepends on locale-base-ru-ru, but it
>>>> |     isn't a build dependency? [build-deps]
>>>> `----
>>>>
>>>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>>> Ping?
>>>
>> Last week, I did a world build and I didn't get this QA warning.
>> Are there any specific build steps or configurations to reproduce the bug?
> ptest?
>

It's in DISTRO_FEATURES.






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

* Re: [PATCH] sed: Skip 'build-deps' QA check for sed-ptest package
  2015-01-04  5:39       ` ChenQi
@ 2015-01-04 18:52         ` Otavio Salvador
  0 siblings, 0 replies; 6+ messages in thread
From: Otavio Salvador @ 2015-01-04 18:52 UTC (permalink / raw)
  To: ChenQi; +Cc: Patches and discussions about the oe-core layer

On Sun, Jan 4, 2015 at 3:39 AM, ChenQi <Qi.Chen@windriver.com> wrote:
> On 01/04/2015 12:31 PM, Otavio Salvador wrote:
>>
>> On Sun, Jan 4, 2015 at 12:03 AM, ChenQi <Qi.Chen@windriver.com> wrote:
>>>
>>> On 01/01/2015 03:16 AM, Otavio Salvador wrote:
>>>>
>>>> On Fri, Dec 26, 2014 at 2:10 PM, Otavio Salvador
>>>> <otavio@ossystems.com.br> wrote:
>>>>>
>>>>> The sed-ptest package has runtime dependencies which are used only
>>>>> when running the tests. So the 'build-dep' QA check can be skipped for
>>>>> this specific package. Fixes the following QA error:
>>>>>
>>>>> ,----
>>>>> | WARNING: QA Issue: sed-ptest rdepends on locale-base-ru-ru, but it
>>>>> |     isn't a build dependency? [build-deps]
>>>>> `----
>>>>>
>>>>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>>>>
>>>> Ping?
>>>>
>>> Last week, I did a world build and I didn't get this QA warning.
>>> Are there any specific build steps or configurations to reproduce the
>>> bug?
>>
>> ptest?
>>
>
> It's in DISTRO_FEATURES.

So using today's tip I have:

NOTE: Preparing RunQueue
NOTE: Executing SetScene Tasks
NOTE: Executing RunQueue Tasks
WARNING: QA Issue: sed-ptest rdepends on locale-base-ru-ru, but it
isn't a build dependency? [build-deps]
NOTE: Tasks Summary: Attempted 718 tasks of which 701 didn't need to
be rerun and all succeeded.
NOTE: Writing buildhistory

To  reproduce it, you need to have ptest in distro features but you
need to 'build' sed.

So you can do:

bitbake sed -c cleansstate ; bitbake sed

This should show off the warning.

-- 
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] 6+ messages in thread

end of thread, other threads:[~2015-01-04 18:52 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-26 16:10 [PATCH] sed: Skip 'build-deps' QA check for sed-ptest package Otavio Salvador
2014-12-31 19:16 ` Otavio Salvador
2015-01-04  2:03   ` ChenQi
2015-01-04  4:31     ` Otavio Salvador
2015-01-04  5:39       ` ChenQi
2015-01-04 18:52         ` Otavio Salvador

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