Re: [oe] [meta-oe][PATCH v2] lvm2: Add dependency on bash

2013-07-30 Thread Paul Eggleton
On Friday 26 July 2013 19:43:45 Martin Jansa wrote:
 On Fri, Jul 26, 2013 at 05:25:29PM +0300, Mihai Prica wrote:
  Fixes rootfs generation error if bash is not included in the image.
 
 With rpm right?

If it's what you're suggesting technically it's broken with any package 
backend, to some degree.
 
 Find which script it has in shebang and add bash only to that package
 RDEPENDS, if it's something which isn't always needed put it in separate
 package.

Or if practical patch the script(s) to remove the bashisms and change the 
shebang to #!/bin/sh instead.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [meta-oe][PATCH 1/2] xinput-calibrator: remove

2013-07-30 Thread Paul Eggleton
This is now part of OE-Core.

Signed-off-by: Paul Eggleton paul.eggle...@linux.intel.com
---
 .../xinput-calibrator/xinput-calibrator.service| 11 --
 .../xinput-calibrator/xinput-calibrator_git.bb | 42 --
 2 files changed, 53 deletions(-)
 delete mode 100644 
meta-oe/recipes-graphics/xinput-calibrator/xinput-calibrator/xinput-calibrator.service
 delete mode 100644 
meta-oe/recipes-graphics/xinput-calibrator/xinput-calibrator_git.bb

diff --git 
a/meta-oe/recipes-graphics/xinput-calibrator/xinput-calibrator/xinput-calibrator.service
 
b/meta-oe/recipes-graphics/xinput-calibrator/xinput-calibrator/xinput-calibrator.service
deleted file mode 100644
index 2301876..000
--- 
a/meta-oe/recipes-graphics/xinput-calibrator/xinput-calibrator/xinput-calibrator.service
+++ /dev/null
@@ -1,11 +0,0 @@
-[Unit]
-Description=Loads xinput calibration data
-ConditionPathExists=/etc/pointercal.xinput
-Requires=display-manager.service
-After=display-manager.service
-
-[Service]
-ExecStart=/usr/bin/xinput_calibrator_once.sh
-
-[Install]
-WantedBy=multi-user.target
diff --git 
a/meta-oe/recipes-graphics/xinput-calibrator/xinput-calibrator_git.bb 
b/meta-oe/recipes-graphics/xinput-calibrator/xinput-calibrator_git.bb
deleted file mode 100644
index 640aef1..000
--- a/meta-oe/recipes-graphics/xinput-calibrator/xinput-calibrator_git.bb
+++ /dev/null
@@ -1,42 +0,0 @@
-DESCRIPTION = A generic touchscreen calibration program for X.Org
-HOMEPAGE = http://www.freedesktop.org/wiki/Software/xinput_calibrator;
-LICENSE = MIT-X
-LIC_FILES_CHKSUM = 
file://src/calibrator.cpp;endline=22;md5=1bcba08f67cdb56f34021557898e4b5a
-DEPENDS = virtual/libx11 libxi
-
-PV = 0.7.5+git${SRCPV}
-PR = r6
-
-inherit autotools systemd
-
-SRCREV = c01c5af807cb4b0157b882ab07a893df9a810111
-SRC_URI = git://github.com/tias/xinput_calibrator.git;protocol=git \
-   file://xinput-calibrator.service
-
-S = ${WORKDIR}/git
-
-# force native X11 ui as we don't have gtk+ in DEPENDS
-EXTRA_OECONF += --with-gui=x11
-
-do_install_append() {
-install -d ${D}${bindir}
-install -m 0755 scripts/xinput_calibrator_pointercal.sh 
${D}${bindir}/xinput_calibrator_once.sh
-
-install -d ${D}${sysconfdir}/xdg/autostart
-sed -i -e 's,^Exec=.*,Exec=${bindir}/xinput_calibrator_once.sh,' 
scripts/xinput_calibrator.desktop
-install -m 0644 scripts/xinput_calibrator.desktop 
${D}${sysconfdir}/xdg/autostart
-
-if ${@base_contains('DISTRO_FEATURES','systemd','true','false',d)}; then
-install -d ${D}${systemd_unitdir}/system
-install -m 0644 ${WORKDIR}/xinput-calibrator.service 
${D}${systemd_unitdir}/system
-fi
-}
-
-FILES_${PN} += ${sysconfdir}/xdg/autostart
-RDEPENDS_${PN} = xinput xterm
-RRECOMMENDS_${PN} = pointercal-xinput
-
-RPROVIDES_${PN} += ${PN}-systemd
-RREPLACES_${PN} += ${PN}-systemd
-RCONFLICTS_${PN} += ${PN}-systemd
-SYSTEMD_SERVICE_${PN} = ${PN}.service
-- 
1.8.1.2

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


[oe] [meta-oe][PATCH 0/2] Remove recipes that are now in OE-Core

2013-07-30 Thread Paul Eggleton
The following changes since commit 47fc1cce49936e0a1c9f083172a7fd45c7434aac:

  connman: Rename bbappend to match new version from oe-core (2013-07-30 
11:35:03 +0200)

are available in the git repository at:

  git://git.openembedded.org/meta-openembedded-contrib paule/xinput-calibrator
  
http://cgit.openembedded.org/cgit.cgi/meta-openembedded-contrib/log/?h=paule/xinput-calibrator

Paul Eggleton (2):
  xinput-calibrator: remove
  pointercal-xinput: remove

 .../pointercal-xinput/pointercal.xinput|  1 -
 .../xinput-calibrator/pointercal-xinput_0.0.bb | 19 --
 .../xinput-calibrator/xinput-calibrator.service| 11 --
 .../xinput-calibrator/xinput-calibrator_git.bb | 42 --
 4 files changed, 73 deletions(-)
 delete mode 100644 
meta-oe/recipes-graphics/xinput-calibrator/pointercal-xinput/pointercal.xinput
 delete mode 100644 
meta-oe/recipes-graphics/xinput-calibrator/pointercal-xinput_0.0.bb
 delete mode 100644 
meta-oe/recipes-graphics/xinput-calibrator/xinput-calibrator/xinput-calibrator.service
 delete mode 100644 
meta-oe/recipes-graphics/xinput-calibrator/xinput-calibrator_git.bb

-- 
1.8.1.2

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


[oe] [meta-oe][PATCH 2/2] pointercal-xinput: remove

2013-07-30 Thread Paul Eggleton
This is now part of OE-Core.

Signed-off-by: Paul Eggleton paul.eggle...@linux.intel.com
---
 .../pointercal-xinput/pointercal.xinput   |  1 -
 .../xinput-calibrator/pointercal-xinput_0.0.bb| 19 ---
 2 files changed, 20 deletions(-)
 delete mode 100644 
meta-oe/recipes-graphics/xinput-calibrator/pointercal-xinput/pointercal.xinput
 delete mode 100644 
meta-oe/recipes-graphics/xinput-calibrator/pointercal-xinput_0.0.bb

diff --git 
a/meta-oe/recipes-graphics/xinput-calibrator/pointercal-xinput/pointercal.xinput
 
b/meta-oe/recipes-graphics/xinput-calibrator/pointercal-xinput/pointercal.xinput
deleted file mode 100644
index 9633fc5..000
--- 
a/meta-oe/recipes-graphics/xinput-calibrator/pointercal-xinput/pointercal.xinput
+++ /dev/null
@@ -1 +0,0 @@
-# replace with valid machine specific pointercal.xinput
diff --git 
a/meta-oe/recipes-graphics/xinput-calibrator/pointercal-xinput_0.0.bb 
b/meta-oe/recipes-graphics/xinput-calibrator/pointercal-xinput_0.0.bb
deleted file mode 100644
index bfb952f..000
--- a/meta-oe/recipes-graphics/xinput-calibrator/pointercal-xinput_0.0.bb
+++ /dev/null
@@ -1,19 +0,0 @@
-DESCRIPTION = Touchscreen calibration data from xinput-calibrator
-LICENSE = MIT
-LIC_FILES_CHKSUM = 
file://${COREBASE}/meta/COPYING.MIT;md5=3da9cfbcb788c80a0384361b4de20420
-
-PR = r4
-
-SRC_URI = file://pointercal.xinput
-S = ${WORKDIR}
-
-do_install() {
-# Only install file if it has a contents
-if [ -s ${S}/pointercal.xinput ]; then
-install -d ${D}${sysconfdir}/
-install -m 0644 ${S}/pointercal.xinput ${D}${sysconfdir}/
-fi
-}
-
-PACKAGE_ARCH = ${MACHINE_ARCH}
-CONFFILES_${PN} = ${sysconfdir}/pointercal.xinput
-- 
1.8.1.2

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


Re: [oe] [meta-oe][PATCH] iperf: Fix license

2013-07-30 Thread Paul Barker
On 29 July 2013 10:45, Guo Chunrong-B40290 b40...@freescale.com wrote:
 Hello,

 I'd appreciate if someone can review.


I'll take a quick look...

 -LICENSE = NewBSD
 +LICENSE = BSD

Comparing http://sourceforge.net/p/iperf/code/82/tree/trunk/COPYING
and the common licenses in openembedded-core, it looks like
BSD-2-Clause would be more accurate. Could you take a look and let
me know if you agree? Currently on the move so can't diff the files
myself.

-- 
Paul Barker

Email: p...@paulbarker.me.uk
http://www.paulbarker.me.uk
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH v2] lvm2: Add dependency on bash

2013-07-30 Thread Prica, Mihai
 -Original Message-
 From: Paul Eggleton [mailto:paul.eggle...@linux.intel.com]
 
 On Friday 26 July 2013 19:43:45 Martin Jansa wrote:
  On Fri, Jul 26, 2013 at 05:25:29PM +0300, Mihai Prica wrote:
   Fixes rootfs generation error if bash is not included in the image.
 
  With rpm right?
 
 If it's what you're suggesting technically it's broken with any package 
 backend, to
 some degree.

For the tests I used rpm.

 
  Find which script it has in shebang and add bash only to that package
  RDEPENDS, if it's something which isn't always needed put it in
  separate package.
 
 Or if practical patch the script(s) to remove the bashisms and change the 
 shebang
 to #!/bin/sh instead.

I've looked over the scripts and there seem to be 4 files which are using bash. 
The files 
are part of the core lvm2 package. I've used checkbashism and there are not 
many 
changes that need to be done to switch to /bin/sh. I'll send an updated patch. 

Thanks,
Mihai

 
 Cheers,
 Paul
 
 --
 
 Paul Eggleton
 Intel Open Source Technology Centre
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-networking][PATCH] ebtables: Load ebtables kernel module

2013-07-30 Thread Joe MacDonald
Merged, thanks.
-J.

[[oe] [meta-networking][PATCH] ebtables: Load ebtables kernel module] On 
13.07.26 (Fri 17:20) Mihai Prica wrote:

 Ebtables needs some kernel configs options to work properly. The standard 
 yocto
 kernel config includes these as modules. Add code to load the ebtables module.
 
 Signed-off-by: Mihai Prica mihai.pr...@intel.com
 ---
  .../recipes-filter/ebtables/ebtables_2.0.10-4.bb   |5 -
  1 file changed, 4 insertions(+), 1 deletion(-)
 
 diff --git a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb 
 b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
 index 78e069a..6ac886e 100644
 --- a/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
 +++ b/meta-networking/recipes-filter/ebtables/ebtables_2.0.10-4.bb
 @@ -2,10 +2,13 @@ SUMMARY = Utility for basic Ethernet frame filtering on a 
 Linux bridge, MAC NAT
  LICENSE = GPLv2
  LIC_FILES_CHKSUM = file://COPYING;md5=53b4a93871a28ab1488fdbd2e73e
  SECTION = console/network
 -PR = r2
 +PR = r3
  
  RDEPENDS_${PN} += perl
  
 +RRECOMMENDS_${PN} += kernel-module-ebtables \
 +
 +
  SRC_URI = ${SOURCEFORGE_MIRROR}/ebtables/ebtables-v${PV}.tar.gz \
 file://installnonroot.patch \
 file://01debian_defaultconfig.patch \
-- 
-Joe MacDonald.
:wq


signature.asc
Description: Digital signature
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH v2] lvm2: Add dependency on bash

2013-07-30 Thread Martin Jansa
On Tue, Jul 30, 2013 at 12:15:19PM +0100, Paul Eggleton wrote:
 On Friday 26 July 2013 19:43:45 Martin Jansa wrote:
  On Fri, Jul 26, 2013 at 05:25:29PM +0300, Mihai Prica wrote:
   Fixes rootfs generation error if bash is not included in the image.
  
  With rpm right?
 
 If it's what you're suggesting technically it's broken with any package 
 backend, to some degree.

Agreed, I was just commenting on it because rootfs generation error if
bash is not included is quite vague and it could be also bashism in
postinst script.

  Find which script it has in shebang and add bash only to that package
  RDEPENDS, if it's something which isn't always needed put it in separate
  package.
 
 Or if practical patch the script(s) to remove the bashisms and change the 
 shebang to #!/bin/sh instead.
 
 Cheers,
 Paul
 
 -- 
 
 Paul Eggleton
 Intel Open Source Technology Centre

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


signature.asc
Description: Digital signature
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][PATCH] iperf: Fix license

2013-07-30 Thread Guo Chunrong-B40290
Hello,paul barker

Thank you for your comments. 
I agree it . 

Thanks,
chunrong

-Original Message-
From: paul.betaf...@gmail.com [mailto:paul.betaf...@gmail.com] On Behalf Of 
Paul Barker
Sent: Tuesday, July 30, 2013 10:15 PM
To: Openembedded Discussion
Cc: Guo Chunrong-B40290; Yu Zongchun-B40527
Subject: Re: [oe] [meta-oe][PATCH] iperf: Fix license

On 29 July 2013 10:45, Guo Chunrong-B40290 b40...@freescale.com wrote:
 Hello,

 I'd appreciate if someone can review.


I'll take a quick look...

 -LICENSE = NewBSD
 +LICENSE = BSD

Comparing http://sourceforge.net/p/iperf/code/82/tree/trunk/COPYING
and the common licenses in openembedded-core, it looks like BSD-2-Clause 
would be more accurate. Could you take a look and let me know if you agree? 
Currently on the move so can't diff the files myself.

--
Paul Barker

Email: p...@paulbarker.me.uk
http://www.paulbarker.me.uk


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


[oe] [meta-oe,v2][PATCH] iperf: Fix license

2013-07-30 Thread b40290
From: Chunrong Guo b40...@freescale.com

 *the current version of Iperf has a false LICENSE set,
 so fix license with the generic BSD license

Signed-off-by: Chunrong Guo b40...@freescale.com
---
 meta-oe/recipes-benchmark/iperf/iperf_2.0.4.bb |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/meta-oe/recipes-benchmark/iperf/iperf_2.0.4.bb 
b/meta-oe/recipes-benchmark/iperf/iperf_2.0.4.bb
index 8107701..4261c95 100644
--- a/meta-oe/recipes-benchmark/iperf/iperf_2.0.4.bb
+++ b/meta-oe/recipes-benchmark/iperf/iperf_2.0.4.bb
@@ -1,7 +1,7 @@
 DESCRIPTION = Iperf is a tool to measure maximum TCP bandwidth, allowing the 
tuning of various parameters and UDP characteristics
 HOMEPAGE = http://dast.nlanr.net/Projects/Iperf/;
 SECTION = console/network
-LICENSE = NewBSD
+LICENSE = BSD-2-Clause
 LIC_FILES_CHKSUM = file://COPYING;md5=e8478eae9f479e39bc34975193360298
 
 SRC_URI =  \
-- 
1.7.5.4


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