diff options
| author | Robert Yang <liezhi.yang@windriver.com> | 2012-06-07 16:37:09 -0700 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2012-06-21 12:04:39 +0100 |
| commit | 107ec956591cd3398f2cbcf8500ee0bc5b8e7bcc (patch) | |
| tree | d9f591335f01324bc796c0f28a2cb2e85d4529b4 /meta | |
| parent | af3e5039e8ecd74a1abc9677dd4220de8f3fdda5 (diff) | |
| download | poky-107ec956591cd3398f2cbcf8500ee0bc5b8e7bcc.tar.gz | |
apt 0.7.14: do_compile failed with gcc 4.7
apt do_compile failed with gcc 4.7:
deb/deblistparser.cc: In member function 'virtual short unsigned int debListParser::VersionHash()':
deb/deblistparser.cc:212:13: error: redeclaration of 'char* I'
deb/deblistparser.cc:202:22: error: 'const char** I' previously declared here
Backport the patch from the upstream would fix the problem, both target and
native apt need it.
[YOCTO #2488]
(From OE-Core rev: 80c1ab1248ff38ba97cf5780fc05ff1321e14e10)
Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
Signed-off-by: Elizabeth Flanagan <elizabeth.flanagan@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Signed-off-by: Joshua Lock <josh@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta')
| -rw-r--r-- | meta/recipes-devtools/apt/apt-0.7.14/remove-redeclaration.patch | 63 | ||||
| -rw-r--r-- | meta/recipes-devtools/apt/apt-native_0.7.14.bb | 2 | ||||
| -rw-r--r-- | meta/recipes-devtools/apt/apt.inc | 1 | ||||
| -rw-r--r-- | meta/recipes-devtools/apt/apt_0.7.14.bb | 2 |
4 files changed, 66 insertions, 2 deletions
diff --git a/meta/recipes-devtools/apt/apt-0.7.14/remove-redeclaration.patch b/meta/recipes-devtools/apt/apt-0.7.14/remove-redeclaration.patch new file mode 100644 index 0000000000..8d7c891545 --- /dev/null +++ b/meta/recipes-devtools/apt/apt-0.7.14/remove-redeclaration.patch | |||
| @@ -0,0 +1,63 @@ | |||
| 1 | Fix build errors on gcc 4.7: | ||
| 2 | |||
| 3 | deb/deblistparser.cc: In member function 'virtual short unsigned int debListParser::VersionHash()': | ||
| 4 | deb/deblistparser.cc:212:13: error: redeclaration of 'char* I' | ||
| 5 | deb/deblistparser.cc:202:22: error: 'const char** I' previously declared here | ||
| 6 | |||
| 7 | Upstream-Status: Backport | ||
| 8 | Signed-off-by: Robert Yang <liezhi.yang@windriver.com> | ||
| 9 | --- | ||
| 10 | apt-pkg/deb/deblistparser.cc | 10 +++++----- | ||
| 11 | cmdline/apt-get.cc | 8 ++++---- | ||
| 12 | 2 files changed, 9 insertions(+), 9 deletions(-) | ||
| 13 | |||
| 14 | diff --git a/apt-pkg/deb/deblistparser.cc b/apt-pkg/deb/deblistparser.cc | ||
| 15 | --- a/apt-pkg/deb/deblistparser.cc | ||
| 16 | +++ b/apt-pkg/deb/deblistparser.cc | ||
| 17 | @@ -209,18 +209,18 @@ unsigned short debListParser::VersionHash() | ||
| 18 | /* Strip out any spaces from the text, this undoes dpkgs reformatting | ||
| 19 | of certain fields. dpkg also has the rather interesting notion of | ||
| 20 | reformatting depends operators < -> <= */ | ||
| 21 | - char *I = S; | ||
| 22 | + char *J = S; | ||
| 23 | for (; Start != End; Start++) | ||
| 24 | { | ||
| 25 | if (isspace(*Start) == 0) | ||
| 26 | - *I++ = tolower(*Start); | ||
| 27 | + *J++ = tolower(*Start); | ||
| 28 | if (*Start == '<' && Start[1] != '<' && Start[1] != '=') | ||
| 29 | - *I++ = '='; | ||
| 30 | + *J++ = '='; | ||
| 31 | if (*Start == '>' && Start[1] != '>' && Start[1] != '=') | ||
| 32 | - *I++ = '='; | ||
| 33 | + *J++ = '='; | ||
| 34 | } | ||
| 35 | |||
| 36 | - Result = AddCRC16(Result,S,I - S); | ||
| 37 | + Result = AddCRC16(Result,S,J - S); | ||
| 38 | } | ||
| 39 | |||
| 40 | return Result; | ||
| 41 | diff --git a/cmdline/apt-get.cc b/cmdline/apt-get.cc | ||
| 42 | --- a/cmdline/apt-get.cc | ||
| 43 | +++ b/cmdline/apt-get.cc | ||
| 44 | @@ -1752,12 +1752,12 @@ bool DoInstall(CommandLine &CmdL) | ||
| 45 | if ((*Cache)[I].Install() == false) | ||
| 46 | continue; | ||
| 47 | |||
| 48 | - const char **J; | ||
| 49 | - for (J = CmdL.FileList + 1; *J != 0; J++) | ||
| 50 | - if (strcmp(*J,I.Name()) == 0) | ||
| 51 | + const char **K; | ||
| 52 | + for (K = CmdL.FileList + 1; *K != 0; K++) | ||
| 53 | + if (strcmp(*K,I.Name()) == 0) | ||
| 54 | break; | ||
| 55 | |||
| 56 | - if (*J == 0) { | ||
| 57 | + if (*K == 0) { | ||
| 58 | List += string(I.Name()) + " "; | ||
| 59 | VersionsList += string(Cache[I].CandVersion) + "\n"; | ||
| 60 | } | ||
| 61 | -- | ||
| 62 | 1.7.1 | ||
| 63 | |||
diff --git a/meta/recipes-devtools/apt/apt-native_0.7.14.bb b/meta/recipes-devtools/apt/apt-native_0.7.14.bb index ec3fd20a8c..ca5476b6b2 100644 --- a/meta/recipes-devtools/apt/apt-native_0.7.14.bb +++ b/meta/recipes-devtools/apt/apt-native_0.7.14.bb | |||
| @@ -1,6 +1,6 @@ | |||
| 1 | require apt-native.inc | 1 | require apt-native.inc |
| 2 | 2 | ||
| 3 | PR = "r7" | 3 | PR = "r8" |
| 4 | 4 | ||
| 5 | SRC_URI += "file://nodoc.patch \ | 5 | SRC_URI += "file://nodoc.patch \ |
| 6 | file://noconfigure.patch \ | 6 | file://noconfigure.patch \ |
diff --git a/meta/recipes-devtools/apt/apt.inc b/meta/recipes-devtools/apt/apt.inc index 61d9602c27..0d241a9ab9 100644 --- a/meta/recipes-devtools/apt/apt.inc +++ b/meta/recipes-devtools/apt/apt.inc | |||
| @@ -7,6 +7,7 @@ SRC_URI = "${DEBIAN_MIRROR}/main/a/apt/apt_${PV}.tar.gz \ | |||
| 7 | file://use-host.patch \ | 7 | file://use-host.patch \ |
| 8 | file://localefixes.patch \ | 8 | file://localefixes.patch \ |
| 9 | file://makerace.patch \ | 9 | file://makerace.patch \ |
| 10 | file://remove-redeclaration.patch \ | ||
| 10 | " | 11 | " |
| 11 | 12 | ||
| 12 | inherit autotools gettext | 13 | inherit autotools gettext |
diff --git a/meta/recipes-devtools/apt/apt_0.7.14.bb b/meta/recipes-devtools/apt/apt_0.7.14.bb index 8464b7f904..9ace6d25df 100644 --- a/meta/recipes-devtools/apt/apt_0.7.14.bb +++ b/meta/recipes-devtools/apt/apt_0.7.14.bb | |||
| @@ -3,7 +3,7 @@ RDEPENDS_${PN} = "dpkg" | |||
| 3 | LIC_FILES_CHKSUM = "file://COPYING.GPL;md5=0636e73ff0215e8d672dc4c32c317bb3" | 3 | LIC_FILES_CHKSUM = "file://COPYING.GPL;md5=0636e73ff0215e8d672dc4c32c317bb3" |
| 4 | require apt.inc | 4 | require apt.inc |
| 5 | 5 | ||
| 6 | PR = "r10" | 6 | PR = "r11" |
| 7 | 7 | ||
| 8 | SRC_URI += "file://nodoc.patch \ | 8 | SRC_URI += "file://nodoc.patch \ |
| 9 | file://includes-fix.patch " | 9 | file://includes-fix.patch " |
