All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Krzysztof Taborski" <taborskikrzysztof@gmail.com>
To: <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH v2] perl: native modules will not trigger build perl for target.
Date: Fri, 8 Dec 2017 18:46:42 +0100	[thread overview]
Message-ID: <00b101d3704c$8297acc0$87c70640$@gmail.com> (raw)
In-Reply-To: <20171204181640.31225-1-taborskikrzysztof@gmail.com>

Hello,
I would like to request feedback about my patch.

Br,
Krzysztof Taborski

-----Original Message-----
From: Krzysztof Taborski [mailto:taborskikrzysztof@gmail.com] 
Sent: Monday, December 04, 2017 7:17 PM
To: openembedded-core@lists.openembedded.org
Cc: Krzysztof Taborski
Subject: [OE-core][PATCH v2] perl: native modules will not trigger build
perl for target.

Currently building perl-native modules triggers build perl for target due to
PACKAGES_DYNAMIC regex.

This commit will cause, that perl native modules will trigger perl-native
build.

Signed-off-by: Krzysztof Taborski <taborskikrzysztof@gmail.com>
---
 meta/recipes-devtools/perl/perl-native_5.24.1.bb | 2 ++
 meta/recipes-devtools/perl/perl_5.24.1.bb        | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/perl/perl-native_5.24.1.bb
b/meta/recipes-devtools/perl/perl-native_5.24.1.bb
index 48113f05a9..a81dc4d703 100644
--- a/meta/recipes-devtools/perl/perl-native_5.24.1.bb
+++ b/meta/recipes-devtools/perl/perl-native_5.24.1.bb
@@ -134,3 +134,5 @@ EOF
 
 # Fix the path in sstate
 SSTATE_SCAN_FILES += "*.pm *.pod *.h *.pl *.sh"
+PACKAGES_DYNAMIC_class-native += "^perl-module-.*native$"
+
diff --git a/meta/recipes-devtools/perl/perl_5.24.1.bb
b/meta/recipes-devtools/perl/perl_5.24.1.bb
index 6b27b22d22..3dc7805da9 100644
--- a/meta/recipes-devtools/perl/perl_5.24.1.bb
+++ b/meta/recipes-devtools/perl/perl_5.24.1.bb
@@ -323,7 +323,7 @@ python split_perl_packages () {
     d.setVar(d.expand("RRECOMMENDS_${PN}-modules"), ' '.join(packages))  }
 
-PACKAGES_DYNAMIC += "^perl-module-.*"
+PACKAGES_DYNAMIC += "^perl-module-.*(?<!\-native)$"
 PACKAGES_DYNAMIC_class-nativesdk += "^nativesdk-perl-module-.*"
 
 RPROVIDES_perl-lib = "perl-lib"
--
2.13.6




  reply	other threads:[~2017-12-08 17:46 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-04 18:16 [PATCH v2] perl: native modules will not trigger build perl for target Krzysztof Taborski
2017-12-08 17:46 ` Krzysztof Taborski [this message]
     [not found] ` <00b201d3704c$8357c870$8a075950$@gmail.com>
2017-12-12  8:34   ` Krzysztof Taborski
2017-12-18 15:51     ` Krzysztof Taborski
2018-05-08 13:08       ` Krzysztof Taborski
2018-05-08 14:11         ` Alexander Kanavin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='00b101d3704c$8297acc0$87c70640$@gmail.com' \
    --to=taborskikrzysztof@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.