* [PATCH 0/8] denzil pull request
@ 2012-06-18 19:56 Scott Garman
2012-06-18 19:56 ` [PATCH 1/8] documentation: Updated Manual revision history table Scott Garman
` (7 more replies)
0 siblings, 8 replies; 18+ messages in thread
From: Scott Garman @ 2012-06-18 19:56 UTC (permalink / raw)
To: poky
Hello,
This is a request for the poky denzil branch, based on a set of
commits I've been testing from my sgarman/denzil-next branch.
These commits have undergone the following testing on the autobuilder:
nightly:
http://autobuilder.yoctoproject.org:8010/builders/nightly/builds/518
nightly-multilib:
http://autobuilder.yoctoproject.org:8010/builders/nightly-multilib/builds/114
nightly-non-gpl3:
http://autobuilder.yoctoproject.org:8010/builders/nightly-non-gpl3/builds/115
nightly-tiny:
http://autobuilder.yoctoproject.org:8010/builders/nightly-tiny/builds/97
nightly-meta-intel:
http://autobuilder.yoctoproject.org:8010/builders/nightly-meta-intel/builds/106
Scott
The following changes since commit 119e1b7dc97a82ff0791d1b4ba519361ff46cbd1:
poky.conf: use correct version string for Ubuntu 12.04 (2012-06-14 11:27:19 +0100)
are available in the git repository at:
git://git.yoctoproject.org/poky-contrib sgarman/denzil-next-testing
http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=sgarman/denzil-next-pull2
Gary Thomas (1):
webkit-gtk: Apply work around for all PowerPC targets
Laurentiu Palcu (1):
sanity.bbclass: warn the user if BBPATH contains wrong entries
Nitin A Kamble (1):
eglibc: package mtrace separately
Richard Purdie (2):
openjade-native: Ensure we reautoconf the package
poky-tiny: eliminate mtrace rdepends
Saul Wold (2):
quilt: added ac_cv_path_BASH to CACHED_CONFIGUREVARS
eglibc: added ac_cv_path_ to CACHED_CONFIGUREVARS
Scott Rifenbark (1):
documentation: Updated Manual revision history table
documentation/adt-manual/adt-manual.xml | 2 +-
documentation/bsp-guide/bsp-guide.xml | 2 +-
documentation/dev-manual/dev-manual.xml | 2 +-
documentation/kernel-manual/kernel-manual.xml | 2 +-
documentation/poky-ref-manual/poky-ref-manual.xml | 2 +-
meta-yocto/conf/distro/poky-tiny.conf | 5 ++
meta/classes/sanity.bbclass | 13 ++-
meta/recipes-core/eglibc/eglibc-package.inc | 7 +-
meta/recipes-core/eglibc/eglibc.inc | 2 +
meta/recipes-core/eglibc/eglibc_2.13.bb | 2 +-
meta/recipes-core/eglibc/eglibc_2.15.bb | 2 +-
meta/recipes-core/tasks/task-core-tools-debug.bb | 4 +
.../openjade/openjade-1.3.2/reautoconf.patch | 83 ++++++++++++++++++++
.../openjade/openjade-native_1.3.2.bb | 5 ++
meta/recipes-devtools/quilt/quilt_0.51.bb | 5 +-
meta/recipes-sato/webkit/webkit-gtk_svn.bb | 15 ++--
16 files changed, 137 insertions(+), 16 deletions(-)
create mode 100644 meta/recipes-devtools/openjade/openjade-1.3.2/reautoconf.patch
--
1.7.9.5
^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH 1/8] documentation: Updated Manual revision history table
2012-06-18 19:56 [PATCH 0/8] denzil pull request Scott Garman
@ 2012-06-18 19:56 ` Scott Garman
2012-06-18 19:56 ` [PATCH 2/8] openjade-native: Ensure we reautoconf the package Scott Garman
` (6 subsequent siblings)
7 siblings, 0 replies; 18+ messages in thread
From: Scott Garman @ 2012-06-18 19:56 UTC (permalink / raw)
To: poky
From: Scott Rifenbark <scott.m.rifenbark@intel.com>
Using July 2012 for the release date.
Signed-off-by: Scott Rifenbark <scott.m.rifenbark@intel.com>
---
documentation/adt-manual/adt-manual.xml | 2 +-
documentation/bsp-guide/bsp-guide.xml | 2 +-
documentation/dev-manual/dev-manual.xml | 2 +-
documentation/kernel-manual/kernel-manual.xml | 2 +-
documentation/poky-ref-manual/poky-ref-manual.xml | 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/documentation/adt-manual/adt-manual.xml b/documentation/adt-manual/adt-manual.xml
index 33ca4c7..8eb89dd 100644
--- a/documentation/adt-manual/adt-manual.xml
+++ b/documentation/adt-manual/adt-manual.xml
@@ -51,7 +51,7 @@
</revision>
<revision>
<revnumber>1.2.1</revnumber>
- <date>Sometime in 2012</date>
+ <date>July 2012</date>
<revremark>Released with the Yocto Project 1.2.1 Release.</revremark>
</revision>
</revhistory>
diff --git a/documentation/bsp-guide/bsp-guide.xml b/documentation/bsp-guide/bsp-guide.xml
index 792be2f..fe28d91 100644
--- a/documentation/bsp-guide/bsp-guide.xml
+++ b/documentation/bsp-guide/bsp-guide.xml
@@ -63,7 +63,7 @@
</revision>
<revision>
<revnumber>1.2.1</revnumber>
- <date>Sometime in 2012</date>
+ <date>July 2012</date>
<revremark>Released with the Yocto Project 1.2.1 Release.</revremark>
</revision>
</revhistory>
diff --git a/documentation/dev-manual/dev-manual.xml b/documentation/dev-manual/dev-manual.xml
index 966e056..d8b48f4 100644
--- a/documentation/dev-manual/dev-manual.xml
+++ b/documentation/dev-manual/dev-manual.xml
@@ -41,7 +41,7 @@
</revision>
<revision>
<revnumber>1.2.1</revnumber>
- <date>Sometime in 2012</date>
+ <date>July 2012</date>
<revremark>Released with the Yocto Project 1.2.1 Release.</revremark>
</revision>
</revhistory>
diff --git a/documentation/kernel-manual/kernel-manual.xml b/documentation/kernel-manual/kernel-manual.xml
index 9fcadc2..0deef70 100644
--- a/documentation/kernel-manual/kernel-manual.xml
+++ b/documentation/kernel-manual/kernel-manual.xml
@@ -56,7 +56,7 @@
</revision>
<revision>
<revnumber>1.2.1</revnumber>
- <date>Sometime in 2012</date>
+ <date>July 2012</date>
<revremark>Released with the Yocto Project 1.2.1 Release.</revremark>
</revision>
</revhistory>
diff --git a/documentation/poky-ref-manual/poky-ref-manual.xml b/documentation/poky-ref-manual/poky-ref-manual.xml
index 76fc5eb..b4a120b 100644
--- a/documentation/poky-ref-manual/poky-ref-manual.xml
+++ b/documentation/poky-ref-manual/poky-ref-manual.xml
@@ -70,7 +70,7 @@
</revision>
<revision>
<revnumber>1.2.1</revnumber>
- <date>Sometime in 2012</date>
+ <date>July 2012</date>
<revremark>Released with the Yocto Project 1.2.1 Release.</revremark>
</revision>
</revhistory>
--
1.7.9.5
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH 2/8] openjade-native: Ensure we reautoconf the package
2012-06-18 19:56 [PATCH 0/8] denzil pull request Scott Garman
2012-06-18 19:56 ` [PATCH 1/8] documentation: Updated Manual revision history table Scott Garman
@ 2012-06-18 19:56 ` Scott Garman
2012-06-18 19:56 ` [PATCH 3/8] webkit-gtk: Apply work around for all PowerPC targets Scott Garman
` (5 subsequent siblings)
7 siblings, 0 replies; 18+ messages in thread
From: Scott Garman @ 2012-06-18 19:56 UTC (permalink / raw)
To: poky
From: Richard Purdie <richard.purdie@linuxfoundation.org>
Currently since configure.in in is in a subdirectory, we don't reautoconf the
recipe. We really need to do this, to update things like the libtool script used
and fix various issues such as those that could creep in if a reautoconf is
triggered for some reason. Since this source only calls AM_INIT_AUTOMAKE to gain the
PACKAGE and VERSION definitions and that macro now errors if Makefile.am doesn't
exist, we need to add these definitions manually.
These changes avoid failures like:
----
| ...
| DssslApp.cxx:117:36: error: 'PACKAGE' was not declared in this scope
| DssslApp.cxx:118:36: error: 'VERSION' was not declared in this scope
| make[2]: *** [DssslApp.lo] Error 1
----
(From OE-Core rev: 6f0d830ed047f700702645d454054dfd6fea6f25)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
.../openjade/openjade-1.3.2/reautoconf.patch | 83 ++++++++++++++++++++
.../openjade/openjade-native_1.3.2.bb | 5 ++
2 files changed, 88 insertions(+)
create mode 100644 meta/recipes-devtools/openjade/openjade-1.3.2/reautoconf.patch
diff --git a/meta/recipes-devtools/openjade/openjade-1.3.2/reautoconf.patch b/meta/recipes-devtools/openjade/openjade-1.3.2/reautoconf.patch
new file mode 100644
index 0000000..1a23a4a
--- /dev/null
+++ b/meta/recipes-devtools/openjade/openjade-1.3.2/reautoconf.patch
@@ -0,0 +1,83 @@
+Ensure we reautoconf the packag
+
+Currently since configure.in in is in a subdirectory, we don't reautoconf the
+recipe. We really need to do this, to update things like the libtool script used
+and fix various issues such as those that could creep in if a reautoconf is
+triggered for some reason. Since this source only calls AM_INIT_AUTOMAKE to
+gain the PACKAGE and VERSION definitions and that macro now errors if Makefile.am
+doesn't exist, we need to add these definitions manually.
+
+These changes avoid failures like:
+----
+| ...
+| DssslApp.cxx:117:36: error: 'PACKAGE' was not declared in this scope
+| DssslApp.cxx:118:36: error: 'VERSION' was not declared in this scope
+| make[2]: *** [DssslApp.lo] Error 1 ----
+
+Upstream-Status: Pending
+
+RP 2012/6/12
+
+Index: openjade-1.3.2/acinclude.m4
+===================================================================
+--- /dev/null 1970-01-01 00:00:00.000000000 +0000
++++ openjade-1.3.2/acinclude.m4 2012-06-12 12:48:54.871365344 +0000
+@@ -0,0 +1,39 @@
++dnl Configure-time switch with default
++dnl
++dnl Each switch defines an --enable-FOO and --disable-FOO option in
++dnl the resulting configure script.
++dnl
++dnl Usage:
++dnl smr_SWITCH(name, description, default, pos-def, neg-def)
++dnl
++dnl where:
++dnl
++dnl name name of switch; generates --enable-name & --disable-name
++dnl options
++dnl description help string is set to this prefixed by "enable" or
++dnl "disable", whichever is the non-default value
++dnl default either "on" or "off"; specifies default if neither
++dnl --enable-name nor --disable-name is specified
++dnl pos-def a symbol to AC_DEFINE if switch is on (optional)
++dnl neg-def a symbol to AC_DEFINE if switch is off (optional)
++dnl
++AC_DEFUN(smr_SWITCH, [
++ AC_MSG_CHECKING(whether to enable $2)
++ AC_ARG_ENABLE(
++ $1,
++ ifelse($3, on,
++ [ --disable-[$1] disable [$2]],
++ [ --enable-[$1] enable [$2]]),
++ [ if test "$enableval" = yes; then
++ AC_MSG_RESULT(yes)
++ ifelse($4, , , AC_DEFINE($4))
++ else
++ AC_MSG_RESULT(no)
++ ifelse($5, , , AC_DEFINE($5))
++ fi ],
++ ifelse($3, on,
++ [ AC_MSG_RESULT(yes)
++ ifelse($4, , , AC_DEFINE($4)) ],
++ [ AC_MSG_RESULT(no)
++ ifelse($5, , , AC_DEFINE($5))]))])
++
+Index: openjade-1.3.2/config/configure.in
+===================================================================
+--- openjade-1.3.2.orig/config/configure.in 2012-06-12 12:47:20.735365445 +0000
++++ openjade-1.3.2/config/configure.in 2012-06-12 12:48:17.507364080 +0000
+@@ -12,9 +12,12 @@
+ dnl Credits: this autoconf script was largely "inspired" <g> by the
+ dnl autoconf script around SP made by Henry Thompson.
+ dnl
+-AC_INIT(dsssl)
++AC_INIT([openjade], [1.3.2])
+ AC_CONFIG_AUX_DIR(config)
+-AM_INIT_AUTOMAKE( openjade, 1.3.2)
++AC_SUBST([PACKAGE], [openjade])
++AC_SUBST([VERSION], [1.3.2])
++AC_DEFINE_UNQUOTED(PACKAGE, "$PACKAGE")
++AC_DEFINE_UNQUOTED(VERSION, "$VERSION")
+ TOP=`pwd`
+ AC_SUBST(TOP)
+ dnl
diff --git a/meta/recipes-devtools/openjade/openjade-native_1.3.2.bb b/meta/recipes-devtools/openjade/openjade-native_1.3.2.bb
index 18d311e..5b29c1f 100644
--- a/meta/recipes-devtools/openjade/openjade-native_1.3.2.bb
+++ b/meta/recipes-devtools/openjade/openjade-native_1.3.2.bb
@@ -14,6 +14,7 @@ RDEPENDS_${PN} = "sgml-common-native"
SRC_URI = "${SOURCEFORGE_MIRROR}/openjade/openjade-${PV}.tar.gz \
file://makefile.patch \
+ file://reautoconf.patch \
file://user-declared-default-constructor.patch"
SRC_URI[md5sum] = "7df692e3186109cc00db6825b777201e"
@@ -38,6 +39,10 @@ CFLAGS =+ "-I${S}/include"
SSTATEPOSTINSTFUNCS += "openjade_sstate_postinst"
SYSROOT_PREPROCESS_FUNCS += "openjade_sysroot_preprocess"
+# configure.in needs to be reloacted to trigger reautoconf
+do_configure_prepend () {
+ cp ${S}/config/configure.in ${S}/
+}
# We need to do this else the source interdependencies aren't generated and
# build failures can result (e.g. zero size style/Makefile.dep file)
--
1.7.9.5
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH 3/8] webkit-gtk: Apply work around for all PowerPC targets
2012-06-18 19:56 [PATCH 0/8] denzil pull request Scott Garman
2012-06-18 19:56 ` [PATCH 1/8] documentation: Updated Manual revision history table Scott Garman
2012-06-18 19:56 ` [PATCH 2/8] openjade-native: Ensure we reautoconf the package Scott Garman
@ 2012-06-18 19:56 ` Scott Garman
2012-06-18 19:56 ` [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries Scott Garman
` (4 subsequent siblings)
7 siblings, 0 replies; 18+ messages in thread
From: Scott Garman @ 2012-06-18 19:56 UTC (permalink / raw)
To: poky
From: Gary Thomas <gary@mlbassoc.com>
The current patch for bug #1570 only applies to qemuppc but should be
applicable for all PowerPC targets. Also update the patch so that
only one language backend, either ICU or PANGO, is built.
Also remove some old customizations (dependencies on darwin) as these
should now be handled in a layer specific .bbappend file.
(From OE-Core rev: 2782caf0f07f61cab92d2981c5e82914f22bd700)
Signed-off-by: Gary Thomas <gary@mlbassoc.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
meta/recipes-sato/webkit/webkit-gtk_svn.bb | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/meta/recipes-sato/webkit/webkit-gtk_svn.bb b/meta/recipes-sato/webkit/webkit-gtk_svn.bb
index 9381486..716afce 100644
--- a/meta/recipes-sato/webkit/webkit-gtk_svn.bb
+++ b/meta/recipes-sato/webkit/webkit-gtk_svn.bb
@@ -7,10 +7,13 @@ LIC_FILES_CHKSUM = "file://Source/WebCore/rendering/RenderApplet.h;endline=22;md
file://Source/WebKit/gtk/webkit/webkit.h;endline=21;md5=b4fbe9f4a944f1d071dba1d2c76b3351 \
file://Source/JavaScriptCore/parser/Parser.h;endline=23;md5=2f3cff0ad0a9c486da5a376928973a90"
-DEPENDS = "zlib enchant gnome-keyring libsoup-2.4 curl icu libxml2 cairo libxslt libxt libidn gnutls gtk+ gstreamer gst-plugins-base flex-native gperf-native perl-native-runtime sqlite3"
+# Choice of language backends - icu has issues on Big Endian machines so use pango
+ICU_LIB = "icu"
+ICU_LIB_powerpc = "pango"
+
+DEPENDS = "zlib enchant gnome-keyring libsoup-2.4 curl libxml2 cairo libxslt libxt libidn gnutls \
+ gtk+ gstreamer gst-plugins-base flex-native gperf-native perl-native-runtime sqlite3 ${ICU_LIB}"
DEPENDS += " ${@base_contains('DISTRO_FEATURES', 'opengl', 'virtual/libgl', '', d)}"
-DEPENDS_darwin8 = "curl icu libxml2 cairo libxslt libidn gnutls gtk+ gstreamer flex-native gperf-native perl-native-runtime sqlite3"
-DEPENDS_append_qemuppc += "pango"
SRCREV_FORMAT = "source"
@@ -48,10 +51,10 @@ EXTRA_OECONF = "\
UNICODE_CFLAGS=-D_REENTRANT \
"
-#default unicode backend icu breaks in cross-compile when target and host owns different endian type
-EXTRA_OECONF_append_qemuppc += "--with-unicode-backend=glib"
+#default unicode backend icu breaks in cross-compile when target and host are different endian type
+EXTRA_OECONF_append_powerpc += "--with-unicode-backend=glib"
-CPPFLAGS_append_qemuppc += "-I${STAGING_INCDIR}/pango-1.0 \
+CPPFLAGS_append_powerpc += "-I${STAGING_INCDIR}/pango-1.0 \
-I${STAGING_LIBDIR}/glib-2.0/include \
-I${STAGING_INCDIR}/glib-2.0"
--
1.7.9.5
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries
2012-06-18 19:56 [PATCH 0/8] denzil pull request Scott Garman
` (2 preceding siblings ...)
2012-06-18 19:56 ` [PATCH 3/8] webkit-gtk: Apply work around for all PowerPC targets Scott Garman
@ 2012-06-18 19:56 ` Scott Garman
2012-06-21 15:51 ` Khem Raj
2012-06-18 19:56 ` [PATCH 5/8] eglibc: package mtrace separately Scott Garman
` (3 subsequent siblings)
7 siblings, 1 reply; 18+ messages in thread
From: Scott Garman @ 2012-06-18 19:56 UTC (permalink / raw)
To: poky
From: Laurentiu Palcu <laurentiu.palcu@intel.com>
If BBPATH references the working directory, the user is warned and asked
to fix the problem.
[Yocto #1465]
(From OE-Core rev: 73f10ae955ba217078fa2c5288b736ae8a30b184)
Signed-off-by: Laurentiu Palcu <laurentiu.palcu@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Reworked commit to fix merge conflicts with denzil branch.
Signed-off-by: Scott Garman <scott.a.garman@intel.com>
---
meta/classes/sanity.bbclass | 13 ++++++++++++-
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
index 9755694..298372b 100644
--- a/meta/classes/sanity.bbclass
+++ b/meta/classes/sanity.bbclass
@@ -310,7 +310,18 @@ def check_sanity(sanity_data):
if "." in data.getVar('PATH', sanity_data, True).split(":"):
messages = messages + "PATH contains '.' which will break the build, please remove this"
- if data.getVar('TARGET_ARCH', sanity_data, True) == "arm":
+ bbpaths = sanity_data.getVar('BBPATH', True).split(":")
+ if "." in bbpaths or "" in bbpaths:
+ # TODO: change the following message to fatal when all BBPATH issues
+ # are fixed
+ bb.warn("BBPATH references the current directory, either through " \
+ "an empty entry, or a '.'.\n\t This is unsafe and means your "\
+ "layer configuration is adding empty elements to BBPATH.\n\t "\
+ "Please check your layer.conf files and other BBPATH " \
+ "settings to remove the current working directory " \
+ "references.");
+
+ if sanity_data.getVar('TARGET_ARCH', True) == "arm":
# This path is no longer user-readable in modern (very recent) Linux
try:
if os.path.exists("/proc/sys/vm/mmap_min_addr"):
--
1.7.9.5
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH 5/8] eglibc: package mtrace separately
2012-06-18 19:56 [PATCH 0/8] denzil pull request Scott Garman
` (3 preceding siblings ...)
2012-06-18 19:56 ` [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries Scott Garman
@ 2012-06-18 19:56 ` Scott Garman
2012-06-18 19:56 ` [PATCH 6/8] poky-tiny: eliminate mtrace rdepends Scott Garman
` (2 subsequent siblings)
7 siblings, 0 replies; 18+ messages in thread
From: Scott Garman @ 2012-06-18 19:56 UTC (permalink / raw)
To: poky
From: Nitin A Kamble <nitin.a.kamble@intel.com>
add libc-mtrace as dependency for task-core-tools-debug
now eglibc-mtrace gets included in an sdk image and not in a non-sdk image.
This does not affect builds with uclibc.
This fixes bug: [YOCTO# 2374]
(From OE-Core rev: a2ad2870a54108027ed33e977470f5f72e0bce1b)
Signed-off-by: Nitin A Kamble <nitin.a.kamble@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
meta/recipes-core/eglibc/eglibc-package.inc | 7 ++++++-
meta/recipes-core/eglibc/eglibc_2.13.bb | 2 +-
meta/recipes-core/tasks/task-core-tools-debug.bb | 4 ++++
3 files changed, 11 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-core/eglibc/eglibc-package.inc b/meta/recipes-core/eglibc/eglibc-package.inc
index 0db221f..3c371a7 100644
--- a/meta/recipes-core/eglibc/eglibc-package.inc
+++ b/meta/recipes-core/eglibc/eglibc-package.inc
@@ -20,11 +20,12 @@ USE_LDCONFIG ?= "1"
PKGSUFFIX = ""
PKGSUFFIX_virtclass-nativesdk = "-nativesdk"
-PACKAGES = "${PN}-dbg ${PN} catchsegv${PKGSUFFIX} sln${PKGSUFFIX} nscd${PKGSUFFIX} ldd${PKGSUFFIX} ${PN}-utils eglibc-extra-nss${PKGSUFFIX} eglibc-thread-db${PKGSUFFIX} ${PN}-pic ${PN}-dev ${PN}-staticdev ${PN}-doc libcidn${PKGSUFFIX} libmemusage${PKGSUFFIX} libsegfault${PKGSUFFIX} ${PN}-pcprofile libsotruss${PKGSUFFIX}"
+PACKAGES = "${PN}-dbg ${PN} catchsegv${PKGSUFFIX} sln${PKGSUFFIX} nscd${PKGSUFFIX} ldd${PKGSUFFIX} ${PN}-mtrace ${PN}-utils eglibc-extra-nss${PKGSUFFIX} eglibc-thread-db${PKGSUFFIX} ${PN}-pic ${PN}-dev ${PN}-staticdev ${PN}-doc libcidn${PKGSUFFIX} libmemusage${PKGSUFFIX} libsegfault${PKGSUFFIX} ${PN}-pcprofile libsotruss${PKGSUFFIX}"
# The ld.so in this eglibc supports the GNU_HASH
RPROVIDES_${PN} = "glibc${PKGSUFFIX} rtld(GNU_HASH)"
RPROVIDES_${PN}-utils = "glibc${PKGSUFFIX}-utils"
+RPROVIDES_${PN}-mtrace = "glibc${PKGSUFFIX}-mtrace libc-mtrace"
RPROVIDES_${PN}-pic = "glibc${PKGSUFFIX}-pic"
RPROVIDES_${PN}-dev = "glibc${PKGSUFFIX}-dev"
RPROVIDES_${PN}-staticdev = "glibc${PKGSUFFIX}-staticdev"
@@ -48,11 +49,13 @@ FILES_${PN}-dev_append += "${bindir}/rpcgen ${libdir}/*_nonshared.a \
${base_libdir}/*_nonshared.a ${base_libdir}/*.o ${datadir}/aclocal"
FILES_${PN}-staticdev_append += "${libdir}/*.a ${base_libdir}/*.a"
FILES_nscd${PKGSUFFIX} = "${sbindir}/nscd*"
+FILES_${PN}-mtrace = "${bindir}/mtrace"
FILES_${PN}-utils = "${bindir}/* ${sbindir}/*"
FILES_${PN}-dbg += "${libexecdir}/*/.debug ${libdir}/audit/.debug"
FILES_catchsegv${PKGSUFFIX} = "${bindir}/catchsegv"
RDEPENDS_catchsegv${PKGSUFFIX} = "libsegfault"
RDEPENDS_${PN}-utils += "bash"
+RDEPENDS_${PN}-mtrace += "perl"
FILES_${PN}-pcprofile = "${base_libdir}/libpcprofile.so"
FILES_eglibc-thread-db${PKGSUFFIX} = "${base_libdir}/libthread_db.so.* ${base_libdir}/libthread_db-*.so"
RPROVIDES_${PN}-dev += "libc-dev"
@@ -68,6 +71,8 @@ SUMMARY_ldd = "print shared library dependencies"
DESCRIPTION_ldd = "/usr/bin/ldd prints shared library dependencies for each program or shared library specified on the command line."
SUMMARY_${PN}-utils = "Miscellaneous utilities provided by eglibc"
DESCRIPTION_${PN}-utils = "Miscellaneous utilities including getconf, iconf, locale, gencat, tzselect, zic, rpcinfo, ..."
+SUMMARY_${PN}-mtrace = "mtrace utility provided by eglibc"
+DESCRIPTION_${PN}-mtrace = "mtrace utility provided by eglibc"
DESCRIPTION_libsotruss = "Library to support sotruss which traces calls through PLTs"
inherit libc-common multilib_header
diff --git a/meta/recipes-core/eglibc/eglibc_2.13.bb b/meta/recipes-core/eglibc/eglibc_2.13.bb
index d8a41dc..73cd0aa 100644
--- a/meta/recipes-core/eglibc/eglibc_2.13.bb
+++ b/meta/recipes-core/eglibc/eglibc_2.13.bb
@@ -3,7 +3,7 @@ require eglibc.inc
SRCREV = "15508"
DEPENDS += "gperf-native"
-PR = "r26"
+PR = "r27"
PR_append = "+svnr${SRCPV}"
EGLIBC_BRANCH="eglibc-2_13"
diff --git a/meta/recipes-core/tasks/task-core-tools-debug.bb b/meta/recipes-core/tasks/task-core-tools-debug.bb
index 3c92f3f..1dddbb8 100644
--- a/meta/recipes-core/tasks/task-core-tools-debug.bb
+++ b/meta/recipes-core/tasks/task-core-tools-debug.bb
@@ -17,10 +17,14 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
ALLOW_EMPTY = "1"
+MTRACE = ""
+MTRACE_libc-glibc = "libc-mtrace"
+
RDEPENDS_${PN} = "\
gdb \
gdbserver \
tcf-agent \
rsync \
strace \
+ ${MTRACE} \
"
--
1.7.9.5
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH 6/8] poky-tiny: eliminate mtrace rdepends
2012-06-18 19:56 [PATCH 0/8] denzil pull request Scott Garman
` (4 preceding siblings ...)
2012-06-18 19:56 ` [PATCH 5/8] eglibc: package mtrace separately Scott Garman
@ 2012-06-18 19:56 ` Scott Garman
2012-06-18 19:56 ` [PATCH 7/8] quilt: added ac_cv_path_BASH to CACHED_CONFIGUREVARS Scott Garman
2012-06-18 19:56 ` [PATCH 8/8] eglibc: added ac_cv_path_ " Scott Garman
7 siblings, 0 replies; 18+ messages in thread
From: Scott Garman @ 2012-06-18 19:56 UTC (permalink / raw)
To: poky
From: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
meta-yocto/conf/distro/poky-tiny.conf | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/meta-yocto/conf/distro/poky-tiny.conf b/meta-yocto/conf/distro/poky-tiny.conf
index 8aa8383..a34c7dc 100644
--- a/meta-yocto/conf/distro/poky-tiny.conf
+++ b/meta-yocto/conf/distro/poky-tiny.conf
@@ -114,3 +114,8 @@ IMAGE_FSTYPES = "ext2 cpio.gz"
# variants of all compatible machines, but that leads to a lot
# more machine configs to maintain long term.
MACHINE_ESSENTIAL_EXTRA_RDEPENDS = ""
+
+# The mtrace script included by eglibc is a perl script. This means the system
+# will build perl in case this package is installed. Since we don't care about
+# this script for the purposes of tiny, remove the dependency from here.
+RDEPENDS_${PN}-mtrace_pn-eglibc = ""
--
1.7.9.5
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH 7/8] quilt: added ac_cv_path_BASH to CACHED_CONFIGUREVARS
2012-06-18 19:56 [PATCH 0/8] denzil pull request Scott Garman
` (5 preceding siblings ...)
2012-06-18 19:56 ` [PATCH 6/8] poky-tiny: eliminate mtrace rdepends Scott Garman
@ 2012-06-18 19:56 ` Scott Garman
2012-06-18 19:56 ` [PATCH 8/8] eglibc: added ac_cv_path_ " Scott Garman
7 siblings, 0 replies; 18+ messages in thread
From: Scott Garman @ 2012-06-18 19:56 UTC (permalink / raw)
To: poky
From: Saul Wold <sgw@linux.intel.com>
On Fedora 17, bash has moved to /usr/bin/bash and the configure process finds it
on the host machine there, this ensures that it is set correctly for the target.
[YOCTO #2363]
(From OE-Core rev: 958934049e4b3b84e5eaa9c6b0d672cf2c4a693e)
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Reworked commit to fix merge conflicts with denzil branch.
Signed-off-by: Scott Garman <scott.a.garman@intel.com>
---
meta/recipes-devtools/quilt/quilt_0.51.bb | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/meta/recipes-devtools/quilt/quilt_0.51.bb b/meta/recipes-devtools/quilt/quilt_0.51.bb
index 4e51b49..81a7a9a 100644
--- a/meta/recipes-devtools/quilt/quilt_0.51.bb
+++ b/meta/recipes-devtools/quilt/quilt_0.51.bb
@@ -4,7 +4,10 @@ RDEPENDS_${PN} += "patch diffstat bzip2 util-linux"
SRC_URI += "file://aclocal.patch \
file://gnu_patch_test_fix_target.patch \
"
-PR = "r4"
+PR = "r5"
+
+
+CACHED_CONFIGUREVARS += "ac_cv_path_BASH=${base_bindir}/bash"
PERLPATH = "${bindir}/env perl"
PERLPATH_virtclass-nativesdk = "/usr/bin/env perl"
--
1.7.9.5
^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH 8/8] eglibc: added ac_cv_path_ to CACHED_CONFIGUREVARS
2012-06-18 19:56 [PATCH 0/8] denzil pull request Scott Garman
` (6 preceding siblings ...)
2012-06-18 19:56 ` [PATCH 7/8] quilt: added ac_cv_path_BASH to CACHED_CONFIGUREVARS Scott Garman
@ 2012-06-18 19:56 ` Scott Garman
7 siblings, 0 replies; 18+ messages in thread
From: Scott Garman @ 2012-06-18 19:56 UTC (permalink / raw)
To: poky
From: Saul Wold <sgw@linux.intel.com>
On Fedora 17, bash has moved to /usr/bin/bash and the configure process finds it
on the host machine there, this ensures that it is set correctly for the target.
[YOCTO #2363]
(From OE-Core rev: 133c2a12d13028a3829c87a59a2ff4fe21f57260)
Signed-off-by: Saul Wold <sgw@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Reworked commit to fix merge conflicts with denzil branch.
Signed-off-by: Scott Garman <scott.a.garman@intel.com>
---
meta/recipes-core/eglibc/eglibc.inc | 2 ++
meta/recipes-core/eglibc/eglibc_2.13.bb | 2 +-
meta/recipes-core/eglibc/eglibc_2.15.bb | 2 +-
3 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-core/eglibc/eglibc.inc b/meta/recipes-core/eglibc/eglibc.inc
index 8e3fe50..51061bc 100644
--- a/meta/recipes-core/eglibc/eglibc.inc
+++ b/meta/recipes-core/eglibc/eglibc.inc
@@ -32,6 +32,8 @@ require eglibc-options.inc
LEAD_SONAME = "libc.so"
+CACHED_CONFIGUREVARS += "ac_cv_path_KSH=${base_bindir}/bash \
+ ac_cv_path_BASH_SHELL=${base_bindir}/bash"
GLIBC_EXTRA_OECONF ?= ""
GLIBC_EXTRA_OECONF_virtclass-nativesdk = ""
INHIBIT_DEFAULT_DEPS = "1"
diff --git a/meta/recipes-core/eglibc/eglibc_2.13.bb b/meta/recipes-core/eglibc/eglibc_2.13.bb
index 73cd0aa..690f99d 100644
--- a/meta/recipes-core/eglibc/eglibc_2.13.bb
+++ b/meta/recipes-core/eglibc/eglibc_2.13.bb
@@ -3,7 +3,7 @@ require eglibc.inc
SRCREV = "15508"
DEPENDS += "gperf-native"
-PR = "r27"
+PR = "r28"
PR_append = "+svnr${SRCPV}"
EGLIBC_BRANCH="eglibc-2_13"
diff --git a/meta/recipes-core/eglibc/eglibc_2.15.bb b/meta/recipes-core/eglibc/eglibc_2.15.bb
index 713efc3..f5219d1 100644
--- a/meta/recipes-core/eglibc/eglibc_2.15.bb
+++ b/meta/recipes-core/eglibc/eglibc_2.15.bb
@@ -3,7 +3,7 @@ require eglibc.inc
SRCREV = "17386"
DEPENDS += "gperf-native"
-PR = "r6"
+PR = "r7"
PR_append = "+svnr${SRCPV}"
EGLIBC_BRANCH="eglibc-2_15"
--
1.7.9.5
^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries
2012-06-18 19:56 ` [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries Scott Garman
@ 2012-06-21 15:51 ` Khem Raj
2012-06-21 20:32 ` Andrei Gherzan
2012-06-21 20:57 ` Scott Garman
0 siblings, 2 replies; 18+ messages in thread
From: Khem Raj @ 2012-06-21 15:51 UTC (permalink / raw)
To: Scott Garman; +Cc: poky
On Mon, Jun 18, 2012 at 12:56 PM, Scott Garman <scott.a.garman@intel.com> wrote:
> From: Laurentiu Palcu <laurentiu.palcu@intel.com>
>
> If BBPATH references the working directory, the user is warned and asked
> to fix the problem.
>
> [Yocto #1465]
>
> (From OE-Core rev: 73f10ae955ba217078fa2c5288b736ae8a30b184)
>
> Signed-off-by: Laurentiu Palcu <laurentiu.palcu@intel.com>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>
> Reworked commit to fix merge conflicts with denzil branch.
>
> Signed-off-by: Scott Garman <scott.a.garman@intel.com>
> ---
> meta/classes/sanity.bbclass | 13 ++++++++++++-
> 1 file changed, 12 insertions(+), 1 deletion(-)
>
> diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
> index 9755694..298372b 100644
> --- a/meta/classes/sanity.bbclass
> +++ b/meta/classes/sanity.bbclass
> @@ -310,7 +310,18 @@ def check_sanity(sanity_data):
> if "." in data.getVar('PATH', sanity_data, True).split(":"):
> messages = messages + "PATH contains '.' which will break the build, please remove this"
>
> - if data.getVar('TARGET_ARCH', sanity_data, True) == "arm":
> + bbpaths = sanity_data.getVar('BBPATH', True).split(":")
> + if "." in bbpaths or "" in bbpaths:
> + # TODO: change the following message to fatal when all BBPATH issues
> + # are fixed
> + bb.warn("BBPATH references the current directory, either through " \
> + "an empty entry, or a '.'.\n\t This is unsafe and means your "\
> + "layer configuration is adding empty elements to BBPATH.\n\t "\
> + "Please check your layer.conf files and other BBPATH " \
> + "settings to remove the current working directory " \
> + "references.");
> +
> + if sanity_data.getVar('TARGET_ARCH', True) == "arm":
No I get this warning when I use OE-Core standalone. The reason is for OE-Core
BBPATH .= ":${LAYERDIR}"
and there are no more entries prepended since its the only layer so
leading : is causing
this warning to show up.
> # This path is no longer user-readable in modern (very recent) Linux
> try:
> if os.path.exists("/proc/sys/vm/mmap_min_addr"):
> --
> 1.7.9.5
>
> _______________________________________________
> poky mailing list
> poky@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/poky
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries
2012-06-21 15:51 ` Khem Raj
@ 2012-06-21 20:32 ` Andrei Gherzan
2012-06-21 21:00 ` Andrei Gherzan
2012-06-21 20:57 ` Scott Garman
1 sibling, 1 reply; 18+ messages in thread
From: Andrei Gherzan @ 2012-06-21 20:32 UTC (permalink / raw)
To: Khem Raj; +Cc: poky
[-- Attachment #1: Type: text/plain, Size: 2358 bytes --]
On Thu, Jun 21, 2012 at 6:51 PM, Khem Raj <raj.khem@gmail.com> wrote:
> On Mon, Jun 18, 2012 at 12:56 PM, Scott Garman <scott.a.garman@intel.com>
> wrote:
> > From: Laurentiu Palcu <laurentiu.palcu@intel.com>
> >
> > If BBPATH references the working directory, the user is warned and asked
> > to fix the problem.
> >
> > [Yocto #1465]
> >
> > (From OE-Core rev: 73f10ae955ba217078fa2c5288b736ae8a30b184)
> >
> > Signed-off-by: Laurentiu Palcu <laurentiu.palcu@intel.com>
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> >
> > Reworked commit to fix merge conflicts with denzil branch.
> >
> > Signed-off-by: Scott Garman <scott.a.garman@intel.com>
> > ---
> > meta/classes/sanity.bbclass | 13 ++++++++++++-
> > 1 file changed, 12 insertions(+), 1 deletion(-)
> >
> > diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
> > index 9755694..298372b 100644
> > --- a/meta/classes/sanity.bbclass
> > +++ b/meta/classes/sanity.bbclass
> > @@ -310,7 +310,18 @@ def check_sanity(sanity_data):
> > if "." in data.getVar('PATH', sanity_data, True).split(":"):
> > messages = messages + "PATH contains '.' which will break the
> build, please remove this"
> >
> > - if data.getVar('TARGET_ARCH', sanity_data, True) == "arm":
> > + bbpaths = sanity_data.getVar('BBPATH', True).split(":")
> > + if "." in bbpaths or "" in bbpaths:
> > + # TODO: change the following message to fatal when all BBPATH
> issues
> > + # are fixed
> > + bb.warn("BBPATH references the current directory, either
> through " \
> > + "an empty entry, or a '.'.\n\t This is unsafe and means
> your "\
> > + "layer configuration is adding empty elements to
> BBPATH.\n\t "\
> > + "Please check your layer.conf files and other BBPATH "
> \
> > + "settings to remove the current working directory "
> \
> > + "references.");
> > +
> > + if sanity_data.getVar('TARGET_ARCH', True) == "arm":
>
> No I get this warning when I use OE-Core standalone. The reason is for
> OE-Core
>
> BBPATH .= ":${LAYERDIR}"
>
> and there are no more entries prepended since its the only layer so
> leading : is causing
> this warning to show up.
>
>
I confirm this.
@g
[-- Attachment #2: Type: text/html, Size: 3307 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries
2012-06-21 15:51 ` Khem Raj
2012-06-21 20:32 ` Andrei Gherzan
@ 2012-06-21 20:57 ` Scott Garman
1 sibling, 0 replies; 18+ messages in thread
From: Scott Garman @ 2012-06-21 20:57 UTC (permalink / raw)
To: Khem Raj; +Cc: poky
On 06/21/2012 08:51 AM, Khem Raj wrote:
> On Mon, Jun 18, 2012 at 12:56 PM, Scott Garman<scott.a.garman@intel.com> wrote:
>> From: Laurentiu Palcu<laurentiu.palcu@intel.com>
>>
>> If BBPATH references the working directory, the user is warned and asked
>> to fix the problem.
>>
>> [Yocto #1465]
>>
>> (From OE-Core rev: 73f10ae955ba217078fa2c5288b736ae8a30b184)
>>
>> Signed-off-by: Laurentiu Palcu<laurentiu.palcu@intel.com>
>> Signed-off-by: Richard Purdie<richard.purdie@linuxfoundation.org>
>>
>> Reworked commit to fix merge conflicts with denzil branch.
>>
>> Signed-off-by: Scott Garman<scott.a.garman@intel.com>
>> ---
>> meta/classes/sanity.bbclass | 13 ++++++++++++-
>> 1 file changed, 12 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
>> index 9755694..298372b 100644
>> --- a/meta/classes/sanity.bbclass
>> +++ b/meta/classes/sanity.bbclass
>> @@ -310,7 +310,18 @@ def check_sanity(sanity_data):
>> if "." in data.getVar('PATH', sanity_data, True).split(":"):
>> messages = messages + "PATH contains '.' which will break the build, please remove this"
>>
>> - if data.getVar('TARGET_ARCH', sanity_data, True) == "arm":
>> + bbpaths = sanity_data.getVar('BBPATH', True).split(":")
>> + if "." in bbpaths or "" in bbpaths:
>> + # TODO: change the following message to fatal when all BBPATH issues
>> + # are fixed
>> + bb.warn("BBPATH references the current directory, either through " \
>> + "an empty entry, or a '.'.\n\t This is unsafe and means your "\
>> + "layer configuration is adding empty elements to BBPATH.\n\t "\
>> + "Please check your layer.conf files and other BBPATH " \
>> + "settings to remove the current working directory " \
>> + "references.");
>> +
>> + if sanity_data.getVar('TARGET_ARCH', True) == "arm":
>
> No I get this warning when I use OE-Core standalone. The reason is for OE-Core
>
> BBPATH .= ":${LAYERDIR}"
>
> and there are no more entries prepended since its the only layer so
> leading : is causing
> this warning to show up.
>
>> # This path is no longer user-readable in modern (very recent) Linux
>> try:
>> if os.path.exists("/proc/sys/vm/mmap_min_addr"):
>> --
>> 1.7.9.5
Richard didn't end up pulling this in, and explained his reasoning on
the oe-core ML. So no worries.
Scott
--
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries
2012-06-21 20:32 ` Andrei Gherzan
@ 2012-06-21 21:00 ` Andrei Gherzan
2012-06-21 21:14 ` Martin Jansa
0 siblings, 1 reply; 18+ messages in thread
From: Andrei Gherzan @ 2012-06-21 21:00 UTC (permalink / raw)
To: Khem Raj; +Cc: poky
[-- Attachment #1: Type: text/plain, Size: 3501 bytes --]
In a qemumips build with master poky:
WARNING: BBPATH references the current directory, either through an empty
entry, or a '.'.
This is unsafe and means your layer configuration is adding empty elements
to BBPATH.
Please check your layer.conf files and other BBPATH settings to remove the
current working directory references.
And this is because BBPATH contains:
/home/agherzan/work/personal/yocto/poky-contrib/meta-yocto::/home/agherzan/work/personal/yocto/poky-contrib/meta
Well it's strange that:
meta: BBPATH .= ":${LAYERDIR}"
meta-yocto: BBPATH := "${LAYERDIR}:${BBPATH}"
This explains the way BBPATH above ended
like: /home/agherzan/work/personal/yocto/poky-contrib/meta-yocto::/home/agherzan/work/personal/yocto/poky-contrib/meta
Why don't we assign BBPATH as this in meta:
BBPATH .= "${LAYERDIR}"
And then in all other layers use: meta-yocto: BBPATH :=
"${LAYERDIR}:${BBPATH}". In this way the variable would be correctly
generated.
@g
On Thu, Jun 21, 2012 at 11:32 PM, Andrei Gherzan <andrei@gherzan.ro> wrote:
> On Thu, Jun 21, 2012 at 6:51 PM, Khem Raj <raj.khem@gmail.com> wrote:
>
>> On Mon, Jun 18, 2012 at 12:56 PM, Scott Garman <scott.a.garman@intel.com>
>> wrote:
>> > From: Laurentiu Palcu <laurentiu.palcu@intel.com>
>> >
>> > If BBPATH references the working directory, the user is warned and asked
>> > to fix the problem.
>> >
>> > [Yocto #1465]
>> >
>> > (From OE-Core rev: 73f10ae955ba217078fa2c5288b736ae8a30b184)
>> >
>> > Signed-off-by: Laurentiu Palcu <laurentiu.palcu@intel.com>
>> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>> >
>> > Reworked commit to fix merge conflicts with denzil branch.
>> >
>> > Signed-off-by: Scott Garman <scott.a.garman@intel.com>
>> > ---
>> > meta/classes/sanity.bbclass | 13 ++++++++++++-
>> > 1 file changed, 12 insertions(+), 1 deletion(-)
>> >
>> > diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
>> > index 9755694..298372b 100644
>> > --- a/meta/classes/sanity.bbclass
>> > +++ b/meta/classes/sanity.bbclass
>> > @@ -310,7 +310,18 @@ def check_sanity(sanity_data):
>> > if "." in data.getVar('PATH', sanity_data, True).split(":"):
>> > messages = messages + "PATH contains '.' which will break the
>> build, please remove this"
>> >
>> > - if data.getVar('TARGET_ARCH', sanity_data, True) == "arm":
>> > + bbpaths = sanity_data.getVar('BBPATH', True).split(":")
>> > + if "." in bbpaths or "" in bbpaths:
>> > + # TODO: change the following message to fatal when all BBPATH
>> issues
>> > + # are fixed
>> > + bb.warn("BBPATH references the current directory, either
>> through " \
>> > + "an empty entry, or a '.'.\n\t This is unsafe and
>> means your "\
>> > + "layer configuration is adding empty elements to
>> BBPATH.\n\t "\
>> > + "Please check your layer.conf files and other BBPATH "
>> \
>> > + "settings to remove the current working directory "
>> \
>> > + "references.");
>> > +
>> > + if sanity_data.getVar('TARGET_ARCH', True) == "arm":
>>
>> No I get this warning when I use OE-Core standalone. The reason is for
>> OE-Core
>>
>> BBPATH .= ":${LAYERDIR}"
>>
>> and there are no more entries prepended since its the only layer so
>> leading : is causing
>> this warning to show up.
>>
>>
> I confirm this.
>
> @g
>
>
[-- Attachment #2: Type: text/html, Size: 5338 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries
2012-06-21 21:00 ` Andrei Gherzan
@ 2012-06-21 21:14 ` Martin Jansa
2012-06-21 21:24 ` Andrei Gherzan
0 siblings, 1 reply; 18+ messages in thread
From: Martin Jansa @ 2012-06-21 21:14 UTC (permalink / raw)
To: Andrei Gherzan; +Cc: poky
[-- Attachment #1: Type: text/plain, Size: 4340 bytes --]
On Fri, Jun 22, 2012 at 12:00:09AM +0300, Andrei Gherzan wrote:
> In a qemumips build with master poky:
>
> WARNING: BBPATH references the current directory, either through an empty
> entry, or a '.'.
> This is unsafe and means your layer configuration is adding empty elements
> to BBPATH.
> Please check your layer.conf files and other BBPATH settings to remove the
> current working directory references.
>
> And this is because BBPATH contains:
> /home/agherzan/work/personal/yocto/poky-contrib/meta-yocto::/home/agherzan/work/personal/yocto/poky-contrib/meta
>
> Well it's strange that:
> meta: BBPATH .= ":${LAYERDIR}"
> meta-yocto: BBPATH := "${LAYERDIR}:${BBPATH}"
>
> This explains the way BBPATH above ended
> like: /home/agherzan/work/personal/yocto/poky-contrib/meta-yocto::/home/agherzan/work/personal/yocto/poky-contrib/meta
>
> Why don't we assign BBPATH as this in meta:
> BBPATH .= "${LAYERDIR}"
>
> And then in all other layers use: meta-yocto: BBPATH :=
> "${LAYERDIR}:${BBPATH}". In this way the variable would be correctly
> generated.
IMHO it's better to change meta-yocto/conf/layer to
BBPATH .= ":${LAYERDIR}"
so the BBPATH order is the same as order of layers in bblayers.conf
I've changed it lately in
meta-initramfs http://patchwork.openembedded.org/patch/28487/
meta-handheld http://patchwork.openembedded.org/patch/28485/
so it would be more consistent to change it in meta-yocto too.
Cheers,
> On Thu, Jun 21, 2012 at 11:32 PM, Andrei Gherzan <andrei@gherzan.ro> wrote:
>
> > On Thu, Jun 21, 2012 at 6:51 PM, Khem Raj <raj.khem@gmail.com> wrote:
> >
> >> On Mon, Jun 18, 2012 at 12:56 PM, Scott Garman <scott.a.garman@intel.com>
> >> wrote:
> >> > From: Laurentiu Palcu <laurentiu.palcu@intel.com>
> >> >
> >> > If BBPATH references the working directory, the user is warned and asked
> >> > to fix the problem.
> >> >
> >> > [Yocto #1465]
> >> >
> >> > (From OE-Core rev: 73f10ae955ba217078fa2c5288b736ae8a30b184)
> >> >
> >> > Signed-off-by: Laurentiu Palcu <laurentiu.palcu@intel.com>
> >> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> >> >
> >> > Reworked commit to fix merge conflicts with denzil branch.
> >> >
> >> > Signed-off-by: Scott Garman <scott.a.garman@intel.com>
> >> > ---
> >> > meta/classes/sanity.bbclass | 13 ++++++++++++-
> >> > 1 file changed, 12 insertions(+), 1 deletion(-)
> >> >
> >> > diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
> >> > index 9755694..298372b 100644
> >> > --- a/meta/classes/sanity.bbclass
> >> > +++ b/meta/classes/sanity.bbclass
> >> > @@ -310,7 +310,18 @@ def check_sanity(sanity_data):
> >> > if "." in data.getVar('PATH', sanity_data, True).split(":"):
> >> > messages = messages + "PATH contains '.' which will break the
> >> build, please remove this"
> >> >
> >> > - if data.getVar('TARGET_ARCH', sanity_data, True) == "arm":
> >> > + bbpaths = sanity_data.getVar('BBPATH', True).split(":")
> >> > + if "." in bbpaths or "" in bbpaths:
> >> > + # TODO: change the following message to fatal when all BBPATH
> >> issues
> >> > + # are fixed
> >> > + bb.warn("BBPATH references the current directory, either
> >> through " \
> >> > + "an empty entry, or a '.'.\n\t This is unsafe and
> >> means your "\
> >> > + "layer configuration is adding empty elements to
> >> BBPATH.\n\t "\
> >> > + "Please check your layer.conf files and other BBPATH "
> >> \
> >> > + "settings to remove the current working directory "
> >> \
> >> > + "references.");
> >> > +
> >> > + if sanity_data.getVar('TARGET_ARCH', True) == "arm":
> >>
> >> No I get this warning when I use OE-Core standalone. The reason is for
> >> OE-Core
> >>
> >> BBPATH .= ":${LAYERDIR}"
> >>
> >> and there are no more entries prepended since its the only layer so
> >> leading : is causing
> >> this warning to show up.
> >>
> >>
> > I confirm this.
> >
> > @g
> >
> >
> _______________________________________________
> poky mailing list
> poky@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/poky
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries
2012-06-21 21:14 ` Martin Jansa
@ 2012-06-21 21:24 ` Andrei Gherzan
2012-06-22 5:39 ` Martin Jansa
0 siblings, 1 reply; 18+ messages in thread
From: Andrei Gherzan @ 2012-06-21 21:24 UTC (permalink / raw)
To: Martin Jansa; +Cc: poky
[-- Attachment #1: Type: text/plain, Size: 1489 bytes --]
On Fri, Jun 22, 2012 at 12:14 AM, Martin Jansa <martin.jansa@gmail.com>wrote:
> On Fri, Jun 22, 2012 at 12:00:09AM +0300, Andrei Gherzan wrote:
> > In a qemumips build with master poky:
> >
> > WARNING: BBPATH references the current directory, either through an empty
> > entry, or a '.'.
> > This is unsafe and means your layer configuration is adding empty
> elements
> > to BBPATH.
> > Please check your layer.conf files and other BBPATH settings to remove
> the
> > current working directory references.
> >
> > And this is because BBPATH contains:
> >
> /home/agherzan/work/personal/yocto/poky-contrib/meta-yocto::/home/agherzan/work/personal/yocto/poky-contrib/meta
> >
> > Well it's strange that:
> > meta: BBPATH .= ":${LAYERDIR}"
> > meta-yocto: BBPATH := "${LAYERDIR}:${BBPATH}"
> >
> > This explains the way BBPATH above ended
> > like:
> /home/agherzan/work/personal/yocto/poky-contrib/meta-yocto::/home/agherzan/work/personal/yocto/poky-contrib/meta
> >
> > Why don't we assign BBPATH as this in meta:
> > BBPATH .= "${LAYERDIR}"
> >
> > And then in all other layers use: meta-yocto: BBPATH :=
> > "${LAYERDIR}:${BBPATH}". In this way the variable would be correctly
> > generated.
>
> IMHO it's better to change meta-yocto/conf/layer to
> BBPATH .= ":${LAYERDIR}"
> so the BBPATH order is the same as order of layers in bblayers.conf
>
> This wouldn't fix it. As BBPATH would still have an empty value: the first
-> ":bla:bla"
@g
[-- Attachment #2: Type: text/html, Size: 1982 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries
2012-06-21 21:24 ` Andrei Gherzan
@ 2012-06-22 5:39 ` Martin Jansa
2012-06-22 5:58 ` Khem Raj
0 siblings, 1 reply; 18+ messages in thread
From: Martin Jansa @ 2012-06-22 5:39 UTC (permalink / raw)
To: Andrei Gherzan; +Cc: poky
[-- Attachment #1: Type: text/plain, Size: 3971 bytes --]
On Fri, Jun 22, 2012 at 12:24:05AM +0300, Andrei Gherzan wrote:
> On Fri, Jun 22, 2012 at 12:14 AM, Martin Jansa <martin.jansa@gmail.com>wrote:
>
> > On Fri, Jun 22, 2012 at 12:00:09AM +0300, Andrei Gherzan wrote:
> > > In a qemumips build with master poky:
> > >
> > > WARNING: BBPATH references the current directory, either through an empty
> > > entry, or a '.'.
> > > This is unsafe and means your layer configuration is adding empty
> > elements
> > > to BBPATH.
> > > Please check your layer.conf files and other BBPATH settings to remove
> > the
> > > current working directory references.
> > >
> > > And this is because BBPATH contains:
> > >
> > /home/agherzan/work/personal/yocto/poky-contrib/meta-yocto::/home/agherzan/work/personal/yocto/poky-contrib/meta
> > >
> > > Well it's strange that:
> > > meta: BBPATH .= ":${LAYERDIR}"
> > > meta-yocto: BBPATH := "${LAYERDIR}:${BBPATH}"
> > >
> > > This explains the way BBPATH above ended
> > > like:
> > /home/agherzan/work/personal/yocto/poky-contrib/meta-yocto::/home/agherzan/work/personal/yocto/poky-contrib/meta
> > >
> > > Why don't we assign BBPATH as this in meta:
> > > BBPATH .= "${LAYERDIR}"
> > >
> > > And then in all other layers use: meta-yocto: BBPATH :=
> > > "${LAYERDIR}:${BBPATH}". In this way the variable would be correctly
> > > generated.
> >
> > IMHO it's better to change meta-yocto/conf/layer to
> > BBPATH .= ":${LAYERDIR}"
> > so the BBPATH order is the same as order of layers in bblayers.conf
> >
> > This wouldn't fix it. As BBPATH would still have an empty value: the first
> -> ":bla:bla"
Yes, this alone won't fix it for everybody, but will allow you to define
such order of BBPATH appends, so that if you start with non-empty value
then it will work fine.
For example my setup has TOPDIR as first item in BBPATH:
bblayers.conf:
BBPATH ?= "${TOPDIR}"
BBFILES ?= ""
# Add your overlay location to BBLAYERS
# Make sure to have a conf/layers.conf in there
BBLAYERS = " \
${TOPDIR}/meta-jama \
${TOPDIR}/meta-smartphone/meta-shr \
${TOPDIR}/meta-smartphone/meta-aurora \
${TOPDIR}/meta-smartphone/meta-fso \
${TOPDIR}/meta-openembedded/meta-oe \
${TOPDIR}/meta-openembedded/toolchain-layer \
${TOPDIR}/meta-openembedded/meta-efl \
${TOPDIR}/meta-openembedded/meta-gnome \
${TOPDIR}/meta-openembedded/meta-gpe \
${TOPDIR}/meta-openembedded/meta-initramfs \
${TOPDIR}/meta-openembedded/meta-systemd \
${TOPDIR}/meta-smartphone/meta-osmocombb \
${TOPDIR}/meta-smartphone/meta-nokia \
${TOPDIR}/meta-smartphone/meta-htc \
${TOPDIR}/meta-smartphone/meta-palm \
${TOPDIR}/meta-smartphone/meta-openmoko \
${TOPDIR}/meta-smartphone/meta-samsung \
${TOPDIR}/meta-handheld \
${TOPDIR}/meta-browser \
${TOPDIR}/openembedded-core/meta \
"
and resulting BBPATH respects that correctly
bitbake -e | grep ^BBPATH | sed 's/:/:^M/g'
BBPATH="/OE/shr-core:
/OE/shr-core/meta-jama:
/OE/shr-core/meta-smartphone/meta-shr:
/OE/shr-core/meta-smartphone/meta-aurora:
/OE/shr-core/meta-smartphone/meta-fso:
/OE/shr-core/meta-openembedded/meta-oe:
/OE/shr-core/meta-openembedded/toolchain-layer:
/OE/shr-core/meta-openembedded/meta-efl:
/OE/shr-core/meta-openembedded/meta-gnome:
/OE/shr-core/meta-openembedded/meta-gpe:
/OE/shr-core/meta-openembedded/meta-initramfs:
/OE/shr-core/meta-openembedded/meta-systemd:
/OE/shr-core/meta-smartphone/meta-osmocombb:
/OE/shr-core/meta-smartphone/meta-nokia:
/OE/shr-core/meta-smartphone/meta-htc:
/OE/shr-core/meta-smartphone/meta-palm:
/OE/shr-core/meta-smartphone/meta-openmoko:
/OE/shr-core/meta-smartphone/meta-samsung:
/OE/shr-core/meta-handheld:
/OE/shr-core/meta-browser:
/OE/shr-core/openembedded-core/meta"
see https://bugzilla.yoctoproject.org/show_bug.cgi?id=1465 for few more
details and reason why this BBPATH check started.
Cheers,
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries
2012-06-22 5:39 ` Martin Jansa
@ 2012-06-22 5:58 ` Khem Raj
2012-06-22 14:57 ` Richard Purdie
0 siblings, 1 reply; 18+ messages in thread
From: Khem Raj @ 2012-06-22 5:58 UTC (permalink / raw)
To: Martin Jansa; +Cc: poky
On Thu, Jun 21, 2012 at 10:39 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
>
> bblayers.conf:
> BBPATH ?= "${TOPDIR}"
>
> BBFILES ?= ""
yeah I guess a way is to always have BBPATH non empty by default and
then layers append their respective
BBPATHS to it and all falls into place.
In OE-Core it still wont work since there BBPATH is empty by default
and we can not assume some initial value for
it since user may place OE-Core anywhere in his layer heirarchy
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries
2012-06-22 5:58 ` Khem Raj
@ 2012-06-22 14:57 ` Richard Purdie
0 siblings, 0 replies; 18+ messages in thread
From: Richard Purdie @ 2012-06-22 14:57 UTC (permalink / raw)
To: Khem Raj; +Cc: poky
On Thu, 2012-06-21 at 22:58 -0700, Khem Raj wrote:
> On Thu, Jun 21, 2012 at 10:39 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
> >
> > bblayers.conf:
> > BBPATH ?= "${TOPDIR}"
> >
> > BBFILES ?= ""
>
> yeah I guess a way is to always have BBPATH non empty by default and
> then layers append their respective
> BBPATHS to it and all falls into place.
>
> In OE-Core it still wont work since there BBPATH is empty by default
> and we can not assume some initial value for
> it since user may place OE-Core anywhere in his layer heirarchy
I've proposed a patch for this on the OE-Core list. It uses BBPATH ?=
"${TOPDIR}" which I came to independently but matches what Martin is
doing :). We need the local build directory in BBPATH for local.conf
Cheers,
Richard
^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2012-06-22 14:57 UTC | newest]
Thread overview: 18+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-18 19:56 [PATCH 0/8] denzil pull request Scott Garman
2012-06-18 19:56 ` [PATCH 1/8] documentation: Updated Manual revision history table Scott Garman
2012-06-18 19:56 ` [PATCH 2/8] openjade-native: Ensure we reautoconf the package Scott Garman
2012-06-18 19:56 ` [PATCH 3/8] webkit-gtk: Apply work around for all PowerPC targets Scott Garman
2012-06-18 19:56 ` [PATCH 4/8] sanity.bbclass: warn the user if BBPATH contains wrong entries Scott Garman
2012-06-21 15:51 ` Khem Raj
2012-06-21 20:32 ` Andrei Gherzan
2012-06-21 21:00 ` Andrei Gherzan
2012-06-21 21:14 ` Martin Jansa
2012-06-21 21:24 ` Andrei Gherzan
2012-06-22 5:39 ` Martin Jansa
2012-06-22 5:58 ` Khem Raj
2012-06-22 14:57 ` Richard Purdie
2012-06-21 20:57 ` Scott Garman
2012-06-18 19:56 ` [PATCH 5/8] eglibc: package mtrace separately Scott Garman
2012-06-18 19:56 ` [PATCH 6/8] poky-tiny: eliminate mtrace rdepends Scott Garman
2012-06-18 19:56 ` [PATCH 7/8] quilt: added ac_cv_path_BASH to CACHED_CONFIGUREVARS Scott Garman
2012-06-18 19:56 ` [PATCH 8/8] eglibc: added ac_cv_path_ " Scott Garman
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.