Important fixes for rc3
cover Important fixes for rc3
The patches in this set are all important IMO and I do think they ought to be included.
The following changes since commit bde1744018afd4616e114b20ffdc21b9abddcedc:
hello-mod: Move hello-mod from meta to meta-skeleton (2012-04-10 13:16:00 +0100)
are available in the git repository at:
git://github.com/OSSystems/oe-core master
https://github.com/OSSystems/oe-core/tree/HEAD
Otavio Salvador (3):
connman: backport test script fixes distrodata.bbclass: fix comment typo
flex: backport Debian patches to fix generated code warnings
meta/classes/distrodata.bbclass | 2 +- .../connman/test-set-ipv4-method-api-fix.patch | 50 +++++++++++++++++ .../connman/test-set-ipv6-method-api-fix.patch | 55 ++++++++++++++++++ meta/recipes-connectivity/connman/connman_0.79.bb | 4 + .../flex/flex/avoid-FORTIFY-warnings.patch | 59 ++++++++++++++++++++ .../flex/int-is-not-the-same-size-as-size_t.patch | 42 ++++++++++++++ meta/recipes-devtools/flex/flex_2.5.35.bb | 5 +- 7 files changed, 215 insertions(+), 2 deletions(-) create mode 100644 meta/recipes-connectivity/connman/connman/test-set-ipv4-method-api-fix.patch create mode 100644 meta/recipes-connectivity/connman/connman/test-set-ipv6-method-api-fix.patch create mode 100644 meta/recipes-devtools/flex/flex/avoid-FORTIFY-warnings.patch create mode 100644 meta/recipes-devtools/flex/flex/int-is-not-the-same-size-as-size_t.patch
Signature · 1 line
1.7.2.5
PATCH 1/3 connman: backport test script fixes
Those fixes are required to get the test scripts to work with current 0.79 DBus API.
--- .../connman/test-set-ipv4-method-api-fix.patch | 50 ++++++++++++++++++ .../connman/test-set-ipv6-method-api-fix.patch | 55 ++++++++++++++++++++ meta/recipes-connectivity/connman/connman_0.79.bb | 4 ++ 3 files changed, 109 insertions(+), 0 deletions(-) create mode 100644 meta/recipes-connectivity/connman/connman/test-set-ipv4-method-api-fix.patch create mode 100644 meta/recipes-connectivity/connman/connman/test-set-ipv6-method-api-fix.patch
meta/recipes-connectivity/connman/connman/test-set-ipv4-method-api-fix.patch new file +50 -0
@@ -0,0 +1,50 @@ +From 0b817dd3096f5c3ca8f0b868ee55ea550054137b Mon Sep 17 00:00:00 2001 +From: Daniel Wagner <[email protected]> +Date: Mon, 19 Mar 2012 14:03:27 +0100 +Subject: [PATCH 3/3] set-ipv4-method: The arguments have to be wrapped into a + variant + +Fixes BMC#24986 + +Upstream-Status: Backport + +Signed-off-by: Otavio Salvador <[email protected]> +--- + test/set-ipv4-method | 11 +++++++---- + 1 file changed, 7 insertions(+), 4 deletions(-) + +diff --git a/test/set-ipv4-method b/test/set-ipv4-method +index 820b24b..235113f 100755 +--- a/test/set-ipv4-method ++++ b/test/set-ipv4-method +@@ -3,6 +3,9 @@ + import sys + import dbus + ++def make_variant(string): ++ return dbus.String(string, variant_level=1) ++ + def print_usage(): + print "Usage: %s <service> [off|dhcp|manual <address> [netmask] [gateway]]" % (sys.argv[0]) + +@@ -20,13 +23,13 @@ properties = service.GetProperties() + + print "Setting method %s for %s" % (sys.argv[2], sys.argv[1]) + +-ipv4_configuration = { "Method": sys.argv[2] } ++ipv4_configuration = { "Method": make_variant(sys.argv[2]) } + if (len(sys.argv) > 3): +- ipv4_configuration["Address"] = sys.argv[3] ++ ipv4_configuration["Address"] = make_variant(sys.argv[3]) + if (len(sys.argv) > 4): +- ipv4_configuration["Netmask"] = sys.argv[4] ++ ipv4_configuration["Netmask"] = make_variant(sys.argv[4]) + if (len(sys.argv) > 5): +- ipv4_configuration["Gateway"] = sys.argv[5] ++ ipv4_configuration["Gateway"] = make_variant(sys.argv[5]) + + service.SetProperty("IPv4.Configuration", ipv4_configuration) + print "New IPv4.Configuration: ", ipv4_configuration +-- +1.7.9.5 +
meta/recipes-connectivity/connman/connman/test-set-ipv6-method-api-fix.patch new file +55 -0
55 lines not on this page · raw message
meta/recipes-connectivity/connman/connman_0.79.bb +4 -0
@@ -1,11 +1,15 @@ require connman.inc +PR = "r1" + # 0.79 tag SRCREV = "442b1fe603e005814f592a3dbcf0d0bfb13f961c" SRC_URI = "git://git.kernel.org/pub/scm/network/connman/connman.git \ file://add_xuser_dbus_permission.patch \ file://ethernet_default.patch \ file://disable_alg-test.patch \ + file://test-set-ipv4-method-api-fix.patch \ + file://test-set-ipv6-method-api-fix.patch \ file://connman" S = "${WORKDIR}/git" PR = "r1"
Signature · 1 line
1.7.2.5
Re: [PATCH 1/3] connman: backport test script fixes
On Wed, Apr 11, 2012 at 00:19, Otavio Salvador <[email protected]> wrote:
...
Quoted 10 lines
meta/recipes-connectivity/connman/connman_0.79.bb +0
-0@@ -1,11 +1,15 @@require connman.inc
meta/recipes-connectivity/connman/connman_0.79.bb +2
-0+PR = "r1" +# 0.79 tag
SRCREV = "442b1fe603e005814f592a3dbcf0d0bfb13f961c"
SRC_URI = "git://git.kernel.org/pub/scm/network/connman/connman.git\
Quoted 8 lines
file://add_xuser_dbus_permission.patch \
file://ethernet_default.patch \
file://disable_alg-test.patch \
+ file://test-set-ipv4-method-api-fix.patch \
+ file://test-set-ipv6-method-api-fix.patch \
file://connman"
S = "${WORKDIR}/git"
PR = "r1"
When picking it, please fix the PR. It has been messed up in merging sorry.
Signature · 3 lines
Otavio Salvador O.S. Systems E-mail: [email protected] http://www.ossystems.com.br Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
PATCH 2/3 distrodata.bbclass: fix comment typo
In reply to Otavio SalvadorFixes:
- xf86-intput-synaptics-12.6.9.tar.gz
+ xf86-input-synaptics-12.6.9.tar.gz
--- meta/classes/distrodata.bbclass | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
meta/classes/distrodata.bbclass +1 -1
@@ -237,7 +237,7 @@ python do_checkpkg() { such as: gnome-common-2.20.0.tar.gz (most common format) gtk+-2.90.1.tar.gz - xf86-intput-synaptics-12.6.9.tar.gz + xf86-input-synaptics-12.6.9.tar.gz dri2proto-2.3.tar.gz blktool_4.orig.tar.gz libid3tag-0.15.1b.tar.gz
Signature · 1 line
1.7.2.5
PATCH 3/3 flex: backport Debian patches to fix generated code warnings
In reply to Otavio SalvadorThe generated parser had warnings regarding signess and return check which makes Linux Kernel's perf tool from 3.4 release to fail without those patches.
--- .../flex/flex/avoid-FORTIFY-warnings.patch | 59 ++++++++++++++++++++ .../flex/int-is-not-the-same-size-as-size_t.patch | 42 ++++++++++++++ meta/recipes-devtools/flex/flex_2.5.35.bb | 5 +- 3 files changed, 105 insertions(+), 1 deletions(-) create mode 100644 meta/recipes-devtools/flex/flex/avoid-FORTIFY-warnings.patch create mode 100644 meta/recipes-devtools/flex/flex/int-is-not-the-same-size-as-size_t.patch
meta/recipes-devtools/flex/flex/avoid-FORTIFY-warnings.patch new file +59 -0
@@ -0,0 +1,59 @@ +Wrap ECHO macro to avoid FORTIFY warnings + +This change will enable flex scanners to be compiled with +-D_FORTIFY_SOURCE=2. + +Signed-off-by: Manoj Srivastava <[email protected]> +Signed-off-by: Otavio Salvador <[email protected]> + +Origin: Cherry picked from Debian +Upstream-Status: Pending + +--- + flex.skl | 2 +- + scan.c | 2 +- + skel.c | 2 +- + 3 files changed, 3 insertions(+), 3 deletions(-) + +diff --git a/flex.skl b/flex.skl +index 6ebf2fd..eaa355e 100644 +--- a/flex.skl ++++ b/flex.skl +@@ -1074,7 +1074,7 @@ m4_ifdef( [[M4_YY_NOT_IN_HEADER]], + /* This used to be an fputs(), but since the string might contain NUL's, + * we now use fwrite(). + */ +-#define ECHO fwrite( yytext, yyleng, 1, yyout ) ++#define ECHO do { if (fwrite( yytext, yyleng, 1, yyout )) {} } while (0) + %endif + %if-c++-only C++ definition + #define ECHO LexerOutput( yytext, yyleng ) +diff --git a/scan.c b/scan.c +index d7f5db3..1a079bf 100644 +--- a/scan.c ++++ b/scan.c +@@ -2093,7 +2093,7 @@ static int input (void ); + /* This used to be an fputs(), but since the string might contain NUL's, + * we now use fwrite(). + */ +-#define ECHO fwrite( yytext, yyleng, 1, yyout ) ++#define ECHO do { if (fwrite( yytext, yyleng, 1, yyout )) {} } while (0) + #endif + + /* Gets input and stuffs it into "buf". number of characters read, or YY_NULL, +diff --git a/skel.c b/skel.c +index ee9f6ef..b8a2b4b 100644 +--- a/skel.c ++++ b/skel.c +@@ -1141,7 +1141,7 @@ const char *skel[] = { + "/* This used to be an fputs(), but since the string might contain NUL's,", + " * we now use fwrite().", + " */", +- "#define ECHO fwrite( yytext, yyleng, 1, yyout )", ++ "#define ECHO do { if (fwrite( yytext, yyleng, 1, yyout )) {} } while (0)", + "%endif", + "%if-c++-only C++ definition", + "#define ECHO LexerOutput( yytext, yyleng )", +-- +1.7.9.5 +
meta/recipes-devtools/flex/flex/int-is-not-the-same-size-as-size_t.patch new file +42 -0
42 lines not on this page · raw message
meta/recipes-devtools/flex/flex_2.5.35.bb +4 -1
@@ -1,8 +1,11 @@ require flex.inc -PR = "r2" +PR = "r3" LICENSE="BSD" LIC_FILES_CHKSUM = "file://COPYING;md5=e4742cf92e89040b39486a6219b68067" BBCLASSEXTEND = "native" +SRC_URI += "file://avoid-FORTIFY-warnings.patch \ + file://int-is-not-the-same-size-as-size_t.patch" + SRC_URI[md5sum] = "10714e50cea54dc7a227e3eddcd44d57" SRC_URI[sha256sum] = "0becbd4b2b36b99c67f8c22ab98f7f80c9860aec70f0350a0018f29a88704e7b"
Signature · 1 line
1.7.2.5
Quoted 7 lines · Otavio Salvador
The patches in this set are all important IMO and I do think they ought to be included.
The following changes since commit
bde1744018afd4616e114b20ffdc21b9abddcedc:hello-mod: Move hello-mod from meta to meta-skeleton (2012-04-10 13:16:00 +0100)
Quoted 8 lines
are available in the git repository at:
git://github.com/OSSystems/oe-coremaster
https://github.com/OSSystems/oe-core/tree/HEADOtavio Salvador (3):
connman: backport test script fixes distrodata.bbclass: fix comment typo
flex: backport Debian patches to fix generated code warnings
A typo fix is not high priority for -rc3. Sending patches which you then request me to manually fix up for -rc3 is a bit of a joke.
I'm actually seriously considering reverting the recent connman version updates since they've totally broken connman-gnome and all the UI connman integration. I appreciate this would upset some people so we're going to try and fix connman-gnome but I'm not happy to be in this position at this point in the release cycle. The test scripts are the least of my worries to be quite honest.
Cheers,
Richard
Quoted 2 lines · Richard Purdie
A typo fix is not high priority for -rc3. Sending patches which you then request me to manually fix up for -rc3 is a bit of a joke.
The typo I agree and I can resend the patch but I didn't because it was 1am here and I was falling sleep. If you check, this patch were send long time ago and not received any comment.
Quoted 6 lines
I'm actually seriously considering reverting the recent connman version updates since they've totally broken connman-gnome and all the UI connman integration. I appreciate this would upset some people so we're going to try and fix connman-gnome but I'm not happy to be in this position at this point in the release cycle. The test scripts are the least of my worries to be quite honest.
Yes; I agree this is unfortunate but connman last version has fixed many issues, specially regarding hidden wifi management (the main reason we wanted to update to it at O.S. Systems). Another important point is that its API is mostly stable now and putting a 1.2 with previous API will force rework from any user using it when 1.3 is released so this is not the way to go IMO.
Signature · 3 lines
Otavio Salvador O.S. Systems E-mail: [email protected] http://www.ossystems.com.br Mobile: +55 53 9981-7854 http://projetos.ossystems.com.br
Saul Wold
AppliedQuoted 24 lines · Otavio Salvador
The patches in this set are all important IMO and I do think they ought to be included.
The following changes since commit
bde1744018afd4616e114b20ffdc21b9abddcedc:hello-mod: Move hello-mod from meta to meta-skeleton (2012-04-10 13:16:00 +0100)are available in the git repository at:
git://github.com/OSSystems/oe-coremaster
https://github.com/OSSystems/oe-core/tree/HEADOtavio Salvador (3):
connman: backport test script fixes distrodata.bbclass: fix comment typo
flex: backport Debian patches to fix generated code warningsmeta/classes/distrodata.bbclass | 2 +- .../connman/test-set-ipv4-method-api-fix.patch | 50 +++++++++++++++++ .../connman/test-set-ipv6-method-api-fix.patch | 55 ++++++++++++++++++ meta/recipes-connectivity/connman/connman_0.79.bb | 4 + .../flex/flex/avoid-FORTIFY-warnings.patch | 59 ++++++++++++++++++++ .../flex/int-is-not-the-same-size-as-size_t.patch | 42 ++++++++++++++ meta/recipes-devtools/flex/flex_2.5.35.bb | 5 +- 7 files changed, 215 insertions(+), 2 deletions(-)
Quoted 5 lines
create mode 100644
meta/recipes-connectivity/connman/connman/test-set-ipv4-method-api-fix.patchcreate mode 100644meta/recipes-connectivity/connman/connman/test-set-ipv6-method-api-fix.patchcreate mode 100644meta/recipes-devtools/flex/flex/avoid-FORTIFY-warnings.patchcreate mode 100644meta/recipes-devtools/flex/flex/int-is-not-the-same-size-as-size_t.patch
Merged into OE-Core
Thanks
Sau!