[oe] [meta-oe][PATCH] krb5: use PACKAGE_BEFORE_PN

2024-01-03 Thread Enrico Scholz via lists.openembedded.org
From: Enrico Scholz 

The 'krb5-config' tool should be in ${PN}-dev (as intented by
binconfig class).

Use PACKAGE_BEFORE_PN for extra packages so that -dev is handled
before -user.

Signed-off-by: Enrico Scholz 
---
 .../recipes-connectivity/krb5/krb5_1.20.2.bb  | 42 +--
 1 file changed, 21 insertions(+), 21 deletions(-)

diff --git a/meta-oe/recipes-connectivity/krb5/krb5_1.20.2.bb 
b/meta-oe/recipes-connectivity/krb5/krb5_1.20.2.bb
index 54e6b778b0c2..e1fe5e567178 100644
--- a/meta-oe/recipes-connectivity/krb5/krb5_1.20.2.bb
+++ b/meta-oe/recipes-connectivity/krb5/krb5_1.20.2.bb
@@ -103,26 +103,26 @@ do_install:append() {
 -i ${D}${bindir}/krb5-config
 }
 
-PACKAGES =+ "${PN}-admin-server \
- ${PN}-gss-samples \
- ${PN}-k5tls \
- ${PN}-kdc \
- ${PN}-kdc-ldap \
- ${PN}-kpropd \
- ${PN}-otp \
- ${PN}-pkinit \
- ${PN}-spake \
- ${PN}-user \
- libgssapi-krb5 \
- libgssrpc \
- libk5crypto \
- libkadm5clnt-mit \
- libkadm5srv-mit \
- libkdb5 \
- libkrad \
- libkrb5 \
- libkrb5support \
- libverto"
+PACKAGE_BEFORE_PN =+ "${PN}-admin-server \
+  ${PN}-gss-samples \
+  ${PN}-k5tls \
+  ${PN}-kdc \
+  ${PN}-kdc-ldap \
+  ${PN}-kpropd \
+  ${PN}-otp \
+  ${PN}-pkinit \
+  ${PN}-spake \
+  ${PN}-user \
+  libgssapi-krb5 \
+  libgssrpc \
+  libk5crypto \
+  libkadm5clnt-mit \
+  libkadm5srv-mit \
+  libkdb5 \
+  libkrad \
+  libkrb5 \
+  libkrb5support \
+  libverto"
 
 FILES:${PN} = "${libdir}/krb5/plugins/preauth/test.so"
 FILES:${PN}-doc += "${datadir}/examples"
@@ -198,4 +198,4 @@ pkg_postinst:${PN}-kdc () {
 BBCLASSEXTEND = "native nativesdk"
 
 inherit multilib_script
-MULTILIB_SCRIPTS = "${PN}-user:${bindir}/krb5-config"
+MULTILIB_SCRIPTS = "${PN}-dev:${bindir}/krb5-config"
-- 
2.43.0


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



[oe] [meta-multimedia] [PATCH] sox: remove ffmpeg dependency

2023-04-03 Thread Enrico Scholz via lists.openembedded.org
From: Enrico Scholz 

ffmpeg format handler was removed from sox in 2013 ([1]).  Drop it
also from DEPENDS.

This makes sox generally available without the need to whitelist the
commercial ffmpeg license.

[1] 
https://sourceforge.net/p/sox/code/ci/5ae4049727d4f29036ad541bde5863c850aa7755

Signed-off-by: Enrico Scholz 
---
 meta-multimedia/recipes-multimedia/sox/sox_14.4.2.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-multimedia/recipes-multimedia/sox/sox_14.4.2.bb 
b/meta-multimedia/recipes-multimedia/sox/sox_14.4.2.bb
index ace8743f3afd..011cbc2a9df0 100644
--- a/meta-multimedia/recipes-multimedia/sox/sox_14.4.2.bb
+++ b/meta-multimedia/recipes-multimedia/sox/sox_14.4.2.bb
@@ -4,7 +4,7 @@ and can apply different effects and filters to the audio data."
 HOMEPAGE = "http://sox.sourceforge.net;
 SECTION = "audio"
 
-DEPENDS = "libpng ffmpeg libsndfile1 libtool"
+DEPENDS = "libpng libsndfile1 libtool"
 
 PACKAGECONFIG ??= "${@bb.utils.filter('DISTRO_FEATURES', 'alsa pulseaudio', 
d)} \
magic \
-- 
2.39.2


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



[oe] [meta-oe] nodejs-oe-cache-native: initial checkin

2022-05-19 Thread Enrico Scholz via lists.openembedded.org
This implements an 'npm cache add' like functionality but allows to
specify the key of the data and sets metadata which are required to
find the data.

It is used to cache information as done during 'npm install'.

Keyformat and metadata are nodejs version specific.

Signed-off-by: Enrico Scholz 
---
 .../nodejs/nodejs-oe-cache-16.14/oe-npm-cache | 77 +++
 .../nodejs/nodejs-oe-cache-native_16.14.bb| 21 +
 2 files changed, 98 insertions(+)
 create mode 100755 
meta-oe/recipes-devtools/nodejs/nodejs-oe-cache-16.14/oe-npm-cache
 create mode 100644 
meta-oe/recipes-devtools/nodejs/nodejs-oe-cache-native_16.14.bb

diff --git a/meta-oe/recipes-devtools/nodejs/nodejs-oe-cache-16.14/oe-npm-cache 
b/meta-oe/recipes-devtools/nodejs/nodejs-oe-cache-16.14/oe-npm-cache
new file mode 100755
index 0..f59620764
--- /dev/null
+++ b/meta-oe/recipes-devtools/nodejs/nodejs-oe-cache-16.14/oe-npm-cache
@@ -0,0 +1,77 @@
+#!/usr/bin/env node
+
+/// Usage: oe-npm-cache
+/// ... meta - metainformation about package
+///   tgz  - tarball
+
+const process = require("node:process");
+
+module.paths.unshift("@@libdir@@/node_modules/npm/node_modules");
+
+const cacache = require('cacache')
+const fs = require('fs')
+
+// argv[0] is 'node', argv[1] is this script
+const cache_dir = process.argv[2]
+const type  = process.argv[3]
+const key   = process.argv[4]
+const file  = process.argv[5]
+
+const data = fs.readFileSync(file)
+
+// metadata content is highly nodejs dependent; when cache entries are not
+// found, place debug statements in 'make-fetch-happen/lib/cache/policy.js'
+// (CachePolicy::satisfies())
+const xlate = {
+'meta': {
+   'key_prefix': 'make-fetch-happen:request-cache:',
+   'metadata': function() {
+   return {
+   time: Date.now(),
+   url:  key,
+   reqHeaders: {
+   'accept': 'application/vnd.npm.install-v1+json; q=1.0, 
application/json; q=0.8, */*',
+   },
+   resHeaders: {
+   "content-type": "application/json",
+   "status": 200,
+   },
+   options: {
+   compress: true,
+   }
+   };
+   },
+},
+
+'tgz': {
+   'key_prefix': 'make-fetch-happen:request-cache:',
+   'metadata': function() {
+   return {
+   time: Date.now(),
+   url:  key,
+   reqHeaders: {
+   'accept': '*/*',
+   },
+   resHeaders: {
+   "content-type": "application/octet-stream",
+   "status": 200,
+   },
+   options: {
+   compress: true,
+   },
+   };
+   },
+},
+};
+
+const info = xlate[type];
+let opts = {}
+
+if (info.metadata) {
+opts['metadata'] = info.metadata();
+}
+
+cacache.put(cache_dir, info.key_prefix + key, data, opts)
+.then(integrity => {
+   console.log(`Saved content of ${key} (${file}).`);
+})
diff --git a/meta-oe/recipes-devtools/nodejs/nodejs-oe-cache-native_16.14.bb 
b/meta-oe/recipes-devtools/nodejs/nodejs-oe-cache-native_16.14.bb
new file mode 100644
index 0..a61dd5018
--- /dev/null
+++ b/meta-oe/recipes-devtools/nodejs/nodejs-oe-cache-native_16.14.bb
@@ -0,0 +1,21 @@
+DESCRIPTION = "OE helper for manipulating npm cache"
+LICENSE = "Apache-2.0"
+LIC_FILES_CHKSUM = 
"file://${COMMON_LICENSE_DIR}/Apache-2.0;md5=89aea4e17d99a7cacdbeed46a0096b10"
+
+SRC_URI = "\
+file://oe-npm-cache \
+"
+
+inherit native
+
+B = "${WORKDIR}/build"
+
+do_configure() {
+sed -e 's!@@libdir@@!${libdir}!g' < '${WORKDIR}/oe-npm-cache' > 
'${B}/oe-npm-cache'
+}
+
+do_install() {
+install -D -p -m 0755 ${B}/oe-npm-cache ${D}${bindir}/oe-npm-cache
+}
+
+RDEPENDS:${PN} = "nodejs-native"
-- 
2.36.1


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



Re: [oe] [OE-core] Inclusive Language Proposal for YP/OE

2022-02-04 Thread Enrico Scholz via lists.openembedded.org
Alexander Kanavin  writes:

>> > Would CANCEL be clearer to you than HALT?
>>
>> mmmh for me as a developer (and non-native english speaker), "cancel"
>> means some ordered ending of an operation.
>>
>> But the condition above causes an emergency abort.
>>
>
> Cancel is the same as abort: a request to immediately stop the operation
> (or not even start it) without reaching the originally requested
> outcome.

https://english.stackexchange.com/questions/535153/what-is-the-difference-between-cancel-and-abort

| Cancel implies the action is rescinded before it implements...
|
| Abort is an emergency procedure to interrupt...


> Halt implies the operation is not going to continue

Really?

https://translate.google.com/?sl=en=de=halt=translate says

| a suspension of movement or activity, typically a temporary one.

Examples in

  https://dictionary.cambridge.org/de/worterbuch/englisch/halt

sound like there is a chance to continue after the "halt" (show the
permit, resolve the pay dispute).


When we want to continue this inclusion BS, then "halt" seems to be
offending to some people because it can mean

| ... having a physical disability.



Enrico

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



Re: [oe] [OE-core] Inclusive Language Proposal for YP/OE

2022-02-04 Thread Enrico Scholz via lists.openembedded.org
Bryan Evenson  writes:

>> >> > For BB_DISKMON_DIRS, the actions "ABORT, STOPTASKS and WARN"
>> >> > would become "HALT, NO_NEW_TASKS and "WARN".
>> >>
>> >> I am not an native english speaker, but for "HALT" I will have to
>> >> think twice whether it will pause the operation or abort it.  I would
>> >> stay at "ABORT" because it makes much more clear what happens.
>
> Would CANCEL be clearer to you than HALT?

mmmh for me as a developer (and non-native english speaker), "cancel"
means some ordered ending of an operation.

But the condition above causes an emergency abort.

I do not know how this can be described without using "abort" or some
extensively long terms.


>> >> > BB_HASHCONFIG_WHITELIST -> BB_HASHCONFIG_IGNORE_VARS
>> >> > BB_SETSCENE_ENFORCE_WHITELIST -> BB_SETSCENE_ENFORCE_IGNORE_TASKS
>> >> > BB_HASHBASE_WHITELIST -> BB_BASEHASH_IGNORE_VARS
>> >> > MULTI_PROVIDER_WHITELIST -> BB_MULTI_PROVIDER_ALLOWED
>> >>
>> >> The new variable names sound like boolean flags, not like lists.
>
> Would BB_HASHCONFIG_IGNORELIST or BB_HASHCONFIG_ALLOWEDLIST make more
> sense to you?

yes; it is much better.  But should it be an "IGNORELIST" or an
"IGNORE*D*LIST"?



Enrico
[who is irritated how people and especially developer can waste their
(and other developers) time in trying to change something which was
completely fine before]

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



Re: [oe] [OE-core] Inclusive Language Proposal for YP/OE

2022-02-04 Thread Enrico Scholz via lists.openembedded.org
Alexander Kanavin  writes:

>> > For BB_DISKMON_DIRS, the actions "ABORT, STOPTASKS and WARN" would
>> > become "HALT, NO_NEW_TASKS and "WARN".
>>
>> I am not an native english speaker, but for "HALT" I will have to
>> think twice whether it will pause the operation or abort it.  I would
>> stay at "ABORT" because it makes much more clear what happens.
>
> I'm not taking a stand here, but just providing the context for where
> the push for avoidance of 'abort' comes from:
> https://inclusivenaming.org/word-lists/tier-1/
>
> Keep in mind that for non-native english speakers none of these words
> are emotionally charged; they're just terms.

Yes; these are terms.  But it should be possible to understand the
meaning of these terms without using a special "inclusivenaming"
dictionary.  Else, we could just use "A", "B" and "C" for the actions
above and explain these "terms" in some documentation later.


>> > BB_HASHCONFIG_WHITELIST -> BB_HASHCONFIG_IGNORE_VARS
>> > BB_SETSCENE_ENFORCE_WHITELIST -> BB_SETSCENE_ENFORCE_IGNORE_TASKS
>> > BB_HASHBASE_WHITELIST -> BB_BASEHASH_IGNORE_VARS
>> > MULTI_PROVIDER_WHITELIST -> BB_MULTI_PROVIDER_ALLOWED
>>
>> The new variable names sound like boolean flags, not like lists.
>
> I'd say the context should make it clear that they are lists

It is bad when a context is required to understand the meaning of a
variable.

And where do you see a context when local.conf contains

| BB_HASHCONFIG_IGNORE_VARS = "true"



Enrico

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



Re: [oe] [OE-core] Inclusive Language Proposal for YP/OE

2022-02-04 Thread Enrico Scholz via lists.openembedded.org
"Jon Mason"  writes:

> For BB_DISKMON_DIRS, the actions "ABORT, STOPTASKS and WARN" would
> become "HALT, NO_NEW_TASKS and "WARN".

I am not an native english speaker, but for "HALT" I will have to think
twice whether it will pause the operation or abort it.  I would stay at
"ABORT" because it makes much more clear what happens.


> BB_HASHCONFIG_WHITELIST -> BB_HASHCONFIG_IGNORE_VARS
> BB_SETSCENE_ENFORCE_WHITELIST -> BB_SETSCENE_ENFORCE_IGNORE_TASKS
> BB_HASHBASE_WHITELIST -> BB_BASEHASH_IGNORE_VARS
> MULTI_PROVIDER_WHITELIST -> BB_MULTI_PROVIDER_ALLOWED

The new variable names sound like boolean flags, not like lists.


> SSTATE_DUPWHITELIST -> SSTATE_ALLOW_OVERLAP_FILES
> CVE_CHECK_PN_WHITELIST -> CVE_CHECK_SKIPRECIPE
> CVE_CHECK_WHITELIST -> CVE_CHECK_IGNORECVE
> SYSROOT_DIRS_BLACKLIST -> SYSROOT_DIRS_IGNORE
> LICENSE_FLAGS_WHITELIST -> LICENSE_FLAGS_ACCEPTED
> UNKNOWN_CONFIGURE_WHITELIST -> UNKNOWN_CONFIGURE_OPT_IGNORE
> SDK_LOCAL_CONF_BLACKLIST -> ESDK_LOCALCONF_REMOVE
> SDK_LOCAL_CONF_WHITELIST -> ESDK_LOCALCONF_ALLOW
> SDK_INHERIT_BLACKLIST -> ESDK_CLASS_INHERIT_DISABLE

ditto; sounds like flags but not like lists


Enrico

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



Re: [oe] [PATCH 02/14] autofs: add bash to RDEPENDS_autofs

2014-09-11 Thread Enrico Scholz
Robert Yang liezhi.y...@windriver.com writes:

 possible bashism in autofs/etc/init.d/autofs line 116 ($foo should be 
 eval_gettext foo):

 +RDEPENDS_${PN} += bash

Is this bashism only in the initscript, or somewhere else too?  If it is
only in the initscript, the RDEPENDS should be a RRECOMMENDS because
package seems to contain also a systemd service.


Enrico
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-qt5] fix do_rootfs for images with dev-pkgs in IMAGE_FEATURES

2013-09-26 Thread Enrico Scholz
Burton, Ross ross.bur...@intel.com writes:

 I ran into this problem the other day, and after speaking to Ross
 Burton, he suggested that the better way to fix this is with:

 RDEPENDS_${PN}-dev = 

 I'm starting to get the feeling that the dev-pkgs feature is also a
 little broken, as it seems as though it doesn't handle cases where
 sub-packages are installed, but the main package is not.

 Well, the problem is the default dependency on the main package in the
 -dev package, which happens whether or not the main package even
 exists.  That's why I prefer removing the dependency instead of
 creating empty packages just for the bad dependency.

ack.  This causes also problems with populate_sdk as it can add
conflicting packages to the package set.  E.g. when having 'dropbear'
and 'openssh-sftp-server' in the image, the -dev RDEPENDS tries to add
the conflicting 'dropbear' and 'openssh' packages to the sdk.

Instead of (or better: additional to) changing 'RDEPENDS_${PN}-dev', I
would prefer the removal of 'ALLOW_EMPTY_${PN}-dev' from bitbake.conf.
There is not reason to create -dev packages for applications.

In 'meta/conf/bitbake.conf' the '${PN}' in

|:RDEPENDS_${PN}-dev = ${PN} (= ${EXTENDPKGV})

should be replaced by something which means the library package(s).


Enrico
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [PATCH] [meta-systemd] dropbear: read configuration from /etc/default/dropbear

2013-01-22 Thread Enrico Scholz
Patch allows to specify additional parameters (e.g. '-B') by reading
/etc/default/dropbear as a systemd environment file.  It applies
$DROPBEAR_EXTRA_ARGS which used already by the oe-core sysv initscript.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 meta-systemd/meta-oe/recipes-core/dropbear/dropbear/dropbear@.service | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/meta-systemd/meta-oe/recipes-core/dropbear/dropbear/dropbear@.service 
b/meta-systemd/meta-oe/recipes-core/dropbear/dropbear/dropbear@.service
index 4fba3f0..813788f 100644
--- a/meta-systemd/meta-oe/recipes-core/dropbear/dropbear/dropbear@.service
+++ b/meta-systemd/meta-oe/recipes-core/dropbear/dropbear/dropbear@.service
@@ -4,6 +4,7 @@ Requires=dropbearkey.service
 After=syslog.target dropbearkey.service
 
 [Service]
-ExecStart=-/usr/sbin/dropbear -i -r /etc/dropbear/dropbear_rsa_host_key -p 22
+EnvironmentFile=-/etc/default/dropbear
+ExecStart=-/usr/sbin/dropbear -i -r /etc/dropbear/dropbear_rsa_host_key -p 22 
$DROPBEAR_EXTRA_ARGS
 ExecReload=/bin/kill -HUP $MAINPID
 StandardInput=socket
-- 
1.7.11.7


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


Re: [oe] Khem Raj : dhcp: Add systemd unit file for dhclient

2013-01-11 Thread Enrico Scholz
Samuel Stirtzel s.stirtzel-gM/ye1e23mwn+bqq9rb...@public.gmane.org
writes:

 +ExecStart=/sbin/dhclient -cf /etc/dhcp/dhclient.conf -q $INTERFACES -lf 
 /var/lib/dhcp/dhclient.leases
 ...
 systemd[1]: PID file /var/run/dhclient.pid not readable (yet?) after start.
 [FAILED] Failed to start Dynamic Host Configuration Protocol (DHCP).
 See 'systemctl status dhclient.service' for details.


 Any idea why it is missing?

probably a '--no-pid' option


Enrico

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


[oe] [PATCH] meta-systemd: select recipes based on active collections

2012-11-26 Thread Enrico Scholz
Recipes of inactivate layers must been BBMASK'ed to prevent build errors.
This is very unflexible and causes high maintenance effort for keeping
the project configuration in sync with the actual meta-systemd layout.

With this patch, the .bbappend files within the meta-* directories are
ignored unless the corresponding collection has been selected.  As it
relies on the order of BBFILES and BBFILE_COLLECTIONS expansion, a
sanity check has been added which aborts the build when assumptions do
not apply anymore.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 meta-systemd/conf/layer.conf | 16 +++-
 meta-systemd/invalid.bb  |  2 ++
 2 files changed, 17 insertions(+), 1 deletion(-)
 create mode 100644 meta-systemd/invalid.bb

diff --git a/meta-systemd/conf/layer.conf b/meta-systemd/conf/layer.conf
index 3310abc..e8fd5ca 100644
--- a/meta-systemd/conf/layer.conf
+++ b/meta-systemd/conf/layer.conf
@@ -12,7 +12,21 @@
 BBPATH .= :${LAYERDIR}
 
 # We have a recipes directory, add to BBFILES
-BBFILES += ${LAYERDIR}/recipes-*/*/*.bb ${LAYERDIR}/*/recipes-*/*/*.bbappend
+BBFILES += \
+  ${LAYERDIR}/recipes-*/*/*.bb \
+  ${LAYERDIR}/oe-core/recipes-*/*.bbappend \
+  ${LAYERDIR}/meta-oe/recipes-*/*/*.bbappend \
+  ${@base_contains('BBFILE_COLLECTIONS', 'systemd-layer',\
+   '', '${LAYERDIR}/invalid.bb', d)} \
+  ${@base_contains('BBFILE_COLLECTIONS', 'gnome-layer',\
+   '${LAYERDIR}/meta-gnome/recipes-*/*/*.bbappend', '', d)} \
+  ${@base_contains('BBFILE_COLLECTIONS', 'efl-layer',\
+   '${LAYERDIR}/meta-efl/recipes-*/*/*.bbappend', '', d)} \
+  ${@base_contains('BBFILE_COLLECTIONS', 'multimedia-layer',\
+   '${LAYERDIR}/meta-multimedia/recipes-*/*/*.bbappend', '', 
d)} \
+  ${@base_contains('BBFILE_COLLECTIONS', 'networking',\
+   '${LAYERDIR}/meta-networking/recipes-*/*/*.bbappend', '', 
d)} \
+
 
 BBFILE_COLLECTIONS += systemd-layer
 BBFILE_PATTERN_systemd-layer := ^${LAYERDIR}/
diff --git a/meta-systemd/invalid.bb b/meta-systemd/invalid.bb
new file mode 100644
index 000..b16787b
--- /dev/null
+++ b/meta-systemd/invalid.bb
@@ -0,0 +1,2 @@
+Content of this file is invalid and is used by conf/layer.conf to
+ensure that BBFILES is expanded after BBFILE_COLLECTIONS.
-- 
1.7.11.7


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


[oe] [PATCH] systemd.bbclass: move disable operation into prerm() script

2012-11-16 Thread Enrico Scholz
The 'systemctl disable' operation should not be done in postrm() because
the .service file does not exist anymore and the script will fail hence.

After moving this command into prerm(), postrm() becomes empty and can
be removed completely.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 meta-systemd/classes/systemd.bbclass | 8 
 1 file changed, 8 deletions(-)

diff --git a/meta-systemd/classes/systemd.bbclass 
b/meta-systemd/classes/systemd.bbclass
index f26c6d1..e1916a0 100644
--- a/meta-systemd/classes/systemd.bbclass
+++ b/meta-systemd/classes/systemd.bbclass
@@ -22,9 +22,7 @@ systemd_prerm() {
 if [ -z $D ]; then
 systemctl stop ${SYSTEMD_SERVICE}
 fi
-}
 
-systemd_postrm() {
 systemctl disable ${SYSTEMD_SERVICE}
 }
 
@@ -116,12 +114,6 @@ python populate_packages_prepend () {
 prerm += localdata.getVar('systemd_prerm', 1)
 d.setVar('pkg_prerm_%s' % pkg, prerm)
 
-postrm = localdata.getVar('pkg_postrm', 1)
-if not postrm:
-postrm = '#!/bin/sh\n'
-postrm += localdata.getVar('systemd_postrm', 1)
-d.setVar('pkg_postrm_%s' % pkg, postrm)
-
 # add files to FILES_*-systemd if existent and not already done
 def systemd_append_file(pkg_systemd, file_append):
 appended = False
-- 
1.7.11.7


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


Re: [oe] [PATCH] gitpkgv.bbclass: cache GITPKGV result

2012-11-16 Thread Enrico Scholz
Otavio Salvador ota...@ossystems.com.br writes:

 This patch caches the result of 'git rev-list' into the sources cache
 directory and uses it on the next run.  Because collisions of the sha1
 hash are very unlikely, the git revision is used directly as the key.
 ...
 I'd prefer to have the hashes cache in same DL_DIR so we don't have another
 directory there. It is a file so I see no need for another dir. What do you
 think?

good idea; I will submit an updated patch soon.


Enrico

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


[oe] [PATCH v2] gitpkgv.bbclass: cache GITPKGV result

2012-11-16 Thread Enrico Scholz
gitpkgv runs the 'git rev-list | wc -l' several times when processing a
package using GITPKGV.  This takes ages for packages like the linux
kernel which has a) a large repository and b) lots of subpackages.

This patch caches the result of 'git rev-list' and uses it on the next
run.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 meta-oe/classes/gitpkgv.bbclass | 30 --
 1 file changed, 24 insertions(+), 6 deletions(-)

diff --git a/meta-oe/classes/gitpkgv.bbclass b/meta-oe/classes/gitpkgv.bbclass
index 165ba1e..7bdc538 100644
--- a/meta-oe/classes/gitpkgv.bbclass
+++ b/meta-oe/classes/gitpkgv.bbclass
@@ -50,6 +50,7 @@ def gitpkgv_drop_tag_prefix(version):
 def get_git_pkgv(d, use_tags):
 import os
 import bb
+from pipes import quote
 
 src_uri = d.getVar('SRC_URI', 1).split()
 fetcher = bb.fetch2.Fetch(src_uri, d)
@@ -71,22 +72,39 @@ def get_git_pkgv(d, use_tags):
 
 found = True
 
-cwd = os.getcwd()
-os.chdir(url.localpath)
+vars = { 'repodir' : quote(url.localpath),
+ 'rev' : quote(rev) }
 
-commits = bb.fetch2.runfetchcmd(git rev-list %s -- 2 
/dev/null | wc -l % rev, d, quiet=True).strip()
+rev = bb.fetch2.get_srcrev(d).split('+')[1]
+rev_file = os.path.join(url.localpath, oe-gitpkgv_ + rev)
+
+if not os.path.exists(rev_file) or 
os.path.getsize(rev_file)==0:
+commits = bb.fetch2.runfetchcmd(
+cd %(repodir)s  
+git rev-list %(rev)s -- 2 /dev/null 
+| wc -l % vars,
+d, quiet=True).strip().lstrip('0')
+
+if commits != :
+oe.path.remove(rev_file, recurse=False)
+open(rev_file, w).write(%d\n % int(commits))
+else:
+commits = 0
+else:
+commits = open(rev_file, r).readline(128).strip()
 
 if use_tags:
 try:
-output = bb.fetch2.runfetchcmd(git describe %s 
2/dev/null % rev, d, quiet=True).strip()
+output = bb.fetch2.runfetchcmd(
+cd %(repodir)s  
+git describe %(rev)s 2/dev/null % vars,
+d, quiet=True).strip()
 ver = gitpkgv_drop_tag_prefix(output)
 except Exception:
 ver = 0.0-%s-g%s % (commits, rev[:7])
 else:
 ver = %s+%s % (commits, rev[:7])
 
-os.chdir(cwd)
-
 format = format.replace(name, ver)
 
 if found:
-- 
1.7.11.7


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


[oe] [PATCH] libtool: fixed parallel build related race

2012-05-20 Thread Enrico Scholz
While building libtool, the libtool script itself will be regenerated
because OE modifies a dependency[1]. With -jX, this operation (--
removal, creation of non-x file, 'chmod a+x') can happen at a time when
the script is going to be executed.  This can cause errors like:

| arm-linux-gnueabi-libtool: compile:  ccache arm-linux-gnueabi-gcc ...
| ...
| /bin/sh ./config.status libtool
| ...
| arm-linux-gnueabi-libtool: compile:  ccache arm-linux-gnueabi-gcc ...
| /bin/sh: ./arm-linux-gnueabi-libtool: Permission denied
| make[2]: *** [libltdl/libltdl_libltdl_la-lt__alloc.lo] Error 126

I am not sure whether the custom do_compile_prepend() is still needed.
For now only the issue above will be fixed by executing ./config.status
yet again.

[1] see 648290d5bf4d6ff50d3643bb7ad902dfc23aa702

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 meta/recipes-devtools/libtool/libtool-2.4.2.inc |3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/meta/recipes-devtools/libtool/libtool-2.4.2.inc 
b/meta/recipes-devtools/libtool/libtool-2.4.2.inc
index 084062a..c1ef7c4 100644
--- a/meta/recipes-devtools/libtool/libtool-2.4.2.inc
+++ b/meta/recipes-devtools/libtool/libtool-2.4.2.inc
@@ -8,7 +8,7 @@ LICENSE = GPLv2  LGPLv2.1
 LIC_FILES_CHKSUM = file://COPYING;md5=751419260aa954499f7abaabaa882bbe \
 file://libltdl/COPYING.LIB;md5=e3eda01d9815f8d24aae2dbd89b68b06
 
-INC_PR = r2
+INC_PR = r3
 
 SRC_URI = ${GNU_MIRROR}/libtool/libtool-${PV}.tar.gz \
file://trailingslash.patch \
@@ -27,6 +27,7 @@ do_compile_prepend () {
# Sometimes this file doesn't get rebuilt, force the issue
rm -f ${S}/libltdl/config/ltmain.sh
make libltdl/config/ltmain.sh
+   ./config.status
 }
 
 inherit autotools
-- 
1.7.7.6


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


Re: [oe] x86_64

2011-05-11 Thread Enrico Scholz
Phil Blundell ph...@gnu.org writes:

 Changing the architecture to x86-64 would be a possibility, 

no; will cause too much pain because all the ./config.guess scripts
expect 'x86_64'.

I kept TARGET_ARCH='x86_64' but added an additional OVERRIDES where '_'
is replaced by '-'.


Enrico

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


Re: [oe] x86_64

2011-05-10 Thread Enrico Scholz
Jaap de Jong jaap.dej...@nedap.com writes:

 TARGET_ARCH   = x86_64

did this ever work?  OE uses hardcoded 'lib' (instead of 'lib64') at
various places.  At least gcc has

| #define GLIBC_DYNAMIC_LINKER64 /lib64/ld-linux-x86-64.so.2

which causes 64 bit binaries to look for the interpreter at /lib64.

I recall issues with openssl too which autodetects lib64 directories but
OE packages lib only.



Enrico

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


Re: [oe] x86_64

2011-05-10 Thread Enrico Scholz
Phil Blundell ph...@gnu.org writes:

 x86_64 seems to be what's currently used as the architecture name.  If
 you want a value for ${MACHINE} then I guess something like
 x86_64-generic would do.  I don't think there's any prohibition
 against underscores in machine names.

OVERRIDES don't work correctly with this. E.g. (bitbake 1.10) a recipe
with

| xx = default
| xx_x86_64 = XX

results into

| OVERRIDES=...:x86_64:...
| # xx_x86_64=XX
| xx_x86_64=XX
| # xx=default
| xx=default


Adding an additional x86-64 to OVERRIDES in the machine configuration
and using it by

| xx_x86-64 = XX

gives the expected

| # xx_x86-64=XX
| xx_x86-64=XX
| # xx=XX
| xx=XX



Enrico

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


Re: [oe] x86_64

2011-05-10 Thread Enrico Scholz
Michael Smith msm...@cbnco.com writes:

 did this ever work?  OE uses hardcoded 'lib' (instead of 'lib64') at
 various places.

 This is my MACHINE config:
 ...
 # Some packages install files into /lib/packagename, not /lib64/packagename.
 FILES_${PN} += ${prefix}/lib/${PN}/*
 FILES_${PN}-dbg += ${prefix}/lib/${PN}/.debug

I think, this will break/change some packages...


After creating (manually) lib64 - lib symlinks in the sysroot, I managed
to build a non trivial 64-bit system (e.g. with vdr and perl stuff) with
(nearly) standard settings (2011.03-maintenance).  But necessary tricks
like the BUILD_VENDOR and some minor fixes (like these in openssl) caused
me to see this as very experimental.


 And gcc-cross:
 EXTRA_OECONF_append_x86_64 =  --disable-multilib

should not be needed anymore; flag is set by
847c0dadb9f55af5cc4556486cde3d9dcc2129af


Enrico

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


[oe] [PATCH 1/2] bitbake.conf: use HOST_CC_ARCH in CPP

2011-04-26 Thread Enrico Scholz
Using plain 'gcc -E' for CPP is wrong because some macros are defined
only for certain ${HOST_CC_ARCH} flags.  Checks using ${CPP} might
return wrong results hence.

Omitting ${HOST_CC_ARCH} from 'gcc -E' can also break builds for
compilers configured e.g. for '-mcpu=cortex-m3'.  Such gccs require
'-mthumb'; even when invoked with '-E':

| $ arm-linux-uclibceabi-gcc -E -  /dev/null
| # 1 stdin
| stdin:1:0: error: target CPU does not support ARM mode
| $

| $ arm-linux-uclibceabi-gcc -mthumb -E -  /dev/null
| # 1 stdin
| # 1 built-in
| # 1 command-line
| # 1 stdin
| $

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 conf/bitbake.conf |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/conf/bitbake.conf b/conf/bitbake.conf
index 17a494d..5fb90e2 100644
--- a/conf/bitbake.conf
+++ b/conf/bitbake.conf
@@ -473,7 +473,7 @@ TOOLCHAIN_SYSPATH ?= ${TOOLCHAIN_PATH}/${TARGET_SYS}
 export CC = ${CCACHE}${HOST_PREFIX}gcc ${HOST_CC_ARCH}${TOOLCHAIN_OPTIONS}
 export CXX = ${CCACHE}${HOST_PREFIX}g++ ${HOST_CC_ARCH}${TOOLCHAIN_OPTIONS}
 export F77 = ${CCACHE}${HOST_PREFIX}g77 ${HOST_CC_ARCH}${TOOLCHAIN_OPTIONS}
-export CPP = ${HOST_PREFIX}gcc -E${TOOLCHAIN_OPTIONS}
+export CPP = ${HOST_PREFIX}gcc -E ${HOST_CC_ARCH}${TOOLCHAIN_OPTIONS}
 export LD = ${HOST_PREFIX}ld${TOOLCHAIN_OPTIONS}
 export CCLD = ${CC}
 export AR = ${HOST_PREFIX}ar
@@ -488,7 +488,7 @@ PYTHON = ${@sys.executable}
 export BUILD_CC = ${CCACHE}${BUILD_PREFIX}gcc ${BUILD_CC_ARCH}
 export BUILD_CXX = ${CCACHE}${BUILD_PREFIX}g++ ${BUILD_CC_ARCH}
 export BUILD_F77 = ${CCACHE}${BUILD_PREFIX}g77 ${BUILD_CC_ARCH}
-export BUILD_CPP = ${BUILD_PREFIX}cpp
+export BUILD_CPP = ${BUILD_PREFIX}cpp ${BUILD_CC_ARCH}
 export BUILD_LD = ${BUILD_PREFIX}ld
 export BUILD_CCLD = ${BUILD_PREFIX}gcc
 export BUILD_AR = ${BUILD_PREFIX}ar
-- 
1.7.4.4


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


[oe] [PATCH 2/2] *native.bbclass: use HOST_CC_ARCH in CPP

2011-04-26 Thread Enrico Scholz
Using plain 'gcc -E' for CPP is wrong because some macros are defined
only for certain ${HOST_CC_ARCH} flags.  Checks using ${CPP} might
return wrong results hence.

Omitting ${HOST_CC_ARCH} from 'gcc -E' can also break builds for
compilers configured e.g. for '-mcpu=cortex-m3'.  Such gccs require
'-mthumb'; even when invoked with '-E':

| $ arm-linux-uclibceabi-gcc -E -  /dev/null
| # 1 stdin
| stdin:1:0: error: target CPU does not support ARM mode
| $

| $ arm-linux-uclibceabi-gcc -mthumb -E -  /dev/null
| # 1 stdin
| # 1 built-in
| # 1 command-line
| # 1 stdin
| $

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 classes/canadian-native.bbclass |2 +-
 classes/native.bbclass  |2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/classes/canadian-native.bbclass b/classes/canadian-native.bbclass
index 3cd003c..50ecc11 100644
--- a/classes/canadian-native.bbclass
+++ b/classes/canadian-native.bbclass
@@ -37,7 +37,7 @@ LDFLAGS = ${SDK_LDFLAGS}
 export CC = ${CCACHE}${SDK_PREFIX}gcc ${HOST_CC_ARCH}
 export CXX = ${CCACHE}${SDK_PREFIX}g++ ${HOST_CC_ARCH}
 export F77 = ${CCACHE}${SDK_PREFIX}g77 ${HOST_CC_ARCH}
-export CPP = ${SDK_PREFIX}gcc -E
+export CPP = ${SDK_PREFIX}gcc -E ${HOST_CC_ARCH}
 export LD = ${SDK_PREFIX}ld
 #export CCLD = ${CC}
 export AR = ${SDK_PREFIX}ar
diff --git a/classes/native.bbclass b/classes/native.bbclass
index 1e7a6ec..0f65792 100644
--- a/classes/native.bbclass
+++ b/classes/native.bbclass
@@ -54,7 +54,7 @@ export CONFIG_SITE = 
 export CC = ${CCACHE}${HOST_PREFIX}gcc ${HOST_CC_ARCH}
 export CXX = ${CCACHE}${HOST_PREFIX}g++ ${HOST_CC_ARCH}
 export F77 = ${CCACHE}${HOST_PREFIX}g77 ${HOST_CC_ARCH}
-export CPP = ${HOST_PREFIX}gcc -E
+export CPP = ${HOST_PREFIX}gcc -E ${HOST_CC_ARCH}
 export LD = ${HOST_PREFIX}ld
 export CCLD = ${CC}
 export AR = ${HOST_PREFIX}ar
-- 
1.7.4.4


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


[oe] [PATCH] openssl: enforce libdir=lib

2011-04-26 Thread Enrico Scholz
From: Enrico Scholz enrico.sch...@informatik.tu-chemnitz.de

openssl tries to detect multlib environments and assigns libdir=lib64
e.g. for x86_64 targets.  This breaks OE assumptions about the location
of pkgconfig files and causes build failures.

As OE is not multilib aware, use always a static lib value for libdir.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/openssl/openssl.inc |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/recipes/openssl/openssl.inc b/recipes/openssl/openssl.inc
index 24b517d..a2eaaa3 100644
--- a/recipes/openssl/openssl.inc
+++ b/recipes/openssl/openssl.inc
@@ -97,7 +97,7 @@ do_configure () {
 if [ x$useprefix = x ]; then
 useprefix=/
 fi
-   perl ./Configure ${EXTRA_OECONF} shared --prefix=$useprefix 
--openssldir=${libdir}/ssl $target
+   perl ./Configure ${EXTRA_OECONF} shared --prefix=$useprefix 
--libdir=lib --openssldir=${libdir}/ssl $target
 
eval ${@base_contains('DISTRO_FEATURES', 'largefile', '', 'sed -i -e 
/_FILE_OFFSET_BITS/,/#endif/d ${S}/crypto/bio/bss_file.c', d)}
eval ${@base_contains('DISTRO_FEATURES', 'ipv6', '', 'sed -i -e 
/AF_INET6/,/break/d ${S}/crypto/bio/bss_dgram.c', d)}
-- 
1.7.4.4


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


[oe] [PATCH] tcp-wrappers: place LIBS after object files

2011-04-26 Thread Enrico Scholz
From: Enrico Scholz enrico.sch...@informatik.tu-chemnitz.de

This fixes builds with '-Wl,-as-needed'. Patch changes the previous
'ldflags.patch' files instead of adding a new one.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
Cc: Jeremy Lainé jeremy.la...@m4x.org
---
 .../tcp-wrappers/tcp-wrappers-7.6/ldflags.patch|   22 +++
 1 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/recipes/tcp-wrappers/tcp-wrappers-7.6/ldflags.patch 
b/recipes/tcp-wrappers/tcp-wrappers-7.6/ldflags.patch
index 2e89765..cac8615 100644
--- a/recipes/tcp-wrappers/tcp-wrappers-7.6/ldflags.patch
+++ b/recipes/tcp-wrappers/tcp-wrappers-7.6/ldflags.patch
@@ -1,13 +1,25 @@
-Index: tcp_wrappers_7.6.orig/Makefile
+Index: tcp_wrappers_7.6/Makefile
 ===
 tcp_wrappers_7.6.orig.orig/Makefile2009-04-06 10:55:47.0 
+
-+++ tcp_wrappers_7.6.orig/Makefile 2009-04-06 10:57:04.0 +
-@@ -748,31 +748,31 @@
+--- tcp_wrappers_7.6.orig/Makefile
 tcp_wrappers_7.6/Makefile
+@@ -724,9 +724,9 @@ SHLIBSOMAJ = shared/libwrap.so.$(SOMAJOR
+ SHLIBSO   = shared/libwrap.so
+ SHLIBFLAGS= -Lshared -lwrap
+ 
+-SHLINKFLAGS = -shared -Xlinker -soname -Xlinker libwrap.so.$(SOMAJOR) -lc 
$(LIBS)
++SHLINKFLAGS = -shared -Xlinker -soname -Xlinker libwrap.so.$(SOMAJOR)
+ SHCFLAGS = -fPIC -shared -D_REENTRANT
+-SHLIB_OBJ= $(addprefix shared/, $(LIB_OBJ));
++SHLIB_OBJ= $(addprefix shared/, $(LIB_OBJ))
+ 
+ all other: config-check tcpd tcpdmatch try-from safe_finger tcpdchk $(LIB)
+ 
+@@ -748,31 +748,31 @@ $(LIB):  $(LIB_OBJ)
  
  $(SHLIB): $(SHLIB_OBJ)
rm -f $(SHLIB)
 -  $(CC) -o $(SHLIB) $(SHLINKFLAGS) $(SHLIB_OBJ)
-+  $(CC) $(LDFLAGS) -o $(SHLIB) $(SHLINKFLAGS) $(SHLIB_OBJ)
++  $(CC) $(LDFLAGS) -o $(SHLIB) $(SHLINKFLAGS) $(SHLIB_OBJ) $(LIBS)
ln -sf $(notdir $(SHLIB)) $(SHLIBSOMAJ)
ln -sf $(notdir $(SHLIBSOMAJ)) $(SHLIBSO)
  
-- 
1.7.4.4


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


[oe] [PATCH] gitpkgv.bbclass: cache GITPKGV result

2011-04-26 Thread Enrico Scholz
gitpkgv runs the 'git rev-list | wc -l' several times when processing a
package using GITPKGV.  This takes ages for packages like the linux
kernel which has a) a large repository and b) lots of subpackages.

This patch caches the result of 'git rev-list' into the sources cache
directory and uses it on the next run.  Because collisions of the sha1
hash are very unlikely, the git revision is used directly as the key.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 classes/gitpkgv.bbclass |   29 +++--
 1 files changed, 23 insertions(+), 6 deletions(-)

diff --git a/classes/gitpkgv.bbclass b/classes/gitpkgv.bbclass
index bedceb9..76c660b 100644
--- a/classes/gitpkgv.bbclass
+++ b/classes/gitpkgv.bbclass
@@ -50,8 +50,11 @@ def gitpkgv_drop_tag_prefix(version):
 def get_git_pkgv(d, use_tags):
import os
import bb
+   from pipes import quote
 
urls = bb.data.getVar('SRC_URI', d, 1).split()
+   cachedir = bb.data.expand(${DL_DIR}/gitpkgv, d)
+   bb.mkdirhier(cachedir)
 
for url in urls:
(type, host, path, user, pswd, parm) = 
bb.decodeurl(bb.data.expand(url, d))
@@ -63,22 +66,36 @@ def get_git_pkgv(d, use_tags):
return None
 
rev = bb.fetch.get_srcrev(d).split('+')[1]
+   rev_file = os.path.join(cachedir, rev)
 
-   cwd = os.getcwd()
-   os.chdir(repodir)
+   vars = { 'repodir' : quote(repodir),
+'rev' : quote(rev),
+  }
 
-   commits = bb.fetch.runfetchcmd(git rev-list %s -- 2 
/dev/null | wc -l % rev, d, quiet=True).strip()
+   if not os.path.exists(rev_file) or 
os.path.getsize(rev_file)==0:
+   commits = bb.fetch.runfetchcmd(
+  cd %(repodir)s  
+  git rev-list %(rev)s -- 2 /dev/null | 
wc -l % vars,
+  d, quiet=True).strip()
+
+   if commits != 0:
+   oe.path.remove(rev_file, recurse=False)
+   open(rev_file, w).write(%d\n % 
commits)
+   else:
+   commits = open(rev_file, 
r).readline(128).strip()
 
if use_tags:
try:
-   ver = 
gitpkgv_drop_tag_prefix(bb.fetch.runfetchcmd(git describe %s 2/dev/null % 
rev, d, quiet=True).strip())
+   tmp = bb.fetch.runfetchcmd(
+   cd %(repodir)s  
+   git describe %(rev)s 2/dev/null % 
vars,
+   d, quiet=True).strip()
+   ver = gitpkgv_drop_tag_prefix(tmp)
except Exception:
ver = 0.0-%s-g%s % (commits, rev[:7])
else:
ver = %s+%s % (commits, rev[:7])
 
-   os.chdir(cwd)
-
return ver
 
return 0+0
-- 
1.7.4.4


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


[oe] [PATCH] dietlibc: rewrote recipes

2011-04-26 Thread Enrico Scholz
dietlibc was splitted into target and cross recipes.  These recipes
build now the latest stable version, from upstream CVS or from a github
repository with lot of fixes.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/dietlibc/dietlibc-0.32.inc |   24 ++
 recipes/dietlibc/dietlibc-0.32/ccache.patch|   74 ---
 recipes/dietlibc/dietlibc-0.32/diethome.patch  |   24 --
 recipes/dietlibc/dietlibc-0.32/getrlimit.patch |   12 ---
 recipes/dietlibc/dietlibc-common.inc   |   81 +
 recipes/dietlibc/dietlibc-cross.inc|   20 +
 recipes/dietlibc/dietlibc-cross_0.32.bb|2 +
 recipes/dietlibc/dietlibc-cross_cvs.bb |2 +
 recipes/dietlibc/dietlibc-cross_git.bb |2 +
 recipes/dietlibc/dietlibc-cvs.inc  |   27 +++
 recipes/dietlibc/dietlibc-git.inc  |   15 
 recipes/dietlibc/dietlibc.inc  |   79 
 recipes/dietlibc/dietlibc_0.31.bb  |   11 ---
 recipes/dietlibc/dietlibc_0.32.bb  |   16 ++---
 recipes/dietlibc/dietlibc_cvs.bb   |9 +++
 recipes/dietlibc/dietlibc_git.bb   |8 ++
 recipes/dietlibc/files/ai_addrconfig.patch |   13 
 recipes/dietlibc/files/ccache.patch|   36 +-
 recipes/dietlibc/files/getrlimit.patch |   12 +++
 recipes/dietlibc/files/runtests-X.sh   |   93 
 20 files changed, 367 insertions(+), 193 deletions(-)
 create mode 100644 recipes/dietlibc/dietlibc-0.32.inc
 delete mode 100644 recipes/dietlibc/dietlibc-0.32/ccache.patch
 delete mode 100644 recipes/dietlibc/dietlibc-0.32/diethome.patch
 delete mode 100644 recipes/dietlibc/dietlibc-0.32/getrlimit.patch
 create mode 100644 recipes/dietlibc/dietlibc-common.inc
 create mode 100644 recipes/dietlibc/dietlibc-cross.inc
 create mode 100644 recipes/dietlibc/dietlibc-cross_0.32.bb
 create mode 100644 recipes/dietlibc/dietlibc-cross_cvs.bb
 create mode 100644 recipes/dietlibc/dietlibc-cross_git.bb
 create mode 100644 recipes/dietlibc/dietlibc-cvs.inc
 create mode 100644 recipes/dietlibc/dietlibc-git.inc
 delete mode 100644 recipes/dietlibc/dietlibc_0.31.bb
 create mode 100644 recipes/dietlibc/dietlibc_cvs.bb
 create mode 100644 recipes/dietlibc/dietlibc_git.bb
 delete mode 100644 recipes/dietlibc/files/ai_addrconfig.patch
 create mode 100644 recipes/dietlibc/files/getrlimit.patch
 create mode 100644 recipes/dietlibc/files/runtests-X.sh

diff --git a/recipes/dietlibc/dietlibc-0.32.inc 
b/recipes/dietlibc/dietlibc-0.32.inc
new file mode 100644
index 000..df94324
--- /dev/null
+++ b/recipes/dietlibc/dietlibc-0.32.inc
@@ -0,0 +1,24 @@
+SRC_URI =  \
+http://www.kernel.org/pub/linux/libs/dietlibc/dietlibc-${PV}.tar.bz2 \
+file://ccache.patch \
+file://ceil.patch \
+file://getrlimit.patch \
+
+SRC_URI[md5sum] = 0098761c17924c15e21d25acdda4a8b5
+SRC_URI[sha256sum] = 
6613a2cae3b39e340779735d7500d284f5a691c5ed67e59a6057e0888726e458
+
+# SMP build is broken
+PARALLEL_MAKE = 
+
+# prefer the 'git' version which fixes lot of issues
+DEFAULT_PREFERENCE = -2
+
+# ARM EABI not fixable without complex patches
+DEFAULT_PREFERENCE_arm = -99
+
+do_configure_append () {
+install -d -m 0755 bin-${DIETLIBC_BUILD_ARCH}
+install -d -m 0755 bin-${DIETLIBC_TARGET_ARCH}
+
+sed -i -e 's!\$(CROSS)strip!$(STRIP)!g' Makefile
+}
diff --git a/recipes/dietlibc/dietlibc-0.32/ccache.patch 
b/recipes/dietlibc/dietlibc-0.32/ccache.patch
deleted file mode 100644
index fb067f0..000
--- a/recipes/dietlibc/dietlibc-0.32/ccache.patch
+++ /dev/null
@@ -1,74 +0,0 @@
-Index: dietlibc-0.32/diet.c
-===
 dietlibc-0.32.orig/diet.c  2009-03-17 00:17:58.0 +0100
-+++ dietlibc-0.32/diet.c   2009-10-02 14:12:14.306929201 +0200
-@@ -132,7 +132,12 @@
- }
-   }
-   {
--char *cc=argv[1];
-+char *cc;
-+if (!strcmp(argv[1],ccache)) {
-+  cc=argv[2];
-+} else {
-+  cc=argv[1];
-+}
- char *tmp=strchr(cc,0)-2;
- char *tmp2,*tmp3;
- if (tmpcc) goto donttouch;
-@@ -144,7 +149,7 @@
-   if (tmp3tmp2) tmp2=tmp3;
-   if (tmp2-cc90) error(platform name too long!\n);
-   shortplatform=platform+len;
--  memmove(shortplatform,argv[1],(size_t)(tmp2-cc));
-+  memmove(shortplatform,cc,(size_t)(tmp2-cc));
-   platform[tmp2-cc+len]=0;
-   if (shortplatform[0]=='i'  shortplatform[2]=='8'  
shortplatform[3]=='6') shortplatform[1]='3';
- } else {
-@@ -291,6 +296,9 @@
- 
-   dest=newargv;
-   *dest++=argv[1];
-+  if (strcmp(argv[1],ccache) == 0) {
-+   *dest++=argv[2];
-+  }
-   if (argv[2]) {
-   if (!strcmp(argv[2],-V)) {
- *dest++=argv[2];
-@@ -300,9 +308,9 @@
-   } else if (!memcmp(argv[2],-V,2)) {
- *dest++=argv[2];
- ++argv;
----argc

[oe] [PATCH] dietlibc.bbclass: require dietlibc-cross

2011-04-26 Thread Enrico Scholz
After recent dietlibc refactoring, the functionality of the old
'dietlibc' recipe is now provided by 'dietlibc-cross'.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 classes/dietlibc.bbclass |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/classes/dietlibc.bbclass b/classes/dietlibc.bbclass
index 2676385..588c085 100644
--- a/classes/dietlibc.bbclass
+++ b/classes/dietlibc.bbclass
@@ -1,4 +1,4 @@
-DEPENDS =+ dietlibc
+DEPENDS =+ dietlibc-cross
 
 def dietlibc_after_parse(d):
 import bb
-- 
1.7.4.4


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


Re: [oe] [PATCH] openssl: enforce libdir=lib

2011-04-26 Thread Enrico Scholz
Michael Smith msm...@cbnco.com writes:

 -perl ./Configure ${EXTRA_OECONF} shared --prefix=$useprefix 
 --openssldir=${libdir}/ssl $target
 +perl ./Configure ${EXTRA_OECONF} shared --prefix=$useprefix 
 --libdir=lib --openssldir=${libdir}/ssl $target

 Could you change lib to ${libdir}? For x86_64 builds my libdir is 
 /usr/lib64.

ok; I will use `basename ${libdir}`.

Just for interest: how did you configured the x86_64 target to use
lib64 directories?  By manually overriding all the *libdir variables in
bitbake.conf?  How well are recipes prepared to use ${*libdir} instead
of static /lib?


Enrico

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


[oe] [PATCH, v2] openssl: enforce libdir=lib

2011-04-26 Thread Enrico Scholz
From: Enrico Scholz enrico.sch...@informatik.tu-chemnitz.de

openssl tries to detect multlib environments and assigns libdir=lib64
e.g. for x86_64 targets.  This breaks OE assumptions about the location
of pkgconfig files and causes build failures.

Patch gives the directory name to openssl which is used as libdir by
OE.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/openssl/openssl.inc |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/recipes/openssl/openssl.inc b/recipes/openssl/openssl.inc
index 24b517d..b029f3b 100644
--- a/recipes/openssl/openssl.inc
+++ b/recipes/openssl/openssl.inc
@@ -97,7 +97,7 @@ do_configure () {
 if [ x$useprefix = x ]; then
 useprefix=/
 fi
-   perl ./Configure ${EXTRA_OECONF} shared --prefix=$useprefix 
--openssldir=${libdir}/ssl $target
+   perl ./Configure ${EXTRA_OECONF} shared --prefix=$useprefix 
--libdir=`basename ${libdir}` --openssldir=${libdir}/ssl $target
 
eval ${@base_contains('DISTRO_FEATURES', 'largefile', '', 'sed -i -e 
/_FILE_OFFSET_BITS/,/#endif/d ${S}/crypto/bio/bss_file.c', d)}
eval ${@base_contains('DISTRO_FEATURES', 'ipv6', '', 'sed -i -e 
/AF_INET6/,/break/d ${S}/crypto/bio/bss_dgram.c', d)}
-- 
1.7.4.4


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


Re: [oe] [PATCH] tcp-wrappers: place LIBS after object files

2011-04-26 Thread Enrico Scholz
Paul Menzel paulepan...@users.sourceforge.net writes:

 From: Enrico Scholz enrico.sch...@informatik.tu-chemnitz.de

 it seems you need to update your Git configuration to your common
 email address.

Both addresses are in use; some patches are created at home when I work
on a personal project (- tu-chemnitz.de address), some patches at work
(- sigma-chemnitz.de address).


 The diff could have been smaller, but no big problem.

how? three places must be changed (removal of $(LIBS) from old LINKFLAGS,
removal of superflous ';' and adding of $(LIBS) to link command).


 +-SHLIB_OBJ= $(addprefix shared/, $(LIB_OBJ));
 ++SHLIB_OBJ= $(addprefix shared/, $(LIB_OBJ))

 The »;« was superfluous, since no command followed?

yes; and it would break the build because it terminates the $(CC) command
and $(LIBS) would be interpreted as the start of a new command.

 ++   $(CC) $(LDFLAGS) -o $(SHLIB) $(SHLINKFLAGS) $(SHLIB_OBJ) $(LIBS)


 Is this patch already upstream?

woglinde gave his ack on irc and I submitted it.



Enrico

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


Re: [oe] [PATCH, v2] openssl: enforce libdir=lib

2011-04-26 Thread Enrico Scholz
Koen Kooi k...@dominion.thruhere.net writes:

 +perl ./Configure ${EXTRA_OECONF} shared --prefix=$useprefix 
 --libdir=`basename ${libdir}`

 Why not plain ${base_libdir}?

openssl's '--libdir' option expects a relative name. E.g. there is done

| $libdir=lib$multilib if $libdir eq ;
| ...
|   s/^LIBDIR=.*$/LIBDIR=$libdir/;
| ...
|   my $foo = $prefix/$libdir/engines;

in Configure and

|   echo 'libdir=$${exec_prefix}/$(LIBDIR)'; \

in Makefile.


Using absolute paths (as in ${base_libdir}) would create paths with
double '/'.  Perhaps, this would be a cosmetical issue only but using
plain 'lib' (or 'lib64') is better imo.


Enrico

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


Re: [oe] [PATCH] tcp-wrappers: place LIBS after object files

2011-04-26 Thread Enrico Scholz
Paul Menzel paulepan...@users.sourceforge.net writes:

 For documentation purpose it would be great if you could add such
 acknowledgments to the commit message.

ok; I will add it the next time.


 But I meant upstream of TCP Wrapper.

tcp-wrappers have an upstream? afais, the tarball in the package is from
1997...


Enrico

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


Re: [oe] [PATCH 1/5] binconfig.bbclass: search configs in D instead of S

2011-04-20 Thread Enrico Scholz
Paul Menzel paulepan...@users.sourceforge.net writes:

  Searching for binconfig in S makes more problems than it
  brings benefits. Searching in the installed root seems to be more logical.
 ...
 I am adding Enrico to the CC since he did a lot of Ncurses work and
 maybe can also share his opinion.

yes; I submitted a similar patch [1] because it is required by ncurses.

Acked-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de



Enrico

Footnotes: 
[1]  http://patchwork.openembedded.org/patch/2248

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


[oe] [PATCH] opkg-utils: convert mtime to int before comparing it

2011-02-23 Thread Enrico Scholz
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 patch was applied as 49e11b32906fc1becd9218bb3125f6fc8fa03fa4 to
old ipkg-utils already.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/opkg-utils/opkg-utils/mtime-int.patch |   27 +
 recipes/opkg-utils/opkg-utils_svn.bb  |5 ++-
 2 files changed, 30 insertions(+), 2 deletions(-)
 create mode 100644 recipes/opkg-utils/opkg-utils/mtime-int.patch

diff --git a/recipes/opkg-utils/opkg-utils/mtime-int.patch 
b/recipes/opkg-utils/opkg-utils/mtime-int.patch
new file mode 100644
index 000..efec3c0
--- /dev/null
+++ b/recipes/opkg-utils/opkg-utils/mtime-int.patch
@@ -0,0 +1,27 @@
+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.
+
+Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
+
+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/recipes/opkg-utils/opkg-utils_svn.bb 
b/recipes/opkg-utils/opkg-utils_svn.bb
index 951580a..6b8d3ea 100644
--- a/recipes/opkg-utils/opkg-utils_svn.bb
+++ b/recipes/opkg-utils/opkg-utils_svn.bb
@@ -6,10 +6,11 @@ RDEPENDS_${PN} = python
 RDEPENDS_${PN}_virtclass-native = 
 SRCREV = 4747
 PV = 0.1.8+svnr${SRCPV}
-PR = r5
+PR = r6
 
 SRC_URI = svn://svn.openmoko.org/trunk/src/host/;module=opkg-utils;proto=http 
\
-   file://index-ignore-filenotfound.patch
+   file://index-ignore-filenotfound.patch \
+   file://mtime-int.patch
 
 S = ${WORKDIR}/opkg-utils
 
-- 
1.7.4


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


[oe] [PATCH] cortex-m3: adjusted target cflags

2011-02-07 Thread Enrico Scholz
Explicit '-mthumb' is required because Cortex-M3 does not support ARM
instructions and actual flags cause gcc to fail with 'error: target CPU
does not support ARM mode'.

The '-mcpu=cortex-m3' is shorter than '-march=armv7-m -mtune=cortex-m3'
and enables workarounds like '-mfix-cortex-m3-ldrd'.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 conf/machine/include/tune-cortexm3.inc |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/conf/machine/include/tune-cortexm3.inc 
b/conf/machine/include/tune-cortexm3.inc
index 6da9aee..a66a33c 100644
--- a/conf/machine/include/tune-cortexm3.inc
+++ b/conf/machine/include/tune-cortexm3.inc
@@ -1,4 +1,4 @@
 # valid options for -march: `armv7', `armv7-m'
-TARGET_CC_ARCH = -march=armv7-m -mtune=cortex-m3 -mfpu=vfp -mfloat-abi=softfp
+TARGET_CC_ARCH = -mthumb -mcpu=cortex-m3 -mfpu=vfp -mfloat-abi=softfp
 FEED_ARCH = armv7
 BASE_PACKAGE_ARCH = armv7
-- 
1.7.4


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


[oe] [PATCH] gcc-4.5: added thumb2 build patch

2011-02-07 Thread Enrico Scholz
Fix for http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43999 and is
required to build gcc for Thumb2 only architectures (e.g. Cortex-M3).

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/gcc/gcc-4.5.inc|3 ++-
 recipes/gcc/gcc-4.5/thumb2-build.patch |   11 +++
 2 files changed, 13 insertions(+), 1 deletions(-)
 create mode 100644 recipes/gcc/gcc-4.5/thumb2-build.patch

diff --git a/recipes/gcc/gcc-4.5.inc b/recipes/gcc/gcc-4.5.inc
index b630528..9ba2e22 100644
--- a/recipes/gcc/gcc-4.5.inc
+++ b/recipes/gcc/gcc-4.5.inc
@@ -8,7 +8,7 @@ DEPENDS = mpfr gmp libmpc libelf
 NATIVEDEPS = mpfr-native gmp-native libmpc-native
 
 
-INC_PR = r31
+INC_PR = r32
 
 SRCREV = 168622
 PV = 4.5
@@ -166,6 +166,7 @@ SRC_URI = 
svn://gcc.gnu.org/svn/gcc/branches;module=${BRANCH} \
 # file://linaro/gcc-4.5-linaro-r99457.patch \
   file://gcc-scalar-widening-pr45847.patch \
   file://gcc-arm-volatile-bitfield-fix.patch \
+  file://thumb2-build.patch \
  
 
 SRC_URI_append_mips64 =  file://mips64-nomultilib.patch 
diff --git a/recipes/gcc/gcc-4.5/thumb2-build.patch 
b/recipes/gcc/gcc-4.5/thumb2-build.patch
new file mode 100644
index 000..8fefbfa
--- /dev/null
+++ b/recipes/gcc/gcc-4.5/thumb2-build.patch
@@ -0,0 +1,11 @@
+Index: gcc/config/arm/lib1funcs.asm
+===
+--- a/gcc/config/arm/lib1funcs.asm (revision 163231)
 b/gcc/config/arm/lib1funcs.asm (working copy)
+@@ -642,5 +642,5 @@
+   cmp \dividend, #0   @ Early termination?
+-  do_it   hs, t
++  do_it   ne, t
+   movnes  \curbit,   \curbit,  lsr #4 @ No, any more bits to do?
+   movne   \divisor,  \divisor, lsr #4
+   bne 1b
-- 
1.7.4


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


[oe] [PATCH, v2] gcc-4.5: added thumb2 build patch

2011-02-07 Thread Enrico Scholz
Fix for http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43999 and is
required to build gcc for Thumb2 only architectures (e.g. Cortex-M3).

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/gcc/gcc-4.5.inc|3 ++-
 recipes/gcc/gcc-4.5/thumb2-build.patch |   16 
 2 files changed, 18 insertions(+), 1 deletions(-)
 create mode 100644 recipes/gcc/gcc-4.5/thumb2-build.patch

diff --git a/recipes/gcc/gcc-4.5.inc b/recipes/gcc/gcc-4.5.inc
index b630528..9ba2e22 100644
--- a/recipes/gcc/gcc-4.5.inc
+++ b/recipes/gcc/gcc-4.5.inc
@@ -8,7 +8,7 @@ DEPENDS = mpfr gmp libmpc libelf
 NATIVEDEPS = mpfr-native gmp-native libmpc-native
 
 
-INC_PR = r31
+INC_PR = r32
 
 SRCREV = 168622
 PV = 4.5
@@ -166,6 +166,7 @@ SRC_URI = 
svn://gcc.gnu.org/svn/gcc/branches;module=${BRANCH} \
 # file://linaro/gcc-4.5-linaro-r99457.patch \
   file://gcc-scalar-widening-pr45847.patch \
   file://gcc-arm-volatile-bitfield-fix.patch \
+  file://thumb2-build.patch \
  
 
 SRC_URI_append_mips64 =  file://mips64-nomultilib.patch 
diff --git a/recipes/gcc/gcc-4.5/thumb2-build.patch 
b/recipes/gcc/gcc-4.5/thumb2-build.patch
new file mode 100644
index 000..75e765e
--- /dev/null
+++ b/recipes/gcc/gcc-4.5/thumb2-build.patch
@@ -0,0 +1,16 @@
+Fix for http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43999
+
+Patch was taken from comment #3 (Jing Yu) there; the patch attached
+to the named bugreport is wrong.
+
+Index: gcc/config/arm/lib1funcs.asm
+===
+--- a/gcc/config/arm/lib1funcs.asm (revision 163231)
 b/gcc/config/arm/lib1funcs.asm (working copy)
+@@ -642,5 +642,5 @@
+   cmp \dividend, #0   @ Early termination?
+-  do_it   hs, t
++  do_it   ne, t
+   movnes  \curbit,   \curbit,  lsr #4 @ No, any more bits to do?
+   movne   \divisor,  \divisor, lsr #4
+   bne 1b
-- 
1.7.4


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


Re: [oe] [PATCH] cortex-m3: adjusted target cflags

2011-02-07 Thread Enrico Scholz
Khem Raj raj.k...@gmail.com writes:

 The '-mcpu=cortex-m3' is shorter than '-march=armv7-m -mtune=cortex-m3'
 and enables workarounds like '-mfix-cortex-m3-ldrd'.

 Does -mcpu=cortex-m3 also cause gcc to tune for cortex-m3 ?

Yes; logic in gcc/config/arm/arm.c is:

|if (i == ARM_OPT_SET_CPU || i == ARM_OPT_SET_TUNE)
|  arm_tune = (enum processor_type) (sel - ptr-processors);
|
|if (i == ARM_OPT_SET_ARCH)
|  target_arch_cpu = sel-core;
|
|if (i == ARM_OPT_SET_CPU)
|  selected_cpu = (enum processor_type) (sel - ptr-processors);

'-mcpu' causes the highest optimization level (both for the used instruction
set (-- '-march') and the soft optimizations (-- '-mcpu')).



Enrico

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


Re: [oe] [PATCH] cortex-m3: adjusted target cflags

2011-02-07 Thread Enrico Scholz
Khem Raj raj.k...@gmail.com writes:

 Can you check what does gcc emits in an assembly file when using
 -mcpu=cortex-m3 in your case. If it emits sdiv instruction or not

 int foo (long a, long b)
 {
 return a/b;
 }

$ arm-linux-uclibceabi-gcc -mthumb -mcpu=cortex-m3 -mfpu=vfp -mfloat-abi=softfp 
-O1 -c x.c
$ arm-linux-uclibceabi-objdump -d x.o
 foo:
   0:   fb90 f0f1   sdivr0, r0, r1
   4:   4770bx  lr
   6:   bf00nop



Enrico

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


Re: [oe] [PATCH 2/5] perl-native: Switch to userelocatableinc and not shared libperl

2011-01-31 Thread Enrico Scholz
Tom Rini tom_r...@mentor.com writes:

 - for PERLSCRIPT in `grep -rIl ${bindir}/perl ${D}${bindir}`; do
 - sed -i -e 's|^#!${bindir}/perl|#!/usr/bin/env perl|' $PERLSCRIPT
 + for PERLSCRIPT in `grep -rIEl '#!.*/perl' ${D}${bindir}`; do
 + sed -i -e '1s|^#!.*|#!/usr/bin/env perl|' $PERLSCRIPT

This will break shebang lines with interpreter options (e.g. 
'#! /usr/bin/perl -T'):

| $ echo '#! /usr/bin/env perl -T'  /tmp/x.pl
| $ chmod +x /tmp/x.pl
| $ /tmp/x.pl
| /usr/bin/env: perl -T: No such file or directory



Enrico

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


Re: [oe] [PATCH] zlib: substitute sharedlibdir

2011-01-14 Thread Enrico Scholz
Khem Raj raj.k...@gmail.com writes:

 @sharedlibdir@ is not substituted so that linkerflags in zlib.pc expand to
 ... -L@sharedlibdir@ which breaks e.g. glib builds.

 I think zlib has --sharedlibdir configure option since 1.2.4.4+
 so you could try to use it instead of patching

afais, the ./configure will be completely replaced due to the
0001-autotools.patch (added by a2393dbf941554098).  So there exists no
such option anymore.


 --- a/recipes/zlib/zlib-1.2.5/0001-autotools.patch
 +++ b/recipes/zlib/zlib-1.2.5/0001-autotools.patch
 @@ -33,7 +33,7 @@ new file mode 100644
   index 000..177c89a
   --- /dev/null
   +++ b/configure.ac


Enrico

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


Re: [oe] [PATCH] e2fsprogs: change mode of non-links only

2011-01-14 Thread Enrico Scholz
Khem Raj raj.k...@gmail.com writes:

 alternatively you can exclude patches directory from feeding into
 chmod

I do not think that excluding a single, implementation specific directory
is a good idea. When this or a future package contains symlinks to outside
the buildroot, problem would be triggered again.

Doing blindly 'find | xargs chmod' is just wrong...

I will submit a new patch which executes 'chmod -R u=rwx .' (this does
not dereference symlinks) instead of using the find pipe.


 -find ./ -print|xargs chmod u=rwX
 +find ./ ! -type l -print|xargs chmod u=rwX



Enrico

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


Re: [oe] [PATCH 3/3] mtd-utils: add 1.4.1

2011-01-13 Thread Enrico Scholz
Eric BĂ©nard e...@eukrea.com writes:

 Is there a reason we're perpetuating lzo1 for newer stuff rather than
 switching to lzo2?

 lzo1 seems necessary :
 compr_lzo.c:29:19: fatal error: lzo1x.h: No such file or directory

mtd-utils build fine here after removing *all* the lzo1 related patches
and the do_configure_prepend().


Enrico

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


[oe] [PATCH] e2fsprogs: change mode of non-links only

2011-01-13 Thread Enrico Scholz
From: Enrico Scholz enrico.sch...@informatik.tu-chemnitz.de

When building as a dedicated user, compilation fails else in a way like

| + cd /e2fsprogs-1.41.12-r27/e2fsprogs-1.41.12
| + do_compile
| + find ./ -print
| + xargs chmod u=rwX
| chmod: changing permissions of `./patches/llseek-uclibc.patch': Operation not 
permitted
| ERROR: Function do_compile failed

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/e2fsprogs/e2fsprogs.inc |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/recipes/e2fsprogs/e2fsprogs.inc b/recipes/e2fsprogs/e2fsprogs.inc
index 34b9950..bdb4bbb 100644
--- a/recipes/e2fsprogs/e2fsprogs.inc
+++ b/recipes/e2fsprogs/e2fsprogs.inc
@@ -23,7 +23,7 @@ EXTRA_OECONF = 
${@['','--disable-tls'][bb.data.getVar('ARM_ABI', d, 1) == 'oabi
 PARALLEL_MAKE = 
 
 do_compile_prepend () {
-   find ./ -print|xargs chmod u=rwX
+   find ./ ! -type l -print|xargs chmod u=rwX
( cd util; ${BUILD_CC} subst.c -o subst )
 }
 
-- 
1.7.3.4


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


[oe] [PATCH] zlib: substitute sharedlibdir

2011-01-13 Thread Enrico Scholz
From: Enrico Scholz enrico.sch...@informatik.tu-chemnitz.de

@sharedlibdir@ is not substituted so that linkerflags in zlib.pc expand to
... -L@sharedlibdir@ which breaks e.g. glib builds.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/zlib/zlib-1.2.5/0001-autotools.patch |4 +++-
 recipes/zlib/zlib_1.2.5.bb   |2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/recipes/zlib/zlib-1.2.5/0001-autotools.patch 
b/recipes/zlib/zlib-1.2.5/0001-autotools.patch
index 6642860..f5c50f1 100644
--- a/recipes/zlib/zlib-1.2.5/0001-autotools.patch
+++ b/recipes/zlib/zlib-1.2.5/0001-autotools.patch
@@ -33,7 +33,7 @@ new file mode 100644
 index 000..177c89a
 --- /dev/null
 +++ b/configure.ac
-@@ -0,0 +1,17 @@
+@@ -0,0 +1,19 @@
 +AC_INIT(zlib,1.2.5)
 +AC_CONFIG_SRCDIR(adler32.c)
 +AM_INIT_AUTOMAKE(zlibs,1.2.5)
@@ -45,6 +45,8 @@ index 000..177c89a
 +
 +AC_HEADER_STDC
 +
++AC_SUBST(sharedlibdir,$libdir)
++
 +AC_CONFIG_FILES([
 +Makefile
 +zlib.pc
diff --git a/recipes/zlib/zlib_1.2.5.bb b/recipes/zlib/zlib_1.2.5.bb
index 34419f0..1f51404 100644
--- a/recipes/zlib/zlib_1.2.5.bb
+++ b/recipes/zlib/zlib_1.2.5.bb
@@ -1,6 +1,6 @@
 include zlib.inc
 
-PR = ${INC_PR}.0
+PR = ${INC_PR}.1
 
 SRC_URI += file://0001-autotools.patch 
 
-- 
1.7.3.4


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


[oe] [PATCH] gdbm: use full qualified permissions for chmod operations

2011-01-13 Thread Enrico Scholz
From: Enrico Scholz enrico.sch...@informatik.tu-chemnitz.de

An umask of 022 can cause 'chmod' to fail in a way like

| $ chmod -w X
| chmod: X: new permissions are r--rw-r--, not r--r--r--

It is better to specify 'a-w' because chmod is not affected by the
umask then and changes all permission bits.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/gdbm/gdbm-1.8.3/chmod.patch |   23 +++
 recipes/gdbm/gdbm_1.8.3.bb  |3 ++-
 2 files changed, 25 insertions(+), 1 deletions(-)
 create mode 100644 recipes/gdbm/gdbm-1.8.3/chmod.patch

diff --git a/recipes/gdbm/gdbm-1.8.3/chmod.patch 
b/recipes/gdbm/gdbm-1.8.3/chmod.patch
new file mode 100644
index 000..11c65d9
--- /dev/null
+++ b/recipes/gdbm/gdbm-1.8.3/chmod.patch
@@ -0,0 +1,23 @@
+With an umask of 022 the build might fail with
+
+| chmod -w gdbm.h
+| chmod: gdbm.h: new permissions are r--rw, not r--r-
+| make: *** [gdbm.h] Error 1
+
+Index: gdbm-1.8.3/Makefile.in
+===
+--- gdbm-1.8.3.orig/Makefile.in
 gdbm-1.8.3/Makefile.in
+@@ -167,10 +167,10 @@ libgdbm_compat.la: $(C_LOBJS) gdbm.h
+ gdbm.h:   gdbm.proto gdbmerrno.h gdbm.proto2
+   rm -f gdbm.h
+   cp $(srcdir)/gdbm.proto gdbm.h
+-  chmod +w gdbm.h
++  chmod u+w gdbm.h
+   grep _ $(srcdir)/gdbmerrno.h  gdbm.h
+   cat $(srcdir)/gdbm.proto2  gdbm.h
+-  chmod -w gdbm.h
++  chmod a-w gdbm.h
+ 
+ testgdbm: testgdbm.o libgdbm.la @LIBOBJS@
+   $(LIBTOOL) $(CC) $(LDFLAGS) -o testgdbm testgdbm.o libgdbm.la @LIBOBJS@
diff --git a/recipes/gdbm/gdbm_1.8.3.bb b/recipes/gdbm/gdbm_1.8.3.bb
index 9c91ae9..24c541a 100644
--- a/recipes/gdbm/gdbm_1.8.3.bb
+++ b/recipes/gdbm/gdbm_1.8.3.bb
@@ -8,7 +8,8 @@ PR = r5
 
 SRC_URI = ${GNU_MIRROR}/gdbm/gdbm-${PV}.tar.gz \
   file://makefile.patch \
-   file://libtool-mode.patch
+   file://libtool-mode.patch \
+   file://chmod.patch
 
 inherit autotools
 
-- 
1.7.3.4


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


Re: [oe] [PATCH] eglibc: use HTTP for svn checkout

2010-12-15 Thread Enrico Scholz
Ryan D Phillips ryan_d_phill...@dell.com writes:

 I would like to see this patch get into the OE tree to make the tree
 more robust to proxies. Currently, this is the only customization I
 need for the projects I work on.

fwiw, I wrote a local patch for bitbake

  https://www.cvg.de/people/ensc/0004-svn-tunnel.patch

which allows fetching of svn:// uris through an http proxy (which must
be configured to allow CONNECTs to svn servers).  I thought it would be
some local hack but when there is wider interest I will submit it to the
bitbake ml.


Enrico

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


Re: [oe] [PATCH] cacaoh: patches to get do_install do its work

2010-10-26 Thread Enrico Scholz
Khem Raj raj.k...@gmail.com writes:

 -install -m 0755 src/cacaoh/.libs/cacaoh 
 ${STAGING_BINDIR}/cacaoh-cldc-${PV}
 +install -m 0755 src/cacaoh/cacaoh ${STAGING_BINDIR}/cacaoh-cldc-${PV}

 * libtool creates sometimes a wrapper script for executables (e.g. when
   they are linked against local libraries) and places the real executable
   in .libs/.  Manually installing one of them is error prone and it might
   break with other linker flags or libtool versions.

 the binary in .libs do not exist so the link is final in this case.

Really?  The author of the original recipe had an environment where the
executable was in .libs/.  Are you really sure that such an environment
does not exist anymore? Or will the recipe now silently fail by installing
the shell wrapper sometimes?


 but in any case If you can come up with a patch to install it using
 libtool that would be nice.

Sorry; providing a patch means testing it (regardless whether it is
trivial).  Here, I am not sure whether 'libtool' or the platform
prefixed 'arch-libtool' is required and how to express
'arch-libtool' in oe.

I gave my comment to a patch which fixed a broken recipe by replacing it
with another broken recipe.  So I expect that the author of this patch
fixes and tests his work.

For completeness; install command should be

| libtool --mode=install install -m 0755 src/cacaoh/cacaoh 
${STAGING_BINDIR}/cacaoh-cldc-${PV}



Enrico

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


Re: [oe] [PATCH] cacaoh: patches to get do_install do its work

2010-10-25 Thread Enrico Scholz
Khem Raj raj.k...@gmail.com writes:

 -    install -m 0755 src/cacaoh/.libs/cacaoh 
 ${STAGING_BINDIR}/cacaoh-cldc-${PV}
 +    install -m 0755 src/cacaoh/cacaoh ${STAGING_BINDIR}/cacaoh-cldc-${PV}

 This is the wrong way to install libtool binaries/libraries.  Please
 use 'libtool --mode=install install -m ...' (or the platform prefixed
 'libtool' program).

 whats the difference ?

* 'libtool --mode=install' is the documented way[1] to install libtool
  binaries (in case of libraries, a 'libtool --mode=finish' might be
  required too)

* libtool creates sometimes a wrapper script for executables (e.g. when
  they are linked against local libraries) and places the real executable
  in .libs/.  Manually installing one of them is error prone and it might
  break with other linker flags or libtool versions.


Enrico

Footnotes: 
[1]  info libtool - Installing libraries + Installing executables


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


Re: [oe] [PATCH] cacaoh: patches to get do_install do its work

2010-10-24 Thread Enrico Scholz
Jaap de Jong jaap.dej...@nedap.com writes:

 -install -m 0755 src/cacaoh/.libs/cacaoh 
 ${STAGING_BINDIR}/cacaoh-cldc-${PV}
 +install -m 0755 src/cacaoh/cacaoh ${STAGING_BINDIR}/cacaoh-cldc-${PV}

This is the wrong way to install libtool binaries/libraries.  Please
use 'libtool --mode=install install -m ...' (or the platform prefixed
'libtool' program).


Enrico

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


Re: [oe] [PATCH v3] package.bbclass: copy dotfiles in root D to PKGD

2010-10-19 Thread Enrico Scholz
Michael Smith msm...@cbnco.com writes:

 + os.system('rm -rf %s/*' % (pkgcopy))

Can be written as

  os.system('rm -rf %s' % (pkgcopy))

Every SUSv3 compliant 'cp' will create the 'pkgcopy' directory.


 + os.system('cp -pPR %s/. %s/' % (installdest, pkgcopy))


Enrico

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


Re: [oe] [PATCH v3] package.bbclass: copy dotfiles in root D to PKGD

2010-10-19 Thread Enrico Scholz
Frans Meulenbroeks fransmeulenbro...@gmail.com writes:

 +     os.system('rm -rf %s/*' % (pkgcopy))

 Can be written as

  os.system('rm -rf %s' % (pkgcopy))
 ...
 +     os.system('cp -pPR %s/. %s/' % (installdest, pkgcopy))


 No this is not good. The rm command you give will also remove the dir.

Why is this bad?  The dir is recreated by the 'cp'.


 The cp command will copy %s/.

Only the final permissions of dstdir may differ between

  rm -rf dstdir
  cp -pPR srcdir/. dstdir/

and

  rm -rf dstdir
  mkdir dstdir
  cp -pPR srcdir/. dstdir/


In first case, dstdir gets permissions of srcdir. In the second it
keeps the permissions from dstdir.

Else, content (both the files and their attributes) of dstdir will be
the same for both operations.


Enrico

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


Re: [oe] [PATCH v2] package.bbclass: copy dotfiles in root D to PKGD

2010-10-18 Thread Enrico Scholz
Frans Meulenbroeks fransmeulenbro...@gmail.com writes:

 - os.system('cp -pPR %s/* %s/' % (dest, dvar))
 + os.system('cp -pPR %s/* %s/.??* %s/' % (dest, dest, dvar))

why not os.system('cp -pPR %s/. %s/' % (dest, dvar))?


Enrico

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


Re: [oe] [PATCH] cacao-native-0.99.3: fix `install: cannot stat `src/cacaoh/.libs/cacaoh': No such file or directory`

2010-10-15 Thread Enrico Scholz
Paul Menzel paulepan...@users.sourceforge.net writes:

 -  install -m 0755 src/cacaoh/.libs/cacaoh ${D}${bindir}/cacaoh-${PV}
 +  install -m 0755 src/cacaoh/cacaoh ${D}${bindir}/cacaoh-${PV}

Location of the binary depends on libtool version and other factors so
that both paths are possible.

| libtool --mode=install install -m 0755 src/cacaoh/cacaoh ...

is more portable.



Enrico

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


Re: [oe] [PATCH] sane-toolchain.inc: Make gcc 4.5 eglibc 2.12 as default sane versions

2010-09-28 Thread Enrico Scholz
Enrico Scholz enrico.sch...@sigma-chemnitz.de writes:

 Probably not on ARMv5 ;) There 5/3 == 3 holds and ld.so crashes due to
 related errors.

sorry; my fault.  I applied a broken patch from [1].


Enrico

Footnotes: 
[1]  http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43999

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


Re: [oe] nano RRECOMMENDS ncurses-terminfo

2010-09-25 Thread Enrico Scholz
Martin Jansa martin.ja...@gmail.com writes:

 +# actually RDEPENDS, but ncurses-terminfo is only in ncurses-5.7, so 
 RRECOMMENDS for now
 +RRECOMMENDS_${PN} = ncurses-terminfo
 ...
 Is there something special in ncurses-terminfo 5.7 that nano needs?

 There wasn't any common /etc/terminfo in base ncurses-5.7 package and I
 guess it's still isn't. 

There was added a 

+RSUGGESTS_${PN}-libtinfo = ${PN}-terminfo
+RRECOMMENDS_${PN}-libtinfo = ${PN}-terminfo-base

a month ago which should install these dependencies.


Enrico

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


Re: [oe] Qt and detecting iWMMXT

2010-09-25 Thread Enrico Scholz
Holger Freyther holger...@freyther.de writes:

 when building for XScale one could pass -iwmmxt to the configure
 script. 

I would avoid -march=iwmmxt because

a) there seems to be kernel bug which does not restore IWMMXT state on
   ELF loading[1]

b) gcc 4.4.4 is bugged for this architecture (both ICE and generation of
   inefficent code) and fails e.g. to bootstrap itself or glibc


When you really need iwmmxt support, I recommend to add '.arch iwmmxt'
to (inline) assembler instead of using -march=iwmmxt.


Enrico

Footnotes: 
[1]  http://article.gmane.org/gmane.linux.ports.arm.general/11795

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


[oe] ncurses/libtermcap fixes

2010-09-09 Thread Enrico Scholz
Hi,

in the last days I committed several changes to the ncurses receipt
which should fix:

* broken widec builds for native/sdk targets

* missing -ltermcap

* implicit -ltinfo on -lncurses (-- this might require a 

  |  bitbake ncurses -c clean

  to take an effect because staging does not change symbolic links to
  regular files on updates)


I saw several patches which deal with this issues and which are probably
not needed anymore.  So, please give recent snapshot a try and see whether
problems are still there.



Enrico


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


Re: [oe] cmake-native trouble making

2010-09-08 Thread Enrico Scholz
Jaap de Jong jaap.dej...@nedap.com writes:

 $ bitbake cmake-native
 
 | [ 87%] Building CXX object
 Source/CMakeFiles/CPackLib.dir/CPack/cmCPackNSISGenerator.o
 | /usr/bin/ld: cannot find -ltinfo
 | collect2: ld returned 1 exit status
 | make[2]: *** [bin/ccmake] Error 1
 | make[1]: *** [Source/CMakeFiles/ccmake.dir/all] Error 2
 | make[1]: *** Waiting for unfinished jobs
 

 Is cmake-native_2.8.2.bb missing this line:

 RDEPENDS = ncurses-terminfo

no, 'DEPENDS = ncurses-native'; BUT:

* it will cause problems on (host)distributions which do not have libtinfo
  (e.g. RHEL5) and have ncurses-native in their ASSUME_PROVIDED

* some hours ago I added a libtermcap.so compatibility symlink to
  ncurses(-native) so that 5ea3047995421d99f7f3537cb8f9ae23f3185a9f
  might not be needed anymore

* build of the curses frontend should be disabled for cmake-native because
  we do automated builds only; I do not know atm how to configure cmake
  for this...


Enrico

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


Re: [oe] libtermcap

2010-09-08 Thread Enrico Scholz
Frans Meulenbroeks fransmeulenbro...@gmail.com writes:

 Eric peeked into the termcap problem. I've tested his patch and it
 works. He will probably push a patch later today.

 The termcap problem from asterisk is not related. It cannot find
 libtermcap. This one does not exist any more with ncurses 5.7.

 People how should we fix this? 

I added 9e58422868a03eeafa72781477174850d1c5abe7 some hours ago so that
'-ltermcap' should work again.  Patch does not break my builds but I do
not know whether it unbreaks the problematic ones ;)

Things are too different across the distributions (some have the
ncurses-config binconfig tools, other the .pc files, some have
libtinfo, some not) to find *the* correct solution.


Enrico

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


Re: [oe] cmake-native trouble making

2010-09-08 Thread Enrico Scholz
Tom Rini tom_r...@mentor.com writes:

 | /usr/bin/ld: cannot find -ltinfo

 no, 'DEPENDS = ncurses-native'; BUT:

 * it will cause problems on (host)distributions which do not have libtinfo
   (e.g. RHEL5) and have ncurses-native in their ASSUME_PROVIDED

 Isn't the whole point of ASSUME_PROVIDED that user be aware?

Would be too arbitrary in this case because it forces people to check by
trial  error whether ASSUME_PROVIDED are working.  A raw guideline for
adding an entry to ASSUME_PROVIDED should be, whether a corresponding,
recent -devel package has been installed on the host.

imo, packages should not have so strict requirements on the actual
provider but check in their configure task what to use (this includes
autodetection of libtinfo vs. libtermcap vs. libncurses).


For cmake, problem above can by solved by an initial configuration
script having

| SET (BUILD_CursesDialog OFF CACHE BOOL  FORCE)


 * some hours ago I added a libtermcap.so compatibility symlink to
   ncurses(-native) so that 5ea3047995421d99f7f3537cb8f9ae23f3185a9f
   might not be needed anymore

 Um, is there a reason this wasn't just done as a ln within the recipe?

It's a mix of personal preference, following Fedora's packaging of this
file and the hope to use some more of the power of linker scripts (giving
out a deprecated warning, add AS_NEEDED, linking against multiple files).
There was no opposition when I wrote about linker scripts neither.

Perhaps I should have made it inline by 'echo INPUT(-ltinfo)  ...' but
I did not saw this when I committed the patch.



Enrico

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


Re: [oe] libtermcap

2010-09-08 Thread Enrico Scholz
Tom Rini tom_r...@mentor.com writes:

 I added 9e58422868a03eeafa72781477174850d1c5abe7 some hours ago so
 that '-ltermcap' should work again.  Patch does not break my builds
 but I do not know whether it unbreaks the problematic ones ;)

 Things are too different across the distributions (some have the
 ncurses-config binconfig tools, other the .pc files, some have
 libtinfo, some not) to find *the* correct solution.

 Maybe I'm jumping in too late here.  Why is the correct solution not
 make our ncurses-native build and install what's required by other
 recipes, fix broken recipes that look for the wrong thing ?  

the wrong and right thing is difficultly to define for libtermcap.
Generally, patches should be in a manner that they can be accepted by
upstream.  That's not the case for 's!-ltermcap!-ltinfo!'


 IMHO, if you ASSUME_PROVIDED and things break, thats not our problem,
 its your problem (Doctor, it hurts when I do *this*.  Don't do *that*
 !)

OE has still the ncurses-5.4 receipt which will break packages depending
on hardcoded -ltinfo.



Enrico

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


Re: [oe] [PATCH v2] ncurses-5.7 : fix sdk build + add libtermcap link

2010-08-29 Thread Enrico Scholz
Eric BĂ©nard e...@eukrea.com writes:

 -'narrowc --with-ticlib' \
 -'widec   --enable-widec --without-progs'; do
 +'narrowc  --with-ticlib' \
 +'widec--with-ticlib --enable-widec --without-progs'; do

can you place this in a separate patch please?


 + cd ${D}${libdir}
 + ln -sf libncurses.so libtermcap.so
 + ln -sf libncurses.a libtermcap.a

This part is wrong; libtermcap functionality is provided by libtinfo and
does not need whole libncurses.

A linker script libtermcap.so with

| INPUT(-ltinfo)

might be a better solution for .so.


 + ln -sf libncurses.so.5 libtermcap.so.5

This is not needed; DL_NEEDED will be libtinfo.so.X both if you create a
symlink or use the linkerscript.  There is nothing with a libtermcap.so.5
SONAME either.


 -PACKAGES =  \
 -  ncurses-dbg \
 -  ncurses-dev \
 -  ncurses-doc \
 -  ncurses-tools \
 -  ncurses \
 -  ncurses-static \
 -  ncurses-terminfo \
 +PACKAGES +=  \
 +  ${PN}-tools \
 +  ${PN}-terminfo \
  

please put this into a separate patch.


Enrico

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


Re: [oe] [PATCH] alsa-utils-alsamixer: add ncurses-terminfo RDEPENDS

2010-08-29 Thread Enrico Scholz
Eric BĂ©nard e...@eukrea.com writes:

 +RDEPENDS_alsa-utils-alsamixer = ncurses-terminfo

I think this should be solved in ncurses because it affects other
packages too; e.g. moving its /etc/terminfo files from 'ncurses' into
an own package and rrecommending this in the libtinfo package.


Enrico

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


Re: [oe] [PATCH] alsa-utils-alsamixer: add ncurses-terminfo RDEPENDS

2010-08-29 Thread Enrico Scholz
Martin Jansa martin.ja...@gmail.com writes:

  +RDEPENDS_alsa-utils-alsamixer = ncurses-terminfo
 
 I think this should be solved in ncurses because it affects other
 packages too; e.g. moving its /etc/terminfo files from 'ncurses' into
 an own package and rrecommending this in the libtinfo package.

 I've already added this to few recipes:
 http://git.openembedded.org/cgit.cgi/openembedded/log/?qt=grepq=terminfo

The 'ncurses-terminfo' is a large package (500kB) and contains definitions
for exotic terminal.  The definitions on 'ncurses' take only 6kB and cover
perhaps 95% of all use cases (linux, xterm, vt100, screen).

I will move them into an own -terminfo-base package; atm I am not sure
whether I should write

| RSUGGESTS_${PN}-libtinfo = ${PN}-terminfo
| RRECOMMENDS_${PN}-libtinfo = ${PN}-terminfo-base

or

| RPROVIDES_${PN}-terminfo = virtual/terminfo
| RPROVIDES_${PN}-terminfo-base = virtual/terminfo
| RDEPENDS_${PN}-libtinfo = virtual/terminfo

I tend to the first variant.


Enrico

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


Re: [oe] [PATCH] jpeg_8b: Allow using older versions of automake

2010-08-29 Thread Enrico Scholz
Chase Maupin chasemaupi...@gmail.com writes:

 * Added patch to allow using older versions of automake such
   as 1.10.3 when building jpeg version 8b.
 * Only use the AM_SILENT_RULES if it is available.

 Signed-off-by: Chase Maupin chase.mau...@ti.com

Acked-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de


 +
 file://0001-configure-make-jpeg-work-with-older-autotools.patch;apply=yes \

I suggest to remove the explicit ';apply=yes'.



Enrico

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


Re: [oe] [PATCH v2] ncurses-5.7 : fix sdk build + add libtermcap link

2010-08-29 Thread Enrico Scholz
Eric BĂ©nard e...@eukrea.com writes:

 -'narrowc --with-ticlib' \
 -'widec   --enable-widec --without-progs'; do
 +'narrowc  --with-ticlib' \
 +'widec--with-ticlib --enable-widec --without-progs'; do

 can you place this in a separate patch please?

 I'm not fully sure of this change, do you think this is needed or is
 this a mistake ?

I do not see a reason, not to apply '--with-ticlib' to widec.  I changed
this in git.


Thanks for noticing it
Enrico

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


Re: [oe] ncurses 5.7 doesn't build libtermcap anymore?

2010-08-29 Thread Enrico Scholz
Eric BĂ©nard e...@eukrea.com writes:

 5.7 is also broken for sdk build, 

I think that I fixed widec builds for native targets in git. 'sdk' and
'native' works on Fedora 13 now; please check other distributions.


Enrico

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


[oe] [PATCH] oe.path: added 'recurse' argument to remove()

2010-08-16 Thread Enrico Scholz
This makes it possible to specify whether the equivalent of 'rm -rf' or
only this of 'rm -f' is wanted.  Due to backward compatibility it
defaults to the recursive variant.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 lib/oe/path.py |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/oe/path.py b/lib/oe/path.py
index f58c013..3d64cfa 100644
--- a/lib/oe/path.py
+++ b/lib/oe/path.py
@@ -43,13 +43,13 @@ def format_display(path, metadata):
 else:
 return rel
 
-def remove(path):
+def remove(path, recurse=True):
 Equivalent to rm -f or rm -rf
 import os, errno, shutil
 try:
 os.unlink(path)
 except OSError, exc:
-if exc.errno == errno.EISDIR:
+if recurse and exc.errno == errno.EISDIR:
 shutil.rmtree(path)
 elif exc.errno != errno.ENOENT:
 raise
-- 
1.7.2.1


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


[oe] Usage of NATIVE_INSTALL_WORKS

2010-08-16 Thread Enrico Scholz
Hi,

http://wiki.openembedded.org/index.php/Legacy_staging states that
NATIVE_INSTALL_WORKS must be set when there is a non trivial
do_install() function and BBCLASSEXTEND is used.

But 

| git grep NATIVE_INSTALL_WORKS conf/ classes/ lib/ 

shows only one place where this variable is evaluated:

| classes/staging.bbclass:elif bb.data.getVar('NATIVE_INSTALL_WORKS', d, 1) 
== 1:
| classes/staging.bbclass-legacy = False

And there, it is used only in the is_legacy_staging() function, to
override legacy/non-legacy detection results.


Is there still any use for this variable in modern staging?  Or shall it
be purged from non-legacy recipes?


Enrico

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


[oe] [PATCH] liboil: removed old cruft before 0.3.15

2010-07-30 Thread Enrico Scholz
Versions before 0.3.15 are more than 2 years old and not referenced
anywhere.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/liboil/liboil-0.3.11/ppc-detect-fpu.patch |   11 ---
 recipes/liboil/liboil_0.3.10.bb   |   15 ---
 recipes/liboil/liboil_0.3.11.bb   |   18 --
 recipes/liboil/liboil_0.3.14.bb   |   21 -
 recipes/liboil/liboil_0.3.9.bb|   15 ---
 5 files changed, 0 insertions(+), 80 deletions(-)
 delete mode 100644 recipes/liboil/liboil-0.3.11/ppc-detect-fpu.patch
 delete mode 100644 recipes/liboil/liboil_0.3.10.bb
 delete mode 100644 recipes/liboil/liboil_0.3.11.bb
 delete mode 100644 recipes/liboil/liboil_0.3.14.bb
 delete mode 100644 recipes/liboil/liboil_0.3.9.bb

diff --git a/recipes/liboil/liboil-0.3.11/ppc-detect-fpu.patch 
b/recipes/liboil/liboil-0.3.11/ppc-detect-fpu.patch
deleted file mode 100644
index 49485e3..000
--- a/recipes/liboil/liboil-0.3.11/ppc-detect-fpu.patch
+++ /dev/null
@@ -1,11 +0,0 @@
 /m4/orig-as-gcc-inline-assembly.m4 2007-03-17 04:03:30.0 +0200
-+++ /m4/as-gcc-inline-assembly.m4  2007-11-20 12:20:24.0 +0200
-@@ -38,7 +38,7 @@
- [
-   AC_MSG_CHECKING([if compiler supports FPU instructions on PowerPC])
- 
--  AC_TRY_COMPILE([], [__asm__ (fadd 0,0,0::) ], [flag_ok=yes], [flag_ok=no])
-+  AC_TRY_COMPILE([], [double x = 0; __asm__ (fadd 0,0,0::f (x):32) ], 
[flag_ok=yes], [flag_ok=no])
- 
-   if test X$flag_ok = Xyes ; then
- $1
diff --git a/recipes/liboil/liboil_0.3.10.bb b/recipes/liboil/liboil_0.3.10.bb
deleted file mode 100644
index 7e5f951..000
--- a/recipes/liboil/liboil_0.3.10.bb
+++ /dev/null
@@ -1,15 +0,0 @@
-DESCRIPTION = Liboil is a library of simple functions that are optimized for 
various CPUs.
-HOMEPAGE = http://liboil.freedesktop.org/;
-LICENSE = liboil
-PR = r1
-
-SRC_URI = http://liboil.freedesktop.org/download/${P}.tar.gz;
-
-inherit autotools pkgconfig
-
-do_stage() {
-   autotools_stage_all
-}
-
-SRC_URI[md5sum] = 770f656bca8166dab33b322d5886a4bf
-SRC_URI[sha256sum] = 
4587753a023550a066abe59a7e467c1af35b9f5e81bfb40a12463ad2df8f088c
diff --git a/recipes/liboil/liboil_0.3.11.bb b/recipes/liboil/liboil_0.3.11.bb
deleted file mode 100644
index d8e055a..000
--- a/recipes/liboil/liboil_0.3.11.bb
+++ /dev/null
@@ -1,18 +0,0 @@
-DESCRIPTION = Liboil is a library of simple functions that are optimized for 
various CPUs.
-HOMEPAGE = http://liboil.freedesktop.org/;
-LICENSE = liboil
-PR =r3
-
-DEPENDS = glib-2.0
-
-SRC_URI = http://liboil.freedesktop.org/download/${P}.tar.gz \
-   file://ppc-detect-fpu.patch
-
-inherit autotools pkgconfig
-
-do_stage() {
-   autotools_stage_all
-}
-
-SRC_URI[md5sum] = 65ce3266be385d0c7cd9a115742f
-SRC_URI[sha256sum] = 
deb6530d40ecad2b849d68912500e6d03bab4ac63de7a97340d05af1d7cf0ad4
diff --git a/recipes/liboil/liboil_0.3.14.bb b/recipes/liboil/liboil_0.3.14.bb
deleted file mode 100644
index 92db760..000
--- a/recipes/liboil/liboil_0.3.14.bb
+++ /dev/null
@@ -1,21 +0,0 @@
-DESCRIPTION = Liboil is a library of simple functions that are optimized for 
various CPUs.
-HOMEPAGE = http://liboil.freedesktop.org/;
-LICENSE = liboil
-PR = r1
-
-# The configure script seems to have bogus checks, so let's not make it the 
default
-DEFAULT_PREFERENCE = -1
-
-DEPENDS = glib-2.0
-
-SRC_URI = http://liboil.freedesktop.org/download/${P}.tar.gz \
-  
-
-inherit autotools pkgconfig
-
-do_stage() {
-   autotools_stage_all
-}
-
-SRC_URI[md5sum] = 7a9f719e50656b795c06c0167012a84f
-SRC_URI[sha256sum] = 
a40c09db6ee24e03aa81f58329a57e45e55462a2e0f619b3796d4bf942c23f2f
diff --git a/recipes/liboil/liboil_0.3.9.bb b/recipes/liboil/liboil_0.3.9.bb
deleted file mode 100644
index c066fa4..000
--- a/recipes/liboil/liboil_0.3.9.bb
+++ /dev/null
@@ -1,15 +0,0 @@
-DESCRIPTION = Liboil is a library of simple functions that are optimized for 
various CPUs.
-HOMEPAGE = http://liboil.freedesktop.org/;
-LICENSE = liboil
-PR = r1
-
-SRC_URI = http://liboil.freedesktop.org/download/${P}.tar.gz;
-
-inherit autotools pkgconfig
-
-do_stage() {
-   autotools_stage_all
-}
-
-SRC_URI[md5sum] = 5d139b1fb16f0e93f0c84290ad2aaff8
-SRC_URI[sha256sum] = 
dd4d680ce95e586a1bbe767e1a7b25e1c53f842b8be6cf4e30e89cfa8232dd90
-- 
1.7.2


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


[oe] [PATCH] ncurses: added 5.7 recipe

2010-07-30 Thread Enrico Scholz
It uses ideas from the old 5.4 recipe but brings a lot of new features
like widec libs, pkgconfig files or splitted library packages.

As lot of packages depend on ncurses, whole distribution must be
probably rebuild when updating to this new version.

Due to this and the complex recipe, it has a negative DEFAULT_PREFERENCE
for now.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/ncurses/ncurses-5.7/tic-hang.patch |   25 
 recipes/ncurses/ncurses_5.7.bb |  187 
 2 files changed, 212 insertions(+), 0 deletions(-)
 create mode 100644 recipes/ncurses/ncurses-5.7/tic-hang.patch
 create mode 100644 recipes/ncurses/ncurses_5.7.bb

diff --git a/recipes/ncurses/ncurses-5.7/tic-hang.patch 
b/recipes/ncurses/ncurses-5.7/tic-hang.patch
new file mode 100644
index 000..1678b88
--- /dev/null
+++ b/recipes/ncurses/ncurses-5.7/tic-hang.patch
@@ -0,0 +1,25 @@
+'tic' of some linux distributions (e.g. fedora 11) hang in an infinite
+loop when processing the original file.
+
+Index: ncurses-5.7/misc/terminfo.src
+===
+--- ncurses-5.7.orig/misc/terminfo.src
 ncurses-5.7/misc/terminfo.src
+@@ -3706,12 +3706,11 @@ konsole-xf3x|KDE console window with key
+ # The value for kbs reflects local customization rather than the settings used
+ # for XFree86 xterm.
+ konsole-xf4x|KDE console window with keyboard for XFree86 4.x xterm,
+-  kend=\EOF, khome=\EOH, use=konsole+pcfkeys,
+-  use=konsole-vt100,
+-# Konsole does not implement shifted cursor-keys.
+-konsole+pcfkeys|konsole subset of xterm+pcfkeys,
+-  kLFT@, kRIT@, kcbt=\E[Z, kind@, kri@, kDN@, kUP@, use=xterm+pcc2,
+-  use=xterm+pcf0,
++  kend=\EOF, kf1=\EOP, kf13=\EO2P, kf14=\EO2Q, kf15=\EO2R,
++  kf16=\EO2S, kf17=\E[15;2~, kf18=\E[17;2~, kf19=\E[18;2~,
++  kf2=\EOQ, kf20=\E[19;2~, kf21=\E[20;2~, kf22=\E[21;2~,
++  kf23=\E[23;2~, kf24=\E[24;2~, kf3=\EOR, kf4=\EOS,
++  khome=\EOH, use=konsole-vt100,
+ # KDE's vt100 keyboard has no relationship to any terminal that DEC made, 
but
+ # it is still useful for deriving the other entries.
+ konsole-vt100|KDE console window with vt100 (sic) keyboard,
diff --git a/recipes/ncurses/ncurses_5.7.bb b/recipes/ncurses/ncurses_5.7.bb
new file mode 100644
index 000..8afb1d6
--- /dev/null
+++ b/recipes/ncurses/ncurses_5.7.bb
@@ -0,0 +1,187 @@
+DESCRIPTION = Ncurses library
+HOMEPAGE = http://www.gnu.org/software/ncurses/ncurses.html;
+LICENSE = MIT
+SECTION = libs
+PATCHDATE = 20100501
+PKGV = ${PV}.${PATCHDATE}
+PR = r8
+
+# recipe is experimental for now...
+DEFAULT_PREFERENCE = -1
+
+DEPENDS = ncurses-native
+DEPENDS_virtclass-native = 
+
+inherit autotools_stage binconfig
+
+SRC_URI = ${GNU_MIRROR}/ncurses/ncurses-${PV}.tar.gz;name=tarball \
+
ftp://invisible-island.net/ncurses/5.7/ncurses-5.7-20100424-patch.sh.bz2;apply=yes;name=p20100424sh
 \
+\
+
ftp://invisible-island.net/ncurses/5.7/ncurses-5.7-${PATCHDATE}.patch.gz;name=p20100501
 \
+file://tic-hang.patch \
+
+
+SRC_URI[tarball.md5sum] = cce05daf61a64501ef6cd8da1f727ec6
+SRC_URI[tarball.sha256sum] = 
0a9bdea5c7de8ded5c9327ed642915f2cc380753f12d4ad120ef7da3ea3498f4
+SRC_URI[p20100424sh.md5sum] = 3a5f76613f0f7ec3e0e73b835bc24864
+SRC_URI[p20100424sh.sha256sum] = 
1e9d70d2d1fe1fea471868832c52f1b9cc6065132102e49e2a3755f2f4f5be53
+SRC_URI[p20100501.md5sum] = 6518cfa5d45e9069a1e042468161448b
+SRC_URI[p20100501.sha256sum] = 
a97ccc30e4bd6fbb89564f3058db0fe84bd35cfefee831556c500793b477abde
+
+PARALLEL_MAKE = 
+EXTRA_AUTORECONF = -I m4
+
+# Whether to enable separate widec libraries; must be 'true' or 'false'
+ENABLE_WIDEC = true
+# Build breaks on Ubuntu else :(
+ENABLE_WIDEC_virtclass-native = false
+
+
+# Override the function from the autotools class; ncurses requires a
+# patched autoconf213 to generate the configure script. This autoconf
+# is not available so that the shipped script will be used.
+do_configure() {
+for i in \
+'narrowc --with-ticlib' \
+'widec   --enable-widec --without-progs'; do
+set -- $i
+mkdir -p $1
+cd $1
+shift
+
+oe_runconf \
+--disable-static \
+--without-debug \
+--without-ada \
+--enable-hard-tabs \
+--enable-xmc-glitch \
+--enable-colorfgbg \
+
--with-termpath='${sysconfdir}/termcap:${datadir}/misc/termcap' \
+
--with-terminfo-dirs='${sysconfdir}/terminfo:${datadir}/terminfo' \
+--with-shared \
+--disable-big-core \
+--program-prefix= \
+--with-termlib=tinfo \
+--enable-sigwinch \
+--enable-pc-files

Re: [oe] native glibc-initial-2.9 unpack error

2010-07-07 Thread Enrico Scholz
Khem Raj raj.k...@gmail.com writes:

 right. OE needed a fix too
 http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=5ac52371b09a25518b60e23e80c8c812cdea6e36

 which means that bitbake fix should also go
 into 1.8 an 1.10 to keep building oe .dev with those versions of bitbake.

I still think that adding a small fix to OE like

| if type == 'file' and host != '':
|   path = host + path

after the bb.decodeurl() call would be better than to require that
everybody uses a patched bitbake version.



Enrico

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


Re: [oe] native glibc-initial-2.9 unpack error

2010-07-06 Thread Enrico Scholz
Jan Paesmans jan.paesm...@gmail.com writes:

 cp: cannot create regular file `/ld.so.conf': Permission denied

bb.decodeurl (bitbake 1.10) seems to be buggy:

|  import bb
|  bb.decodeurl(file://ld.so.conf)
| ('file', '', 'ld.so.conf', '', '', {})
|  bb.decodeurl(file://etc/ld.so.conf)
| ('file', 'etc', '/ld.so.conf', '', '', {})
|  bb.decodeurl(file://etc/test/ld.so.conf)
| ('file', 'etc', '/test/ld.so.conf', '', '', {})



Enrico

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


Re: [oe] [PATCH v2] opkg recipe overhaul - removal of opkg-nogpg and opkg-nogpg-nocurl.

2010-07-01 Thread Enrico Scholz
Graham Gower graham.go...@gmail.com writes:

 None of the distros in OE 

At least the openmoko SHR distribution fetches ipkg from network and
requires curl support hence.


 appear to want/need gpg or curl support in opkg; so have opkg.inc
 remove them by default.

I do not think that supporting only offline mode (due to removal of
curl support) is a good idea.  Although the opportunistic signature
verification is not secure, it would be imo bad to remove gpg support
completely because updating embedded devices in a secure manner is
important.

A better way might be to conditionalize your '--disable-*' options
resp. make them depend on some DISTRO_FEATURE.


 +EXTRA_OECONF +=  --disable-gpg \ 
 +  --disable-openssl \ 
 +  --disable-ssl-curl \
 +  --disable-curl


Enrico

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


Re: [oe] [PATCH v2] opkg recipe overhaul - removal of opkg-nogpg and opkg-nogpg-nocurl.

2010-07-01 Thread Enrico Scholz
Koen Kooi k.k...@student.utwente.nl writes:

 A better way might be to conditionalize your '--disable-*' options
 resp. make them depend on some DISTRO_FEATURE.

 Introducing USE flags would be a step backwards, so NAK on that.

how would you reenable gpg and curl for https support then?


Enrico

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


[oe] [PATCH] binconfig.bbclass: take *-config from ${D}, not from ${S}

2010-06-24 Thread Enrico Scholz
It is not always right to take *-config files from source tree;
e.g. the buildsystem of recent ncurses takes an 'ncurses-config'
template and installs it as 'ncurses5-config' and 'ncursesw5-config'.

This patch uses the installed *-config files which are having the
expected name and content.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 classes/binconfig.bbclass |5 -
 1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/classes/binconfig.bbclass b/classes/binconfig.bbclass
index b3b2236..c1116fc 100644
--- a/classes/binconfig.bbclass
+++ b/classes/binconfig.bbclass
@@ -49,7 +49,10 @@ binconfig_package_preprocess () {
 SYSROOT_PREPROCESS_FUNCS += binconfig_sysroot_preprocess
 
 binconfig_sysroot_preprocess () {
-   for config in `find ${S} -name '${BINCONFIG_GLOB}'`; do
+   # take *-config from ${D}; Using ${PKGD} is not possible
+   # because it might not be populated yet and copies there will
+   # be altered by the function above.
+   for config in `find ${D} -name '${BINCONFIG_GLOB}'`; do
configname=`basename $config`
install -d ${SYSROOT_DESTDIR}${STAGING_BINDIR_CROSS}
cat $config | sed $...@get_binconfig_mangle(d)}  
${SYSROOT_DESTDIR}${STAGING_BINDIR_CROSS}/$configname
-- 
1.7.0.1


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


Re: [oe] [PATCH] ipkg-utils: convert mtime to int before comparing it

2010-06-21 Thread Enrico Scholz
Khem Raj raj.k...@gmail.com writes:

 On (15/06/10 15:03), Enrico Scholz wrote:
 The st_mtime attribute (which is a float) is compared against a value
 from the timestamp database, which was stored as an integer there.


 Is it stored as integer or long int? Did you try it on x86_64 box.

As 'int'. The ipkg-make-index-track-stamps.patch stores them as

+pkgsStamps[basename] = fnameStat.st_mtime
 ...
+pkgsStampsFile.write(%d %s\n % (pkgsStamps[f], f))

and reads them back as

+s, f = l.split( , 1)
+pkgsStamps[f] = int(s)


 -+  if pkgsStamps.has_key(basename) and fnameStat.st_mtime == 
 pkgsStamps[basename]:
 ++  if pkgsStamps.has_key(basename) and int(fnameStat.st_mtime) == 
 pkgsStamps[basename]:



Enrico

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


Re: [oe] Proposal to remove libattr recipe

2010-06-17 Thread Enrico Scholz
Henning Heinold hein...@inf.fu-berlin.de writes:

 the libattr from xfsprogs clashes with the attr recipe from udev and I
 can find any recipe which uses libattr, besides libcap2 which could
 use attr from udev too. So my proposal is to remove the libattr recipe
 and switch libcap2 to use attr from udev.

Because 'xfs' is upstream for attr + acl it would be probably better to
do the opposite thing (remove these general libraries from udev/ and
update the xfsprogs/ version).  Or, remove both versions and move the
best one into toplevel /attr + /acl.


Enrico

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


Re: [oe] We need newer ncurses Was: Host Environment affect on target builds

2010-06-05 Thread Enrico Scholz
Martin Jansa martin.ja...@gmail.com writes:

 BTW: 5.4 is 6years old, 5.7 ~2years old (latest)

 Is someone already working on newer ncurses recipes?

https://www.cvg.de/people/ensc/ncurses_5.7.bb

Problem with ncurses is, that it needs a patched autoconf 2.13 to do
the full autoconf step.  Recipe above uses shipped ./configure. Newer
ncurses need some termcap/tinfo related changes in multiple packages
too and I did not had time to clean them all up for submission.


Enrico

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


[oe] [PATCH] libvpx: disabled explicitly the generation of docs

2010-06-04 Thread Enrico Scholz
When buildsystem has 'doxygen' and 'php' the documentation will be
created and the path in do_install (vpx-vp8-nopost-nodocs) won't be
found anymore:

| make[2]: Leaving directory `.../libvpx-0.9.0-r2/libvpx-0.9.0'
| + cp -r '.../libvpx-0.9.0/vpx-vp8-nopost-nodocs*0.9.0/*' .../image/usr/
| cp: cannot stat `.../libvpx-0.9.0/vpx-vp8-nopost-nodocs*0.9.0/*': No such 
file or directory
| ERROR: Function do_install failed NOTE: package libvpx-0.9.0-r2: task 
do_install: Failed

This patch makes the build more deterministic.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
Cc: Koen Kooi k...@openembedded.org
---
 recipes/webm/libvpx_0.9.0.bb |3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/recipes/webm/libvpx_0.9.0.bb b/recipes/webm/libvpx_0.9.0.bb
index a59d94d..b55604b 100644
--- a/recipes/webm/libvpx_0.9.0.bb
+++ b/recipes/webm/libvpx_0.9.0.bb
@@ -1,7 +1,7 @@
 DESCRIPTION = vpx Multi-Format Codec SDK
 LICENSE = VP8
 
-PR = r2
+PR = r3
 
 SRC_URI = http://webm.googlecode.com/files/libvpx-${PV}.tar.bz2;
 
@@ -20,6 +20,7 @@ VPXTARGET ?= generic-gnu
 
 CONFIGUREOPTS =  \
   --target=${VPXTARGET} \
+  --disable-install_docs \
   --enable-vp8 \
   --enable-libs \
 
-- 
1.7.0.1


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


[oe] [PATCH] encodings/font-util: swapped inter-DEPENDS

2010-06-04 Thread Enrico Scholz
'encodings' uses font-util's pkg-config file to determine location of
the X11 font directory.  When 'font-util' is not staged yet (as it was
the case), it will use buildhost information which point to something
like /usr/share/X11 (which is outside of the staging dir) and which are
not writable.

This results into an empty 'encodings' package on first bitbake run in a
pristine tree; 'encodings' must be rebuilt so that it can use font-util
information from previous runs.

As 'font-util' itself does not require 'encodings' during build-time,
there were swapped the DEPENDS.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
Cc: David-John Willis john.wil...@distant-earth.com
Cc: Martin Jansa martin.ja...@gmail.com
---
 recipes/xorg-font/encodings_1.0.3.bb |4 ++--
 recipes/xorg-font/font-util_1.1.1.bb |4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/recipes/xorg-font/encodings_1.0.3.bb 
b/recipes/xorg-font/encodings_1.0.3.bb
index 168a67a..bc95836 100644
--- a/recipes/xorg-font/encodings_1.0.3.bb
+++ b/recipes/xorg-font/encodings_1.0.3.bb
@@ -1,8 +1,8 @@
 require xorg-font-common.inc
 PE = 1
-PR = ${INC_PR}.0
+PR = ${INC_PR}.1
 
-DEPENDS = mkfontscale-native font-util-native
+DEPENDS = mkfontscale-native font-util-native font-util
 
 SRC_URI[archive.md5sum] = 29637480e7ce12546668631067c19ced
 SRC_URI[archive.sha256sum] = 
07d9766d4aebe05ac572599f8f903a4fe06ab667a8182f2b6cc03e5fb3b80f53
diff --git a/recipes/xorg-font/font-util_1.1.1.bb 
b/recipes/xorg-font/font-util_1.1.1.bb
index f42b505..3f9b4c8 100644
--- a/recipes/xorg-font/font-util_1.1.1.bb
+++ b/recipes/xorg-font/font-util_1.1.1.bb
@@ -4,11 +4,11 @@ PACKAGE_ARCH = ${BASE_PACKAGE_ARCH}
 
 DESCRIPTION = X font utils.
 
-DEPENDS = encodings util-macros
+DEPENDS = util-macros
 RDEPENDS_${PN} = mkfontdir mkfontscale encodings
 
 PE = 1
-PR = ${INC_PR}.1
+PR = ${INC_PR}.2
 
 do_configure_prepend() {
 sed -i 
s#MAPFILES_PATH=\`pkg-config#MAPFILES_PATH=\`PKG_CONFIG_PATH=\${STAGING_LIBDIR_NATIVE}/pkg-config\
 pkg-config#g fontutil.m4.in
-- 
1.7.0.1


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


[oe] [PATCH 3/3] patch.bbclass: use oe_param_bool()

2010-05-27 Thread Enrico Scholz
This patch replaces a complicated if statement with a more simple
code.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 classes/patch.bbclass |   12 +---
 1 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/classes/patch.bbclass b/classes/patch.bbclass
index 7b0c444..8fe8c00 100644
--- a/classes/patch.bbclass
+++ b/classes/patch.bbclass
@@ -46,15 +46,13 @@ python patch_do_patch() {
local = os.path.join(workdir, base)
ext = os.path.splitext(base)[1]
 
-   if apply in parm:
-   apply = parm[apply]
-   if apply != yes:
-   if apply != no:
-   bb.msg.warn(None, Unsupported value 
'%s' for 'apply' url param in '%s', please use 'yes' or 'no' % (apply, url))
-   continue
+   is_patch = ext in (.diff, .patch)
+
+   if not oe_param_bool(parm, 'apply', is_patch):
+   continue
elif patch in parm:
bb.msg.warn(None, Deprecated usage of 'patch' url 
param in '%s', please use 'apply={yes,no}' % url)
-   elif ext not in (.diff, .patch):
+   elif not is_patch:
continue
 
if not local:
-- 
1.7.0.1


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


[oe] [PATCH 2/3] base.bbclass: fixed 'apply=no' behavior

2010-05-27 Thread Enrico Scholz
After the recent change to autodetect patchfiles in SRC_URI's, files
with 'apply=no' were not copied to workdir anymore.  This broke e.g.
dropbear which marks a patch as 'apply=no' and applies it manually.

This patch evaluates content of 'apply' parameter and copies files to
workdir when it is not a patch or the parameter is false.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 classes/base.bbclass |3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/classes/base.bbclass b/classes/base.bbclass
index 21b5646..f21d8ad 100644
--- a/classes/base.bbclass
+++ b/classes/base.bbclass
@@ -217,7 +217,8 @@ def oe_unpack_file(file, data, url = None):
cmd = 'cp -pPR %s %s/%s/' % (file, os.getcwd(), destdir)
else:
(type, host, path, user, pswd, parm) = bb.decodeurl(url)
-   if not 'apply' in parm and not 'patch' in parm:
+   is_patch = path.endswith('.diff') or path.endswith('.patch')
+   if not oe_param_bool(parm, 'apply', is_patch) and not 
oe_param_bool(parm, 'patch', False):
# The destdir handling was specifically done for 
FILESPATH
# items.  So, only do so for file:// entries.
if type == file:
-- 
1.7.0.1


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


[oe] [PATCH (rebase) 2/3] freetype: build with system zlib

2010-05-27 Thread Enrico Scholz
Problems mentioned in 4187d3df8838506f6578a2b0f2fb98f98c41e151 (Aug
2006) seem to went away and build succeeds with system zlib now.  Using
system libraries is preferable because it saves space and allows better
tracking of security issues.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/freetype/freetype_2.3.12.bb |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/recipes/freetype/freetype_2.3.12.bb 
b/recipes/freetype/freetype_2.3.12.bb
index fb57094..6e0641e 100644
--- a/recipes/freetype/freetype_2.3.12.bb
+++ b/recipes/freetype/freetype_2.3.12.bb
@@ -2,6 +2,7 @@ DESCRIPTION = Freetype font rendering library
 SECTION = libs
 LICENSE = freetype
 PR = r0
+DEPENDS = zlib
 
 SRC_URI = \
   ${SOURCEFORGE_MIRROR}/freetype/freetype-${PV}.tar.bz2 \
@@ -18,7 +19,6 @@ inherit autotools pkgconfig binconfig
 
 LIBTOOL = ${HOST_SYS}-libtool
 EXTRA_OEMAKE = 'LIBTOOL=${LIBTOOL}'
-EXTRA_OECONF = --without-zlib
 
 do_configure() {
cd builds/unix
-- 
1.7.0.1


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


[oe] [PATCH (rebase) 3/3] freetype: enable bytecode interpreter

2010-05-27 Thread Enrico Scholz
From: Enrico Scholz enrico.sch...@informatik.tu-chemnitz.de

Bytecode stuff is patent encumbered so it can be enabled for
non-enterprise distributions only.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 .../freetype-2.3.12/bytecode-interpreter.patch |   22 
 recipes/freetype/freetype_2.3.12.bb|1 +
 2 files changed, 23 insertions(+), 0 deletions(-)
 create mode 100644 recipes/freetype/freetype-2.3.12/bytecode-interpreter.patch

diff --git a/recipes/freetype/freetype-2.3.12/bytecode-interpreter.patch 
b/recipes/freetype/freetype-2.3.12/bytecode-interpreter.patch
new file mode 100644
index 000..599e67d
--- /dev/null
+++ b/recipes/freetype/freetype-2.3.12/bytecode-interpreter.patch
@@ -0,0 +1,22 @@
+Index: freetype-2.3.12/include/freetype/config/ftoption.h
+===
+--- freetype-2.3.12.orig/include/freetype/config/ftoption.h
 freetype-2.3.12/include/freetype/config/ftoption.h
+@@ -93,7 +93,7 @@ FT_BEGIN_HEADER
+   /* This is done to allow FreeType clients to run unmodified, forcing */
+   /* them to display normal gray-level anti-aliased glyphs.*/
+   /*   */
+-/* #define FT_CONFIG_OPTION_SUBPIXEL_RENDERING */
++#define FT_CONFIG_OPTION_SUBPIXEL_RENDERING
+ 
+ 
+   /*/
+@@ -497,7 +497,7 @@ FT_BEGIN_HEADER
+   /*   Do not #undef this macro here, since the build system might */
+   /*   define it for certain configurations only.  */
+   /*   */
+-/* #define TT_CONFIG_OPTION_BYTECODE_INTERPRETER */
++#define TT_CONFIG_OPTION_BYTECODE_INTERPRETER
+ 
+ 
+   /*/
diff --git a/recipes/freetype/freetype_2.3.12.bb 
b/recipes/freetype/freetype_2.3.12.bb
index 6e0641e..04e445c 100644
--- a/recipes/freetype/freetype_2.3.12.bb
+++ b/recipes/freetype/freetype_2.3.12.bb
@@ -9,6 +9,7 @@ SRC_URI = \
   file://no-hardcode.patch \
   file://fix-configure.patch \
   file://libtool-tag.patch \
+  
$...@base_conditional('ENTERPRISE_DISTRO','1','','file://bytecode-interpreter.patch',d)}
 \
 
 S = ${WORKDIR}/freetype-${PV}
 
-- 
1.7.0.1


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


[oe] [PATCH (rebase) 1/3] freetype: updated to 2.3.12

2010-05-27 Thread Enrico Scholz
That's (nearly) a copy of the old 2.3.11 recipe.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 .../freetype/freetype-2.3.12/fix-configure.patch   |   13 ++
 recipes/freetype/freetype-2.3.12/libtool-tag.patch |   20 ++
 recipes/freetype/freetype_2.3.12.bb|   40 
 3 files changed, 73 insertions(+), 0 deletions(-)
 create mode 100644 recipes/freetype/freetype-2.3.12/fix-configure.patch
 create mode 100644 recipes/freetype/freetype-2.3.12/libtool-tag.patch
 create mode 100644 recipes/freetype/freetype_2.3.12.bb

diff --git a/recipes/freetype/freetype-2.3.12/fix-configure.patch 
b/recipes/freetype/freetype-2.3.12/fix-configure.patch
new file mode 100644
index 000..78e76eb
--- /dev/null
+++ b/recipes/freetype/freetype-2.3.12/fix-configure.patch
@@ -0,0 +1,13 @@
+Index: freetype-2.3.6/builds/unix/configure.ac
+===
+--- freetype-2.3.6.orig/builds/unix/configure.ac
 freetype-2.3.6/builds/unix/configure.ac
+@@ -506,8 +506,6 @@ AC_SUBST([FT2_EXTRA_LIBS])
+ AC_SUBST([SYSTEM_ZLIB])
+
+
+-LT_INIT(win32-dll)
+-
+ AC_SUBST([hardcode_libdir_flag_spec])
+ AC_SUBST([wl])
+ AC_SUBST([build_libtool_libs])
diff --git a/recipes/freetype/freetype-2.3.12/libtool-tag.patch 
b/recipes/freetype/freetype-2.3.12/libtool-tag.patch
new file mode 100644
index 000..356a256
--- /dev/null
+++ b/recipes/freetype/freetype-2.3.12/libtool-tag.patch
@@ -0,0 +1,20 @@
+--- freetype-2.3.6/builds/unix/unix-cc.in
 freetype-2.3.6/builds/unix/unix-cc.in
+@@ -88,7 +88,7 @@
+ #
+ #
+ CCraw := $(CC)
+-CC:= $(LIBTOOL) --mode=compile $(CCraw)
++CC:= $(LIBTOOL) --tag=CC --mode=compile $(CCraw)
+
+ # Linker flags.
+ #
+@@ -105,7 +105,7 @@
+
+ # Library linking
+ #
+-LINK_LIBRARY = $(LIBTOOL) --mode=link $(CCraw) -o $@ $(OBJECTS_LIST) \
++LINK_LIBRARY = $(LIBTOOL) --tag=CC --mode=link $(CCraw) -o $@ $(OBJECTS_LIST) 
\
+   -rpath $(libdir) -version-info $(version_info) \
+   $(LDFLAGS) -no-undefined \
+   # -export-symbols $(EXPORTS_LIST)
diff --git a/recipes/freetype/freetype_2.3.12.bb 
b/recipes/freetype/freetype_2.3.12.bb
new file mode 100644
index 000..fb57094
--- /dev/null
+++ b/recipes/freetype/freetype_2.3.12.bb
@@ -0,0 +1,40 @@
+DESCRIPTION = Freetype font rendering library
+SECTION = libs
+LICENSE = freetype
+PR = r0
+
+SRC_URI = \
+  ${SOURCEFORGE_MIRROR}/freetype/freetype-${PV}.tar.bz2 \
+  file://no-hardcode.patch \
+  file://fix-configure.patch \
+  file://libtool-tag.patch \
+
+S = ${WORKDIR}/freetype-${PV}
+
+SRC_URI[md5sum] = e974a82e5939be8e05ee65f07275d7c5
+SRC_URI[sha256sum] = 
3b96438f016a62b676c1d2089c00ca777f710d19f6aefa66ccf068d360db3e92
+
+inherit autotools pkgconfig binconfig
+
+LIBTOOL = ${HOST_SYS}-libtool
+EXTRA_OEMAKE = 'LIBTOOL=${LIBTOOL}'
+EXTRA_OECONF = --without-zlib
+
+do_configure() {
+   cd builds/unix
+   libtoolize --force --copy
+   gnu-configize --force
+   aclocal -I .
+   autoconf
+   cd ${S}
+   oe_runconf
+}
+
+do_compile_prepend() {
+   ${BUILD_CC} -o objs/apinames src/tools/apinames.c
+}
+
+BBCLASSEXTEND = native
+
+FILES_${PN} = ${libdir}/lib*.so.*
+FILES_${PN}-dev += ${bindir}
-- 
1.7.0.1


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


Re: [oe] developmet setup

2010-05-27 Thread Enrico Scholz
Raffaele Recalcati lamiapost...@gmail.com writes:

 I was searching a clean way to develop in OE my new kernel and my new
 applications.

I wrote a recipe[1] which generates a special 'Makefile'[2].  It can be
used for developing an application in a way like

| $ make -C ~/src/my-app -f workspace/Makefile.ipan7 all
| $ make -f workspace/Makefile.ipan7 shell

and sets all the crosscompiler information.  There exist some extensions
for U-Boot[3] or common tasks like pkg-installation in NFS
rootfilesystems[4]. Sample commands are

| $ make -C ~/src/u-boot -f workspace/Makefile.ipan7 CFG=u-boot

or 

| $ make -f workspace/Makefile.ipan7 CFG=pkg pkg-install P=busybox
| $ make -f workspace/Makefile.ipan7 CFG=pkg pkg-upgrade


We started with a variation for the kernel which was placed into a
kernel recipe[5] and generates a more simple makefile[6]. It should be
probably ported to the previous, more modern method but it works and
I do not have time for it :(


Our development model is to work on NFS rootfs where applications are
installed on by the methods above. There are OE recipes for these apps
which fetch either from git repositories or fetch normal tarballs of
these apps.



Enrico

Footnotes: 
[1]  https://www.cvg.de/people/ensc/oe/elito-develcomp.bb
[2]  https://www.cvg.de/people/ensc/oe/Makefile.ipan7
[3]  https://www.cvg.de/people/ensc/oe/u-boot.mk
[4]  https://www.cvg.de/people/ensc/oe/pkg.mk
[5]  https://www.cvg.de/people/ensc/oe/elito-kernel.inc
[6]  https://www.cvg.de/people/ensc/oe/Makefile.kernel.ipan7

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


[oe] [PATCH] packaged-staging.bbclass: added missing space in find call

2010-05-23 Thread Enrico Scholz
Old find statement failed with something like

  $ find . \( -name XXX\)
  find: invalid expression; I was expecting to find a ')' somewhere but did not 
see one.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 classes/packaged-staging.bbclass |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/classes/packaged-staging.bbclass b/classes/packaged-staging.bbclass
index 62a78e4..83f3578 100644
--- a/classes/packaged-staging.bbclass
+++ b/classes/packaged-staging.bbclass
@@ -19,7 +19,7 @@ PSTAGE_PKGPN  = 
$...@bb.data.expand('staging-${PN}-${MULTIMACH_ARCH}${TARGET_
 PSTAGE_PKGNAME= 
${PSTAGE_PKGPN}_${PSTAGE_PKGVERSION}_${PSTAGE_PKGARCH}.ipk
 PSTAGE_PKG= ${PSTAGE_DIR}/${PSTAGE_PKGPATH}/${PSTAGE_PKGNAME}
 PSTAGE_WORKDIR   = ${TMPDIR}/pstage
-PSTAGE_SCAN_CMD ?= find ${PSTAGE_TMPDIR_STAGE} \( -name *.la -o -name 
*-config\) -type f
+PSTAGE_SCAN_CMD ?= find ${PSTAGE_TMPDIR_STAGE} \( -name *.la -o -name 
*-config \) -type f
 
 PSTAGE_NATIVEDEPENDS = \
 shasum-native \
-- 
1.7.0.1


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


[oe] [PATCH (updated)] autotools.bbclass: enhanced oe_runconf() to accept quoted arguments

2010-05-23 Thread Enrico Scholz
This patch allows to pass quoted strings in EXTRA_OECONF.  E.g. with
this patch, it is possible to do

| EXTRA_OECONF = --with-build-cflags='${BUILD_CFLAGS}'

where 'BUILD_CFLAGS' contains multiple, whitespace separated arguments.

Such flags can be also passed to oe_runconf() directly:

| oe_runconf --with-build-cflags=${BUILD_CFLAGS}

This patch might create incompatibilities for recipes which workaround
the
current limitations.

As a sideeffect, 'set -x' debug output is now much nicer because
superflous whitespaces in the configure call are removed.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 classes/autotools.bbclass |8 
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/classes/autotools.bbclass b/classes/autotools.bbclass
index 9bb4f6c..2e53cc6 100644
--- a/classes/autotools.bbclass
+++ b/classes/autotools.bbclass
@@ -64,12 +64,12 @@ CONFIGUREOPTS =  --build=${BUILD_SYS} \
 
 oe_runconf () {
if [ -x ${S}/configure ] ; then
-   cfgcmd=${S}/configure \
+   set -- ${S}/configure \
${CONFIGUREOPTS} \
${EXTRA_OECONF} \
-   $@
-   oenote Running $cfgcmd...
-   $cfgcmd || oefatal oe_runconf failed 
+   $@
+   oenote Running $*...
+   $@ || oefatal oe_runconf failed 
else
oefatal no configure script found
fi
-- 
1.7.0.1


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


Re: [oe] [PATCH (updated)] autotools.bbclass: enhanced oe_runconf() to accept quoted arguments

2010-05-23 Thread Enrico Scholz
Chris Larson clar...@kergoth.com writes:

  This patch allows to pass quoted strings in EXTRA_OECONF.

 I fail to see how this is a problem.  Seems to work fine for every
 other app, including make.

EXTRA_OEMAKE handling is implemented as

| ${MAKE} ${EXTRA_OEMAKE} $@

but EXTRA_OECONF as

| cfgcmd=... ${EXTRA_OECONF} $@
| $cfgcmd


'make' arguments are expanded by bitbake which preserve the quoting.

The EXTRA_OECONF content was put into a shell variable and this variable
be executed.  This does not preserve quoting (and I am not aware of any
way how this can be with variable assignments).


 Is configure's argument passing that stupid, that it can't split by =
 and use the right side for the value?

It is not a configure issue, but a shell one.


Enrico

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


Re: [oe] [PATCH (updated)] autotools.bbclass: enhanced oe_runconf() to accept quoted arguments

2010-05-23 Thread Enrico Scholz
Chris Larson clar...@kergoth.com writes:

 Ahh, I see, that makes it clear, thanks for clarifying.  I have a local
 patch that turns cfgcmd into a bitbake variable, and runs ${cfgcmd}
 $@.
 What do you think about that approach as an alternative?  

It will work...  I do not know the story behind 'cfgcmd' but I guess it
was added to avoid code duplication in 'oenote Running $cfgcmd...'.

This 'oenote' is probably useful for debugging.  But for me, tracing
program invocations is such an elementary instrument to analyze problems
that I patched[1] 'bitbake' some time ago to enable shell monitoring
globally.

Without this (for me redundant) 'oenote' message, we could call ./configure
directly (like in oe_runmake) without any shell or python helper variables.




Enrico

Footnotes: 
[1]  http://www.ensc.de/bitbake/0003-trace.patch
 -- shell calls will be recorded in logfile

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


[oe] [PATCH 2/3] freetype: build with system zlib

2010-05-23 Thread Enrico Scholz
Problems mentioned in 4187d3df8838506f6578a2b0f2fb98f98c41e151 (Aug
2006) seem to went away and build succeeds with system zlib now.  Using
system libraries is preferable because it saves space and allows better
tracking of security issues.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 recipes/freetype/freetype_2.3.12.bb |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/recipes/freetype/freetype_2.3.12.bb 
b/recipes/freetype/freetype_2.3.12.bb
index de9ff29..b1f8d6f 100644
--- a/recipes/freetype/freetype_2.3.12.bb
+++ b/recipes/freetype/freetype_2.3.12.bb
@@ -1,7 +1,8 @@
 DESCRIPTION = Freetype font rendering library
 SECTION = libs
 LICENSE = freetype
-PR = r0
+PR = r1
+DEPENDS = zlib
 
 SRC_URI = \
   ${SOURCEFORGE_MIRROR}/freetype/freetype-${PV}.tar.bz2 \
@@ -18,7 +19,6 @@ inherit autotools pkgconfig binconfig
 
 LIBTOOL = ${HOST_SYS}-libtool
 EXTRA_OEMAKE = 'LIBTOOL=${LIBTOOL}'
-EXTRA_OECONF = --without-zlib
 
 do_configure() {
cd builds/unix
-- 
1.7.0.1


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


[oe] [PATCH 3/3] freetype: enable bytecode interpreter

2010-05-23 Thread Enrico Scholz
From: Enrico Scholz enrico.sch...@informatik.tu-chemnitz.de

Bytecode stuff is patent encumbered so it can be enabled for
non-enterprise distributions only.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 .../freetype-2.3.12/bytecode-interpreter.patch |   22 
 recipes/freetype/freetype_2.3.12.bb|1 +
 2 files changed, 23 insertions(+), 0 deletions(-)
 create mode 100644 recipes/freetype/freetype-2.3.12/bytecode-interpreter.patch

diff --git a/recipes/freetype/freetype-2.3.12/bytecode-interpreter.patch 
b/recipes/freetype/freetype-2.3.12/bytecode-interpreter.patch
new file mode 100644
index 000..599e67d
--- /dev/null
+++ b/recipes/freetype/freetype-2.3.12/bytecode-interpreter.patch
@@ -0,0 +1,22 @@
+Index: freetype-2.3.12/include/freetype/config/ftoption.h
+===
+--- freetype-2.3.12.orig/include/freetype/config/ftoption.h
 freetype-2.3.12/include/freetype/config/ftoption.h
+@@ -93,7 +93,7 @@ FT_BEGIN_HEADER
+   /* This is done to allow FreeType clients to run unmodified, forcing */
+   /* them to display normal gray-level anti-aliased glyphs.*/
+   /*   */
+-/* #define FT_CONFIG_OPTION_SUBPIXEL_RENDERING */
++#define FT_CONFIG_OPTION_SUBPIXEL_RENDERING
+ 
+ 
+   /*/
+@@ -497,7 +497,7 @@ FT_BEGIN_HEADER
+   /*   Do not #undef this macro here, since the build system might */
+   /*   define it for certain configurations only.  */
+   /*   */
+-/* #define TT_CONFIG_OPTION_BYTECODE_INTERPRETER */
++#define TT_CONFIG_OPTION_BYTECODE_INTERPRETER
+ 
+ 
+   /*/
diff --git a/recipes/freetype/freetype_2.3.12.bb 
b/recipes/freetype/freetype_2.3.12.bb
index b1f8d6f..53372fc 100644
--- a/recipes/freetype/freetype_2.3.12.bb
+++ b/recipes/freetype/freetype_2.3.12.bb
@@ -9,6 +9,7 @@ SRC_URI = \
   file://no-hardcode.patch;patch=1 \
   file://fix-configure.patch;patch=1 \
   file://libtool-tag.patch;patch=1 \
+  
$...@base_conditional('ENTERPRISE_DISTRO','1','','file://bytecode-interpreter.patch;patch=1',d)}
 \
 
 S = ${WORKDIR}/freetype-${PV}
 
-- 
1.7.0.1


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


[oe] [PATCH 1/3] freetype: updated to 2.3.12

2010-05-23 Thread Enrico Scholz
That's (nearly) a copy of the old 2.3.11 recipe.

Signed-off-by: Enrico Scholz enrico.sch...@sigma-chemnitz.de
---
 .../freetype/freetype-2.3.12/fix-configure.patch   |   13 ++
 recipes/freetype/freetype-2.3.12/libtool-tag.patch |   20 ++
 recipes/freetype/freetype_2.3.12.bb|   40 
 3 files changed, 73 insertions(+), 0 deletions(-)
 create mode 100644 recipes/freetype/freetype-2.3.12/fix-configure.patch
 create mode 100644 recipes/freetype/freetype-2.3.12/libtool-tag.patch
 create mode 100644 recipes/freetype/freetype_2.3.12.bb

diff --git a/recipes/freetype/freetype-2.3.12/fix-configure.patch 
b/recipes/freetype/freetype-2.3.12/fix-configure.patch
new file mode 100644
index 000..ecd9673
--- /dev/null
+++ b/recipes/freetype/freetype-2.3.12/fix-configure.patch
@@ -0,0 +1,13 @@
+Index: freetype-2.3.6/builds/unix/configure.ac
+===
+--- freetype-2.3.6.orig/builds/unix/configure.ac
 freetype-2.3.6/builds/unix/configure.ac
+@@ -506,8 +506,6 @@ AC_SUBST([FT2_EXTRA_LIBS])
+ AC_SUBST([SYSTEM_ZLIB])
+ 
+ 
+-LT_INIT(win32-dll)
+-
+ AC_SUBST([hardcode_libdir_flag_spec])
+ AC_SUBST([wl])
+ AC_SUBST([build_libtool_libs])
diff --git a/recipes/freetype/freetype-2.3.12/libtool-tag.patch 
b/recipes/freetype/freetype-2.3.12/libtool-tag.patch
new file mode 100644
index 000..2f096cd
--- /dev/null
+++ b/recipes/freetype/freetype-2.3.12/libtool-tag.patch
@@ -0,0 +1,20 @@
+--- freetype-2.3.6/builds/unix/unix-cc.in  
 freetype-2.3.6/builds/unix/unix-cc.in  
+@@ -88,7 +88,7 @@ 
+ #
+ #
+ CCraw := $(CC)
+-CC:= $(LIBTOOL) --mode=compile $(CCraw)
++CC:= $(LIBTOOL) --tag=CC --mode=compile $(CCraw)
+ 
+ # Linker flags.
+ #
+@@ -105,7 +105,7 @@ 
+ 
+ # Library linking
+ #
+-LINK_LIBRARY = $(LIBTOOL) --mode=link $(CCraw) -o $@ $(OBJECTS_LIST) \
++LINK_LIBRARY = $(LIBTOOL) --tag=CC --mode=link $(CCraw) -o $@ $(OBJECTS_LIST) 
\
+   -rpath $(libdir) -version-info $(version_info) \
+   $(LDFLAGS) -no-undefined \
+   # -export-symbols $(EXPORTS_LIST)
diff --git a/recipes/freetype/freetype_2.3.12.bb 
b/recipes/freetype/freetype_2.3.12.bb
new file mode 100644
index 000..de9ff29
--- /dev/null
+++ b/recipes/freetype/freetype_2.3.12.bb
@@ -0,0 +1,40 @@
+DESCRIPTION = Freetype font rendering library
+SECTION = libs
+LICENSE = freetype
+PR = r0
+
+SRC_URI = \
+  ${SOURCEFORGE_MIRROR}/freetype/freetype-${PV}.tar.bz2 \
+  file://no-hardcode.patch;patch=1 \
+  file://fix-configure.patch;patch=1 \
+  file://libtool-tag.patch;patch=1 \
+
+S = ${WORKDIR}/freetype-${PV}
+
+SRC_URI[md5sum] = e974a82e5939be8e05ee65f07275d7c5
+SRC_URI[sha256sum] = 
3b96438f016a62b676c1d2089c00ca777f710d19f6aefa66ccf068d360db3e92
+
+inherit autotools pkgconfig binconfig
+
+LIBTOOL = ${HOST_SYS}-libtool
+EXTRA_OEMAKE = 'LIBTOOL=${LIBTOOL}'
+EXTRA_OECONF = --without-zlib
+
+do_configure() {
+   cd builds/unix
+   libtoolize --force --copy
+   gnu-configize --force
+   aclocal -I .
+   autoconf
+   cd ${S}
+   oe_runconf
+}
+
+do_compile_prepend() {
+   ${BUILD_CC} -o objs/apinames src/tools/apinames.c
+}
+
+BBCLASSEXTEND = native
+
+FILES_${PN} = ${libdir}/lib*.so.*
+FILES_${PN}-dev += ${bindir}
-- 
1.7.0.1


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


Re: [oe] [PATCH 3/3] freetype: enable bytecode interpreter

2010-05-23 Thread Enrico Scholz
pieterg piet...@gmx.com writes:

 Bytecode stuff is patent encumbered so it can be enabled for
 non-enterprise distributions only.

 The bytecode patents have expired, 

afaik (I am not a lawer) that's only true for the second part
(TT_CONFIG_OPTION_BYTECODE_INTERPRETER) of the patch.  The ClearType
patents (which are violated by FT_CONFIG_OPTION_SUBPIXEL_RENDERING) were
filed in 1999 and are probably still valid.

I should have been clearer in the patch description and mention that it
enables also subpixel rendering.


Enrico

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


Re: [oe] [PATCH 1/2] kernel bbclass: convert to new style staging (from poky)

2010-04-17 Thread Enrico Scholz
Koen Kooi koen.k...@gmail.com writes:

   if [ -e include/asm ] ; then
   # This link is generated only in kernel before 2.6.33-rc1, 
 don't stage it for newer kernels
   ASMDIR=`readlink include/asm`
  
 - mkdir -p ${STAGING_KERNEL_DIR}/include/$ASMDIR
 - cp -fR include/$ASMDIR/* ${STAGING_KERNEL_DIR}/include/$ASMDIR/
 + mkdir -p $kerneldir/include/$ASMDIR
 + cp -fR include/$ASMDIR/* $kerneldir/include/$ASMDIR/
   fi
 +
 + cp -fR include/$ASMDIR/* $kerneldir/include/$ASMDIR/

this breaks on recent kernels because they do no have include/asm and
ASMDIR won't be defined here.


   if [ -e arch/${ARCH}/include/asm/ ] ; then 
 - if [ -e include/asm ] ; then
 - cp -fR arch/${ARCH}/include/asm/* 
 ${STAGING_KERNEL_DIR}/include/$ASMDIR/
 - fi
 - install -d ${STAGING_KERNEL_DIR}/arch/${ARCH}/include
 - cp -fR arch/${ARCH}/* ${STAGING_KERNEL_DIR}/arch/${ARCH}/   
 + cp -fR arch/${ARCH}/include/asm/* $kerneldir/include/$ASMDIR/

ditto



Enrico

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


Re: [oe] [PATCH] dbus: use $(LIBTOOL) to install 'dbus-daemon'

2010-04-15 Thread Enrico Scholz
Sebastian Spaeth sebast...@sspaeth.de writes:

 Changing Makefile.am to use native automake methods to install this
 files would be a better but more invasive fix.

 Looks reasonable, although I am always for going with the better fixes
 in general :-)

I will work on a better patch which will be sent to upstream.  But I can
not say when I have time to work on it.


Enrico

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


[oe] [PATCH] dbus: use $(LIBTOOL) to install 'dbus-daemon'

2010-04-14 Thread Enrico Scholz
Depending on autotools/libtool version and configure flags, dbus-daemon
will be built sometimes below .libs/ and sometimes in base directory.

Using 'libtool --mode=install' is a portable way to deal with this.

Changing Makefile.am to use native automake methods to install this
files would be a better but more invasive fix.
---
 recipes/dbus/dbus-1.2.24/fix-install-daemon.patch |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/recipes/dbus/dbus-1.2.24/fix-install-daemon.patch 
b/recipes/dbus/dbus-1.2.24/fix-install-daemon.patch
index c317863..5e25d12 100644
--- a/recipes/dbus/dbus-1.2.24/fix-install-daemon.patch
+++ b/recipes/dbus/dbus-1.2.24/fix-install-daemon.patch
@@ -7,7 +7,7 @@ Index: dbus-0.94/bus/Makefile.am
chmod 755 $(DESTDIR)$(DBUS_DAEMONDIR); \
fi
 -  $(INSTALL_PROGRAM) dbus-daemon $(DESTDIR)$(DBUS_DAEMONDIR)
-+  $(INSTALL_PROGRAM) .libs/dbus-daemon $(DESTDIR)$(DBUS_DAEMONDIR)
++  $(LIBTOOL) --mode=install $(INSTALL_PROGRAM) dbus-daemon 
$(DESTDIR)$(DBUS_DAEMONDIR)
$(mkinstalldirs) $(DESTDIR)$(localstatedir)/run/dbus
$(mkinstalldirs) $(DESTDIR)$(configdir)/system.d
$(mkinstalldirs) $(DESTDIR)$(datadir)/dbus-1/services
-- 
1.7.0.1


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


[oe] [PATCH] package.bbclass: search for dangling links in installation directory

2010-04-14 Thread Enrico Scholz
The old dangling link detection used os.stat() on the file and
registered it as a dangling link when it was not found. This causes
problems when the link is an absolute one because it will check
files of the host system.

E.g. when there is a link

.../tmp/work/../etc/cron.root - /var/spool/cron/root

Then

* existence of /var/spool/cron/root but not of
  ../tmp/work/.../var/spool/cron/root will be checked

* the build aborts because a 'permission denied' exception is thrown
  but only 'not found' be catched

This patch uses os.lstat() to check whether source file is a link, adds
it to the installation root direction and checks for the existence of
this file then.

As this is only a QA mechanism which does not affect the resulting
package, there is checked for generic OSErrors only, not only for
ENOENT.  Other problems (e.g. EPERM) will be reported by other
functions.
---
 classes/package.bbclass |   14 --
 1 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/classes/package.bbclass b/classes/package.bbclass
index 990c3f6..ae8e92e 100644
--- a/classes/package.bbclass
+++ b/classes/package.bbclass
@@ -482,15 +482,17 @@ python populate_packages () {
path = os.path.join(root, f)
rpath = path[len(inst_root):]
pkg_files[pkg].append(rpath)
-   try:
-   s = os.stat(path)
-   except OSError, (err, strerror):
-   if err != errno.ENOENT:
-   raise
+   if os.path.islink(path):
target = os.readlink(path)
if target[0] != '/':
target = 
os.path.join(root[len(inst_root):], target)
-   
dangling_links[pkg].append(os.path.normpath(target))
+
+   rtarget = os.path.join(inst_root, 
target[1:])
+
+   try:
+   os.lstat(rtarget)
+   except OSError:
+   
dangling_links[pkg].append(os.path.normpath(target))
 
for pkg in package_list:
rdepends = explode_deps(bb.data.getVar('RDEPENDS_' + pkg, d, 0) 
or bb.data.getVar('RDEPENDS', d, 0) or )
-- 
1.7.0.1


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


  1   2   >