Re: [OE-core] [PATCH] resolvconf: add missing dependencies to RDEPENDS

2021-03-09 Thread Frans Meulenbroeks via lists.openembedded.org
 Freaking yahoo email seems to have no option any more to quote decently.Guess 
it is time for a new email provider :-)Anyway my response below prefixed with 
FM:

Op dinsdag 9 maart 2021 00:26:05 CET schreef Khem Raj : 
 
 
 

On 3/8/21 2:25 PM, Andre McCurdy wrote:
> On Mon, Mar 8, 2021 at 2:15 PM Khem Raj  wrote:
>>
>> On 3/8/21 2:11 PM, Andre McCurdy wrote:
>>> On Mon, Mar 8, 2021 at 11:30 AM Khem Raj  wrote:
>>>>
>>>> From: Frans Meulenbroeks 
>>>>
>>>> resolvconf uses flock and readlink.
>>>> So explicitly added dependencies on util-linux (for flock) and coreutils
>>>> (for readlink).
>>>> Note that the options used with flock and readlink are not part of the
>>>> busybox implementations so full-fledged implementations are needed.
>>>>
>>>> Signed-off-by: Khem Raj 
>>>> ---
>>>>    meta/recipes-connectivity/resolvconf/resolvconf_1.87.bb | 2 +-
>>>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/meta/recipes-connectivity/resolvconf/resolvconf_1.87.bb 
>>>> b/meta/recipes-connectivity/resolvconf/resolvconf_1.87.bb
>>>> index 6df96b699c..6da8cd6023 100644
>>>> --- a/meta/recipes-connectivity/resolvconf/resolvconf_1.87.bb
>>>> +++ b/meta/recipes-connectivity/resolvconf/resolvconf_1.87.bb
>>>> @@ -9,7 +9,7 @@ LICENSE = "GPLv2+"
>>>>    LIC_FILES_CHKSUM = "file://COPYING;md5=c93c0550bd3173f4504b2cbd8991e50b"
>>>>    AUTHOR = "Thomas Hood"
>>>>    HOMEPAGE = "http://packages.debian.org/resolvconf;
>>>> -RDEPENDS_${PN} = "bash"
>>>> +RDEPENDS_${PN} = "bash coreutils util-linux"
>>>
>>> util-linux-flock has its own package.
>>
>> yeah and busybox also provides flock too. Lets drop it
> 
> Busybox provides flock, but it doesn't support the -w option (wait
> with timeout). Either the util-linux flock is required or resolvconf
> needs to be patched to drop the -w option.
> 

right, its to explore possibilities to avoid the deps.
FM: While I am all in favor of avoiding these dependencies, I'm not versed 
enough in the resolvconf code to judge the impact of dropping -w, so my 
preference would be to depend on util-linux-flock.


>>> Patching resolvconf to use readlink -f (instead of readlink -m) should
>>> be sufficient to allow it to work with busybox readlink.
FM: this will most likely work. I can make a patch for that, no problem.
>>>
>>>>    SRC_URI = 
>>>>"git://salsa.debian.org/debian/resolvconf.git;protocol=https;branch=unstable
>>>> \
>>>>              file://fix-path-for-busybox.patch \
>>>> --
>>>> 2.30.1

On a general note: in the current form the package is broken, so some fix is 
desirable. 

Kind regards, Frans.
  

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#149165): 
https://lists.openembedded.org/g/openembedded-core/message/149165
Mute This Topic: https://lists.openembedded.org/mt/81183177/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [OE-core] [oe] Invitation: OpenEmbedded public TSC/workgroup meeting

2013-09-10 Thread Frans Meulenbroeks
2013/9/10 Paul Eggleton paul.eggle...@linux.intel.com


  I assume that an IRC log will be posted after the meeting, is that
 correct?

 That's correct.

#oe is permanently logged (unless the bot is dead)
logs are at http://ibot.rikers.org/%23oe/

Enjoy!
Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core


[OE-core] [PATCH] mtd-utils: split into multiple packages

2013-01-21 Thread Frans Meulenbroeks
This allows one not to install rarely used or fs specific files
As discussed in 
http://thread.gmane.org/gmane.comp.handhelds.openembedded.core/31744/focus=31804

Signed-off-by: Frans Meulenbroeks fransmeulenbro...@gmail.com
---
 meta/recipes-devtools/mtd/mtd-utils_1.5.0.bb |8 +++-
 1 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-devtools/mtd/mtd-utils_1.5.0.bb 
b/meta/recipes-devtools/mtd/mtd-utils_1.5.0.bb
index 12ad9d4..5797204 100644
--- a/meta/recipes-devtools/mtd/mtd-utils_1.5.0.bb
+++ b/meta/recipes-devtools/mtd/mtd-utils_1.5.0.bb
@@ -13,7 +13,7 @@ SRC_URI = git://git.infradead.org/mtd-utils.git;protocol=git 
\
 
 S = ${WORKDIR}/git/
 
-PR = r2
+PR = r3
 
 EXTRA_OEMAKE = 'CC=${CC}' 'RANLIB=${RANLIB}' 'AR=${AR}' 'CFLAGS=${CFLAGS} 
-I${S}/include -DWITHOUT_XATTR' 'BUILDDIR=${S}'
 
@@ -21,6 +21,12 @@ do_install () {
oe_runmake install DESTDIR=${D} SBINDIR=${sbindir} MANDIR=${mandir} 
INCLUDEDIR=${includedir}
 }
 
+PACKAGES =+ mtd-utils-jffs2 mtd-utils-ubifs mtd-utils-misc
+
+FILES_mtd-utils-jffs2 = ${sbindir}/mkfs.jffs2 ${sbindir}/jffs2dump 
${sbindir}/jffs2reader ${sbindir}/sumtool
+FILES_mtd-utils-ubifs = ${sbindir}/mkfs.ubifs ${sbindir}/ubi*
+FILES_mtd-utils-misc = ${sbindir}/nftl* ${sbindir}/ftl* ${sbindir}/rfd* 
${sbindir}/doc* ${sbindir}/serve_image ${sbindir}/recv_image
+
 PARALLEL_MAKE = 
 
 BBCLASSEXTEND = native
-- 
1.7.0.4


___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] split mtd-utils

2013-01-21 Thread Frans Meulenbroeks
patch submitted.
Enjoy.
Frans


2013/1/11 Frans Meulenbroeks fransmeulenbro...@gmail.com




 2013/1/11 Eric Bénard e...@eukrea.com

 Hi Frans,

 [...]

 IIRC sumtol is used for jffs2 images (to create the erase block
 summary) so that may better fit into mtd-utils-jffs2

 Also, nftl* rfd* doc* ftl* serve_image may go into a separate packate
 (mtd-utils-misc ?) as I believe those tools are not very usefull in
 most configurations using mtd with nowadays's flashs.

 Eric

 Hi Eric,

 Thanks for the feedback. I'm not into the details of all tools in the
 package (I typically only use the flash* mtd* and jffs2 things)
 Seems a good plan to me to put these in a misc package
 (actually I also considered moving all other stuff that is in my proposal
 in mtd-utils into a differently named package, e.g. mtd-utils-core) and
 have mtd-utils as an umbrella package dragging in what it does now (so it
 should RDEPEND on the meta-utils-* packages but have no other stuff in it).

 Best regards, Frans.

 PS: partially related: I have filed yocto bug 3702 asking for more
 fine-grained control on what is included in an image.
 https://bugzilla.yoctoproject.org/show_bug.cgi?id=3702

___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] split mtd-utils

2013-01-11 Thread Frans Meulenbroeks
Along the lines of the oe classic patch I propose to split as follows:


mtd-utils-jffs2 package containing
jffs2dump
jffs2reader
mkfs.jffs2

mtd-utils-ubifs package containing:
mkfs.ubifs
ubiattach
ubicrc32
ubidetach
ubiformat
ubimkvol
ubinfo
ubinize
ubirename
ubirmvol
ubirsvol
ubiupdatevol

mtd-utils package containing:
docfdisk
doc_loadbios
flashcp
flash_erase
flash_eraseall
flash_lock
flash_otp_dump
flash_otp_info
flash_unlock
ftl_check
ftl_format
mtd_debug
mtdinfo
nanddump
nandtest
nandwrite
nftldump
nftl_format
recv_image
rfddump
rfdformat
serve_image
sumtool

Or do we need to make additional packages (like e.g. for (n)tfl )

Best regards, Frans




2013/1/10 Frans Meulenbroeks fransmeulenbro...@gmail.com

 Cool, I'll have a peek at the oe classic recipe and give it a stab along
 the same spirit.

 Best regards, Frans


 2013/1/10 Andrea Adami andrea.ad...@gmail.com

 On Wed, Jan 9, 2013 at 6:17 PM, Richard Purdie
 richard.pur...@linuxfoundation.org wrote:
  On Wed, 2013-01-09 at 17:12 +0100, Frans Meulenbroeks wrote:
  As I am involved in embedded systems where flash is somewhat sparse
  I'm always eager to save a few bytes where possible.
 
  Today I noticed that mtd-utils (1.5.0 from danny) generates for my
  architecture (powerpc) roughly 780k of binaries in usr/sbin. 423k of
  it is due to ubifs related files.
 
  Would it be desired to put this in a separate package?
 
  e.g. mtd-utils-ubi and mtd-utils-nonubi with mtd-utils itself being
  empty but rdepend on those two?
  that way mtd-utils will still give all packages but those only wanting
  the non ubi stuff can limit themselves to that.
 
  If desired I can give this a stab.
 
  Sounds like a sensible split to me...
 
  Cheers,
 
  Richard
 
 
 
  ___
  Openembedded-core mailing list
  Openembedded-core@lists.openembedded.org
  http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

 OE-Classic already has one packaging split:
 PACKAGES =+ mkfs-jffs2 mkfs-ubifs

 Recently I faced this same issue: for ubiattach we have to install
 full mtd-utils (700KiB)
 FYI there is a recipe for ubi-utils-klibc for more extreme size
 optimization.


 http://cgit.openembedded.org/meta-openembedded/tree/meta-initramfs/recipes-devtools/mtd

 Cheers

 Andrea



___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] split mtd-utils

2013-01-11 Thread Frans Meulenbroeks
2013/1/11 Eric Bénard e...@eukrea.com

 Hi Frans,

 [...]

 IIRC sumtol is used for jffs2 images (to create the erase block
 summary) so that may better fit into mtd-utils-jffs2

 Also, nftl* rfd* doc* ftl* serve_image may go into a separate packate
 (mtd-utils-misc ?) as I believe those tools are not very usefull in
 most configurations using mtd with nowadays's flashs.

 Eric

Hi Eric,

Thanks for the feedback. I'm not into the details of all tools in the
package (I typically only use the flash* mtd* and jffs2 things)
Seems a good plan to me to put these in a misc package
(actually I also considered moving all other stuff that is in my proposal
in mtd-utils into a differently named package, e.g. mtd-utils-core) and
have mtd-utils as an umbrella package dragging in what it does now (so it
should RDEPEND on the meta-utils-* packages but have no other stuff in it).

Best regards, Frans.

PS: partially related: I have filed yocto bug 3702 asking for more
fine-grained control on what is included in an image.
https://bugzilla.yoctoproject.org/show_bug.cgi?id=3702
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] split mtd-utils

2013-01-10 Thread Frans Meulenbroeks
Cool, I'll have a peek at the oe classic recipe and give it a stab along
the same spirit.

Best regards, Frans


2013/1/10 Andrea Adami andrea.ad...@gmail.com

 On Wed, Jan 9, 2013 at 6:17 PM, Richard Purdie
 richard.pur...@linuxfoundation.org wrote:
  On Wed, 2013-01-09 at 17:12 +0100, Frans Meulenbroeks wrote:
  As I am involved in embedded systems where flash is somewhat sparse
  I'm always eager to save a few bytes where possible.
 
  Today I noticed that mtd-utils (1.5.0 from danny) generates for my
  architecture (powerpc) roughly 780k of binaries in usr/sbin. 423k of
  it is due to ubifs related files.
 
  Would it be desired to put this in a separate package?
 
  e.g. mtd-utils-ubi and mtd-utils-nonubi with mtd-utils itself being
  empty but rdepend on those two?
  that way mtd-utils will still give all packages but those only wanting
  the non ubi stuff can limit themselves to that.
 
  If desired I can give this a stab.
 
  Sounds like a sensible split to me...
 
  Cheers,
 
  Richard
 
 
 
  ___
  Openembedded-core mailing list
  Openembedded-core@lists.openembedded.org
  http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

 OE-Classic already has one packaging split:
 PACKAGES =+ mkfs-jffs2 mkfs-ubifs

 Recently I faced this same issue: for ubiattach we have to install
 full mtd-utils (700KiB)
 FYI there is a recipe for ubi-utils-klibc for more extreme size
 optimization.


 http://cgit.openembedded.org/meta-openembedded/tree/meta-initramfs/recipes-devtools/mtd

 Cheers

 Andrea

___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


[OE-core] [PATCH] inotify-tools: new recipe

2013-01-07 Thread Frans Meulenbroeks
From: Frans Meulenbroeks frans.meulenbro...@axon.tv

- mostly taken from oe classic
- updated to latest version (3.14)
- adopted for new home location (project moved from sourceforge to github)

Signed-off-by: Frans Meulenbroeks fransmeulenbro...@gmail.com
---

This patch is provided as is, in the hope that is it useful for someone.
I have no plans to become the maintainer of this recipe or so.

I was not too sure if this is something for oe-core or better suited for meta-oe
The patch is against oe-core.
I've also decided to put it in recipes-extended.

If there is a better layer than oe-core or a better dir than recipes-extended 
feel free
to put it in a more appropriate location.
(and if you do not like it, feel free to ignore)

Have fun!
Frans

 .../inotify-tools/inotify-tools_3.14.bb|   16 
 1 files changed, 16 insertions(+), 0 deletions(-)
 create mode 100644 meta/recipes-extended/inotify-tools/inotify-tools_3.14.bb

diff --git a/meta/recipes-extended/inotify-tools/inotify-tools_3.14.bb 
b/meta/recipes-extended/inotify-tools/inotify-tools_3.14.bb
new file mode 100644
index 000..3597225
--- /dev/null
+++ b/meta/recipes-extended/inotify-tools/inotify-tools_3.14.bb
@@ -0,0 +1,16 @@
+DESCRIPTION = command line utilities for the Linux inotify filesystem change 
notification system.
+AUTHOR = Rohan McGovern
+HOMEPAGE = https://github.com/rvoicilas/inotify-tools/wiki;
+LICENSE = GPLv2
+LIC_FILES_CHKSUM = file://COPYING;md5=ac6c26e52aea428ee7f56dc2c56424c6
+
+PR = r0
+
+SRC_URI = 
http://github.com/downloads/rvoicilas/inotify-tools/inotify-tools-${PV}.tar.gz;
+
+EXTRA_OECONF = --disable-doxygen
+
+inherit autotools
+
+SRC_URI[md5sum] = b43d95a0fa8c45f8bab3aec9672cf30c
+SRC_URI[sha256sum] = 
222bcca8893d7bf8a1ce207fb39ceead5233b5015623d099392e95197676c92f
-- 
1.7.0.4


___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [oe] Minutes: OE Technical Steering Committee 19 June 2012

2012-07-02 Thread Frans Meulenbroeks

  b. elections
  - remove from this list, put back when needed - 2 years from last
 December
  = Jefro to document on wiki  remind the board in October 2013


This is not correct.

From http://www.openembedded.org/wiki/TSC

Each member serves a two-year term. During election time, one member may
stand for election every two months. The next election will be for Richard
Purdie's seat in July, 2013.

So probably this should be initiated in April or May 2013, not Oct.

FM
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


[OE-core] DEFAULT_PREFERENCE = -1

2012-01-26 Thread Frans Meulenbroeks
Dear all,

Triggered by the discussion on oe devel on splitting meta-oe I decided to
do some quick greps on who uses DEFAULT_PREFERENCE in oe-core and meta-oe.

Below are the results.
My questions:
- are recipes with D_P = -1 good enough to enter oe core and/or meta oe.
- is it desirable to have a D_P -1 in an inc file (as happens in
mesa-dri.inc)

Best regards, Frans.

PS: I found the -2 ones especially odd. This seems only useful if there is
someone else already with a -1, which might make the recipe even more
questionable.

frans@frans-desktop:~/workspace$ grep -r DEFAULT_PREFERENCE
openembedded-core
openembedded-core/meta/recipes-core/uclibc/uclibc_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-sato/matchbox-theme-sato/matchbox-theme-sato_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-gnome/gnome/gobject-introspection_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-kernel/oprofile/oprofile_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-connectivity/gypsy/gypsy_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-qt/qt4/qt4-embedded_4.8.0.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-qt/qt4/qt4-x11-free_4.8.0.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-qt/qt4/qt4-tools-nativesdk_4.8.0.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-qt/qt4/qt4-native_4.8.0.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-graphics/libmatchbox/libmatchbox_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-graphics/mutter/mutter_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-graphics/xcb/libxcb_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-graphics/xcb/xcb-proto_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-graphics/clutter/cogl_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-graphics/clutter/clutter_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-graphics/mesa/mesa-dri_git.bb:DEFAULT_PREFERENCE
= -2
openembedded-core/meta/recipes-graphics/mesa/mesa-dri.inc:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-graphics/mesa/qemugl_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-graphics/mesa/mesa-xlib_git.bb:DEFAULT_PREFERENCE
= -2
openembedded-core/meta/recipes-devtools/opkg/opkg-nogpg_svn.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-devtools/opkg/opkg-nogpg_0.1.8.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-devtools/pkgconfig/pkgconfig_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-devtools/qemu/qemu_git.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-devtools/subversion/subversion_1.7.2.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-devtools/binutils/binutils_csl-arm-2008q1.bb:DEFAULT_PREFERENCE
= -1
openembedded-core/meta/recipes-devtools/pseudo/pseudo_git.bb:DEFAULT_PREFERENCE
= -1
frans@frans-desktop:~/workspace$ grep -r DEFAULT_PREFERENCE
meta-openembedded/
meta-openembedded/meta-efl/recipes-support/libsoup/libsoup-2.4_2.37.2.bb:DEFAULT_PREFERENCE
= -1
meta-openembedded/meta-efl/recipes-efl/efl/eet_svn.bb:DEFAULT_PREFERENCE =
-1
meta-openembedded/meta-efl/recipes-efl/efl/eina_svn.bb:DEFAULT_PREFERENCE =
-1
meta-openembedded/meta-efl/recipes-efl/efl/edbus_svn.bb:DEFAULT_PREFERENCE
= -1
meta-openembedded/meta-efl/recipes-efl/efl/efreet_svn.bb:DEFAULT_PREFERENCE
= -1
meta-openembedded/meta-efl/recipes-efl/efl/edje_svn.bb:DEFAULT_PREFERENCE =
-1
meta-openembedded/meta-efl/recipes-efl/efl/embryo_svn.bb:DEFAULT_PREFERENCE
= -1
meta-openembedded/meta-efl/recipes-efl/efl/eeze_svn.bb:DEFAULT_PREFERENCE =
-1
meta-openembedded/meta-efl/recipes-efl/efl/evas_svn.bb:DEFAULT_PREFERENCE =
-1
meta-openembedded/meta-efl/recipes-efl/efl/eio_svn.bb:DEFAULT_PREFERENCE =
-1
meta-openembedded/meta-efl/recipes-efl/efl/ecore_svn.bb:DEFAULT_PREFERENCE
= -1
meta-openembedded/meta-oe/recipes-core/udev/udev_175.bb:DEFAULT_PREFERENCE
= -1
meta-openembedded/meta-oe/recipes-graphics/tslib/tslib_git.bb:DEFAULT_PREFERENCE
= -1
meta-openembedded/meta-oe/recipes-devtools/nodejs/nodejs_0.6.8.bb:DEFAULT_PREFERENCE
= -1
meta-openembedded/contrib/oe-stylize.py:'DEFAULT_PREFERENCE',
meta-openembedded/meta-gnome/recipes-gnome/gnome-panel/gnome-panel3_3.0.2.bb:DEFAULT_PREFERENCE
= -1
meta-openembedded/meta-gnome/recipes-gnome/gobject-introspection/gobject-introspection_git.bb:DEFAULT_PREFERENCE
= -1
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


[OE-core] should recipes be distro agnostic?

2012-01-25 Thread Frans Meulenbroeks
By coincidence I bumped into this in oe-core:

./meta/recipes-devtools/apt/apt.inc:EXTRA_OECONF_append_angstrom = 
ac_cv_glibc_ver=libc6.4

That also triggered me to do this:

frans@frans-desktop:~/workspace/openembedded-core$ grep -r _poky .
./meta/recipes-sato/pcmanfm/pcmanfm_0.9.10.bb:DEPENDS_append_poky = 
libowl
./meta/recipes-sato/pcmanfm/pcmanfm_0.9.10.bb:SRC_URI_append_poky = 
file://owl-window-menu.patch
./meta/recipes-sato/leafpad/leafpad_0.8.18.1.bb:DEPENDS_append_poky = 
libowl
./meta/recipes-sato/leafpad/leafpad_0.8.18.1.bb:SRC_URI_append_poky += 
file://owl-menu.patch;apply=yes 
./meta/recipes-sato/pimlico/dates.inc:OWL_poky = --enable-owl
--disable-dnd
./meta/recipes-sato/pimlico/dates.inc:DEPENDS_append_poky =  libowl
./meta/recipes-sato/pimlico/dates.inc:SRC_URI_append_poky = 
file://dates-owl-window-menu.patch;apply=yes 
./meta/recipes-sato/pimlico/tasks_0.19.bb:OWL_poky = --with-owl
./meta/recipes-sato/pimlico/contacts.inc:DEPENDS_append_poky =  libowl
./meta/recipes-sato/pimlico/contacts.inc:OWL_poky = --enable-owl
./meta/recipes-sato/pimlico/contacts.inc:SRC_URI_append_poky = 
file://contacts-owl-window-menu.patch;apply=yes 
./meta/recipes-sato/pimlico/tasks.inc:DEPENDS_append_poky =  libowl
./meta/recipes-sato/pimlico/tasks.inc:OWL_poky ?= --enable-owl
./meta/recipes-sato/puzzles/oh-puzzles_git.bb:DEPENDS_append_poky = 
libowl
./meta/recipes-sato/puzzles/oh-puzzles_git.bb:SRC_URI_append_poky = 
file://oh-puzzles-owl-menu.patch;striplevel=0 
./meta/recipes-devtools/gcc/gcc-runtime_4.6.bb:EXTRA_OECONF_append_poky-lsb
=  --enable-clocale=gnu

I'd hoped oe-core to be core and not know things about distro's etc.
E.g. the first case with apt, this should probably be fixed with a
.bbappend in the angstrom layer, shouldn't it.
And the poky things in a poky layer.

(at least that is what I expected/hoped)

Btw did the same thing on meta-oe:
no _poky's, two for _angstrom:
./meta-oe/recipes-core/systemd/systemd_git.bb:SYSTEMDDISTRO_angstrom =
angstrom
./meta-gnome/recipes-gnome/system-tools/system-tools-backends_2.10.2.bb:SRC_URI_append_angstrom
=  \

In general I expected recipes to be without distro or hw suffixes (such a
suffx is an indication that the recipe is probably not at the proper place)

Best regards, Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 2/2] u-boot: added generating the env utils (fixes bug 1800)

2012-01-02 Thread Frans Meulenbroeks
Triggered by the remarks of Anders I've been digging into this a little bit
more.
I suggest to ignore tools_fwenv.patch, and only add the changes to the .inc
file.

For fw_env.h a better solution exists, namely using the value from the
config file.
I'll try to submit a patch to upstream to fix that.

Best regards (and a prosperous 2012!)
Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


[OE-core] [PATCH v2 1/2] u-boot: improved DESCRIPTION, fixed HOMEPAGE

2011-12-28 Thread Frans Meulenbroeks
HOMEPAGE pointed to an sf.net page that said that the
page did not exist any more and after that it redirected
to the denx.de page; updated HOMEPAGE to point to that page
Also improved the description to use the official capitalisation

Signed-off-by: Frans Meulenbroeks fransmeulenbro...@gmail.com
---
 meta/recipes-bsp/u-boot/u-boot.inc |2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-bsp/u-boot/u-boot.inc 
b/meta/recipes-bsp/u-boot/u-boot.inc
index 544acd2..a3f50bc 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -1,3 +1,3 @@
+DESCRIPTION = U-Boot - the Universal Boot Loader
+HOMEPAGE = http://www.denx.de/wiki/U-Boot/WebHome;
-DESCRIPTION = U-boot bootloader
-HOMEPAGE = http://u-boot.sf.net;
 SECTION = bootloaders
-- 
1.7.8.1


___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 1/2] u-boot: improved DESCRIPTION, fixed HOMEPAGE

2011-12-28 Thread Frans Meulenbroeks
2011/12/28 Ni Qingliang niqingli...@insigma.com.cn

 I have just done a seperate env tools bb for my project, but use the
 config file to assign the env size and address (which is generated on
 booting from /proc/mtd).

 in the fdt, the u-boot env has a seperate partition identified by it's
 name u-boot environment

 and have some problem:
 1. the fw_printenv will exit with 0 even though can't read the env (CRC
 error).


Apparently the author of the program did not consider this as an error.
See tools/env/fw_env.c



 2. the default env vars in fw_printenv(used when CRC error) are not same
 as it in the u-boot.


If I recall correctly this is addressed in the 2nd patch I posted.
From memory:
Issue is that fw_env.h redefines CONFIG_BOOTCOMMAND. This gives you a
different boot command.
By putting #ifdef's around it the default one is taken

Enjoy!
Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


[OE-core] non-fetching recipes

2011-12-28 Thread Frans Meulenbroeks
Hi,

I noticed that several recipes are non-fetchers. These are mostly the ones
with sources hosted on kernel.org.
Is there an issue with kernel.org again?

I noticed this for oe classic where libpam and util-linux-ng did not fetch
(as well as opkg-utils but that one is from openmoko).
oe maintenance branch seems to have the same problems as well as using
binutils 2.21 which is not available any more. master does not have this as
it has been moved to 2.21a.

I'm crossposting this to oe-core, because I noticed it uses the same
location for libpam (and probably util-linux).

Someone might want to run some fetchalls.

Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


[OE-core] [PATCH 1/2] u-boot: improved DESCRIPTION, fixed HOMEPAGE

2011-12-27 Thread Frans Meulenbroeks
HOMEPAGE pointed to an sf.net page that said that the
page did not exist any more and after that it redirected
to the denx.de page; updated HOMEPAGE to point to that page
Also improved the description to use the official capitalisation

Signed-off-by: Frans Meulenbroeks fransmeulenbro...@gmail.com
---
 meta/recipes-bsp/u-boot/u-boot.inc |2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/meta/recipes-bsp/u-boot/u-boot.inc 
b/meta/recipes-bsp/u-boot/u-boot.inc
index 544acd2..a3f50bc 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -1,3 +1,5 @@
+DESCRIPTION = U-Boot - the Universal Boot Loader
+HOMEPAGE = http://www.denx.de/wiki/U-Boot/WebHome;
 DESCRIPTION = U-boot bootloader
 HOMEPAGE = http://u-boot.sf.net;
 SECTION = bootloaders
-- 
1.7.8.1


___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


[OE-core] [PATCH 2/2] u-boot: added generating the env utils (fixes bug 1800)

2011-12-27 Thread Frans Meulenbroeks
This adds building the u-boot tools for target.
(fw_printenv, fw_setenv).
They are put into a separate package
With these files it becomes possible to modify
the u-boot environment when linux is booted

The #ifdef's in fw_env.h were needed to avoid compilation issues
The constants have been changed to more meaningful defaults
(as 128k flash pages are much more common than 4k pages)
but can be overridden using a configuration file.

This resolves bug 1800

Signed-off-by: Frans Meulenbroeks fransmeulenbro...@gmail.com

---
Note: development and testing was done under openembedded classic.
I've tested it there quite heavily (for mpc8313 if that matters).
I was requested to provide a patch for oe-core too (to resolve bug 1800)
so here it is, although at the moment I am unable to test the patch for oe-core.
I suggest that this is carefully reviewed and tested before being merged.

Frans.
---
 meta/recipes-bsp/u-boot/files/tools_fwenv.patch |   48 +++
 meta/recipes-bsp/u-boot/u-boot.inc  |   21 -
 meta/recipes-bsp/u-boot/u-boot_2011.03.bb   |3 +-
 meta/recipes-bsp/u-boot/u-boot_2011.06.bb   |3 +-
 4 files changed, 70 insertions(+), 5 deletions(-)
 create mode 100644 meta/recipes-bsp/u-boot/files/tools_fwenv.patch

diff --git a/meta/recipes-bsp/u-boot/files/tools_fwenv.patch 
b/meta/recipes-bsp/u-boot/files/tools_fwenv.patch
new file mode 100644
index 000..8e87800
--- /dev/null
+++ b/meta/recipes-bsp/u-boot/files/tools_fwenv.patch
@@ -0,0 +1,48 @@
+Index: u-boot-2011.03/tools/env/fw_env.h
+===
+--- u-boot-2011.03.orig/tools/env/fw_env.h 2011-03-31 23:45:36.0 
+0200
 u-boot-2011.03/tools/env/fw_env.h  2011-12-02 13:55:24.667278362 +0100
+@@ -27,25 +27,35 @@
+  * See included fw_env.config sample file (TRAB board)
+  * for notes on configuration.
+  */
+-#define CONFIG_FILE /etc/fw_env.config
++//#define CONFIG_FILE /etc/fw_env.config
++
++#include config.h
+ 
+ #define HAVE_REDUND /* For systems with 2 env sectors */
+-#define DEVICE1_NAME  /dev/mtd1
+-#define DEVICE2_NAME  /dev/mtd2
++#define DEVICE1_NAME  /dev/mtd0
++#define DEVICE2_NAME  /dev/mtd0
+ #define DEVICE1_OFFSET0x
+-#define ENV1_SIZE 0x4000
+-#define DEVICE1_ESIZE 0x4000
+-#define DEVICE2_OFFSET0x
+-#define ENV2_SIZE 0x4000
+-#define DEVICE2_ESIZE 0x4000
++#define ENV1_SIZE 0x2000
++#define DEVICE1_ESIZE 0x2
++#define DEVICE1_ENVSECTORS 1
++#define DEVICE2_OFFSET0x2
++#define ENV2_SIZE 0x2000
++#define DEVICE2_ESIZE 0x2
++#define DEVICE2_ENVSECTORS 1
+ 
++#ifndef CONFIG_BAUDRATE
+ #define CONFIG_BAUDRATE   115200
++#endif
++#ifndef CONFIG_BOOTDELAY
+ #define CONFIG_BOOTDELAY  5   /* autoboot after 5 seconds */
++#endif
++#ifndef CONFIG_BOOTCOMMAND
+ #define CONFIG_BOOTCOMMAND
\
+   bootp;
\
+   setenv bootargs root=/dev/nfs nfsroot=${serverip}:${rootpath} 
\
+   ip=${ipaddr}:${serverip}:${gatewayip}:${netmask}:${hostname}::off;
\
+   bootm
++#endif
+ 
+ extern int   fw_printenv(int argc, char *argv[]);
+ extern char *fw_getenv  (char *name);
diff --git a/meta/recipes-bsp/u-boot/u-boot.inc 
b/meta/recipes-bsp/u-boot/u-boot.inc
index a3f50bc..bbfc6d6 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -32,16 +32,31 @@ do_compile () {
unset CPPFLAGS
oe_runmake ${UBOOT_MACHINE}
oe_runmake ${UBOOT_MAKE_TARGET}
+   oe_runmake tools env HOSTCC=${CC}
 }
 
 do_install () {
-install -d ${D}/boot
-install ${S}/${UBOOT_BINARY} ${D}/boot/${UBOOT_IMAGE}
-ln -sf ${UBOOT_IMAGE} ${D}/boot/${UBOOT_BINARY}
+   install -d ${D}/boot
+   install ${S}/${UBOOT_BINARY} ${D}/boot/${UBOOT_IMAGE}
+   ln -sf ${UBOOT_IMAGE} ${D}/boot/${UBOOT_BINARY}
+
+   if [ -e ${WORKDIR}/fw_env.config ] ; then
+   install -d ${D}${base_sbindir}
+   install -d ${D}${sysconfdir}
+   install -m 644 ${WORKDIR}/fw_env.config 
${D}${sysconfdir}/fw_env.config
+   install -m 755 ${S}/tools/env/fw_printenv 
${D}${base_sbindir}/fw_printenv
+   install -m 755 ${S}/tools/env/fw_printenv 
${D}${base_sbindir}/fw_setenv
+   fi
+
 }
 
 FILES_${PN} = /boot
 
+PACKAGES += ${PN}-fw-utils
+FILES_${PN}-fw-utils = ${sysconfdir} ${base_sbindir}
+# u-boot doesn't use LDFLAGS for fw files, needs to get fixed, but until then:
+INSANE_SKIP_${PN}-fw-utils = True
+
 do_deploy () {
install ${S}/u-boot.bin ${DEPLOYDIR}/${UBOOT_IMAGE}
 
diff --git a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb 
b/meta/recipes-bsp/u-boot/u-boot_2011.03.bb
index 1ebdbea..fe10f58 100644
--- a/meta/recipes-bsp/u-boot/u-boot_2011.03.bb
+++ b/meta/recipes-bsp/u-boot/u

[OE-core] [PATCH] u-boot: renamed dir from uboot to the more appropriate u-boot

2011-12-08 Thread Frans Meulenbroeks
recipe dirs normally have the name of the (main) package and the official name 
is u-boot

Signed-off-by: Frans Meulenbroeks fransmeulenbro...@gmail.com
---
 .../u-boot/files/fix-arm920t-eabi.patch|   22 +
 ...Drop-config.h-include-in-tools-imximage.h.patch |   39 +
 ...ove-LDSCRIPT-processing-to-the-top-level-.patch |   82 
 ...Drop-config.h-include-in-tools-imximage.h.patch |   39 +
 ...ove-LDSCRIPT-processing-to-the-top-level-.patch |   82 
 meta/recipes-bsp/u-boot/u-boot-mkimage_2011.03.bb  |   31 
 meta/recipes-bsp/u-boot/u-boot-mkimage_2011.06.bb  |   29 +++
 meta/recipes-bsp/u-boot/u-boot.inc |   50 
 meta/recipes-bsp/u-boot/u-boot_2011.03.bb  |   26 ++
 meta/recipes-bsp/u-boot/u-boot_2011.06.bb  |   26 ++
 .../recipes-bsp/uboot/files/fix-arm920t-eabi.patch |   22 -
 ...Drop-config.h-include-in-tools-imximage.h.patch |   39 -
 ...ove-LDSCRIPT-processing-to-the-top-level-.patch |   82 
 ...Drop-config.h-include-in-tools-imximage.h.patch |   39 -
 ...ove-LDSCRIPT-processing-to-the-top-level-.patch |   82 
 meta/recipes-bsp/uboot/u-boot-mkimage_2011.03.bb   |   31 
 meta/recipes-bsp/uboot/u-boot-mkimage_2011.06.bb   |   29 ---
 meta/recipes-bsp/uboot/u-boot.inc  |   50 
 meta/recipes-bsp/uboot/u-boot_2011.03.bb   |   26 --
 meta/recipes-bsp/uboot/u-boot_2011.06.bb   |   26 --
 20 files changed, 426 insertions(+), 426 deletions(-)
 create mode 100644 meta/recipes-bsp/u-boot/files/fix-arm920t-eabi.patch
 create mode 100644 
meta/recipes-bsp/u-boot/u-boot-mkimage-2011.03/0001-Drop-config.h-include-in-tools-imximage.h.patch
 create mode 100644 
meta/recipes-bsp/u-boot/u-boot-mkimage-2011.03/0002-config.mk-move-LDSCRIPT-processing-to-the-top-level-.patch
 create mode 100644 
meta/recipes-bsp/u-boot/u-boot-mkimage/0001-Drop-config.h-include-in-tools-imximage.h.patch
 create mode 100644 
meta/recipes-bsp/u-boot/u-boot-mkimage/0002-config.mk-move-LDSCRIPT-processing-to-the-top-level-.patch
 create mode 100644 meta/recipes-bsp/u-boot/u-boot-mkimage_2011.03.bb
 create mode 100644 meta/recipes-bsp/u-boot/u-boot-mkimage_2011.06.bb
 create mode 100644 meta/recipes-bsp/u-boot/u-boot.inc
 create mode 100644 meta/recipes-bsp/u-boot/u-boot_2011.03.bb
 create mode 100644 meta/recipes-bsp/u-boot/u-boot_2011.06.bb
 delete mode 100644 meta/recipes-bsp/uboot/files/fix-arm920t-eabi.patch
 delete mode 100644 
meta/recipes-bsp/uboot/u-boot-mkimage-2011.03/0001-Drop-config.h-include-in-tools-imximage.h.patch
 delete mode 100644 
meta/recipes-bsp/uboot/u-boot-mkimage-2011.03/0002-config.mk-move-LDSCRIPT-processing-to-the-top-level-.patch
 delete mode 100644 
meta/recipes-bsp/uboot/u-boot-mkimage/0001-Drop-config.h-include-in-tools-imximage.h.patch
 delete mode 100644 
meta/recipes-bsp/uboot/u-boot-mkimage/0002-config.mk-move-LDSCRIPT-processing-to-the-top-level-.patch
 delete mode 100644 meta/recipes-bsp/uboot/u-boot-mkimage_2011.03.bb
 delete mode 100644 meta/recipes-bsp/uboot/u-boot-mkimage_2011.06.bb
 delete mode 100644 meta/recipes-bsp/uboot/u-boot.inc
 delete mode 100644 meta/recipes-bsp/uboot/u-boot_2011.03.bb
 delete mode 100644 meta/recipes-bsp/uboot/u-boot_2011.06.bb

diff --git a/meta/recipes-bsp/u-boot/files/fix-arm920t-eabi.patch 
b/meta/recipes-bsp/u-boot/files/fix-arm920t-eabi.patch
new file mode 100644
index 000..69cb758
--- /dev/null
+++ b/meta/recipes-bsp/u-boot/files/fix-arm920t-eabi.patch
@@ -0,0 +1,22 @@
+---
+ cpu/arm920t/config.mk |4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+--- u-boot-1.3.2.orig/cpu/arm920t/config.mk
 u-boot-1.3.2/cpu/arm920t/config.mk
+@@ -22,13 +22,13 @@
+ #
+ 
+ PLATFORM_RELFLAGS += -fno-strict-aliasing  -fno-common -ffixed-r8 \
+   -msoft-float
+ 
+-PLATFORM_CPPFLAGS += -march=armv4
++#PLATFORM_CPPFLAGS += -march=armv4
+ # =
+ #
+ # Supply options according to compiler version
+ #
+ # =
+-PLATFORM_CPPFLAGS +=$(call cc-option,-mapcs-32,-mabi=apcs-gnu)
++PLATFORM_CPPFLAGS +=$(call cc-option)
+ PLATFORM_RELFLAGS +=$(call cc-option,-mshort-load-bytes,$(call 
cc-option,-malignment-traps,))
diff --git 
a/meta/recipes-bsp/u-boot/u-boot-mkimage-2011.03/0001-Drop-config.h-include-in-tools-imximage.h.patch
 
b/meta/recipes-bsp/u-boot/u-boot-mkimage-2011.03/0001-Drop-config.h-include-in-tools-imximage.h.patch
new file mode 100644
index 000..0405834
--- /dev/null
+++ 
b/meta/recipes-bsp/u-boot/u-boot-mkimage-2011.03/0001-Drop-config.h-include-in-tools-imximage.h.patch
@@ -0,0 +1,39 @@
+From ce56e089ddb51dbd81bb2c86b1646d77447afe39 Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Lo=C3=AFc=20Minier?= loic.min...@linaro.org
+Date: Thu, 3 Feb 2011 15:07:01 +0100
+Subject

[OE-core] Fwd: You have been unsubscribed from the Openembedded-core mailing list

2011-12-02 Thread Frans Meulenbroeks
Why have I been unsubscribed from the list??

Are we censoring users that voice opinions that for whatever reason do not
meet the list maintainer ??
Or did I say something that violates the list etiquette? If so I would have
appreciated a message about that (and perhaps a warning upfront). Not that
I can recall having said something wrong

Frans.


-- Forwarded message --
From: openembedded-core-boun...@lists.openembedded.org
Date: 2011/12/2
Subject: You have been unsubscribed from the Openembedded-core mailing list
To: fransmeulenbro...@gmail.com
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Fwd: You have been unsubscribed from the Openembedded-core mailing list

2011-12-02 Thread Frans Meulenbroeks
2011/12/2 Phil Blundell ph...@gnu.org

 On Fri, 2011-12-02 at 12:02 +0100, Frans Meulenbroeks wrote:
  Why have I been unsubscribed from the list??

 That is rather bizarre.  Mailman will unsubscribe you if too many mails
 are returned undeliverable, but you ought to get a warning before that
 happens.  I'll have a look at the logs and see if I can figure out
 what's going on.  Did it allow you to re-subscribe?

 Yes, I could resubscribe (obviously, otherwise i could not post to the
list as afaik only subscribers may post)

Thanks for looking into this.

Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Coordinating inter-layer dependencies

2011-12-02 Thread Frans Meulenbroeks
Ideally a specific revision of a layer should specify on which revision it
depends on/was tested with, so people can reproduce that same setup.

Randomly mixing revisions is a recipe for problems.
Expecting that a layer informs its users that something is changing is
probably not workable. How would a layer maintainer know what layers are
actually depending on it (especially for things like oe-core)?

If one is using the head of a layer one is living on the bleeding edge (and
hence sometimes one bleeds) (and doing production work based on that seems
unwise).

Then again, I seem to recall that somewhere in the spring we agreed on a
deprecation policy where high impact changes like for toolchain would be
done like:
- commit new version
- wait a while (2 weeks?)
- remove old version

thus giving people a chance to migrate.

Frans.

(PS: of course it is also possible to proceed in a perfectly synchronised
lockstep, but that probably requires some central coordination and some
staging trees or so to test against before the lockstep is performed)
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] shared-mime-info: fix ordering of PACKAGES

2011-11-20 Thread Frans Meulenbroeks
2011/11/19 Martin Jansa martin.ja...@gmail.com

 On Sat, Nov 19, 2011 at 04:02:49PM +0100, Frans Meulenbroeks wrote:
  2011/11/18 Paul Eggleton paul.eggle...@linux.intel.com
 
   On Friday 18 November 2011 15:28:28 Frans Meulenbroeks wrote:
Is ordering of PACKAGES relevant? If so, please enlighten me why.
  
   It is. The packaging code reads this in order and the first package to
   match a
   file is the package that the file will go into; thus if you want to
 add a
   package that will include files that would normally be in one of the
   default
   packages you need to prepend to PACKAGES so that your new package gets
 the
   files first.
  
   Ah, ok. understood.
  Not fully sure if the current behaviour is the most desirable one.
  Is it still possible to put a file into two packages by explicitly adding
  it to both?

 And then make resulting packages RCONFLICTing each other? Imho this is
 bad idea and it's good that this wasn't ever possible afaik.

I was thinking of the situation where a recipe could generate two (indeed
conflicting) packages. E.g. a minimal version and a more expanded version.
Or versions with different options enabled. Or a static and a
dynamic linked version (which still could share e.g. a common config file).
I can also imagine one wants (or must) add a file to all packages. This
could perhaps apply to some license files.

 Also it might be somewhat confusing that a line that adds say a dir to a
 package does not necessarily get all fies in that dir in the package
 Personally I think I prefer to have all added in that case, irrespective
if
 a previous package already took a file from it.

Why confusing?
 should only one package include all /usr/lib/foo and then separte
 subpackages package only some parts of it so we won't pull whole big
 /usr/lib/foo when we need in some image only
 /usr/lib/foo/feature1 and /usr/lib/foo/feature2 in other package? And
 what should shlibs report as feature1 provider? PN_feature1 or PN (which
 also includes all /usr/lib/foo/feature1 files)?

 Sorry but this idea seems wrong to me..

I'm not saying we should move into that direction, I'm more exporing
whether there is a use case for this and whether it is desired or not.

If it is felt that allowing adding a file to two packages is not good, then
I feel that if a file could be candidate for insertion of two packages,
this could be flagged as an error to be resolved in the recipe (by more
precisely specifying the package contents). That way ordering within
PACKAGES becomes irrelevant (and as such it removes a source of silent
errors, like the one that started this thread).

Frans.


  Alternately it could be seen as a QA issue if a file could have been
 added
  to two packages (in which case a more strict pattern should have been
 used
  somewhere)
 
  Frans

  ___
  Openembedded-core mailing list
  Openembedded-core@lists.openembedded.org
  http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


 --
 Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com

 ___
 Openembedded-core mailing list
 Openembedded-core@lists.openembedded.org
 http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] shared-mime-info: fix ordering of PACKAGES

2011-11-19 Thread Frans Meulenbroeks
2011/11/18 Paul Eggleton paul.eggle...@linux.intel.com

 On Friday 18 November 2011 15:28:28 Frans Meulenbroeks wrote:
  Is ordering of PACKAGES relevant? If so, please enlighten me why.

 It is. The packaging code reads this in order and the first package to
 match a
 file is the package that the file will go into; thus if you want to add a
 package that will include files that would normally be in one of the
 default
 packages you need to prepend to PACKAGES so that your new package gets the
 files first.

 Ah, ok. understood.
Not fully sure if the current behaviour is the most desirable one.
Is it still possible to put a file into two packages by explicitly adding
it to both?

Also it might be somewhat confusing that a line that adds say a dir to a
package does not necessarily get all fies in that dir in the package
Personally I think I prefer to have all added in that case, irrespective if
a previous package already took a file from it.
Alternately it could be seen as a QA issue if a file could have been added
to two packages (in which case a more strict pattern should have been used
somewhere)

Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] shared-mime-info: fix ordering of PACKAGES

2011-11-18 Thread Frans Meulenbroeks
2011/11/18 Koen Kooi k...@dominion.thruhere.net

 Signed-off-by: Koen Kooi k...@dominion.thruhere.net
 ---
  .../shared-mime-info/shared-mime-info.inc  |2 +-
  .../shared-mime-info/shared-mime-info_0.91.bb  |2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

 diff --git a/meta/recipes-support/shared-mime-info/shared-mime-info.inc
 b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
 index 57e829c..d7fa67c 100644
 --- a/meta/recipes-support/shared-mime-info/shared-mime-info.inc
 +++ b/meta/recipes-support/shared-mime-info/shared-mime-info.inc
 @@ -19,7 +19,7 @@ FILES_${PN}-dev +=
 ${datadir}/pkgconfig/shared-mime-info.pc

  # freedesktop.org.xml is only required when updating the mime database,
  # package it separately
 -PACKAGES += shared-mime-info-data
 +PACKAGES =+ shared-mime-info-data
  FILES_shared-mime-info-data =
 ${datadir}/mime/packages/freedesktop.org.xml
  RDEPENDS_shared-mime-info-data = shared-mime-info

 diff --git a/meta/recipes-support/shared-mime-info/
 shared-mime-info_0.91.bb b/meta/recipes-support/shared-mime-info/
 shared-mime-info_0.91.bb
 index bc4a8f7..7ae6e10 100644
 --- a/meta/recipes-support/shared-mime-info/shared-mime-info_0.91.bb
 +++ b/meta/recipes-support/shared-mime-info/shared-mime-info_0.91.bb
 @@ -1,5 +1,5 @@
  require shared-mime-info.inc
 -PR = r2
 +PR = r3

  SRC_URI[md5sum] = 982a211560ba4c47dc791ccff34e8fbc
  SRC_URI[sha256sum] =
 98cfebe1d809afb24934e634373821e2a1dfa86fc6462cab230589a1c80988bd
 --
 1.7.2.5


For my education:
Is ordering of PACKAGES relevant? If so, please enlighten me why. If not
why this patch?

Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] serial-console: IMAGE_FEATURE, MACHINE_FEATURE or DISTRO_FEATURE?

2011-10-24 Thread Frans Meulenbroeks
2011/10/24 Koen Kooi k...@dominion.thruhere.net

 Hi,

 Otavio is cleaning up systemd in meta-oe to work with his initramfs setup
 and we stumbled onto a challenge that we'd like some more feedback on.

 The old sysvinit recipes always hardcode a serial getty into inittab,
 systemd installs a getty@port.service file instead. Otavio doesn't need
 (or want) a getty on serial in his setup, but I need (and want) it on all my
 setups. So we agreed that it needs to be a feature :)

 I nitially suggested MACHINE_FEATURE, which Otavio implemented here:
 http://patchwork.openembedded.org/patch/13713/ , but I'm having second
 thoughs about it and am leaning toward an IMAGE_FEATURE instead. Since this
 involves *_FEATURE and we want systemd to move into oe-core eventually I'm
 moving the discussion from OE-devel to OE-core. Not much discussion has been
 going on, so I hope we'll get some more feedback here. I'll try to
 buttonhole some more OE developers at ELC and report back as well.
  http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core



Technically it is probably both.
I've seen embedded systems without a console. That seems to indicate a
machine feature.
Then again even if you have a console you might not want to enable it (e.g.
in order not to give the customer root access or even info wrt printk etc).
That seems to indicate a machine feature.
Probably the answer is inbetween: A machine feature HAS_CONSOLE or something
like that and an image feature CONSOLE_ENABLED.

My 2 cents.

Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 2/5] kernel.bbclass: respect MACHINE_KERNEL_PR

2011-10-20 Thread Frans Meulenbroeks
2011/10/20 Koen Kooi k...@dominion.thruhere.net


 Op 20 okt. 2011, om 13:21 heeft Richard Purdie het volgende geschreven:

  On Thu, 2011-10-20 at 08:23 +0200, Koen Kooi wrote:
  Op 28 sep. 2011, om 22:04 heeft Otavio Salvador het volgende geschreven:
 
  On Wed, Sep 28, 2011 at 16:50, Richard Purdie
  richard.pur...@linuxfoundation.org wrote:
  This patch improves the current situation and I don't foresee the
  autoPR code working soon
 
  Which is why we need to switch to that model and shake out the issues
  sooner than later. Enough is enough with the PR madness and we need to
  get to grips and fix it.
 
  I fully agree this is the way to go but this doesn't mean we ought to
  hold this patch until all this happens. This patch allows the removal
  of the kernel.bbclass from meta-oe so reducing the delta between
  oe-core and meta-oe.
 
  So a month later and no sign of the mythical working
  auto-PR-incrementer or work on it.
 
  A month where we were stabilising for a release. Its on the 1.2 feature
  list and as it happens I've been hearing questions about what is needed
  here.
 
  So can this patch go in? It would mean we can drop kernel.bbclass
  from meta-oe.
 
  I *HATE* this PR bumping stuff. I've just been told we likely need to
  bump the PR for anything using libGL which once again shows that build
  system basically failing to automate building things.
 
  So I'm drawing a line here and no, we can't take this. If its fine to
  expect people to bump PR values manually for lib* changes, its fine for
  kernels too. I'd suggest you do drop this from meta-oe and we start
  building up pressure for the problem to get fixed properly rather than
  letting people wallpaper over the cracks.

 I have products to ship, so treating meta-oe as a plaything and break this
 for the sake of breaking it is unacceptable. I'll let oe-core have the
 monopoly on high-qualitily, but broken metadata.
 .


Didn't we have a TSC to escalate discussions and disagreements between
developers to???

Best regards, Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Bringing an image from OE to OE-Core

2011-10-06 Thread Frans Meulenbroeks
2011/10/6 Saul Wold saul.w...@intel.com

 On 10/05/2011 01:03 PM, Philip Balister wrote:

 On 10/05/2011 03:35 PM, Khem Raj wrote:

 On Wed, Oct 5, 2011 at 7:14 AM, Philip Balisterphi...@balister.org
 wrote:


 1) I do not want rpm in the image. This would confuse my customer base.
 2) I am tired of dropbear, I want openssh only.
 3) I need the full versions of tools, not the busybox ones.
 4) I am not limited to gpv2 software.

 Richard, it looks to me like we should add an item for the next Yocto
 development cycle to review how images are built and try to make the
 base
 stuff in oe-core more usable by everyone. We need to define what
 choices are
 made by distros. For example opkg, rpm, no package management in image.
 Images may want dropbear or openssh.

 Short term, I think I'll copy the tasks/images into my bsp and get some
 stuff together for testing. I'd like a better long term solution though.


 There always will be customizations needed. But we can strive for
 better basic blocks


 Sure. The immediate things I noticed are rpm being installed and lack of
 a way to chose between dropbear/openssh.

 I think it is worth having a conversation to find out if when can make
 it easier for users to create images, with a small number of knobs to
 turn.

  I agree, your 4 items above make sense and we could create a set of tasks
 that can be use it as building blocks, I think that task-core-basic could be
 a starting point for that.

 We did work to enable the selection of either openssh/dropbear but at an
 IMAGE_FEATURE level, not as a DISTRO_FEATURE or virtual.

 Let's see what you come up with for your tasks and we can go from there.

 Thanks
 Sau!

 The usability of tasks in general greatly depends on the use case you are
in.
To give an idea of our use case.
We make embedded systems to which the user only can interact in a specified
way. No online upgrading of packages let alone installing new ones (actually
some of the boards do not even have a network interface).
As we are limited on flash space our images are hand crafted, only to
contain those packages that we really need.
Wrt kernel: we hardly use modules, almost everything is configured into the
kernel (as we exactly know our hardware).

Then again if you have more space and or need to be more flexible tasks
definitely can have their merits. (especially when mapping machine features
to kernel modules etc).

For the specific case with dropbear.
OE classic task-base.bb contained
DISTRO_SSH_DAEMON ?= dropbear
and then used the variable in the image list
OE classic task-poky.bb hardcoded dropbear.

Personally I don't really see too much value in having the package in a var.
It is more work to type ${DISTRO_SSH_DAEMON} in the IMAGE-INSTALL than it is
to type dropbear or openssh.

Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] MINUTES: OE-TSC meeting 16 Sept 2011

2011-09-29 Thread Frans Meulenbroeks
2011/9/28 Jeff Osier-Mixon je...@jefro.net

 Hi Phil,

 These are the recent meetings for which I have records. If you are missing
 the minutes for any of these, please let me know and I will repost. Perhaps
 I should set up a wiki page with these so we don't have to depend solely on
 the mailing list archives.

 4-Aug-2011
 11-Aug-2011
 1-Sept-2011
 16-Sept-2011


aug 11 and sept 1 didn't make it here either.
We used to have a wiki page for storing the minutes, but apparently it is
not kept up to date: http://www.openembedded.org/wiki/TSC

Frans

PS: board, if you read this, were we not supposed to have bi-monthly TSC
elections ???
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] MINUTES: OE-TSC meeting 16 Sept 2011

2011-09-29 Thread Frans Meulenbroeks
PS: apparently the minutes didn't make it to the TSC mailing list archive
either: http://lists.linuxtogo.org/pipermail/tsc/
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] [PATCH] raptor: add recipe

2011-07-28 Thread Frans Meulenbroeks
2011/7/28 Ben Gardiner bengardi...@nanometrics.ca

 Signed-off-by: Frans Meulenbroeks fransmeulenbro...@gmail.com


Eh, yes and no.

I did write the original recipe and commited it on 14 aug 2010 with a
sign-off message.
As such this got my signoff, but I am not sure the signoff should be
repeated if this moved to oe-core.

My sign-off at that time is definitely not to be used as a suggestion it is
ok for oe-core (actually I feel this is not something for oe-core)

And (in reply to Koen), this recipe was tested by me for openembedded when I
submitted it. Afaik I have never build it with oe-core/meta-oe.
Reason to add it then was because flickcurl needed it (and I needed
flickcurl).

Frans.


 Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
 Signed-off-by: Tom Rini tom_r...@mentor.com

 This recipe is a port of recipes/raptor/raptor_1.4.21.bb from
 git://git.openembedded.org/openembedded, commits
 01e8e9f325d8d251e852e7a5704b5fe50880e1ad 'raptor: added recipe' and
 f1d24b5a986233f869364eb109476f5184e76d10 'raptor: add libxml2 DEPENDS'

 The recipe was modified by adding a LIC_FILES_CHKSUM assignment.

 Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca

 ---

 Not tested in the oe-core + meta-openembedded et. al. layers. We are not
 ready yet to migrate from the 2011.03-maintenance branch. This patch is
 proposed for inclusion in oe-core so that we can meet the new policy
 requirements for inclusion in the 2011.03-maintenance branch and submit
 a subsequent pull-request for commit
 f1d24b5a986233f869364eb109476f5184e76d10 raptor: add libxml2 DEPENDS
 ---
  meta/recipes-support/raptor/raptor_1.4.21.bb |   13 +
  1 files changed, 13 insertions(+), 0 deletions(-)
  create mode 100644 meta/recipes-support/raptor/raptor_1.4.21.bb

 diff --git 
 a/meta/recipes-support/raptor/raptor_1.4.21.bbb/meta/recipes-support/raptor/
 raptor_1.4.21.bb
 new file mode 100644
 index 000..84fa69e
 --- /dev/null
 +++ b/meta/recipes-support/raptor/raptor_1.4.21.bb
 @@ -0,0 +1,13 @@
 +DESCRIPTION = Raptor RDF Parser Library
 +SECTION = libs
 +HOMEPAGE = http://librdf.org/raptor/;
 +LICENSE = LGPL 2.1/GPL 2/Apache 2.0
 +LIC_FILES_CHKSUM = file://COPYING;md5=751419260aa954499f7abaabaa882bbe
 +PR = r0
 +SRC_URI = http://download.librdf.org/source/raptor-${PV}.tar.gz;
 +DEPENDS = libxml2
 +
 +inherit autotools pkgconfig
 +
 +SRC_URI[md5sum] = 992061488af7a9e2d933df6b694bb876
 +SRC_URI[sha256sum] =
 db3172d6f3c432623ed87d7d609161973d2f7098e3d2233d0702fbcc22cfd8ca
 --
 1.7.4.1


___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] [PATCH] raptor: add recipe

2011-07-28 Thread Frans Meulenbroeks
2011/7/28 Ben Gardiner bengardi...@nanometrics.ca

 Hi Koen, Frans,

 On Thu, Jul 28, 2011 at 6:48 AM, Koen Kooi k...@dominion.thruhere.net
 wrote:
 
  Op 28 jul. 2011, om 05:38 heeft Ben Gardiner het volgende geschreven:
 
  Signed-off-by: Frans Meulenbroeks fransmeulenbro...@gmail.com
  Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
  Signed-off-by: Tom Rini tom_r...@mentor.com
 
  This recipe is a port of recipes/raptor/raptor_1.4.21.bb from
  git://git.openembedded.org/openembedded, commits
  01e8e9f325d8d251e852e7a5704b5fe50880e1ad 'raptor: added recipe' and
  f1d24b5a986233f869364eb109476f5184e76d10 'raptor: add libxml2 DEPENDS'
 
  Why does this need to be in OE-core and not in some other layer? It
 doesn't look so core to me, especially since nothing in oe-core (or
 meta-oe/efl/gnome for that matter) depends on it

 Because oe-core's meta/recipes-support (naively) seemed like a
 reasonable place for it. I would be happy to re-spin the patch for
 locating it in any layer. Please advise me.


oe-core is only for core recipes. Not sure about the exact definition, but I
suspect it is something like recipes that (virtually) everybody needs.
I'm not sure about layer policies, but maybe this could go to meta-oe.


 It's true that nothing depends on it. As Frans mentions later it's
 only dependent in oe.dev is flickrcurl. Raptor is both a library and a
 utility; we use the latter so raptor itself is a dependency of our
 images.

 
  The recipe was modified by adding a LIC_FILES_CHKSUM assignment.
 
  Signed-off-by: Ben Gardiner bengardi...@nanometrics.ca
 
  ---
 
  Not tested in the oe-core + meta-openembedded et. al. layers.

 I'm sorry for the confusion -- this recipe is/was tested in oe.dev and
 2011.03-maintenance branches.


I'd say that before submitting you should as a minimum test that it builds
properly in the layer you are submitting it (so e.g. we're sure all depends
are there).


  So why are you sending it if it isn't tested?
 
  We are not
  ready yet to migrate from the 2011.03-maintenance branch. This patch is
  proposed for inclusion in oe-core so that we can meet the new policy
  requirements for inclusion in the 2011.03-maintenance branch
 
  Where does it say it's OK to do a pull request for untested recipes? And
 where does it say it needs to be in OE-core first?

 My understanding of Tom's post to the oe.dev mailing list is that
 commits in pull-requests for 2011.03 need to first be oe-core or one
 of its layers.


No idea here.
But why would add a patch for LIC_FILES_CHECKSUM in the maintenance tree.
If I recall correctly those are not needed in 2011.03 (but if it is and it
is missing from the recipe, I think it should be added, if not, probably
leave the recipe as is).


 In Message-ID: 4dfa7108.5020...@mentor.com On Thu, Jun 16, 2011 at
 5:09 PM, Tom Rini tom_r...@mentor.com wrote:
  -BEGIN PGP SIGNED MESSAGE-
  Hash: SHA1
 
  Hi all,
 
  As part of this weeks TSC meeting, an agenda item was brought up about
  the 2011.03-maintenance branch and oe-core / etc.  I have now updated
  the policy about where changes need to be before they can be included in
  2011.03-maintenance to include being in oe-core / meta-oe or other
  relevant public layer instead of being only in the oe.dev master branch.
   This is not a policy change, but a clarification of what was there
  previously.  Thanks all!
 

 On Thu, Jul 28, 2011 at 7:29 AM, Frans Meulenbroeks
 fransmeulenbro...@gmail.com wrote:
 
 
  2011/7/28 Ben Gardiner bengardi...@nanometrics.ca
 
  Signed-off-by: Frans Meulenbroeks fransmeulenbro...@gmail.com
 
  Eh, yes and no.
 
  I did write the original recipe and commited it on 14 aug 2010 with a
  sign-off message.
  As such this got my signoff, but I am not sure the signoff should be
  repeated if this moved to oe-core.

 Fair enough. It is your Sign-off to give or take.

 I was following the patch message guidelines. Example: Importaing
 from Elsewhere Modified in particular:


 http://wiki.openembedded.org/index.php/Commit_Patch_Message_Guidelines#Example:_Importing_from_Elsewhere_Modified


Ah ok, Those guidelines are fairly new and I was not too aware of them. If
it is in accordance with the guidelines it is fine with me to keep them.
I was mainly triggered by the signed-off lines at the beginning of your
message (which I don't think are in accordance to the above mentioned
guidelines).
The only concern I have is that the sign-off would be interpreted as my
blessing for this patch in oe-core. If it only is used as an indicator of
the origin or as a statement that I released and was entitled to release the
original stuff, that is ok (as that is still appropriate).


 I will remove your Sign-off in v2.

  My sign-off at that time is definitely not to be used as a suggestion it
 is
  ok for oe-core (actually I feel this is not something for oe-core)

 Understood -- I am happy to re-spin for inclusion in any layer. Please
 advise.

 Best Regards,
 Ben Gardiner

Re: [OE-core] [PATCH] [PATCH] raptor: add recipe

2011-07-28 Thread Frans Meulenbroeks
2011/7/28 Ben Gardiner bengardi...@nanometrics.ca



 Well, I would not submit _this_ patch for pull-request. I was hoping
 to get this version of raptor included and then submit a pull-request
 for a cherry-pick of the patch to add libxml2 to DEPENDS.


Ah, missed that you also added libxml2.
Not sure about the policies of the maintenance branch, but I suspect the
proper way would be to fix and test in openembedded then submit a pull
request.
Then again maybe the policies for the maintenance branch are different.

[...]

 
 http://wiki.openembedded.org/index.php/Commit_Patch_Message_Guidelines#Example:_Importing_from_Elsewhere_Modified
 
  Ah ok, Those guidelines are fairly new and I was not too aware of them.
 If
  it is in accordance with the guidelines it is fine with me to keep them.
  I was mainly triggered by the signed-off lines at the beginning of your
  message (which I don't think are in accordance to the above mentioned
  guidelines).


 Oh. I thought they were [in accordance]. I guess there would usually
 be a commit body before the initial Signed-off-by:'s ; however, there
 was no original commit body to preserve.


Ah ok, understood.
Actually, I guess we should preserve the first line of the commit too.
This is what git log says:

frans@frans-desktop:~/workspace/openembedded/recipes/raptor$ git log *
commit 01e8e9f325d8d251e852e7a5704b5fe50880e1ad
Author: Frans Meulenbroeks fransmeulenbro...@gmail.com
Date:   Sat Aug 14 21:27:49 2010 +0200

raptor: added recipe

Signed-off-by: Frans Meulenbroeks fransmeulenbro...@gmail.com



Best regards, Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] libiconv checksum wrong

2011-07-20 Thread Frans Meulenbroeks
2011/7/20 Phil Blundell ph...@gnu.org

 Since updating this morning I'm getting

 | ERROR: Function 'File: '/home/pb/sources/libiconv-1.13.1.tar.gz' has
 md5 checksum 7ab33ebd26687c744a37264a330bbe9a when
 d42b97f6ef5dd0ba4469d520ed732fed was expected (from URL:
 'ftp://ftp.gnu.org/gnu/libiconv/libiconv-1.13.1.tar.gz')' failed
 NOTE: package libiconv-1.13.1-r0: task do_fetch: Failed

 The offending commit appears to be
 bd1e627bef29830f9346b4b984905fd8fcf2e716.  As far as I can tell from a
 quick inspection of the files, the checksum that was added for
 libiconv-1.13.1.tar.gz in that commit is actually the md5sum of the
 earlier version 1.11.1.

 This seems to indicate that the added 1.13.1 recipe has not been build by
neither the submitter nor the committer (let alone tested by someone).
is that the kind of quality we want to achieve in oe-core ?

Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] libiconv checksum wrong

2011-07-20 Thread Frans Meulenbroeks
2011/7/20 Khem Raj raj.k...@gmail.com

 On 07/20/2011 04:55 AM, Frans Meulenbroeks wrote:

 2011/7/20 Phil Blundellph...@gnu.org

  Since updating this morning I'm getting

 | ERROR: Function 'File: '/home/pb/sources/libiconv-1.**13.1.tar.gz' has
 md5 checksum 7ab33ebd26687c744a37264a330bbe**9a when
 d42b97f6ef5dd0ba4469d520ed732f**ed was expected (from URL:
 'ftp://ftp.gnu.org/gnu/**libiconv/libiconv-1.13.1.tar.**gz'ftp://ftp.gnu.org/gnu/libiconv/libiconv-1.13.1.tar.gz%27)'
 failed
 NOTE: package libiconv-1.13.1-r0: task do_fetch: Failed

 The offending commit appears to be
 bd1e627bef29830f9346b4b984905f**d8fcf2e716.  As far as I can tell from a
 quick inspection of the files, the checksum that was added for
 libiconv-1.13.1.tar.gz in that commit is actually the md5sum of the
 earlier version 1.11.1.

 This seems to indicate that the added 1.13.1 recipe has not been build by

 neither the submitter nor the committer (let alone tested by someone).
 is that the kind of quality we want to achieve in oe-core ?


 there is already patch sent to fix it.
 http://patches.openembedded.**org/patch/7933/http://patches.openembedded.org/patch/7933/
 libiconv is not default provider of virtual/libiconv on eglibc/glibc based
 systems and sometimes that can trip you over. It happens and as long as we
 find it and fix it quickly I don't see a problem. Do you ?

Some people seem to think differently about this. I still recall the pile of
shit Koen dumped upon me about a year ago when I accidentally removed a
version of openssh or so that was still used. Even though the problem was
fixed very quickly after it was brought to my attention. Ah well. As Orwell
already said all animals are equal but some animals are more equal than
others.


 IMO we should not get so pedantic that people start getting scared of
 making changes


It was by no means my intention to be pedantic.

Then again I *do* think it is good practice if someone creates a new recipe
that (s)he tests it before submitting it.

And my impression was that one of the goals of YP and oe-core was to
increase the quality level.
One of the ways to increase quality is to do a build after pulling changes
and before committing them.
(at least I feel that is one of the ways to increase quality, and yes there
are other ways too).

And where people work, mistakes happen. One can accept that, but one can
also see if there are ways to improve and avoid that a problem re-occurs.

My two cents. Do with it whatever you want. And remember: have fun!

Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] libiconv checksum wrong

2011-07-20 Thread Frans Meulenbroeks
2011/7/20 Richard Purdie richard.pur...@linuxfoundation.org

 On Wed, 2011-07-20 at 16:26 +0200, Frans Meulenbroeks wrote
  there is already patch sent to fix it.
  http://patches.openembedded.org/patch/7933/
  libiconv is not default provider of virtual/libiconv on
  eglibc/glibc based systems and sometimes that can trip you
  over. It happens and as long as we find it and fix it quickly
  I don't see a problem. Do you ?
  Some people seem to think differently about this. I still recall the
  pile of shit Koen dumped upon me about a year ago when I accidentally
  removed a version of openssh or so that was still used. Even though
  the problem was fixed very quickly after it was brought to my
  attention. Ah well. As Orwell already said all animals are equal but
  some animals are more equal than others.
 
 
  IMO we should not get so pedantic that people start getting
  scared of making changes
 
  It was by no means my intention to be pedantic.
 
  Then again I *do* think it is good practice if someone creates a new
  recipe that (s)he tests it before submitting it.
 
  And my impression was that one of the goals of YP and oe-core was to
  increase the quality level.
  One of the ways to increase quality is to do a build after pulling
  changes and before committing them.
  (at least I feel that is one of the ways to increase quality, and yes
  there are other ways too).
 
  And where people work, mistakes happen. One can accept that, but one
  can also see if there are ways to improve and avoid that a problem
  re-occurs.

 I think its fair to ask how this happened and it appears to be due to
 PREFERRED_VERSION and/or PROVIDER confusion. Its unfortunate but I think
 the people involved will not do it again :).

 I don't think its entirely fair to immediately bring into question the
 overall quality goals as we are continuing to work towards those and
 this is an exception, not the norm.

 Cheers,

 Richard, Saul, all,

I understand that these things can happen. It is a pretty basic mistake, but
I made my share of those as well. Unfortunately some community members were
not too forgiving in those situations :-(
I feel as long as those mistakes lead to improvement of the process and we
learn from it, I have no problem with it (and I'm sure this has happened in
this case).
What slightly irridated me was the email of Khem which stated as long as we
find it and fix it quickly I don't see a problem. I'm not sure if that is
the most desirable way of working, and definitely not a statement I had
expected from a TSC member.

Wrt commits: in the past I have been picking up oe postings from other
people without commit access and commit these for them. When I did that I
always made sure to review and to build the changed or new recipe.
I think that is good practice. Now I know that a lot of patches are being
thrown to Richard, so this might not be too workable to do it that way. It
could help though to do some automated tests after taking patches and before
pushing to the oe-core repo. (ideally it would be an (incremental?) build of
world (if there is a bitbake world in YP, actually never attempted that, but
I can also imagine a fetchall or so).

And yeah, on reading back maybe I should have formulated things a little bit
more diplomatic.

Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 03/10] intltool: inherit perlnative

2011-07-20 Thread Frans Meulenbroeks
General remark:

I suspect lots of recipes depend on perl if only in their configure scripts.
I guess doing per-patckage staging could improve the visibility of this.

Also wrt /usr/bin/env perl:
I haven't peeked into the source of env, but ideally non-native recipes
should maybe not rely on host packages, only on the ones we build.
E.g. no /usr/bin in PATH wehn building these (and yes, I know that is not
trivial and will trigger some bugs and it also does not help against hard
coded paths, and probalby some progs will cause a pain (e.g. /bin/sh)).
Maybe a chrooted system would be better, but I vaguely recall that this was
once discussed (and presumably rejected).

Another thing we could do is have  our own env program which (in this case)
does not return a path to perl if the native package is not there (and
thereby causing things to fail instead of silently using the host perl).

Have fun!
Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 0/8] Bug Fixes for M2 Rc2

2011-07-08 Thread Frans Meulenbroeks
2011/7/8 Saul Wold s...@linux.intel.com

 Richard,

 This address the bash, xcb missing from libx11, locale issue and
 the lack of space due to zypper eating space.


I don't think all the dependencies on bash are needed.

E.g:  eglibc: Add RDEPENDS on bash
eglibc itself does not need bash to be present at runtime (or maybe only in
some eglibc configurations).
I am currently happily running an eglibc system with busybox sh without any
problem at all.
Footprintwise for embedded systems that is desirable and as such it is quite
a common setup.

Frans.




 Sau!


 The following changes since commit
 f1fc6d084b079dea21ff1a30b815496452042490:

  pulseaudio: add 0.9.23 (2011-07-07 13:44:36 +0100)

 are available in the git repository at:
  git://git.openembedded.org/openembedded-core-contrib sgw/fix

 http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=sgw/fix

 Saul Wold (8):
  rootfs_rpm: Add 50Meg to image size for zypper
  image.bbclass: add LINGUAS_INSTALL to depends list
  gnome-doc-utils: Add RDEPENDS on bash
  quilt: Add RDEPENDS on bash
  console-tools: Add RDEPENDS on bash
  usbutils: Add RDEPENDS on bash
  eglibc: Add RDEPENDS on bash
  libx11: enable xcb support

  meta/classes/image.bbclass |2 +-
  meta/classes/image_types.bbclass   |2 +-
  meta/classes/rootfs_rpm.bbclass|5 +
  meta/recipes-bsp/usbutils/usbutils_0.91.bb |5 +++--
  .../console-tools/console-tools_0.3.2.bb   |3 ++-
  meta/recipes-core/eglibc/eglibc-package.inc|2 +-
  meta/recipes-core/eglibc/eglibc.inc|2 +-
  meta/recipes-core/eglibc/eglibc_2.12.bb|2 +-
  meta/recipes-core/eglibc/eglibc_2.13.bb|2 +-
  meta/recipes-devtools/quilt/quilt.inc  |2 ++
  meta/recipes-devtools/quilt/quilt_0.48.bb  |2 +-
  meta/recipes-gnome/gnome/gnome-doc-utils.inc   |2 ++
  meta/recipes-gnome/gnome/gnome-doc-utils_0.20.6.bb |2 +-
  meta/recipes-graphics/xorg-lib/libx11_1.3.4.bb |   11 ---
  14 files changed, 30 insertions(+), 14 deletions(-)

 --
 1.7.3.4


 ___
 Openembedded-core mailing list
 Openembedded-core@lists.openembedded.org
 http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 7/8] eglibc: Add RDEPENDS on bash

2011-07-08 Thread Frans Meulenbroeks
2011/7/8 Saul Wold s...@linux.intel.com

 [YOCTO #1214]

 Signed-off-by: Saul Wold s...@linux.intel.com
 ---
  meta/recipes-core/eglibc/eglibc-package.inc |2 +-
  meta/recipes-core/eglibc/eglibc.inc |2 +-
  meta/recipes-core/eglibc/eglibc_2.12.bb |2 +-
  meta/recipes-core/eglibc/eglibc_2.13.bb |2 +-
  4 files changed, 4 insertions(+), 4 deletions(-)

 diff --git a/meta/recipes-core/eglibc/eglibc-package.inc
 b/meta/recipes-core/eglibc/eglibc-package.inc
 index 1c6626c..f0fac76 100644
 --- a/meta/recipes-core/eglibc/eglibc-package.inc
 +++ b/meta/recipes-core/eglibc/eglibc-package.inc
 @@ -69,7 +69,7 @@ FILES_eglibc-utils = ${bindir}/* ${sbindir}/*
  FILES_${PN}-dbg += ${libexecdir}/*/.debug ${libdir}/audit/.debug
  FILES_catchsegv${PKGSUFFIX} = ${bindir}/catchsegv
  RDEPENDS_catchsegv${PKGSUFFIX} = libsegfault
 -EDEPENDS_eglibc-utils = libsotruss
 +RDEPENDS_eglibc-utils += libsotruss bash
  FILES_eglibc-pcprofile = ${base_libdir}/libpcprofile.so
  FILES_eglibc-thread-db${PKGSUFFIX} = ${base_libdir}/libthread_db*
  RPROVIDES_eglibc-dev += libc-dev
 diff --git a/meta/recipes-core/eglibc/eglibc.inc
 b/meta/recipes-core/eglibc/eglibc.inc
 index 74afb9d..058d58e 100644
 --- a/meta/recipes-core/eglibc/eglibc.inc
 +++ b/meta/recipes-core/eglibc/eglibc.inc
 @@ -22,7 +22,7 @@ siteconfig_do_siteconfig_gencache_prepend =  \
  # nptl needs unwind support in gcc, which can't be built without glibc.
  DEPENDS = virtual/${TARGET_PREFIX}gcc-intermediate linux-libc-headers
  #this leads to circular deps, so lets not add it yet
 -#RDEPENDS_ldd +=  bash
 +RDEPENDS_ldd +=  bash


Ah ok, now see that this is only for ldd.
In that case the commit message is misleading. (and probably also not
conformant with the commit message policy).

Frans


  # nptl needs libgcc but dlopens it, so our shlibs code doesn't detect this
  #RDEPENDS_${PN} += ${@['','libgcc']['nptl' in '${GLIBC_ADDONS}']}
  PROVIDES = virtual/libc virtual/${TARGET_PREFIX}libc-for-gcc
 diff --git 
 a/meta/recipes-core/eglibc/eglibc_2.12.bbb/meta/recipes-core/eglibc/
 eglibc_2.12.bb
 index 85d58fa..fd7b485 100644
 --- a/meta/recipes-core/eglibc/eglibc_2.12.bb
 +++ b/meta/recipes-core/eglibc/eglibc_2.12.bb
 @@ -1,7 +1,7 @@
  require eglibc.inc

  DEPENDS += gperf-native
 -PR = r18
 +PR = r19

  SRCREV = 14158

 diff --git 
 a/meta/recipes-core/eglibc/eglibc_2.13.bbb/meta/recipes-core/eglibc/
 eglibc_2.13.bb
 index 7986131..be65787 100644
 --- a/meta/recipes-core/eglibc/eglibc_2.13.bb
 +++ b/meta/recipes-core/eglibc/eglibc_2.13.bb
 @@ -4,7 +4,7 @@ SRCREV = 14157

  DEPENDS += gperf-native
  FILESPATHPKG =. eglibc-svn:
 -PR = r5
 +PR = r6
  PR_append = +svnr${SRCPV}

  EGLIBC_BRANCH=eglibc-2_13
 --
 1.7.3.4


 ___
 Openembedded-core mailing list
 Openembedded-core@lists.openembedded.org
 http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 7/8] eglibc: Add RDEPENDS on bash

2011-07-08 Thread Frans Meulenbroeks
2011/7/8 Koen Kooi k...@dominion.thruhere.net



 Op 8 jul. 2011 om 07:53 heeft Frans Meulenbroeks 
 fransmeulenbro...@gmail.com het volgende geschreven:


 -#RDEPENDS_ldd += bash
 +RDEPENDS_ldd += bash


 Ah ok, now see that this is only for ldd.
 In that case the commit message is misleading. (and probably also not
 conformant with the commit message policy).


 there's an ldd-unbash patch in . dev


 Seems to me that that is a better alternative then

Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH] libxml-parser-perl: convert to BBCLASSEXTEND, merge in OE fixes:wq

2011-07-02 Thread Frans Meulenbroeks
commit headers do not need to reveal that you are using vi :-)
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Proposal: recipe feature switches

2011-07-01 Thread Frans Meulenbroeks
2011/7/1 Richard Purdie richard.pur...@linuxfoundation.org

 On Thu, 2011-06-30 at 18:10 +0100, Chris Elston wrote:
  We're looking to base an embedded development on oe-core in the near
  future, and one of the things we must have is the ability to configure
  features in/out of recipes per distro.
 
  At the moment some recipes have a configuration / set of dependencies
  baked in to the recipe.  For example, gstreamer has theora, ogg and
  vorbis hard-coded in to it's EXTRA_OECONF and DEPENDS.  Currently, you'd
  have to override the DEPENDS and EXTRA_OECONF for the recipe in the
  distro layer - which means moving knowledge about what EXTRA_OECONF
  flags bring in which dependencies outside of the recipe.  It seems that
  this metadata is better contained in the recipe.
 
  I'm proposing something along the lines of the following:

 This is actually along the lines of what I've been thinking too!

  diff --git a/meta/classes/utils.bbclass b/meta/classes/utils.bbclass
  index 9930a24..4f1c54f 100644
  --- a/meta/classes/utils.bbclass
  +++ b/meta/classes/utils.bbclass
  @@ -84,6 +84,15 @@ def oe_system(d, cmd, **kwargs):
   kwargs[shell] = True
   return oe_popen(d, cmd, **kwargs).wait()
 
  +def oe_package_feature_switch(feature, switch_on, switch_off,
  dependencies, d):
  +oeconf = d.getVar(EXTRA_OECONF, True)
  +if feature in d.getVar(PACKAGE_FEATURES, True).split():
  +d.setVar(EXTRA_OECONF, oeconf +   + switch_on)
  +depends = d.getVar(DEPENDS, True)
  +d.setVar(DEPENDS, depends +   + dependencies)
  +else:
  +d.setVar(EXTRA_OECONF, oeconf +   + switch_off)
  +
   oe_soinstall() {
# Purpose: Install shared library file and
#  create the necessary links
  diff --git
  a/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.32.bb
  b/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.32.bb
  index f81011f..6fc6405 100644
  --- a/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.32.bb
  +++ b/meta/recipes-multimedia/gstreamer/gst-plugins-base_0.10.32.bb
  @@ -6,10 +6,12 @@ LIC_FILES_CHKSUM =
  file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
 
  file://COPYING.LIB;md5=55ca817ccb7d5b5b66355690e9abc605 \
 
 
 file://gst/ffmpegcolorspace/utils.c;beginline=1;endline=20;md5=9c83a200b8e597b26ca29df20fc6ecd0
 
  -DEPENDS += virtual/libx11 alsa-lib freetype gnome-vfs liboil libogg
  libvorbis libxv libtheora avahi util-linux tremor
  +DEPENDS += virtual/libx11 alsa-lib freetype gnome-vfs liboil libxv
  avahi util-linux tremor
   RDEPENDS_${PN} += gnome-vfs-plugin-file gnome-vfs-plugin-http
  gnome-vfs-plugin-ftp \
gnome-vfs-plugin-sftp
 
  +PACKAGE_FEATURES ?= ogg vorbis theora
  +
   SRC_URI +=  file://gst-plugins-base-tremor.patch
 
   SRC_URI[md5sum] = 2920af2b3162f3d9fbaa7fabc8ed4d38
  @@ -21,6 +23,12 @@ inherit gettext
 
   EXTRA_OECONF += --disable-freetypetest --disable-pango
 
  +python () {
  + oe_package_feature_switch(ogg, --enable-ogg, --disable-ogg,
  libogg, d)
  + oe_package_feature_switch(vorbis, --enable-vorbis,
  --disable-vorbis, libvorbis, d)
  + oe_package_feature_switch(theora, --enable-theora,
  --disable-theora, libtheora, d)
  +}
  +
   do_configure_prepend() {
# This m4 file contains nastiness which conflicts with libtool
 2.2.2
rm -f ${S}/m4/lib-link.m4
 
  Then in your layer, you could configure which plugins gstreamer should
  be built with like this:
 
  PACKAGE_FEATURES_pn-gst-plugins-base = ogg theora
 
  Or just filter out the package features that you don't want.
 
  With a little extra work, it would also be possible to check the list of
  package features a distro layer requests against those the recipe
  actually implements.  Then we could get an early warning when we pull
  oe-core and a recipe has stopped implementing a feature.

 How about something like the syntax for the recipe:

 PACKAGECONFIG[ogg] = --enable-ogg, --disable-ogg, libogg
 PACKAGECONFIG[vorbis] = --enable-vorbis, --disable-vorbis, libvorbis
 PACKAGECONFIG[theora] = --enable-theora, --disable-theora, libtheora

 and then in base.bbclass we can simply:

 for flag in (d.getVarFlags(PACKAGECONFIG) or {}):
manipulate DEPENDS/EXTRA_OECONF

 I'm also strongly tempted to tie this into the DISTRO_FEATURES variable
 instead of individual PACKAGE_FEATURES variables to make it clear how
 this is intended to be used.

 If we do this, we are going to need to exercise some measure of control
 as someone is going to want an enable/disable option for every config
 option for every package. How do we decide which make sense and which
 don't?

 Note that the more options you have, the more combinations there are to
 test and the less testing the system likely gets. I would therefore like
 to see some kind of natural pressure to only add very useful
 DISTRO_FEATURES. Maybe a criteria like it affecting 5 or more recipes?


Good idea.
Personally I'd like to also bring footprint 

Re: [OE-core] Proposal: recipe feature switches

2011-07-01 Thread Frans Meulenbroeks
2011/7/1 Koen Kooi k...@dominion.thruhere.net


 Op 1 jul 2011, om 10:55 heeft Frans Meulenbroeks het volgende geschreven:

 
  Good idea.
  Personally I'd like to also bring footprint into the equation. If a
 feature drags in lots of additional packages, it is interesting to make it
 configurable.
  My favourite example: bluez dragging in all kind of rendering stuff
 (through DEPENDS) even though the hardware functionality might not be there
 (e.g. you have BT but not audio).

 Which is a great example, since that doesn't impact footprint at all, it's
 an alsa *plugin* that will get produced.


 bluez.inc:DEPENDS = gstreamer gst-plugins-base dbus glib-2.0

I don't think gstreamer is really needed or desired if you e.g. just want to
do some tethering over bluetooth, or in my case, connect to a BT HID, and
they do contribute to both the build time and the footprint.

Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Proposal: recipe feature switches

2011-07-01 Thread Frans Meulenbroeks
Oh by the way, and the message you might be see saying:

This message may not have been sent by: fransmeulenbro...@gmail.com Learn
more Report phishing

This is really google not having their stuff properly configured.
I'm sending email from their web client (https://mail.google.com/)
And yes, it is me :-)

Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Proposal: recipe feature switches

2011-07-01 Thread Frans Meulenbroeks
2011/7/1 Koen Kooi k...@dominion.thruhere.net


 Op 1 jul 2011, om 11:26 heeft Frans Meulenbroeks het volgende geschreven:

 
 
  2011/7/1 Koen Kooi k...@dominion.thruhere.net
 
  Op 1 jul 2011, om 10:55 heeft Frans Meulenbroeks het volgende geschreven:
 
  
   Good idea.
   Personally I'd like to also bring footprint into the equation. If a
 feature drags in lots of additional packages, it is interesting to make it
 configurable.
   My favourite example: bluez dragging in all kind of rendering stuff
 (through DEPENDS) even though the hardware functionality might not be there
 (e.g. you have BT but not audio).
 
  Which is a great example, since that doesn't impact footprint at all,
 it's an alsa *plugin* that will get produced.
 
 
  bluez.inc:DEPENDS = gstreamer gst-plugins-base dbus glib-2.0
 
  I don't think gstreamer is really needed or desired if you e.g. just want
 to do some tethering over bluetooth, or in my case, connect to a BT HID, and
 they do contribute to both the build time and the footprint.

 Again, a plugin, so no footprint issues.

 Well, as far as I know if it is in DEPENDS it will implicitly end up in
RDEPENDS and will become part of the image that contains bluez, unless
special precautions are taken, so it seems to me the image will grow as
plugins do consume space.

Anyway, my current project does not use or need bluez and I use a hand
crafted image picking only the things that I need so haven't really been
bitten by this recently.

Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] PRIORITY

2011-07-01 Thread Frans Meulenbroeks
2011/7/1 Phil Blundell p...@pbcl.net

 On Fri, 2011-07-01 at 11:15 +0100, Richard Purdie wrote:
  On Fri, 2011-07-01 at 10:41 +0100, Phil Blundell wrote:
   We had some discussion of this a few weeks ago but I'm not sure it ever
   reached a firm conclusion.
  
   Is everyone agreed that the PRIORITY variable does not serve any useful
   purpose in oe-core and can/should be removed from the recipes?
 
  I think that was the conclusion, yes. I'll take a patch (or a
  commandline I guess).

 I did:

 $ find . -regex .*\.\(bb\|inc\)$ | xargs sed -i '/^PRIORITY = .*$/d'

 which seemed to produce the desired results.  I can send a patch if you
 like but it would be somewhat large and fairly uninteresting.

 I'd suggest a regex that matches for any whitespace in order also to catch
things like:
./libgpevtype/libgpevtype_0.50.bb:PRIORITY= optional

Have fun! Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Multilib Development Update

2011-06-21 Thread Frans Meulenbroeks
As I already asked before: what is the benefit having this in embedded
systems?

If I am doing an embedded system I know the target hardware, and there is no
need to have e.g. both 32 and 64 bit libs.

Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Question about apply eglibc configurability to create minimal image

2011-06-09 Thread Frans Meulenbroeks
2011/6/9 Mark Hatle mark.ha...@windriver.com

 On 6/9/11 5:57 AM, Kang Kai wrote:
  Hi Mark,
 
  I am focus on eglibc itself compilation with disabling all the
  configurable options, right now eglibc can be compiled with disable all
  the configurable options.
 
  But when I build core-image-minimal in a clear new directory, some
  packages build failed and they need eglibc supports, such as

 core-image-minimal is simply to large of an image to see some of the
 advantages
 of the eglibc configuration.  Realistically the advantages come on single
 application or small (busybox + single application) systems.


Then again if you are in that usecase you're probably better off (spacewise)
using static linking

Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 0/5] network based PR service

2011-05-19 Thread Frans Meulenbroeks
2011/5/19 Lianhao Lu lianhao...@intel.com

 From: Lianhao Lu lianhao...@intel.com

 This series of 5 patches implemented the network based PR service and
 enabled
 the poky to use it during the task do_package and do_package_write_xxx. By
 using the network based PR service and the basichash for
 BB_SIGNATURE_HANDLER,
 the poky user may not need to bump the PR manually everytime he/she changes
 the recipe. The package will get automatically rebuilt and new revision
 number
 reflecting the change will be included in the package feed.

 The first patch [PATCH 1/5] Added the PR service. implemented the network
 based PR service on the server side. It is a XMLRPC server with the sqlite3
 as the backend database. The users query an automated value from this
 service
 by giving  a tuple of (version, checksum). The returned value will
 be used as a part of the package revision value during the package feed
 creation process. The following algorihtm is used so this PR service could
 give
 multiple build servers new values for the new index tuple and same values
 for
 the same index tuple:

 IF the index tuple(version, checksum) is found,  THEN
return the value.
 ELSE
find the max value in the database with the same version.
increment it and save it into the database along with the index tuple.
return the incremented value.
(Note: If no matching version is found, save and return the value of 0).
 ENDIF

 To start the network based PR service daemon, run the following command
 after
 sourcing the environment file oe-init-build-env:

  bitbake-prserv --start

 See bitbake-prserv --help to see a detailed description of the options.

 The remaining 4 patches enable the poky to use the PR service. In order to
 use
 it, the user needs to set 2 varialbes of PRSERV_HOST and PRSERV_PORT (which
 specify the PR service ip address and port, default is localhost and
 8585
 respectively) in the file conf/local.conf under the building directory.
 Unsetting these 2 variables would disable the poky to use the PR service
 for
 backward compatibility.

 The following important new variables are introduced in bitbake.conf:

 PRFORMAT: The final value goes into the package revision. Default is
 something
 like PR.PRAUTO, where PRAUTO is the value returned by the PR service.

 PRAUTOINX: The version part of the index tuple for the PR service. Default
 is PF.
 The checksum of that tuple is the task checksum of do_package.

 Pull URL: git://git.pokylinux.org/poky-contrib.git
  Branch: llu/PR-service
  Browse:
 http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=llu/PR-service

 Thanks,
Lianhao Lu lianhao...@intel.com
 ---


 Lianhao Lu (5):
  Added the PR service.
  conf/bitbake.conf: Added variables for PR service.
  classes/package(prserv).bbclass: Added PR service support.
  classes/package_xxx.class: Added PR service support.
  meta-yocto/local.conf.sample: Added PRSERV_HOST and PRSERV_PORT.

  bitbake/bin/bitbake-prserv|   53 ++
  bitbake/lib/prserv/__init__.py|   11 ++
  bitbake/lib/prserv/db.py  |  100 +++
  bitbake/lib/prserv/serv.py|  198
 +
  meta-yocto/conf/local.conf.sample |5 +
  meta/classes/package.bbclass  |   36 +--
  meta/classes/package_deb.bbclass  |6 +-
  meta/classes/package_ipk.bbclass  |6 +-
  meta/classes/package_rpm.bbclass  |4 +-
  meta/classes/package_tar.bbclass  |4 +-
  meta/classes/prserv.bbclass   |   29 ++
  meta/conf/bitbake.conf|   14 +++-
  12 files changed, 443 insertions(+), 23 deletions(-)
  create mode 100755 bitbake/bin/bitbake-prserv
  create mode 100644 bitbake/lib/prserv/__init__.py
  create mode 100644 bitbake/lib/prserv/db.py
  create mode 100644 bitbake/lib/prserv/serv.py
  create mode 100644 meta/classes/prserv.bbclass


 What should I do to either disable this for some recipes, or use a
different (private) server?
We do not wish to rely on an external server for proprietary recipes (or for
recipes for which we made a local change in an overlay).
How is this case handled?

Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 0/5] network based PR service

2011-05-19 Thread Frans Meulenbroeks
2011/5/19 Richard Purdie richard.pur...@linuxfoundation.org

 On Thu, 2011-05-19 at 13:01 +0200, Frans Meulenbroeks wrote:
  What should I do to either disable this for some recipes, or use a
  different (private) server?
  We do not wish to rely on an external server for proprietary recipes (or
 for
  recipes for which we made a local change in an overlay).
  How is this case handled?

 This could be handled by doing things like setting:

 PRSERV_HOST_pn-myprivaterecipe = somelocalhost

 or

 PRSERV_HOST_pn-myprivaterecipe = 

 to disable it.

 One thing we lack is a good way to apply changes like this, only if code
 is within a given layer. Even in that case, it should be possible with
 anonymous python to look at the location of the current .bb file and
 then conditionally set these variables as appropriate to the right
 server.

 Certainly this needs to be thought about and documented but I think
 there are ways to do it.

 Pardon my ignorance, but I do not really understand the complete flow and
way of working.

Anyway if I do something like:
 PRSERV_HOST_pn-myprivaterecipe = 
would I still be able to use PR in my recipe (like I do today)?

Also we do have the issue that it is desired to be able to rebuild without
network connectivity (e.g. while temporary offline while travelling). Would
that still be possible?

For layers, one solution could be to allow variable overriding on the
overlay level. I can imagine there are more uses for that (and I understand
this requires changes to the bitbake machinery).

Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 0/5] network based PR service

2011-05-19 Thread Frans Meulenbroeks
2011/5/19 Richard Purdie richard.pur...@linuxfoundation.org

 On Thu, 2011-05-19 at 14:02 +0200, Frans Meulenbroeks wrote:
  2011/5/19 Richard Purdie richard.pur...@linuxfoundation.org
 
   On Thu, 2011-05-19 at 13:01 +0200, Frans Meulenbroeks wrote:
What should I do to either disable this for some recipes, or use a
different (private) server?
We do not wish to rely on an external server for proprietary recipes
 (or
   for
recipes for which we made a local change in an overlay).
How is this case handled?
  
   This could be handled by doing things like setting:
  
   PRSERV_HOST_pn-myprivaterecipe = somelocalhost
  
   or
  
   PRSERV_HOST_pn-myprivaterecipe = 
  
   to disable it.
  
   One thing we lack is a good way to apply changes like this, only if
 code
   is within a given layer. Even in that case, it should be possible with
   anonymous python to look at the location of the current .bb file and
   then conditionally set these variables as appropriate to the right
   server.
  
   Certainly this needs to be thought about and documented but I think
   there are ways to do it.
  
   Pardon my ignorance, but I do not really understand the complete flow
 and
  way of working.
 
  Anyway if I do something like:
   PRSERV_HOST_pn-myprivaterecipe = 
  would I still be able to use PR in my recipe (like I do today)?

 Yes, since the PR server appends to this.

  Also we do have the issue that it is desired to be able to rebuild
 without
  network connectivity (e.g. while temporary offline while travelling).
 Would
  that still be possible?

 You could use a local PR server. Obviously connecting to one central
 server without any network connectivity isn't going to happen so we have
 to be realistic about expectations.

 To make a perfect rebuild the local PR server would need a dump of the
 database on the central server. There isn't code for that at the moment
 and I don't think its the highest priority task out there or the most
 important use case but its certainly possible for someone to add.


I'd say it would already be nice if some caching is being done locally (just
like is done with e.g. downloads).


  For layers, one solution could be to allow variable overriding on the
  overlay level. I can imagine there are more uses for that (and I
 understand
  this requires changes to the bitbake machinery).

 There is certainly a use case for something like this. The exact
 implementation and workings needs a lot more thought and discussion
 though. I believe its at least already possible in anonymous python (and
 if not, any extensions needed shouldn't be invasive by comparison).

 Hm. you consider this PR change to be non-invasive?

BTW I am not saying it is not good, and I understand the problem that you
want to solve, but I feel this could require some more thought wrt the
issues I raised before in this thread (and some more documentation and usage
info).

Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 2/2] opkg-utils: Allow uppercase lettlers in package names

2011-05-19 Thread Frans Meulenbroeks
2011/5/19 Khem Raj raj.k...@gmail.com

 Some packages have uppercase letters in their names e.g.
 efikamx kernel. We allow uppercase character with
 uppercase-letters.patch

 Import mtime-int.patch from OE commit
 c81990aecae3bdf70a7c924699776b248ab7a006

 Signed-off-by: Khem Raj raj.k...@gmail.com
 ---
  .../opkg-utils/opkg-utils/mtime-int.patch  |   29 +
  .../opkg-utils/opkg-utils/uppercase-letters.patch  |   44
 
  meta/recipes-devtools/opkg-utils/opkg-utils_svn.bb |8 +++-
  3 files changed, 79 insertions(+), 2 deletions(-)
  create mode 100644
 meta/recipes-devtools/opkg-utils/opkg-utils/mtime-int.patch
  create mode 100644
 meta/recipes-devtools/opkg-utils/opkg-utils/uppercase-letters.patch

 diff --git a/meta/recipes-devtools/opkg-utils/opkg-utils/mtime-int.patch
 b/meta/recipes-devtools/opkg-utils/opkg-utils/mtime-int.patch
 new file mode 100644
 index 000..fdbce21
 --- /dev/null
 +++ b/meta/recipes-devtools/opkg-utils/opkg-utils/mtime-int.patch
 @@ -0,0 +1,29 @@
 +Convert mtime to int before comparing it
 +
 +The st_mtime attribute (which is a float) is compared against a value
 +from the timestamp database, which was stored as an integer there.
 +
 +When working on a filesystem with precise timestamps the comparision
 +will fail nearly everytime hence.
 +
 +Although it might be possible to enhance the database to store the
 +fractional part too, this will complicate things more than we would
 +gain by this change.


This part is not covered by the commit message. Was it intended to be in
here.?
Rest of the patch looks fine to me.

Frans

 +
 +Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
 +
 +Upstream-Status: Pending
 +
 +Index: opkg-utils/opkg-make-index
 +===
 +--- opkg-utils.orig/opkg-make-index
  opkg-utils/opkg-make-index
 +@@ -100,7 +100,7 @@ for filename in files:
 +  pkg = None
 +  fnameStat = os.stat(filename)
 +  if old_pkg_hash.has_key(basename):
 +-  if pkgsStamps.has_key(basename) and fnameStat.st_mtime ==
 pkgsStamps[basename]:
 ++  if pkgsStamps.has_key(basename) and int(fnameStat.st_mtime) ==
 pkgsStamps[basename]:
 + if (verbose):
 +sys.stderr.write(Found %s in Packages\n % (filename,))
 + pkg = old_pkg_hash[basename]
 diff --git
 a/meta/recipes-devtools/opkg-utils/opkg-utils/uppercase-letters.patch
 b/meta/recipes-devtools/opkg-utils/opkg-utils/uppercase-letters.patch
 new file mode 100644
 index 000..689d780
 --- /dev/null
 +++ b/meta/recipes-devtools/opkg-utils/opkg-utils/uppercase-letters.patch
 @@ -0,0 +1,44 @@
 +when building packages, accept uppercase letters in the package name
 +Fixes issues like
 +
 +
 +| Packaged contents of kernel into
 /home/kraj/work/angstrom/build/tmp-angstrom_2010_x-eglibc/work/efikamx-angstrom-linux-gnueabi/linux-efikamx-2.6.31-r0/deploy-ipks/efikamx/kernel_2.6.31-r0_efikamx.ipk
 +| kernel-2.6.31.12-ER1
 +| *** Error: Package name  contains illegal characters, (other than
 [a-z0-9.+-])
 +|
 +| opkg-build: Please fix the above errors and try again.
 +| ERROR: Function 'opkg-build execution failed' failed
 +
 +Signed-off-by: Khem Raj raj.k...@gmail.com
 +
 +Upstream-Status: Pending
 +Index: opkg-utils/opkg-buildpackage
 +===
 +--- opkg-utils.orig/opkg-buildpackage
  opkg-utils/opkg-buildpackage
 +@@ -65,8 +65,8 @@ pkg_appears_sane_control() {
 +   required_field Maintainer /dev/null
 +   required_field Description /dev/null
 +
 +-  if echo $pkg | grep '[^a-z0-9.+-]'; then
 +-  echo opkg-build: Error: Package name $name contains
 illegal characters, (other than [a-z0-9.+-])
 ++  if echo $pkg | grep '[^a-zA-Z0-9.+-]'; then
 ++  echo opkg-build: Error: Package name $name contains
 illegal characters, (other than [a-zA-Z0-9.+-])
 +   PKG_ERROR=1;
 +   fi
 +
 +Index: opkg-utils/opkg-build
 +===
 +--- opkg-utils.orig/opkg-build
  opkg-utils/opkg-build
 +@@ -118,8 +118,8 @@ You probably want to chown these to a sy
 +   disallowed_filename=`disallowed_field Filename`
 +   [ $? -ne 0 ]  PKG_ERROR=1
 +
 +-  if echo $pkg | grep '[^a-z0-9.+-]'; then
 +-  echo *** Error: Package name $name contains illegal
 characters, (other than [a-z0-9.+-]) 2
 ++  if echo $pkg | grep '[^a-zA-Z0-9.+-]'; then
 ++  echo *** Error: Package name $name contains illegal
 characters, (other than [a-zA-Z0-9.+-]) 2
 +   PKG_ERROR=1;
 +   fi
 +
 diff --git 
 a/meta/recipes-devtools/opkg-utils/opkg-utils_svn.bbb/meta/recipes-devtools/opkg-utils/
 opkg-utils_svn.bb
 index 0db6f80..7e4c6e4 100644
 --- a/meta/recipes-devtools/opkg-utils/opkg-utils_svn.bb
 +++ b/meta/recipes-devtools/opkg-utils/opkg-utils_svn.bb
 @@ -9,10 +9,13 @@ RDEPENDS_${PN} = 

Re: [OE-core] [PATCH 5/6] conf/distro/include/default-distrovars.inc: Create set of default 'distro' variable values

2011-05-11 Thread Frans Meulenbroeks
2011/5/10 Richard Purdie richard.pur...@linuxfoundation.org

 On Tue, 2011-05-10 at 16:26 +0200, Frans Meulenbroeks wrote:
  Some minor remarks on the default-distrovars.inc contents:

 To quote the email prefacing this patch series:

 
 I did dump a load of default variables into default-distrovars.inc,
 I'm not calling that file finished, I just had to draw the line
 somewhere and start a discussion about this :)
 

 but we can have this discussion.


I saw that note. Idea was indeed to initiate the discussion and make
suggestions for improvements.



  2011/5/10 Richard Purdie richard.pur...@linuxfoundation.org
  [...]
 
   diff --git a/meta/conf/distro/include/default-distrovars.inc
   b/meta/conf/distro/include/default-distrovars.inc
   new file mode 100644
   index 000..ab26a30
   --- /dev/null
   +++ b/meta/conf/distro/include/default-distrovars.inc
   @@ -0,0 +1,43 @@
   +QA_LOGFILE = ${TMPDIR}/qa.log
  
  Should this be here? I would expect the class file that uses this to have
  this as default and/or have this in local.conf.sample.
  This does not seem too distro specific

 If this is unset, the logfile isn't generated at all with the current
 defaults. I don't claim the current defaults are correct but its a
 separate patch if we do want to change the defaults.


It is good to to have the qa file made by default, but I am not sure whether
it is distro specific.
Actually I probably would expect things like QA_LOGFILE to live in the same
spot as where TMPDIR and DOWNLOADDIR etc are defined


   +
   +IMAGE_ROOTFS_SIZE_ext2 ?= 131072
  
  Hm, again something that is probalby not distro related. And why only for
  _ext2 and not eg for _ext3 or for _jffs2
  Personally I always stuff this in my image recipe

 Left there pending a review of where various image sizes are set.


Ok understood


   +
   +OEINCLUDELOGS ?= yes
  
  in local.conf.sample?

 If its there, this can be dropped, yes. I think forcing it on by default
 may be a good thing though. I'm not 100% sure that variable even still
 exists tbh.


In openembedded/classes and openembedded/site this var does not exist. I do
not have an oe-core/yocto tree handy to check if it is there.
BTW I do have BBINCLUDELOGS in my local.conf.sample. As far as I know this
is a bitbake var and actually being used.


   +KERNEL_CONSOLE ?= ttyS0
  
  I'm inclined to put this into the machine conf. Is this really a distro
  thing?

 No, left for compatibility pending cleanup.

   +
   +require conf/distro/include/preferred-xorg-versions.inc
   +
   +PCMCIA_MANAGER ?= pcmciautils
   +
   +IMAGE_LINGUAS ?= en-us en-gb
   +LIMIT_BUILT_LOCALES ?= POSIX en_US en_GB
   +ENABLE_BINARY_LOCALE_GENERATION ?= 1
   +LOCALE_UTF8_ONLY ?= 0
  
  Again something I tend to do in image recipes; then again this might be
  because I do not use feeds, only make images.

 These ones are more distro policy and whilst an image can change them a
 default is good in those cases. If there are defaults set elsewhere we
 should review and pick the correct ones.

   +DISTRO_FEATURES ?= alsa bluetooth ext2 irda pcmcia usbgadget usbhost
 wifi
   nfs zeroconf pci
  
  Is this the complete range? I would expect all features to be enabled by
  default (and let MACHINE_FEATURES reduce that set)

 Needs further investigation.

   +
   +DISTRO_EXTRA_RDEPENDS += task-core-boot
   +DISTRO_EXTRA_RRECOMMENDS += kernel-module-af-packet
   +
   +IMAGE_FEATURES ?= 
  
  Has a var with ?= assignment of an empty string any meaning. Thought an
  empty string would be the default.

 No, its not.

   +
   +# This is a list of packages that are used by the build system to
 build
   the distribution, they are not
   +# directly part of the distribution.
   +HOSTTOOLS_WHITELIST_GPLv3 ?= 
   +WHITELIST_GPLv3 ?= less
  
  I'm not sure why less is listed in the line above:
  the less license is not gpl (at least not less 443)
 
 
   +LGPLv2_WHITELIST_GPLv3 ?= libassuan gnutls libtasn1 libidn libgcc
   gcc-runtime
   +
   +# This is a list of packages that require a commercial license to ship
   +# product. If shipped as part of an image these packages may have
   +# implications so they are disabled by default
   +COMMERCIAL_LICENSE ?= lame gst-fluendo-mp3 libmad mpeg2dec ffmpeg
 qmmp
   +COMMERCIAL_AUDIO_PLUGINS ?= 
   +# COMMERCIAL_AUDIO_PLUGINS ?= gst-plugins-ugly-mad
   gst-plugins-ugly-mpegaudioparse
   +COMMERCIAL_VIDEO_PLUGINS ?= 
   +# COMMERCIAL_VIDEO_PLUGINS ?= gst-plugins-ugly-mpeg2dec
   gst-plugins-ugly-mpegstream gst-plugins-bad-mpegvideoparse
   +COMMERCIAL_QT ?= 
   +# COMMERCIAL_QT ?= qmmp
   +# Set of common licenses used for license.bbclass
   +COMMON_LICENSE_DIR ??= ${COREBASE}/meta/files/common-licenses
  
  maybe the above could be in a separate license.inc file or so.embedded

 Possibly, setting these to defaults is good though.

   +
   +BB_GENERATE_MIRROR_TARBALLS ??= 0
  
  local.conf.sample ?

 I regard local.conf.sample as things a new user cares about. I don't
 think this fits

Re: [OE-core] [PATCH 5/6] conf/distro/include/default-distrovars.inc: Create set of default 'distro' variable values

2011-05-10 Thread Frans Meulenbroeks
Some minor remarks on the default-distrovars.inc contents:

2011/5/10 Richard Purdie richard.pur...@linuxfoundation.org
[...]

 diff --git a/meta/conf/distro/include/default-distrovars.inc
 b/meta/conf/distro/include/default-distrovars.inc
 new file mode 100644
 index 000..ab26a30
 --- /dev/null
 +++ b/meta/conf/distro/include/default-distrovars.inc
 @@ -0,0 +1,43 @@
 +QA_LOGFILE = ${TMPDIR}/qa.log

Should this be here? I would expect the class file that uses this to have
this as default and/or have this in local.conf.sample.
This does not seem too distro specific

 +
 +IMAGE_ROOTFS_SIZE_ext2 ?= 131072

Hm, again something that is probalby not distro related. And why only for
_ext2 and not eg for _ext3 or for _jffs2
Personally I always stuff this in my image recipe

 +
 +OEINCLUDELOGS ?= yes

in local.conf.sample?

 +KERNEL_CONSOLE ?= ttyS0

I'm inclined to put this into the machine conf. Is this really a distro
thing?

 +
 +require conf/distro/include/preferred-xorg-versions.inc
 +
 +PCMCIA_MANAGER ?= pcmciautils
 +
 +IMAGE_LINGUAS ?= en-us en-gb
 +LIMIT_BUILT_LOCALES ?= POSIX en_US en_GB
 +ENABLE_BINARY_LOCALE_GENERATION ?= 1
 +LOCALE_UTF8_ONLY ?= 0

Again something I tend to do in image recipes; then again this might be
because I do not use feeds, only make images.

 +
 +DISTRO_FEATURES ?= alsa bluetooth ext2 irda pcmcia usbgadget usbhost wifi
 nfs zeroconf pci

Is this the complete range? I would expect all features to be enabled by
default (and let MACHINE_FEATURES reduce that set)

 +
 +DISTRO_EXTRA_RDEPENDS += task-core-boot
 +DISTRO_EXTRA_RRECOMMENDS += kernel-module-af-packet
 +
 +IMAGE_FEATURES ?= 

Has a var with ?= assignment of an empty string any meaning. Thought an
empty string would be the default.

 +
 +# This is a list of packages that are used by the build system to build
 the distribution, they are not
 +# directly part of the distribution.
 +HOSTTOOLS_WHITELIST_GPLv3 ?= 
 +WHITELIST_GPLv3 ?= less

I'm not sure why less is listed in the line above:
the less license is not gpl (at least not less 443)


 +LGPLv2_WHITELIST_GPLv3 ?= libassuan gnutls libtasn1 libidn libgcc
 gcc-runtime
 +
 +# This is a list of packages that require a commercial license to ship
 +# product. If shipped as part of an image these packages may have
 +# implications so they are disabled by default
 +COMMERCIAL_LICENSE ?= lame gst-fluendo-mp3 libmad mpeg2dec ffmpeg qmmp
 +COMMERCIAL_AUDIO_PLUGINS ?= 
 +# COMMERCIAL_AUDIO_PLUGINS ?= gst-plugins-ugly-mad
 gst-plugins-ugly-mpegaudioparse
 +COMMERCIAL_VIDEO_PLUGINS ?= 
 +# COMMERCIAL_VIDEO_PLUGINS ?= gst-plugins-ugly-mpeg2dec
 gst-plugins-ugly-mpegstream gst-plugins-bad-mpegvideoparse
 +COMMERCIAL_QT ?= 
 +# COMMERCIAL_QT ?= qmmp
 +# Set of common licenses used for license.bbclass
 +COMMON_LICENSE_DIR ??= ${COREBASE}/meta/files/common-licenses

maybe the above could be in a separate license.inc file or so.

 +
 +BB_GENERATE_MIRROR_TARBALLS ??= 0

local.conf.sample ?

Enjoy! Frans
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 0/1] poky-default-revisions: move SRCREV to every recipe

2011-05-04 Thread Frans Meulenbroeks
2011/5/4 Richard Purdie richard.pur...@linuxfoundation.org

 On Wed, 2011-05-04 at 22:05 +0800, Yu Ke wrote:
  From: Yu Ke ke...@intel.com
 
  move the SRCREV from poky-default-revisions.inc to its corresponding
 recipe,
  in this case, those non poky distro can also use its SRCREV.
 
  Pull URL: git://git.pokylinux.org/poky-contrib.git
Branch: kyu3/srcrev-recipe
Browse:
 http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=kyu3/srcrev-recipe
 
  Thanks,
  Yu Ke ke...@intel.com
  ---
 
 
  Yu Ke (1):
poky-default-revisions: move the SRCREV to recipe file

 Merged to master, thanks for resolving this issue! :)

 Cheers,

 Richard

 I would have preferred a more standardised placement of SRCREV.
Most of the time the SRCREV is before the PV, but not always (and sometimes
separated with an empty line and sometimes not).

Also there is at least one error introduced:

diff --git 
a/meta/recipes-devtools/yaffs2/yaffs2-utils_cvs.bbb/meta/recipes-devtools/yaffs2/
yaffs2-utils_cvs.bb
index 6171fe5..c729c7c 100644
--- 
a/meta/recipes-devtools/yaffs2/yaffs2-utils_cvs.bbhttp://git.pokylinux.org/cgit.cgi/poky-contrib/tree/meta/recipes-devtools/yaffs2/yaffs2-utils_cvs.bb?h=kyu3/srcrev-recipeid=d2e078aa046ae6c4f169695f546cf229db5be1f7
+++ 
b/meta/recipes-devtools/yaffs2/yaffs2-utils_cvs.bbhttp://git.pokylinux.org/cgit.cgi/poky-contrib/tree/meta/recipes-devtools/yaffs2/yaffs2-utils_cvs.bb?h=kyu3/srcrev-recipeid=c75b49ff5cfd10f5187af2b66a0a8d5513460374
@@ -1,3 +1,4 @@
require yaffs2-utils.inc
PR = r1
+SRCDAT = 20071107

That should be SRCDATE. There might be more of these, my eye just fell on
this one.

Frans

PS: speaking of yaffs2 utils: it could be considered to move to a somewhat
newer version (not that I care as I do not use yaffs2)
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] [PATCH 0/1] poky-default-revisions: move SRCREV to every recipe

2011-05-04 Thread Frans Meulenbroeks
2011/5/4 Richard Purdie richard.pur...@linuxfoundation.org

 On Wed, 2011-05-04 at 16:24 +0200, Frans Meulenbroeks wrote:
  Most of the time the SRCREV is before the PV, but not always (and
 sometimes
  separated with an empty line and sometimes not).

 Patches welcome...


I know. It was more a hint for future changes.
Also this probably should be part of a more global style cleanup to make
things more according to the style guide.

[...]

 
  PS: speaking of yaffs2 utils: it could be considered to move to a
 somewhat
  newer version (not that I care as I do not use yaffs2)

 It is something we should look at going, yes although I don't think
 yaffs2 has changed that much in a while.

 The latest changes are about 13 months old, but there seem to be quite some
changes since 2007 (although quite some seem cosmetical).

(btw there is also an abiword cvs recipe with a srcdate of 2007; somehow odd
given the number of regular releases abiword has).

Frans.
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core


Re: [OE-core] Proposed Multilib Implementation Brainstorming

2011-04-06 Thread Frans Meulenbroeks
2011/4/5 Richard Purdie richard.pur...@linuxfoundation.org:

[...]

 Does this make sense to everyone, are there any questions/ objections/
 concerns/ things I've missed?

I think most embedded systems would only use one lib. To take your
lib/lib64 example:
If I am developing for an embedded system I know whether it will run
as 32 or 64 bit, so there is no need to have both.

multilib has its merits when it comes to supporting multiple hardware systems.
However as in the embedded world one is typically targeting a specific
hardware configuration.
(actually I don't recall having seen requests for multilib on the ML
before, although I could have missed it).

Also I'm somewhat worried by the actual complexity this adds (to the
build process and the recipes, and timewise probably also to the
bootstrap process as additional packages have to be built).

Not sure if that is a desirable route forward, but if we (we as in OE
members + developers) feel that OE should go that way, I would
sugggest to have a way to opt-in or opt-out

Frans

___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core