[PATCH 0/1] insane.bbclass: skip dev-deps QA checking for ptest package

[PATCH 0/1] insane.bbclass: skip dev-deps QA checking for ptest package

[Robert Yang]

The following changes since commit dd09fab685de2eaf04aa5ab60f8220b89c1deae9:

  tune-core2.inc: set X86ARCH32 to i686 (instead of i586) (2015-07-02 23:08:27 +0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib rbt/insane
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/insane

Robert Yang (1):
  insane.bbclass: skip dev-deps QA checking for ptest package

 meta/classes/insane.bbclass |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

-- 
1.7.9.5

[PATCH 1/1] insane.bbclass: skip dev-deps QA checking for ptest package

[Robert Yang]

The ptest pkg such as perl-ptest may need .h, .so or other files in dev
package to run test cases, so skipping dev-deps checkin for ptest
package.

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/classes/insane.bbclass |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/insane.bbclass b/meta/classes/insane.bbclass
index 4537eec..d0eaa6b 100644
--- a/meta/classes/insane.bbclass
+++ b/meta/classes/insane.bbclass
@@ -792,7 +792,8 @@ def package_qa_check_rdepends(pkg, pkgdest, skip, taskdeps, packages, d):
                 if "-dbg" in rdepend and "debug-deps" not in skip:
                     error_msg = "%s rdepends on %s" % (pkg,rdepend)
                     sane = package_qa_handle_error("debug-deps", error_msg, d)
-                if (not "-dev" in pkg and not "-staticdev" in pkg) and rdepend.endswith("-dev") and "dev-deps" not in skip:
+                if (not pkg.endswith('-ptest') and not "-dev" in pkg and not "-staticdev" in pkg) \
+                        and rdepend.endswith("-dev") and "dev-deps" not in skip:
                     error_msg = "%s rdepends on %s" % (pkg, rdepend)
                     sane = package_qa_handle_error("dev-deps", error_msg, d)
                 if rdepend not in packages:
-- 
1.7.9.5

Re: [PATCH 1/1] insane.bbclass: skip dev-deps QA checking for ptest package

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 391 bytes --]

On 6 July 2015 at 11:01, Robert Yang <liezhi.yang@windriver.com> wrote:

> The ptest pkg such as perl-ptest may need .h, .so or other files in dev
> package to run test cases, so skipping dev-deps checkin for ptest
> package.
>

Is this a common occurrence or is perl the first instance of this? This
warning can be skipped with INSANE_SKIP_${PN}-ptest = "dev-deps", right?

Ross

[-- Attachment #2: Type: text/html, Size: 816 bytes --]

Re: [PATCH 1/1] insane.bbclass: skip dev-deps QA checking for ptest package

[Robert Yang]

On 07/11/2015 06:26 AM, Burton, Ross wrote:
>
> On 6 July 2015 at 11:01, Robert Yang <liezhi.yang@windriver.com
> <mailto:liezhi.yang@windriver.com>> wrote:
>
>     The ptest pkg such as perl-ptest may need .h, .so or other files in dev
>     package to run test cases, so skipping dev-deps checkin for ptest
>     package.
>
>
> Is this a common occurrence or is perl the first instance of this? This warning
> can be skipped with INSANE_SKIP_${PN}-ptest = "dev-deps", right?

I only know perl atm, I will try more.

// Robert

>
> Ross