RFC populate perl-native into its own directory
Get series with b4
b4 am [email protected]Versions newest first
- v1 RFC Latest16 patches 27 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/16 | native.bbclass: allow a native package to be populated into its own dirmeta/ | +5 | 3 replies | |
| 2/16 | perl-native: populate into its own dirmeta/ | +4 | 0 replies | |
| 3/16 | perlnative.bbclass: add the filemeta/ | +2 | 0 replies | |
| 4/16 | gnu-config-native: should depend on perl-native-runtime rather than perl-nativemeta/ | +2 | 0 replies | |
| 5/16 | libcap: should depend on perl-native-runtime rather than perl-nativemeta/ meta/ | +3 | 0 replies | |
| 6/16 | openssl: should depend on perl-native-runtime rather than perl-nativemeta/ meta/ | +2 | 0 replies | |
| 7/16 | git: should depend on perl-native-runtime rather than perl-nativemeta/ meta/ | +3 | 2 replies | |
| 8/16 | coreutils: remove unnecessary dependency on perlmeta/ meta/ | +6 | 0 replies | |
| 9/16 | dpkg: should depend on perl-native-runtime rather than perl-nativemeta/ meta/ | +2 | 0 replies | |
| 10/16 | webkit-gtk: should depend on perl-native-runtime rather than perl-nativemeta/ | +3 | 0 replies | |
| 11/16 | perl: inherit perlnativemeta/ | +5 | 0 replies | |
| 12/16 | cpan.bbclass, cpan-base.bbclas: update them for the perlnative changemeta/ meta/ | +13 | 0 replies | |
| 13/16 | libxml-parser-perl: inherit perlnativemeta/ meta/ | +2 | 0 replies | |
| 14/16 | libconvert-asn1-perl: fix EXTRA_PERLFLAGS due to the perl-native changemeta/ | +2 | 0 replies | |
| 15/16 | libxml-simple-perl: fix EXTRA_PERLFLAGS due the the perlnative changemeta/ meta/ | +3 | 0 replies | |
| 16/16 | icon-naming-utils-native: inherit perlnativemeta/ | +2 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [RFC v1 PATCH 00/16] populate perl-native into its own directory
Currently both perl-native (a.k.a. ${STAGING_BINDIR_NATIVE}/perl )and perl-native-runtime(a.k.a. the system perl, or /usr/bin/perl) appear in the PATH when bitbake is running.
This can cause some race conditions: many places detecting perl from PATH can't make sure which path will be used as this depends on when perl-native's populate_sysroot is finished, e.g., automake-native and autoconf-native could use perl-native-runtime while gnu-config-native could use perl-native and this inconsistent usages can cause trouble, e.g., bug 941.
And, as RP suggested, "the time to use perl-native instead of perl-native-runtime is where any perl module is being built, perl itself is being built or anything that has an explicit dependency on the perl version present".
So I made the following changes to try to address the aboves issues:
1) populate perl-native into its own directory so it won't appear in PATH by default, and add perlnative.bbclass for these recipes that really depend on perl-native;
2) check all perl-native references and correct the ones that should be perl-native-runtime;
3) fix any building issues due to the new location of perl-native, especially cpan and cpan-base .bbclass.
With the changes, bug 941 doesn't appear.
Tests I did are:
I tried "bitbale core-image-sato-sdk and meta-toolchain-gmae" in x86_32 and x86_64 Ubuntu hosts and everything seems building fine.
Please review the changes and comment on them. Thanks!
--------------------------
The following changes since commit 0736b136efac283449c9218c01740eba4a7bc8cc:
are available in the git repository at:
git://git.pokylinux.org/poky-contrib dcui/master_perl-native
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dcui/master_perl-native
Dexuan Cui (16):
native.bbclass: allow a native package to be populated into its own
dir
perl-native: populate into its own dir
perlnative.bbclass: add the file
gnu-config-native: should depend on perl-native-runtime rather than
perl-native
libcap: should depend on perl-native-runtime rather than perl-native
openssl: should depend on perl-native-runtime rather than perl-native
git: should depend on perl-native-runtime rather than perl-native
coreutils: remove unnecessary dependency on perl
dpkg: should depend on perl-native-runtime rather than perl-native
webkit-gtk: should depend on perl-native-runtime rather than
perl-native
perl: inherit perlnative
cpan.bbclass, cpan-base.bbclas: update them for the perlnative change
libxml-parser-perl: inherit perlnative
libconvert-asn1-perl: fix EXTRA_PERLFLAGS due to the perl-native
change
libxml-simple-perl: fix EXTRA_PERLFLAGS due the the perlnative change
icon-naming-utils-native: inherit perlnative
meta/classes/cpan-base.bbclass | 10 +++++++--- meta/classes/cpan.bbclass | 12 ++++++------ meta/classes/native.bbclass | 5 +++++ meta/classes/perlnative.bbclass | 2 ++ meta/recipes-connectivity/openssl/openssl.inc | 2 +- .../recipes-connectivity/openssl/openssl_0.9.8r.bb | 2 +- meta/recipes-core/coreutils/coreutils_6.9.bb | 6 +++--- meta/recipes-core/coreutils/coreutils_8.9.bb | 6 +++--- meta/recipes-devtools/dpkg/dpkg.inc | 2 +- meta/recipes-devtools/dpkg/dpkg_1.15.8.7.bb | 2 +- meta/recipes-devtools/git/git.inc | 4 ++-- meta/recipes-devtools/git/git_1.7.5.1.bb | 2 +- .../gnu-config/gnu-config_20080123.bb | 4 ++-- .../icon-naming-utils-native_0.8.7.bb | 4 ++-- .../perl/libxml-parser-perl-native_2.40.bb | 2 +- .../perl/libxml-parser-perl_2.40.bb | 2 +- .../perl/libxml-simple-perl-native_2.18.bb | 2 +- .../perl/libxml-simple-perl_2.18.bb | 4 ++-- meta/recipes-devtools/perl/perl-native_5.12.3.bb | 5 ++++- meta/recipes-devtools/perl/perl_5.12.3.bb | 10 +++++----- .../perl/libconvert-asn1-perl_0.22.bb | 4 ++-- meta/recipes-sato/webkit/webkit-gtk_svn.bb | 6 +++--- meta/recipes-support/libcap/libcap.inc | 4 ++-- meta/recipes-support/libcap/libcap_2.20.bb | 2 +- 24 files changed, 59 insertions(+), 45 deletions(-) create mode 100644 meta/classes/perlnative.bbclass
Signature · 1 line
1.7.2