* [PATCH 1/8] base.bbclass: Commentary typo fix
2013-02-05 15:55 [PATCH 0/8] misc tweaks Bernhard Reutner-Fischer
@ 2013-02-05 15:55 ` Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 2/8] uclibc: add c6x arch Bernhard Reutner-Fischer
` (6 subsequent siblings)
7 siblings, 0 replies; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 15:55 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
---
meta/classes/base.bbclass | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index 4662d3b..8147a81 100644
--- a/meta/classes/base.bbclass
+++ b/meta/classes/base.bbclass
@@ -160,7 +160,7 @@ def pkgarch_mapping(d):
d.setVar("TUNE_PKGARCH", "armv7a")
def preferred_ml_updates(d):
- # If any PREFERRED_PROVIDER or PREFERRED_VERSIONS are set,
+ # If any PREFERRED_PROVIDER or PREFERRED_VERSION are set,
# we need to mirror these variables in the multilib case;
# likewise the PNBLACKLIST flags.
multilibs = d.getVar('MULTILIBS', True) or ""
--
1.7.10.4
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH 2/8] uclibc: add c6x arch
2013-02-05 15:55 [PATCH 0/8] misc tweaks Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 1/8] base.bbclass: Commentary typo fix Bernhard Reutner-Fischer
@ 2013-02-05 15:55 ` Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 3/8] uclibc: turn on kernel module support Bernhard Reutner-Fischer
` (5 subsequent siblings)
7 siblings, 0 replies; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 15:55 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
---
meta/recipes-core/uclibc/uclibc-config.inc | 1 +
1 file changed, 1 insertion(+)
diff --git a/meta/recipes-core/uclibc/uclibc-config.inc b/meta/recipes-core/uclibc/uclibc-config.inc
index fd36fa5..0c7a0ce 100644
--- a/meta/recipes-core/uclibc/uclibc-config.inc
+++ b/meta/recipes-core/uclibc/uclibc-config.inc
@@ -9,6 +9,7 @@ alpha \
arm \
avr32 \
bfin \
+c6x \
cris \
e1 \
frv \
--
1.7.10.4
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH 3/8] uclibc: turn on kernel module support
2013-02-05 15:55 [PATCH 0/8] misc tweaks Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 1/8] base.bbclass: Commentary typo fix Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 2/8] uclibc: add c6x arch Bernhard Reutner-Fischer
@ 2013-02-05 15:55 ` Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 4/8] Revert "Add ptest for glib." Bernhard Reutner-Fischer
` (4 subsequent siblings)
7 siblings, 0 replies; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 15:55 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
---
meta/recipes-core/uclibc/uclibc-git/uClibc.distro | 2 ++
1 file changed, 2 insertions(+)
diff --git a/meta/recipes-core/uclibc/uclibc-git/uClibc.distro b/meta/recipes-core/uclibc/uclibc-git/uClibc.distro
index 5affb5a..c309e6f 100644
--- a/meta/recipes-core/uclibc/uclibc-git/uClibc.distro
+++ b/meta/recipes-core/uclibc/uclibc-git/uClibc.distro
@@ -157,6 +157,8 @@ WARNINGS="-Wall"
DO_C99_MATH=y
UCLIBC_HAS_LONG_DOUBLE_MATH=y
UCLIBC_HAS_FENV=y
+UCLIBC_LINUX_MODULE_26=y
+# UCLIBC_LINUX_MODULE_24 is not set
UCLIBC_LINUX_SPECIFIC=y
UCLIBC_HAS_REALTIME=y
UCLIBC_HAS_ADVANCED_REALTIME=y
--
1.7.10.4
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 15:55 [PATCH 0/8] misc tweaks Bernhard Reutner-Fischer
` (2 preceding siblings ...)
2013-02-05 15:55 ` [PATCH 3/8] uclibc: turn on kernel module support Bernhard Reutner-Fischer
@ 2013-02-05 15:55 ` Bernhard Reutner-Fischer
2013-02-05 16:44 ` Richard Purdie
` (2 more replies)
2013-02-05 15:55 ` [PATCH 5/8] busybox: add ifup's ifstate dir to package Bernhard Reutner-Fischer
` (3 subsequent siblings)
7 siblings, 3 replies; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 15:55 UTC (permalink / raw)
To: openembedded-core
This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
This breaks core-image-sato-sdk for TCLIBC="uclibc".
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
---
.../glib-2.0/glib-2.0/Makefile-ptest.patch | 61 --------------------
meta/recipes-core/glib-2.0/glib-2.0/run-ptest | 3 -
meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb | 22 -------
3 files changed, 86 deletions(-)
delete mode 100644 meta/recipes-core/glib-2.0/glib-2.0/Makefile-ptest.patch
delete mode 100644 meta/recipes-core/glib-2.0/glib-2.0/run-ptest
diff --git a/meta/recipes-core/glib-2.0/glib-2.0/Makefile-ptest.patch b/meta/recipes-core/glib-2.0/glib-2.0/Makefile-ptest.patch
deleted file mode 100644
index cb36c09..0000000
--- a/meta/recipes-core/glib-2.0/glib-2.0/Makefile-ptest.patch
+++ /dev/null
@@ -1,61 +0,0 @@
-Add 'ptest' target to Makefile, to run tests without checking dependencies.
-Add 'install-ptest' target, to copy needed files to ptest dir.
-
-Signed-off-by: Björn Stenberg <bjst@enea.com>
-Upstream-status: Pending
----
- Makefile.decl | 43 +++++++++++++++++++++++++++++++++++++++++++
- 1 file changed, 43 insertions(+)
-
---- a/Makefile.decl 2012-03-12 01:42:39.000000000 +0100
-+++ b/Makefile.decl 2012-11-26 13:42:08.436865192 +0100
-@@ -25,6 +25,49 @@
- test-nonrecursive:
- endif
-
-+ptest: ptest-nonrecursive
-+ @ for subdir in $(SUBDIRS) .; do \
-+ test -d "$$subdir" -a "$$subdir" != "." -a "$$subdir" != "po" && \
-+ ( test -d $$subdir && cd $$subdir && $(MAKE) $(AM_MAKEFLAGS) $@ ) \
-+ done
-+
-+ptest-nonrecursive:
-+ @test -z "${TEST_PROGS}" || ${GTESTER} --keep-going --verbose ${TEST_PROGS}
-+
-+install-ptest: install-ptest-nonrecursive
-+ @test -z "$(SUBDIRS)" || for subdir in $(SUBDIRS); do \
-+ test "$$subdir" == "." || \
-+ (cd $$subdir && $(MAKE) DESTDIR=$(DESTDIR)/$$subdir $@ ) \
-+ done
-+
-+# install files used for ptest:
-+# - copy EXTRA_DIST (for test data in glib/tests)
-+# - run dist-hook (for glib/tests/bookmarks/ and glib/tests/markups)
-+# - copy TEST_PROGS (the test programs)
-+# - copy INSTALL_PROGS (for glib/gtester)
-+# - avoid libtool wrappers by copying binaries from .libs
-+# - strip Makefile: target from all Makefiles
-+install-ptest-nonrecursive:
-+ @mkdir -p $(DESTDIR)
-+ @if [ -n "${TEST_PROGS}" ]; then \
-+ if [ -n "${EXTRA_DIST}" ]; then \
-+ cp ${EXTRA_DIST} $(DESTDIR); \
-+ fi; \
-+ grep -q dist-hook: Makefile && \
-+ $(MAKE) distdir=$(DESTDIR) dist-hook; \
-+ if [ -d .libs ] ; then \
-+ cd .libs; \
-+ fi; \
-+ cp ${TEST_PROGS} $(DESTDIR); \
-+ fi;
-+ @if [ -n "${INSTALL_PROGS}" ]; then \
-+ if [ -d .libs ] ; then \
-+ cd .libs; \
-+ fi; \
-+ cp ${INSTALL_PROGS} $(DESTDIR); \
-+ fi
-+ @test -f Makefile && sed 's/^Makefile:/_Makefile:/' <Makefile >$(DESTDIR)/Makefile
-+
- # test-report: run tests in subdirs and generate report
- # perf-report: run tests in subdirs with -m perf and generate report
- # full-report: like test-report: with -m perf and -m slow
diff --git a/meta/recipes-core/glib-2.0/glib-2.0/run-ptest b/meta/recipes-core/glib-2.0/glib-2.0/run-ptest
deleted file mode 100644
index 3deb586..0000000
--- a/meta/recipes-core/glib-2.0/glib-2.0/run-ptest
+++ /dev/null
@@ -1,3 +0,0 @@
-#!/bin/sh
-
-make -k ptest
diff --git a/meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb b/meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb
index e1d44e2..e4293c8 100644
--- a/meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb
+++ b/meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb
@@ -13,8 +13,6 @@ SRC_URI = "${GNOME_MIRROR}/glib/${SHRT_VER}/glib-${PV}.tar.xz \
file://configure-libtool.patch \
file://glib-2.0_fix_for_x32.patch \
file://obsolete_automake_macros.patch \
- file://Makefile-ptest.patch \
- file://run-ptest \
"
SRC_URI[md5sum] = "a4ca31e258273c3761e3de2edd607661"
SRC_URI[sha256sum] = "855fcbf87cb93065b488358e351774d8a39177281023bae58c286f41612658a7"
@@ -22,24 +20,6 @@ SRC_URI[sha256sum] = "855fcbf87cb93065b488358e351774d8a39177281023bae58c286f4161
SRC_URI_append_class-native = " file://glib-gettextize-dir.patch"
BBCLASSEXTEND = "native nativesdk"
-RDEPENDS_${PN}-ptest += "\
- eglibc-gconv-utf-16 \
- eglibc-charmap-utf-8 \
- eglibc-gconv-cp1255 \
- eglibc-charmap-cp1255 \
- eglibc-gconv-utf-32 \
- eglibc-gconv-utf-7 \
- eglibc-charmap-invariant \
- eglibc-localedata-translit-cjk-variants \
- tzdata \
- tzdata-americas \
- tzdata-asia \
- tzdata-europe \
- tzdata-posix \
- python-pygobject \
- python-dbus \
- "
-
do_configure_prepend() {
sed -i -e '1s,#!.*,#!${USRBINPATH}/env python,' ${S}/gio/gdbus-2.0/codegen/gdbus-codegen.in
}
@@ -56,6 +36,4 @@ do_install_append() {
if [ -f ${D}${bindir}/glib-mkenums ]; then
sed -i -e '1s,#!.*perl,#! ${USRBINPATH}/env perl,' ${D}${bindir}/glib-mkenums
fi
-
- ptest_do_install
}
--
1.7.10.4
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 15:55 ` [PATCH 4/8] Revert "Add ptest for glib." Bernhard Reutner-Fischer
@ 2013-02-05 16:44 ` Richard Purdie
2013-02-05 16:44 ` Burton, Ross
2013-02-05 16:46 ` Saul Wold
2 siblings, 0 replies; 27+ messages in thread
From: Richard Purdie @ 2013-02-05 16:44 UTC (permalink / raw)
To: Bernhard Reutner-Fischer, Björn Stenberg; +Cc: openembedded-core
On Tue, 2013-02-05 at 16:55 +0100, Bernhard Reutner-Fischer wrote:
> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>
> This breaks core-image-sato-sdk for TCLIBC="uclibc".
>
> Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
> ---
> .../glib-2.0/glib-2.0/Makefile-ptest.patch | 61 --------------------
> meta/recipes-core/glib-2.0/glib-2.0/run-ptest | 3 -
> meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb | 22 -------
> 3 files changed, 86 deletions(-)
> delete mode 100644 meta/recipes-core/glib-2.0/glib-2.0/Makefile-ptest.patch
> delete mode 100644 meta/recipes-core/glib-2.0/glib-2.0/run-ptest
How about we fix ptest for glib? The revert isn't really the direction
we want to take the project. At least discussing this with Björn (cc'd)
and/or filing a bug would be a good idea before we move to reverting it.
Cheers,
Richard
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 15:55 ` [PATCH 4/8] Revert "Add ptest for glib." Bernhard Reutner-Fischer
2013-02-05 16:44 ` Richard Purdie
@ 2013-02-05 16:44 ` Burton, Ross
2013-02-05 16:47 ` Bernhard Reutner-Fischer
2013-02-05 16:46 ` Saul Wold
2 siblings, 1 reply; 27+ messages in thread
From: Burton, Ross @ 2013-02-05 16:44 UTC (permalink / raw)
To: Bernhard Reutner-Fischer; +Cc: openembedded-core
On 5 February 2013 15:55, Bernhard Reutner-Fischer
<rep.dot.nop@gmail.com> wrote:
> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>
> This breaks core-image-sato-sdk for TCLIBC="uclibc".
Ouch. How about we fix it instead of reverting this?
Ross
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 16:44 ` Burton, Ross
@ 2013-02-05 16:47 ` Bernhard Reutner-Fischer
2013-02-05 16:59 ` Burton, Ross
0 siblings, 1 reply; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 16:47 UTC (permalink / raw)
To: Burton, Ross; +Cc: openembedded-core
On 5 February 2013 17:44, Burton, Ross <ross.burton@intel.com> wrote:
> On 5 February 2013 15:55, Bernhard Reutner-Fischer
> <rep.dot.nop@gmail.com> wrote:
>> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>>
>> This breaks core-image-sato-sdk for TCLIBC="uclibc".
>
> Ouch. How about we fix it instead of reverting this?
I do not have time to look into this, sorry. But yes, of course fixing
it for real would be better..
thanks,
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 15:55 ` [PATCH 4/8] Revert "Add ptest for glib." Bernhard Reutner-Fischer
2013-02-05 16:44 ` Richard Purdie
2013-02-05 16:44 ` Burton, Ross
@ 2013-02-05 16:46 ` Saul Wold
2013-02-05 17:13 ` Bernhard Reutner-Fischer
2 siblings, 1 reply; 27+ messages in thread
From: Saul Wold @ 2013-02-05 16:46 UTC (permalink / raw)
To: Bernhard Reutner-Fischer; +Cc: openembedded-core
On 02/05/2013 07:55 AM, Bernhard Reutner-Fischer wrote:
> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>
> This breaks core-image-sato-sdk for TCLIBC="uclibc".
>
What's the issue? Is it fixable instead of reverting the ptest patch? I
think this would be the wrong approach to this kind of issue as we want
to expand the usage of ptest.
Thanks
Sau!
> Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
> ---
> .../glib-2.0/glib-2.0/Makefile-ptest.patch | 61 --------------------
> meta/recipes-core/glib-2.0/glib-2.0/run-ptest | 3 -
> meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb | 22 -------
> 3 files changed, 86 deletions(-)
> delete mode 100644 meta/recipes-core/glib-2.0/glib-2.0/Makefile-ptest.patch
> delete mode 100644 meta/recipes-core/glib-2.0/glib-2.0/run-ptest
>
> diff --git a/meta/recipes-core/glib-2.0/glib-2.0/Makefile-ptest.patch b/meta/recipes-core/glib-2.0/glib-2.0/Makefile-ptest.patch
> deleted file mode 100644
> index cb36c09..0000000
> --- a/meta/recipes-core/glib-2.0/glib-2.0/Makefile-ptest.patch
> +++ /dev/null
> @@ -1,61 +0,0 @@
> -Add 'ptest' target to Makefile, to run tests without checking dependencies.
> -Add 'install-ptest' target, to copy needed files to ptest dir.
> -
> -Signed-off-by: Björn Stenberg <bjst@enea.com>
> -Upstream-status: Pending
> ----
> - Makefile.decl | 43 +++++++++++++++++++++++++++++++++++++++++++
> - 1 file changed, 43 insertions(+)
> -
> ---- a/Makefile.decl 2012-03-12 01:42:39.000000000 +0100
> -+++ b/Makefile.decl 2012-11-26 13:42:08.436865192 +0100
> -@@ -25,6 +25,49 @@
> - test-nonrecursive:
> - endif
> -
> -+ptest: ptest-nonrecursive
> -+ @ for subdir in $(SUBDIRS) .; do \
> -+ test -d "$$subdir" -a "$$subdir" != "." -a "$$subdir" != "po" && \
> -+ ( test -d $$subdir && cd $$subdir && $(MAKE) $(AM_MAKEFLAGS) $@ ) \
> -+ done
> -+
> -+ptest-nonrecursive:
> -+ @test -z "${TEST_PROGS}" || ${GTESTER} --keep-going --verbose ${TEST_PROGS}
> -+
> -+install-ptest: install-ptest-nonrecursive
> -+ @test -z "$(SUBDIRS)" || for subdir in $(SUBDIRS); do \
> -+ test "$$subdir" == "." || \
> -+ (cd $$subdir && $(MAKE) DESTDIR=$(DESTDIR)/$$subdir $@ ) \
> -+ done
> -+
> -+# install files used for ptest:
> -+# - copy EXTRA_DIST (for test data in glib/tests)
> -+# - run dist-hook (for glib/tests/bookmarks/ and glib/tests/markups)
> -+# - copy TEST_PROGS (the test programs)
> -+# - copy INSTALL_PROGS (for glib/gtester)
> -+# - avoid libtool wrappers by copying binaries from .libs
> -+# - strip Makefile: target from all Makefiles
> -+install-ptest-nonrecursive:
> -+ @mkdir -p $(DESTDIR)
> -+ @if [ -n "${TEST_PROGS}" ]; then \
> -+ if [ -n "${EXTRA_DIST}" ]; then \
> -+ cp ${EXTRA_DIST} $(DESTDIR); \
> -+ fi; \
> -+ grep -q dist-hook: Makefile && \
> -+ $(MAKE) distdir=$(DESTDIR) dist-hook; \
> -+ if [ -d .libs ] ; then \
> -+ cd .libs; \
> -+ fi; \
> -+ cp ${TEST_PROGS} $(DESTDIR); \
> -+ fi;
> -+ @if [ -n "${INSTALL_PROGS}" ]; then \
> -+ if [ -d .libs ] ; then \
> -+ cd .libs; \
> -+ fi; \
> -+ cp ${INSTALL_PROGS} $(DESTDIR); \
> -+ fi
> -+ @test -f Makefile && sed 's/^Makefile:/_Makefile:/' <Makefile >$(DESTDIR)/Makefile
> -+
> - # test-report: run tests in subdirs and generate report
> - # perf-report: run tests in subdirs with -m perf and generate report
> - # full-report: like test-report: with -m perf and -m slow
> diff --git a/meta/recipes-core/glib-2.0/glib-2.0/run-ptest b/meta/recipes-core/glib-2.0/glib-2.0/run-ptest
> deleted file mode 100644
> index 3deb586..0000000
> --- a/meta/recipes-core/glib-2.0/glib-2.0/run-ptest
> +++ /dev/null
> @@ -1,3 +0,0 @@
> -#!/bin/sh
> -
> -make -k ptest
> diff --git a/meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb b/meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb
> index e1d44e2..e4293c8 100644
> --- a/meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb
> +++ b/meta/recipes-core/glib-2.0/glib-2.0_2.34.3.bb
> @@ -13,8 +13,6 @@ SRC_URI = "${GNOME_MIRROR}/glib/${SHRT_VER}/glib-${PV}.tar.xz \
> file://configure-libtool.patch \
> file://glib-2.0_fix_for_x32.patch \
> file://obsolete_automake_macros.patch \
> - file://Makefile-ptest.patch \
> - file://run-ptest \
> "
> SRC_URI[md5sum] = "a4ca31e258273c3761e3de2edd607661"
> SRC_URI[sha256sum] = "855fcbf87cb93065b488358e351774d8a39177281023bae58c286f41612658a7"
> @@ -22,24 +20,6 @@ SRC_URI[sha256sum] = "855fcbf87cb93065b488358e351774d8a39177281023bae58c286f4161
> SRC_URI_append_class-native = " file://glib-gettextize-dir.patch"
> BBCLASSEXTEND = "native nativesdk"
>
> -RDEPENDS_${PN}-ptest += "\
> - eglibc-gconv-utf-16 \
> - eglibc-charmap-utf-8 \
> - eglibc-gconv-cp1255 \
> - eglibc-charmap-cp1255 \
> - eglibc-gconv-utf-32 \
> - eglibc-gconv-utf-7 \
> - eglibc-charmap-invariant \
> - eglibc-localedata-translit-cjk-variants \
> - tzdata \
> - tzdata-americas \
> - tzdata-asia \
> - tzdata-europe \
> - tzdata-posix \
> - python-pygobject \
> - python-dbus \
> - "
> -
> do_configure_prepend() {
> sed -i -e '1s,#!.*,#!${USRBINPATH}/env python,' ${S}/gio/gdbus-2.0/codegen/gdbus-codegen.in
> }
> @@ -56,6 +36,4 @@ do_install_append() {
> if [ -f ${D}${bindir}/glib-mkenums ]; then
> sed -i -e '1s,#!.*perl,#! ${USRBINPATH}/env perl,' ${D}${bindir}/glib-mkenums
> fi
> -
> - ptest_do_install
> }
>
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 16:46 ` Saul Wold
@ 2013-02-05 17:13 ` Bernhard Reutner-Fischer
2013-02-05 18:39 ` Saul Wold
0 siblings, 1 reply; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 17:13 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
On 5 February 2013 17:46, Saul Wold <sgw@linux.intel.com> wrote:
> On 02/05/2013 07:55 AM, Bernhard Reutner-Fischer wrote:
>>
>> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>>
>> This breaks core-image-sato-sdk for TCLIBC="uclibc".
>>
> What's the issue? Is it fixable instead of reverting the ptest patch? I
The issue is that it incorrectly and unconditionally pulls in eglibc stuff:
-RDEPENDS_${PN}-ptest += "\
- eglibc-gconv-utf-16 \
- eglibc-charmap-utf-8 \
- eglibc-gconv-cp1255 \
- eglibc-charmap-cp1255 \
- eglibc-gconv-utf-32 \
- eglibc-gconv-utf-7 \
- eglibc-charmap-invariant \
- eglibc-localedata-translit-cjk-variants \
- tzdata \
- tzdata-americas \
- tzdata-asia \
- tzdata-europe \
- tzdata-posix \
- python-pygobject \
- python-dbus \
- "
> think this would be the wrong approach to this kind of issue as we want to
> expand the usage of ptest.
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 17:13 ` Bernhard Reutner-Fischer
@ 2013-02-05 18:39 ` Saul Wold
2013-02-05 19:16 ` Bernhard Reutner-Fischer
2013-02-06 9:02 ` Björn Stenberg
0 siblings, 2 replies; 27+ messages in thread
From: Saul Wold @ 2013-02-05 18:39 UTC (permalink / raw)
To: Björn Stenberg; +Cc: openembedded-core
On 02/05/2013 09:13 AM, Bernhard Reutner-Fischer wrote:
> On 5 February 2013 17:46, Saul Wold <sgw@linux.intel.com> wrote:
>> On 02/05/2013 07:55 AM, Bernhard Reutner-Fischer wrote:
>>>
>>> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>>>
>>> This breaks core-image-sato-sdk for TCLIBC="uclibc".
>>>
>> What's the issue? Is it fixable instead of reverting the ptest patch? I
>
> The issue is that it incorrectly and unconditionally pulls in eglibc stuff:
>
> -RDEPENDS_${PN}-ptest += "\
> - eglibc-gconv-utf-16 \
> - eglibc-charmap-utf-8 \
> - eglibc-gconv-cp1255 \
> - eglibc-charmap-cp1255 \
> - eglibc-gconv-utf-32 \
> - eglibc-gconv-utf-7 \
> - eglibc-charmap-invariant \
> - eglibc-localedata-translit-cjk-variants \
> - tzdata \
> - tzdata-americas \
> - tzdata-asia \
> - tzdata-europe \
> - tzdata-posix \
> - python-pygobject \
> - python-dbus \
> - "
>
Bjorn,
Do you have some time to look at this and see if you can make use of
TCLIBC to use the correct library and locale package info based on which
libc is being used?
I guess you did not originally think about the uclibc case.
Thanks
Sau!
>> think this would be the wrong approach to this kind of issue as we want to
>> expand the usage of ptest.
>
>
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 18:39 ` Saul Wold
@ 2013-02-05 19:16 ` Bernhard Reutner-Fischer
2013-02-05 19:36 ` Bernhard Reutner-Fischer
2013-02-06 9:02 ` Björn Stenberg
1 sibling, 1 reply; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 19:16 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
On 5 February 2013 19:39, Saul Wold <sgw@linux.intel.com> wrote:
> On 02/05/2013 09:13 AM, Bernhard Reutner-Fischer wrote:
>>
>> On 5 February 2013 17:46, Saul Wold <sgw@linux.intel.com> wrote:
>>>
>>> On 02/05/2013 07:55 AM, Bernhard Reutner-Fischer wrote:
>>>>
>>>>
>>>> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>>>>
>>>> This breaks core-image-sato-sdk for TCLIBC="uclibc".
>>>>
>>> What's the issue? Is it fixable instead of reverting the ptest patch? I
>>
>>
>> The issue is that it incorrectly and unconditionally pulls in eglibc
>> stuff:
>>
>> -RDEPENDS_${PN}-ptest += "\
>> - eglibc-gconv-utf-16 \
>> - eglibc-charmap-utf-8 \
>> - eglibc-gconv-cp1255 \
>> - eglibc-charmap-cp1255 \
>> - eglibc-gconv-utf-32 \
>> - eglibc-gconv-utf-7 \
>> - eglibc-charmap-invariant \
>> - eglibc-localedata-translit-cjk-variants \
>> - tzdata \
>> - tzdata-americas \
>> - tzdata-asia \
>> - tzdata-europe \
>> - tzdata-posix \
>> - python-pygobject \
>> - python-dbus \
>> - "
>>
> Bjorn,
>
> Do you have some time to look at this and see if you can make use of TCLIBC
> to use the correct library and locale package info based on which libc is
> being used?
would most likely DTRT, yep.
I'm curious what you would need from uClibc to satisfy that ptest thing?
What does it do, what is it good for? :)
RDEPENDS_${PN}-ptest += "\
tzdata \
tzdata-americas \
tzdata-asia \
tzdata-europe \
tzdata-posix \
python-pygobject \
python-dbus \
"
RDEPENDS_${PN}-ptest_append_libc-eglibc = "\
eglibc-gconv-utf-16 \
eglibc-charmap-utf-8 \
eglibc-gconv-cp1255 \
eglibc-charmap-cp1255 \
eglibc-gconv-utf-32 \
eglibc-gconv-utf-7 \
eglibc-charmap-invariant \
eglibc-localedata-translit-cjk-variants \
"
RDEPENDS_${PN}-ptest_append_libc-eglibc = "did not look"
TIA && cheers,
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 19:16 ` Bernhard Reutner-Fischer
@ 2013-02-05 19:36 ` Bernhard Reutner-Fischer
2013-02-05 19:59 ` Saul Wold
0 siblings, 1 reply; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 19:36 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
On 5 February 2013 20:16, Bernhard Reutner-Fischer
<rep.dot.nop@gmail.com> wrote:
> On 5 February 2013 19:39, Saul Wold <sgw@linux.intel.com> wrote:
>> On 02/05/2013 09:13 AM, Bernhard Reutner-Fischer wrote:
>>>
>>> On 5 February 2013 17:46, Saul Wold <sgw@linux.intel.com> wrote:
>>>>
>>>> On 02/05/2013 07:55 AM, Bernhard Reutner-Fischer wrote:
>>>>>
>>>>>
>>>>> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>>>>>
>>>>> This breaks core-image-sato-sdk for TCLIBC="uclibc".
>>>>>
>>>> What's the issue? Is it fixable instead of reverting the ptest patch? I
>>>
>>>
>>> The issue is that it incorrectly and unconditionally pulls in eglibc
>>> stuff:
>>>
>>> -RDEPENDS_${PN}-ptest += "\
>>> - eglibc-gconv-utf-16 \
>>> - eglibc-charmap-utf-8 \
>>> - eglibc-gconv-cp1255 \
>>> - eglibc-charmap-cp1255 \
>>> - eglibc-gconv-utf-32 \
>>> - eglibc-gconv-utf-7 \
>>> - eglibc-charmap-invariant \
>>> - eglibc-localedata-translit-cjk-variants \
>>> - tzdata \
>>> - tzdata-americas \
>>> - tzdata-asia \
>>> - tzdata-europe \
>>> - tzdata-posix \
>>> - python-pygobject \
>>> - python-dbus \
>>> - "
>>>
>> Bjorn,
>>
>> Do you have some time to look at this and see if you can make use of TCLIBC
>> to use the correct library and locale package info based on which libc is
>> being used?
>
> would most likely DTRT, yep.
Well, given that i see e.g. glibc-gconv-utf-16 nowhere in
openembedded-core i suspect they are misplaced and should live in the
layer that provides them.
> I'm curious what you would need from uClibc to satisfy that ptest thing?
> What does it do, what is it good for? :)
Is e.g. glibc-gconv-utf-16 locale support or something like that?
thanks,
>
> RDEPENDS_${PN}-ptest += "\
> tzdata \
> tzdata-americas \
> tzdata-asia \
> tzdata-europe \
> tzdata-posix \
> python-pygobject \
> python-dbus \
> "
> RDEPENDS_${PN}-ptest_append_libc-eglibc = "\
> eglibc-gconv-utf-16 \
> eglibc-charmap-utf-8 \
> eglibc-gconv-cp1255 \
> eglibc-charmap-cp1255 \
> eglibc-gconv-utf-32 \
> eglibc-gconv-utf-7 \
> eglibc-charmap-invariant \
> eglibc-localedata-translit-cjk-variants \
> "
> RDEPENDS_${PN}-ptest_append_libc-eglibc = "did not look"
>
> TIA && cheers,
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 19:36 ` Bernhard Reutner-Fischer
@ 2013-02-05 19:59 ` Saul Wold
2013-02-06 0:01 ` Khem Raj
0 siblings, 1 reply; 27+ messages in thread
From: Saul Wold @ 2013-02-05 19:59 UTC (permalink / raw)
To: Bernhard Reutner-Fischer; +Cc: openembedded-core
On 02/05/2013 11:36 AM, Bernhard Reutner-Fischer wrote:
> On 5 February 2013 20:16, Bernhard Reutner-Fischer
> <rep.dot.nop@gmail.com> wrote:
>> On 5 February 2013 19:39, Saul Wold <sgw@linux.intel.com> wrote:
>>> On 02/05/2013 09:13 AM, Bernhard Reutner-Fischer wrote:
>>>>
>>>> On 5 February 2013 17:46, Saul Wold <sgw@linux.intel.com> wrote:
>>>>>
>>>>> On 02/05/2013 07:55 AM, Bernhard Reutner-Fischer wrote:
>>>>>>
>>>>>>
>>>>>> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>>>>>>
>>>>>> This breaks core-image-sato-sdk for TCLIBC="uclibc".
>>>>>>
>>>>> What's the issue? Is it fixable instead of reverting the ptest patch? I
>>>>
>>>>
>>>> The issue is that it incorrectly and unconditionally pulls in eglibc
>>>> stuff:
>>>>
>>>> -RDEPENDS_${PN}-ptest += "\
>>>> - eglibc-gconv-utf-16 \
>>>> - eglibc-charmap-utf-8 \
>>>> - eglibc-gconv-cp1255 \
>>>> - eglibc-charmap-cp1255 \
>>>> - eglibc-gconv-utf-32 \
>>>> - eglibc-gconv-utf-7 \
>>>> - eglibc-charmap-invariant \
>>>> - eglibc-localedata-translit-cjk-variants \
>>>> - tzdata \
>>>> - tzdata-americas \
>>>> - tzdata-asia \
>>>> - tzdata-europe \
>>>> - tzdata-posix \
>>>> - python-pygobject \
>>>> - python-dbus \
>>>> - "
>>>>
>>> Bjorn,
>>>
>>> Do you have some time to look at this and see if you can make use of TCLIBC
>>> to use the correct library and locale package info based on which libc is
>>> being used?
>>
>> would most likely DTRT, yep.
>
> Well, given that i see e.g. glibc-gconv-utf-16 nowhere in
> openembedded-core i suspect they are misplaced and should live in the
> layer that provides them.
>
It's a DYNAMIC package built in eglibc-locale.inc.
>> I'm curious what you would need from uClibc to satisfy that ptest thing?
>> What does it do, what is it good for? :)
>
Bjorn question at this point!
> Is e.g. glibc-gconv-utf-16 locale support or something like that?
>
probably, could the uclibc-gconv provide equivalent?
Sau!
> thanks,
>>
>> RDEPENDS_${PN}-ptest += "\
>> tzdata \
>> tzdata-americas \
>> tzdata-asia \
>> tzdata-europe \
>> tzdata-posix \
>> python-pygobject \
>> python-dbus \
>> "
>> RDEPENDS_${PN}-ptest_append_libc-eglibc = "\
>> eglibc-gconv-utf-16 \
>> eglibc-charmap-utf-8 \
>> eglibc-gconv-cp1255 \
>> eglibc-charmap-cp1255 \
>> eglibc-gconv-utf-32 \
>> eglibc-gconv-utf-7 \
>> eglibc-charmap-invariant \
>> eglibc-localedata-translit-cjk-variants \
>> "
>> RDEPENDS_${PN}-ptest_append_libc-eglibc = "did not look"
>>
>> TIA && cheers,
>
>
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 19:59 ` Saul Wold
@ 2013-02-06 0:01 ` Khem Raj
0 siblings, 0 replies; 27+ messages in thread
From: Khem Raj @ 2013-02-06 0:01 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
On Tue, Feb 5, 2013 at 11:59 AM, Saul Wold <sgw@linux.intel.com> wrote:
> On 02/05/2013 11:36 AM, Bernhard Reutner-Fischer wrote:
>>
>> On 5 February 2013 20:16, Bernhard Reutner-Fischer
>> <rep.dot.nop@gmail.com> wrote:
>>>
>>> On 5 February 2013 19:39, Saul Wold <sgw@linux.intel.com> wrote:
>>>>
>>>> On 02/05/2013 09:13 AM, Bernhard Reutner-Fischer wrote:
>>>>>
>>>>>
>>>>> On 5 February 2013 17:46, Saul Wold <sgw@linux.intel.com> wrote:
>>>>>>
>>>>>>
>>>>>> On 02/05/2013 07:55 AM, Bernhard Reutner-Fischer wrote:
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> This reverts commit dec6f912d68f52748f645b6af411f189cf464f85.
>>>>>>>
>>>>>>> This breaks core-image-sato-sdk for TCLIBC="uclibc".
>>>>>>>
>>>>>> What's the issue? Is it fixable instead of reverting the ptest patch?
>>>>>> I
>>>>>
>>>>>
>>>>>
>>>>> The issue is that it incorrectly and unconditionally pulls in eglibc
>>>>> stuff:
>>>>>
>>>>> -RDEPENDS_${PN}-ptest += "\
>>>>> - eglibc-gconv-utf-16 \
>>>>> - eglibc-charmap-utf-8 \
>>>>> - eglibc-gconv-cp1255 \
>>>>> - eglibc-charmap-cp1255 \
>>>>> - eglibc-gconv-utf-32 \
>>>>> - eglibc-gconv-utf-7 \
>>>>> - eglibc-charmap-invariant \
>>>>> - eglibc-localedata-translit-cjk-variants \
>>>>> - tzdata \
>>>>> - tzdata-americas \
>>>>> - tzdata-asia \
>>>>> - tzdata-europe \
>>>>> - tzdata-posix \
>>>>> - python-pygobject \
>>>>> - python-dbus \
>>>>> - "
>>>>>
>>>> Bjorn,
>>>>
>>>> Do you have some time to look at this and see if you can make use of
>>>> TCLIBC
>>>> to use the correct library and locale package info based on which libc
>>>> is
>>>> being used?
>>>
>>>
>>> would most likely DTRT, yep.
>>
>>
>> Well, given that i see e.g. glibc-gconv-utf-16 nowhere in
>> openembedded-core i suspect they are misplaced and should live in the
>> layer that provides them.
>>
> It's a DYNAMIC package built in eglibc-locale.inc.
>
>
>>> I'm curious what you would need from uClibc to satisfy that ptest thing?
>>> What does it do, what is it good for? :)
>>
>>
> Bjorn question at this point!
>
>
>> Is e.g. glibc-gconv-utf-16 locale support or something like that?
>>
> probably, could the uclibc-gconv provide equivalent?
>
split the ptest recipe out into its own and make it eglibc only recipe
locales and gconv stuff wont fly with uclibc
> Sau!
>
>
>
>> thanks,
>>>
>>>
>>> RDEPENDS_${PN}-ptest += "\
>>> tzdata \
>>> tzdata-americas \
>>> tzdata-asia \
>>> tzdata-europe \
>>> tzdata-posix \
>>> python-pygobject \
>>> python-dbus \
>>> "
>>> RDEPENDS_${PN}-ptest_append_libc-eglibc = "\
>>> eglibc-gconv-utf-16 \
>>> eglibc-charmap-utf-8 \
>>> eglibc-gconv-cp1255 \
>>> eglibc-charmap-cp1255 \
>>> eglibc-gconv-utf-32 \
>>> eglibc-gconv-utf-7 \
>>> eglibc-charmap-invariant \
>>> eglibc-localedata-translit-cjk-variants \
>>> "
>>> RDEPENDS_${PN}-ptest_append_libc-eglibc = "did not look"
>>>
>>> TIA && cheers,
>>
>>
>>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH 4/8] Revert "Add ptest for glib."
2013-02-05 18:39 ` Saul Wold
2013-02-05 19:16 ` Bernhard Reutner-Fischer
@ 2013-02-06 9:02 ` Björn Stenberg
1 sibling, 0 replies; 27+ messages in thread
From: Björn Stenberg @ 2013-02-06 9:02 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
Saul Wold wrote:
> >-RDEPENDS_${PN}-ptest += "\
> >- eglibc-gconv-utf-16 \
> >- eglibc-charmap-utf-8 \
> >- eglibc-gconv-cp1255 \
> >- eglibc-charmap-cp1255 \
> >- eglibc-gconv-utf-32 \
> >- eglibc-gconv-utf-7 \
> >- eglibc-charmap-invariant \
> >- eglibc-localedata-translit-cjk-variants \
> >- tzdata \
> >- tzdata-americas \
> >- tzdata-asia \
> >- tzdata-europe \
> >- tzdata-posix \
> >- python-pygobject \
> >- python-dbus \
> >- "
> Do you have some time to look at this and see if you can make use of
> TCLIBC to use the correct library and locale package info based on
> which libc is being used?
>
> I guess you did not originally think about the uclibc case.
Correct, I did not.
The list of packages added by ptest are things that are used by the glib test suite. Without this data, some tests will fail.
I'll look into making it more uclibc friendly.
--
Björn
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH 5/8] busybox: add ifup's ifstate dir to package
2013-02-05 15:55 [PATCH 0/8] misc tweaks Bernhard Reutner-Fischer
` (3 preceding siblings ...)
2013-02-05 15:55 ` [PATCH 4/8] Revert "Add ptest for glib." Bernhard Reutner-Fischer
@ 2013-02-05 15:55 ` Bernhard Reutner-Fischer
2013-02-12 5:46 ` Saul Wold
2013-02-05 15:55 ` [PATCH 6/8] busybox: update-modules script needs id(1) Bernhard Reutner-Fischer
` (2 subsequent siblings)
7 siblings, 1 reply; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 15:55 UTC (permalink / raw)
To: openembedded-core
ifupdown stores its ifstate into CONFIG_IFIPDOWN_IFSTATE_PATH.
Fixes:
ifup: can't open '/var/run/ifstate': No such file or directory
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
---
meta/recipes-core/busybox/busybox.inc | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/meta/recipes-core/busybox/busybox.inc b/meta/recipes-core/busybox/busybox.inc
index 972e7d0..972df6d 100644
--- a/meta/recipes-core/busybox/busybox.inc
+++ b/meta/recipes-core/busybox/busybox.inc
@@ -199,6 +199,10 @@ do_install () {
install -m 644 ${WORKDIR}/mdev.conf ${D}${sysconfdir}/mdev.conf
fi
fi
+ IFUPDOWN_IFSTATE_PATH=`awk '/^CONFIG_IFUPDOWN_IFSTATE_PATH/{split($0,x,/=/);p=x[2];gsub("\"","",p);print p;}' ${WORKDIR}/defconfig`
+ if test -n "$IFUPDOWN_IFSTATE_PATH"; then
+ install -m 0755 -d ${D}/"$IFUPDOWN_IFSTATE_PATH"
+ fi
install -m 0644 ${S}/busybox.links ${D}${sysconfdir}
}
--
1.7.10.4
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH 5/8] busybox: add ifup's ifstate dir to package
2013-02-05 15:55 ` [PATCH 5/8] busybox: add ifup's ifstate dir to package Bernhard Reutner-Fischer
@ 2013-02-12 5:46 ` Saul Wold
2013-02-12 9:14 ` Bernhard Reutner-Fischer
0 siblings, 1 reply; 27+ messages in thread
From: Saul Wold @ 2013-02-12 5:46 UTC (permalink / raw)
To: Bernhard Reutner-Fischer; +Cc: openembedded-core
On 02/05/2013 07:55 AM, Bernhard Reutner-Fischer wrote:
> ifupdown stores its ifstate into CONFIG_IFIPDOWN_IFSTATE_PATH.
> Fixes:
> ifup: can't open '/var/run/ifstate': No such file or directory
>
> Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
> ---
> meta/recipes-core/busybox/busybox.inc | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/meta/recipes-core/busybox/busybox.inc b/meta/recipes-core/busybox/busybox.inc
> index 972e7d0..972df6d 100644
> --- a/meta/recipes-core/busybox/busybox.inc
> +++ b/meta/recipes-core/busybox/busybox.inc
> @@ -199,6 +199,10 @@ do_install () {
> install -m 644 ${WORKDIR}/mdev.conf ${D}${sysconfdir}/mdev.conf
> fi
> fi
> + IFUPDOWN_IFSTATE_PATH=`awk '/^CONFIG_IFUPDOWN_IFSTATE_PATH/{split($0,x,/=/);p=x[2];gsub("\"","",p);print p;}' ${WORKDIR}/defconfig`
> + if test -n "$IFUPDOWN_IFSTATE_PATH"; then
Should this also be ${D} in the test, other wise you might be testing
the host!
Sau!
> + install -m 0755 -d ${D}/"$IFUPDOWN_IFSTATE_PATH"
> + fi
> install -m 0644 ${S}/busybox.links ${D}${sysconfdir}
> }
>
>
^ permalink raw reply [flat|nested] 27+ messages in thread* Re: [PATCH 5/8] busybox: add ifup's ifstate dir to package
2013-02-12 5:46 ` Saul Wold
@ 2013-02-12 9:14 ` Bernhard Reutner-Fischer
0 siblings, 0 replies; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-12 9:14 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
On Mon, Feb 11, 2013 at 09:46:31PM -0800, Saul Wold wrote:
>On 02/05/2013 07:55 AM, Bernhard Reutner-Fischer wrote:
>>ifupdown stores its ifstate into CONFIG_IFIPDOWN_IFSTATE_PATH.
>>Fixes:
>>ifup: can't open '/var/run/ifstate': No such file or directory
>>
>>Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
>>---
>> meta/recipes-core/busybox/busybox.inc | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>>diff --git a/meta/recipes-core/busybox/busybox.inc b/meta/recipes-core/busybox/busybox.inc
>>index 972e7d0..972df6d 100644
>>--- a/meta/recipes-core/busybox/busybox.inc
>>+++ b/meta/recipes-core/busybox/busybox.inc
>>@@ -199,6 +199,10 @@ do_install () {
>> install -m 644 ${WORKDIR}/mdev.conf ${D}${sysconfdir}/mdev.conf
>> fi
>> fi
>>+ IFUPDOWN_IFSTATE_PATH=`awk '/^CONFIG_IFUPDOWN_IFSTATE_PATH/{split($0,x,/=/);p=x[2];gsub("\"","",p);print p;}' ${WORKDIR}/defconfig`
>>+ if test -n "$IFUPDOWN_IFSTATE_PATH"; then
>Should this also be ${D} in the test, other wise you might be testing
>the host!
WORKDIR is correct, see meta/conf/bitbake.conf setting WORKDIR.
thanks,
>
>
>Sau!
>
>>+ install -m 0755 -d ${D}/"$IFUPDOWN_IFSTATE_PATH"
>>+ fi
>> install -m 0644 ${S}/busybox.links ${D}${sysconfdir}
>> }
>>
>>
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH 6/8] busybox: update-modules script needs id(1)
2013-02-05 15:55 [PATCH 0/8] misc tweaks Bernhard Reutner-Fischer
` (4 preceding siblings ...)
2013-02-05 15:55 ` [PATCH 5/8] busybox: add ifup's ifstate dir to package Bernhard Reutner-Fischer
@ 2013-02-05 15:55 ` Bernhard Reutner-Fischer
2013-02-05 16:38 ` Richard Purdie
2013-02-05 15:55 ` [PATCH 7/8] base.bbclass: remove redundant setting of MAKE Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 8/8] create-pull-request: Error message on missing -u Bernhard Reutner-Fischer
7 siblings, 1 reply; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 15:55 UTC (permalink / raw)
To: openembedded-core
The update-modules script calls id(1), so turn on id in this case.
Bump PR.
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
---
meta/recipes-core/busybox/busybox.inc | 1 +
meta/recipes-core/busybox/busybox_1.20.2.bb | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/meta/recipes-core/busybox/busybox.inc b/meta/recipes-core/busybox/busybox.inc
index 972df6d..b0a3cbe 100644
--- a/meta/recipes-core/busybox/busybox.inc
+++ b/meta/recipes-core/busybox/busybox.inc
@@ -59,6 +59,7 @@ def features_to_busybox_settings(d):
busybox_cfg('ipv6', distro_features, 'CONFIG_FEATURE_IFUPDOWN_IPV6', cnf, rem)
busybox_cfg('wifi', distro_features, 'CONFIG_RFKILL', cnf, rem)
busybox_cfg('bluetooth', distro_features, 'CONFIG_RFKILL', cnf, rem)
+ busybox_cfg('update-modules', distro_features, 'CONFIG_ID', cnf, rem)
return "\n".join(cnf), "\n".join(rem)
# X, Y = ${@features_to_uclibc_settings(d)}
diff --git a/meta/recipes-core/busybox/busybox_1.20.2.bb b/meta/recipes-core/busybox/busybox_1.20.2.bb
index 783261e..3ea100d 100644
--- a/meta/recipes-core/busybox/busybox_1.20.2.bb
+++ b/meta/recipes-core/busybox/busybox_1.20.2.bb
@@ -1,5 +1,5 @@
require busybox.inc
-PR = "r5"
+PR = "r6"
SRC_URI = "http://www.busybox.net/downloads/busybox-${PV}.tar.bz2;name=tarball \
file://B921600.patch \
--
1.7.10.4
^ permalink raw reply related [flat|nested] 27+ messages in thread* Re: [PATCH 6/8] busybox: update-modules script needs id(1)
2013-02-05 15:55 ` [PATCH 6/8] busybox: update-modules script needs id(1) Bernhard Reutner-Fischer
@ 2013-02-05 16:38 ` Richard Purdie
2013-02-05 16:46 ` Bernhard Reutner-Fischer
0 siblings, 1 reply; 27+ messages in thread
From: Richard Purdie @ 2013-02-05 16:38 UTC (permalink / raw)
To: Bernhard Reutner-Fischer; +Cc: openembedded-core
On Tue, 2013-02-05 at 16:55 +0100, Bernhard Reutner-Fischer wrote:
> The update-modules script calls id(1), so turn on id in this case.
> Bump PR.
>
> Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
> ---
> meta/recipes-core/busybox/busybox.inc | 1 +
> meta/recipes-core/busybox/busybox_1.20.2.bb | 2 +-
> 2 files changed, 2 insertions(+), 1 deletion(-)
Wasn't update-modules just removed from OECore?
Cheers,
Richard
^ permalink raw reply [flat|nested] 27+ messages in thread
* Re: [PATCH 6/8] busybox: update-modules script needs id(1)
2013-02-05 16:38 ` Richard Purdie
@ 2013-02-05 16:46 ` Bernhard Reutner-Fischer
0 siblings, 0 replies; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 16:46 UTC (permalink / raw)
To: Richard Purdie; +Cc: openembedded-core
On 5 February 2013 17:38, Richard Purdie
<richard.purdie@linuxfoundation.org> wrote:
> On Tue, 2013-02-05 at 16:55 +0100, Bernhard Reutner-Fischer wrote:
>> The update-modules script calls id(1), so turn on id in this case.
>> Bump PR.
>>
>> Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
>> ---
>> meta/recipes-core/busybox/busybox.inc | 1 +
>> meta/recipes-core/busybox/busybox_1.20.2.bb | 2 +-
>> 2 files changed, 2 insertions(+), 1 deletion(-)
>
> Wasn't update-modules just removed from OECore?
could be, i am still at 8dd608dac1e5658197078d002dc1b3dc8ac83184 here
because i feared that unrelated stuff will fail again on me (see the
Revert in this series for one example).
Please disregard this hunk if it is moot by now..
cheers,
^ permalink raw reply [flat|nested] 27+ messages in thread
* [PATCH 7/8] base.bbclass: remove redundant setting of MAKE
2013-02-05 15:55 [PATCH 0/8] misc tweaks Bernhard Reutner-Fischer
` (5 preceding siblings ...)
2013-02-05 15:55 ` [PATCH 6/8] busybox: update-modules script needs id(1) Bernhard Reutner-Fischer
@ 2013-02-05 15:55 ` Bernhard Reutner-Fischer
2013-02-05 15:55 ` [PATCH 8/8] create-pull-request: Error message on missing -u Bernhard Reutner-Fischer
7 siblings, 0 replies; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 15:55 UTC (permalink / raw)
To: openembedded-core
oe_runmake() tried to set MAKE a second time, remove it since this is
(nowadays?) redundant WRT the normal export MAKE= in the generated
scripts.
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
---
meta/classes/base.bbclass | 1 -
1 file changed, 1 deletion(-)
diff --git a/meta/classes/base.bbclass b/meta/classes/base.bbclass
index 8147a81..3e6fa0e 100644
--- a/meta/classes/base.bbclass
+++ b/meta/classes/base.bbclass
@@ -54,7 +54,6 @@ die() {
}
oe_runmake() {
- if [ x"$MAKE" = x ]; then MAKE=make; fi
bbnote ${MAKE} ${EXTRA_OEMAKE} "$@"
${MAKE} ${EXTRA_OEMAKE} "$@" || die "oe_runmake failed"
}
--
1.7.10.4
^ permalink raw reply related [flat|nested] 27+ messages in thread* [PATCH 8/8] create-pull-request: Error message on missing -u
2013-02-05 15:55 [PATCH 0/8] misc tweaks Bernhard Reutner-Fischer
` (6 preceding siblings ...)
2013-02-05 15:55 ` [PATCH 7/8] base.bbclass: remove redundant setting of MAKE Bernhard Reutner-Fischer
@ 2013-02-05 15:55 ` Bernhard Reutner-Fischer
7 siblings, 0 replies; 27+ messages in thread
From: Bernhard Reutner-Fischer @ 2013-02-05 15:55 UTC (permalink / raw)
To: openembedded-core
The script was erroring out without a hint on what failed.
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
---
scripts/create-pull-request | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/scripts/create-pull-request b/scripts/create-pull-request
index 9692bf1..503248b 100755
--- a/scripts/create-pull-request
+++ b/scripts/create-pull-request
@@ -1,6 +1,6 @@
#!/bin/bash
#
-# Copyright (c) 2010-2011, Intel Corporation.
+# Copyright (c) 2010-2013, Intel Corporation.
# All Rights Reserved
#
# This program is free software; you can redistribute it and/or modify
@@ -124,8 +124,9 @@ if [ -z "$BRANCH" ]; then
BRANCH=$(git branch | grep -e "^\* " | cut -d' ' -f2)
echo "NOTE: Assuming remote branch '$BRANCH', use -b to override."
fi
-
+
if [ -z "$REMOTE_URL" ]; then
+ echo "ERROR: Missing parameter -u, no git remote!"
usage
exit 1
fi
--
1.7.10.4
^ permalink raw reply related [flat|nested] 27+ messages in thread