All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] iproute2: upgrade to 3.4.0
@ 2012-07-17  8:28 Cristian Iorga
  2012-07-17  8:28 ` [PATCH 2/3] pulseaudio: upgrade to 2.0 Cristian Iorga
                   ` (3 more replies)
  0 siblings, 4 replies; 14+ messages in thread
From: Cristian Iorga @ 2012-07-17  8:28 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
---
 .../iproute2/iproute2-3.2.0/configure-cross.patch  |   76 -------------------
 .../iproute2/iproute2-3.4.0/configure-cross.patch  |   77 ++++++++++++++++++++
 meta/recipes-connectivity/iproute2/iproute2.inc    |   10 +--
 .../iproute2/iproute2_3.2.0.bb                     |   11 ---
 .../iproute2/iproute2_3.4.0.bb                     |    9 +++
 5 files changed, 91 insertions(+), 92 deletions(-)
 delete mode 100644 meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
 create mode 100644 meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
 delete mode 100644 meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
 create mode 100644 meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb

diff --git a/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch b/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
deleted file mode 100644
index bed7b84..0000000
--- a/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
+++ /dev/null
@@ -1,76 +0,0 @@
-From 85b0589b4843c03e8e6fd9416d71ea449a73c5c0 Mon Sep 17 00:00:00 2001
-From: Koen Kooi <koen@dominion.thruhere.net>
-Date: Thu, 3 Nov 2011 10:46:16 +0100
-Subject: [PATCH] make configure cross compile safe
-
-According to Kevin Tian:
-Upstream-Status: Pending
-
-Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
-Signed-off-by: Shane Wang <shane.wang@intel.com>
-
-diff -r e4e1b14b9c4a configure
---- a/configure	Sun Jan 22 15:26:47 2012 +0800
-+++ b/configure	Sun Jan 22 15:29:38 2012 +0800
-@@ -2,6 +2,7 @@
- # This is not an autconf generated configure
- #
- INCLUDE=${1:-"$PWD/include"}
-+SYSROOT=$1
- 
- check_atm()
- {
-@@ -13,7 +14,7 @@
- 	return 0;
- }
- EOF
--gcc -I$INCLUDE -o /tmp/atmtest /tmp/atmtest.c -latm >/dev/null 2>&1 
-+$CC -I$INCLUDE -o /tmp/atmtest /tmp/atmtest.c -latm >/dev/null 2>&1 
- if [ $? -eq 0 ]
- then
-     echo "TC_CONFIG_ATM:=y" >>Config
-@@ -47,7 +48,7 @@
- 
- EOF
- 
--if gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
-+if $CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
- then
- 	echo "TC_CONFIG_XT:=y" >>Config
- 	echo "using xtables"
-@@ -84,7 +85,7 @@
- }
- 
- EOF
--gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl >/dev/null 2>&1
-+$CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl >/dev/null 2>&1
- 
- if [ $? -eq 0 ]
- then
-@@ -124,7 +125,7 @@
- }
- 
- EOF
--gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl >/dev/null 2>&1
-+$CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl >/dev/null 2>&1
- 
- if [ $? -eq 0 ]
- then
-@@ -145,7 +146,7 @@
- check_ipt_lib_dir()
- {
- 	IPT_LIB_DIR=""
--	for dir in /lib /usr/lib /usr/local/lib
-+	for dir in $SYSROOT/lib $SYSROOT/usr/lib $SYSROOT/usr/local/lib
- 	do
- 		for file in $dir/{xtables,iptables}/lib*t_*so ; do
- 			if [ -f $file ]; then
-@@ -168,7 +169,7 @@
- 	return 0;
- }
- EOF
--gcc -I$INCLUDE -o /tmp/setnstest /tmp/setnstest.c >/dev/null 2>&1
-+$CC -I$INCLUDE -o /tmp/setnstest /tmp/setnstest.c >/dev/null 2>&1
- if [ $? -eq 0 ]
- then
- 	echo "IP_CONFIG_SETNS:=y" >>Config
diff --git a/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch b/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
new file mode 100644
index 0000000..42f591a
--- /dev/null
+++ b/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
@@ -0,0 +1,77 @@
+From 85b0589b4843c03e8e6fd9416d71ea449a73c5c0 Mon Sep 17 00:00:00 2001
+From: Koen Kooi <koen@dominion.thruhere.net>
+Date: Thu, 3 Nov 2011 10:46:16 +0100
+Subject: [PATCH] make configure cross compile safe
+
+According to Kevin Tian:
+Upstream-Status: Pending
+
+Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
+Signed-off-by: Shane Wang <shane.wang@intel.com>
+
+Index: iproute2-3.4.0/configure
+===================================================================
+--- iproute2-3.4.0.orig/configure
++++ iproute2-3.4.0/configure
+@@ -2,6 +2,7 @@
+ # This is not an autconf generated configure
+ #
+ INCLUDE=${1:-"$PWD/include"}
++SYSROOT=$1
+ 
+ # Make a temp directory in build tree.
+ TMPDIR=$(mktemp -d config.XXXXXX)
+@@ -17,7 +18,7 @@ int main(int argc, char **argv) {
+ 	return 0;
+ }
+ EOF
+-gcc -I$INCLUDE -o $TMPDIR/atmtest $TMPDIR/atmtest.c -latm >/dev/null 2>&1 
++$CC -I$INCLUDE -o $TMPDIR/atmtest $TMPDIR/atmtest.c -latm >/dev/null 2>&1
+ if [ $? -eq 0 ]
+ then
+     echo "TC_CONFIG_ATM:=y" >>Config
+@@ -51,7 +52,7 @@ int main(int argc, char **argv)
+ 
+ EOF
+ 
+-if gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
++if $CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
+ then
+ 	echo "TC_CONFIG_XT:=y" >>Config
+ 	echo "using xtables"
+@@ -88,7 +89,7 @@ int main(int argc, char **argv) {
+ }
+ 
+ EOF
+-gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl >/dev/null 2>&1
++$CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl >/dev/null 2>&1
+ 
+ if [ $? -eq 0 ]
+ then
+@@ -128,7 +129,7 @@ int main(int argc, char **argv) {
+ }
+ 
+ EOF
+-gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl >/dev/null 2>&1
++$CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl >/dev/null 2>&1
+ 
+ if [ $? -eq 0 ]
+ then
+@@ -149,7 +150,7 @@ check_ipt()
+ check_ipt_lib_dir()
+ {
+ 	IPT_LIB_DIR=""
+-	for dir in /lib /usr/lib /usr/local/lib
++	for dir in $SYSROOT/lib $SYSROOT/usr/lib $SYSROOT/usr/local/lib
+ 	do
+ 		for file in $dir/{xtables,iptables}/lib*t_*so ; do
+ 			if [ -f $file ]; then
+@@ -172,7 +173,7 @@ int main(int argc, char **argv)
+ 	return 0;
+ }
+ EOF
+-gcc -I$INCLUDE -o $TMPDIR/setnstest $TMPDIR/setnstest.c >/dev/null 2>&1
++$CC -I$INCLUDE -o $TMPDIR/setnstest $TMPDIR/setnstest.c >/dev/null 2>&1
+ if [ $? -eq 0 ]
+ then
+ 	echo "IP_CONFIG_SETNS:=y" >>Config
diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc b/meta/recipes-connectivity/iproute2/iproute2.inc
index 86847ef..7caf7d0 100644
--- a/meta/recipes-connectivity/iproute2/iproute2.inc
+++ b/meta/recipes-connectivity/iproute2/iproute2.inc
@@ -26,12 +26,12 @@ do_install () {
 # There are only .so files in iproute2
 INSANE_SKIP_${PN} = "dev-so"
 
-FILES_${PN} += "${base_libdir}/tc"
-FILES_${PN}-dbg += "${base_libdir}/tc/.debug"
+FILES_${PN} += "${libdir}/tc"
+FILES_${PN}-dbg += "${libdir}/tc/.debug"
 
-ALTERNATIVE_NAME = "ip"
-ALTERNATIVE_PATH = "${base_sbindir}/ip.iproute2"
-ALTERNATIVE_LINK = "${base_bindir}/ip"
+ALTERNATIVE_${PN} = "ip"
+ALTERNATIVE_TARGET[ip] = "${base_sbindir}/ip.${BPN}"
+ALTERNATIVE_LINK_NAME[ip] = "${base_bindir}/ip"
 ALTERNATIVE_PRIORITY = "100"
 
 PARALLEL_MAKE = ""
diff --git a/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb b/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
deleted file mode 100644
index 5af12af..0000000
--- a/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
+++ /dev/null
@@ -1,11 +0,0 @@
-require iproute2.inc
-
-PR = "r1"
-
-#v3.2.0 tag is "447c118f138171b260ad045ad6e1b17f9ef462e2"
-#but it was not fully tested and had build error, and the next commit fixed it.
-SRCREV = "13603f6a9e46f08576f6284a0ef1ce1fbf94ffe0"
-SRC_URI = "git://git.kernel.org/pub/scm/linux/kernel/git/shemminger/iproute2.git \
-           file://configure-cross.patch"
-S = "${WORKDIR}/git"
-
diff --git a/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb b/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb
new file mode 100644
index 0000000..cf1d521
--- /dev/null
+++ b/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb
@@ -0,0 +1,9 @@
+require iproute2.inc
+
+PR = "r0"
+
+SRC_URI = "${KERNELORG_MIRROR}/linux/utils/net/${BPN}/${BPN}-${PV}.tar.xz \
+           file://configure-cross.patch"
+
+SRC_URI[md5sum] = "879d3fac4e90809598b2864ec4a0cbf8"
+SRC_URI[sha256sum] = "38e846e412b2fa235a447b50c20ad1e9770d1b3ed4d3ab18ca0b18c6e8b79ba4"
-- 
1.7.9.5




^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [PATCH 2/3] pulseaudio: upgrade to 2.0
  2012-07-17  8:28 [PATCH 1/3] iproute2: upgrade to 3.4.0 Cristian Iorga
@ 2012-07-17  8:28 ` Cristian Iorga
  2012-07-17 18:53   ` Saul Wold
  2012-07-17  8:28 ` [PATCH 3/3] wpa-supplicant: upgrade to 1.0 Cristian Iorga
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 14+ messages in thread
From: Cristian Iorga @ 2012-07-17  8:28 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
---
 .../{libcanberra_0.28.bb => libcanberra_0.29.bb}   |   14 +++++++-------
 meta/recipes-multimedia/pulseaudio/pulseaudio.inc  |    2 +-
 .../{pulseaudio_1.1.bb => pulseaudio_2.0.bb}       |    7 +++----
 3 files changed, 11 insertions(+), 12 deletions(-)
 rename meta/recipes-multimedia/pulseaudio/{libcanberra_0.28.bb => libcanberra_0.29.bb} (76%)
 rename meta/recipes-multimedia/pulseaudio/{pulseaudio_1.1.bb => pulseaudio_2.0.bb} (64%)

diff --git a/meta/recipes-multimedia/pulseaudio/libcanberra_0.28.bb b/meta/recipes-multimedia/pulseaudio/libcanberra_0.29.bb
similarity index 76%
rename from meta/recipes-multimedia/pulseaudio/libcanberra_0.28.bb
rename to meta/recipes-multimedia/pulseaudio/libcanberra_0.29.bb
index e69c89d..b3c0e3d 100644
--- a/meta/recipes-multimedia/pulseaudio/libcanberra_0.28.bb
+++ b/meta/recipes-multimedia/pulseaudio/libcanberra_0.29.bb
@@ -5,14 +5,14 @@ LIC_FILES_CHKSUM = "file://LGPL;md5=2d5025d4aa3495befef8f17206a5b0a1 \
                     file://src/canberra.h;beginline=7;endline=24;md5=c616c687cf8da540a14f917e0d23ab03"
 
 DEPENDS = "gtk+ pulseaudio alsa-lib libtool libvorbis"
-PR = "r1"
+PR = "r0"
 
 inherit gconf autotools
 
-SRC_URI = "http://0pointer.de/lennart/projects/libcanberra/libcanberra-${PV}.tar.gz"
+SRC_URI = "http://0pointer.de/lennart/projects/libcanberra/libcanberra-${PV}.tar.xz"
 
-SRC_URI[md5sum] = "c198b4811598c4c161ff505e4531b02c"
-SRC_URI[sha256sum] = "eb1f8b2cabad7f07b6e44d606a91d73e1efca4b46daf92bd553e7222bc68868c"
+SRC_URI[md5sum] = "2594093a5d61047bd9cc87e955f86df8"
+SRC_URI[sha256sum] = "127a5ef07805856d63758e5180ebfb241d1f80094fd301c287591a15b8cfcd72"
 
 EXTRA_OECONF = " --disable-oss --disable-ltdl-install" 
 
@@ -25,8 +25,8 @@ PACKAGES += "${PN}-gnome"
 FILES_${PN} += "${libdir}/gtk-2.0/modules/ ${datadir}/gnome"
 FILES_${PN}-gnome += "${datadir}/gdm/autostart/LoginWindow/libcanberra-ready-sound.desktop \
 	              ${libdir}/gnome-settings-daemon-3.0/gtk-modules/canberra-gtk-module.desktop"
-FILES_${PN}-dev += "${libdir}/libcanberra-0.28/libcanberra-*.so \
-		    ${libdir}/libcanberra-0.28/libcanberra-*.la \
+FILES_${PN}-dev += "${libdir}/libcanberra-0.29/libcanberra-*.so \
+		    ${libdir}/libcanberra-0.29/libcanberra-*.la \
 		    ${datadir}/vala/vapi"
-FILES_${PN}-dbg += "${libdir}/gtk-2.0/modules/.debug ${libdir}/libcanberra-0.28/.debug"
+FILES_${PN}-dbg += "${libdir}/gtk-2.0/modules/.debug ${libdir}/libcanberra-0.29/.debug"
 
diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
index c3d6d98..cd09940 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
@@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1"
 LIC_FILES_CHKSUM = "file://GPL;md5=4325afd396febcb659c36b49533135d4 \
                     file://LGPL;md5=2d5025d4aa3495befef8f17206a5b0a1 \
                     file://src/modules/bluetooth/proximity-helper.c;beginline=1;endline=25;md5=e4cc710e40a4d900c2c294167bc58270 \
-                    file://src/pulsecore/vector.h;beginline=1;endline=21;md5=1b561f0a2ca47b1fa49862a6ccaacff1"
+                    file://src/pulsecore/resampler.h;beginline=4;endline=23;md5=c3d539b93f8c82a1780bfa3cfa544a95"
 DEPENDS = "libatomics-ops liboil avahi libsamplerate0 libsndfile1 libtool \
            ${@base_contains('DISTRO_FEATURES', 'x11', 'virtual/libx11 libxtst libice libsm libxcb gtk+', '', d)}"
 # optional
diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_1.1.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.0.bb
similarity index 64%
rename from meta/recipes-multimedia/pulseaudio/pulseaudio_1.1.bb
rename to meta/recipes-multimedia/pulseaudio/pulseaudio_2.0.bb
index e88a3a5..7e57e5e 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio_1.1.bb
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.0.bb
@@ -1,6 +1,6 @@
 require pulseaudio.inc
 
-PR = "r10"
+PR = "r0"
 
 DEPENDS += "libjson gdbm speex libxml-parser-perl-native"
 
@@ -8,11 +8,10 @@ inherit gettext perlnative
 
 SRC_URI = "http://freedesktop.org/software/pulseaudio/releases/pulseaudio-${PV}.tar.xz \
   file://volatiles.04_pulse \
-  file://pulseaudio_fix_for_x32.patch \
 "
 
-SRC_URI[md5sum] = "17d21df798cee407b769c6355fae397a"
-SRC_URI[sha256sum] = "6fe531136f6ebce2d35872a2d2c914278cdc5dcdd5eea516dc52c81f9001f5ee"
+SRC_URI[md5sum] = "9bbde657c353fe675c3b693054175a8e"
+SRC_URI[sha256sum] = "28b42edd42f4879a6884af5f0ec11801ac001eb7582881215b36649aa37e2061"
 
 do_compile_prepend() {
     cd ${S}
-- 
1.7.9.5




^ permalink raw reply related	[flat|nested] 14+ messages in thread

* [PATCH 3/3] wpa-supplicant: upgrade to 1.0
  2012-07-17  8:28 [PATCH 1/3] iproute2: upgrade to 3.4.0 Cristian Iorga
  2012-07-17  8:28 ` [PATCH 2/3] pulseaudio: upgrade to 2.0 Cristian Iorga
@ 2012-07-17  8:28 ` Cristian Iorga
  2012-07-17 19:01   ` Saul Wold
  2012-07-17  9:24 ` [PATCH 1/3] iproute2: upgrade to 3.4.0 Andrei Gherzan
  2012-07-17 18:58 ` Saul Wold
  3 siblings, 1 reply; 14+ messages in thread
From: Cristian Iorga @ 2012-07-17  8:28 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
---
 ...a-supplicant-0.7.inc => wpa-supplicant-1.0.inc} |    4 ++--
 .../99_wpa_supplicant                              |    0
 .../defconfig-gnutls                               |    0
 .../wpa-supplicant.sh                              |    0
 .../wpa_supplicant.conf                            |    0
 .../wpa_supplicant.conf-sane                       |    0
 .../wpa-supplicant/wpa-supplicant_0.7.3.bb         |    6 ------
 .../wpa-supplicant/wpa-supplicant_1.0.bb           |    6 ++++++
 8 files changed, 8 insertions(+), 8 deletions(-)
 rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.inc => wpa-supplicant-1.0.inc} (95%)
 rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/99_wpa_supplicant (100%)
 rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/defconfig-gnutls (100%)
 rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/wpa-supplicant.sh (100%)
 rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/wpa_supplicant.conf (100%)
 rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/wpa_supplicant.conf-sane (100%)
 delete mode 100644 meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb
 create mode 100644 meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb

diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.inc b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0.inc
similarity index 95%
rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.inc
rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0.inc
index 6c84202..0c18b30 100644
--- a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.inc
+++ b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0.inc
@@ -4,8 +4,8 @@ BUGTRACKER = "http://hostap.epitest.fi/bugz/"
 SECTION = "network"
 LICENSE = "GPLv2 | BSD"
 LIC_FILES_CHKSUM = "file://../COPYING;md5=c54ce9345727175ff66d17b67ff51f58 \
-                    file://../README;md5=54cfc88015d3ce83f7156e63c6bb1738 \
-                    file://wpa_supplicant.c;beginline=1;endline=17;md5=acdc5a4b0d6345f21f136eace747260e"
+                    file://../README;md5=9bee93996fecdb70f807baceb29d84b9 \
+                    file://wpa_supplicant.c;beginline=1;endline=17;md5=372fb65dc23b228aec4e40e64de3e467"
 DEPENDS = "gnutls dbus libnl"
 RRECOMMENDS_${PN} = "wpa-supplicant-passphrase wpa-supplicant-cli"
 
diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/99_wpa_supplicant b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/99_wpa_supplicant
similarity index 100%
rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/99_wpa_supplicant
rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/99_wpa_supplicant
diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/defconfig-gnutls b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/defconfig-gnutls
similarity index 100%
rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/defconfig-gnutls
rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/defconfig-gnutls
diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa-supplicant.sh b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa-supplicant.sh
similarity index 100%
rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa-supplicant.sh
rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa-supplicant.sh
diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_supplicant.conf b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_supplicant.conf
similarity index 100%
rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_supplicant.conf
rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_supplicant.conf
diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_supplicant.conf-sane b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_supplicant.conf-sane
similarity index 100%
rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_supplicant.conf-sane
rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_supplicant.conf-sane
diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb
deleted file mode 100644
index 6f6378b..0000000
--- a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb
+++ /dev/null
@@ -1,6 +0,0 @@
-require wpa-supplicant-0.7.inc
-
-PR = "r12"
-
-SRC_URI[md5sum] = "f516f191384a9a546e3f5145c08addda"
-SRC_URI[sha256sum] = "d0cd50caa85346ccc376dcda5ed3c258eef19a93b3cade39d25760118ad59443"
diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb
new file mode 100644
index 0000000..8ede3e5
--- /dev/null
+++ b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb
@@ -0,0 +1,6 @@
+require wpa-supplicant-1.0.inc
+
+PR = "r0"
+
+SRC_URI[md5sum] = "8650f6aa23646ef634402552d0669640"
+SRC_URI[sha256sum] = "91d41d473a5aafa2e25dd6577ebda975c4d4a3188850a53e31feaf7c04482b9c"
-- 
1.7.9.5




^ permalink raw reply related	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/3] iproute2: upgrade to 3.4.0
  2012-07-17  8:28 [PATCH 1/3] iproute2: upgrade to 3.4.0 Cristian Iorga
  2012-07-17  8:28 ` [PATCH 2/3] pulseaudio: upgrade to 2.0 Cristian Iorga
  2012-07-17  8:28 ` [PATCH 3/3] wpa-supplicant: upgrade to 1.0 Cristian Iorga
@ 2012-07-17  9:24 ` Andrei Gherzan
  2012-07-17 13:08   ` Paul Eggleton
  2012-07-17 18:58 ` Saul Wold
  3 siblings, 1 reply; 14+ messages in thread
From: Andrei Gherzan @ 2012-07-17  9:24 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 9474 bytes --]

On Tue, Jul 17, 2012 at 11:28 AM, Cristian Iorga
<cristian.iorga@intel.com>wrote:

> Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
> ---
>  .../iproute2/iproute2-3.2.0/configure-cross.patch  |   76
> -------------------
>  .../iproute2/iproute2-3.4.0/configure-cross.patch  |   77
> ++++++++++++++++++++
>  meta/recipes-connectivity/iproute2/iproute2.inc    |   10 +--
>  .../iproute2/iproute2_3.2.0.bb                     |   11 ---
>  .../iproute2/iproute2_3.4.0.bb                     |    9 +++
>  5 files changed, 91 insertions(+), 92 deletions(-)
>  delete mode 100644
> meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
>  create mode 100644
> meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
>  delete mode 100644 meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
>  create mode 100644 meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb
>
> diff --git
> a/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
> b/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
> deleted file mode 100644
> index bed7b84..0000000
> ---
> a/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
> +++ /dev/null
> @@ -1,76 +0,0 @@
> -From 85b0589b4843c03e8e6fd9416d71ea449a73c5c0 Mon Sep 17 00:00:00 2001
> -From: Koen Kooi <koen@dominion.thruhere.net>
> -Date: Thu, 3 Nov 2011 10:46:16 +0100
> -Subject: [PATCH] make configure cross compile safe
> -
> -According to Kevin Tian:
> -Upstream-Status: Pending
> -
> -Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> -Signed-off-by: Shane Wang <shane.wang@intel.com>
> -
> -diff -r e4e1b14b9c4a configure
> ---- a/configure        Sun Jan 22 15:26:47 2012 +0800
> -+++ b/configure        Sun Jan 22 15:29:38 2012 +0800
> -@@ -2,6 +2,7 @@
> - # This is not an autconf generated configure
> - #
> - INCLUDE=${1:-"$PWD/include"}
> -+SYSROOT=$1
> -
> - check_atm()
> - {
> -@@ -13,7 +14,7 @@
> -       return 0;
> - }
> - EOF
> --gcc -I$INCLUDE -o /tmp/atmtest /tmp/atmtest.c -latm >/dev/null 2>&1
> -+$CC -I$INCLUDE -o /tmp/atmtest /tmp/atmtest.c -latm >/dev/null 2>&1
> - if [ $? -eq 0 ]
> - then
> -     echo "TC_CONFIG_ATM:=y" >>Config
> -@@ -47,7 +48,7 @@
> -
> - EOF
> -
> --if gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL
> $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> -+if $CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL
> $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> - then
> -       echo "TC_CONFIG_XT:=y" >>Config
> -       echo "using xtables"
> -@@ -84,7 +85,7 @@
> - }
> -
> - EOF
> --gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl
> >/dev/null 2>&1
> -+$CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl
> >/dev/null 2>&1
> -
> - if [ $? -eq 0 ]
> - then
> -@@ -124,7 +125,7 @@
> - }
> -
> - EOF
> --gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl
> >/dev/null 2>&1
> -+$CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl
> >/dev/null 2>&1
> -
> - if [ $? -eq 0 ]
> - then
> -@@ -145,7 +146,7 @@
> - check_ipt_lib_dir()
> - {
> -       IPT_LIB_DIR=""
> --      for dir in /lib /usr/lib /usr/local/lib
> -+      for dir in $SYSROOT/lib $SYSROOT/usr/lib $SYSROOT/usr/local/lib
> -       do
> -               for file in $dir/{xtables,iptables}/lib*t_*so ; do
> -                       if [ -f $file ]; then
> -@@ -168,7 +169,7 @@
> -       return 0;
> - }
> - EOF
> --gcc -I$INCLUDE -o /tmp/setnstest /tmp/setnstest.c >/dev/null 2>&1
> -+$CC -I$INCLUDE -o /tmp/setnstest /tmp/setnstest.c >/dev/null 2>&1
> - if [ $? -eq 0 ]
> - then
> -       echo "IP_CONFIG_SETNS:=y" >>Config
> diff --git
> a/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
> b/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
> new file mode 100644
> index 0000000..42f591a
> --- /dev/null
> +++
> b/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
> @@ -0,0 +1,77 @@
> +From 85b0589b4843c03e8e6fd9416d71ea449a73c5c0 Mon Sep 17 00:00:00 2001
> +From: Koen Kooi <koen@dominion.thruhere.net>
> +Date: Thu, 3 Nov 2011 10:46:16 +0100
> +Subject: [PATCH] make configure cross compile safe
> +
> +According to Kevin Tian:
> +Upstream-Status: Pending
> +
> +Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> +Signed-off-by: Shane Wang <shane.wang@intel.com>
> +
> +Index: iproute2-3.4.0/configure
> +===================================================================
> +--- iproute2-3.4.0.orig/configure
> ++++ iproute2-3.4.0/configure
> +@@ -2,6 +2,7 @@
> + # This is not an autconf generated configure
> + #
> + INCLUDE=${1:-"$PWD/include"}
> ++SYSROOT=$1
> +
> + # Make a temp directory in build tree.
> + TMPDIR=$(mktemp -d config.XXXXXX)
> +@@ -17,7 +18,7 @@ int main(int argc, char **argv) {
> +       return 0;
> + }
> + EOF
> +-gcc -I$INCLUDE -o $TMPDIR/atmtest $TMPDIR/atmtest.c -latm >/dev/null 2>&1
> ++$CC -I$INCLUDE -o $TMPDIR/atmtest $TMPDIR/atmtest.c -latm >/dev/null 2>&1
> + if [ $? -eq 0 ]
> + then
> +     echo "TC_CONFIG_ATM:=y" >>Config
> +@@ -51,7 +52,7 @@ int main(int argc, char **argv)
> +
> + EOF
> +
> +-if gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL
> $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> ++if $CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL
> $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> + then
> +       echo "TC_CONFIG_XT:=y" >>Config
> +       echo "using xtables"
> +@@ -88,7 +89,7 @@ int main(int argc, char **argv) {
> + }
> +
> + EOF
> +-gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl
> >/dev/null 2>&1
> ++$CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl
> >/dev/null 2>&1
> +
> + if [ $? -eq 0 ]
> + then
> +@@ -128,7 +129,7 @@ int main(int argc, char **argv) {
> + }
> +
> + EOF
> +-gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl
> >/dev/null 2>&1
> ++$CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl
> >/dev/null 2>&1
> +
> + if [ $? -eq 0 ]
> + then
> +@@ -149,7 +150,7 @@ check_ipt()
> + check_ipt_lib_dir()
> + {
> +       IPT_LIB_DIR=""
> +-      for dir in /lib /usr/lib /usr/local/lib
> ++      for dir in $SYSROOT/lib $SYSROOT/usr/lib $SYSROOT/usr/local/lib
> +       do
> +               for file in $dir/{xtables,iptables}/lib*t_*so ; do
> +                       if [ -f $file ]; then
> +@@ -172,7 +173,7 @@ int main(int argc, char **argv)
> +       return 0;
> + }
> + EOF
> +-gcc -I$INCLUDE -o $TMPDIR/setnstest $TMPDIR/setnstest.c >/dev/null 2>&1
> ++$CC -I$INCLUDE -o $TMPDIR/setnstest $TMPDIR/setnstest.c >/dev/null 2>&1
> + if [ $? -eq 0 ]
> + then
> +       echo "IP_CONFIG_SETNS:=y" >>Config
> diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc
> b/meta/recipes-connectivity/iproute2/iproute2.inc
> index 86847ef..7caf7d0 100644
> --- a/meta/recipes-connectivity/iproute2/iproute2.inc
> +++ b/meta/recipes-connectivity/iproute2/iproute2.inc
> @@ -26,12 +26,12 @@ do_install () {
>  # There are only .so files in iproute2
>  INSANE_SKIP_${PN} = "dev-so"
>
> -FILES_${PN} += "${base_libdir}/tc"
> -FILES_${PN}-dbg += "${base_libdir}/tc/.debug"
> +FILES_${PN} += "${libdir}/tc"
> +FILES_${PN}-dbg += "${libdir}/tc/.debug"
>
> -ALTERNATIVE_NAME = "ip"
> -ALTERNATIVE_PATH = "${base_sbindir}/ip.iproute2"
> -ALTERNATIVE_LINK = "${base_bindir}/ip"
> +ALTERNATIVE_${PN} = "ip"
> +ALTERNATIVE_TARGET[ip] = "${base_sbindir}/ip.${BPN}"
> +ALTERNATIVE_LINK_NAME[ip] = "${base_bindir}/ip"
>  ALTERNATIVE_PRIORITY = "100"
>
>
You could state this change in the description.


>  PARALLEL_MAKE = ""
> diff --git a/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bbb/meta/recipes-connectivity/iproute2/
> iproute2_3.2.0.bb
> deleted file mode 100644
> index 5af12af..0000000
> --- a/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
> +++ /dev/null
> @@ -1,11 +0,0 @@
> -require iproute2.inc
> -
> -PR = "r1"
> -
> -#v3.2.0 tag is "447c118f138171b260ad045ad6e1b17f9ef462e2"
> -#but it was not fully tested and had build error, and the next commit
> fixed it.
> -SRCREV = "13603f6a9e46f08576f6284a0ef1ce1fbf94ffe0"
> -SRC_URI = "git://
> git.kernel.org/pub/scm/linux/kernel/git/shemminger/iproute2.git \
> -           file://configure-cross.patch"
> -S = "${WORKDIR}/git"
> -
> diff --git a/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bbb/meta/recipes-connectivity/iproute2/
> iproute2_3.4.0.bb
> new file mode 100644
> index 0000000..cf1d521
> --- /dev/null
> +++ b/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb
> @@ -0,0 +1,9 @@
> +require iproute2.inc
> +
> +PR = "r0"
>
>
Not needed anymore. (The same mention for wpa-s. and pulseaudio.)


> +SRC_URI = "${KERNELORG_MIRROR}/linux/utils/net/${BPN}/${BPN}-${PV}.tar.xz
> \
> +           file://configure-cross.patch"
> +
> +SRC_URI[md5sum] = "879d3fac4e90809598b2864ec4a0cbf8"
> +SRC_URI[sha256sum] =
> "38e846e412b2fa235a447b50c20ad1e9770d1b3ed4d3ab18ca0b18c6e8b79ba4"
> --
> 1.7.9.5
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>

[-- Attachment #2: Type: text/html, Size: 12344 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/3] iproute2: upgrade to 3.4.0
  2012-07-17  9:24 ` [PATCH 1/3] iproute2: upgrade to 3.4.0 Andrei Gherzan
@ 2012-07-17 13:08   ` Paul Eggleton
  2012-07-17 13:27     ` Koen Kooi
  2012-07-17 13:29     ` Andrei Gherzan
  0 siblings, 2 replies; 14+ messages in thread
From: Paul Eggleton @ 2012-07-17 13:08 UTC (permalink / raw)
  To: openembedded-core

On Tuesday 17 July 2012 12:24:41 Andrei Gherzan wrote:
> On Tue, Jul 17, 2012 at 11:28 AM, Cristian Iorga
> <cristian.iorga@intel.com>wrote:
> > +PR = "r0"
> 
> Not needed anymore. (The same mention for wpa-s. and pulseaudio.)

This is not a hard-and-fast rule for OE-Core AFAIK.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/3] iproute2: upgrade to 3.4.0
  2012-07-17 13:08   ` Paul Eggleton
@ 2012-07-17 13:27     ` Koen Kooi
  2012-07-17 13:41       ` Paul Eggleton
  2012-07-17 13:29     ` Andrei Gherzan
  1 sibling, 1 reply; 14+ messages in thread
From: Koen Kooi @ 2012-07-17 13:27 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer


Op 17 jul. 2012, om 15:08 heeft Paul Eggleton het volgende geschreven:

> On Tuesday 17 July 2012 12:24:41 Andrei Gherzan wrote:
>> On Tue, Jul 17, 2012 at 11:28 AM, Cristian Iorga
>> <cristian.iorga@intel.com>wrote:
>>> +PR = "r0"
>> 
>> Not needed anymore. (The same mention for wpa-s. and pulseaudio.)
> 
> This is not a hard-and-fast rule for OE-Core AFAIK.

It would be *very* nice to get rid of all PR = r0 instances in the complete oe metaverse.


^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/3] iproute2: upgrade to 3.4.0
  2012-07-17 13:08   ` Paul Eggleton
  2012-07-17 13:27     ` Koen Kooi
@ 2012-07-17 13:29     ` Andrei Gherzan
  1 sibling, 0 replies; 14+ messages in thread
From: Andrei Gherzan @ 2012-07-17 13:29 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

[-- Attachment #1: Type: text/plain, Size: 498 bytes --]

On Tue, Jul 17, 2012 at 4:08 PM, Paul Eggleton <
paul.eggleton@linux.intel.com> wrote:

> On Tuesday 17 July 2012 12:24:41 Andrei Gherzan wrote:
> > On Tue, Jul 17, 2012 at 11:28 AM, Cristian Iorga
> > <cristian.iorga@intel.com>wrote:
> > > +PR = "r0"
> >
> > Not needed anymore. (The same mention for wpa-s. and pulseaudio.)
>
> This is not a hard-and-fast rule for OE-Core AFAIK.
>
>
Indeed but as we have this it's a good thing to use it and clean a little
bit the recipes.

ag

[-- Attachment #2: Type: text/html, Size: 885 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/3] iproute2: upgrade to 3.4.0
  2012-07-17 13:27     ` Koen Kooi
@ 2012-07-17 13:41       ` Paul Eggleton
  0 siblings, 0 replies; 14+ messages in thread
From: Paul Eggleton @ 2012-07-17 13:41 UTC (permalink / raw)
  To: Koen Kooi; +Cc: openembedded-core

On Tuesday 17 July 2012 15:27:01 Koen Kooi wrote:
> Op 17 jul. 2012, om 15:08 heeft Paul Eggleton het volgende geschreven:
> > On Tuesday 17 July 2012 12:24:41 Andrei Gherzan wrote:
> >> On Tue, Jul 17, 2012 at 11:28 AM, Cristian Iorga
> >> 
> >> <cristian.iorga@intel.com>wrote:
> >>> +PR = "r0"
> >> 
> >> Not needed anymore. (The same mention for wpa-s. and pulseaudio.)
> > 
> > This is not a hard-and-fast rule for OE-Core AFAIK.
> 
> It would be *very* nice to get rid of all PR = r0 instances in the complete
> oe metaverse. 

I haven't yet heard a compelling argument as to why it's such a great idea to 
keep removing it. For my part, leaving PR = "r0" just means the PR line stays 
in the same place instead of jumping around when people remove and add it in 
different locations within the recipe.

Of course, once we move to using the PR server this is a moot point.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/3] pulseaudio: upgrade to 2.0
  2012-07-17  8:28 ` [PATCH 2/3] pulseaudio: upgrade to 2.0 Cristian Iorga
@ 2012-07-17 18:53   ` Saul Wold
  2012-07-18 12:40     ` Iorga, Cristian
  0 siblings, 1 reply; 14+ messages in thread
From: Saul Wold @ 2012-07-17 18:53 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 07/17/2012 01:28 AM, Cristian Iorga wrote:
> Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
> ---
>   .../{libcanberra_0.28.bb => libcanberra_0.29.bb}   |   14 +++++++-------
>   meta/recipes-multimedia/pulseaudio/pulseaudio.inc  |    2 +-
>   .../{pulseaudio_1.1.bb => pulseaudio_2.0.bb}       |    7 +++----
>   3 files changed, 11 insertions(+), 12 deletions(-)
>   rename meta/recipes-multimedia/pulseaudio/{libcanberra_0.28.bb => libcanberra_0.29.bb} (76%)
>   rename meta/recipes-multimedia/pulseaudio/{pulseaudio_1.1.bb => pulseaudio_2.0.bb} (64%)
>
You seem to be updating 2 recipes here, please separate them out as 2 
discrete commits.

see below also.

> diff --git a/meta/recipes-multimedia/pulseaudio/libcanberra_0.28.bb b/meta/recipes-multimedia/pulseaudio/libcanberra_0.29.bb
> similarity index 76%
> rename from meta/recipes-multimedia/pulseaudio/libcanberra_0.28.bb
> rename to meta/recipes-multimedia/pulseaudio/libcanberra_0.29.bb
> index e69c89d..b3c0e3d 100644
> --- a/meta/recipes-multimedia/pulseaudio/libcanberra_0.28.bb
> +++ b/meta/recipes-multimedia/pulseaudio/libcanberra_0.29.bb
> @@ -5,14 +5,14 @@ LIC_FILES_CHKSUM = "file://LGPL;md5=2d5025d4aa3495befef8f17206a5b0a1 \
>                       file://src/canberra.h;beginline=7;endline=24;md5=c616c687cf8da540a14f917e0d23ab03"
>
>   DEPENDS = "gtk+ pulseaudio alsa-lib libtool libvorbis"
> -PR = "r1"
> +PR = "r0"
>
>   inherit gconf autotools
>
> -SRC_URI = "http://0pointer.de/lennart/projects/libcanberra/libcanberra-${PV}.tar.gz"
> +SRC_URI = "http://0pointer.de/lennart/projects/libcanberra/libcanberra-${PV}.tar.xz"
>
> -SRC_URI[md5sum] = "c198b4811598c4c161ff505e4531b02c"
> -SRC_URI[sha256sum] = "eb1f8b2cabad7f07b6e44d606a91d73e1efca4b46daf92bd553e7222bc68868c"
> +SRC_URI[md5sum] = "2594093a5d61047bd9cc87e955f86df8"
> +SRC_URI[sha256sum] = "127a5ef07805856d63758e5180ebfb241d1f80094fd301c287591a15b8cfcd72"
>
>   EXTRA_OECONF = " --disable-oss --disable-ltdl-install"
>
> @@ -25,8 +25,8 @@ PACKAGES += "${PN}-gnome"
>   FILES_${PN} += "${libdir}/gtk-2.0/modules/ ${datadir}/gnome"
>   FILES_${PN}-gnome += "${datadir}/gdm/autostart/LoginWindow/libcanberra-ready-sound.desktop \
>   	              ${libdir}/gnome-settings-daemon-3.0/gtk-modules/canberra-gtk-module.desktop"
> -FILES_${PN}-dev += "${libdir}/libcanberra-0.28/libcanberra-*.so \
> -		    ${libdir}/libcanberra-0.28/libcanberra-*.la \
> +FILES_${PN}-dev += "${libdir}/libcanberra-0.29/libcanberra-*.so \
> +		    ${libdir}/libcanberra-0.29/libcanberra-*.la \
>   		    ${datadir}/vala/vapi"
> -FILES_${PN}-dbg += "${libdir}/gtk-2.0/modules/.debug ${libdir}/libcanberra-0.28/.debug"
> +FILES_${PN}-dbg += "${libdir}/gtk-2.0/modules/.debug ${libdir}/libcanberra-0.29/.debug"
>
Can we change these hardcoded version numbers to ${PV} instead? or -*?

> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> index c3d6d98..cd09940 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> @@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1"
>   LIC_FILES_CHKSUM = "file://GPL;md5=4325afd396febcb659c36b49533135d4 \
>                       file://LGPL;md5=2d5025d4aa3495befef8f17206a5b0a1 \
>                       file://src/modules/bluetooth/proximity-helper.c;beginline=1;endline=25;md5=e4cc710e40a4d900c2c294167bc58270 \
> -                    file://src/pulsecore/vector.h;beginline=1;endline=21;md5=1b561f0a2ca47b1fa49862a6ccaacff1"
> +                    file://src/pulsecore/resampler.h;beginline=4;endline=23;md5=c3d539b93f8c82a1780bfa3cfa544a95"

What changed here and why, anytime we change LIC_FILES_CHKSUM you should 
ensure it's commented in the commit message.

>   DEPENDS = "libatomics-ops liboil avahi libsamplerate0 libsndfile1 libtool \
>              ${@base_contains('DISTRO_FEATURES', 'x11', 'virtual/libx11 libxtst libice libsm libxcb gtk+', '', d)}"
>   # optional
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_1.1.bb b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.0.bb
> similarity index 64%
> rename from meta/recipes-multimedia/pulseaudio/pulseaudio_1.1.bb
> rename to meta/recipes-multimedia/pulseaudio/pulseaudio_2.0.bb
> index e88a3a5..7e57e5e 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_1.1.bb
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.0.bb
> @@ -1,6 +1,6 @@
>   require pulseaudio.inc
>
> -PR = "r10"
> +PR = "r0"
>
>   DEPENDS += "libjson gdbm speex libxml-parser-perl-native"
>
> @@ -8,11 +8,10 @@ inherit gettext perlnative
>
>   SRC_URI = "http://freedesktop.org/software/pulseaudio/releases/pulseaudio-${PV}.tar.xz \
>     file://volatiles.04_pulse \
> -  file://pulseaudio_fix_for_x32.patch \
>   "
Why are you removing this patch? Also, you don't remove the 
corresponding file?  Did you test x32?


>
> -SRC_URI[md5sum] = "17d21df798cee407b769c6355fae397a"
> -SRC_URI[sha256sum] = "6fe531136f6ebce2d35872a2d2c914278cdc5dcdd5eea516dc52c81f9001f5ee"
> +SRC_URI[md5sum] = "9bbde657c353fe675c3b693054175a8e"
> +SRC_URI[sha256sum] = "28b42edd42f4879a6884af5f0ec11801ac001eb7582881215b36649aa37e2061"
>
>   do_compile_prepend() {
>       cd ${S}
>

Thanks
	Sau!



^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/3] iproute2: upgrade to 3.4.0
  2012-07-17  8:28 [PATCH 1/3] iproute2: upgrade to 3.4.0 Cristian Iorga
                   ` (2 preceding siblings ...)
  2012-07-17  9:24 ` [PATCH 1/3] iproute2: upgrade to 3.4.0 Andrei Gherzan
@ 2012-07-17 18:58 ` Saul Wold
  2012-07-18 11:50   ` Iorga, Cristian
  3 siblings, 1 reply; 14+ messages in thread
From: Saul Wold @ 2012-07-17 18:58 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 07/17/2012 01:28 AM, Cristian Iorga wrote:
> Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
> ---
>   .../iproute2/iproute2-3.2.0/configure-cross.patch  |   76 -------------------
>   .../iproute2/iproute2-3.4.0/configure-cross.patch  |   77 ++++++++++++++++++++
>   meta/recipes-connectivity/iproute2/iproute2.inc    |   10 +--
>   .../iproute2/iproute2_3.2.0.bb                     |   11 ---
>   .../iproute2/iproute2_3.4.0.bb                     |    9 +++
>   5 files changed, 91 insertions(+), 92 deletions(-)
>   delete mode 100644 meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
>   create mode 100644 meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
>   delete mode 100644 meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
>   create mode 100644 meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb
>
> diff --git a/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch b/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
> deleted file mode 100644
> index bed7b84..0000000
> --- a/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
> +++ /dev/null
> @@ -1,76 +0,0 @@
> -From 85b0589b4843c03e8e6fd9416d71ea449a73c5c0 Mon Sep 17 00:00:00 2001
> -From: Koen Kooi <koen@dominion.thruhere.net>
> -Date: Thu, 3 Nov 2011 10:46:16 +0100
> -Subject: [PATCH] make configure cross compile safe
> -
> -According to Kevin Tian:
> -Upstream-Status: Pending
> -
> -Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> -Signed-off-by: Shane Wang <shane.wang@intel.com>
> -
> -diff -r e4e1b14b9c4a configure
> ---- a/configure	Sun Jan 22 15:26:47 2012 +0800
> -+++ b/configure	Sun Jan 22 15:29:38 2012 +0800
> -@@ -2,6 +2,7 @@
> - # This is not an autconf generated configure
> - #
> - INCLUDE=${1:-"$PWD/include"}
> -+SYSROOT=$1
> -
> - check_atm()
> - {
> -@@ -13,7 +14,7 @@
> - 	return 0;
> - }
> - EOF
> --gcc -I$INCLUDE -o /tmp/atmtest /tmp/atmtest.c -latm >/dev/null 2>&1
> -+$CC -I$INCLUDE -o /tmp/atmtest /tmp/atmtest.c -latm >/dev/null 2>&1
> - if [ $? -eq 0 ]
> - then
> -     echo "TC_CONFIG_ATM:=y" >>Config
> -@@ -47,7 +48,7 @@
> -
> - EOF
> -
> --if gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> -+if $CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> - then
> - 	echo "TC_CONFIG_XT:=y" >>Config
> - 	echo "using xtables"
> -@@ -84,7 +85,7 @@
> - }
> -
> - EOF
> --gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl >/dev/null 2>&1
> -+$CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl >/dev/null 2>&1
> -
> - if [ $? -eq 0 ]
> - then
> -@@ -124,7 +125,7 @@
> - }
> -
> - EOF
> --gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl >/dev/null 2>&1
> -+$CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl >/dev/null 2>&1
> -
> - if [ $? -eq 0 ]
> - then
> -@@ -145,7 +146,7 @@
> - check_ipt_lib_dir()
> - {
> - 	IPT_LIB_DIR=""
> --	for dir in /lib /usr/lib /usr/local/lib
> -+	for dir in $SYSROOT/lib $SYSROOT/usr/lib $SYSROOT/usr/local/lib
> - 	do
> - 		for file in $dir/{xtables,iptables}/lib*t_*so ; do
> - 			if [ -f $file ]; then
> -@@ -168,7 +169,7 @@
> - 	return 0;
> - }
> - EOF
> --gcc -I$INCLUDE -o /tmp/setnstest /tmp/setnstest.c >/dev/null 2>&1
> -+$CC -I$INCLUDE -o /tmp/setnstest /tmp/setnstest.c >/dev/null 2>&1
> - if [ $? -eq 0 ]
> - then
> - 	echo "IP_CONFIG_SETNS:=y" >>Config
> diff --git a/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch b/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
> new file mode 100644
> index 0000000..42f591a
> --- /dev/null
> +++ b/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
> @@ -0,0 +1,77 @@
> +From 85b0589b4843c03e8e6fd9416d71ea449a73c5c0 Mon Sep 17 00:00:00 2001
> +From: Koen Kooi <koen@dominion.thruhere.net>
> +Date: Thu, 3 Nov 2011 10:46:16 +0100
> +Subject: [PATCH] make configure cross compile safe
> +
> +According to Kevin Tian:
> +Upstream-Status: Pending
> +
> +Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> +Signed-off-by: Shane Wang <shane.wang@intel.com>
> +
> +Index: iproute2-3.4.0/configure
> +===================================================================
> +--- iproute2-3.4.0.orig/configure
> ++++ iproute2-3.4.0/configure
> +@@ -2,6 +2,7 @@
> + # This is not an autconf generated configure
> + #
> + INCLUDE=${1:-"$PWD/include"}
> ++SYSROOT=$1
> +
> + # Make a temp directory in build tree.
> + TMPDIR=$(mktemp -d config.XXXXXX)
> +@@ -17,7 +18,7 @@ int main(int argc, char **argv) {
> + 	return 0;
> + }
> + EOF
> +-gcc -I$INCLUDE -o $TMPDIR/atmtest $TMPDIR/atmtest.c -latm >/dev/null 2>&1
> ++$CC -I$INCLUDE -o $TMPDIR/atmtest $TMPDIR/atmtest.c -latm >/dev/null 2>&1
> + if [ $? -eq 0 ]
> + then
> +     echo "TC_CONFIG_ATM:=y" >>Config
> +@@ -51,7 +52,7 @@ int main(int argc, char **argv)
> +
> + EOF
> +
> +-if gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> ++if $CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> + then
> + 	echo "TC_CONFIG_XT:=y" >>Config
> + 	echo "using xtables"
> +@@ -88,7 +89,7 @@ int main(int argc, char **argv) {
> + }
> +
> + EOF
> +-gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl >/dev/null 2>&1
> ++$CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl >/dev/null 2>&1
> +
> + if [ $? -eq 0 ]
> + then
> +@@ -128,7 +129,7 @@ int main(int argc, char **argv) {
> + }
> +
> + EOF
> +-gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl >/dev/null 2>&1
> ++$CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl >/dev/null 2>&1
> +
> + if [ $? -eq 0 ]
> + then
> +@@ -149,7 +150,7 @@ check_ipt()
> + check_ipt_lib_dir()
> + {
> + 	IPT_LIB_DIR=""
> +-	for dir in /lib /usr/lib /usr/local/lib
> ++	for dir in $SYSROOT/lib $SYSROOT/usr/lib $SYSROOT/usr/local/lib
> + 	do
> + 		for file in $dir/{xtables,iptables}/lib*t_*so ; do
> + 			if [ -f $file ]; then
> +@@ -172,7 +173,7 @@ int main(int argc, char **argv)
> + 	return 0;
> + }
> + EOF
> +-gcc -I$INCLUDE -o $TMPDIR/setnstest $TMPDIR/setnstest.c >/dev/null 2>&1
> ++$CC -I$INCLUDE -o $TMPDIR/setnstest $TMPDIR/setnstest.c >/dev/null 2>&1
> + if [ $? -eq 0 ]
> + then
> + 	echo "IP_CONFIG_SETNS:=y" >>Config
> diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc b/meta/recipes-connectivity/iproute2/iproute2.inc
> index 86847ef..7caf7d0 100644
> --- a/meta/recipes-connectivity/iproute2/iproute2.inc
> +++ b/meta/recipes-connectivity/iproute2/iproute2.inc
> @@ -26,12 +26,12 @@ do_install () {
>   # There are only .so files in iproute2
>   INSANE_SKIP_${PN} = "dev-so"
>
> -FILES_${PN} += "${base_libdir}/tc"
> -FILES_${PN}-dbg += "${base_libdir}/tc/.debug"
> +FILES_${PN} += "${libdir}/tc"
> +FILES_${PN}-dbg += "${libdir}/tc/.debug"
>
> -ALTERNATIVE_NAME = "ip"
> -ALTERNATIVE_PATH = "${base_sbindir}/ip.iproute2"
> -ALTERNATIVE_LINK = "${base_bindir}/ip"
> +ALTERNATIVE_${PN} = "ip"
> +ALTERNATIVE_TARGET[ip] = "${base_sbindir}/ip.${BPN}"
> +ALTERNATIVE_LINK_NAME[ip] = "${base_bindir}/ip"
>   ALTERNATIVE_PRIORITY = "100"
>

Did you use buildhistory to compare the changes to the package?

Sau!

>   PARALLEL_MAKE = ""
> diff --git a/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb b/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
> deleted file mode 100644
> index 5af12af..0000000
> --- a/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
> +++ /dev/null
> @@ -1,11 +0,0 @@
> -require iproute2.inc
> -
> -PR = "r1"
> -
> -#v3.2.0 tag is "447c118f138171b260ad045ad6e1b17f9ef462e2"
> -#but it was not fully tested and had build error, and the next commit fixed it.
> -SRCREV = "13603f6a9e46f08576f6284a0ef1ce1fbf94ffe0"
> -SRC_URI = "git://git.kernel.org/pub/scm/linux/kernel/git/shemminger/iproute2.git \
> -           file://configure-cross.patch"
> -S = "${WORKDIR}/git"
> -
> diff --git a/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb b/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb
> new file mode 100644
> index 0000000..cf1d521
> --- /dev/null
> +++ b/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb
> @@ -0,0 +1,9 @@
> +require iproute2.inc
> +
> +PR = "r0"
> +
> +SRC_URI = "${KERNELORG_MIRROR}/linux/utils/net/${BPN}/${BPN}-${PV}.tar.xz \
> +           file://configure-cross.patch"
> +
> +SRC_URI[md5sum] = "879d3fac4e90809598b2864ec4a0cbf8"
> +SRC_URI[sha256sum] = "38e846e412b2fa235a447b50c20ad1e9770d1b3ed4d3ab18ca0b18c6e8b79ba4"
>




^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 3/3] wpa-supplicant: upgrade to 1.0
  2012-07-17  8:28 ` [PATCH 3/3] wpa-supplicant: upgrade to 1.0 Cristian Iorga
@ 2012-07-17 19:01   ` Saul Wold
  2012-07-18 12:09     ` Iorga, Cristian
  0 siblings, 1 reply; 14+ messages in thread
From: Saul Wold @ 2012-07-17 19:01 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On 07/17/2012 01:28 AM, Cristian Iorga wrote:
> Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
> ---
>   ...a-supplicant-0.7.inc => wpa-supplicant-1.0.inc} |    4 ++--
>   .../99_wpa_supplicant                              |    0
>   .../defconfig-gnutls                               |    0
>   .../wpa-supplicant.sh                              |    0
>   .../wpa_supplicant.conf                            |    0
>   .../wpa_supplicant.conf-sane                       |    0
>   .../wpa-supplicant/wpa-supplicant_0.7.3.bb         |    6 ------
>   .../wpa-supplicant/wpa-supplicant_1.0.bb           |    6 ++++++
>   8 files changed, 8 insertions(+), 8 deletions(-)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.inc => wpa-supplicant-1.0.inc} (95%)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/99_wpa_supplicant (100%)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/defconfig-gnutls (100%)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/wpa-supplicant.sh (100%)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/wpa_supplicant.conf (100%)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/wpa_supplicant.conf-sane (100%)
>   delete mode 100644 meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb
>   create mode 100644 meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb
>
> diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.inc b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0.inc
> similarity index 95%
> rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.inc
> rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0.inc
> index 6c84202..0c18b30 100644
> --- a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.inc
> +++ b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0.inc
> @@ -4,8 +4,8 @@ BUGTRACKER = "http://hostap.epitest.fi/bugz/"
>   SECTION = "network"
>   LICENSE = "GPLv2 | BSD"
>   LIC_FILES_CHKSUM = "file://../COPYING;md5=c54ce9345727175ff66d17b67ff51f58 \
> -                    file://../README;md5=54cfc88015d3ce83f7156e63c6bb1738 \
> -                    file://wpa_supplicant.c;beginline=1;endline=17;md5=acdc5a4b0d6345f21f136eace747260e"
> +                    file://../README;md5=9bee93996fecdb70f807baceb29d84b9 \
> +                    file://wpa_supplicant.c;beginline=1;endline=17;md5=372fb65dc23b228aec4e40e64de3e467"

As mentioned before, what changed and why?  This should be in the commit 
message.

Sau!

>   DEPENDS = "gnutls dbus libnl"
>   RRECOMMENDS_${PN} = "wpa-supplicant-passphrase wpa-supplicant-cli"
>
> diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/99_wpa_supplicant b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/99_wpa_supplicant
> similarity index 100%
> rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/99_wpa_supplicant
> rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/99_wpa_supplicant
> diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/defconfig-gnutls b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/defconfig-gnutls
> similarity index 100%
> rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/defconfig-gnutls
> rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/defconfig-gnutls
> diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa-supplicant.sh b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa-supplicant.sh
> similarity index 100%
> rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa-supplicant.sh
> rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa-supplicant.sh
> diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_supplicant.conf b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_supplicant.conf
> similarity index 100%
> rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_supplicant.conf
> rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_supplicant.conf
> diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_supplicant.conf-sane b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_supplicant.conf-sane
> similarity index 100%
> rename from meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_supplicant.conf-sane
> rename to meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_supplicant.conf-sane
> diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb
> deleted file mode 100644
> index 6f6378b..0000000
> --- a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb
> +++ /dev/null
> @@ -1,6 +0,0 @@
> -require wpa-supplicant-0.7.inc
> -
> -PR = "r12"
> -
> -SRC_URI[md5sum] = "f516f191384a9a546e3f5145c08addda"
> -SRC_URI[sha256sum] = "d0cd50caa85346ccc376dcda5ed3c258eef19a93b3cade39d25760118ad59443"
> diff --git a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb
> new file mode 100644
> index 0000000..8ede3e5
> --- /dev/null
> +++ b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb
> @@ -0,0 +1,6 @@
> +require wpa-supplicant-1.0.inc
> +
> +PR = "r0"
> +
> +SRC_URI[md5sum] = "8650f6aa23646ef634402552d0669640"
> +SRC_URI[sha256sum] = "91d41d473a5aafa2e25dd6577ebda975c4d4a3188850a53e31feaf7c04482b9c"
>




^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 1/3] iproute2: upgrade to 3.4.0
  2012-07-17 18:58 ` Saul Wold
@ 2012-07-18 11:50   ` Iorga, Cristian
  0 siblings, 0 replies; 14+ messages in thread
From: Iorga, Cristian @ 2012-07-18 11:50 UTC (permalink / raw)
  To: Saul Wold, Patches and discussions about the oe-core layer

Hi Saul,

No, I did not used buildhistory at this point.
I will activate it.

But I followed the changes carefully.
Does it count?

Thanks,
Cristian

-----Original Message-----
From: Saul Wold [mailto:sgw@linux.intel.com] 
Sent: Tuesday, July 17, 2012 9:58 PM
To: Patches and discussions about the oe-core layer
Cc: Iorga, Cristian
Subject: Re: [OE-core] [PATCH 1/3] iproute2: upgrade to 3.4.0

On 07/17/2012 01:28 AM, Cristian Iorga wrote:
> Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
> ---
>   .../iproute2/iproute2-3.2.0/configure-cross.patch  |   76 -------------------
>   .../iproute2/iproute2-3.4.0/configure-cross.patch  |   77 ++++++++++++++++++++
>   meta/recipes-connectivity/iproute2/iproute2.inc    |   10 +--
>   .../iproute2/iproute2_3.2.0.bb                     |   11 ---
>   .../iproute2/iproute2_3.4.0.bb                     |    9 +++
>   5 files changed, 91 insertions(+), 92 deletions(-)
>   delete mode 100644 meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.patch
>   create mode 100644 meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.patch
>   delete mode 100644 meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
>   create mode 100644 
> meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb
>
> diff --git 
> a/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.pa
> tch 
> b/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.pa
> tch
> deleted file mode 100644
> index bed7b84..0000000
> --- 
> a/meta/recipes-connectivity/iproute2/iproute2-3.2.0/configure-cross.pa
> tch
> +++ /dev/null
> @@ -1,76 +0,0 @@
> -From 85b0589b4843c03e8e6fd9416d71ea449a73c5c0 Mon Sep 17 00:00:00 
> 2001
> -From: Koen Kooi <koen@dominion.thruhere.net>
> -Date: Thu, 3 Nov 2011 10:46:16 +0100
> -Subject: [PATCH] make configure cross compile safe
> -
> -According to Kevin Tian:
> -Upstream-Status: Pending
> -
> -Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> -Signed-off-by: Shane Wang <shane.wang@intel.com>
> -
> -diff -r e4e1b14b9c4a configure
> ---- a/configure	Sun Jan 22 15:26:47 2012 +0800
> -+++ b/configure	Sun Jan 22 15:29:38 2012 +0800
> -@@ -2,6 +2,7 @@
> - # This is not an autconf generated configure
> - #
> - INCLUDE=${1:-"$PWD/include"}
> -+SYSROOT=$1
> -
> - check_atm()
> - {
> -@@ -13,7 +14,7 @@
> - 	return 0;
> - }
> - EOF
> --gcc -I$INCLUDE -o /tmp/atmtest /tmp/atmtest.c -latm >/dev/null 2>&1
> -+$CC -I$INCLUDE -o /tmp/atmtest /tmp/atmtest.c -latm >/dev/null 2>&1
> - if [ $? -eq 0 ]
> - then
> -     echo "TC_CONFIG_ATM:=y" >>Config
> -@@ -47,7 +48,7 @@
> -
> - EOF
> -
> --if gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL 
> $(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> -+if $CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL 
> -+$(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> - then
> - 	echo "TC_CONFIG_XT:=y" >>Config
> - 	echo "using xtables"
> -@@ -84,7 +85,7 @@
> - }
> -
> - EOF
> --gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl 
> >/dev/null 2>&1
> -+$CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl 
> -+>/dev/null 2>&1
> -
> - if [ $? -eq 0 ]
> - then
> -@@ -124,7 +125,7 @@
> - }
> -
> - EOF
> --gcc -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl 
> >/dev/null 2>&1
> -+$CC -I$INCLUDE $IPTC -o /tmp/ipttest /tmp/ipttest.c $IPTL -ldl 
> -+>/dev/null 2>&1
> -
> - if [ $? -eq 0 ]
> - then
> -@@ -145,7 +146,7 @@
> - check_ipt_lib_dir()
> - {
> - 	IPT_LIB_DIR=""
> --	for dir in /lib /usr/lib /usr/local/lib
> -+	for dir in $SYSROOT/lib $SYSROOT/usr/lib $SYSROOT/usr/local/lib
> - 	do
> - 		for file in $dir/{xtables,iptables}/lib*t_*so ; do
> - 			if [ -f $file ]; then
> -@@ -168,7 +169,7 @@
> - 	return 0;
> - }
> - EOF
> --gcc -I$INCLUDE -o /tmp/setnstest /tmp/setnstest.c >/dev/null 2>&1
> -+$CC -I$INCLUDE -o /tmp/setnstest /tmp/setnstest.c >/dev/null 2>&1
> - if [ $? -eq 0 ]
> - then
> - 	echo "IP_CONFIG_SETNS:=y" >>Config
> diff --git 
> a/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.pa
> tch 
> b/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cross.pa
> tch
> new file mode 100644
> index 0000000..42f591a
> --- /dev/null
> +++ b/meta/recipes-connectivity/iproute2/iproute2-3.4.0/configure-cros
> +++ s.patch
> @@ -0,0 +1,77 @@
> +From 85b0589b4843c03e8e6fd9416d71ea449a73c5c0 Mon Sep 17 00:00:00 
> +2001
> +From: Koen Kooi <koen@dominion.thruhere.net>
> +Date: Thu, 3 Nov 2011 10:46:16 +0100
> +Subject: [PATCH] make configure cross compile safe
> +
> +According to Kevin Tian:
> +Upstream-Status: Pending
> +
> +Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> +Signed-off-by: Shane Wang <shane.wang@intel.com>
> +
> +Index: iproute2-3.4.0/configure
> +===================================================================
> +--- iproute2-3.4.0.orig/configure
> ++++ iproute2-3.4.0/configure
> +@@ -2,6 +2,7 @@
> + # This is not an autconf generated configure  #  
> +INCLUDE=${1:-"$PWD/include"}
> ++SYSROOT=$1
> +
> + # Make a temp directory in build tree.
> + TMPDIR=$(mktemp -d config.XXXXXX)
> +@@ -17,7 +18,7 @@ int main(int argc, char **argv) {
> + 	return 0;
> + }
> + EOF
> +-gcc -I$INCLUDE -o $TMPDIR/atmtest $TMPDIR/atmtest.c -latm >/dev/null 
> +2>&1
> ++$CC -I$INCLUDE -o $TMPDIR/atmtest $TMPDIR/atmtest.c -latm >/dev/null 
> ++2>&1
> + if [ $? -eq 0 ]
> + then
> +     echo "TC_CONFIG_ATM:=y" >>Config @@ -51,7 +52,7 @@ int main(int 
> +argc, char **argv)
> +
> + EOF
> +
> +-if gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL 
> +$(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> ++if $CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL 
> ++$(pkg-config xtables --cflags --libs) -ldl >/dev/null 2>&1
> + then
> + 	echo "TC_CONFIG_XT:=y" >>Config
> + 	echo "using xtables"
> +@@ -88,7 +89,7 @@ int main(int argc, char **argv) {  }
> +
> + EOF
> +-gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl 
> +>/dev/null 2>&1
> ++$CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl 
> ++>/dev/null 2>&1
> +
> + if [ $? -eq 0 ]
> + then
> +@@ -128,7 +129,7 @@ int main(int argc, char **argv) {  }
> +
> + EOF
> +-gcc -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl 
> +>/dev/null 2>&1
> ++$CC -I$INCLUDE $IPTC -o $TMPDIR/ipttest $TMPDIR/ipttest.c $IPTL -ldl 
> ++>/dev/null 2>&1
> +
> + if [ $? -eq 0 ]
> + then
> +@@ -149,7 +150,7 @@ check_ipt()
> + check_ipt_lib_dir()
> + {
> + 	IPT_LIB_DIR=""
> +-	for dir in /lib /usr/lib /usr/local/lib
> ++	for dir in $SYSROOT/lib $SYSROOT/usr/lib $SYSROOT/usr/local/lib
> + 	do
> + 		for file in $dir/{xtables,iptables}/lib*t_*so ; do
> + 			if [ -f $file ]; then
> +@@ -172,7 +173,7 @@ int main(int argc, char **argv)
> + 	return 0;
> + }
> + EOF
> +-gcc -I$INCLUDE -o $TMPDIR/setnstest $TMPDIR/setnstest.c >/dev/null 
> +2>&1
> ++$CC -I$INCLUDE -o $TMPDIR/setnstest $TMPDIR/setnstest.c >/dev/null 
> ++2>&1
> + if [ $? -eq 0 ]
> + then
> + 	echo "IP_CONFIG_SETNS:=y" >>Config
> diff --git a/meta/recipes-connectivity/iproute2/iproute2.inc 
> b/meta/recipes-connectivity/iproute2/iproute2.inc
> index 86847ef..7caf7d0 100644
> --- a/meta/recipes-connectivity/iproute2/iproute2.inc
> +++ b/meta/recipes-connectivity/iproute2/iproute2.inc
> @@ -26,12 +26,12 @@ do_install () {
>   # There are only .so files in iproute2
>   INSANE_SKIP_${PN} = "dev-so"
>
> -FILES_${PN} += "${base_libdir}/tc"
> -FILES_${PN}-dbg += "${base_libdir}/tc/.debug"
> +FILES_${PN} += "${libdir}/tc"
> +FILES_${PN}-dbg += "${libdir}/tc/.debug"
>
> -ALTERNATIVE_NAME = "ip"
> -ALTERNATIVE_PATH = "${base_sbindir}/ip.iproute2"
> -ALTERNATIVE_LINK = "${base_bindir}/ip"
> +ALTERNATIVE_${PN} = "ip"
> +ALTERNATIVE_TARGET[ip] = "${base_sbindir}/ip.${BPN}"
> +ALTERNATIVE_LINK_NAME[ip] = "${base_bindir}/ip"
>   ALTERNATIVE_PRIORITY = "100"
>

Did you use buildhistory to compare the changes to the package?

Sau!

>   PARALLEL_MAKE = ""
> diff --git a/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb 
> b/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
> deleted file mode 100644
> index 5af12af..0000000
> --- a/meta/recipes-connectivity/iproute2/iproute2_3.2.0.bb
> +++ /dev/null
> @@ -1,11 +0,0 @@
> -require iproute2.inc
> -
> -PR = "r1"
> -
> -#v3.2.0 tag is "447c118f138171b260ad045ad6e1b17f9ef462e2"
> -#but it was not fully tested and had build error, and the next commit fixed it.
> -SRCREV = "13603f6a9e46f08576f6284a0ef1ce1fbf94ffe0"
> -SRC_URI = "git://git.kernel.org/pub/scm/linux/kernel/git/shemminger/iproute2.git \
> -           file://configure-cross.patch"
> -S = "${WORKDIR}/git"
> -
> diff --git a/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb 
> b/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb
> new file mode 100644
> index 0000000..cf1d521
> --- /dev/null
> +++ b/meta/recipes-connectivity/iproute2/iproute2_3.4.0.bb
> @@ -0,0 +1,9 @@
> +require iproute2.inc
> +
> +PR = "r0"
> +
> +SRC_URI = "${KERNELORG_MIRROR}/linux/utils/net/${BPN}/${BPN}-${PV}.tar.xz \
> +           file://configure-cross.patch"
> +
> +SRC_URI[md5sum] = "879d3fac4e90809598b2864ec4a0cbf8"
> +SRC_URI[sha256sum] = "38e846e412b2fa235a447b50c20ad1e9770d1b3ed4d3ab18ca0b18c6e8b79ba4"
>




^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 3/3] wpa-supplicant: upgrade to 1.0
  2012-07-17 19:01   ` Saul Wold
@ 2012-07-18 12:09     ` Iorga, Cristian
  0 siblings, 0 replies; 14+ messages in thread
From: Iorga, Cristian @ 2012-07-18 12:09 UTC (permalink / raw)
  To: Saul Wold, Patches and discussions about the oe-core layer

Working on fixing the commit with proper comments.

Thanks for the review.

/Cristian

-----Original Message-----
From: Saul Wold [mailto:sgw@linux.intel.com] 
Sent: Tuesday, July 17, 2012 10:01 PM
To: Patches and discussions about the oe-core layer
Cc: Iorga, Cristian
Subject: Re: [OE-core] [PATCH 3/3] wpa-supplicant: upgrade to 1.0

On 07/17/2012 01:28 AM, Cristian Iorga wrote:
> Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
> ---
>   ...a-supplicant-0.7.inc => wpa-supplicant-1.0.inc} |    4 ++--
>   .../99_wpa_supplicant                              |    0
>   .../defconfig-gnutls                               |    0
>   .../wpa-supplicant.sh                              |    0
>   .../wpa_supplicant.conf                            |    0
>   .../wpa_supplicant.conf-sane                       |    0
>   .../wpa-supplicant/wpa-supplicant_0.7.3.bb         |    6 ------
>   .../wpa-supplicant/wpa-supplicant_1.0.bb           |    6 ++++++
>   8 files changed, 8 insertions(+), 8 deletions(-)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.inc => wpa-supplicant-1.0.inc} (95%)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/99_wpa_supplicant (100%)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/defconfig-gnutls (100%)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/wpa-supplicant.sh (100%)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/wpa_supplicant.conf (100%)
>   rename meta/recipes-connectivity/wpa-supplicant/{wpa-supplicant-0.7.3 => wpa-supplicant-1.0}/wpa_supplicant.conf-sane (100%)
>   delete mode 100644 meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb
>   create mode 100644 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb
>
> diff --git 
> a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.inc 
> b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0.inc
> similarity index 95%
> rename from 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.inc
> rename to 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0.inc
> index 6c84202..0c18b30 100644
> --- a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.inc
> +++ b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0.inc
> @@ -4,8 +4,8 @@ BUGTRACKER = "http://hostap.epitest.fi/bugz/"
>   SECTION = "network"
>   LICENSE = "GPLv2 | BSD"
>   LIC_FILES_CHKSUM = "file://../COPYING;md5=c54ce9345727175ff66d17b67ff51f58 \
> -                    file://../README;md5=54cfc88015d3ce83f7156e63c6bb1738 \
> -                    file://wpa_supplicant.c;beginline=1;endline=17;md5=acdc5a4b0d6345f21f136eace747260e"
> +                    file://../README;md5=9bee93996fecdb70f807baceb29d84b9 \
> +                    file://wpa_supplicant.c;beginline=1;endline=17;md5=372fb65dc23b228aec4e40e64de3e467"

As mentioned before, what changed and why?  This should be in the commit message.

Sau!

>   DEPENDS = "gnutls dbus libnl"
>   RRECOMMENDS_${PN} = "wpa-supplicant-passphrase wpa-supplicant-cli"
>
> diff --git 
> a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/99_wpa
> _supplicant 
> b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/99_wpa_s
> upplicant
> similarity index 100%
> rename from 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/99_wpa_s
> upplicant rename to 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/99_wpa_sup
> plicant diff --git 
> a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/defcon
> fig-gnutls 
> b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/defconfi
> g-gnutls
> similarity index 100%
> rename from 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/defconfi
> g-gnutls rename to 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/defconfig-
> gnutls diff --git 
> a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa-su
> pplicant.sh 
> b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa-supp
> licant.sh
> similarity index 100%
> rename from 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa-supp
> licant.sh rename to 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa-suppli
> cant.sh diff --git 
> a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_su
> pplicant.conf 
> b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_supp
> licant.conf
> similarity index 100%
> rename from 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_supp
> licant.conf rename to 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_suppli
> cant.conf diff --git 
> a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_su
> pplicant.conf-sane 
> b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_supp
> licant.conf-sane
> similarity index 100%
> rename from 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-0.7.3/wpa_supp
> licant.conf-sane rename to 
> meta/recipes-connectivity/wpa-supplicant/wpa-supplicant-1.0/wpa_suppli
> cant.conf-sane diff --git 
> a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb 
> b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb
> deleted file mode 100644
> index 6f6378b..0000000
> --- a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_0.7.3.bb
> +++ /dev/null
> @@ -1,6 +0,0 @@
> -require wpa-supplicant-0.7.inc
> -
> -PR = "r12"
> -
> -SRC_URI[md5sum] = "f516f191384a9a546e3f5145c08addda"
> -SRC_URI[sha256sum] = "d0cd50caa85346ccc376dcda5ed3c258eef19a93b3cade39d25760118ad59443"
> diff --git 
> a/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb 
> b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb
> new file mode 100644
> index 0000000..8ede3e5
> --- /dev/null
> +++ b/meta/recipes-connectivity/wpa-supplicant/wpa-supplicant_1.0.bb
> @@ -0,0 +1,6 @@
> +require wpa-supplicant-1.0.inc
> +
> +PR = "r0"
> +
> +SRC_URI[md5sum] = "8650f6aa23646ef634402552d0669640"
> +SRC_URI[sha256sum] = "91d41d473a5aafa2e25dd6577ebda975c4d4a3188850a53e31feaf7c04482b9c"
>




^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [PATCH 2/3] pulseaudio: upgrade to 2.0
  2012-07-17 18:53   ` Saul Wold
@ 2012-07-18 12:40     ` Iorga, Cristian
  0 siblings, 0 replies; 14+ messages in thread
From: Iorga, Cristian @ 2012-07-18 12:40 UTC (permalink / raw)
  To: Saul Wold, Patches and discussions about the oe-core layer

Hello Saul,

libcanberra is just a small recipe inside pulseaudio recipe.
This is the reason for submitting as a single patch.
If is really necessary, I will re-submit them as two patches.

Regards,
Cristian

-----Original Message-----
From: Saul Wold [mailto:sgw@linux.intel.com] 
Sent: Tuesday, July 17, 2012 9:53 PM
To: Patches and discussions about the oe-core layer
Cc: Iorga, Cristian
Subject: Re: [OE-core] [PATCH 2/3] pulseaudio: upgrade to 2.0

On 07/17/2012 01:28 AM, Cristian Iorga wrote:
> Signed-off-by: Cristian Iorga <cristian.iorga@intel.com>
> ---
>   .../{libcanberra_0.28.bb => libcanberra_0.29.bb}   |   14 +++++++-------
>   meta/recipes-multimedia/pulseaudio/pulseaudio.inc  |    2 +-
>   .../{pulseaudio_1.1.bb => pulseaudio_2.0.bb}       |    7 +++----
>   3 files changed, 11 insertions(+), 12 deletions(-)
>   rename meta/recipes-multimedia/pulseaudio/{libcanberra_0.28.bb => libcanberra_0.29.bb} (76%)
>   rename meta/recipes-multimedia/pulseaudio/{pulseaudio_1.1.bb => 
> pulseaudio_2.0.bb} (64%)
>
You seem to be updating 2 recipes here, please separate them out as 2 discrete commits.

see below also.

> diff --git a/meta/recipes-multimedia/pulseaudio/libcanberra_0.28.bb 
> b/meta/recipes-multimedia/pulseaudio/libcanberra_0.29.bb
> similarity index 76%
> rename from meta/recipes-multimedia/pulseaudio/libcanberra_0.28.bb
> rename to meta/recipes-multimedia/pulseaudio/libcanberra_0.29.bb
> index e69c89d..b3c0e3d 100644
> --- a/meta/recipes-multimedia/pulseaudio/libcanberra_0.28.bb
> +++ b/meta/recipes-multimedia/pulseaudio/libcanberra_0.29.bb
> @@ -5,14 +5,14 @@ LIC_FILES_CHKSUM = "file://LGPL;md5=2d5025d4aa3495befef8f17206a5b0a1 \
>                       file://src/canberra.h;beginline=7;endline=24;md5=c616c687cf8da540a14f917e0d23ab03"
>
>   DEPENDS = "gtk+ pulseaudio alsa-lib libtool libvorbis"
> -PR = "r1"
> +PR = "r0"
>
>   inherit gconf autotools
>
> -SRC_URI = "http://0pointer.de/lennart/projects/libcanberra/libcanberra-${PV}.tar.gz"
> +SRC_URI = "http://0pointer.de/lennart/projects/libcanberra/libcanberra-${PV}.tar.xz"
>
> -SRC_URI[md5sum] = "c198b4811598c4c161ff505e4531b02c"
> -SRC_URI[sha256sum] = "eb1f8b2cabad7f07b6e44d606a91d73e1efca4b46daf92bd553e7222bc68868c"
> +SRC_URI[md5sum] = "2594093a5d61047bd9cc87e955f86df8"
> +SRC_URI[sha256sum] = "127a5ef07805856d63758e5180ebfb241d1f80094fd301c287591a15b8cfcd72"
>
>   EXTRA_OECONF = " --disable-oss --disable-ltdl-install"
>
> @@ -25,8 +25,8 @@ PACKAGES += "${PN}-gnome"
>   FILES_${PN} += "${libdir}/gtk-2.0/modules/ ${datadir}/gnome"
>   FILES_${PN}-gnome += "${datadir}/gdm/autostart/LoginWindow/libcanberra-ready-sound.desktop \
>   	              ${libdir}/gnome-settings-daemon-3.0/gtk-modules/canberra-gtk-module.desktop"
> -FILES_${PN}-dev += "${libdir}/libcanberra-0.28/libcanberra-*.so \
> -		    ${libdir}/libcanberra-0.28/libcanberra-*.la \
> +FILES_${PN}-dev += "${libdir}/libcanberra-0.29/libcanberra-*.so \
> +		    ${libdir}/libcanberra-0.29/libcanberra-*.la \
>   		    ${datadir}/vala/vapi"
> -FILES_${PN}-dbg += "${libdir}/gtk-2.0/modules/.debug ${libdir}/libcanberra-0.28/.debug"
> +FILES_${PN}-dbg += "${libdir}/gtk-2.0/modules/.debug ${libdir}/libcanberra-0.29/.debug"
>
Can we change these hardcoded version numbers to ${PV} instead? or -*?

> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc 
> b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> index c3d6d98..cd09940 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> @@ -6,7 +6,7 @@ LICENSE = "GPLv2+ & LGPLv2.1"
>   LIC_FILES_CHKSUM = "file://GPL;md5=4325afd396febcb659c36b49533135d4 \
>                       file://LGPL;md5=2d5025d4aa3495befef8f17206a5b0a1 \
>                       file://src/modules/bluetooth/proximity-helper.c;beginline=1;endline=25;md5=e4cc710e40a4d900c2c294167bc58270 \
> -                    file://src/pulsecore/vector.h;beginline=1;endline=21;md5=1b561f0a2ca47b1fa49862a6ccaacff1"
> +                    file://src/pulsecore/resampler.h;beginline=4;endline=23;md5=c3d539b93f8c82a1780bfa3cfa544a95"

What changed here and why, anytime we change LIC_FILES_CHKSUM you should ensure it's commented in the commit message.

>   DEPENDS = "libatomics-ops liboil avahi libsamplerate0 libsndfile1 libtool \
>              ${@base_contains('DISTRO_FEATURES', 'x11', 'virtual/libx11 libxtst libice libsm libxcb gtk+', '', d)}"
>   # optional
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio_1.1.bb 
> b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.0.bb
> similarity index 64%
> rename from meta/recipes-multimedia/pulseaudio/pulseaudio_1.1.bb
> rename to meta/recipes-multimedia/pulseaudio/pulseaudio_2.0.bb
> index e88a3a5..7e57e5e 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio_1.1.bb
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio_2.0.bb
> @@ -1,6 +1,6 @@
>   require pulseaudio.inc
>
> -PR = "r10"
> +PR = "r0"
>
>   DEPENDS += "libjson gdbm speex libxml-parser-perl-native"
>
> @@ -8,11 +8,10 @@ inherit gettext perlnative
>
>   SRC_URI = "http://freedesktop.org/software/pulseaudio/releases/pulseaudio-${PV}.tar.xz \
>     file://volatiles.04_pulse \
> -  file://pulseaudio_fix_for_x32.patch \
>   "
Why are you removing this patch? Also, you don't remove the corresponding file?  Did you test x32?


>
> -SRC_URI[md5sum] = "17d21df798cee407b769c6355fae397a"
> -SRC_URI[sha256sum] = "6fe531136f6ebce2d35872a2d2c914278cdc5dcdd5eea516dc52c81f9001f5ee"
> +SRC_URI[md5sum] = "9bbde657c353fe675c3b693054175a8e"
> +SRC_URI[sha256sum] = "28b42edd42f4879a6884af5f0ec11801ac001eb7582881215b36649aa37e2061"
>
>   do_compile_prepend() {
>       cd ${S}
>

Thanks
	Sau!



^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2012-07-18 12:52 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-17  8:28 [PATCH 1/3] iproute2: upgrade to 3.4.0 Cristian Iorga
2012-07-17  8:28 ` [PATCH 2/3] pulseaudio: upgrade to 2.0 Cristian Iorga
2012-07-17 18:53   ` Saul Wold
2012-07-18 12:40     ` Iorga, Cristian
2012-07-17  8:28 ` [PATCH 3/3] wpa-supplicant: upgrade to 1.0 Cristian Iorga
2012-07-17 19:01   ` Saul Wold
2012-07-18 12:09     ` Iorga, Cristian
2012-07-17  9:24 ` [PATCH 1/3] iproute2: upgrade to 3.4.0 Andrei Gherzan
2012-07-17 13:08   ` Paul Eggleton
2012-07-17 13:27     ` Koen Kooi
2012-07-17 13:41       ` Paul Eggleton
2012-07-17 13:29     ` Andrei Gherzan
2012-07-17 18:58 ` Saul Wold
2012-07-18 11:50   ` Iorga, Cristian

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.